[11.x] Test Improvements (#56849)

Signed-off-by: Mior Muhammad Zaki <[email protected]>

Author: crynobone

composer.json

@@ -111,7 +111,7 @@
         "league/flysystem-read-only": "^3.25.1",
         "league/flysystem-sftp-v3": "^3.25.1",
         "mockery/mockery": "^1.6.10",
-        "orchestra/testbench-core": "^9.16.0",
+        "orchestra/testbench-core": "^9.16.1",
         "pda/pheanstalk": "^5.0.6",
         "php-http/discovery": "^1.15",
         "phpstan/phpstan": "^2.0",

src/Illuminate/Testing/PendingCommand.php

@@ -17,6 +17,7 @@
 use Symfony\Component\Console\Helper\Table;
 use Symfony\Component\Console\Input\ArrayInput;
 use Symfony\Component\Console\Output\BufferedOutput;
+use Symfony\Component\Console\Output\ConsoleOutput;
 use Symfony\Component\Console\Question\ChoiceQuestion;
 
 class PendingCommand
@@ -359,6 +360,27 @@ public function run()
         return $exitCode;
     }
 
+    /**
+     * Debug the command.
+     *
+     * @return never
+     */
+    public function dd()
+    {
+        $consoleOutput = new OutputStyle(new ArrayInput($this->parameters), new ConsoleOutput());
+        $exitCode = $this->app->make(Kernel::class)->call($this->command, $this->parameters, $consoleOutput);
+
+        $streamOutput = $consoleOutput->getOutput()->getStream();
+        $output = stream_get_contents($streamOutput);
+
+        fclose($streamOutput);
+
+        dd([
+            'exitCode' => $exitCode,
+            'output' => $output,
+        ]);
+    }
+
     /**
      * Determine if expected questions / choices / outputs are fulfilled.
      *

tests/Filesystem/FilesystemTest.php

@@ -14,6 +14,8 @@
 use PHPUnit\Framework\TestCase;
 use SplFileInfo;
 
+use function Orchestra\Testbench\terminate;
+
 class FilesystemTest extends TestCase
 {
     private static $tempDir;
@@ -547,7 +549,7 @@ public function testSharedGet()
                 $files->put(self::$tempDir.'/file.txt', $content, true);
                 $read = $files->get(self::$tempDir.'/file.txt', true);
 
-                exit(strlen($read) === strlen($content) ? 1 : 0);
+                terminate($this, strlen($read) === strlen($content) ? 1 : 0);
             }
         }
 

tests/Testing/Console/RouteListCommandTest.php

@@ -8,7 +8,6 @@
 use Illuminate\Foundation\Testing\Concerns\InteractsWithDeprecationHandling;
 use Illuminate\Http\RedirectResponse;
 use Illuminate\Routing\Controller;
-use Illuminate\Support\Facades\Facade;
 use Orchestra\Testbench\TestCase;
 
 class RouteListCommandTest extends TestCase
@@ -123,6 +122,10 @@ public function testRouteCanBeFilteredByAction()
     {
         $this->withoutDeprecationHandling();
 
+        RouteListCommand::resolveTerminalWidthUsing(function () {
+            return 82;
+        });
+
         $this->router->get('/', function () {
             //
         });
@@ -135,7 +138,7 @@ public function testRouteCanBeFilteredByAction()
                 '  GET|HEAD       foo/{user} Illuminate\Tests\Testing\Console\FooController@show'
             )->expectsOutput('')
             ->expectsOutput(
-                '                                                  Showing [1] routes'
+                '                                                              Showing [1] routes'
             )
             ->expectsOutput('');
     }
@@ -156,15 +159,6 @@ public function testDisplayRoutesExceptVendor()
             ->expectsOutput('                                                  Showing [3] routes')
             ->expectsOutput('');
     }
-
-    protected function tearDown(): void
-    {
-        parent::tearDown();
-
-        Facade::setFacadeApplication(null);
-
-        RouteListCommand::resolveTerminalWidthUsing(null);
-    }
 }
 
 class FooController extends Controller