Add long-running failure as well

Luc45 · Luc45 · commit fcd5a1deee4b · 2024-02-08T11:28:21.000-03:00
diff --git a/src/Symfony/Component/Process/Tests/ProcessTest.php b/src/Symfony/Component/Process/Tests/ProcessTest.php
@@ -1553,6 +1553,18 @@ public function testMultipleCallsToProcGetStatus()
         $this->assertSame(0, $process->getExitCode());
     }
 
+    public function testFailingProcessWithMultipleCallsToProcGetStatus()
+    {
+        $process = $this->getProcess('exit 123');
+        $process->start(function () use ($process) {
+            $process->isRunning();
+        });
+        while ($process->isRunning()) {
+            usleep(1000);
+        }
+        $this->assertSame(123, $process->getExitCode());
+    }
+
     /**
      * @group time-sensitive
      */
@@ -1568,9 +1580,12 @@ public function testLongRunningProcessWithMultipleCallsToProcGetStatus()
         $this->assertSame(0, $process->getExitCode());
     }
 
-    public function testFailingProcessWithMultipleCallsToProcGetStatus()
+    /**
+     * @group time-sensitive
+     */
+    public function testLongRunningProcessWithMultipleCallsToProcGetStatusError()
     {
-        $process = $this->getProcess('exit 123');
+        $process = $this->getProcess('sleep 2; echo "failure"; exit 123');
         $process->start(function () use ($process) {
             $process->isRunning();
         });
@@ -1580,7 +1595,6 @@ public function testFailingProcessWithMultipleCallsToProcGetStatus()
         $this->assertSame(123, $process->getExitCode());
     }
 
-
     /**
      * @group transient-on-windows
      */
