Merge pull request #111 from cspray/inspect-tests

Adds test for inspecting shares
rdlowrey · Jul 26, 2015 · cfce81b · cfce81b
2 parents 66f9862 + 366d4e3
commit cfce81b
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/lib/Injector.php b/lib/Injector.php
@@ -302,6 +302,7 @@ public function inspect($nameFilter = null, $typeFilter = null)
             self::I_ALIASES => "aliases",
             self::I_SHARES => "shares"
         );
+
         foreach ($types as $type => $source) {
             if ($typeFilter & $type) {
                 $result[$type] = $this->filter($this->{$source}, $name);
@@ -315,8 +316,8 @@ private function filter($source, $name)
     {
         if (empty($name)) {
             return $source;
-        } elseif (isset($source[$name])) {
-            return $source[$name];
+        } elseif (array_key_exists($name, $source)) {
+            return array($name => $source[$name]);
         } else {
             return array();
         }

diff --git a/test/InjectorTest.php b/test/InjectorTest.php
@@ -1000,4 +1000,13 @@ public function testDebugMake()
             $this->assertEquals('auryn\test\depinterface', $chain[1]);
         }
     }
+
+    public function testInspectShares()
+    {
+        $injector = new Injector();
+        $injector->share('Auryn\Test\SomeClassName');
+
+        $inspection = $injector->inspect('Auryn\Test\SomeClassName', Injector::I_SHARES);
+        $this->assertArrayHasKey('auryn\test\someclassname', $inspection[Injector::I_SHARES]);
+    }
 }