Merge pull request #5 from jdreesen/controller-resolver-as-shared-service

Override ControllerResolver as a shared service

Author: mnapoli

src/Application.php

@@ -43,7 +43,9 @@ public function __construct(ContainerBuilder $containerBuilder = null, array $va
         parent::__construct($values);
 
         // Override the controller resolver with ours
-        $this['resolver'] = new ControllerResolver($this->phpdi);
+        $this['resolver'] = $this->share(function () {
+            return new ControllerResolver($this->phpdi);
+        });
     }
 
     public function offsetGet($id)

tests/ApplicationTest.php

@@ -42,4 +42,15 @@ public function the_application_should_expose_phpdi_container()
 
         $this->assertInstanceOf('DI\Container', $app->getPhpDi());
     }
+
+    /**
+     * @test
+     */
+    public function the_controller_resolver_should_be_registered_as_a_service()
+    {
+        $app = new Application();
+
+        $this->assertInstanceOf('Closure', $app->raw('resolver'));
+        $this->assertInstanceOf('DI\Bridge\Silex\Controller\ControllerResolver', $app['resolver']);
+    }
 }