From 1c36943b8da127b1347b80a5fb356dd6a4af0d72 Mon Sep 17 00:00:00 2001 From: jungkumseok Date: Fri, 12 Oct 2018 09:13:42 -0700 Subject: [PATCH] test: fix `assert.strictEqual` arguments in test/parallel/test-c-ares.js When using `assert.strictEqual`, the first argument must be the actual value and the second argument must be the expected value. PR-URL: https://github.com/nodejs/node/pull/23448 Reviewed-By: Ruben Bridgewater Reviewed-By: Colin Ihrig Reviewed-By: James M Snell Reviewed-By: Trivikram Kamat Reviewed-By: Sakthipriyan Vairamani --- test/parallel/test-c-ares.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/test/parallel/test-c-ares.js b/test/parallel/test-c-ares.js index 8b1cf79868c328..45e7d46829b00a 100644 --- a/test/parallel/test-c-ares.js +++ b/test/parallel/test-c-ares.js @@ -46,20 +46,20 @@ const dnsPromises = dns.promises; dns.lookup(null, common.mustCall((error, result, addressType) => { assert.ifError(error); - assert.strictEqual(null, result); - assert.strictEqual(4, addressType); + assert.strictEqual(result, null); + assert.strictEqual(addressType, 4); })); dns.lookup('127.0.0.1', common.mustCall((error, result, addressType) => { assert.ifError(error); - assert.strictEqual('127.0.0.1', result); - assert.strictEqual(4, addressType); + assert.strictEqual(result, '127.0.0.1'); + assert.strictEqual(addressType, 4); })); dns.lookup('::1', common.mustCall((error, result, addressType) => { assert.ifError(error); - assert.strictEqual('::1', result); - assert.strictEqual(6, addressType); + assert.strictEqual(result, '::1'); + assert.strictEqual(addressType, 6); })); [