Make it possible to always allow direct deps (#23)

src/Config.php

@@ -16,6 +16,7 @@ public function getDefaultConfig()
     {
         return (object) [
             'always_update_all' => 0,
+            'always_allow_direct_dependencies' => 0,
             'allow_list' => [],
             'update_dev_dependencies' => 1,
             'check_only_direct_dependencies' => 1,
@@ -87,6 +88,11 @@ public function getLabelsSecurity() : array
         return $this->config->labels_security;
     }
 
+    public function shouldAlwaysAllowDirect() : bool
+    {
+        return (bool) $this->config->always_allow_direct_dependencies;
+    }
+
     public function hasConfigForKey($key)
     {
         return !empty($this->configOptionsSet[$key]);

tests/UnitTest.php

@@ -57,6 +57,17 @@ protected function runTestExpectedResultFromFixture($filename, $expected_result,
         self::assertEquals($expected_result, $data->{$method}());
     }
 
+    /**
+     * Test the different things we can set in always allow direct.
+     *
+     * @dataProvider getAlwaysAllowDirect
+     */
+    public function testAlwaysDirect($filename, $expected_result)
+    {
+        $data = $this->createDataFromFixture($filename);
+        self::assertEquals($expected_result, $data->shouldAlwaysAllowDirect());
+    }
+
     /**
      * Test the different things we can set in run scripts, and what we expect from it.
      *
@@ -741,6 +752,36 @@ public function emptyConfigs()
         ];
     }
 
+    public function getAlwaysAllowDirect()
+    {
+        return [
+            [
+                'empty.json',
+                false,
+            ],
+            [
+                'always_allow_direct_dependencies.json',
+                true
+            ],
+            [
+                'always_allow_direct_dependencies2.json',
+                true
+            ],
+            [
+                'always_allow_direct_dependencies3.json',
+                true
+            ],
+            [
+                'always_allow_direct_dependencies4.json',
+                false
+            ],
+            [
+                'always_allow_direct_dependencies5.json',
+                false
+            ],
+        ];
+    }
+
     public function getRunScriptData()
     {
         return [

tests/fixtures/always_allow_direct_dependencies.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "always_allow_direct_dependencies": "yes_of_course_this_should_be_an_int_but_this_evaluates_to_true"
+    }
+  }
+}

tests/fixtures/always_allow_direct_dependencies2.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "always_allow_direct_dependencies": 1
+    }
+  }
+}

tests/fixtures/always_allow_direct_dependencies3.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "always_allow_direct_dependencies": "1"
+    }
+  }
+}

tests/fixtures/always_allow_direct_dependencies4.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "always_allow_direct_dependencies": "0"
+    }
+  }
+}

tests/fixtures/always_allow_direct_dependencies5.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "always_allow_direct_dependencies": 0
+    }
+  }
+}