fix: output for tool command to also contain error output (#22)

stickeeuk · Jan 3, 2023 · ef9585d · ef9585d
1 parent 5895fe7
commit ef9585d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/app/Commands/ToolCommand.php b/app/Commands/ToolCommand.php
@@ -119,7 +119,8 @@ private function process(array $command): array
             }
         } else {
             $taskSuccess = true;
-            $output = $process->getOutput();
+            $output = $process->getErrorOutput();
+            $output .= $process->getOutput();
         }
 
         $this->newLine();

diff --git a/tests/Feature/FixCommandTest.php b/tests/Feature/FixCommandTest.php
@@ -16,12 +16,12 @@
 
 it('works', function () {
     $this->artisan('fix')
-        ->expectsOutputToContain('Fixed all files')
+        ->expectsOutputToContain('stickee')
         ->assertExitCode(0);
 });
 
 it('allows passing options', function () {
-    $this->artisan('fix --dry-run')
-        ->expectsOutputToContain('Checked all files')
+    $this->artisan('fix -- --dry-run --verbose')
+        ->expectsOutputToContain('Legend:')
         ->assertExitCode(0);
 });