spiral / framework

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/app/src/ViewEngine/TestEngine.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
 
     public function compile(string $path, ContextInterface $context): void
     {
-        if ($path == 'custom:error') {
+        if ($path == 'custom:error'){
             throw new EngineException('Unable to compile custom:error');
         }
     }

Braces +2 added lines, -1 removed lines

@@ -31,7 +31,8 @@
 
     public function compile(string $path, ContextInterface $context): void
     {
-        if ($path == 'custom:error') {
+        if ($path == 'custom:error')
+        {
             throw new EngineException('Unable to compile custom:error');
         }
     }

tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -12,4 +12,4 @@
 ini_set('display_errors', '1');
 
 //Composer
-require dirname(__DIR__) . '/vendor/autoload.php';
+require dirname(__DIR__).'/vendor/autoload.php';

src/Filter/InputScope.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      */
     public function getValue(string $source, string $name = null)
     {
-        if (!method_exists($this->input, $source)) {
+        if (!method_exists($this->input, $source)){
             throw new InputException("Undefined input source '{$source}'");
         }
 

Braces +2 added lines, -1 removed lines

@@ -41,7 +41,8 @@
      */
     public function getValue(string $source, string $name = null)
     {
-        if (!method_exists($this->input, $source)) {
+        if (!method_exists($this->input, $source))
+        {
             throw new InputException("Undefined input source '{$source}'");
         }
 

src/Bootloader/Http/RouterBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
     private function route(ServerRequestInterface $request): RouteInterface
     {
         $route = $request->getAttribute(Router::ROUTE_ATTRIBUTE, null);
-        if ($route === null) {
+        if ($route === null){
             throw new ScopeException('Unable to resolve Route, invalid request scope');
         }
 

Braces +2 added lines, -1 removed lines

@@ -57,7 +57,8 @@
     private function route(ServerRequestInterface $request): RouteInterface
     {
         $route = $request->getAttribute(Router::ROUTE_ATTRIBUTE, null);
-        if ($route === null) {
+        if ($route === null)
+        {
             throw new ScopeException('Unable to resolve Route, invalid request scope');
         }
 

src/Bootloader/Http/CookiesBootloader.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
     private function cookieQueue(ServerRequestInterface $request): CookieQueue
     {
         $cookieQueue = $request->getAttribute(CookieQueue::ATTRIBUTE, null);
-        if ($cookieQueue === null) {
+        if ($cookieQueue === null){
             throw new ScopeException('Unable to resolve CookieQueue, invalid request scope');
         }
 

Braces +2 added lines, -1 removed lines

@@ -72,7 +72,8 @@
     private function cookieQueue(ServerRequestInterface $request): CookieQueue
     {
         $cookieQueue = $request->getAttribute(CookieQueue::ATTRIBUTE, null);
-        if ($cookieQueue === null) {
+        if ($cookieQueue === null)
+        {
             throw new ScopeException('Unable to resolve CookieQueue, invalid request scope');
         }
 

tests/Framework/GRPC/DispatcherTest.php

Spacing +13 added lines, -13 removed lines

@@ -22,14 +22,14 @@ 
     public function setUp(): void
     {
         exec('protoc 2>&1', $out);
-        if (strpos(join("\n", $out), '--php_out') === false) {
+        if (strpos(join("\n", $out), '--php_out') === false){
             $this->markTestSkipped('Protoc binary is missing');
         }
 
         parent::setUp();
 
         $fs = new Files();
-        $proto = $fs->normalizePath($this->app->dir('app') . 'proto/service.proto');
+        $proto = $fs->normalizePath($this->app->dir('app').'proto/service.proto');
 
         // protoc can't figure relative paths
         $proto = str_replace('Framework/../', '', $proto);
@@ -38,7 +38,7 @@ 
             'proto' => $proto
         ]);
 
-        file_put_contents($this->app->dir('app') . 'src/Service/EchoService.php', GenerateTest::SERVICE);
+        file_put_contents($this->app->dir('app').'src/Service/EchoService.php', GenerateTest::SERVICE);
     }
 
     public function tearDown(): void
@@ -47,12 +47,12 @@ 
 
         $fs = new Files();
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/Service')) {
-            $fs->deleteDirectory($this->app->dir('app') . 'src/Service');
+        if ($fs->isDirectory($this->app->dir('app').'src/Service')){
+            $fs->deleteDirectory($this->app->dir('app').'src/Service');
         }
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata')) {
-            $fs->deleteDirectory($this->app->dir('app') . 'src/GPBMetadata');
+        if ($fs->isDirectory($this->app->dir('app').'src/GPBMetadata')){
+            $fs->deleteDirectory($this->app->dir('app').'src/GPBMetadata');
         }
     }
 
@@ -78,7 +78,7 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping"
@@ -89,7 +89,7 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('send')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out){
                 $msg = new Message();
                 $msg->mergeFromString($out);
                 $this->assertSame('hello', $msg->getMsg());
@@ -100,7 +100,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })
@@ -120,7 +120,7 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid"
@@ -131,7 +131,7 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out){
                 $this->assertContains('Method `Invalid` not found', $out);
                 return true;
             })
@@ -139,7 +139,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })

Braces +18 added lines, -9 removed lines

@@ -22,7 +22,8 @@ 
     public function setUp(): void
     {
         exec('protoc 2>&1', $out);
-        if (strpos(join("\n", $out), '--php_out') === false) {
+        if (strpos(join("\n", $out), '--php_out') === false)
+        {
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -47,11 +48,13 @@ 
 
         $fs = new Files();
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/Service')) {
+        if ($fs->isDirectory($this->app->dir('app') . 'src/Service'))
+        {
             $fs->deleteDirectory($this->app->dir('app') . 'src/Service');
         }
 
-        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata')) {
+        if ($fs->isDirectory($this->app->dir('app') . 'src/GPBMetadata'))
+        {
             $fs->deleteDirectory($this->app->dir('app') . 'src/GPBMetadata');
         }
     }
@@ -78,7 +81,8 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping"
@@ -89,7 +93,8 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('send')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out)
+            {
                 $msg = new Message();
                 $msg->mergeFromString($out);
                 $this->assertSame('hello', $msg->getMsg());
@@ -100,7 +105,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })
@@ -120,7 +126,8 @@ 
         $msg->setMsg('hello');
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid"
@@ -131,7 +138,8 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out)
+            {
                 $this->assertContains('Method `Invalid` not found', $out);
                 return true;
             })
@@ -139,7 +147,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })

tests/Framework/Jobs/DispatcherTest.php

Spacing +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
         $this->assertNull($this->app->getEnvironment()->get('FIRED'));
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.TestJob"
@@ -69,7 +69,7 @@ 
         $this->app->getContainer()->bind(Worker::class, $w);
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.ErrorJob"
@@ -85,7 +85,7 @@ 
         $w->shouldReceive('receive')->once()->andReturn(null);
 
         $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);
@@ -93,7 +93,7 @@ 
         $this->app->get(JobDispatcher::class)->serve();
 
         $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);

Braces +4 added lines, -2 removed lines

@@ -41,7 +41,8 @@ 
         $this->assertNull($this->app->getEnvironment()->get('FIRED'));
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.TestJob"
@@ -69,7 +70,8 @@ 
         $this->app->getContainer()->bind(Worker::class, $w);
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "id": "1",
                   "job": "spiral.app.job.ErrorJob"

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());