spiral / framework

tests/Framework/ConsoleTest.php

Spacing +3 added lines, -3 removed lines

@@ -31,12 +31,12 @@
 
         $fs = new Files();
 
-        if ($fs->isDirectory(__DIR__ . '/../app/migrations')) {
-            $fs->deleteDirectory(__DIR__ . '/../app/migrations');
+        if ($fs->isDirectory(__DIR__.'/../app/migrations')){
+            $fs->deleteDirectory(__DIR__.'/../app/migrations');
         }
 
         $runtime = $this->app->get(DirectoriesInterface::class)->get('runtime');
-        if ($fs->isDirectory($runtime)) {
+        if ($fs->isDirectory($runtime)){
             $fs->deleteDirectory($runtime);
         }
     }

Braces +4 added lines, -2 removed lines

@@ -31,12 +31,14 @@
 
         $fs = new Files();
 
-        if ($fs->isDirectory(__DIR__ . '/../app/migrations')) {
+        if ($fs->isDirectory(__DIR__ . '/../app/migrations'))
+        {
             $fs->deleteDirectory(__DIR__ . '/../app/migrations');
         }
 
         $runtime = $this->app->get(DirectoriesInterface::class)->get('runtime');
-        if ($fs->isDirectory($runtime)) {
+        if ($fs->isDirectory($runtime))
+        {
             $fs->deleteDirectory($runtime);
         }
     }

tests/Framework/BaseTest.php

Spacing +4 added lines, -4 removed lines

@@ -17,10 +17,10 @@
     public function makeApp(array $env = []): TestApp
     {
         return TestApp::init([
-            'root'    => __DIR__ . '/../..',
-            'app'     => __DIR__ . '/../app',
-            'runtime' => sys_get_temp_dir() . '/spiral',
-            'cache'   => sys_get_temp_dir() . '/spiral',
+            'root'    => __DIR__.'/../..',
+            'app'     => __DIR__.'/../app',
+            'runtime' => sys_get_temp_dir().'/spiral',
+            'cache'   => sys_get_temp_dir().'/spiral',
         ], new Environment($env), false);
     }
 }

tests/Framework/SnapshotTest.php

Spacing +2 added lines, -2 removed lines

@@ -20,9 +20,9 @@
     {
         $app = $this->makeApp();
 
-        try {
+        try{
             throw new \Error('test error');
-        } catch (\Error $e) {
+        }catch (\Error $e){
             /** @var SnapshotInterface $s */
             $s = $app->get(SnapshotterInterface::class)->register($e);
         }

Braces +5 added lines, -2 removed lines

@@ -20,9 +20,12 @@
     {
         $app = $this->makeApp();
 
-        try {
+        try
+        {
             throw new \Error('test error');
-        } catch (\Error $e) {
+        }
+        catch (\Error $e)
+        {
             /** @var SnapshotInterface $s */
             $s = $app->get(SnapshotterInterface::class)->register($e);
         }

tests/Framework/Module/PublishTest.php

Spacing +8 added lines, -8 removed lines

@@ -15,12 +15,12 @@ 
 
 class PublishTest extends ConsoleTest
 {
-    protected const TEST_FILE   = __DIR__ . '/test.txt';
-    protected const TEST_FILE_2 = __DIR__ . '/PublishTest.php';
+    protected const TEST_FILE   = __DIR__.'/test.txt';
+    protected const TEST_FILE_2 = __DIR__.'/PublishTest.php';
 
     public function tearDown(): void
     {
-        if (file_exists(self::TEST_FILE)) {
+        if (file_exists(self::TEST_FILE)){
             unlink(self::TEST_FILE);
         }
 
@@ -176,7 +176,7 @@ 
         $this->runCommandDebug('publish', [
             'type'   => 'replace',
             'target' => '@runtime',
-            'source' => __DIR__ . '/*',
+            'source' => __DIR__.'/*',
             'mode'   => 'runtime'
         ]);
 
@@ -194,7 +194,7 @@ 
         $this->runCommandDebug('publish', [
             'type'   => 'follow',
             'target' => '@runtime',
-            'source' => __DIR__ . '/*',
+            'source' => __DIR__.'/*',
             'mode'   => 'runtime'
         ]);
 
@@ -212,7 +212,7 @@ 
         $this->runCommandDebug('publish', [
             'type'   => 'follow',
             'target' => '@runtime/test.txt',
-            'source' => self::TEST_FILE . 'invalid',
+            'source' => self::TEST_FILE.'invalid',
             'mode'   => 'runtime'
         ]);
     }
@@ -227,13 +227,13 @@ 
         $this->runCommandDebug('publish', [
             'type'   => 'follow',
             'target' => '@runtime/test.txt',
-            'source' => self::TEST_FILE . 'invalid/*',
+            'source' => self::TEST_FILE.'invalid/*',
             'mode'   => 'runtime'
         ]);
     }
 
     protected function file(string $dir, string $name)
     {
-        return $this->app->get(DirectoriesInterface::class)->get($dir) . $name;
+        return $this->app->get(DirectoriesInterface::class)->get($dir).$name;
     }
 }

Braces +2 added lines, -1 removed lines

@@ -20,7 +20,8 @@
 
     public function tearDown(): void
     {
-        if (file_exists(self::TEST_FILE)) {
+        if (file_exists(self::TEST_FILE))
+        {
             unlink(self::TEST_FILE);
         }
 

tests/Framework/Http/SessionTest.php

Spacing +6 added lines, -6 removed lines

@@ -18,7 +18,7 @@ 
 {
     public function testSetSid(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -32,7 +32,7 @@ 
 
     public function testSessionResume(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -56,7 +56,7 @@ 
 
     public function testSessionRegenerateId(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -73,7 +73,7 @@ 
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
 
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             $this->session()->regenerateID(false);
 
             return ++$this->session()->getSection('cli')->value;
@@ -92,7 +92,7 @@ 
 
     public function testDestroySession(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -105,7 +105,7 @@ 
         ]);
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function (){
             $this->session()->destroy();
             $this->assertFalse($this->session()->isStarted());
 

Braces +12 added lines, -6 removed lines

@@ -18,7 +18,8 @@ 
 {
     public function testSetSid(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -32,7 +33,8 @@ 
 
     public function testSessionResume(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -56,7 +58,8 @@ 
 
     public function testSessionRegenerateId(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -73,7 +76,8 @@ 
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
 
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             $this->session()->regenerateID(false);
 
             return ++$this->session()->getSection('cli')->value;
@@ -92,7 +96,8 @@ 
 
     public function testDestroySession(): void
     {
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             return ++$this->session()->getSection('cli')->value;
         });
 
@@ -105,7 +110,8 @@ 
         ]);
         $this->assertSame(200, $result->getStatusCode());
         $this->assertSame('2', $result->getBody()->__toString());
-        $this->http->setHandler(function () {
+        $this->http->setHandler(function ()
+        {
             $this->session()->destroy();
             $this->assertFalse($this->session()->isStarted());
 

tests/Framework/Http/SapiTest.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 
     public function setUp(): void
     {
-        $this->bufferEmitter = new class() implements EmitterInterface {
+        $this->bufferEmitter = new class() implements EmitterInterface{
             public $response;
 
             public function emit(ResponseInterface $response): bool
@@ -59,7 +59,7 @@ 
         $e = $this->bufferEmitter;
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -68,7 +68,7 @@ 
         $this->app->get(SapiDispatcher::class)->serve($e);
 
         $files = $this->app->get(FilesInterface::class)->getFiles(
-            $this->app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $this->app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);
@@ -85,7 +85,7 @@ 
         ]);
 
         $files = $app->get(FilesInterface::class)->getFiles(
-            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(0, $files);
@@ -94,7 +94,7 @@ 
         $app->get(SapiDispatcher::class)->serve($e);
 
         $files = $app->get(FilesInterface::class)->getFiles(
-            $app->get(DirectoriesInterface::class)->get('runtime') . '/snapshots/'
+            $app->get(DirectoriesInterface::class)->get('runtime').'/snapshots/'
         );
 
         $this->assertCount(1, $files);

Braces +2 added lines, -1 removed lines

@@ -25,7 +25,8 @@
 
     public function setUp(): void
     {
-        $this->bufferEmitter = new class() implements EmitterInterface {
+        $this->bufferEmitter = new class() implements EmitterInterface
+        {
             public $response;
 
             public function emit(ResponseInterface $response): bool

tests/app/src/Controller/TestController.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
     public function input(InputScope $i)
     {
-        return 'value: ' . $i->withPrefix('section')->getValue('query', 'value');
+        return 'value: '.$i->withPrefix('section')->getValue('query', 'value');
     }
 
     public function error(): void

tests/app/src/Controller/AuthController.php

Spacing +3 added lines, -3 removed lines

@@ -28,7 +28,7 @@ 
 
     public function do(GuardInterface $guard)
     {
-        if (!$guard->allows('do')) {
+        if (!$guard->allows('do')){
             throw new ControllerException("Unauthorized permission 'do'", ControllerException::FORBIDDEN);
         }
 
@@ -37,7 +37,7 @@ 
 
     public function token(AuthContextInterface $authContext)
     {
-        if ($authContext->getToken() !== null) {
+        if ($authContext->getToken() !== null){
             return $authContext->getToken()->getID();
         }
 
@@ -55,7 +55,7 @@ 
 
     public function token2()
     {
-        if ($this->auth->getToken() !== null) {
+        if ($this->auth->getToken() !== null){
             return $this->auth->getToken()->getID();
         }
 

Braces +7 added lines, -3 removed lines

@@ -26,10 +26,12 @@ 
         $this->auth = $auth;
     }
 
-    public function do(GuardInterface $guard)
+    public function do{
+        (GuardInterface $guard)
     {
         if (!$guard->allows('do')) {
             throw new ControllerException("Unauthorized permission 'do'", ControllerException::FORBIDDEN);
+    }
         }
 
         return 'ok';
@@ -37,7 +39,8 @@ 
 
     public function token(AuthContextInterface $authContext)
     {
-        if ($authContext->getToken() !== null) {
+        if ($authContext->getToken() !== null)
+        {
             return $authContext->getToken()->getID();
         }
 
@@ -55,7 +58,8 @@ 
 
     public function token2()
     {
-        if ($this->auth->getToken() !== null) {
+        if ($this->auth->getToken() !== null)
+        {
             return $this->auth->getToken()->getID();
         }
 

tests/Framework/Http/WebsocketsTest.php

Spacing +9 added lines, -9 removed lines

@@ -38,7 +38,7 @@ 
 
     public function testAllowServerAccess(): void
     {
-        $this->init(static function () {
+        $this->init(static function (){
             return true;
         });
 
@@ -50,7 +50,7 @@ 
 
     public function testDenyServerAccess(): void
     {
-        $this->init(static function () {
+        $this->init(static function (){
             return false;
         });
 
@@ -62,7 +62,7 @@ 
 
     public function testServerAccessWithDependency(): void
     {
-        $this->init(static function (ServerRequestInterface $request) {
+        $this->init(static function (ServerRequestInterface $request){
             return $request instanceof ServerRequestInterface;
         });
 
@@ -124,7 +124,7 @@ 
 
     public function testTopicAccessOK(): void
     {
-        $this->init(null, function () {
+        $this->init(null, function (){
             return true;
         });
 
@@ -136,7 +136,7 @@ 
 
     public function testTopicWildcardAccessFail(): void
     {
-        $this->init(null, function () {
+        $this->init(null, function (){
             return true;
         });
 
@@ -151,7 +151,7 @@ 
         $this->init(
             null,
             null,
-            function () {
+            function (){
                 return true;
             }
         );
@@ -167,7 +167,7 @@ 
         $this->init(
             null,
             null,
-            function ($id) {
+            function ($id){
                 return $id === '1';
             }
         );
@@ -183,7 +183,7 @@ 
         $this->init(
             null,
             null,
-            function ($id) {
+            function ($id){
                 return $id === '1';
             }
         );
@@ -204,7 +204,7 @@ 
             'CYCLE_AUTH'                 => true,
             'RR_BROADCAST_PATH'          => '/ws/',
             'WS_SERVER_CALLBACK'         => null,
-            'WS_TOPIC_CALLBACK'          => function (AuthContextInterface $authContext) {
+            'WS_TOPIC_CALLBACK'          => function (AuthContextInterface $authContext){
                 return $authContext->getToken() !== null;
             },
             'WS_TOPIC_WILDCARD_CALLBACK' => null,

Braces +18 added lines, -9 removed lines

@@ -38,7 +38,8 @@ 
 
     public function testAllowServerAccess(): void
     {
-        $this->init(static function () {
+        $this->init(static function ()
+        {
             return true;
         });
 
@@ -50,7 +51,8 @@ 
 
     public function testDenyServerAccess(): void
     {
-        $this->init(static function () {
+        $this->init(static function ()
+        {
             return false;
         });
 
@@ -62,7 +64,8 @@ 
 
     public function testServerAccessWithDependency(): void
     {
-        $this->init(static function (ServerRequestInterface $request) {
+        $this->init(static function (ServerRequestInterface $request)
+        {
             return $request instanceof ServerRequestInterface;
         });
 
@@ -124,7 +127,8 @@ 
 
     public function testTopicAccessOK(): void
     {
-        $this->init(null, function () {
+        $this->init(null, function ()
+        {
             return true;
         });
 
@@ -136,7 +140,8 @@ 
 
     public function testTopicWildcardAccessFail(): void
     {
-        $this->init(null, function () {
+        $this->init(null, function ()
+        {
             return true;
         });
 
@@ -151,7 +156,8 @@ 
         $this->init(
             null,
             null,
-            function () {
+            function ()
+            {
                 return true;
             }
         );
@@ -167,7 +173,8 @@ 
         $this->init(
             null,
             null,
-            function ($id) {
+            function ($id)
+            {
                 return $id === '1';
             }
         );
@@ -183,7 +190,8 @@ 
         $this->init(
             null,
             null,
-            function ($id) {
+            function ($id)
+            {
                 return $id === '1';
             }
         );
@@ -204,7 +212,8 @@ 
             'CYCLE_AUTH'                 => true,
             'RR_BROADCAST_PATH'          => '/ws/',
             'WS_SERVER_CALLBACK'         => null,
-            'WS_TOPIC_CALLBACK'          => function (AuthContextInterface $authContext) {
+            'WS_TOPIC_CALLBACK'          => function (AuthContextInterface $authContext)
+            {
                 return $authContext->getToken() !== null;
             },
             'WS_TOPIC_WILDCARD_CALLBACK' => null,