test: utilize common.mustCall() on child exit

sreepurnajasti · gibfahn · commit ec28fe68ad17 · 2017-12-20T01:58:51.000Z
PR-URL: #16996 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>
diff --git a/test/parallel/test-child-process-fork-exec-argv.js b/test/parallel/test-child-process-fork-exec-argv.js
@@ -20,7 +20,7 @@
 // USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 'use strict';
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 const child_process = require('child_process');
 const spawn = child_process.spawn;
@@ -43,7 +43,7 @@ if (process.argv[2] === 'fork'){
 out += chunk;
 });
 
- child.on('exit', function(){
+ child.on('exit', common.mustCall(function(){
 assert.deepStrictEqual(JSON.parse(out), execArgv);
- });
+ }));
 }

-Original file line number
+Diff line change
 // USE OR OTHER DEALINGS IN THE SOFTWARE.
 'use strict';
 -require('../common');
 +constcommon=require('../common');
 constassert=require('assert');
 constchild_process=require('child_process');
 constspawn=child_process.spawn;
 out+=chunk;
 });
 -child.on('exit',function(){
 +child.on('exit',common.mustCall(function(){
 assert.deepStrictEqual(JSON.parse(out),execArgv);
 -});
 +}));
+}