spiral / framework

src/Http/tests/PipelineTest.php

Braces +4 added lines, -2 removed lines

@@ -60,7 +60,8 @@ 
 
     public function testMiddlewareProcessingEventShouldBeDispatched(): void
     {
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface
+        {
             public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
             {
                 return (new ResponseFactory(new HttpConfig(['headers' => []])))->createResponse(200);
@@ -89,7 +90,8 @@ 
         $this->container->getBinder('http')
             ->bind(ServerRequestInterface::class, static fn(CurrentRequest $cr): ?ServerRequestInterface => $cr->get());
 
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface
+        {
             public function process(
                 ServerRequestInterface $request,
                 RequestHandlerInterface $handler,

Spacing +4 added lines, -4 removed lines

@@ -33,7 +33,7 @@ 
 
         self::assertSame(200, $response->getStatusCode());
         self::assertSame('OK', $response->getReasonPhrase());
-        self::assertSame('response', (string) $response->getBody());
+        self::assertSame('response', (string)$response->getBody());
     }
 
     public function testHandle(): void
@@ -46,7 +46,7 @@ 
 
         self::assertSame(200, $response->getStatusCode());
         self::assertSame('OK', $response->getReasonPhrase());
-        self::assertSame('response', (string) $response->getBody());
+        self::assertSame('response', (string)$response->getBody());
     }
 
     public function testHandleException(): void
@@ -59,7 +59,7 @@ 
 
     public function testMiddlewareProcessingEventShouldBeDispatched(): void
     {
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface{
             public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
             {
                 return (new ResponseFactory(new HttpConfig(['headers' => []])))->createResponse(200);
@@ -88,7 +88,7 @@ 
         $this->container->getBinder('http')
             ->bind(ServerRequestInterface::class, static fn(CurrentRequest $cr): ?ServerRequestInterface => $cr->get());
 
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface{
             public function process(
                 ServerRequestInterface $request,
                 RequestHandlerInterface $handler,

src/Distribution/src/Bootloader/DistributionBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
             static function (DistributionConfig $config): DistributionInterface {
                 $manager = new Manager($config->getDefaultDriver());
 
-                foreach ($config->getResolvers() as $name => $resolver) {
+                foreach ($config->getResolvers() as $name => $resolver){
                     $manager->add($name, $resolver);
                 }
 

Braces +2 added lines, -1 removed lines

@@ -48,7 +48,8 @@
             static function (DistributionConfig $config): DistributionInterface {
                 $manager = new Manager($config->getDefaultDriver());
 
-                foreach ($config->getResolvers() as $name => $resolver) {
+                foreach ($config->getResolvers() as $name => $resolver)
+                {
                     $manager->add($name, $resolver);
                 }
 

src/Router/src/Router.php

Braces +49 added lines, -22 removed lines

@@ -68,13 +68,17 @@ 
         return $this->tracer->trace(
             name: 'Routing',
             callback: function (SpanInterface $span) use ($request): ResponseInterface {
-                try {
+                try
+                {
                     $route = $this->matchRoute($request, $routeName);
-                } catch (RouteException $e) {
+                }
+                catch (RouteException $e)
+                {
                     throw new RouterException('Invalid route definition', $e->getCode(), $e);
                 }
 
-                if ($route === null) {
+                if ($route === null)
+                {
                     $this->eventDispatcher?->dispatch(new RouteNotFound($request));
                     throw new RouteNotFoundException($request->getUri());
                 }
@@ -109,7 +113,8 @@ 
 
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name]))
+        {
             return $this->routes[$name];
         }
 
@@ -118,7 +123,8 @@ 
 
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default))
+        {
             return $this->routes + [null => $this->default];
         }
 
@@ -127,9 +133,12 @@ 
 
     public function uri(string $route, iterable $parameters = []): UriInterface
     {
-        try {
+        try
+        {
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException) {
+        }
+        catch (UndefinedRouteException)
+        {
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -141,9 +150,11 @@ 
         /** @var GroupRegistry $groups */
         $groups = $this->container->get(GroupRegistry::class);
 
-        foreach ($routes->getCollection() as $name => $configurator) {
+        foreach ($routes->getCollection() as $name => $configurator)
+        {
             $target = $configurator->target;
-            if ($configurator->core !== null && $target instanceof AbstractTarget) {
+            if ($configurator->core !== null && $target instanceof AbstractTarget)
+            {
                 $target = $target->withCore($configurator->core);
             }
 
@@ -152,21 +163,25 @@ 
                 : \ltrim($configurator->pattern, '/');
             $route = new Route($pattern, $target, $configurator->defaults);
 
-            if ($configurator->middleware !== null) {
+            if ($configurator->middleware !== null)
+            {
                 $route = $route->withMiddleware(...$configurator->middleware);
             }
 
-            if ($configurator->methods !== null) {
+            if ($configurator->methods !== null)
+            {
                 $route = $route->withVerbs(...$configurator->methods);
             }
 
-            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME)
+            {
                 $group = $groups->getGroup($configurator->group ?? $groups->getDefaultGroup());
                 $group->setPrefix($configurator->prefix);
                 $group->addRoute($name, $route);
             }
 
-            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME)
+            {
                 $this->setDefault($route);
             }
         }
@@ -177,17 +192,20 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, ?string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route)
+        {
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null)
+            {
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null)
+        {
             return $this->default->match($request);
         }
 
@@ -200,14 +218,18 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer())
+        {
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
 
-        try {
+        try
+        {
             $uriHandler = $route->getUriHandler();
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
             $uriHandler = $this->uriHandler;
         }
 
@@ -239,11 +261,16 @@ 
         /**
          * @var Matches $matches
          */
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name']))
+        {
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        }
+        elseif ($this->default !== null)
+        {
             $routeObject = $this->default;
-        } else {
+        }
+        else
+        {
             throw new UndefinedRouteException(\sprintf('Unable to locate route candidate for `%s`', $route));
         }
 

Spacing +26 added lines, -26 removed lines

@@ -52,9 +52,9 @@ 
         private readonly ContainerInterface $container,
         private readonly ?EventDispatcherInterface $eventDispatcher = null,
         ?TracerInterface $tracer = null,
-    ) {
+    ){
         $this->tracer = $tracer ?? new NullTracer();
-        $this->basePath = '/' . \ltrim($basePath, '/');
+        $this->basePath = '/'.\ltrim($basePath, '/');
     }
 
     /**
@@ -68,19 +68,19 @@ 
         return $this->tracer->trace(
             name: 'Routing',
             callback: function (SpanInterface $span) use ($request): ResponseInterface {
-                try {
+                try{
                     $route = $this->matchRoute($request, $routeName);
-                } catch (RouteException $e) {
+                }catch (RouteException $e){
                     throw new RouterException('Invalid route definition', $e->getCode(), $e);
                 }
 
-                if ($route === null) {
+                if ($route === null){
                     $this->eventDispatcher?->dispatch(new RouteNotFound($request));
                     throw new RouteNotFoundException($request->getUri());
                 }
 
                 $span
-                    ->setAttribute('request.uri', (string) $request->getUri())
+                    ->setAttribute('request.uri', (string)$request->getUri())
                     ->setAttribute('route.name', $routeName)
                     ->setAttribute('route.matches', $route->getMatches() ?? []);
 
@@ -109,7 +109,7 @@ 
 
     public function getRoute(string $name): RouteInterface
     {
-        if (isset($this->routes[$name])) {
+        if (isset($this->routes[$name])){
             return $this->routes[$name];
         }
 
@@ -118,7 +118,7 @@ 
 
     public function getRoutes(): array
     {
-        if (!empty($this->default)) {
+        if (!empty($this->default)){
             return $this->routes + [null => $this->default];
         }
 
@@ -127,9 +127,9 @@ 
 
     public function uri(string $route, iterable $parameters = []): UriInterface
     {
-        try {
+        try{
             return $this->getRoute($route)->uri($parameters);
-        } catch (UndefinedRouteException) {
+        }catch (UndefinedRouteException){
             //In some cases route name can be provided as controller:action pair, we can try to
             //generate such route automatically based on our default/fallback route
             return $this->castRoute($route)->uri($parameters);
@@ -141,9 +141,9 @@ 
         /** @var GroupRegistry $groups */
         $groups = $this->container->get(GroupRegistry::class);
 
-        foreach ($routes->getCollection() as $name => $configurator) {
+        foreach ($routes->getCollection() as $name => $configurator){
             $target = $configurator->target;
-            if ($configurator->core !== null && $target instanceof AbstractTarget) {
+            if ($configurator->core !== null && $target instanceof AbstractTarget){
                 $target = $target->withCore($configurator->core);
             }
 
@@ -152,21 +152,21 @@ 
                 : \ltrim($configurator->pattern, '/');
             $route = new Route($pattern, $target, $configurator->defaults);
 
-            if ($configurator->middleware !== null) {
+            if ($configurator->middleware !== null){
                 $route = $route->withMiddleware(...$configurator->middleware);
             }
 
-            if ($configurator->methods !== null) {
+            if ($configurator->methods !== null){
                 $route = $route->withVerbs(...$configurator->methods);
             }
 
-            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if (!isset($this->routes[$name]) && $name !== RoutingConfigurator::DEFAULT_ROUTE_NAME){
                 $group = $groups->getGroup($configurator->group ?? $groups->getDefaultGroup());
                 $group->setPrefix($configurator->prefix);
                 $group->addRoute($name, $route);
             }
 
-            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME) {
+            if ($name === RoutingConfigurator::DEFAULT_ROUTE_NAME){
                 $this->setDefault($route);
             }
         }
@@ -177,17 +177,17 @@ 
      */
     protected function matchRoute(ServerRequestInterface $request, ?string &$routeName = null): ?RouteInterface
     {
-        foreach ($this->routes as $name => $route) {
+        foreach ($this->routes as $name => $route){
             // Matched route will return new route instance with matched parameters
             $matched = $route->match($request);
 
-            if ($matched !== null) {
+            if ($matched !== null){
                 $routeName = $name;
                 return $matched;
             }
         }
 
-        if ($this->default !== null) {
+        if ($this->default !== null){
             return $this->default->match($request);
         }
 
@@ -200,14 +200,14 @@ 
      */
     protected function configure(RouteInterface $route): RouteInterface
     {
-        if ($route instanceof ContainerizedInterface && !$route->hasContainer()) {
+        if ($route instanceof ContainerizedInterface && !$route->hasContainer()){
             // isolating route in a given container
             $route = $route->withContainer($this->container);
         }
 
-        try {
+        try{
             $uriHandler = $route->getUriHandler();
-        } catch (\Throwable) {
+        }catch (\Throwable){
             $uriHandler = $this->uriHandler;
         }
 
@@ -230,7 +230,7 @@ 
                 $route,
                 $matches,
             )
-        ) {
+        ){
             throw new UndefinedRouteException(
                 "Unable to locate route or use default route with 'name/controller:action' pattern",
             );
@@ -239,11 +239,11 @@ 
         /**
          * @var Matches $matches
          */
-        if (!empty($matches['name'])) {
+        if (!empty($matches['name'])){
             $routeObject = $this->getRoute($matches['name']);
-        } elseif ($this->default !== null) {
+        } elseif ($this->default !== null){
             $routeObject = $this->default;
-        } else {
+        }else{
             throw new UndefinedRouteException(\sprintf('Unable to locate route candidate for `%s`', $route));
         }
 

src/Core/tests/ExceptionsTest.php

Spacing +4 added lines, -4 removed lines

@@ -135,9 +135,9 @@ 
     {
         $container = new Container();
 
-        try {
+        try{
             $container->get('invalid');
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             self::assertSame(<<<MARKDOWN
                 Can't resolve `invalid`: undefined class or binding `invalid`.
                 Container trace list:
@@ -166,9 +166,9 @@ 
     {
         $this->expectException(ContainerException::class);
 
-        try {
+        try{
             $container->get(ClassWithUndefinedDependency::class);
-        } catch (ContainerException $e) {
+        }catch (ContainerException $e){
             self::assertSame($message, $e->getMessage());
 
             throw $e;

Braces +10 added lines, -4 removed lines

@@ -135,9 +135,12 @@ 
     {
         $container = new Container();
 
-        try {
+        try
+        {
             $container->get('invalid');
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             self::assertSame(<<<MARKDOWN
                 Can't resolve `invalid`: undefined class or binding `invalid`.
                 Container trace list:
@@ -166,9 +169,12 @@ 
     {
         $this->expectException(ContainerException::class);
 
-        try {
+        try
+        {
             $container->get(ClassWithUndefinedDependency::class);
-        } catch (ContainerException $e) {
+        }
+        catch (ContainerException $e)
+        {
             self::assertSame($message, $e->getMessage());
 
             throw $e;

src/Core/tests/Scope/SideEffectTest.php

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
 
         $logger = $root->runScope(
             new Scope('test'),
-            static fn(?ScopeIndicatorLogger $logger): ?ScopeIndicatorLogger => $logger,
+            static fn(?ScopeIndicatorLogger $logger) : ?ScopeIndicatorLogger => $logger,
         );
 
         self::assertNotNull($logger);
@@ -77,7 +77,7 @@ 
 
         $logger = $root->runScope(
             new Scope('test'),
-            static fn(?ScopeIndicatorLogger $logger): ?ScopeIndicatorLogger => $logger,
+            static fn(?ScopeIndicatorLogger $logger) : ?ScopeIndicatorLogger => $logger,
         );
 
         self::assertNotNull($logger);

src/Core/tests/SingletonsTest.php

Braces +5 added lines, -2 removed lines

@@ -181,7 +181,9 @@ 
     public function testHasShouldReturnTrueWhenSingletonIsAlreadyConstructed(): void
     {
         $container = new Container();
-        $class = new #[Singleton] class {};
+        $class = new #[Singleton] class
+        {
+};
 
         self::assertFalse($container->has($class::class));
 
@@ -194,7 +196,8 @@ 
     {
         return new
         #[Singleton]
-        class {
+        class
+        {
             public string $baz = 'baz';
         };
     }

src/Tokenizer/tests/Bootloader/TokenizerListenerBootloaderTest.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
         $kernel = TestCoreWithTokenizer::create(directories: ['root' => __DIR__], container: $container);
         $kernel->run();
 
-        self::assertContains(\dirname(__DIR__) . '/Fixtures/Bootloader', $container->get(TokenizerConfig::class)->getDirectories());
+        self::assertContains(\dirname(__DIR__).'/Fixtures/Bootloader', $container->get(TokenizerConfig::class)->getDirectories());
     }
 
 }

src/Tokenizer/tests/Attribute/TargetAttributeTest.php

Spacing +2 added lines, -2 removed lines

@@ -20,10 +20,10 @@
     public function testToString(): void
     {
         $attribute = new TargetAttribute('foo');
-        self::assertSame('3dc18b19eed74479a03c069dec2e8724', (string) $attribute);
+        self::assertSame('3dc18b19eed74479a03c069dec2e8724', (string)$attribute);
 
         $attribute = new TargetAttribute('foo', 'bar');
-        self::assertSame('52ec767c53f3898bf6de6f6e88125dc8', (string) $attribute);
+        self::assertSame('52ec767c53f3898bf6de6f6e88125dc8', (string)$attribute);
     }
 
     public function testFilterAttrWithArgs(): void

src/Tokenizer/tests/Attribute/TargetClassTest.php

Spacing +2 added lines, -2 removed lines

@@ -12,9 +12,9 @@
     public function testToString(): void
     {
         $attribute = new TargetClass('foo');
-        self::assertSame('3319d33aad20ad375d27dcd03c879454', (string) $attribute);
+        self::assertSame('3319d33aad20ad375d27dcd03c879454', (string)$attribute);
 
         $attribute = new TargetClass('foo', 'bar');
-        self::assertSame('a95c8a2cfc901290939df93183ce98d6', (string) $attribute);
+        self::assertSame('a95c8a2cfc901290939df93183ce98d6', (string)$attribute);
     }
 }
\ No newline at end of file