djsmithme / Router

Src/Router/Handler/ClosureHandler.php

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Ds\Router\Handler;
 
 use Ds\Router\RouteHandlerInterface;
-use Psr\Http\Message\ResponseInterface;
 
 /**
  * Closure Route Handler.

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
      * @param callable $closure
      * @return Closure
      */
-    public static function add($preResponse = '', callable $closure){
+    public static function add($preResponse = '', callable $closure) {
         return new Closure($preResponse, $closure);
     }
 
@@ -43,10 +43,10 @@ 
      */
     function getResponse($vars = [])
     {
-        if ($this->resolved === null){
+        if ($this->resolved === null) {
             throw new \Exception("Handler must be resolved before calling getContents().");
         }
-        return call_user_func($this->closure,$vars);
+        return call_user_func($this->closure, $vars);
     }
 
     /**

Tests/Router/RouterTest.php

Spacing +11 added lines, -11 removed lines

@@ -44,45 +44,45 @@
         );
     }
 
-    public function testGetAdaptor(){
+    public function testGetAdaptor() {
         $expected = $this->adaptor;
         $actual = $this->router->getAdaptor();
         $this->assertEquals($expected, $actual);
     }
 
-    public function testWithAdaptor(){
+    public function testWithAdaptor() {
         $newAdaptor = $this->getMockBuilder('\Ds\Router\AdaptorInterface')->getMock();
         $newRouter = $this->router->withAdaptor($newAdaptor);
         $this->assertNotSame($this->adaptor, $newRouter->getAdaptor());
     }
 
-    public function testGetRouterResponse(){
+    public function testGetRouterResponse() {
 
         $request = $this->getMockBuilder('\Psr\Http\Message\RequestInterface')->getMock();
 
-        $routerResponse = new RouterResponse(200,[],null,[]);
+        $routerResponse = new RouterResponse(200, [], null, []);
 
-        $this->adaptor->expects( $this->once() )
+        $this->adaptor->expects($this->once())
             ->method('match')
             ->willReturn($routerResponse);
 
-        $this->routes->expects( $this->once() )
+        $this->routes->expects($this->once())
             ->method('getRoutes')
             ->willReturn([]);
 
         $response = $this->router->getRouterResponse($request);
-        $this->assertInstanceOf('\Ds\Router\RouterResponseInterface' , $response);
+        $this->assertInstanceOf('\Ds\Router\RouterResponseInterface', $response);
 
     }
 
-    public function testWithRoutes(){
+    public function testWithRoutes() {
 
         $routes = $this->getMockBuilder('\Ds\Router\RoutesInterface')->getMock();
         $this->router->withRoutes($routes);
 
-        $reflector = new \ReflectionClass( '\Ds\Router\Router' );
-        $prop = $reflector->getProperty( 'routes' );
-        $prop->setAccessible( true );
+        $reflector = new \ReflectionClass('\Ds\Router\Router');
+        $prop = $reflector->getProperty('routes');
+        $prop->setAccessible(true);
 
         $this->assertSame($routes, $prop->getValue($this->router));