Merge 0d419db into 4e9b096

violinist-dev · Sep 14, 2021 · 7de3e72 · 7de3e72
2 parents 4e9b096 + 0d419db
commit 7de3e72
Show file tree

Hide file tree

Showing 5 changed files with 67 additions and 0 deletions.
diff --git a/src/Config.php b/src/Config.php
@@ -85,6 +85,15 @@ public function getBundledPackagesForPackage($package_name)
         return [];
     }
 
+    public function getAssignees()
+    {
+        if (!is_array($this->config->assignees)) {
+            return [];
+        }
+
+        return $this->config->assignees;
+    }
+
     public function getBlockList()
     {
         if (!is_array($this->config->blocklist)) {

diff --git a/tests/UnitTest.php b/tests/UnitTest.php
@@ -135,12 +135,47 @@ public function testOnlyDirectDependencies($filename, $expected_result)
         self::assertEquals($expected_result, $data->shouldCheckDirectOnly());
     }
 
+    /**
+     * Test the assignees part.
+     *
+     * @dataProvider getAssignees
+     */
+    public function testAssignees($filename, $expected_reesult)
+    {
+        $data = $this->createDataFromFixture($filename);
+        self::assertEquals($expected_reesult, $data->getAssignees());
+    }
+
     protected function createDataFromFixture($filename)
     {
         $file_contents = json_decode(file_get_contents(__DIR__ . '/fixtures/' . $filename));
         return Config::createFromComposerData($file_contents);
     }
 
+    public function getAssignees()
+    {
+        return [
+            [
+                'empty.json',
+                [],
+            ],
+            [
+                'assignees.json',
+                [],
+            ],
+            [
+                'assignees2.json',
+                [],
+            ],
+            [
+                'assignees3.json',
+                [
+                    "test_user"
+                ],
+            ],
+        ];
+    }
+
     public function getBranchPrefix()
     {
         return [

diff --git a/tests/fixtures/assignees.json b/tests/fixtures/assignees.json
@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "assignees": "should_be_array"
+    }
+  }
+}
diff --git a/tests/fixtures/assignees2.json b/tests/fixtures/assignees2.json
@@ -0,0 +1,7 @@
+{
+  "extra": {
+    "violinist": {
+      "assignees": []
+    }
+  }
+}
diff --git a/tests/fixtures/assignees3.json b/tests/fixtures/assignees3.json
@@ -0,0 +1,9 @@
+{
+  "extra": {
+    "violinist": {
+      "assignees": [
+        "test_user"
+      ]
+    }
+  }
+}