From 34a921825bdda9e5f0a51096a34b43ac9bcacb34 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);