spiral / framework

rector.php

Spacing +32 added lines, -32 removed lines

@@ -18,55 +18,55 @@ 
 
 return RectorConfig::configure()
     ->withPaths([
-        __DIR__ . '/src/*/src',
-        __DIR__ . '/src/*/tests',
-        __DIR__ . '/tests',
+        __DIR__.'/src/*/src',
+        __DIR__.'/src/*/tests',
+        __DIR__.'/tests',
     ])
     ->withParallel()
     ->withSkip([
         IfIssetToCoalescingRector::class,
         RemoveUnusedPrivatePropertyRector::class => [
-            __DIR__ . '/src/Scaffolder/src/Command/BootloaderCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/CommandCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/ConfigCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/ControllerCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/FilterCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/JobHandlerCommand.php',
-            __DIR__ . '/src/Scaffolder/src/Command/MiddlewareCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/BootloaderCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/CommandCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/ConfigCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/ControllerCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/FilterCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/JobHandlerCommand.php',
+            __DIR__.'/src/Scaffolder/src/Command/MiddlewareCommand.php',
         ],
         RemoveUnusedPrivateMethodRector::class => [
-            __DIR__ . '/src/Boot/src/Bootloader/ConfigurationBootloader.php',
-            __DIR__ . '/src/Broadcasting/src/Bootloader/BroadcastingBootloader.php',
-            __DIR__ . '/src/Cache/src/Bootloader/CacheBootloader.php',
-            __DIR__ . '/src/Serializer/src/Bootloader/SerializerBootloader.php',
-            __DIR__ . '/src/Validation/src/Bootloader/ValidationBootloader.php',
-            __DIR__ . '/src/Translator/tests/IndexerTest.php',
-            __DIR__ . '/src/Tokenizer/tests/ReflectionFileTest.php',
-            __DIR__ . '/src/Core/tests/SingletonsTest.php',
+            __DIR__.'/src/Boot/src/Bootloader/ConfigurationBootloader.php',
+            __DIR__.'/src/Broadcasting/src/Bootloader/BroadcastingBootloader.php',
+            __DIR__.'/src/Cache/src/Bootloader/CacheBootloader.php',
+            __DIR__.'/src/Serializer/src/Bootloader/SerializerBootloader.php',
+            __DIR__.'/src/Validation/src/Bootloader/ValidationBootloader.php',
+            __DIR__.'/src/Translator/tests/IndexerTest.php',
+            __DIR__.'/src/Tokenizer/tests/ReflectionFileTest.php',
+            __DIR__.'/src/Core/tests/SingletonsTest.php',
         ],
         RemoveUselessVarTagRector::class => [
-            __DIR__ . '/src/Console/src/Traits/HelpersTrait.php',
+            __DIR__.'/src/Console/src/Traits/HelpersTrait.php',
         ],
         RemoveAlwaysTrueIfConditionRector::class => [
-            __DIR__ . '/src/Boot/src/BootloadManager/Initializer.php',
-            __DIR__ . '/src/Stempler/src/Traverser.php',
-            __DIR__ . '/src/Prototype/src/NodeVisitors/LocateProperties.php',
-            __DIR__ . '/src/Prototype/src/NodeVisitors/RemoveTrait.php',
-            __DIR__ . '/src/Logger/src/ListenerRegistry.php',
-            __DIR__ . '/src/Stempler/src/Transform/Merge/ExtendsParent.php',
+            __DIR__.'/src/Boot/src/BootloadManager/Initializer.php',
+            __DIR__.'/src/Stempler/src/Traverser.php',
+            __DIR__.'/src/Prototype/src/NodeVisitors/LocateProperties.php',
+            __DIR__.'/src/Prototype/src/NodeVisitors/RemoveTrait.php',
+            __DIR__.'/src/Logger/src/ListenerRegistry.php',
+            __DIR__.'/src/Stempler/src/Transform/Merge/ExtendsParent.php',
         ],
         RemoveExtraParametersRector::class => [
-            __DIR__ . '/src/Boot/src/BootloadManager/AbstractBootloadManager.php',
+            __DIR__.'/src/Boot/src/BootloadManager/AbstractBootloadManager.php',
         ],
         RemoveUnusedPrivateMethodParameterRector::class => [
-            __DIR__ . '/src/Core/src/Internal/Factory.php',
-            __DIR__ . '/src/Core/tests/InjectableTest.php',
+            __DIR__.'/src/Core/src/Internal/Factory.php',
+            __DIR__.'/src/Core/tests/InjectableTest.php',
         ],
         RemoveDoubleAssignRector::class => [
-            __DIR__ . '/src/Core/tests/Scope/FinalizeAttributeTest.php',
+            __DIR__.'/src/Core/tests/Scope/FinalizeAttributeTest.php',
         ],
         RemoveUnusedVariableAssignRector::class => [
-            __DIR__ . '/src/Core/tests/ExceptionsTest.php',
+            __DIR__.'/src/Core/tests/ExceptionsTest.php',
         ],
 
         // to be enabled later for bc break 4.x
@@ -75,8 +75,8 @@ 
         RemoveUnusedPromotedPropertyRector::class,
 
         // start with short open tag
-        __DIR__ . '/src/Views/tests/fixtures/other/var.php',
-        __DIR__ . '/tests/app/views/native.php',
+        __DIR__.'/src/Views/tests/fixtures/other/var.php',
+        __DIR__.'/tests/app/views/native.php',
 
         // example code for test
         '*/Fixtures/*',

src/Stempler/tests/Transform/BaseTestCase.php

Spacing +4 added lines, -4 removed lines

@@ -27,7 +27,7 @@ 
 {
     protected function compile(string $source, array $visitors = [], LoaderInterface $loader = null)
     {
-        if ($loader === null) {
+        if ($loader === null){
             $loader = new StringLoader();
             $loader->set('root', $source);
         }
@@ -62,11 +62,11 @@ 
         $builder->getCompiler()->addRenderer(new DynamicRenderer(new DirectiveGroup()));
         $builder->getCompiler()->addRenderer(new HTMLRenderer());
 
-        foreach ($this->getVisitors() as $visitor) {
+        foreach ($this->getVisitors() as $visitor){
             $builder->addVisitor($visitor);
         }
 
-        foreach ($visitors as $visitor) {
+        foreach ($visitors as $visitor){
             $builder->addVisitor($visitor);
         }
 
@@ -80,6 +80,6 @@ 
 
     protected function getFixtureLoader(): LoaderInterface
     {
-        return new DirectoryLoader(__DIR__ . '/../fixtures');
+        return new DirectoryLoader(__DIR__.'/../fixtures');
     }
 }

src/Queue/tests/Interceptor/Push/CoreTest.php

Spacing +3 added lines, -3 removed lines

@@ -24,7 +24,7 @@ 
             $queue = m::mock(QueueInterface::class)
         );
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload)){
             $this->markTestIncomplete('QueueInterface does not support non-array payloads');
             return;
         }
@@ -46,7 +46,7 @@ 
             $queue = m::mock(QueueInterface::class)
         );
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload)){
             $this->markTestIncomplete('QueueInterface does not support non-array payloads');
             return;
         }
