burzum / php-router

src/Router.php

Spacing +5 added lines, -5 removed lines

@@ -183,7 +183,7 @@ 
     public function setBasePath(string $basePath): self
     {
         $basePath = trim($basePath, '/');
-        $this->basePath = $basePath ? '/' . $basePath : '';
+        $this->basePath = $basePath ? '/'.$basePath : '';
         return $this;
     }
 
@@ -220,7 +220,7 @@ 
             $options['host'] = $this->hosts[$scheme][$host];
         }
 
-        $patternKey = md5($options['pattern'] . '-' . $options['name']);
+        $patternKey = md5($options['pattern'].'-'.$options['name']);
         if (isset($this->pattern[$scheme][$host][$patternKey])) {
             $route = $this->pattern[$scheme][$host][$patternKey];
         } else {
@@ -280,7 +280,7 @@ 
             $options['host'] = $this->hosts[$scheme][$host];
         }
 
-        $patternKey = md5($options['pattern'] . '-' . $options['name']);
+        $patternKey = md5($options['pattern'].'-'.$options['name']);
         if (isset($this->pattern[$scheme][$host][$patternKey])) {
             $instance = $this->pattern[$scheme][$host][$patternKey];
         } else {
@@ -293,7 +293,7 @@ 
             $this->pattern[$scheme][$host][$patternKey] = $instance;
         }
 
-        $methods = $options['methods'] ? (array)$options['methods'] : [];
+        $methods = $options['methods'] ? (array) $options['methods'] : [];
         $instance->allow($methods);
         foreach ($methods as $method) {
             $this->routes[$scheme][$host][strtoupper($method)][] = $instance;
@@ -411,7 +411,7 @@ 
             $request = $parsed + $request;
         }
 
-        $request['path'] = (ltrim((string)strtok($request['path'], '?'), '/'));
+        $request['path'] = (ltrim((string) strtok($request['path'], '?'), '/'));
         $request['method'] = strtoupper($request['method']);
 
         return $request;

spec/Suite/Router.spec.php

Spacing +163 added lines, -163 removed lines

@@ -12,32 +12,32 @@ 
 use Psr\Http\Message\ServerRequestInterface;
 use Psr\Http\Message\UriInterface;
 
