test: fix test-process-exec-argv flakiness

santigimeno · evanlucas · commit 89b25fac4ce6 · 2016-05-17T08:37:44.000-05:00
Wait for the `close` event before parsing the child stdout output. Fixes: #6480 PR-URL: #6575 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
diff --git a/test/parallel/test-process-exec-argv.js b/test/parallel/test-process-exec-argv.js
@@ -15,7 +15,7 @@ if (process.argv[2] === 'child') {
     out += chunk;
   });
 
-  child.on('exit', function() {
+  child.on('close', function() {
     assert.deepStrictEqual(JSON.parse(out), execArgv);
   });
 }

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ if (process.argv[2] === 'child') {`
`15`	`15`	`out += chunk;`
`16`	`16`	`});`
`17`	`17`
`18`		`- child.on('exit', function() {`
	`18`	`+ child.on('close', function() {`
`19`	`19`	`assert.deepStrictEqual(JSON.parse(out), execArgv);`
`20`	`20`	`});`
`21`	`21`	`}`