Browse Source

test: cleanup test-child-process-exec-env.js

Replace equal with strictEqual, use const instead of var and
improve test with use of assert.notStrictEqual

PR-URL: https://github.com/nodejs/node/pull/8600
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ilkka Myller <ilkka.myller@nodefield.com>
v6.x
Yevgen Safronov 8 years ago
committed by Jeremiah Senkpiel
parent
commit
c98d0c984d
  1. 22
      test/parallel/test-child-process-exec-env.js

22
test/parallel/test-child-process-exec-env.js

@ -1,11 +1,11 @@
'use strict';
var common = require('../common');
var assert = require('assert');
var exec = require('child_process').exec;
var success_count = 0;
var error_count = 0;
var response = '';
var child;
const common = require('../common');
const assert = require('assert');
const exec = require('child_process').exec;
let success_count = 0;
let error_count = 0;
let response = '';
let child;
function after(err, stdout, stderr) {
if (err) {
@ -13,10 +13,10 @@ function after(err, stdout, stderr) {
console.log('error!: ' + err.code);
console.log('stdout: ' + JSON.stringify(stdout));
console.log('stderr: ' + JSON.stringify(stderr));
assert.equal(false, err.killed);
assert.strictEqual(false, err.killed);
} else {
success_count++;
assert.equal(true, stdout != '');
assert.notStrictEqual('', stdout);
}
}
@ -33,7 +33,7 @@ child.stdout.on('data', function(chunk) {
process.on('exit', function() {
console.log('response: ', response);
assert.equal(1, success_count);
assert.equal(0, error_count);
assert.strictEqual(1, success_count);
assert.strictEqual(0, error_count);
assert.ok(response.indexOf('HELLO=WORLD') >= 0);
});

Loading…
Cancel
Save