spiral / framework

tests/Framework/GRPC/DispatcherTest.php

Spacing +13 added lines, -13 removed lines

@@ -23,7 +23,7 @@ 
     public function setUp(): void
     {
         exec('protoc 2>&1', $out);
-        if (strpos(implode("\n", $out), '--php_out') === false) {
+        if (strpos(implode("\n", $out), '--php_out') === false){
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -32,10 +32,10 @@ 
         $fs = new Files();
 
         $this->runCommandDebug('grpc:generate', [
-            'proto' => realpath($fs->normalizePath($this->app->dir('app') . 'proto/service.proto'))
+            'proto' => realpath($fs->normalizePath($this->app->dir('app').'proto/service.proto'))
         ]);
 
-        $output = $this->app->dir('app') . 'src/Service/EchoService.php';
+        $output = $this->app->dir('app').'src/Service/EchoService.php';
         file_put_contents($output, GenerateTest::SERVICE);
     }
 
@@ -45,12 +45,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');
         }
     }
 
@@ -76,7 +76,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());
@@ -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",
@@ -132,7 +132,7 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out){
                 $this->assertStringContainsString('Method `Invalid` not found', $out);
                 return true;
             })
@@ -140,7 +140,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

@@ -23,7 +23,8 @@ 
     public function setUp(): void
     {
         exec('protoc 2>&1', $out);
-        if (strpos(implode("\n", $out), '--php_out') === false) {
+        if (strpos(implode("\n", $out), '--php_out') === false)
+        {
             $this->markTestSkipped('Protoc binary is missing');
         }
 
@@ -45,11 +46,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');
         }
     }
