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: nodejs#24135
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
  • Loading branch information
michael-zucker authored and kiyomizumia committed Nov 15, 2018
1 parent 7b54571 commit 2a249ca
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 2a249ca

Please sign in to comment.