spiral / framework

src/Files/tests/DirectoriesTest.php

Spacing +24 added lines, -24 removed lines

@@ -25,7 +25,7 @@ 
     public function testEnsureDirectory(): void
     {
         $files = new Files();
-        $directory = self::FIXTURE_DIRECTORY . 'directory/';
+        $directory = self::FIXTURE_DIRECTORY.'directory/';
 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
@@ -39,7 +39,7 @@ 
     public function testEnsureExistedDirectory(): void
     {
         $files = new Files();
-        $directory = self::FIXTURE_DIRECTORY . 'directory/';
+        $directory = self::FIXTURE_DIRECTORY.'directory/';
 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
@@ -58,7 +58,7 @@ 
     public function testEnsureNestedDirectory(): void
     {
         $files = new Files();
-        $directory = self::FIXTURE_DIRECTORY . 'directory/sub/other';
+        $directory = self::FIXTURE_DIRECTORY.'directory/sub/other';
 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
@@ -72,7 +72,7 @@ 
     public function testEnsureNestedDirectoryNoRecursivePermissions(): void
     {
         $files = new Files();
-        $directory = self::FIXTURE_DIRECTORY . 'directory/sub/other';
+        $directory = self::FIXTURE_DIRECTORY.'directory/sub/other';
 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
@@ -86,14 +86,14 @@ 
     public function testEnsureExistedNestedDirectory(): void
     {
         $files = new Files();
-        $directory = self::FIXTURE_DIRECTORY . 'directory/sub/other';
+        $directory = self::FIXTURE_DIRECTORY.'directory/sub/other';
 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
 
-        mkdir(self::FIXTURE_DIRECTORY . 'directory');
-        mkdir(self::FIXTURE_DIRECTORY . 'directory/sub');
-        mkdir(self::FIXTURE_DIRECTORY . 'directory/sub/other');
+        mkdir(self::FIXTURE_DIRECTORY.'directory');
+        mkdir(self::FIXTURE_DIRECTORY.'directory/sub');
+        mkdir(self::FIXTURE_DIRECTORY.'directory/sub/other');
 
         self::assertTrue($files->exists($directory));
         self::assertTrue($files->isDirectory($directory));
@@ -107,13 +107,13 @@ 
     public function testDeleteDirectoryContent(): void
     {
         $files = new Files();
-        $baseDirectory = self::FIXTURE_DIRECTORY . 'directory/';
-        $directory = $baseDirectory . 'sub/other';
+        $baseDirectory = self::FIXTURE_DIRECTORY.'directory/';
+        $directory = $baseDirectory.'sub/other';
 
         $filenames = [
-            $baseDirectory . 'test.file',
-            $directory . 'other.file',
-            $directory . '.sample',
+            $baseDirectory.'test.file',
+            $directory.'other.file',
+            $directory.'.sample',
         ];
 
         self::assertFalse($files->exists($baseDirectory));
@@ -130,7 +130,7 @@ 
         self::assertTrue($files->exists($directory));
         self::assertTrue($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename){
             self::assertFalse($files->exists($filename));
             $files->write($filename, 'random-data');
             self::assertTrue($files->exists($filename));
@@ -144,7 +144,7 @@ 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename){
             self::assertFalse($files->exists($filename));
         }
     }
@@ -152,13 +152,13 @@ 
     public function testDeleteDirectory(): void
     {
         $files = new Files();
-        $baseDirectory = self::FIXTURE_DIRECTORY . 'directory/';
-        $directory = $baseDirectory . 'sub/other';
+        $baseDirectory = self::FIXTURE_DIRECTORY.'directory/';
+        $directory = $baseDirectory.'sub/other';
 
         $filenames = [
-            $baseDirectory . 'test.file',
-            $directory . 'other.file',
-            $directory . '.sample',
+            $baseDirectory.'test.file',
+            $directory.'other.file',
+            $directory.'.sample',
         ];
 
         self::assertFalse($files->exists($baseDirectory));
@@ -175,7 +175,7 @@ 
         self::assertTrue($files->exists($directory));
         self::assertTrue($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename){
             self::assertFalse($files->exists($filename));
             $files->write($filename, 'random-data');
             self::assertTrue($files->exists($filename));
@@ -189,7 +189,7 @@ 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename){
             self::assertFalse($files->exists($filename));
         }
     }
@@ -200,8 +200,8 @@ 
 
         $files = new Files();
 
-        $files->write(self::FIXTURE_DIRECTORY . 'test', 'data');
-        $files->deleteDirectory(self::FIXTURE_DIRECTORY . 'test');
+        $files->write(self::FIXTURE_DIRECTORY.'test', 'data');
+        $files->deleteDirectory(self::FIXTURE_DIRECTORY.'test');
     }
 
     public function testGetFiles(): void

Braces +8 added lines, -4 removed lines

@@ -130,7 +130,8 @@ 
         self::assertTrue($files->exists($directory));
         self::assertTrue($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename)
+        {
             self::assertFalse($files->exists($filename));
             $files->write($filename, 'random-data');
             self::assertTrue($files->exists($filename));
@@ -144,7 +145,8 @@ 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename)
+        {
             self::assertFalse($files->exists($filename));
         }
     }
@@ -175,7 +177,8 @@ 
         self::assertTrue($files->exists($directory));
         self::assertTrue($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename)
+        {
             self::assertFalse($files->exists($filename));
             $files->write($filename, 'random-data');
             self::assertTrue($files->exists($filename));
@@ -189,7 +192,8 @@ 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->isDirectory($directory));
 
-        foreach ($filenames as $filename) {
+        foreach ($filenames as $filename)
+        {
             self::assertFalse($files->exists($filename));
         }
     }

src/Files/tests/IOTest.php

Spacing +20 added lines, -20 removed lines

@@ -27,7 +27,7 @@ 
     {
         $files = new Files();
 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
         self::assertFalse($files->exists($filename));
 
         $files->write($filename, 'some-data');
@@ -40,8 +40,8 @@ 
     {
         $files = new Files();
 
-        $directory = self::FIXTURE_DIRECTORY . '/directory/abc/';
-        $filename = $directory . 'test.txt';
+        $directory = self::FIXTURE_DIRECTORY.'/directory/abc/';
+        $filename = $directory.'test.txt';
 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->exists($filename));
@@ -59,7 +59,7 @@ 
     {
         $files = new Files();
 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
         self::assertFalse($files->exists($filename));
 
         $files->write($filename, 'some-data');
@@ -75,7 +75,7 @@ 
 
         $files = new Files();
 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
         self::assertFalse($files->exists($filename));
 
         $files->read($filename);
@@ -94,14 +94,14 @@ 
         $this->expectException(FileNotFoundException::class);
 
         $files = new Files();
-        $files->getPermissions(self::FIXTURE_DIRECTORY . '/missing');
+        $files->getPermissions(self::FIXTURE_DIRECTORY.'/missing');
     }
 
     public function testAppend(): void
     {
         $files = new Files();
 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
         self::assertFalse($files->exists($filename));
 
         $files->append($filename, 'some-data');
@@ -117,8 +117,8 @@ 
     {
         $files = new Files();
 
-        $directory = self::FIXTURE_DIRECTORY . '/directory/abc/';
-        $filename = $directory . 'test.txt';
+        $directory = self::FIXTURE_DIRECTORY.'/directory/abc/';
+        $filename = $directory.'test.txt';
 
         self::assertFalse($files->exists($directory));
         self::assertFalse($files->exists($filename));
@@ -139,7 +139,7 @@ 
     {
         $files = new Files();
 
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
 
         self::assertFalse($files->exists($filename));
         $files->touch($filename);
@@ -149,7 +149,7 @@ 
     public function testDelete(): void
     {
         $files = new Files();
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
 
         self::assertFalse($files->exists($filename));
 
@@ -163,7 +163,7 @@ 
     public function testDeleteMissingFile(): void
     {
         $files = new Files();
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
 
         self::assertFalse($files->exists($filename));
         $files->delete($filename);
@@ -172,8 +172,8 @@ 
     public function testCopy(): void
     {
         $files = new Files();
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
 
         self::assertFalse($files->exists($filename));
         $files->write($filename, 'some-data');
@@ -196,8 +196,8 @@ 
         $this->expectException(FileNotFoundException::class);
 
         $files = new Files();
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
 
         self::assertFalse($files->exists($filename));
         $files->copy($filename, $destination);
@@ -206,8 +206,8 @@ 
     public function testMove(): void
     {
         $files = new Files();
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
 
         self::assertFalse($files->exists($filename));
         $files->write($filename, 'some-data');
@@ -230,8 +230,8 @@ 
         $this->expectException(FileNotFoundException::class);
 
         $files = new Files();
-        $filename = self::FIXTURE_DIRECTORY . '/test.txt';
-        $destination = self::FIXTURE_DIRECTORY . '/new.txt';
+        $filename = self::FIXTURE_DIRECTORY.'/test.txt';
+        $destination = self::FIXTURE_DIRECTORY.'/new.txt';
 
         self::assertFalse($files->exists($filename));
         $files->move($filename, $destination);

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
         self::assertFalse($files->isDirectory($directory));
 
-        $files->write($filename, 'some-data', FilesInterface::READONLY, true);
+        $files->write($filename, 'some-data', FilesInterface::readonly, true);
 
         self::assertTrue($files->isDirectory($directory));
         self::assertTrue($files->exists($filename));

src/Console/tests/SignatureTest.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -119,7 +119,7 @@
   -o, --id[=ID]         Id option description. (multiple values allowed)
   -Q, --quit            Quit option description.
       --naf[=NAF]       Naf option description. [default: "default"]
-HELP, $core->run(command: 'help', input: ['command_name' => 'foo:bar'])->getOutput()->fetch());
+help, $core->run(command: 'help', input: ['command_name' => 'foo:bar'])->getOutput()->fetch());
     }
 
     public function testDescriptionFromConstant(): void

src/Router/tests/Loader/Configurator/RoutingConfiguratorTest.php

Spacing +3 added lines, -3 removed lines

@@ -17,7 +17,7 @@ 
 
         self::assertCount(0, $routes->getCollection());
 
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php');
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php');
 
         self::assertCount(3, $routes->getCollection());
     }
@@ -28,7 +28,7 @@ 
 
         self::assertCount(0, $routes->getCollection());
 
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php', 'php');
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php', 'php');
 
         self::assertCount(3, $routes->getCollection());
     }
@@ -39,7 +39,7 @@ 
 
         self::assertCount(0, $routes->getCollection());
 
-        $routes->import(\dirname(__DIR__, 2) . '/Fixtures/file.php', 'yaml');
+        $routes->import(\dirname(__DIR__, 2).'/Fixtures/file.php', 'yaml');
 
         self::assertCount(0, $routes->getCollection());
     }

src/Router/tests/Loader/PhpFileLoaderTest.php

Spacing +2 added lines, -2 removed lines

@@ -29,12 +29,12 @@
     {
         $loader = new PhpFileLoader($this->container, $this->container);
 
-        $routes = $loader->load(\dirname(__DIR__) . '/Fixtures/file.php');
+        $routes = $loader->load(\dirname(__DIR__).'/Fixtures/file.php');
         self::assertInstanceOf(RouteCollection::class, $routes);
         self::assertCount(3, $routes);
 
         $this->expectException(LoaderLoadException::class);
-        $loader->load(\dirname(__DIR__) . '/Fixtures/unknown.php');
+        $loader->load(\dirname(__DIR__).'/Fixtures/unknown.php');
     }
 
     public function testSupports(): void

src/Router/tests/RouterTest.php

Spacing +2 added lines, -2 removed lines

@@ -54,7 +54,7 @@ 
         $dispatcher
             ->expects(self::exactly(2))
             ->method('dispatch')
-            ->with($this->callback(static fn (Routing|RouteMatched $event): bool => $event->request instanceof ServerRequest));
+            ->with($this->callback(static fn (Routing | RouteMatched $event): bool => $event->request instanceof ServerRequest));
 
         $router = $this->makeRouter('', $dispatcher);
         $router->setDefault($route);
@@ -90,7 +90,7 @@ 
         $router->import($configurator);
         $this->getContainer()->get(GroupRegistry::class)->registerRoutes($router);
 
-        $uri = (string) $router->uri('foo', ['host' => 'some']);
+        $uri = (string)$router->uri('foo', ['host' => 'some']);
         self::assertSame('some/register', $uri);
         self::assertFalse(\str_contains('https://host.com', $uri));
     }

src/Router/tests/SingleActionTest.php

Spacing +2 added lines, -2 removed lines

@@ -96,9 +96,9 @@
         self::assertSame('echoed', (string)$response->getBody());
 
         $e = null;
-        try {
+        try{
             $router->handle(new ServerRequest('GET', new Uri('/test')));
-        } catch (UndefinedRouteException $e) {
+        }catch (UndefinedRouteException $e){
         }
 
         self::assertNotNull($e, 'Autofill not fired');

Braces +5 added lines, -2 removed lines

@@ -96,9 +96,12 @@
         self::assertSame('echoed', (string)$response->getBody());
 
         $e = null;
-        try {
+        try
+        {
             $router->handle(new ServerRequest('GET', new Uri('/test')));
-        } catch (UndefinedRouteException $e) {
+        }
+        catch (UndefinedRouteException $e)
+        {
         }
 
         self::assertNotNull($e, 'Autofill not fired');

src/Router/tests/RouteGroupTest.php

Spacing +1 added lines, -1 removed lines

@@ -150,7 +150,7 @@
         $route = $this->router->getRoute('name');
         self::assertNotNull($route->match(new ServerRequest('GET', '/api/blog')));
 
-        self::assertSame('/api/blog', (string) $route->uri());
+        self::assertSame('/api/blog', (string)$route->uri());
     }
 
     public static function routePrefixDataProvider(): iterable

src/Stempler/tests/TraverserTest.php

Spacing +7 added lines, -7 removed lines

@@ -32,10 +32,10 @@ 
         $doc = $this->parse('<a href="url"></a>');
 
         $t = new Traverser();
-        $t->addVisitor(new class() implements VisitorInterface {
+        $t->addVisitor(new class() implements VisitorInterface{
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
             {
-                if ($node instanceof Tag && $node->name === 'a') {
+                if ($node instanceof Tag && $node->name === 'a'){
                     $node->name = 'b';
                 }
 
@@ -58,7 +58,7 @@ 
         $doc = $this->parse('<a href="url"></a>');
 
         $t = new Traverser();
-        $t->addVisitor(new class() implements VisitorInterface {
+        $t->addVisitor(new class() implements VisitorInterface{
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
             {
                 return null;
@@ -66,7 +66,7 @@ 
 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
             {
-                if ($node instanceof Tag && $node->name === 'a') {
+                if ($node instanceof Tag && $node->name === 'a'){
                     $new = new Tag();
                     $new->name = 'link';
                     $new->void = true;
@@ -88,7 +88,7 @@ 
         $doc = $this->parse('<a href="url"><b></b>hello</a>');
 
         $t = new Traverser();
-        $t->addVisitor(new class() implements VisitorInterface {
+        $t->addVisitor(new class() implements VisitorInterface{
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
             {
                 return null;
@@ -96,7 +96,7 @@ 
 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
             {
-                if ($node instanceof Tag && $node->name === 'b') {
+                if ($node instanceof Tag && $node->name === 'b'){
                     return VisitorInterface::REMOVE_NODE;
                 }
 
@@ -123,7 +123,7 @@ 
 
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
     {
-        if ($ctx->getCurrentNode() instanceof Raw) {
+        if ($ctx->getCurrentNode() instanceof Raw){
             self::assertInstanceOf(Tag::class, $ctx->getParentNode());
             self::assertSame('b', $ctx->getParentNode()->name);
 

Braces +14 added lines, -7 removed lines

@@ -32,10 +32,12 @@ 
         $doc = $this->parse('<a href="url"></a>');
 
         $t = new Traverser();
-        $t->addVisitor(new class() implements VisitorInterface {
+        $t->addVisitor(new class() implements VisitorInterface
+        {
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
             {
-                if ($node instanceof Tag && $node->name === 'a') {
+                if ($node instanceof Tag && $node->name === 'a')
+                {
                     $node->name = 'b';
                 }
 
@@ -58,7 +60,8 @@ 
         $doc = $this->parse('<a href="url"></a>');
 
         $t = new Traverser();
-        $t->addVisitor(new class() implements VisitorInterface {
+        $t->addVisitor(new class() implements VisitorInterface
+        {
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
             {
                 return null;
@@ -66,7 +69,8 @@ 
 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
             {
-                if ($node instanceof Tag && $node->name === 'a') {
+                if ($node instanceof Tag && $node->name === 'a')
+                {
                     $new = new Tag();
                     $new->name = 'link';
                     $new->void = true;
@@ -88,7 +92,8 @@ 
         $doc = $this->parse('<a href="url"><b></b>hello</a>');
 
         $t = new Traverser();
-        $t->addVisitor(new class() implements VisitorInterface {
+        $t->addVisitor(new class() implements VisitorInterface
+        {
             public function enterNode(mixed $node, VisitorContext $ctx): mixed
             {
                 return null;
@@ -96,7 +101,8 @@ 
 
             public function leaveNode(mixed $node, VisitorContext $ctx): mixed
             {
-                if ($node instanceof Tag && $node->name === 'b') {
+                if ($node instanceof Tag && $node->name === 'b')
+                {
                     return VisitorInterface::REMOVE_NODE;
                 }
 
@@ -123,7 +129,8 @@ 
 
     public function enterNode(mixed $node, VisitorContext $ctx): mixed
     {
-        if ($ctx->getCurrentNode() instanceof Raw) {
+        if ($ctx->getCurrentNode() instanceof Raw)
+        {
             self::assertInstanceOf(Tag::class, $ctx->getParentNode());
             self::assertSame('b', $ctx->getParentNode()->name);