Merge pull request #107 from SenseException/coverage

Add test for invalid service alias names
bitExpert · Sep 6, 2017 · f0a8fd1 · f0a8fd1
2 parents 027b5eb + d119c00
commit f0a8fd1
Showing 1 changed file with 23 additions and 0 deletions.
diff --git a/tests/bitExpert/Disco/Annotations/AliasUnitTest.php b/tests/bitExpert/Disco/Annotations/AliasUnitTest.php
@@ -61,4 +61,27 @@ public function aliasShouldBeNamedOrTypeAlias()
         self::expectExceptionMessage('Alias should either be a named alias or a type alias!');
         new Alias();
     }
+
+    /**
+     * @test
+     * @dataProvider invalidNameProvider
+     */
+    public function aliasNameCannotBeEmpty($name)
+    {
+        self::expectException(AnnotationException::class);
+        self::expectExceptionMessage('Alias should either be a named alias or a type alias!');
+        new Alias(['value' => ['name' => $name, 'type' => false]]);
+    }
+
+    public function invalidNameProvider()
+    {
+        return [
+            [''],
+            [0],
+            [0.0],
+            [false],
+            [null],
+            [[]],
+        ];
+    }
 }