spiral / framework

src/Csrf/tests/CsrfTest.php

Spacing +9 added lines, -9 removed lines

@@ -37,7 +37,7 @@ 
         $cookies = $this->fetchCookies($response);
 
         self::assertArrayHasKey('csrf-token', $cookies);
-        self::assertSame($cookies['csrf-token'], (string) $response->getBody());
+        self::assertSame($cookies['csrf-token'], (string)$response->getBody());
     }
 
     public function testLengthException(): void
@@ -93,7 +93,7 @@ 
 
         $response = $this->get($core, '/');
         self::assertSame(200, $response->getStatusCode());
-        self::assertSame('all good', (string) $response->getBody());
+        self::assertSame('all good', (string)$response->getBody());
 
         $cookies = $this->fetchCookies($response);
 
@@ -112,7 +112,7 @@ 
         );
 
         self::assertSame(200, $response->getStatusCode());
-        self::assertSame('all good', (string) $response->getBody());
+        self::assertSame('all good', (string)$response->getBody());
     }
 
     public function testHeaderOK(): void
@@ -124,7 +124,7 @@ 
 
         $response = $this->get($core, '/');
         self::assertSame(200, $response->getStatusCode());
-        self::assertSame('all good', (string) $response->getBody());
+        self::assertSame('all good', (string)$response->getBody());
 
         $cookies = $this->fetchCookies($response);
 
@@ -143,7 +143,7 @@ 
         );
 
         self::assertSame(200, $response->getStatusCode());
-        self::assertSame('all good', (string) $response->getBody());
+        self::assertSame('all good', (string)$response->getBody());
     }
 
     public function testHeaderOKStrict(): void
@@ -173,7 +173,7 @@ 
         );
 
         self::assertSame(200, $response->getStatusCode());
-        self::assertSame('all good', (string) $response->getBody());
+        self::assertSame('all good', (string)$response->getBody());
     }
 
     protected function setUp(): void
