Skip to content

Commit

Permalink
test: modernize test-child-process-flush-stdio
Browse files Browse the repository at this point in the history
PR-URL: nodejs#23504
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
  • Loading branch information
Viacheslav Liakhov authored and BridgeAR committed Oct 15, 2018
1 parent 211f2bc commit aa5cb57
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions test/parallel/test-child-process-flush-stdio.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,25 +9,25 @@ const opts = { shell: common.isWindows };

const p = cp.spawn('echo', [], opts);

p.on('close', common.mustCall(function(code, signal) {
p.on('close', common.mustCall((code, signal) => {
assert.strictEqual(code, 0);
assert.strictEqual(signal, null);
spawnWithReadable();
}));

p.stdout.read();

function spawnWithReadable() {
const spawnWithReadable = () => {
const buffer = [];
const p = cp.spawn('echo', ['123'], opts);
p.on('close', common.mustCall(function(code, signal) {
p.on('close', common.mustCall((code, signal) => {
assert.strictEqual(code, 0);
assert.strictEqual(signal, null);
assert.strictEqual(Buffer.concat(buffer).toString().trim(), '123');
}));
p.stdout.on('readable', function() {
p.stdout.on('readable', () => {
let buf;
while (buf = this.read())
while (buf = p.stdout.read())
buffer.push(buf);
});
}
};

0 comments on commit aa5cb57

Please sign in to comment.