Skip to content

Commit da357d8

Browse files
minor #18519 [Process] Consistently use getProcess() in tests (nicolas-grekas)
This PR was merged into the 2.7 branch. Discussion ---------- [Process] Consistently use getProcess() in tests | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Commits ------- ce13fe7 [Process] Consistently use getProcess() in tests
2 parents 689f27b + ce13fe7 commit da357d8

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Component/Process/Tests/ProcessTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1181,7 +1181,7 @@ public function pipesCodeProvider()
11811181
* @dataProvider provideVariousIncrementals
11821182
*/
11831183
public function testIncrementalOutputDoesNotRequireAnotherCall($stream, $method) {
1184-
$process = new Process(self::$phpBin.' -r '.escapeshellarg('$n = 0; while ($n < 3) { file_put_contents(\''.$stream.'\', $n, 1); $n++; usleep(1000); }'), null, null, null, null);
1184+
$process = $this->getProcess(self::$phpBin.' -r '.escapeshellarg('$n = 0; while ($n < 3) { file_put_contents(\''.$stream.'\', $n, 1); $n++; usleep(1000); }'), null, null, null, null);
11851185
$process->start();
11861186
$result = '';
11871187
$limit = microtime(true) + 3;

0 commit comments

Comments
 (0)