@@ -257,10 +257,10 @@ 
     {
         $result = [];
 
-        foreach ($response->getHeaders() as $header) {
-            foreach ($header as $headerLine) {
+        foreach ($response->getHeaders() as $header){
+            foreach ($header as $headerLine){
                 $chunk = \explode(';', $headerLine);
-                if (\mb_strpos($chunk[0], '=') === false) {
+                if (\mb_strpos($chunk[0], '=') === false){
                     continue;
                 }
 

Braces +6 added lines, -3 removed lines

@@ -257,10 +257,13 @@
     {
         $result = [];
 
-        foreach ($response->getHeaders() as $header) {
-            foreach ($header as $headerLine) {
+        foreach ($response->getHeaders() as $header)
+        {
+            foreach ($header as $headerLine)
+            {
                 $chunk = \explode(';', $headerLine);
-                if (\mb_strpos($chunk[0], '=') === false) {
+                if (\mb_strpos($chunk[0], '=') === false)
+                {
                     continue;
                 }
 

src/Csrf/src/Middleware/CsrfMiddleware.php

Spacing +8 added lines, -8 removed lines

@@ -25,14 +25,14 @@ 
 
     public function __construct(
         private readonly CsrfConfig $config,
-    ) {}
+    ){}
 
     public function process(Request $request, RequestHandlerInterface $handler): Response
     {
         $cookie = null;
-        if (isset($request->getCookieParams()[$this->config->getCookie()])) {
+        if (isset($request->getCookieParams()[$this->config->getCookie()])){
             $token = $request->getCookieParams()[$this->config->getCookie()];
-        } else {
+        }else{
             //Making new token
             $token = $this->random($this->config->getTokenLength());
 
@@ -43,7 +43,7 @@ 
         //CSRF issues must be handled by Firewall middleware
         $response = $handler->handle($request->withAttribute(self::ATTRIBUTE, $token));
 
-        if (!empty($cookie)) {
+        if (!empty($cookie)){
             return $response->withAddedHeader('Set-Cookie', $cookie);
         }
 
@@ -74,12 +74,12 @@ 
      */
     private function random(int $length = 32): string
     {
-        try {
-            if (empty($string = \random_bytes($length))) {
+        try{
+            if (empty($string = \random_bytes($length))){
                 throw new \RuntimeException('Unable to generate random string');
             }
-        } catch (\Throwable $e) {
-            throw new \RuntimeException('Unable to generate random string', (int) $e->getCode(), $e);
+        }catch (\Throwable $e){
+            throw new \RuntimeException('Unable to generate random string', (int)$e->getCode(), $e);
         }
 
         return \substr(\base64_encode($string), 0, $length);

Braces +16 added lines, -7 removed lines

@@ -25,14 +25,18 @@ 
 
     public function __construct(
         private readonly CsrfConfig $config,
-    ) {}
+    ) {
+}
 
     public function process(Request $request, RequestHandlerInterface $handler): Response
     {
         $cookie = null;
-        if (isset($request->getCookieParams()[$this->config->getCookie()])) {
+        if (isset($request->getCookieParams()[$this->config->getCookie()]))
+        {
             $token = $request->getCookieParams()[$this->config->getCookie()];
-        } else {
+        }
+        else
+        {
             //Making new token
             $token = $this->random($this->config->getTokenLength());
 
@@ -43,7 +47,8 @@ 
         //CSRF issues must be handled by Firewall middleware
         $response = $handler->handle($request->withAttribute(self::ATTRIBUTE, $token));
 
-        if (!empty($cookie)) {
+        if (!empty($cookie))
+        {
             return $response->withAddedHeader('Set-Cookie', $cookie);
         }
 
@@ -74,11 +79,15 @@ 
      */
     private function random(int $length = 32): string
     {
-        try {
-            if (empty($string = \random_bytes($length))) {
+        try
+        {
+            if (empty($string = \random_bytes($length)))
+            {
                 throw new \RuntimeException('Unable to generate random string');
             }
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw new \RuntimeException('Unable to generate random string', (int) $e->getCode(), $e);
         }
 

src/Core/tests/Internal/Factory/CommonCasesTest.php

Spacing +1 added lines, -1 removed lines

@@ -89,7 +89,7 @@
 
     public function testClosureFactory(): void
     {
-        $this->bind(Bucket::class, static function ($data) {
+        $this->bind(Bucket::class, static function ($data){
             return new Bucket('via-closure', $data);
         });
 

Braces +4 added lines, -2 removed lines

@@ -89,7 +89,8 @@ 
 
     public function testClosureFactory(): void
     {
-        $this->bind(Bucket::class, static function ($data) {
+        $this->bind(Bucket::class, static function ($data)
+        {
             return new Bucket('via-closure', $data);
         });
 
@@ -129,7 +130,8 @@ 
         $sample = new SampleClass();
 
         $this->bind(Bucket::class, [Factory::class, 'makeBucketWithSample']);
-        $this->bind(SampleClass::class, static function () use ($sample) {
+        $this->bind(SampleClass::class, static function () use ($sample)
+        {
             return $sample;
         });
 

src/Core/tests/Internal/Resolver/TypeValidationTest.php

Spacing +8 added lines, -8 removed lines

@@ -119,7 +119,7 @@ 
     public function testUnionType(): void
     {
         $this->validateClosureArguments(
-            static fn(array|\Traversable $a, array|\Traversable $b) => null,
+            static fn(array | \Traversable $a, array | \Traversable $b) => null,
             [[1, 2, NAN], new \EmptyIterator()],
         );
     }
@@ -127,7 +127,7 @@ 
     public function testNullableUnionType(): void
     {
         $this->validateClosureArguments(
-            static fn(null|array|\Traversable $a, null|array|\Traversable $b) => null,
+            static fn(null | array | \Traversable $a, null | array | \Traversable $b) => null,
             [null, new \EmptyIterator()],
         );
     }
@@ -190,7 +190,7 @@ 
     public function testWrongUnionType(): void
     {
         $this->validateClosureArguments(
-            static fn(array|\Traversable $a) => null,
+            static fn(array | \Traversable $a) => null,
             ['foo'],
             'a',
         );
@@ -324,14 +324,14 @@ 
         ?string $invalidParameter = null,
         string $exceptionClass = InvalidArgumentException::class,
     ): void {
-        try {
+        try{
             $this->createResolver()->validateArguments(new \ReflectionFunction($closure), $arguments);
-        } catch (ValidationException $e) {
+        }catch (ValidationException $e){
             $this->assertInstanceOf($exceptionClass, $e, 'Expected other exception.');
-            if ($invalidParameter === null) {
+            if ($invalidParameter === null){
                 throw $e;
             }
-            if ($e->getParameter() !== $invalidParameter) {
+            if ($e->getParameter() !== $invalidParameter){
                 $this->fail(
                     \sprintf(
                         'The other argument has been failed: `%s` instead of `%s`.',
@@ -343,7 +343,7 @@ 
             $this->assertTrue(true, 'Invalid value has been failed.');
             return;
         }
-        if ($invalidParameter === null) {
+        if ($invalidParameter === null){
             $this->assertTrue(true, 'Valid argument value has been failed.');
             return;
         }

src/Core/tests/Internal/Proxy/ProxyClassRendererTest.php

Spacing +19 added lines, -19 removed lines

@@ -34,24 +34,24 @@ 
         yield [$from(static fn(string $string = self::STRING_CONST) => 0), 'mixed $string = self::STRING_CONST'];
         yield [
             $from(static fn(string $string = ProxyClassRendererTest::STRING_CONST) => 0),
-            'mixed $string = \\' . self::class . '::STRING_CONST',
+            'mixed $string = \\'.self::class.'::STRING_CONST',
         ];
-        yield [$from(static fn(string|int $string = self::INT_CONST) => 0), 'mixed $string = self::INT_CONST'];
+        yield [$from(static fn(string | int $string = self::INT_CONST) => 0), 'mixed $string = self::INT_CONST'];
         yield [$from(static fn(mixed $string = 42) => 0), 'mixed $string = 42'];
         yield [$from(static fn(int $string = 42) => 0), 'mixed $string = 42'];
         yield [$from(static fn(float $string = 42) => 0), 'mixed $string = 42.0'];
         yield [$from(static fn(?bool $string = false) => 0), 'mixed $string = false'];
-        yield [$from(static fn(bool|null $string = true) => 0), 'mixed $string = true'];
+        yield [$from(static fn(bool | null $string = true) => 0), 'mixed $string = true'];
         yield [$from(static fn(?object $string = null) => 0), 'mixed $string = NULL'];
         yield [$from(static fn(?iterable $string = null) => 0), 'mixed $string = NULL'];
         yield [$from(static fn(\Countable&\ArrayAccess $val) => 0), 'mixed $val'];
         yield [$from(static fn(string ...$val) => 0), 'mixed ...$val'];
-        yield [$from(static fn(string|int ...$val) => 0), 'mixed ...$val'];
-        yield [$from(static fn(string|int &$link) => 0), 'mixed &$link'];
+        yield [$from(static fn(string | int ...$val) => 0), 'mixed ...$val'];
+        yield [$from(static fn(string | int &$link) => 0), 'mixed &$link'];
         yield [$from(self::withSelf(...)), 'mixed $self = new self()'];
         yield [$from(static fn(object $link = new \stdClass()) => 0), 'mixed $link = new \stdClass()'];
         yield [
-            $from(static fn(#[Proxy] float|int|\stdClass|null $string = new \stdClass(1, 2, bar: "\n'zero")) => 0),
+            $from(static fn(#[Proxy] float | int | \stdClass | null $string = new \stdClass(1, 2, bar: "\n'zero")) => 0),
             "mixed \$string = new \stdClass(1, 2, bar: '\n\'zero')",
         ];
         yield [
@@ -66,22 +66,22 @@ 
             public const INT_CONST = 42;
 
             #[ExpectedAttribute('public function test1(...$variadic)')]
-            public function test1(...$variadic) {}
+            public function test1(...$variadic){}
 
             #[ExpectedAttribute('public function test2(mixed $string = self::INT_CONST): string|int')]
-            public function test2(string|int $string = self::INT_CONST): string|int {}
+            public function test2(string | int $string = self::INT_CONST): string | int {}
 
             #[ExpectedAttribute('public function test3(mixed $obj = new \stdClass(new \stdClass(), new \stdClass()))')]
-            public function test3(object $obj = new \stdClass(new \stdClass(), new \stdClass())) {}
+            public function test3(object $obj = new \stdClass(new \stdClass(), new \stdClass())){}
 
-            #[ExpectedAttribute('public function test4(): \\' . ProxyClassRendererTest::class)]
+            #[ExpectedAttribute('public function test4(): \\'.ProxyClassRendererTest::class)]
             public function test4(): ProxyClassRendererTest {}
 
             #[ExpectedAttribute('public function &test5(): string')]
             public function &test5(): string {}
         };
 
-        foreach ((new \ReflectionClass($class))->getMethods() as $method) {
+        foreach ((new \ReflectionClass($class))->getMethods() as $method){
             $expected = $method->getAttributes(ExpectedAttribute::class)[0]->newInstance();
 
             yield [$method, $expected->value];
@@ -96,22 +96,22 @@ 
         $from = static fn(\Closure $closure): \ReflectionParameter => new \ReflectionParameter($closure, 0);
 
         yield [$from(static fn(string $string) => 0), 'string'];
-        yield [$from(static fn(string|int $string) => 0), 'string|int'];
+        yield [$from(static fn(string | int $string) => 0), 'string|int'];
         yield [$from(static fn(mixed $string) => 0), 'mixed'];
         yield [$from(static fn(int $string) => 0), 'int'];
         yield [$from(static fn(float $string) => 0), 'float'];
         yield [$from(static fn(?bool $string) => 0), '?bool'];
-        yield [$from(static fn(bool|null $string) => 0), '?bool'];
+        yield [$from(static fn(bool | null $string) => 0), '?bool'];
         yield [$from(static fn(object $string) => 0), 'object'];
         yield [$from(static fn(iterable $string) => 0), 'iterable'];
         yield [$from(static fn(\Countable&\ArrayAccess $val) => 0), '\Countable&\ArrayAccess'];
         yield [$from(static fn(string ...$val) => 0), 'string'];
-        yield [$from(static fn(string|int ...$val) => 0), 'string|int'];
-        yield [$from(static fn(string|int &$link) => 0), 'string|int'];
-        yield [$from(self::withSelf(...)), '\\' . self::class];
+        yield [$from(static fn(string | int ...$val) => 0), 'string|int'];
+        yield [$from(static fn(string | int &$link) => 0), 'string|int'];
+        yield [$from(self::withSelf(...)), '\\'.self::class];
         yield [$from(static fn(object $link) => 0), 'object'];
-        yield [$from(static fn(#[Proxy] float|int|\stdClass|null $string) => 0), '\stdClass|int|float|null'];
-        yield [$from(static fn(SimpleEnum $val) => 0), '\\' . SimpleEnum::class];
+        yield [$from(static fn(#[Proxy] float | int | \stdClass | null $string) => 0), '\stdClass|int|float|null'];
+        yield [$from(static fn(SimpleEnum $val) => 0), '\\'.SimpleEnum::class];
     }
 
     public function testInterfaceWithConstructor(): void
@@ -174,5 +174,5 @@ 
 {
     public function __construct(
         public readonly string $value,
-    ) {}
+    ){}
 }

src/Core/tests/Internal/Proxy/ProxyTest.php

Spacing +7 added lines, -7 removed lines

@@ -100,7 +100,7 @@ 
             $proxy = $$var;
             self::assertSame(['foo', 'bar', 'baz', 69], $proxy->extraVariadic('foo', 'bar', 'baz', 69));
             self::assertSame(
-                ['foo' => 'foo','zap' => 'bar', 'gas' => 69],
+                ['foo' => 'foo', 'zap' => 'bar', 'gas' => 69],
                 $proxy->extraVariadic(foo: 'foo', zap: 'bar', gas: 69),
             );
         });
@@ -185,7 +185,7 @@ 
         $root->bindSingleton(MockInterface::class, Stub\MockInterfaceImpl::class);
         $root->bindSingleton(EmptyInterface::class, Stub\MockInterfaceImpl::class);
 
-        $mock = $root->invoke(static fn(#[Proxy] MockInterface|EmptyInterface $mock) => $mock);
+        $mock = $root->invoke(static fn(#[Proxy] MockInterface | EmptyInterface $mock) => $mock);
 
         self::assertInstanceOf(MockInterfaceImpl::class, $mock);
     }
@@ -238,7 +238,7 @@ 
     {
         \set_error_handler(static function (int $errno, string $error) use ($interface): void {
             self::assertSame(
-                \sprintf('Using `%s` outside of the `foo` scope is deprecated and will be ' .
+                \sprintf('Using `%s` outside of the `foo` scope is deprecated and will be '.
                     'impossible in version 4.0.', $interface),
                 $error,
             );
@@ -295,7 +295,7 @@ 
     {
         \set_error_handler(static function (int $errno, string $error) use ($interface): void {
             self::assertSame(
-                \sprintf('Using `%s` outside of the `a` scope is deprecated and will be ' .
+                \sprintf('Using `%s` outside of the `a` scope is deprecated and will be '.
                     'impossible in version 4.0.', $interface),
                 $error,
             );
@@ -323,7 +323,7 @@ 
     public function testDeprecationProxyConfigDontThrowIfNotConstructed(string $interface): void
     {
         \set_error_handler(static function (int $errno, string $error) use ($interface): void {
-            self::fail('Unexpected error: ' . $error);
+            self::fail('Unexpected error: '.$error);
         });
 
         $root = new Container();
@@ -343,7 +343,7 @@ 
     }
 
     #[DataProvider('invalidDeprecationProxyArgsDataProvider')]
-    public function testDeprecationProxyConfigArgsRequiredException(string|null $scope, string|null $version): void
+    public function testDeprecationProxyConfigArgsRequiredException(string | null $scope, string | null $version): void
     {
         self::expectException(\InvalidArgumentException::class);
         self::expectExceptionMessage('Scope and version or custom message must be provided.');
@@ -355,7 +355,7 @@ 
     {
         $proxy = new Config\Proxy(EmptyInterface::class);
 
-        $this->assertSame(\sprintf('Proxy to `%s`', EmptyInterface::class), (string) $proxy);
+        $this->assertSame(\sprintf('Proxy to `%s`', EmptyInterface::class), (string)$proxy);
     }
 
     public function testProxyConfigNotInterfaceException(): void

src/Core/tests/Internal/Destructor/MemoryLeaksTest.php

Spacing +2 added lines, -2 removed lines

@@ -35,8 +35,8 @@
         $map = new \WeakMap();
 
         $fn = function (\WeakMap $map): void {
-            foreach ($this as $key => $value) {
-                if (\is_object($value)) {
+            foreach ($this as $key => $value){
+                if (\is_object($value)){
                     $map->offsetSet($value, $key);
                 }
             }

Braces +4 added lines, -2 removed lines

@@ -35,8 +35,10 @@
         $map = new \WeakMap();
 
         $fn = function (\WeakMap $map): void {
-            foreach ($this as $key => $value) {
-                if (\is_object($value)) {
+            foreach ($this as $key => $value)
+            {
+                if (\is_object($value))
+                {
                     $map->offsetSet($value, $key);
                 }
             }

src/Core/tests/Internal/Destructor/FinalizerTest.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
 
                 public function __destruct()
                 {
-                    if ($this->closure !== null) {
+                    if ($this->closure !== null){
                         ($this->closure)();
                     }
                 }

Braces +4 added lines, -2 removed lines

@@ -13,12 +13,14 @@
     {
         (static function (): void {
             $container = new Container();
-            $finalizer = new class {
+            $finalizer = new class
+            {
                 public ?\Closure $closure = null;
 
                 public function __destruct()
                 {
-                    if ($this->closure !== null) {
+                    if ($this->closure !== null)
+                    {
                         ($this->closure)();
                     }
                 }

src/Core/tests/ScopesTest.php

Spacing +4 added lines, -4 removed lines

@@ -32,11 +32,11 @@ 
 
         self::assertNull(ContainerScope::getContainer());
 
-        try {
+        try{
             self::assertTrue(ContainerScope::runScope($container, static function (): never {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
         }
 
         self::assertInstanceOf(RuntimeException::class, $e);
@@ -83,14 +83,14 @@ 
             return $c->get('bucket')->getName() == 'b' && $c->has('other');
         }));
 
-        try {
+        try{
             self::assertTrue($c->runScope([
                 'bucket' => new Bucket('b'),
                 'other'  => new SampleClass(),
             ], static function () use ($c): void {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable) {
+        }catch (\Throwable){
         }
 
         self::assertSame('a', $c->get('bucket')->getName());

Braces +13 added lines, -5 removed lines

@@ -32,11 +32,14 @@ 
 
         self::assertNull(ContainerScope::getContainer());
 
-        try {
+        try
+        {
             self::assertTrue(ContainerScope::runScope($container, static function (): never {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
         }
 
         self::assertInstanceOf(RuntimeException::class, $e);
@@ -83,14 +86,17 @@ 
             return $c->get('bucket')->getName() == 'b' && $c->has('other');
         }));
 
-        try {
+        try
+        {
             self::assertTrue($c->runScope([
                 'bucket' => new Bucket('b'),
                 'other'  => new SampleClass(),
             ], static function () use ($c): void {
                 throw new RuntimeException('exception');
             }));
-        } catch (\Throwable) {
+        }
+        catch (\Throwable)
+        {
         }
 
         self::assertSame('a', $c->get('bucket')->getName());
@@ -135,7 +141,9 @@ 
     public function testHasInstanceAfterMakeWithoutAliasInScope(): void
     {
         $container = new Container();
-        $container->bindSingleton('test', new #[Singleton] class {});
+        $container->bindSingleton('test', new #[Singleton] class
+        {
+});
         $container->make('test');
 
         $container->runScoped(static function (Container $container): void {