From e8e5ea3d81d41b10ce4ece75a284a9912b348201 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. This should be reversed, per the assert documentation. Refs: https://nodejs.org/api/assert.html#assert_assert_strictequal_actual_expected_message --- 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() {