diff options
author | Marcos Frony <mfrony@gmail.com> | 2018-10-12 20:15:11 +0300 |
---|---|---|
committer | Myles Borins <mylesborins@google.com> | 2018-11-29 19:38:40 +0300 |
commit | 3c5d31266232fe7a0fea3d0c1088b190377da020 (patch) | |
tree | fd93eb443b87bb01d8ba9928555a00a573efce06 | |
parent | e04a128731a9c019dc59a54e04252ab04c4cfb10 (diff) |
test: reorder asserts arguments
Revert arguments for assert and ensure that the first
argument is always the actual value being tested.
PR-URL: https://github.com/nodejs/node/pull/23534
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
-rw-r--r-- | test/pummel/test-net-pause.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/pummel/test-net-pause.js b/test/pummel/test-net-pause.js index c25f8a0bc1d..ea0e61264e3 100644 --- a/test/pummel/test-net-pause.js +++ b/test/pummel/test-net-pause.js @@ -53,7 +53,7 @@ server.on('listening', function() { setTimeout(function() { chars_recved = recv.length; console.log(`pause at: ${chars_recved}`); - assert.strictEqual(true, chars_recved > 1); + assert.strictEqual(chars_recved > 1, true); client.pause(); setTimeout(function() { console.log(`resume at: ${chars_recved}`); @@ -86,6 +86,6 @@ server.on('listening', function() { server.listen(common.PORT); process.on('exit', function() { - assert.strictEqual(N, recv.length); + assert.strictEqual(recv.length, N); console.error('Exit'); }); |