delta/node-new.git/test/parallel/test-child-process-pipe-dataflow.js, branch Remove-Python-3-tests

test: increase debugging information on failure

2019-05-23T14:53:57+00:00

Increase the information displayed when
test-child-process-pipe-dataflow.js fails.

PR-URL: https://github.com/nodejs/node/pull/27790
Reviewed-By: Gireesh Punathil 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Trivikram Kamat

child_process: only stop readable side of stream passed to proc

2019-04-29T14:31:27+00:00

Closing the underlying resource completely has the unwanted side effect
that the stream can no longer be used at all, including passing it
to other child processes.

What we want to avoid is accidentally reading from the stream;
accordingly, it should be sufficient to stop its readable side
manually, and otherwise leave the underlying resource intact.

Fixes: https://github.com/nodejs/node/issues/27097
Refs: https://github.com/nodejs/node/pull/21209

PR-URL: https://github.com/nodejs/node/pull/27373
Reviewed-By: Matteo Collina 
Reviewed-By: Gireesh Punathil

test: capture stderr from child processes

2019-02-11T04:03:58+00:00

If the test fails with errors from the child commands,
there is no debug info. Suppliment the stderr data
so that we know what to look for.

Refs: https://github.com/nodejs/node/issues/25988
PR-URL: https://github.com/nodejs/node/pull/26007
Reviewed-By: Anna Henningsen 
Reviewed-By: Luigi Pinca 
Reviewed-By: Colin Ihrig

child_process: close pipe ends that are re-piped

2019-02-06T12:20:46+00:00

when t0 and t1 are spawned with t0's outputstream [1, 2] is piped into
t1's input, a new pipe is created which uses a copy of the t0's fd.
This leaves the original copy in Node parent, unattended. Net result is
that when t0 produces data, it gets bifurcated into both the copies

Detect the passed handle to be of 'wrap' type and close after the
native spawn invocation by which time piping would have been over.

Fixes: https://github.com/nodejs/node/issues/9413
Fixes: https://github.com/nodejs/node/issues/18016

PR-URL: https://github.com/nodejs/node/pull/21209
Reviewed-By: Matteo Collina 
Reviewed-By: James M Snell 
Reviewed-By: Anna Henningsen