Skip to content

Commit

Permalink
test: fixed the arguments order in assert.strictEqual
Browse files Browse the repository at this point in the history
This change was initiated from the NodeConfEU session.

PR-URL: #24135
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
  • Loading branch information
michael-zucker authored and BridgeAR committed Nov 13, 2018
1 parent 71545e6 commit a5ac7b9
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions test/parallel/test-child-process-stdin.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ cat.stdin.write('hello');
cat.stdin.write(' ');
cat.stdin.write('world');

assert.strictEqual(true, cat.stdin.writable);
assert.strictEqual(false, cat.stdin.readable);
assert.strictEqual(cat.stdin.writable, true);
assert.strictEqual(cat.stdin.readable, false);

cat.stdin.end();

Expand All @@ -50,9 +50,9 @@ cat.stderr.on('data', common.mustNotCall());
cat.stderr.on('end', common.mustCall());

cat.on('exit', common.mustCall(function(status) {
assert.strictEqual(0, status);
assert.strictEqual(status, 0);
}));

cat.on('close', common.mustCall(function() {
assert.strictEqual('hello world', response);
assert.strictEqual(response, 'hello world');
}));

0 comments on commit a5ac7b9

Please sign in to comment.