djsmithme / Router

Tests/Router/Adaptor/FastRouteTest.php

Spacing +7 added lines, -7 removed lines

@@ -5,7 +5,7 @@ 
 use Ds\Router\Adaptor\FastRoute;
 use Ds\Router\Handler\Handler;
 
-class MockHandler extends Handler{
+class MockHandler extends Handler {
 
     function getResponse($vars = [])
     {
@@ -22,7 +22,7 @@ 
     public $fastRoute;
     public $options;
 
-    public function setUp(){
+    public function setUp() {
 
         $this->options = [
             'handler' => '\Tests\Ds\Router\Adaptor\MockHandler',
@@ -41,22 +41,22 @@ 
         $this->fastRoute = new FastRoute($this->options);
     }
 
-    public function testFindHandler(){
+    public function testFindHandler() {
         $result = $this->fastRoute->findHandler('default');
         $this->assertInstanceOf('\Tests\Ds\Router\Adaptor\MockHandler', $result);
     }
 
-    public function testFindHandlerDefault(){
+    public function testFindHandlerDefault() {
         $result = $this->fastRoute->findHandler('notFound');
         $this->assertInstanceOf('\Tests\Ds\Router\Adaptor\MockHandler', $result);
     }
 
-    public function testCheckCacheFileDisabled(){
+    public function testCheckCacheFileDisabled() {
         $expected = true;
         $actual = $this->fastRoute->checkCacheFile();
         $this->assertEquals($expected, $actual);
     }
 
-    public function testGetCachedDispatcher(){}
-    public function testMatch(){}
+    public function testGetCachedDispatcher() {}
+    public function testMatch() {}
 }
\ No newline at end of file

Tests/Router/Handler/ClassHandlerTest.php

Spacing +17 added lines, -17 removed lines

@@ -4,7 +4,7 @@ 
 
 use Ds\Router\Handler\ClassHandler;
 
-class TestController{
+class TestController {
 
     public $resolveValue;
 
@@ -13,8 +13,8 @@ 
         $this->resolveValue = $resolveValue;
     }
 
-    public function test($var){
-        return 'foo'.$var;
+    public function test($var) {
+        return 'foo' . $var;
     }
 }
 
@@ -29,44 +29,44 @@ 
         $this->controller = 'Tests\Ds\Router\Handler\TestController';
     }
 
-    public function testConstructController(){
-        $reflector = new \ReflectionClass(  $this->handler);
-        $prop = $reflector->getProperty( 'controller' );
+    public function testConstructController() {
+        $reflector = new \ReflectionClass($this->handler);
+        $prop = $reflector->getProperty('controller');
         $prop->setAccessible(true);
         $this->assertEquals($this->controller, $prop->getValue($this->handler));
     }
 
-    public function testConstructMethod(){
-        $reflector = new \ReflectionClass(  $this->handler);
-        $prop = $reflector->getProperty( 'method' );
+    public function testConstructMethod() {
+        $reflector = new \ReflectionClass($this->handler);
+        $prop = $reflector->getProperty('method');
         $prop->setAccessible(true);
         $this->assertEquals('test', $prop->getValue($this->handler));
     }
 
-    public function testConstructPreResponse(){
-        $reflector = new \ReflectionClass(  $this->handler);
-        $prop = $reflector->getProperty( 'preResponse' );
+    public function testConstructPreResponse() {
+        $reflector = new \ReflectionClass($this->handler);
+        $prop = $reflector->getProperty('preResponse');
         $prop->setAccessible(true);
-        $this->assertEquals($this->controller.'::test', $prop->getValue($this->handler));
+        $this->assertEquals($this->controller . '::test', $prop->getValue($this->handler));
     }
 
-    public function testResolveClassInstance(){
+    public function testResolveClassInstance() {
         $handler = $this->handler->resolve();
         $reflector = new \ReflectionClass($handler);
-        $prop = $reflector->getProperty( 'resolved' );
+        $prop = $reflector->getProperty('resolved');
         $prop->setAccessible(true);
         $this->assertInstanceOf($this->controller, $prop->getValue($handler));
     }
 
 
-    public function testGetResponse(){
+    public function testGetResponse() {
         $expected = 'foobar';
         $this->handler->resolve();
         $actual = $this->handler->getResponse('bar');
         $this->assertEquals($expected, $actual);
     }
 
-    public function testGetResponseException(){
+    public function testGetResponseException() {
         $this->setExpectedException('Exception');
         $this->handler->getResponse('bar');
     }

Tests/Router/Handler/HandlerTest.php

Spacing +9 added lines, -9 removed lines

@@ -10,7 +10,7 @@ 
 
 use Ds\Router\Handler\Handler;
 
-class TestHandler extends Handler{
+class TestHandler extends Handler {
 
     public $preResponse = 'pre-response';
     public $name = ['foo' => 'foo'];
@@ -44,7 +44,7 @@ 
         $this->handler = new TestHandler();
     }
 
-    public function testSetName(){
+    public function testSetName() {
         $expected = ['bar'];
         $this->handler->setName($expected);
 
@@ -53,20 +53,20 @@ 
         $reflectionProperty->setAccessible(true);
         $actual = $reflectionProperty->getValue($this->handler);
 
-        $this->assertEquals($expected,$actual);
+        $this->assertEquals($expected, $actual);
     }
 
-    public function testAddName(){
+    public function testAddName() {
         $expected = 'foo';
         $this->handler->addName($expected);
         $reflectionClass = new \ReflectionClass('Tests\Ds\Router\Handler\TestHandler');
         $reflectionProperty = $reflectionClass->getProperty('name');
         $reflectionProperty->setAccessible(true);
         $actual = $reflectionProperty->getValue($this->handler);
-        $this->assertEquals([$expected => $expected],$actual);
+        $this->assertEquals([$expected => $expected], $actual);
     }
     
-    public function testGetNames(){
+    public function testGetNames() {
         $expected = [
             'foo' => 'foo',
         ];
@@ -74,7 +74,7 @@ 
         $this->assertEquals($expected, $this->handler->getNames());
     }
 
-    public function testWithPreResponse(){
+    public function testWithPreResponse() {
      
         $expected = 'preResponse String';
         $this->handler = $this->handler->withPreResponse($expected);
@@ -89,13 +89,13 @@ 
     }
 
 
-    public function testGetPreResponse(){
+    public function testGetPreResponse() {
         $expected = 'preResponse String';
         $this->handler = $this->handler->withPreResponse($expected);
         $this->assertEquals($expected, $this->handler->getPreResponse());
     }
     
-    public function testSetState(){
+    public function testSetState() {
         $actual = TestHandler::__set_state(['state' => 'newState']);
         $expected = new TestHandler('newState');
         $this->assertEquals($expected, $actual);

Tests/Router/RouterResponseTest.php

Spacing +6 added lines, -6 removed lines

@@ -24,13 +24,13 @@ 
         );
     }
 
-    public function testGetHandler(){
+    public function testGetHandler() {
         $handler = $this->response->getHandler();
         $this->assertInstanceOf('\Ds\Router\Interfaces\RouteHandlerInterface', $handler);
         $this->assertSame($this->handler, $handler);
     }
 
-    public function testWithHandler(){
+    public function testWithHandler() {
 
         $newHandler = $this->getMockBuilder('\Ds\Router\Interfaces\RouteHandlerInterface')->getMock();
         $oldHandler = $this->response->getHandler();
@@ -42,11 +42,11 @@ 
      * Returns the Status Code.
      * @return int
      */
-    public function testGetStatusCode(){
-        $this->assertEquals( $this->response->getStatusCode(), 200);
+    public function testGetStatusCode() {
+        $this->assertEquals($this->response->getStatusCode(), 200);
     }
 
-    public function testGetResponseInstance(){
+    public function testGetResponseInstance() {
 
         $httpResponse = $this->getMockBuilder('\Psr\Http\Message\ResponseInterface')->getMock();
 
@@ -54,7 +54,7 @@ 
             ->method('getResponse')
             ->willReturn($httpResponse);
         
-        $response = new RouterResponse( 200, ['get'], $this->handler, []);
+        $response = new RouterResponse(200, ['get'], $this->handler, []);
         $psrResponse = $response->getResponse();
 
         $this->assertInstanceOf('\Psr\Http\Message\ResponseInterface', $psrResponse);

Src/Router/Adaptor/Adaptor.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
      * Return Adaptor Options
      * @return array
      */
-    public function getOptions(){
+    public function getOptions() {
         return $this->options;
     }
 
@@ -45,7 +45,7 @@ 
      * @param array $options
      * @return void
      */
-    public function withOptions(array $options = []){
+    public function withOptions(array $options = []) {
         $this->options = $options;
         return;
     }

Src/Router/Handler/Handler.php

Spacing +6 added lines, -6 removed lines

@@ -47,7 +47,7 @@ 
      * Set Route name;
      * @param $name
      */
-    public function setName(array $name = []){
+    public function setName(array $name = []) {
             $this->name = $name;
     }
 
@@ -55,7 +55,7 @@ 
      * Add Route name;
      * @param $name
      */
-    public function addName($name = ''){
+    public function addName($name = '') {
        $this->name[$name] = $name;
     }
 
@@ -63,7 +63,7 @@ 
      * Get all added names
      * @return mixed
      */
-    public function getNames(){
+    public function getNames() {
         return (array)$this->name;
     }
 
@@ -71,7 +71,7 @@ 
      * Get route (early) middleware pre-response.
      * @return mixed
      */
-    public function getPreResponse(){
+    public function getPreResponse() {
         return $this->preResponse;
     }
 
@@ -80,7 +80,7 @@ 
      * @param $string
      * @return $this
      */
-    public function withPreResponse($string){
+    public function withPreResponse($string) {
 
         $new = clone $this;
         $new->preResponse = $string;
@@ -98,7 +98,7 @@ 
     public static function __set_state($fields) {
 
         $args = [];
-        foreach($fields as $key => $value){
+        foreach ($fields as $key => $value) {
             $args[] = $value;
         }
 

Src/Router/Routes.php

Spacing +4 added lines, -4 removed lines

@@ -49,12 +49,12 @@
     {
 
         $pattern = ltrim($pattern, '/');
-        $pattern = '/'.$pattern;
+        $pattern = '/' . $pattern;
 
-        if ($this->groupedRoute === false){
+        if ($this->groupedRoute === false) {
             $routePattern = $pattern;
-        }else{
-            $routePattern = implode('',$this->dir).$pattern;
+        } else {
+            $routePattern = implode('', $this->dir) . $pattern;
         }
 
         $this->isUnique($method, $routePattern);

Tests/Router/RoutesTest.php

Spacing +10 added lines, -10 removed lines

@@ -14,15 +14,15 @@ 
         $this->routes = new Routes();
     }
 
-    public function testGroup(){
+    public function testGroup() {
 
         $handler = $this->getMockBuilder('Ds\Router\Interfaces\RouteHandlerInterface')->getMock();
         $routes = $this->routes;
 
-        $this->routes->group('/path',function() use ($routes, $handler){
-            $routes->add(['GET','POST'], '/foo', $handler, ['global']);
+        $this->routes->group('/path', function() use ($routes, $handler){
+            $routes->add(['GET', 'POST'], '/foo', $handler, ['global']);
 
-                $routes->group('/sub-dir',function() use ($routes, $handler){
+                $routes->group('/sub-dir', function() use ($routes, $handler){
                     $routes->add(['GET'], '/sub-page', $handler, ['sub-dir']);
                 });
         });
@@ -53,10 +53,10 @@ 
         $this->assertEquals($expected, $actual);
     }
 
-    public function testAdd(){
+    public function testAdd() {
 
         $handler = $this->getMockBuilder('Ds\Router\Interfaces\RouteHandlerInterface')->getMock();
-        $this->routes->add(['GET','POST'], '/foo', $handler, ['global']);
+        $this->routes->add(['GET', 'POST'], '/foo', $handler, ['global']);
 
         $actual = $this->routes->getRoutes();
 
@@ -78,11 +78,11 @@ 
         $this->assertEquals($expected, $actual);
     }
 
-    public function testGetRoutes(){
+    public function testGetRoutes() {
         $handler = $this->getMockBuilder('Ds\Router\Interfaces\RouteHandlerInterface')->getMock();
 
         $this->routes->add('GET', '/foo', $handler, ['global']);
-        $this->routes->add('POST', '/baz', $handler, ['some','other','names']);
+        $this->routes->add('POST', '/baz', $handler, ['some', 'other', 'names']);
 
         $actual = $this->routes->getRoutes();
 
@@ -97,14 +97,14 @@ 
                 'method' => 'POST',
                 'pattern' => '/baz',
                 'handler' => $handler,
-                'name' => ['some','other','names']
+                'name' => ['some', 'other', 'names']
             ]
         ];
 
         $this->assertEquals($expected, $actual);
     }
 
-    public function testIsUnique(){
+    public function testIsUnique() {
         $handler = $this->getMockBuilder('Ds\Router\Interfaces\RouteHandlerInterface')->getMock();
         $this->routes->add('GET', '/foo', $handler, ['global']);
         $this->setExpectedException('\Exception');

Src/Router/Handler/ClassHandler.php

Spacing +13 added lines, -13 removed lines

@@ -24,7 +24,7 @@ 
      * @param string $method
      * @return ClassHandler
      */
-    public static function add($controller, $method){
+    public static function add($controller, $method) {
         return new ClassHandler($controller, $method);
     }
 
@@ -51,19 +51,19 @@ 
      * @return $this
      * @throws \Exception
      */
-    public function resolve(...$arguments){
+    public function resolve(...$arguments) {
 
 
-        $controllerMethod = explode('::',$this->getPreResponse());
+        $controllerMethod = explode('::', $this->getPreResponse());
 
         $this->controller = isset($controllerMethod[0]) ? $controllerMethod[0] : $this->controller;
         $this->method = isset($controllerMethod[1]) ? $controllerMethod[1] : $this->method;
 
-        try{
-            $this->resolveController( $this->controller);
-            $this->resolveMethod( $this->method );
+        try {
+            $this->resolveController($this->controller);
+            $this->resolveMethod($this->method);
             $this->resolved = new $this->controller(...$arguments);
-        }catch(\Exception $e){
+        }catch (\Exception $e) {
             throw new \Exception($e->getMessage());
         }
         return $this;
@@ -77,9 +77,9 @@ 
      * @return ResponseInterface
      * @throws \Exception
      */
-    public function getResponse($vars = [] ){
+    public function getResponse($vars = []) {
 
-        if ($this->resolved === null){
+        if ($this->resolved === null) {
             throw new \Exception("Handler must be resolved before calling getContents().");
         }
 
@@ -96,9 +96,9 @@ 
      * @return bool
      * @throws \Exception controller:{controller} not found
      */
-    private function resolveController($controller){
+    private function resolveController($controller) {
 
-        if ( !class_exists($controller)){
+        if (!class_exists($controller)) {
             throw new \Exception("Controller {$controller} not found");
         }
         return true;
@@ -111,8 +111,8 @@ 
      * @return bool
      * @throws \Exception
      */
-    private function resolveMethod($method){
-        if (method_exists($this->controller,$method)){
+    private function resolveMethod($method) {
+        if (method_exists($this->controller, $method)) {
             return true;
         }
         throw new \Exception("method: {$this->controller}::{$method} not found");