Add config for tags (#20)

src/Config.php

@@ -36,6 +36,8 @@ public function getDefaultConfig()
             'allow_update_indirect_with_direct' => 0,
             'automerge' => 0,
             'automerge_security' => 0,
+            'tags' => [],
+            'tags_security' => [],
         ];
     }
 
@@ -69,6 +71,22 @@ public function setConfig($config)
         }
     }
 
+    public function getTags() : array
+    {
+        if (!is_array($this->config->tags)) {
+            return [];
+        }
+        return $this->config->tags;
+    }
+
+    public function getTagsSecurity() : array
+    {
+        if (!is_array($this->config->tags_security)) {
+            return [];
+        }
+        return $this->config->tags_security;
+    }
+
     public function hasConfigForKey($key)
     {
         return !empty($this->configOptionsSet[$key]);

tests/UnitTest.php

@@ -31,6 +31,32 @@ public function testGetOptionSet($filename, $option, $expected_result)
         self::assertEquals($data->hasConfigForKey($option), $expected_result);
     }
 
+    /**
+     * Test that getTags returns the expected things.
+     *
+     * @dataProvider getTagsConfig
+     */
+    public function testTags($filename, $expected_result)
+    {
+        $this->runTestExpectedResultFromFixture($filename, $expected_result, 'getTags');
+    }
+
+    /**
+     * Test that getTagsSecurity returns the expected things.
+     *
+     * @dataProvider getTagsSecurityConfig
+     */
+    public function testTagsSecurity($filename, $expected_result)
+    {
+        $this->runTestExpectedResultFromFixture($filename, $expected_result, 'getTagsSecurity');
+    }
+
+    protected function runTestExpectedResultFromFixture($filename, $expected_result, $method)
+    {
+        $data = $this->createDataFromFixture($filename);
+        self::assertEquals($expected_result, $data->{$method}());
+    }
+
     /**
      * Test the different things we can set in run scripts, and what we expect from it.
      *
@@ -601,6 +627,74 @@ public function getCommitMessage()
         ];
     }
 
+    public function getTagsConfig()
+    {
+        return [
+            [
+                'empty.json',
+                [],
+            ],
+            [
+                'tags.json',
+                [],
+            ],
+            [
+                'tags2.json',
+                [],
+            ],
+            [
+                'tags3.json',
+                [],
+            ],
+            [
+                'tags4.json',
+                [],
+            ],
+            [
+                'tags5.json',
+                ["tag123"],
+            ],
+            [
+                'tags6.json',
+                ["tag123", "tag456"],
+            ],
+        ];
+    }
+
+    public function getTagsSecurityConfig()
+    {
+        return [
+            [
+                'empty.json',
+                [],
+            ],
+            [
+                'tags_security.json',
+                [],
+            ],
+            [
+                'tags_security2.json',
+                [],
+            ],
+            [
+                'tags_security3.json',
+                [],
+            ],
+            [
+                'tags_security4.json',
+                [],
+            ],
+            [
+                'tags_security5.json',
+                ["tag123"],
+            ],
+            [
+                'tags_security6.json',
+                ["tag123", "tag456"],
+            ],
+        ];
+    }
+
     public function getHasConfig()
     {
         return [

tests/fixtures/tags.json

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

tests/fixtures/tags2.json

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

tests/fixtures/tags3.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "tags": []
+    }
+  }
+}

tests/fixtures/tags4.json

@@ -0,0 +1,9 @@
+{
+  "extra": {
+    "violinist": {
+      "tags": {
+        "tag1": true
+      }
+    }
+  }
+}

tests/fixtures/tags5.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "tags": ["tag123"]
+    }
+  }
+}

tests/fixtures/tags6.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "tags": ["tag123", "tag456"]
+    }
+  }
+}

tests/fixtures/tags_security.json

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

tests/fixtures/tags_security2.json

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

tests/fixtures/tags_security3.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "tags_security": []
+    }
+  }
+}

tests/fixtures/tags_security4.json

@@ -0,0 +1,9 @@
+{
+  "extra": {
+    "violinist": {
+      "tags_security": {
+        "tag1": true
+      }
+    }
+  }
+}

tests/fixtures/tags_security5.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "tags_security": ["tag123"]
+    }
+  }
+}

tests/fixtures/tags_security6.json

@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "tags_security": ["tag123", "tag456"]
+    }
+  }
+}