Add option for allow_update_direct_with_only_dependencies (#16)

* Update Config.php

* Update Config.php

* Create allow_indirect.json

* Update UnitTest.php

* Update UnitTest.php

* Create allow_indirect2.json

* Create allow_indirect3.json

* Update UnitTest.php

src/Config.php

@@ -33,6 +33,7 @@ public function getDefaultConfig()
             'number_of_concurrent_updates' => 0,
             'branch_prefix' => '',
             'commit_message_convention' => '',
+            'allow_update_indirect_with_direct' => 0,
         ];
     }
 
@@ -71,6 +72,11 @@ public function hasConfigForKey($key)
         return !empty($this->configOptionsSet[$key]);
     }
 
+    public function shouldUpdateIndirectWithDirect()
+    {
+        return (bool) $this->config->allow_update_indirect_with_direct;
+    }
+
     public function shouldAlwaysUpdateAll()
     {
         return (bool) $this->config->always_update_all;

tests/UnitTest.php

@@ -191,6 +191,17 @@ public function testUpdatesBeyondConstraint($filename, $expected_result)
         self::assertEquals($expected_result, $data->shouldAllowUpdatesBeyondConstraint());
     }
 
+    /**
+     * Test the allow_update_direct_with_only_dependencies option.
+     *
+     * @dataProvider getAllowUpdateDirect
+     */
+    public function testAllowUpdateDirect($filename, $expected_result)
+    {
+        $data = $this->createDataFromFixture($filename);
+        self::assertEquals($expected_result, $data->shouldUpdateIndirectWithDirect());
+    }
+
     /**
      * Test the always_update_all option.
      *
@@ -297,6 +308,29 @@ public function getAlwaysUpdateAll()
         ];
     }
 
+    public function getAllowUpdateDirect()
+    {
+        return [
+            [
+                'empty.json',
+                false,
+            ],
+            [
+                'allow_indirect.json',
+                true,
+            ],
+            [
+                'allow_indirect2.json',
+                true,
+            ],
+            [
+                'allow_indirect3.json',
+                false,
+            ],
+        ];
+    }
+
+
     public function getUpdatesBeyondConstraint()
     {
         return [

tests/fixtures/allow_indirect.json

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

tests/fixtures/allow_indirect2.json

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

tests/fixtures/allow_indirect3.json

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