spiral / framework

src/Command/Translator/ExportCommand.php

Spacing +8 added lines, -8 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function perform(TranslatorConfig $config, CatalogueManager $manager)
     {
-        if (!$config->hasDumper($this->option('dumper'))) {
+        if (!$config->hasDumper($this->option('dumper'))){
             $this->writeln("<fg=red>Undefined dumper '{$this->option('dumper')}'.</fg=red>");
 
             return;
@@ -51,7 +51,7 @@ 
             $manager->get($this->argument('locale'))
         );
 
-        if ($this->isVerbose() && !empty($mc->getDomains())) {
+        if ($this->isVerbose() && !empty($mc->getDomains())){
             $this->sprintf(
                 "<info>Exporting domain(s):</info> %s\n",
                 join(',', $mc->getDomains())
@@ -66,8 +66,8 @@ 
             'xliff_version'  => '2.0' // forcing default version for xliff dumper only
         ]);
 
-        $this->writeln("Export successfully completed using <info>" . get_class($dumper) . "</info>");
-        $this->writeln("Output: <comment>" . realpath($this->argument('path')) . "</comment>");
+        $this->writeln("Export successfully completed using <info>".get_class($dumper)."</info>");
+        $this->writeln("Output: <comment>".realpath($this->argument('path'))."</comment>");
     }
 
     /**
@@ -86,10 +86,10 @@ 
             $catalogue->getData()
         );
 
-        if ($this->option('fallback')) {
-            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages) {
-                foreach ($messages as $id => $message) {
-                    if (!$messageCatalogue->defines($id, $domain)) {
+        if ($this->option('fallback')){
+            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages){
+                foreach ($messages as $id => $message){
+                    if (!$messageCatalogue->defines($id, $domain)){
                         $messageCatalogue->set($id, $message, $domain);
                     }
                 }

Braces +12 added lines, -6 removed lines

@@ -39,7 +39,8 @@ 
      */
     public function perform(TranslatorConfig $config, CatalogueManager $manager)
     {
-        if (!$config->hasDumper($this->option('dumper'))) {
+        if (!$config->hasDumper($this->option('dumper')))
+        {
             $this->writeln("<fg=red>Undefined dumper '{$this->option('dumper')}'.</fg=red>");
 
             return;
@@ -51,7 +52,8 @@ 
             $manager->get($this->argument('locale'))
         );
 
-        if ($this->isVerbose() && !empty($mc->getDomains())) {
+        if ($this->isVerbose() && !empty($mc->getDomains()))
+        {
             $this->sprintf(
                 "<info>Exporting domain(s):</info> %s\n",
                 join(',', $mc->getDomains())
@@ -86,10 +88,14 @@ 
             $catalogue->getData()
         );
 
-        if ($this->option('fallback')) {
-            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages) {
-                foreach ($messages as $id => $message) {
-                    if (!$messageCatalogue->defines($id, $domain)) {
+        if ($this->option('fallback'))
+        {
+            foreach ($manager->get($config->getFallbackLocale())->getData() as $domain => $messages)
+            {
+                foreach ($messages as $id => $message)
+                {
+                    if (!$messageCatalogue->defines($id, $domain))
+                    {
                         $messageCatalogue->set($id, $message, $domain);
                     }
                 }

tests/app/src/Job/ErrorJob.php

Braces +3 added lines, -1 removed lines

@@ -12,8 +12,10 @@
 
 class ErrorJob extends AbstractJob
 {
-    public function do()
+    public function do{
+        ()
     {
         throw new \ErrorException("bad job");
     }
+    }
 }

tests/app/src/Job/TestJob.php

Braces +3 added lines, -1 removed lines

@@ -13,8 +13,10 @@
 
 class TestJob extends AbstractJob
 {
-    public function do(EnvironmentInterface $env)
+    public function do{
+        (EnvironmentInterface $env)
     {
         $env->set('FIRED', true);
     }
+    }
 }

tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

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

tests/Framework/Encrypter/KeyCommandTest.php

Spacing +9 added lines, -9 removed lines

@@ -23,7 +23,7 @@ 
     public function testMountFileNotFound()
     {
         $out = $this->runCommand('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
 
         $this->assertContains('Unable to find', $out);
@@ -31,18 +31,18 @@ 
 
     public function testReplace()
     {
-        file_put_contents(__DIR__ . '/.env', '{encrypt-key}');
+        file_put_contents(__DIR__.'/.env', '{encrypt-key}');
 
         $out = $this->runCommand('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
 
         $this->assertContains('key has been updated', $out);
 
-        $body = file_get_contents(__DIR__ . '/.env');
+        $body = file_get_contents(__DIR__.'/.env');
         $this->assertContains($body, $out);
 
-        unlink(__DIR__ . '/.env');
+        unlink(__DIR__.'/.env');
     }
 
     public function testReplaceCurrent()
@@ -53,18 +53,18 @@ 
             'ENCRYPTER_KEY' => $key
         ]);
 
-        file_put_contents(__DIR__ . '/.env', $key);
+        file_put_contents(__DIR__.'/.env', $key);
 
         $out = $app->console()->run('encrypt:key', [
-            '-m' => __DIR__ . '/.env'
+            '-m' => __DIR__.'/.env'
         ]);
         $out = $out->getOutput()->fetch();
 
         $this->assertContains('key has been updated', $out);
 
-        $body = file_get_contents(__DIR__ . '/.env');
+        $body = file_get_contents(__DIR__.'/.env');
         $this->assertContains($body, $out);
 
-        unlink(__DIR__ . '/.env');
+        unlink(__DIR__.'/.env');
     }
 }

tests/Framework/GRPC/DispatcherTest.php

Spacing +13 added lines, -13 removed lines

@@ -21,14 +21,14 @@ 
     public function setUp()
     {
         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);
@@ -37,7 +37,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()
@@ -46,12 +46,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');
         }
     }
 
@@ -77,7 +77,7 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping"               
@@ -88,7 +88,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());
@@ -99,7 +99,7 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = null;
                 return true;
             })
@@ -119,7 +119,7 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context){
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid"               
@@ -130,7 +130,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;
             })
@@ -138,7 +138,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

@@ -21,7 +21,8 @@ 
     public function setUp()
     {
         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');
         }
 
@@ -46,11 +47,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');
         }
     }
@@ -77,7 +80,8 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Ping"               
@@ -88,7 +92,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());
@@ -99,7 +104,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })
@@ -119,7 +125,8 @@ 
         $msg->setMsg("hello");
 
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = '{
                   "service": "service.Echo",
                   "method": "Invalid"               
