[Process] Fix test case

symfony · Aug 1, 2023 · ce9850f · ce9850f
1 parent 1a44dc3
commit ce9850f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/Tests/ErrorProcessInitiator.php b/Tests/ErrorProcessInitiator.php
@@ -14,12 +14,12 @@
 use Symfony\Component\Process\Exception\ProcessTimedOutException;
 use Symfony\Component\Process\Process;
 
-require \dirname(__DIR__).'/vendor/autoload.php';
+require is_file(\dirname(__DIR__).'/vendor/autoload.php') ? \dirname(__DIR__).'/vendor/autoload.php' : \dirname(__DIR__, 5).'/vendor/autoload.php';
 
 ['e' => $php] = getopt('e:') + ['e' => 'php'];
 
 try {
-    $process = new Process("exec $php -r \"echo 'ready'; trigger_error('error', E_USER_ERROR);\"");
+    $process = new Process([$php, '-r', "echo 'ready'; trigger_error('error', E_USER_ERROR);"]);
     $process->start();
     $process->setTimeout(0.5);
     while (!str_contains($process->getOutput(), 'ready')) {

diff --git a/Tests/ProcessTest.php b/Tests/ProcessTest.php
@@ -1523,6 +1523,10 @@ public function testWaitStoppedDeadProcess()
         $process->setTimeout(2);
         $process->wait();
         $this->assertFalse($process->isRunning());
+
+        if ('\\' !== \DIRECTORY_SEPARATOR) {
+            $this->assertSame(0, $process->getExitCode());
+        }
     }
 
     public function testEnvCaseInsensitiveOnWindows()