test: fix test-debug-port-from-cmdline
authorJoão Reis <reis@janeasystems.com>
Wed, 15 Jul 2015 12:19:56 +0000 (13:19 +0100)
committercjihrig <cjihrig@gmail.com>
Thu, 16 Jul 2015 01:52:16 +0000 (21:52 -0400)
commit2b4b6006607c33a5699ec53afaf40f987dc11895
tree8de57a9d3a0fc08008eefec22998af396437ee29
parentf95f9ef6ea2eb3e7cc98c3dea372d74b4bb10910
test: fix test-debug-port-from-cmdline

This test was failing because the spawned process was terminated before
anything could be done, by calling child.stdin.end. With this change,
the child's stdin is no longer closed. When the stdin is not a tty,
io.js waits for the whole input before starting, so the child must be
run with --interactive to process the command sent by the parent. The
child is killed explicitly by the parent before it exits.

This test was failing silently because the asserts were not called if
nothing was received from the child. This fix moves assertOutputLines to
always run on exit.

Fixes: https://github.com/nodejs/io.js/issues/2177
Refs: https://github.com/nodejs/io.js/issues/2094
PR-URL: https://github.com/nodejs/io.js/pull/2186
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Alexis Campailla <alexis@janeasystems.com>
test/parallel/test-debug-port-from-cmdline.js