Skip to content

Commit

Permalink
test: refactor the code in test-child-process-spawn-loop.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

PR-URL: nodejs#10605
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
sivaprs authored and italoacasas committed Jan 23, 2017
1 parent 5162337 commit e83d626
Showing 1 changed file with 10 additions and 10 deletions.
20 changes: 10 additions & 10 deletions test/pummel/test-child-process-spawn-loop.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,26 +4,26 @@ const assert = require('assert');

const spawn = require('child_process').spawn;

var SIZE = 1000 * 1024;
var N = 40;
var finished = false;
const SIZE = 1000 * 1024;
const N = 40;
let finished = false;

function doSpawn(i) {
var child = spawn('python', ['-c', 'print ' + SIZE + ' * "C"']);
var count = 0;
const child = spawn('python', ['-c', 'print ' + SIZE + ' * "C"']);
let count = 0;

child.stdout.setEncoding('ascii');
child.stdout.on('data', function(chunk) {
child.stdout.on('data', (chunk) => {
count += chunk.length;
});

child.stderr.on('data', function(chunk) {
child.stderr.on('data', (chunk) => {
console.log('stderr: ' + chunk);
});

child.on('close', function() {
child.on('close', () => {
// + 1 for \n or + 2 for \r\n on Windows
assert.equal(SIZE + (common.isWindows ? 2 : 1), count);
assert.strictEqual(SIZE + (common.isWindows ? 2 : 1), count);
if (i < N) {
doSpawn(i + 1);
} else {
Expand All @@ -34,6 +34,6 @@ function doSpawn(i) {

doSpawn(0);

process.on('exit', function() {
process.on('exit', () => {
assert.ok(finished);
});

0 comments on commit e83d626

Please sign in to comment.