Skip to content

Commit 1c36943

Browse files
jungkumseokjasnell
authored andcommitted
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: #23448 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
1 parent fc963cd commit 1c36943

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

test/parallel/test-c-ares.js

+6-6
Original file line numberDiff line numberDiff line change
@@ -46,20 +46,20 @@ const dnsPromises = dns.promises;
4646

4747
dns.lookup(null, common.mustCall((error, result, addressType) => {
4848
assert.ifError(error);
49-
assert.strictEqual(null, result);
50-
assert.strictEqual(4, addressType);
49+
assert.strictEqual(result, null);
50+
assert.strictEqual(addressType, 4);
5151
}));
5252

5353
dns.lookup('127.0.0.1', common.mustCall((error, result, addressType) => {
5454
assert.ifError(error);
55-
assert.strictEqual('127.0.0.1', result);
56-
assert.strictEqual(4, addressType);
55+
assert.strictEqual(result, '127.0.0.1');
56+
assert.strictEqual(addressType, 4);
5757
}));
5858

5959
dns.lookup('::1', common.mustCall((error, result, addressType) => {
6060
assert.ifError(error);
61-
assert.strictEqual('::1', result);
62-
assert.strictEqual(6, addressType);
61+
assert.strictEqual(result, '::1');
62+
assert.strictEqual(addressType, 6);
6363
}));
6464

6565
[

0 commit comments

Comments
 (0)