From b49f4a93a60de7b0cd4e38e1ea4efa095e8a7e49 Mon Sep 17 00:00:00 2001 From: Joe Shindelar Date: Fri, 12 Oct 2018 11:03:15 -0700 Subject: [PATCH] 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: https://github.com/nodejs/node/pull/23571 Reviewed-By: Joyee Cheung Reviewed-By: James M Snell Reviewed-By: Anna Henningsen Reviewed-By: Ruben Bridgewater Reviewed-By: Colin Ihrig Reviewed-By: Gireesh Punathil Reviewed-By: Trivikram Kamat Reviewed-By: Sakthipriyan Vairamani --- test/parallel/test-net-socket-local-address.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-net-socket-local-address.js b/test/parallel/test-net-socket-local-address.js index 1aa51e0c5c1d87..e80731bc78fb0a 100644 --- a/test/parallel/test-net-socket-local-address.js +++ b/test/parallel/test-net-socket-local-address.js @@ -18,8 +18,8 @@ const server = net.createServer((socket) => { server.on('close', common.mustCall(() => { // client and server should agree on the ports used - assert.deepStrictEqual(clientLocalPorts, serverRemotePorts); - assert.strictEqual(2, conns); + assert.deepStrictEqual(serverRemotePorts, clientLocalPorts); + assert.strictEqual(conns, 2); })); server.listen(0, common.localhostIPv4, connect);