STDOUT/STDERR IO streams do not exist in process isolation #1348
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The PHP CLI SAPI does not auto-register the standard IO streams when a script is piped into STDIN (which is the case in process isolation); cf. - https://bugs.php.net/bug.php?id=43283 - http://php.net/manual/en/features.commandline.io-streams.php - https://gist.github.com/sun/d02c242514c8f34bccdb
sun
added a commit
to sun/drupal
that referenced
this pull request
Jul 24, 2014
sun
added a commit
to sun/drupal
that referenced
this pull request
Jul 26, 2014
|
For consistency with recent, related PRs, I'll recreate this one against master, too. In essence, PHPUnit 4.3 == Mature Process Isolation. (unless we're going to backport sooner) |
|
Recreated for master, including regression test: #1366 |
sun
added a commit
to sun/drupal
that referenced
this pull request
Aug 1, 2014
sun
added a commit
to sun/drupal
that referenced
this pull request
Aug 1, 2014
- Fix process isolation infinitely blocks on Windows. sebastianbergmann/phpunit#1340 - Fix standard IO streams are not defined in process isolation. sebastianbergmann/phpunit#1348 - Fix isolated child process leaks into parent process. sebastianbergmann/phpunit#1351
sun
added a commit
to sun/drupal
that referenced
this pull request
Aug 1, 2014
- Fix process isolation infinitely blocks on Windows. sebastianbergmann/phpunit#1340 - Fix standard IO streams are not defined in process isolation. sebastianbergmann/phpunit#1348 - Fix isolated child process leaks into parent process. sebastianbergmann/phpunit#1351
sun
added a commit
to sun/drupal
that referenced
this pull request
Aug 6, 2014
- Fix process isolation infinitely blocks on Windows. sebastianbergmann/phpunit#1340 - Fix standard IO streams are not defined in process isolation. sebastianbergmann/phpunit#1348 - Fix isolated child process leaks into parent process. sebastianbergmann/phpunit#1351
sun
added a commit
to sun/drupal
that referenced
this pull request
Aug 6, 2014
- Fix process isolation infinitely blocks on Windows. sebastianbergmann/phpunit#1340 - Fix standard IO streams are not defined in process isolation. sebastianbergmann/phpunit#1348 - Fix isolated child process leaks into parent process. sebastianbergmann/phpunit#1351
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PHP CLI SAPI does not auto-register the standard IO streams when a script is piped into STDIN (which is the case in process isolation); cf.
https://bugs.php.net/bug.php?id=43283
http://php.net/manual/en/features.commandline.io-streams.php
https://gist.github.com/sun/d02c242514c8f34bccdb
When a test is executed without process isolation, then the
STDOUTandSTDERRstreams are available. But if the test is executed in a separate process, then attempting to access the streams triggers a PHP Notice (undefined constant), followed by a PHP Warning (fopen).A prominent example of affected code is e.g. vfsStream, which is an officially recommended/endorsed library for PHPUnit.
I'll add a regression test shortly.