@@ -130,7 +137,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;
             })
@@ -138,7 +146,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })

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/I18n/ExportTest.php

Spacing +4 added lines, -4 removed lines

@@ -15,7 +15,7 @@ 
 {
     public function testReset()
     {
-        $this->assertFileNotExists(__DIR__ . '/messages.ru.php');
+        $this->assertFileNotExists(__DIR__.'/messages.ru.php');
 
         $this->runCommandDebug('i18n:index');
         $this->runCommandDebug('configure');
@@ -25,9 +25,9 @@ 
             '--fallback' => 'en',
         ]);
 
-        $this->assertFileExists(__DIR__ . '/messages.ru.php');
+        $this->assertFileExists(__DIR__.'/messages.ru.php');
 
-        $exp = (require __DIR__ . '/messages.ru.php');
+        $exp = (require __DIR__.'/messages.ru.php');
 
         $this->assertArrayHasKey('World', $exp);
         $this->assertSame('Мир', $exp['World']);
@@ -35,6 +35,6 @@ 
         $this->assertArrayHasKey('%s unit|%s units', $exp);
         $this->assertArrayHasKey('This value is required.', $exp);
 
-        unlink(__DIR__ . '/messages.ru.php');
+        unlink(__DIR__.'/messages.ru.php');
     }
 }

tests/Framework/Http/SapiTest.php

Spacing +4 added lines, -4 removed lines

@@ -57,7 +57,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);
@@ -66,7 +66,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);
@@ -83,7 +83,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);
@@ -92,7 +92,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);