From a5ac7b94ea6b3fb048475c3e371d2354addd3631 Mon Sep 17 00:00:00 2001 From: mzucker Date: Tue, 6 Nov 2018 14:37:00 +0000 Subject: [PATCH] test: fixed the arguments order in `assert.strictEqual` This change was initiated from the NodeConfEU session. PR-URL: https://github.com/nodejs/node/pull/24135 Reviewed-By: Colin Ihrig Reviewed-By: Anna Henningsen Reviewed-By: Ruben Bridgewater Reviewed-By: Trivikram Kamat Reviewed-By: Gireesh Punathil --- test/parallel/test-child-process-stdin.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/parallel/test-child-process-stdin.js b/test/parallel/test-child-process-stdin.js index 7f49d90f3a3fb6..29f78df7d0ae88 100644 --- a/test/parallel/test-child-process-stdin.js +++ b/test/parallel/test-child-process-stdin.js @@ -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(); @@ -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'); }));