@@ -76,7 +79,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());
@@ -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",
@@ -132,7 +139,8 @@ 
         )->andReturn($msg->serializeToString());
 
         $w->shouldReceive('error')->once()->with(
-            \Mockery::on(function ($out) {
+            \Mockery::on(function ($out)
+            {
                 $this->assertStringContainsString('Method `Invalid` not found', $out);
                 return true;
             })
@@ -140,7 +148,8 @@ 
 
         // one command only
         $w->shouldReceive('receive')->once()->with(
-            \Mockery::on(function (&$context) {
+            \Mockery::on(function (&$context)
+            {
                 $context = null;
                 return true;
             })

src/Config/src/Loader/DirectoryLoader.php

Spacing +7 added lines, -7 removed lines

@@ -46,9 +46,9 @@ 
      */
     public function has(string $section): bool
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class){
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (file_exists($filename)) {
+            if (file_exists($filename)){
                 return true;
             }
         }
@@ -61,15 +61,15 @@ 
      */
     public function load(string $section): array
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class){
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (!file_exists($filename)) {
+            if (!file_exists($filename)){
                 continue;
             }
 
-            try {
+            try{
                 return $this->getLoader($extension)->loadFile($section, $filename);
-            } catch (LoaderException $e) {
+            }catch (LoaderException $e){
                 throw new LoaderException("Unable to load config `{$section}`.", $e->getCode(), $e);
             }
         }
@@ -83,7 +83,7 @@ 
      */
     private function getLoader(string $extension): FileLoaderInterface
     {
-        if (isset($this->loaders[$extension])) {
+        if (isset($this->loaders[$extension])){
             return $this->loaders[$extension];
         }
 

Braces +15 added lines, -7 removed lines

@@ -46,9 +46,11 @@ 
      */
     public function has(string $section): bool
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class)
+        {
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (file_exists($filename)) {
+            if (file_exists($filename))
+            {
                 return true;
             }
         }
@@ -61,15 +63,20 @@ 
      */
     public function load(string $section): array
     {
-        foreach (static::LOADERS as $extension => $class) {
+        foreach (static::LOADERS as $extension => $class)
+        {
             $filename = sprintf('%s/%s.%s', $this->directory, $section, $extension);
-            if (!file_exists($filename)) {
+            if (!file_exists($filename))
+            {
                 continue;
             }
 
-            try {
+            try
+            {
                 return $this->getLoader($extension)->loadFile($section, $filename);
-            } catch (LoaderException $e) {
+            }
+            catch (LoaderException $e)
+            {
                 throw new LoaderException("Unable to load config `{$section}`.", $e->getCode(), $e);
             }
         }
@@ -83,7 +90,8 @@ 
      */
     private function getLoader(string $extension): FileLoaderInterface
     {
-        if (isset($this->loaders[$extension])) {
+        if (isset($this->loaders[$extension]))
+        {
             return $this->loaders[$extension];
         }
 

src/Config/src/Loader/PhpLoader.php

Spacing +2 added lines, -2 removed lines

@@ -36,11 +36,11 @@
      */
     public function loadFile(string $section, string $filename): array
     {
-        try {
+        try{
             return ContainerScope::runScope($this->container, function () use ($filename) {
                 return (require $filename);
             });
-        } catch (\Throwable $e) {
+        }catch (\Throwable $e){
             throw new LoaderException($e->getMessage(), $e->getCode(), $e);
         }
     }

Braces +7 added lines, -3 removed lines

@@ -36,11 +36,15 @@
      */
     public function loadFile(string $section, string $filename): array
     {
-        try {
-            return ContainerScope::runScope($this->container, function () use ($filename) {
+        try
+        {
+            return ContainerScope::runScope($this->container, function () use ($filename)
+            {
                 return (require $filename);
             });
-        } catch (\Throwable $e) {
+        }
+        catch (\Throwable $e)
+        {
             throw new LoaderException($e->getMessage(), $e->getCode(), $e);
         }
     }

src/Config/src/Loader/JsonLoader.php

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         $content = file_get_contents($filename);
         $data = json_decode($content, true);
 
-        if (is_null($data)) {
+        if (is_null($data)){
             throw new LoaderException(json_last_error_msg(), json_last_error());
         }
 

Braces +2 added lines, -1 removed lines

@@ -23,7 +23,8 @@
         $content = file_get_contents($filename);
         $data = json_decode($content, true);
 
-        if (is_null($data)) {
+        if (is_null($data))
+        {
             throw new LoaderException(json_last_error_msg(), json_last_error());
         }
 

src/Config/src/ConfigManager.php

Spacing +11 added lines, -11 removed lines

@@ -69,11 +69,11 @@ 
      */
     public function setDefaults(string $section, array $data): void
     {
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section])){
             throw new ConfiguratorException("Unable to set default config `{$section}` more than once.");
         }
 
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section])){
             throw new ConfigDeliveredException("Unable to set default config `{$section}`, config has been loaded.");
         }
 
@@ -85,8 +85,8 @@ 
      */
     public function modify(string $section, PatchInterface $patch): array
     {
-        if (isset($this->instances[$section])) {
-            if ($this->strict) {
+        if (isset($this->instances[$section])){
+            if ($this->strict){
                 throw new ConfigDeliveredException(
                     "Unable to patch config `{$section}`, config object has already been delivered."
                 );
@@ -97,9 +97,9 @@ 
 
         $data = $this->getConfig($section);
 
-        try {
+        try{
             return $this->data[$section] = $patch->patch($data);
-        } catch (PatchException $e) {
+        }catch (PatchException $e){
             throw new PatchException("Unable to modify config `{$section}`.", $e->getCode(), $e);
         }
     }
@@ -109,18 +109,18 @@ 
      */
     public function getConfig(string $section = null): array
     {
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section])){
             return $this->data[$section];
         }
 
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section])){
             $data = [];
-            if ($this->loader->has($section)) {
+            if ($this->loader->has($section)){
                 $data = $this->loader->load($section);
             }
 
             $data = array_merge($this->defaults[$section], $data);
-        } else {
+        }else{
             $data = $this->loader->load($section);
         }
 
@@ -133,7 +133,7 @@ 
     public function createInjection(\ReflectionClass $class, string $context = null)
     {
         $config = $class->getConstant('CONFIG');
-        if (isset($this->instances[$config])) {
+        if (isset($this->instances[$config])){
             return $this->instances[$config];
         }
 

Braces +24 added lines, -11 removed lines

@@ -69,11 +69,13 @@ 
      */
     public function setDefaults(string $section, array $data): void
     {
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section]))
+        {
             throw new ConfiguratorException("Unable to set default config `{$section}` more than once.");
         }
 
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section]))
+        {
             throw new ConfigDeliveredException("Unable to set default config `{$section}`, config has been loaded.");
         }
 
@@ -85,8 +87,10 @@ 
      */
     public function modify(string $section, PatchInterface $patch): array
     {
-        if (isset($this->instances[$section])) {
-            if ($this->strict) {
+        if (isset($this->instances[$section]))
+        {
+            if ($this->strict)
+            {
                 throw new ConfigDeliveredException(
                     "Unable to patch config `{$section}`, config object has already been delivered."
                 );
@@ -97,9 +101,12 @@ 
 
         $data = $this->getConfig($section);
 
-        try {
+        try
+        {
             return $this->data[$section] = $patch->patch($data);
-        } catch (PatchException $e) {
+        }
+        catch (PatchException $e)
+        {
             throw new PatchException("Unable to modify config `{$section}`.", $e->getCode(), $e);
         }
     }
@@ -109,18 +116,23 @@ 
      */
     public function getConfig(string $section = null): array
     {
-        if (isset($this->data[$section])) {
+        if (isset($this->data[$section]))
+        {
             return $this->data[$section];
         }
 
-        if (isset($this->defaults[$section])) {
+        if (isset($this->defaults[$section]))
+        {
             $data = [];
-            if ($this->loader->has($section)) {
+            if ($this->loader->has($section))
+            {
                 $data = $this->loader->load($section);
             }
 
             $data = array_merge($this->defaults[$section], $data);
-        } else {
+        }
+        else
+        {
             $data = $this->loader->load($section);
         }
 
@@ -133,7 +145,8 @@ 
     public function createInjection(\ReflectionClass $class, string $context = null)
     {
         $config = $class->getConstant('CONFIG');
-        if (isset($this->instances[$config])) {
+        if (isset($this->instances[$config]))
+        {
             return $this->instances[$config];
         }
 

src/Config/src/Patch/Set.php

Spacing +2 added lines, -2 removed lines

@@ -45,10 +45,10 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try{
             $target = &$this->dotGet($config, $this->key);
             $target = $this->value;
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

Braces +5 added lines, -2 removed lines

@@ -45,10 +45,13 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $target = &$this->dotGet($config, $this->key);
             $target = $this->value;
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

src/Config/src/Patch/Delete.php

Spacing +6 added lines, -6 removed lines

@@ -45,20 +45,20 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try{
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null){
                 unset($target[$this->key]);
-            } else {
-                foreach ($target as $key => $value) {
-                    if ($value === $this->value) {
+            }else{
+                foreach ($target as $key => $value){
+                    if ($value === $this->value){
                         unset($target[$key]);
                         break;
                     }
                 }
             }
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             // doing nothing when section not found
         }
 

Braces +14 added lines, -6 removed lines

@@ -45,20 +45,28 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null)
+            {
                 unset($target[$this->key]);
-            } else {
-                foreach ($target as $key => $value) {
-                    if ($value === $this->value) {
+            }
+            else
+            {
+                foreach ($target as $key => $value)
+                {
+                    if ($value === $this->value)
+                    {
                         unset($target[$key]);
                         break;
                     }
                 }
             }
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             // doing nothing when section not found
         }
 

src/Config/src/Patch/Group.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
      */
     public function patch(array $config): array
     {
-        foreach ($this->patches as $patch) {
+        foreach ($this->patches as $patch){
             $config = $patch->patch($config);
         }
 

Braces +2 added lines, -1 removed lines

@@ -32,7 +32,8 @@
      */
     public function patch(array $config): array
     {
-        foreach ($this->patches as $patch) {
+        foreach ($this->patches as $patch)
+        {
             $config = $patch->patch($config);
         }
 

src/Config/src/Patch/Append.php

Spacing +4 added lines, -4 removed lines

@@ -46,15 +46,15 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try{
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null){
                 $target[$this->key] = $this->value;
-            } else {
+            }else{
                 $target[] = $this->value;
             }
-        } catch (DotNotFoundException $e) {
+        }catch (DotNotFoundException $e){
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }
 

Braces +10 added lines, -4 removed lines

@@ -46,15 +46,21 @@
      */
     public function patch(array $config): array
     {
-        try {
+        try
+        {
             $target = &$this->dotGet($config, $this->position);
 
-            if ($this->key !== null) {
+            if ($this->key !== null)
+            {
                 $target[$this->key] = $this->value;
-            } else {
+            }
+            else
+            {
                 $target[] = $this->value;
             }
-        } catch (DotNotFoundException $e) {
+        }
+        catch (DotNotFoundException $e)
+        {
             throw new PatchException($e->getMessage(), $e->getCode(), $e);
         }