Permalink
Browse files

test: refactor the code in test-child-process-spawn-loop.js

* use const and let instead of var
* use assert.strictEqual instead of assert.equal
* use arrow functions

PR-URL: #10605
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information...
sivaprs authored and jasnell committed Jan 4, 2017
1 parent b9abeec commit 41567ee9d816b3adb94fc7bf67ca3a6cef7edc1f
Showing with 10 additions and 10 deletions.
  1. +10 −10 test/pummel/test-child-process-spawn-loop.js
@@ -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 {
@@ -34,6 +34,6 @@ function doSpawn(i) {
doSpawn(0);
process.on('exit', function() {
process.on('exit', () => {
assert.ok(finished);
});

0 comments on commit 41567ee

Please sign in to comment.