From 134389142edbe09f8881338ee738b6b93a3ec5e0 Mon Sep 17 00:00:00 2001 From: Jason Nutter Date: Fri, 12 Oct 2018 11:25:42 -0700 Subject: [PATCH] test: fix order of parameters to assert.strictEqual Usage of assert.strictEqual in test-net-server-listen-remove-callback.js incorrectly passes the expected value as the first argument and actual value as the second argument. PR-URL: https://github.com/nodejs/node/pull/23590 Reviewed-By: James M Snell Reviewed-By: Ruben Bridgewater Reviewed-By: Colin Ihrig Reviewed-By: Gireesh Punathil Reviewed-By: Trivikram Kamat Reviewed-By: Sakthipriyan Vairamani --- test/parallel/test-net-server-listen-remove-callback.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/parallel/test-net-server-listen-remove-callback.js b/test/parallel/test-net-server-listen-remove-callback.js index 3989b229062304..a874099fb8c782 100644 --- a/test/parallel/test-net-server-listen-remove-callback.js +++ b/test/parallel/test-net-server-listen-remove-callback.js @@ -30,7 +30,7 @@ const server = net.createServer(); server.on('close', function() { const listeners = server.listeners('listening'); console.log('Closed, listeners:', listeners.length); - assert.strictEqual(0, listeners.length); + assert.strictEqual(listeners.length, 0); }); server.listen(0, function() {