test: swap expected and actual arguments in assert.strictEqual()

PR-URL: #23542 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]>
nodejs · Oct 30, 2018 · adc89e3 · adc89e3
1 parent b55fed0
commit adc89e3
Showing 1 changed file with 12 additions and 12 deletions.
diff --git a/test/pummel/test-net-pingpong-delay.js b/test/pummel/test-net-pingpong-delay.js
@@ -35,11 +35,11 @@ function pingPongTest(port, host, on_complete) {
 
     socket.on('data', function(data) {
       console.log(data);
-      assert.strictEqual('PING', data);
-      assert.strictEqual('open', socket.readyState);
-      assert.strictEqual(true, count <= N);
+      assert.strictEqual(data, 'PING');
+      assert.strictEqual(socket.readyState, 'open');
+      assert.strictEqual(count <= N, true);
       setTimeout(function() {
-        assert.strictEqual('open', socket.readyState);
+        assert.strictEqual(socket.readyState, 'open');
         socket.write('PONG');
       }, DELAY);
     });
@@ -51,14 +51,14 @@ function pingPongTest(port, host, on_complete) {
 
     socket.on('end', function() {
       console.log('server-side socket EOF');
-      assert.strictEqual('writeOnly', socket.readyState);
+      assert.strictEqual(socket.readyState, 'writeOnly');
       socket.end();
     });
 
     socket.on('close', function(had_error) {
       console.log('server-side socket.end');
-      assert.strictEqual(false, had_error);
-      assert.strictEqual('closed', socket.readyState);
+      assert.strictEqual(had_error, false);
+      assert.strictEqual(socket.readyState, 'closed');
       socket.server.close();
     });
   });
@@ -69,17 +69,17 @@ function pingPongTest(port, host, on_complete) {
     client.setEncoding('utf8');
 
     client.on('connect', function() {
-      assert.strictEqual('open', client.readyState);
+      assert.strictEqual(client.readyState, 'open');
       client.write('PING');
     });
 
     client.on('data', function(data) {
       console.log(data);
-      assert.strictEqual('PONG', data);
-      assert.strictEqual('open', client.readyState);
+      assert.strictEqual(data, 'PONG');
+      assert.strictEqual(client.readyState, 'open');
 
       setTimeout(function() {
-        assert.strictEqual('open', client.readyState);
+        assert.strictEqual(client.readyState, 'open');
         if (count++ < N) {
           client.write('PING');
         } else {
@@ -97,7 +97,7 @@ function pingPongTest(port, host, on_complete) {
 
     client.on('close', common.mustCall(function() {
       console.log('client.end');
-      assert.strictEqual(N + 1, count);
+      assert.strictEqual(count, N + 1);
       assert.ok(client_ended);
       if (on_complete) on_complete();
     }));