Skip to content

Commit 1757c9c

Browse files
minor symfony#28192 [PhpUnitBridge] Add missing stderr redirection (greg0ire)
This PR was merged into the 3.4 branch. Discussion ---------- [PhpUnitBridge] Add missing stderr redirection | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a This seems to have been overlooked in 6c0e6af, and results in the test suite being polluted on machines where phpdbg is not installed. I updated the code to mimic other occurences of exec in this file. Commits ------- d67b86b Add missing stderr redirection
2 parents 37c39fb + d67b86b commit 1757c9c

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Bridge/PhpUnit/Tests/CoverageListenerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ public function test()
1616
$this->markTestSkipped('This test cannot be run on HHVM.');
1717
}
1818

19-
exec('type phpdbg', $output, $returnCode);
19+
exec('type phpdbg 2> /dev/null', $output, $returnCode);
2020

2121
if (\PHP_VERSION_ID >= 70000 && 0 === $returnCode) {
2222
$php = 'phpdbg -qrr';

0 commit comments

Comments
 (0)