New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add .all
property with interleaved stdout and stderr
#171
Commits on Jan 20, 2019
-
Added result.all intermixed stdout/stderr (concat for sync)
Added test cases for result.all intermixed output. Added fixtures/noop-132 which should output the sequence '123' but due to the unpredictable nature of the stdout/stderr it outputs '132' as the most common cases I've tried, as listed on noop-132, a different number of technique to make it predictable. On execa.sync, as it's using spawnSync internally, I couldn't get the stream and merge them. As such the output of result.all is just a concatenation of stdout + stderr. This makes the result.all in different use case (ex. async vs. sync) unreliable for string comparison, but at least useful for logging or checking on the output.
Commits on Mar 7, 2019
-
Removed
all
stream from execa.sync(); fix creating mixed stream for…… nothing - execa.sync() cannot provide a true interleaved `all` stream because of the implementation of child_process.spawnSync, which does not expose `stdout` and `stderr` streams until termination. We could only concatenate the streams in the end. - Does not create the mixed stream if both streams do not exist.
-
More predictable noop-132 fixture for testing
all
stream outputThe delay 6000ms is approximate, it may not work on every machine.
Commits on Mar 8, 2019
Commits on Mar 9, 2019
-
Using test.serial for testing result.all with lower timeout for noop-…
…132 (1000ms) Testing with test.serial and a delay of only 1000ms seems to pass multiple test runs consistently. Let's see if it passes on CI testing as well.
-