divineniiquaye / flight-routing

Tests/RouterIntegrationTest.php

Spacing +12 added lines, -12 removed lines

@@ -85,7 +85,7 @@ 
 
     public function createInvalidResponseFactory(): callable
     {
-        return function (): void {
+        return function(): void {
             Assert::fail('Response generated when it should not have been');
         };
     }
@@ -162,7 +162,7 @@ 
 
         $finalHandler = $this->prophesize(RequestHandlerInterface::class);
         $finalHandler
-            ->handle(Argument::that(function (ServerRequestInterface $request) use ($method) {
+            ->handle(Argument::that(function(ServerRequestInterface $request) use ($method) {
                 Assert::assertSame($method, $request->getMethod());
 
                 $routeResult = $request->getAttribute(RouteResults::class);
@@ -200,7 +200,7 @@ 
     public function testExplicitWithoutCollector(string $method, array $routes): void
     {
         $router           = $this->getRouter();
-        [$serverRequest,] = $this->psrServerResponseFactory();
+        [$serverRequest, ] = $this->psrServerResponseFactory();
 
         $finalHandler = $this->prophesize(RequestHandlerInterface::class);
         $finalHandler->handle(Argument::any())->shouldNotBeCalled();
@@ -247,7 +247,7 @@ 
     public function testWithoutImplicitMiddleware(string $requestMethod, array $allowedMethods): void
     {
         $router           = $this->getRouteCollection();
-        [$serverRequest,] = $this->psrServerResponseFactory();
+        [$serverRequest, ] = $this->psrServerResponseFactory();
 
         $finalResponse = $this->prophesize(ResponseInterface::class);
         $finalResponse->withStatus(405)->will([$finalResponse, 'reveal']);
@@ -259,7 +259,7 @@ 
         foreach ($allowedMethods as $routeMethod) {
             $route = (new Route([$routeMethod], '/api/v1/me', $finalHandler->reveal()))
                 ->addMiddleware(
-                    function (ServerRequestInterface $request, RequestHandlerInterface $handler) {
+                    function(ServerRequestInterface $request, RequestHandlerInterface $handler) {
                         return $handler->handle($request);
                     }
                 );
@@ -329,11 +329,11 @@ 
             ->willReturn($finalResponse)
             ->shouldBeCalledTimes(1);
 
-        $router->map([$requestMethod], $routePath, function () use ($finalResponse) {
+        $router->map([$requestMethod], $routePath, function() use ($finalResponse) {
             return $finalResponse;
         })
         ->addMiddleware(
-            function (ServerRequestInterface $req, RequestHandlerInterface $handle) use ($middleware, $requestMethod) {
+            function(ServerRequestInterface $req, RequestHandlerInterface $handle) use ($middleware, $requestMethod) {
                 Assert::assertEquals($requestMethod, $req->getMethod());
                 Assert::assertInstanceOf(Next::class, $handle);
 
@@ -374,7 +374,7 @@ 
         array $asserts = []
     ): void {
         $router           = $this->getRouteCollection();
-        [$serverRequest,] = $this->psrServerResponseFactory();
+        [$serverRequest, ] = $this->psrServerResponseFactory();
 
         $router->map([$requestMethod], $routePath, $controller)
         ->setName($routeOptions['name'] ?? null)
@@ -478,10 +478,10 @@ 
         $router = $this->getRouteCollection();
         $router->setNamespace('Flight\\Routing\\Tests\\');
 
-        [$serverRequest,] = $this->psrServerResponseFactory();
+        [$serverRequest, ] = $this->psrServerResponseFactory();
         $path             = $serverRequest->getUri()->withPath('/' . ($asserts['path'] ?? 'group/test'));
 
-        $router->group($groupAttributes, function (RouterProxyInterface $route) use ($asserts): void {
+        $router->group($groupAttributes, function(RouterProxyInterface $route) use ($asserts): void {
             $route->get(
                 'test*<homePageRequestResponse>',
                 isset($asserts['namespace']) ? 'SampleController' : SampleController::class
@@ -515,9 +515,9 @@ 
     public function testImplicitRouteNotFound(): void
     {
         $router           = $this->getRouteCollection();
-        [$serverRequest,] = $this->psrServerResponseFactory();
+        [$serverRequest, ] = $this->psrServerResponseFactory();
 
-        $router->map([HttpMethods::METHOD_GET], '/error', function () {
+        $router->map([HttpMethods::METHOD_GET], '/error', function() {
             return 'This is an error page';
         })->setName('error');
 

Tests/DefaultRouterTest.php

Spacing +21 added lines, -21 removed lines

@@ -43,7 +43,7 @@ 
             '/',
             '/',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return 'Hello World';
             },
             [],
@@ -54,7 +54,7 @@ 
             '/',
             '/',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return '<html><body><h1>Hello World</h1></body></html>';
             },
             [
@@ -71,7 +71,7 @@ 
             '/',
             '/',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return '<?xml version="1.0" encoding="UTF-8"?><route>Hello World</route>';
             },
             [],
@@ -82,7 +82,7 @@ 
             '/',
             '/',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return new Helpers\DumpArrayTest();
             },
             [],
@@ -114,7 +114,7 @@ 
             '/test',
             '/test',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return 'Hello, this is a basic test route';
             },
         ];
@@ -123,7 +123,7 @@ 
             '/test',
             '/test/',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return 'Hello, this is a basic test route';
             },
             [],
@@ -134,7 +134,7 @@ 
             '/test/{home}',
             '/test/cool',
             HttpMethods::METHOD_GET,
-            function (string $home) {
+            function(string $home) {
                 return 'Hello, this is a basic test route on subpage ' . $home;
             },
         ];
@@ -143,7 +143,7 @@ 
             '/test/{home}',
             '/test/cool',
             HttpMethods::METHOD_GET,
-            function (string $home, int $id) {
+            function(string $home, int $id) {
                 return $home . $id;
             },
             ['defaults' => ['id' => 233]],
@@ -154,7 +154,7 @@ 
             '/test[/{home}]',
             '/test',
             HttpMethods::METHOD_GET,
-            function (?string $home) {
+            function(?string $home) {
                 return 'Hello, this is a basic test route on subpage ' . $home;
             },
             [],
@@ -165,7 +165,7 @@ 
             '/test[/{home}]',
             '/test/cool',
             HttpMethods::METHOD_GET,
-            function (?string $home) {
+            function(?string $home) {
                 return 'Hello, this is a basic test route on subpage ' . $home;
             },
             [],
@@ -176,7 +176,7 @@ 
             '//example.com/test',
             '/test',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return 'Hello World';
             },
             ['domain' => 'example.com'],
@@ -186,7 +186,7 @@ 
             '//{id:int}.example.com/test',
             '/test',
             HttpMethods::METHOD_GET,
-            function () {
+            function() {
                 return 'Hello World';
             },
             [
@@ -201,7 +201,7 @@ 
             'https://{id:int}.example.com/{action}',
             '/tests',
             HttpMethods::METHOD_GET,
-            function (string $action) {
+            function(string $action) {
                 return 'Hello World' . $action;
             },
             [
@@ -220,7 +220,7 @@ 
             '/[{action}/[{id}]]',
             '/test/',
             HttpMethods::METHOD_GET,
-            function (?string $action) {
+            function(?string $action) {
                 return $action;
             },
             [],
@@ -231,7 +231,7 @@ 
             '/[{action}/[{id}]]',
             '/test/id',
             HttpMethods::METHOD_GET,
-            function (?string $action) {
+            function(?string $action) {
                 return $action;
             },
             [
@@ -248,7 +248,7 @@ 
             '/user/{id:[0-9-]+}',
             'user/23',
             HttpMethods::METHOD_GET,
-            function (int $id) {
+            function(int $id) {
                 return $id;
             },
         ];
@@ -257,7 +257,7 @@ 
             '/[{lang:[a-z]{2}}/]hello',
             '/hello',
             HttpMethods::METHOD_GET,
-            function (?string $lang) {
+            function(?string $lang) {
                 return $lang;
             },
         ];
@@ -266,7 +266,7 @@ 
             '/[{lang:[a-z]{2}}/]{name}',
             '/en/download',
             HttpMethods::METHOD_GET,
-            function (?string $lang, string $name) {
+            function(?string $lang, string $name) {
                 return $lang . $name;
             },
         ];
@@ -275,7 +275,7 @@ 
             '[{lang:[a-z]{2}}[-{sublang}]/]{name}[/page-{page=<0>}]',
             '/download',
             HttpMethods::METHOD_GET,
-            function (?string $lang, ?string $sublang, string $name, $page) {
+            function(?string $lang, ?string $sublang, string $name, $page) {
                 return $lang . '-' . $sublang . $name . $page;
             },
             [],
@@ -286,7 +286,7 @@ 
             '[{lang:[a-z]{2}}[-{sublang}]/]{name}[/page-{page=<0>}]',
             '/en-us/download',
             HttpMethods::METHOD_GET,
-            function (?string $lang, ?string $sublang, string $name, $page) {
+            function(?string $lang, ?string $sublang, string $name, $page) {
                 return $lang . '-' . $sublang . $name . $page;
             },
             [],
@@ -297,7 +297,7 @@ 
             '[{lang:[a-z]{2}}[-{sublang}]/]{name}[/page-{page=<0>}]',
             '/en-us/download/page-12',
             HttpMethods::METHOD_GET,
-            function (?string $lang, ?string $sublang, string $name, $page) {
+            function(?string $lang, ?string $sublang, string $name, $page) {
                 return $lang . '-' . $sublang . $name . $page;
             },
             [],

src/RouteCollector.php

Spacing +2 added lines, -2 removed lines

@@ -519,7 +519,7 @@ 
     {
         return \array_filter(
             \array_replace($middlewares, $this->getMiddlewaresStack()),
-            function ($middleware) {
+            function($middleware) {
                 return !\in_array($middleware, ['off', 'disable'], true);
             }
         );
@@ -600,7 +600,7 @@ 
     private function checkForDuplicateRoute(string $path, array $methods): void
     {
         $allowed        = [];
-        $matches        = \array_filter($this->getRoutes(), function (Route $route) use ($path, $methods, $allowed) {
+        $matches        = \array_filter($this->getRoutes(), function(Route $route) use ($path, $methods, $allowed) {
             if ($path === $route->getPath()) {
                 foreach ($methods as $method) {
                     if (\in_array($method, $route->getMethods(), true)) {

src/Traits/PathsTrait.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
 
         // Match domain + scheme from pattern...
         if (\preg_match('@^(?:(https?):)?(//[^/]+)@i', $pattern)) {
-            $pattern = \preg_replace_callback('@^(?:(https?):)?(//[^/]+)@i', function ($matches) {
+            $pattern = \preg_replace_callback('@^(?:(https?):)?(//[^/]+)@i', function($matches) {
                 $this->addDomain(isset($matches[1]) ? $matches[0] : $matches[2]);
 
                 return '';

src/Traits/ControllersTrait.php

Spacing +1 added lines, -1 removed lines

@@ -110,7 +110,7 @@
      */
     public function handle(callable $controller, CallableResolverInterface $callableResolver): callable
     {
-        $finalController = function (Request $request, Response $response) use ($controller, $callableResolver) {
+        $finalController = function(Request $request, Response $response) use ($controller, $callableResolver) {
             if (\class_exists(BoundMethod::class)) {
                 return BoundMethod::call(
                     $container = $callableResolver->getContainer(),

src/Services/DefaultFlightRouter.php

Spacing +1 added lines, -1 removed lines

@@ -234,7 +234,7 @@
      */
     private function createDispatcherCallback(): callable
     {
-        return function (RouteInterface $route) {
+        return function(RouteInterface $route) {
             return (new SimpleRouteCompiler())->compile($route);
         };
     }