Skip to content

Commit

Permalink
test: refactor the code in test-util-debug.js
Browse files Browse the repository at this point in the history
* use const and let instead of var
* use assert.strictEqual instead of assert.equal
* use arrow functions
* removed unwanted console log
  • Loading branch information
sivaprs committed Jan 4, 2017
1 parent f44f509 commit 40921a5
Showing 1 changed file with 11 additions and 12 deletions.
23 changes: 11 additions & 12 deletions test/sequential/test-util-debug.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,44 +17,43 @@ function parent() {
}

function test(environ, shouldWrite) {
var expectErr = '';
let expectErr = '';
if (shouldWrite) {
expectErr = 'TUD %PID%: this { is: \'a\' } /debugging/\n' +
'TUD %PID%: number=1234 string=asdf obj={"foo":"bar"}\n';
}
var expectOut = 'ok\n';
const expectOut = 'ok\n';

const spawn = require('child_process').spawn;
var child = spawn(process.execPath, [__filename, 'child'], {
const child = spawn(process.execPath, [__filename, 'child'], {
env: Object.assign(process.env, { NODE_DEBUG: environ })
});

expectErr = expectErr.split('%PID%').join(child.pid);

var err = '';
let err = '';
child.stderr.setEncoding('utf8');
child.stderr.on('data', function(c) {
child.stderr.on('data', (c) => {
err += c;
});

var out = '';
let out = '';
child.stdout.setEncoding('utf8');
child.stdout.on('data', function(c) {
child.stdout.on('data', (c) => {
out += c;
});

child.on('close', common.mustCall(function(c) {
child.on('close', common.mustCall((c) => {
assert(!c);
assert.equal(err, expectErr);
assert.equal(out, expectOut);
console.log('ok %j %j', environ, shouldWrite);
assert.strictEqual(err, expectErr);
assert.strictEqual(out, expectOut);
}));
}


function child() {
const util = require('util');
var debug = util.debuglog('tud');
const debug = util.debuglog('tud');
debug('this', { is: 'a' }, /debugging/);
debug('number=%d string=%s obj=%j', 1234, 'asdf', { foo: 'bar' });
console.log('ok');
Expand Down

0 comments on commit 40921a5

Please sign in to comment.