Renamed TestInjector to MockInjector as more appropriate.

Author: Adam Benson

composer.json

@@ -1,5 +1,5 @@
 {
-    "name": "bigcommerce/test-injector",
+    "name": "bigcommerce/mock-injector",
     "type": "library",
     "description": "Auto-mocking injector for testing DI components",
     "keywords": [
@@ -19,7 +19,7 @@
     "repositories": [
         {
             "type": "vcs",
-            "url": "ssh://git@github.com/bigcommerce-labs/pimple-injector.git"
+            "url": "ssh://git@github.com/bigcommerce-labs/injector.git"
         }
     ],
     "require": {
@@ -29,7 +29,7 @@
     },
     "autoload": {
         "psr-4": {
-            "Bigcommerce\\TestInjector\\": "src/"
+            "Bigcommerce\\MockInjector\\": "src/"
         }
     },
     "autoload-dev": {

composer.lock

@@ -1,5 +1,5 @@
 <?php
-namespace Bigcommerce\TestInjector;
+namespace Bigcommerce\MockInjector;
 
 use Bigcommerce\Injector\InjectorInterface;
 
@@ -11,7 +11,7 @@
  */
 abstract class AutoMockingTest extends \PHPUnit_Framework_TestCase
 {
-    /** @var InjectorInterface|TestInjector */
+    /** @var InjectorInterface|MockInjector */
     protected $injector;
 
     /**
@@ -21,7 +21,7 @@ abstract class AutoMockingTest extends \PHPUnit_Framework_TestCase
     protected function setUp()
     {
         parent::setUp();
-        $this->injector = new TestInjector();
+        $this->injector = new MockInjector();
     }
 
     /**

src/AutoMockingTest.php

@@ -1,5 +1,5 @@
 <?php
-namespace Bigcommerce\TestInjector;
+namespace Bigcommerce\MockInjector;
 
 use Bigcommerce\Injector\Cache\ArrayServiceCache;
 use Bigcommerce\Injector\Injector;
@@ -10,12 +10,12 @@
 use Prophecy\Prophet;
 
 /**
- * The TestInjector is a specialised version of the injector that automatically mocks dependencies of objects being
+ * The MockInjector is a specialised version of the injector that automatically mocks dependencies of objects being
  * tested using the same introspection of parameter types as the default Injector.
- * Note that any prophecy mock auto-generated by the TestInjector can be accessed via the ::getMock method.
+ * Note that any prophecy mock auto-generated by the MockInjector can be accessed via the ::getMock method.
  * @package \Bigcommerce\Injector
  */
-class TestInjector implements InjectorInterface
+class MockInjector implements InjectorInterface
 {
     /**
      * @var InjectorInterface
@@ -27,8 +27,8 @@ class TestInjector implements InjectorInterface
     private $mockingContainer;
 
     /**
-     * TestInjector constructor.
-     * NOTE: The TestInjector intentionally does not use DependencyInjection and instantiates its own
+     * MockInjector constructor.
+     * NOTE: The MockInjector intentionally does not use DependencyInjection and instantiates its own
      * dependencies via poor mans DI as it is used only for testing purposes and is tightly coupled to
      * its dependencies.
      * @param MockingContainer $mockingContainer
@@ -119,4 +119,4 @@ public function invoke($instance, $methodName, $parameters = [])
     {
         return $this->injector->invoke($instance, $methodName, $parameters);
     }
-}
+}

src/TestInjector.php src/MockInjector.phpsrc/TestInjector.php renamed to src/MockInjector.php

@@ -1,5 +1,5 @@
 <?php
-namespace Bigcommerce\TestInjector;
+namespace Bigcommerce\MockInjector;
 
 
 use Interop\Container\ContainerInterface;
@@ -98,4 +98,4 @@ private function createOrGetMock($mockClassName)
         }
         return $this->mocks[$mockClassName];
     }
-}
+}

src/MockingContainer.php

@@ -2,15 +2,15 @@
 namespace Tests;
 
 use Bigcommerce\Injector\InjectorInterface;
-use Bigcommerce\TestInjector\MockingContainer;
-use Bigcommerce\TestInjector\TestInjector;
+use Bigcommerce\MockInjector\MockingContainer;
+use Bigcommerce\MockInjector\MockInjector;
 use Prophecy\Prophecy\ObjectProphecy;
 use Tests\Dummy\DummyDependency;
 use Tests\Dummy\DummySubDependency;
 
 /**
  *
- * @coversDefaultClass \Bigcommerce\TestInjector\TestInjector
+ * @coversDefaultClass \Bigcommerce\MockInjector\TestInjector
  */
 class TestInjectorTest extends \PHPUnit_Framework_TestCase
 {
@@ -32,7 +32,7 @@ protected function setUp()
     public function testCreate()
     {
         $this->injector->create("abc123", [1 => "hello"])->willReturn("cat")->shouldBeCalledTimes(1);
-        $i = new TestInjector($this->mockContainer->reveal(), $this->injector->reveal());
+        $i = new MockInjector($this->mockContainer->reveal(), $this->injector->reveal());
         $this->assertEquals("cat", $i->create("abc123", [1 => "hello"]));
     }
 
@@ -43,7 +43,7 @@ public function testInvoke()
     {
         $obj = new \stdClass();
         $this->injector->invoke($obj, "method1", [1 => "hello"])->willReturn("fish")->shouldBeCalledTimes(1);
-        $i = new TestInjector($this->mockContainer->reveal(), $this->injector->reveal());
+        $i = new MockInjector($this->mockContainer->reveal(), $this->injector->reveal());
         $this->assertEquals("fish", $i->invoke($obj, "method1", [1 => "hello"]));
     }
 
@@ -53,7 +53,7 @@ public function testInvoke()
     public function testCheckPredictions()
     {
         $this->mockContainer->checkPredictions()->shouldBeCalledTimes(1);
-        $i = new TestInjector($this->mockContainer->reveal(), $this->injector->reveal());
+        $i = new MockInjector($this->mockContainer->reveal(), $this->injector->reveal());
         $i->checkPredictions();
     }
 
@@ -63,7 +63,7 @@ public function testCheckPredictions()
     public function testGetAllMocks()
     {
         $this->mockContainer->getAllMocks()->willReturn([1, 2, 6])->shouldBeCalledTimes(1);
-        $i = new TestInjector($this->mockContainer->reveal(), $this->injector->reveal());
+        $i = new MockInjector($this->mockContainer->reveal(), $this->injector->reveal());
         $this->assertEquals([1, 2, 6], $i->getAllMocks());
     }
 
@@ -74,7 +74,7 @@ public function testGetMock()
     {
         $dummy = new \stdClass();
         $this->mockContainer->getMock("blah")->willReturn($dummy)->shouldBeCalledTimes(1);
-        $i = new TestInjector($this->mockContainer->reveal(), $this->injector->reveal());
+        $i = new MockInjector($this->mockContainer->reveal(), $this->injector->reveal());
         $this->assertEquals($dummy, $i->getMock("blah"));
     }
 
@@ -85,7 +85,7 @@ public function testGetMock()
      */
     public function testWithoutDependencies()
     {
-        $i = new TestInjector();
+        $i = new MockInjector();
         /** @var DummyDependency $obj */
         $obj = $i->create(DummyDependency::class);
         $this->assertInstanceOf(DummySubDependency::class, $obj->getDependency());