[BUGFIX] DI fails with CompilerDefinition; solves zendframework/zendf…

…ramework#5738 - Test - assertions on false added
zendframework · Feb 28, 2014 · a53cf81 · a53cf81
1 parent 2d62d4b
commit a53cf81
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/test/Definition/CompilerDefinitionTest.php b/test/Definition/CompilerDefinitionTest.php
@@ -139,8 +139,18 @@ public function testHasMethodParameters()
         $definition->addDirectory(__DIR__ . '/../TestAsset/SetterInjection');
         $definition->addDirectory(__DIR__ . '/../TestAsset/CompilerClasses');
         $definition->compile();
+
+        // constructor injection
         $this->assertTrue($definition->hasMethodParameters('ZendTest\Di\TestAsset\ConstructorInjection\B', '__construct'));
+        // setter injection
         $this->assertTrue($definition->hasMethodParameters('ZendTest\Di\TestAsset\SetterInjection\B', 'setA'));
+        // setter injection with method from derived class
         $this->assertTrue($definition->hasMethodParameters('ZendTest\Di\TestAsset\CompilerClasses\D', 'setB'));
+        // class does not exist
+        $this->assertFalse($definition->hasMethodParameters('ZendTest\Di\TestAsset\ConstructorInjection\BB', '__construct'));
+        // method not existing
+        $this->assertFalse($definition->hasMethodParameters('ZendTest\Di\TestAsset\SetterInjection\B', 'setB'));
+        // method exists but has no parameters
+        $this->assertFalse($definition->hasMethodParameters('ZendTest\Di\TestAsset\SetterInjection\StaticSetter', 'setFoo'));
     }
 }