From 771ce87d4d665cb2a940df7b854eb001a8f3ef02 Mon Sep 17 00:00:00 2001 From: Israel Ortiz Date: Fri, 12 Oct 2018 11:46:07 -0700 Subject: [PATCH] test: fix parameters in test-repl.js fixed order of parameters in assert.strictEqual() assertion functions, first argument provided was the expected value and the second value was the actual value. this is backwards from the documentation for assertions like assert.strictEqual() where the first value being tested and the second value is the expected value --- test/parallel/test-repl.js | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/test/parallel/test-repl.js b/test/parallel/test-repl.js index 49e718da0f0683..f8261dedd98974 100644 --- a/test/parallel/test-repl.js +++ b/test/parallel/test-repl.js @@ -784,8 +784,8 @@ function startTCPRepl() { client.setEncoding('utf8'); client.on('connect', common.mustCall(() => { - assert.strictEqual(true, client.readable); - assert.strictEqual(true, client.writable); + assert.strictEqual(client.readable, true); + assert.strictEqual(client.writable, true); resolveSocket(client); })); @@ -827,8 +827,8 @@ function startUnixRepl() { client.setEncoding('utf8'); client.on('connect', common.mustCall(() => { - assert.strictEqual(true, client.readable); - assert.strictEqual(true, client.writable); + assert.strictEqual(client.readable, true); + assert.strictEqual(client.writable, true); resolveSocket(client); }));