minor #57767 [Process] Fix ProcessTest - testIgnoringSignal for local (thibaut22200)

nicolas-grekas · nicolas-grekas · commit 1dbc465798da · 2024-07-26T14:44:57.000+02:00
This PR was merged into the 7.1 branch. Discussion ---------- [Process] Fix ProcessTest - testIgnoringSignal for local | Q | A | ------------- | --- | Branch? | 7.1 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | Fix Process test (testIgnoringSignal) | License | MIT Fix ProcessTest : testIgnoringSignal(). Tests on 2 independant env. The test was failed. Have to pass an array instead of string in the getProcess() Commits ------- 9567835 Fix ProcessTest - testIgnoringSignal for local
diff --git a/src/Symfony/Component/Process/Tests/ProcessTest.php b/src/Symfony/Component/Process/Tests/ProcessTest.php
@@ -1669,7 +1669,7 @@ public function testIgnoringSignal()
             $this->markTestSkipped('pnctl extension is required.');
         }
 
-        $process = $this->getProcess('sleep 10');
+        $process = $this->getProcess(['sleep', '10']);
         $process->setIgnoredSignals([\SIGTERM]);
 
         $process->start();
@@ -1685,7 +1685,7 @@ public function testNotIgnoringSignal()
             $this->markTestSkipped('pnctl extension is required.');
         }
 
-        $process = $this->getProcess('sleep 10');
+        $process = $this->getProcess(['sleep', '10']);
 
         $process->start();
         $process->stop(timeout: 0.2);

Original file line number	Diff line number	Diff line change
`@@ -1669,7 +1669,7 @@ public function testIgnoringSignal()`
`1669`	`1669`	`$this->markTestSkipped('pnctl extension is required.');`
`1670`	`1670`	`}`
`1671`	`1671`
`1672`		`- $process = $this->getProcess('sleep 10');`
	`1672`	`+ $process = $this->getProcess(['sleep', '10']);`
`1673`	`1673`	`$process->setIgnoredSignals([\SIGTERM]);`
`1674`	`1674`
`1675`	`1675`	`$process->start();`
`@@ -1685,7 +1685,7 @@ public function testNotIgnoringSignal()`
`1685`	`1685`	`$this->markTestSkipped('pnctl extension is required.');`
`1686`	`1686`	`}`
`1687`	`1687`
`1688`		`- $process = $this->getProcess('sleep 10');`
	`1688`	`+ $process = $this->getProcess(['sleep', '10']);`
`1689`	`1689`
`1690`	`1690`	`$process->start();`
`1691`	`1691`	`$process->stop(timeout: 0.2);`