-describe("Router", function () {
+describe("Router", function() {
 
 
-    beforeEach(function () {
+    beforeEach(function() {
 
 
         $this->router = new Router();
-        $this->export = function ($request) {
+        $this->export = function($request) {
 
             return array_intersect_key($request, array_fill_keys(['path', 'method', 'host', 'scheme'], true));
         };
     });
-    describe("->__construct()", function () {
+    describe("->__construct()", function() {
 
 
-        it("formats the basePath", function () {
+        it("formats the basePath", function() {
 
 
             $router = new Router(['basePath' => '/']);
             expect($router->getBasePath())->toBe('');
         });
     });
-    describe("->getBasePath()", function () {
+    describe("->getBasePath()", function() {
 
 
-        it("sets an empty basePath", function () {
+        it("sets an empty basePath", function() {
 
 
             expect($this->router->setBasePath('/'))->toBe($this->router);
@@ -45,7 +45,7 @@ 
             expect($this->router->setBasePath(''))->toBe($this->router);
             expect($this->router->getBasePath())->toBe('');
         });
-        it("adds an leading slash for non empty basePath", function () {
+        it("adds an leading slash for non empty basePath", function() {
 
 
             expect($this->router->setBasePath('app'))->toBe($this->router);
@@ -54,24 +54,24 @@ 
             expect($this->router->getBasePath())->toBe('/base');
         });
     });
-    describe("->bind()", function () {
+    describe("->bind()", function() {
 
 
-        it("binds a named route", function () {
+        it("binds a named route", function() {
 
 
             $r = $this->router;
-            $route = $r->bind('foo/bar', ['name' => 'foo'], function () {
+            $route = $r->bind('foo/bar', ['name' => 'foo'], function() {
                 return 'hello';
             });
             expect(isset($r['foo']))->toBe(true);
             expect($r['foo'])->toBe($route);
         });
-        it("matches on methods", function () {
+        it("matches on methods", function() {
 
 
             $r = $this->router;
-            $r->bind('foo/bar', ['methods' => ['POST', 'PUT']], function () {
+            $r->bind('foo/bar', ['methods' => ['POST', 'PUT']], function() {
             });
             $route = $r->route('foo/bar', 'POST');
             expect($route->Methods())->toBe(['POST', 'PUT']);
@@ -83,11 +83,11 @@ 
                 expect($e->getMessage())->toBe("No route found for `*:*:GET:/bar/foo`.");
             }
         });
-        it("supports lowercase method names", function () {
+        it("supports lowercase method names", function() {
 
 
             $r = $this->router;
-            $r->bind('foo/bar', ['methods' => ['POST', 'PUT']], function () {
+            $r->bind('foo/bar', ['methods' => ['POST', 'PUT']], function() {
             });
             $route = $r->route('foo/bar', 'post');
             expect($route->Methods())->toBe(['POST', 'PUT']);
@@ -99,52 +99,52 @@ 
                 expect($e->getMessage())->toBe("No route found for `*:*:GET:/bar/foo`.");
             }
         });
-        it("matches on same path different methods", function () {
+        it("matches on same path different methods", function() {
 
             $r = $this->router;
-            $r->bind('foo/bar', ['name' => 'foo', 'methods' => ['POST']], function () {
+            $r->bind('foo/bar', ['name' => 'foo', 'methods' => ['POST']], function() {
             });
-            $r->bind('foo/bar', ['name' => 'bar', 'methods' => ['PUT']], function () {
+            $r->bind('foo/bar', ['name' => 'bar', 'methods' => ['PUT']], function() {
             });
             $route = $r->route('foo/bar', 'POST');
             expect($route->name)->toBe('foo');
             $route = $r->route('foo/bar', 'PUT');
             expect($route->name)->toBe('bar');
         });
-        it("throws an exception when the handler is not a closure", function () {
+        it("throws an exception when the handler is not a closure", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $r = $this->router;
                 $r->bind('foo', 'substr');
             };
             expect($closure)->toThrow(new RouterException("The handler needs to be an instance of `Closure` or implements the `__invoke()` magic method."));
         });
-        it("throws an exception when trying to use the `'method'` option", function () {
+        it("throws an exception when trying to use the `'method'` option", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $r = $this->router;
-                $r->bind('foo', ['method' => 'GET'], function () {
+                $r->bind('foo', ['method' => 'GET'], function() {
                 });
             };
             expect($closure)->toThrow(new RouterException("Use the `'methods'` option to limit HTTP verbs on a route binding definition."));
         });
     });
-    describe("->link()", function () {
+    describe("->link()", function() {
 
 
-        it("forwards router base path", function () {
+        it("forwards router base path", function() {
 
 
             $r = $this->router;
             $r->basePath('app');
 
-            $r->group(['host' => 'www.{domain}.com', 'scheme' => 'https'], function ($r) {
+            $r->group(['host' => 'www.{domain}.com', 'scheme' => 'https'], function($r) {
 
-                $r->bind('foo/{bar}', ['name' => 'foo'], function () {
+                $r->bind('foo/{bar}', ['name' => 'foo'], function() {
                 });
             });
             $link = $r->link('foo', [
@@ -153,28 +153,28 @@ 
             ], ['absolute' => true]);
             expect($link)->toBe('https://www.example.com/app/foo/baz');
         });
-        it("maintains prefixes in nested routes", function () {
+        it("maintains prefixes in nested routes", function() {
 
 
             $r = $this->router;
-            $r->group('foo', ['name' => 'foz'], function ($r) {
+            $r->group('foo', ['name' => 'foz'], function($r) {
 
-                $r->group('bar', ['name' => 'baz'], function ($r) {
+                $r->group('bar', ['name' => 'baz'], function($r) {
 
-                    $r->bind('{var1}', ['name' => 'quz'], function () {
+                    $r->bind('{var1}', ['name' => 'quz'], function() {
                     });
                 });
             });
             $link = $r->link('foz.baz.quz', ['var1' => 'hello']);
             expect($link)->toBe('/foo/bar/hello');
         });
-        it("persists persisted parameters in a dispatching context", function () {
+        it("persists persisted parameters in a dispatching context", function() {
 
 
             $r = $this->router;
-            $r->group('{locale:en|fr}', ['persist' => 'locale'], function ($r) {
+            $r->group('{locale:en|fr}', ['persist' => 'locale'], function($r) {
 
-                $r->bind('{controller}/{action}[/{id}]', ['name' => 'controller'], function () {
+                $r->bind('{controller}/{action}[/{id}]', ['name' => 'controller'], function() {
                 });
             });
             $r->route('fr/post/index');
@@ -192,13 +192,13 @@ 
             ]);
             expect($link)->toBe('/en/post/view/5');
         });
-        it("overrides persisted parameters in a dispatching context", function () {
+        it("overrides persisted parameters in a dispatching context", function() {
 
 
             $r = $this->router;
-            $r->group('{locale:en|fr}', ['persist' => 'locale'], function ($r) {
+            $r->group('{locale:en|fr}', ['persist' => 'locale'], function($r) {
 
-                $r->bind('{controller}/{action}[/{id}]', ['name' => 'controller'], function () {
+                $r->bind('{controller}/{action}[/{id}]', ['name' => 'controller'], function() {
                 });
             });
             $r->route('fr/post/index');
@@ -210,24 +210,24 @@ 
             ]);
             expect($link)->toBe('/en/post/view/5');
         });
-        it("throws an exception when no route is found for a specified name", function () {
+        it("throws an exception when no route is found for a specified name", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $this->router->link('not.binded.yet', []);
             };
             expect($closure)->toThrow(new RouterException("No binded route defined for `'not.binded.yet'`, bind it first with `bind()`."));
         });
     });
-    describe("->route()", function () {
+    describe("->route()", function() {
 
 
-        it("routes on a simple route", function () {
+        it("routes on a simple route", function() {
 
 
             $r = $this->router;
-            $r->bind('foo/bar', function () {
+            $r->bind('foo/bar', function() {
             });
             $route = $r->route('foo/bar', 'GET');
             expect($this->export($route->request))->toEqual([
@@ -242,11 +242,11 @@ 
                 expect($e->getMessage())->toBe("No route found for `*:*:GET:/bar/foo`.");
             }
         });
-        it("routes on a named route", function () {
+        it("routes on a named route", function() {
 
 
             $r = $this->router;
-            $r->bind('foo/bar', ['name' => 'foo'], function () {
+            $r->bind('foo/bar', ['name' => 'foo'], function() {
             });
             $route = $r->route('foo/bar', 'GET');
             expect($route->name)->toBe('foo');
@@ -256,11 +256,11 @@ 
                 expect($e->getMessage())->toBe("No route found for `*:*:GET:/bar/foo`.");
             }
         });
-        it("supports empty as index route", function () {
+        it("supports empty as index route", function() {
 
 
             $r = $this->router;
-            $r->bind('', function () {
+            $r->bind('', function() {
             });
             $route = $r->route('', 'GET');
             expect($this->export($route->request))->toEqual([
@@ -270,11 +270,11 @@ 
                 'path'   => ''
             ]);
         });
-        it("supports a slash as indes route", function () {
+        it("supports a slash as indes route", function() {
 
 
             $r = $this->router;
-            $r->bind('/', function () {
+            $r->bind('/', function() {
             });
             $route = $r->route('', 'GET');
             expect($this->export($route->request))->toEqual([
@@ -284,11 +284,11 @@ 
                 'path'   => ''
             ]);
         });
-        it("supports route variables", function () {
+        it("supports route variables", function() {
 
 
             $r = $this->router;
-            $r->get('foo/{param}', function () {
+            $r->get('foo/{param}', function() {
             });
             $route = $r->route('foo/bar', 'GET');
             expect($route->params)->toBe(['param' => 'bar']);
@@ -298,11 +298,11 @@ 
                 expect($e->getMessage())->toBe("No route found for `*:*:GET:/bar/foo`.");
             }
         });
-        it("supports constrained route variables", function () {
+        it("supports constrained route variables", function() {
 
 
             $r = $this->router;
-            $r->get('foo/{var1:\d+}', function () {
+            $r->get('foo/{var1:\d+}', function() {
             });
             $route = $r->route('foo/25', 'GET');
             expect($route->params)->toBe(['var1' => '25']);
@@ -312,22 +312,22 @@ 
                 expect($e->getMessage())->toBe("No route found for `*:*:GET:/foo/bar`.");
             }
         });
-        it("supports optional segments with variables", function () {
+        it("supports optional segments with variables", function() {
 
 
             $r = $this->router;
-            $r->get('foo[/{var1}]', function () {
+            $r->get('foo[/{var1}]', function() {
             });
             $route = $r->route('foo', 'GET');
             expect($route->params)->toBe(['var1' => null]);
             $route = $r->route('foo/bar', 'GET');
             expect($route->params)->toBe(['var1' => 'bar']);
         });
-        it("supports repeatable segments", function () {
+        it("supports repeatable segments", function() {
 
 
             $r = $this->router;
-            $r->get('foo[/:{var1}]*[/bar[/:{var2}]*]', function () {
+            $r->get('foo[/:{var1}]*[/bar[/:{var2}]*]', function() {
             });
             $route = $r->route('foo', 'GET');
             expect($route->params)->toBe([
@@ -345,11 +345,11 @@ 
                 'var2' => ['fuz']
             ]);
         });
-        it("supports optional segments with custom variable regex", function () {
+        it("supports optional segments with custom variable regex", function() {
 
 
             $r = $this->router;
-            $r->get('foo[/{var1:\d+}]', function () {
+            $r->get('foo[/{var1:\d+}]', function() {
             });
             $route = $r->route('foo', 'GET');
             expect($route->params)->toBe(['var1' => null]);
@@ -361,7 +361,7 @@ 
                 expect($e->getMessage())->toBe("No route found for `*:*:GET:/foo/baz`.");
             }
         });
-        it("supports multiple optional segments", function () {
+        it("supports multiple optional segments", function() {
 
 
             $patterns = [
@@ -370,7 +370,7 @@ 
             $r = $this->router;
 
             foreach ($patterns as $pattern) {
-                $r->get($pattern, function () {
+                $r->get($pattern, function() {
                 });
                 $route = $r->route('foo', 'GET');
                 expect($route->params)->toBe(['var1' => 'foo', 'var2' => null]);
@@ -379,13 +379,13 @@ 
                 $r->clear();
             };
         });
-        it("supports host variables", function () {
+        it("supports host variables", function() {
 
 
             $r = $this->router;
-            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.bar'], function () {
+            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.bar'], function() {
             });
-            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.baz'], function () {
+            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.baz'], function() {
             });
             $route = $r->route('foo/25', 'GET', 'foo.biz.bar');
             expect($route->Host()->getPattern())->toBe('foo.{domain}.bar');
@@ -394,27 +394,27 @@ 
             expect($route->Host()->getPattern())->toBe('foo.{domain}.baz');
             expect($route->params)->toBe(['domain' => 'buz', 'var1' => '50']);
         });
-        it("supports constrained host variables", function () {
+        it("supports constrained host variables", function() {
 
 
             $r = $this->router;
-            $r->get('foo/{var1:\d+}', ['host' => '{subdomain:foo}.{domain}.bar'], function () {
+            $r->get('foo/{var1:\d+}', ['host' => '{subdomain:foo}.{domain}.bar'], function() {
             });
             $route = $r->route('foo/25', 'GET', 'foo.biz.bar');
-            expect($route->params)->toBe([ 'subdomain' => 'foo', 'domain' => 'biz', 'var1' => '25']);
+            expect($route->params)->toBe(['subdomain' => 'foo', 'domain' => 'biz', 'var1' => '25']);
             try {
                 $route = $r->route('foo/bar', 'GET', 'foo.biz.bar');
             } catch (RouteNotFoundException $e) {
                 expect($e->getMessage())->toBe("No route found for `*:foo.biz.bar:GET:/foo/bar`.");
             }
         });
-        it("supports absolute URL", function () {
+        it("supports absolute URL", function() {
 
 
             $r = $this->router;
-            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.bar'], function () {
+            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.bar'], function() {
             });
-            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.baz'], function () {
+            $r->get('foo/{var1:\d+}', ['host' => 'foo.{domain}.baz'], function() {
             });
             $route = $r->route('http://foo.biz.bar/foo/25', 'GET');
             expect($route->Host()->getPattern())->toBe('foo.{domain}.bar');
@@ -423,23 +423,23 @@ 
             expect($route->Host()->getPattern())->toBe('foo.{domain}.baz');
             expect($route->params)->toBe(['domain' => 'buz', 'var1' => '50']);
         });
-        it("supports RESTful routes", function () {
+        it("supports RESTful routes", function() {
 
 
             $r = $this->router;
-            $r->get('foo/bar', function () {
+            $r->get('foo/bar', function() {
             });
-            $r->head('foo/bar', function () {
+            $r->head('foo/bar', function() {
             });
-            $r->post('foo/bar', function () {
+            $r->post('foo/bar', function() {
             });
-            $r->put('foo/bar', function () {
+            $r->put('foo/bar', function() {
             });
-            $r->patch('foo/bar', function () {
+            $r->patch('foo/bar', function() {
             });
-            $r->delete('foo/bar', function () {
+            $r->delete('foo/bar', function() {
             });
-            $r->options('foo/bar', function () {
+            $r->options('foo/bar', function() {
             });
             $methods = ['OPTIONS', 'DELETE', 'PATCH', 'PUT', 'POST', 'HEAD', 'GET'];
             $route = $r->route('foo/bar', 'GET');
@@ -457,11 +457,11 @@ 
             $route = $r->route('foo/bar', 'OPTIONS');
             expect($route->Methods())->toBe($methods);
         });
-        it("matches relationships based routes", function () {
+        it("matches relationships based routes", function() {
 
 
             $r = $this->router;
-            $r->get('[/{relations:[^/]+/[^/:][^/]*}]*/comment[/{id:[^/:][^/]*}][/:{action}]', function () {
+            $r->get('[/{relations:[^/]+/[^/:][^/]*}]*/comment[/{id:[^/:][^/]*}][/:{action}]', function() {
             });
             $route = $r->route('blog/1/post/22/comment/:show', 'GET');
             expect($route->params)->toBe([
@@ -473,35 +473,35 @@ 
                 'action' => 'show'
             ]);
         });
-        it("dispatches HEAD requests on matching GET routes if the HEAD routes are missing", function () {
+        it("dispatches HEAD requests on matching GET routes if the HEAD routes are missing", function() {
 
 
             $r = $this->router;
-            $r->get('foo/bar', function () {
+            $r->get('foo/bar', function() {
                 return 'GET';
             });
             $route = $r->route('foo/bar', 'HEAD');
             expect($route->Methods())->toBe(['GET']);
         });
-        it("dispatches HEAD requests on matching HEAD routes", function () {
+        it("dispatches HEAD requests on matching HEAD routes", function() {
 
 
             $r = $this->router;
 
-            $r->head('foo/bar', function () {
+            $r->head('foo/bar', function() {
                 return 'HEAD';
             });
-            $r->get('foo/bar', function () {
+            $r->get('foo/bar', function() {
                 return 'GET';
             });
             $route = $r->route('foo/bar', 'HEAD');
             expect($route->Methods())->toBe(['GET', 'HEAD']);
         });
-        it("supports requests as a list of arguments", function () {
+        it("supports requests as a list of arguments", function() {
 
 
             $r = $this->router;
-            $r->bind('foo/bar', function () {
+            $r->bind('foo/bar', function() {
             });
             $route = $r->route('foo/bar', 'GET');
             expect($this->export($route->request))->toEqual([
@@ -511,10 +511,10 @@ 
                 'path'   => 'foo/bar'
             ]);
         });
-        it("supports requests as an object", function () {
+        it("supports requests as an object", function() {
 
             $r = $this->router;
-            $r->bind('foo/bar', function () {
+            $r->bind('foo/bar', function() {
             });
             $request = Double::instance(['implements' => ServerRequestInterface::class]);
             $uri = Double::instance(['implements' => UriInterface::class]);
@@ -527,11 +527,11 @@ 
             $route = $r->route($request, 'GET');
             expect($route->request)->toBe($request);
         });
-        it("supports requests as an array", function () {
+        it("supports requests as an array", function() {
 
 
             $r = $this->router;
-            $r->bind('foo/bar', function () {
+            $r->bind('foo/bar', function() {
             });
             $route = $r->route(['path' => 'foo/bar'], 'GET');
             expect($this->export($route->request))->toEqual([
@@ -542,59 +542,59 @@ 
             ]);
         });
     });
-    describe("->group()", function () {
+    describe("->group()", function() {
 
 
-        it("supports nested named route", function () {
+        it("supports nested named route", function() {
 
 
             $r = $this->router;
-            $r->group('foo', ['name' => 'foz'], function ($r) use (&$route) {
+            $r->group('foo', ['name' => 'foz'], function($r) use (&$route) {
 
-                $r->group('bar', ['name' => 'baz'], function ($r) use (&$route) {
+                $r->group('bar', ['name' => 'baz'], function($r) use (&$route) {
 
-                    $route = $r->bind('{var1}', ['name' => 'quz'], function () {
+                    $route = $r->bind('{var1}', ['name' => 'quz'], function() {
                     });
                 });
             });
             expect(isset($r['foz.baz.quz']))->toBe(true);
             expect($r['foz.baz.quz'])->toBe($route);
         });
-        it("returns a working scope", function () {
+        it("returns a working scope", function() {
 
 
             $r = $this->router;
-            $route = $r->group('foo', ['name' => 'foz'], function () {
-            })->group('bar', ['name' => 'baz'], function () {
-            })->bind('{var1}', ['name' => 'quz'], function () {
+            $route = $r->group('foo', ['name' => 'foz'], function() {
+            })->group('bar', ['name' => 'baz'], function() {
+            })->bind('{var1}', ['name' => 'quz'], function() {
             });
             expect(isset($r['foz.baz.quz']))->toBe(true);
             expect($r['foz.baz.quz'])->toBe($route);
         });
-        context("with a prefix contraint", function () {
+        context("with a prefix contraint", function() {
 
 
-            beforeEach(function () {
+            beforeEach(function() {
 
 
                 $r = $this->router;
-                $r->group('foo', function ($r) {
+                $r->group('foo', function($r) {
 
-                    $r->bind('{var1}', function () {
+                    $r->bind('{var1}', function() {
                     });
                 });
             });
-            it("respects url's prefix constraint", function () {
+            it("respects url's prefix constraint", function() {
 
 
                 $r = $this->router;
                 $route = $r->route('foo/bar');
                 expect($route->params)->toBe(['var1'   => 'bar']);
             });
-            it("throws an exception when the prefix does not match", function () {
+            it("throws an exception when the prefix does not match", function() {
 
 
-                $closure = function () {
+                $closure = function() {
 
                     $r = $this->router;
                     $route = $r->route('bar/foo', 'GET');
@@ -602,30 +602,30 @@ 
                 expect($closure)->toThrow(new RouteNotFoundException("No route found for `*:*:GET:/bar/foo`."));
             });
         });
-        context("with a prefix contraint and an optional parameter", function () {
+        context("with a prefix contraint and an optional parameter", function() {
 
 
-            beforeEach(function () {
+            beforeEach(function() {
 
 
                 $r = $this->router;
-                $r->group('foo', function ($r) {
+                $r->group('foo', function($r) {
 
-                    $r->bind('[{var1}]', function () {
+                    $r->bind('[{var1}]', function() {
                     });
                 });
             });
-            it("respects url's prefix constraint", function () {
+            it("respects url's prefix constraint", function() {
 
 
                 $r = $this->router;
                 $route = $r->route('foo');
                 expect($route->params)->toBe(['var1' => null]);
             });
-            it("throws an exception when the prefix does not match", function () {
+            it("throws an exception when the prefix does not match", function() {
 
 
-                $closure = function () {
+                $closure = function() {
 
                     $r = $this->router;
                     $route = $r->route('bar/foo', 'GET');
@@ -633,23 +633,23 @@ 
                 expect($closure)->toThrow(new RouteNotFoundException("No route found for `*:*:GET:/bar/foo`."));
             });
         });
-        context("with a host constraint", function () {
+        context("with a host constraint", function() {
 
 
-            beforeEach(function () {
+            beforeEach(function() {
 
 
                 $r = $this->router;
-                $r->group('foo', ['host' => 'foo.{domain}.bar'], function ($r) {
+                $r->group('foo', ['host' => 'foo.{domain}.bar'], function($r) {
 
-                    $r->group('bar', function ($r) {
+                    $r->group('bar', function($r) {
 
-                        $r->bind('{var1}', function () {
+                        $r->bind('{var1}', function() {
                         });
                     });
                 });
             });
-            it("respects url's host constraint", function () {
+            it("respects url's host constraint", function() {
 
 
                 $r = $this->router;
@@ -659,10 +659,10 @@ 
                     'var1'   => 'baz'
                 ]);
             });
-            it("throws an exception when the host does not match", function () {
+            it("throws an exception when the host does not match", function() {
 
 
-                $closure = function () {
+                $closure = function() {
 
                     $r = $this->router;
                     $route = $r->route('http://bar.hello.foo/foo/bar/baz', 'GET');
@@ -670,23 +670,23 @@ 
                 expect($closure)->toThrow(new RouteNotFoundException("No route found for `http:bar.hello.foo:GET:/foo/bar/baz`."));
             });
         });
-        context("with a scheme constraint", function () {
+        context("with a scheme constraint", function() {
 
 
-            beforeEach(function () {
+            beforeEach(function() {
 
 
                 $r = $this->router;
-                $r->group('foo', ['scheme' => 'http'], function ($r) {
+                $r->group('foo', ['scheme' => 'http'], function($r) {
 
-                    $r->group('bar', function ($r) {
+                    $r->group('bar', function($r) {
 
-                        $r->bind('{var1}', function () {
+                        $r->bind('{var1}', function() {
                         });
                     });
                 });
             });
-            it("respects url's scheme constraint", function () {
+            it("respects url's scheme constraint", function() {
 
 
                 $r = $this->router;
@@ -695,10 +695,10 @@ 
                     'var1'   => 'baz'
                 ]);
             });
-            it("throws an exception when route is not found", function () {
+            it("throws an exception when route is not found", function() {
 
 
-                $closure = function () {
+                $closure = function() {
 
                     $r = $this->router;
                     $route = $r->route('https://domain.com/foo/bar/baz', 'GET');
@@ -706,25 +706,25 @@ 
                 expect($closure)->toThrow(new RouteNotFoundException("No route found for `https:domain.com:GET:/foo/bar/baz`."));
             });
         });
-        it("concats namespace values", function () {
+        it("concats namespace values", function() {
 
 
             $r = $this->router;
-            $r->group('foo', ['namespace' => 'My'], function ($r) {
+            $r->group('foo', ['namespace' => 'My'], function($r) {
 
-                $r->group('bar', ['namespace' => 'Name'], function ($r) {
+                $r->group('bar', ['namespace' => 'Name'], function($r) {
 
-                    $r->bind('{var1}', ['namespace' => 'Space'], function () {
+                    $r->bind('{var1}', ['namespace' => 'Space'], function() {
                     });
                 });
             });
             $route = $r->route('foo/bar/baz', 'GET');
             expect($route->namespace)->toBe('My\Name\Space\\');
         });
-        it("throws an exception when the handler is not a closure", function () {
+        it("throws an exception when the handler is not a closure", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $r = $this->router;
                 $r->group('foo', 'substr');
@@ -732,24 +732,24 @@ 
             expect($closure)->toThrow(new RouterException("The handler needs to be an instance of `Closure` or implements the `__invoke()` magic method."));
         });
     });
-    describe("->middleware()", function () {
+    describe("->middleware()", function() {
 
 
-        it("returns global middleware", function () {
+        it("returns global middleware", function() {
 
 
             $r = $this->router;
-            $mw1 = function ($request, $response, $next) {
+            $mw1 = function($request, $response, $next) {
 
-                return '1' . $next() . '1';
+                return '1'.$next().'1';
             };
-            $mw2 = function ($request, $response, $next) {
+            $mw2 = function($request, $response, $next) {
 
-                return '2' . $next() . '2';
+                return '2'.$next().'2';
             };
             $r->apply($mw1, $mw2);
 
-            $next = function () {
+            $next = function() {
                 return '';
             };
             $generator = $r->middleware();
@@ -760,21 +760,21 @@ 
             expect($actual1(null, null, $next))->toBe('11');
         });
     });
-    describe("->apply()", function () {
+    describe("->apply()", function() {
 
 
-        it("applies middlewares globally", function () {
+        it("applies middlewares globally", function() {
 
 
             $r = $this->router;
-            $r->apply(function ($request, $response, $next) {
+            $r->apply(function($request, $response, $next) {
 
-                return '1' . $next() . '1';
-            })->apply(function ($request, $response, $next) {
+                return '1'.$next().'1';
+            })->apply(function($request, $response, $next) {
 
-                return '2' . $next() . '2';
+                return '2'.$next().'2';
             });
-            $route = $r->bind('/foo/bar', function ($route) {
+            $route = $r->bind('/foo/bar', function($route) {
 
                 return 'A';
             });
@@ -782,27 +782,27 @@ 
             $actual = $route->dispatch();
             expect($actual)->toBe('21A12');
         });
-        it("applies middlewares globally and per groups", function () {
+        it("applies middlewares globally and per groups", function() {
 
 
             $r = $this->router;
-            $r->apply(function ($request, $response, $next) {
+            $r->apply(function($request, $response, $next) {
 
-                return '1' . $next() . '1';
+                return '1'.$next().'1';
             });
-            $r->bind('foo/{foo}', ['name' => 'foo'], function () {
+            $r->bind('foo/{foo}', ['name' => 'foo'], function() {
 
                 return 'A';
             });
-            $r->group('bar', function ($r) {
+            $r->group('bar', function($r) {
 
-                $r->bind('{bar}', ['name' => 'bar'], function () {
+                $r->bind('{bar}', ['name' => 'bar'], function() {
 
                     return 'A';
                 });
-            })->apply(function ($request, $response, $next) {
+            })->apply(function($request, $response, $next) {
 
-                return '2' . $next() . '2';
+                return '2'.$next().'2';
             });
             $route = $r->route('foo/foo');
             $actual = $route->dispatch();
@@ -812,15 +812,15 @@ 
             expect($actual)->toBe('21A12');
         });
     });
-    describe("->strategy()", function () {
+    describe("->strategy()", function() {
 
 
-        it("sets a strategy", function () {
+        it("sets a strategy", function() {
 
             $r = $this->router;
-            $mystrategy = function ($router) {
+            $mystrategy = function($router) {
 
-                $router->bind('foo/bar', function () {
+                $router->bind('foo/bar', function() {
 
                     return 'Hello World!';
                 });
@@ -831,11 +831,11 @@ 
             $route = $r->route('foo/bar');
             expect($route->Pattern())->toBe('/foo/bar');
         });
-        it("unsets a strategy", function () {
+        it("unsets a strategy", function() {
 
             $r = $this->router;
 
-            $mystrategy = function () {
+            $mystrategy = function() {
             };
             $r->setStrategy('mystrategy', $mystrategy);
             expect($r->strategy('mystrategy'))->toBe($mystrategy);