@@ -81,7 +81,7 @@ 
                 && $payload === ['baz' => 'baf']
                 && $options->getHeader('foo') === ['bar']);
 
-        ContainerScope::runScope($container, function() use($core) {
+        ContainerScope::runScope($container, function () use($core) {
             $core->callAction('foo', 'bar', [
                 'id' => 'job-id',
                 'payload' => ['baz' => 'baf'],

Braces +6 added lines, -3 removed lines

@@ -24,7 +24,8 @@ 
             $queue = m::mock(QueueInterface::class)
         );
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload))
+        {
             $this->markTestIncomplete('QueueInterface does not support non-array payloads');
             return;
         }
@@ -46,7 +47,8 @@ 
             $queue = m::mock(QueueInterface::class)
         );
 
-        if (!\is_array($payload)) {
+        if (!\is_array($payload))
+        {
             $this->markTestIncomplete('QueueInterface does not support non-array payloads');
             return;
         }
@@ -81,7 +83,8 @@ 
                 && $payload === ['baz' => 'baf']
                 && $options->getHeader('foo') === ['bar']);
 
-        ContainerScope::runScope($container, function() use($core) {
+        ContainerScope::runScope($container, function() use($core)
+        {
             $core->callAction('foo', 'bar', [
                 'id' => 'job-id',
                 'payload' => ['baz' => 'baf'],

src/Boot/tests/EnvironmentTest.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
     /**
      * @throws \Throwable
      */
-    protected function getEnv(array $env, bool $overwite= true): EnvironmentInterface
+    protected function getEnv(array $env, bool $overwite = true): EnvironmentInterface
     {
         $core = TestCore::create(['root' => __DIR__])->run(new Environment($env, $overwite));
 

src/Boot/tests/ConfigsTest.php

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
     {
         $core = TestCore::create([
             'root'   => __DIR__,
-            'config' => __DIR__ . '/config'
+            'config' => __DIR__.'/config'
         ])->run();
 
         /** @var TestConfig $config */
@@ -27,7 +27,7 @@ 
     {
         $core = TestCore::create([
             'root'   => __DIR__,
-            'config' => __DIR__ . '/config',
+            'config' => __DIR__.'/config',
         ])->run();
 
         $configurator = $core->getContainer()->get(ConfiguratorInterface::class);

src/Http/tests/HttpTest.php

Spacing +5 added lines, -5 removed lines

@@ -89,7 +89,7 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function ($req, $resp) {
+        $core->setHandler(function ($req, $resp){
             echo 'hello!';
 
             return $resp->withAddedHeader('hello', 'value');
@@ -105,7 +105,7 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function ($req, $resp) {
+        $core->setHandler(function ($req, $resp){
             echo 'hello!';
             $resp->getBody()->write('world ');
 
@@ -122,7 +122,7 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function () {
+        $core->setHandler(function (){
             ob_start();
             ob_start();
             echo 'hello!';
@@ -232,7 +232,7 @@ 
         $dispatcher
             ->expects(self::exactly(2))
             ->method('dispatch')
-            ->with($this->callback(static fn(RequestReceived|RequestHandled $event): bool => true));
+            ->with($this->callback(static fn(RequestReceived | RequestHandled $event): bool => true));
         $this->container->bind(EventDispatcherInterface::class, $dispatcher);
 
         $core = $this->getCore();
@@ -289,7 +289,7 @@ 
                 TraceKind::SERVER,
             )
             ->willReturnCallback(
-                function ($name, $callback, $attributes, $scoped, $traceKind) {
+                function ($name, $callback, $attributes, $scoped, $traceKind){
                     self::assertSame($attributes, [
                         'http.method' => 'GET',
                         'http.url' => 'http://example.org/path',

Braces +8 added lines, -4 removed lines

@@ -89,7 +89,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function ($req, $resp) {
+        $core->setHandler(function ($req, $resp)
+        {
             echo 'hello!';
 
             return $resp->withAddedHeader('hello', 'value');
@@ -105,7 +106,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function ($req, $resp) {
+        $core->setHandler(function ($req, $resp)
+        {
             echo 'hello!';
             $resp->getBody()->write('world ');
 
@@ -122,7 +124,8 @@ 
     {
         $core = $this->getCore();
 
-        $core->setHandler(function () {
+        $core->setHandler(function ()
+        {
             ob_start();
             ob_start();
             echo 'hello!';
@@ -289,7 +292,8 @@ 
                 TraceKind::SERVER,
             )
             ->willReturnCallback(
-                function ($name, $callback, $attributes, $scoped, $traceKind) {
+                function ($name, $callback, $attributes, $scoped, $traceKind)
+                {
                     self::assertSame($attributes, [
                         'http.method' => 'GET',
                         'http.url' => 'http://example.org/path',

src/Http/tests/Stream/GeneratorStreamTest.php

Spacing +4 added lines, -4 removed lines

@@ -99,9 +99,9 @@ 
         $rValue = 'return-value';
         $stream = $this->createStream(self::DEFAULT_SEQUENCE, $rValue);
 
-        $result = (string) $stream;
+        $result = (string)$stream;
 
-        $this->assertSame(self::DEFAULT_CONTENT_RESULT . $rValue, $result);
+        $this->assertSame(self::DEFAULT_CONTENT_RESULT.$rValue, $result);
     }
 
     public function testToStringWithReturnOnly(): void
@@ -109,7 +109,7 @@ 
         $rValue = 'return-value';
         $stream = $this->createStream([], $rValue);
 
-        $result = (string) $stream;
+        $result = (string)$stream;
 
         $this->assertSame($rValue, $result);
     }
@@ -119,7 +119,7 @@ 
         $stream = $this->createStream();
         $stream->close();
 
-        $this->assertSame('', (string) $stream);
+        $this->assertSame('', (string)$stream);
 
         $this->expectException(RuntimeException::class);
         $this->expectExceptionMessage('Unable to read stream contents.');

src/Http/tests/PipelineTest.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,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);
@@ -89,7 +89,7 @@ 
         $this->container->getBinder('http')
             ->bind(ServerRequestInterface::class, static fn(CurrentRequest $cr) => $cr->get());
 
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface{
             public function process(
                 ServerRequestInterface $request,
                 RequestHandlerInterface $handler,

Braces +6 added lines, -3 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) => $cr->get());
 
-        $middleware = new class implements MiddlewareInterface {
+        $middleware = new class implements MiddlewareInterface
+        {
             public function process(
                 ServerRequestInterface $request,
                 RequestHandlerInterface $handler,
@@ -107,7 +109,8 @@ 
 
         $this->container->runScope(
             new \Spiral\Core\Scope(name: 'http'),
-            function (ScopeInterface $c) use ($middleware) {
+            function (ScopeInterface $c) use ($middleware)
+            {
                 $request = new ServerRequest('GET', '');
                 $handler = new CallableHandler(fn() => 'response', new ResponseFactory(new HttpConfig(['headers' => []])));
 

src/Interceptors/tests/Unit/Context/TargetTest.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
         $target = Target::fromReflectionMethod($reflection, self::class);
 
         self::assertSame($reflection, $target->getReflection());
-        self::assertSame(self::class . '->' . __FUNCTION__, (string)$target);
+        self::assertSame(self::class.'->'.__FUNCTION__, (string)$target);
         self::assertNull($target->getObject());
     }
 
@@ -58,7 +58,7 @@ 
         $target = Target::fromReflectionMethod($reflection, $this);
 
         self::assertSame($reflection, $target->getReflection());
-        self::assertSame(self::class . '->' . __FUNCTION__, (string)$target);
+        self::assertSame(self::class.'->'.__FUNCTION__, (string)$target);
         self::assertNotNull($target->getObject());
     }
 
@@ -124,7 +124,7 @@ 
         $reflection = $target->getReflection();
         self::assertSame($hasReflection, $reflection !== null);
         self::assertNull($target->getObject());
-        if ($hasReflection) {
+        if ($hasReflection){
             self::assertInstanceOf(\ReflectionMethod::class, $reflection);
             self::assertSame($action, $reflection->getName());
         }