Skip to content

Commit

Permalink
test: fix flaky test-stdout-close-catch
Browse files Browse the repository at this point in the history
Make sure that `catch-stdout-error` has written data before the
destination process exits.

Fixes: #6791
PR-URL: #6808
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
  • Loading branch information
santigimeno authored and Myles Borins committed Jun 24, 2016
1 parent 3b94e31 commit 1c4549a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/parallel/test-stdout-close-catch.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ var testScript = path.join(common.fixturesDir, 'catch-stdout-error.js');
var cmd = JSON.stringify(process.execPath) + ' ' +
JSON.stringify(testScript) + ' | ' +
JSON.stringify(process.execPath) + ' ' +
'-pe "process.exit(1);"';
'-pe "process.stdin.on(\'data\' , () => process.exit(1))"';

var child = child_process.exec(cmd);
var output = '';
Expand Down

0 comments on commit 1c4549a

Please sign in to comment.