style: cs fix [skip ci][ci skip]

Author: adhocore

src/Helper/Shell.php

@@ -51,7 +51,7 @@ class Shell
 
     const DEFAULT_STDOUT_WIN = ['pipe', 'w'];
     const DEFAULT_STDOUT_NIX = ['pipe', 'w'];
-    
+
     const DEFAULT_STDERR_WIN = ['pipe', 'w'];
     const DEFAULT_STDERR_NIX = ['pipe', 'w'];
 
@@ -119,6 +119,7 @@ protected function prepareDescriptors(?array $stdin = null, ?array $stdout = nul
         if (!$stderr) {
             $stderr = $win ? self::DEFAULT_STDERR_WIN : self::DEFAULT_STDERR_NIX;
         }
+
         return [
             self::STDIN_DESCRIPTOR_KEY  => $stdin,
             self::STDOUT_DESCRIPTOR_KEY => $stdout,
@@ -132,6 +133,7 @@ protected function isWindows(): bool
         if (defined('PHP_OS')) {
             return 'WIN' === strtoupper(substr(PHP_OS, 0, 3)); // May be 'WINNT' or 'WIN32' or 'Windows'
         }
+
         return '\\' === DIRECTORY_SEPARATOR; // Fallback - Less reliable (Windows 7...)
     }
 
@@ -208,16 +210,17 @@ public function setOptions(
 
         return $this;
     }
-    
+
     /**
      * execute
      * Execute the command with optional stdin, stdout and stderr which override the defaults
-     * If async is set to true, the process will be executed in the background
-     * 
-     * @param  bool $async - default false
-     * @param  ?array $stdin - default null (loads default descriptor)
-     * @param  ?array $stdout - default null (loads default descriptor)
-     * @param  ?array $stderr - default null (loads default descriptor)
+     * If async is set to true, the process will be executed in the background.
+     *
+     * @param bool   $async  - default false
+     * @param ?array $stdin  - default null (loads default descriptor)
+     * @param ?array $stdout - default null (loads default descriptor)
+     * @param ?array $stderr - default null (loads default descriptor)
+     *
      * @return self
      */
     public function execute(bool $async = false, ?array $stdin = null, ?array $stdout = null, ?array $stderr = null): self
@@ -264,6 +267,7 @@ private function setOutputStreamNonBlocking(): bool
         if (!is_resource($this->pipes[self::STDOUT_DESCRIPTOR_KEY])) {
             return false;
         }
+
         return stream_set_blocking($this->pipes[self::STDOUT_DESCRIPTOR_KEY], false);
     }
 
@@ -278,6 +282,7 @@ public function getOutput(): string
         if (!is_resource($this->pipes[self::STDOUT_DESCRIPTOR_KEY])) {
             return '';
         }
+
         return stream_get_contents($this->pipes[self::STDOUT_DESCRIPTOR_KEY]);
     }
 
@@ -286,6 +291,7 @@ public function getErrorOutput(): string
         if (!is_resource($this->pipes[self::STDERR_DESCRIPTOR_KEY])) {
             return '';
         }
+
         return stream_get_contents($this->pipes[self::STDERR_DESCRIPTOR_KEY]);
     }
 

tests/Helper/ShellTest.php

@@ -24,7 +24,7 @@ public function test_get_output()
 
         $shell->execute();
 
-        $this->assertSame("hello", trim($shell->getOutput())); // trim to remove trailing newline which is OS dependent
+        $this->assertSame('hello', trim($shell->getOutput())); // trim to remove trailing newline which is OS dependent
         $this->assertSame(0, $shell->getExitCode());
     }
 

tests/Output/ColorTest.php

@@ -53,12 +53,12 @@ public function test_colors()
         $c = new Color;
 
         // We use PHP_EOL here because it is platform dependent and eol tag will be replaced by it.
-        $this->assertSame(PHP_EOL."abc".PHP_EOL, $c->colors('<eol>abc</eol>'));
+        $this->assertSame(PHP_EOL . 'abc' . PHP_EOL, $c->colors('<eol>abc</eol>'));
         $this->assertSame("\033[0;31mRed\033[0m", $c->colors('<red>Red</end>'));
-        $this->assertSame("\033[1;31mBoldRed".PHP_EOL."\033[0m", $c->colors('<boldRed>BoldRed<eol/></end>'));
-        $this->assertSame("\033[0;36;42mBgGreenCyan\033[0m".PHP_EOL, $c->colors('<bgGreenCyan>BgGreenCyan</end><eol>'));
+        $this->assertSame("\033[1;31mBoldRed" . PHP_EOL . "\033[0m", $c->colors('<boldRed>BoldRed<eol/></end>'));
+        $this->assertSame("\033[0;36;42mBgGreenCyan\033[0m" . PHP_EOL, $c->colors('<bgGreenCyan>BgGreenCyan</end><eol>'));
         $this->assertSame(
-            "\033[0;31mRed\033[0m".PHP_EOL."Normal".PHP_EOL."\033[1;37mBOLD\033[0m",
+            "\033[0;31mRed\033[0m" . PHP_EOL . 'Normal' . PHP_EOL . "\033[1;37mBOLD\033[0m",
             $c->colors("<red>Red</end>\r\nNormal\n<bold>BOLD</end>")
         );
     }