burzum / php-router

spec/Suite/Router.spec.php

Spacing +161 added lines, -161 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->basePath())->toBe('');
         });
     });
-    describe("->basePath()", function () {
+    describe("->basePath()", 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->basePath())->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->basePath())->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,42 +99,42 @@ 
                 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 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', [
@@ -143,28 +143,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');
@@ -182,13 +182,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');
@@ -200,24 +200,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 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([
@@ -232,11 +232,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');
@@ -246,11 +246,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([
@@ -260,11 +260,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([
@@ -274,11 +274,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']);
@@ -288,11 +288,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']);
@@ -302,22 +302,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([
@@ -335,11 +335,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]);
@@ -351,7 +351,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 = [
@@ -360,7 +360,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]);
@@ -369,13 +369,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()->pattern())->toBe('foo.{domain}.bar');
@@ -384,27 +384,27 @@ 
             expect($route->Host()->pattern())->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()->pattern())->toBe('foo.{domain}.bar');
@@ -413,23 +413,23 @@ 
             expect($route->Host()->pattern())->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');
@@ -447,11 +447,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([
@@ -463,35 +463,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([
@@ -501,10 +501,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]);
@@ -517,11 +517,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([
@@ -532,59 +532,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');
@@ -592,30 +592,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');
@@ -623,23 +623,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;
@@ -649,10 +649,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');
@@ -660,23 +660,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;
@@ -685,10 +685,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');
@@ -696,25 +696,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');
@@ -722,24 +722,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();
@@ -750,21 +750,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';
             });
@@ -772,27 +772,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();
@@ -802,15 +802,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!';
                 });
@@ -821,11 +821,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);

spec/Suite/Host.spec.php

Spacing +20 added lines, -20 removed lines

@@ -7,13 +7,13 @@ 
 use Lead\Router\Exception\RouterException;
 use Lead\Router\Host;
 
-describe("Host", function () {
+describe("Host", function() {
 
 
-    describe("->scheme()", function () {
+    describe("->scheme()", function() {
 
 
-        it("gets/sets the scheme", function () {
+        it("gets/sets the scheme", function() {
 
 
             $host = new Host();
@@ -22,10 +22,10 @@ 
             expect($host->scheme())->toBe('https');
         });
     });
-    describe("->pattern()", function () {
+    describe("->pattern()", function() {
 
 
-        it("gets/sets the pattern", function () {
+        it("gets/sets the pattern", function() {
 
 
             $host = new Host();
@@ -37,7 +37,7 @@ 
                 'domain'    => false
             ]);
         });
-        it("updates the regex", function () {
+        it("updates the regex", function() {
 
 
             $host = new Host();
@@ -46,7 +46,7 @@ 
             expect($host->setPattern('foo.{domain}.baz'))->toBe($host);
             expect($host->regex())->toBe('foo\\.([^.]+)\\.baz');
         });
-        it("updates the variables", function () {
+        it("updates the variables", function() {
 
 
             $host = new Host();
@@ -60,10 +60,10 @@ 
             ]);
         });
     });
-    describe("->match()", function () {
+    describe("->match()", function() {
 
 
-        it("returns `true` when host & scheme matches", function () {
+        it("returns `true` when host & scheme matches", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => 'https']);
@@ -72,7 +72,7 @@ 
             expect($host->match(['scheme' => 'https', 'host' => 'biz.bar.baz'], $variables))->toBe(false);
             expect($host->match(['scheme' => 'http', 'host' => 'foo.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when host matches with a wildcard as host's scheme", function () {
+        it("returns `true` when host matches with a wildcard as host's scheme", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => 'https']);
@@ -80,7 +80,7 @@ 
             expect($variables)->toBe(['domain' => 'baz']);
             expect($host->match(['scheme' => '*', 'host' => 'biz.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when host matches with a wildcard as request's scheme", function () {
+        it("returns `true` when host matches with a wildcard as request's scheme", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => '*']);
@@ -88,7 +88,7 @@ 
             expect($variables)->toBe(['domain' => 'baz']);
             expect($host->match(['scheme' => 'https', 'host' => 'biz.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when scheme matches with a wildcard as host's pattern", function () {
+        it("returns `true` when scheme matches with a wildcard as host's pattern", function() {
 
 
             $host = new Host(['pattern' => '*', 'scheme' => 'http']);
@@ -96,7 +96,7 @@ 
             expect($variables)->toBe([]);
             expect($host->match(['scheme' => 'https', 'host' => 'foo.baz.bar'], $variables))->toBe(false);
         });
-        it("returns `true` when scheme matches with a wildcard as request's pattern", function () {
+        it("returns `true` when scheme matches with a wildcard as request's pattern", function() {
 
 
             $host = new Host(['pattern' => 'foo.{domain}.bar', 'scheme' => 'http']);
@@ -105,24 +105,24 @@ 
             expect($host->match(['scheme' => 'https', 'host' => '*'], $variables))->toBe(false);
         });
     });
-    describe("->link()", function () {
+    describe("->link()", function() {
 
 
-        it("builds an host link", function () {
+        it("builds an host link", function() {
 
 
             $host = new Host(['pattern' => 'www.{domain}.com', 'scheme' => 'https']);
             $link = $host->link(['domain' => 'example']);
             expect($link)->toBe('https://www.example.com');
         });
-        it("builds a scheme less host link", function () {
+        it("builds a scheme less host link", function() {
 
 
             $host = new Host(['pattern' => 'www.{domain}.com']);
             $link = $host->link(['domain' => 'example']);
             expect($link)->toBe('//www.example.com');
         });
-        it("overrides scheme when passed as parameter", function () {
+        it("overrides scheme when passed as parameter", function() {
 
 
             $host = new Host(['pattern' => 'www.{domain}.com']);
@@ -131,7 +131,7 @@ 
             ]);
             expect($link)->toBe('http://www.example.com');
         });
-        it("processes complex pattern", function () {
+        it("processes complex pattern", function() {
 
 
             $host = new Host(['pattern' => '[{subdomains}.]*{domain}.com', 'scheme' => 'https']);
@@ -145,10 +145,10 @@ 
                 'domain' => 'example'
             ]))->toBe('https://a.b.c.example.com');
         });
-        it("throws an exception when variables are missing", function () {
+        it("throws an exception when variables are missing", function() {
 
 
-            $closure = function () {
+            $closure = function() {
 
                 $host = new Host(['pattern' => 'www.{domain}.com']);
                 $link = $host->link();

src/Host.php

Spacing +3 added lines, -3 removed lines

@@ -219,7 +219,7 @@ 
             return false;
         }
 
-        if (!preg_match('~^' . $this->regex() . '$~', $host, $matches)) {
+        if (!preg_match('~^'.$this->regex().'$~', $host, $matches)) {
             $hostVariables = null;
             return false;
         }
@@ -245,9 +245,9 @@ 
             $options['host'] = $this->_link($this->token(), $params);
         }
 
-        $scheme = $options['scheme'] !== '*' ? $options['scheme'] . '://' : '//';
+        $scheme = $options['scheme'] !== '*' ? $options['scheme'].'://' : '//';
 
-        return $scheme . $options['host'];
+        return $scheme.$options['host'];
     }
 
     /**