Skip to content

Commit 895d4ab

Browse files
eojthebraveBridgeAR
authored andcommitted
test: fix order of arguments passed to strictEqual
The argument order in the strictEqual check was in the wrong order. The first argument is now the actual value and the second argument is the expected value. PR-URL: nodejs#23571 Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
1 parent 2160168 commit 895d4ab

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

test/parallel/test-net-socket-local-address.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ const server = net.createServer((socket) => {
1818

1919
server.on('close', common.mustCall(() => {
2020
// client and server should agree on the ports used
21-
assert.deepStrictEqual(clientLocalPorts, serverRemotePorts);
22-
assert.strictEqual(2, conns);
21+
assert.deepStrictEqual(serverRemotePorts, clientLocalPorts);
22+
assert.strictEqual(conns, 2);
2323
}));
2424

2525
server.listen(0, common.localhostIPv4, connect);

0 commit comments

Comments
 (0)