test: cleanup test-stdout-close-catch.js

Added common.mustCall in child process on 'close' callback
Changed several 'var' statements to 'const' or 'let' where appropriate
Also linting

PR-URL: https://github.com/nodejs/node/pull/10006
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
This commit is contained in:
Travis Bretton authored and Roman Reiss committed 2016-12-16 22:08:31 +01:00
1 parent 9de2adc04d
commit 4a08ac695c
1 file changed
+19 -16
+19 -16
View File
@@ -1,27 +1,30 @@
'use strict';
var common = require('../common');
var assert = require('assert');
var path = require('path');
var child_process = require('child_process');
const common = require('../common');
const assert = require('assert');
const path = require('path');
const child_process = require('child_process');
var testScript = path.join(common.fixturesDir, 'catch-stdout-error.js');
const testScript = path.join(common.fixturesDir, 'catch-stdout-error.js');
var cmd = JSON.stringify(process.execPath) + ' ' +
JSON.stringify(testScript) + ' | ' +
JSON.stringify(process.execPath) + ' ' +
'-pe "process.stdin.on(\'data\' , () => process.exit(1))"';
const cmd = JSON.stringify(process.execPath) + ' ' +
JSON.stringify(testScript) + ' | ' +
JSON.stringify(process.execPath) + ' ' +
'-pe "process.stdin.on(\'data\' , () => process.exit(1))"';
var child = child_process.exec(cmd);
var output = '';
var outputExpect = { 'code': 'EPIPE',
'errno': 'EPIPE',
'syscall': 'write' };
const child = child_process.exec(cmd);
let output = '';
const outputExpect = {
code: 'EPIPE',
errno: 'EPIPE',
syscall: 'write'
};
child.stderr.on('data', function(c) {
output += c;
});
child.on('close', function(code) {
child.on('close', common.mustCall(function(code) {
try {
output = JSON.parse(output);
} catch (er) {
@@ -31,4 +34,4 @@ child.on('close', function(code) {
assert.deepStrictEqual(output, outputExpect);
console.log('ok');
});
}));