test-child-process-flush-stdio: review changes

Make the spawned process object a const, and add assertions to the event handler that the arguments are as expected
davidvgalbraith · Nov 30, 2015 · 4becc69 · 4becc69
1 parent d13f74f
commit 4becc69
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/test/parallel/test-child-process-flush-stdio.js b/test/parallel/test-child-process-flush-stdio.js
@@ -1,10 +1,14 @@
 'use strict';
 const cp = require('child_process');
 const common = require('../common');
+const assert = require('assert');
 
-var p = cp.spawn('echo');
+const p = cp.spawn('echo');
 
-p.on('close', common.mustCall(function() {}));
+p.on('close', common.mustCall(function(code, signal) {
+ assert.strictEqual(code, 0);
+ assert.strictEqual(signal, null);
+}));
 
 p.stdout.read();