Skip to content

Commit

Permalink
test: fix backward assertion arguments
Browse files Browse the repository at this point in the history
PR-URL: #23616
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
  • Loading branch information
svassr authored and rvagg committed Nov 28, 2018
1 parent df2ad6e commit c80c46f
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-net-server-max-connections.js
Original file line number Diff line number Diff line change
Expand Up @@ -75,10 +75,10 @@ function makeConnection(index) {
}

if (index < server.maxConnections) {
assert.strictEqual(true, gotData,
assert.strictEqual(gotData, true,
`${index} didn't get data, but should have`);
} else {
assert.strictEqual(false, gotData,
assert.strictEqual(gotData, false,
`${index} got data, but shouldn't have`);
}
});
Expand All @@ -103,5 +103,5 @@ function makeConnection(index) {


process.on('exit', function() {
assert.strictEqual(N, closes);
assert.strictEqual(closes, N);
});

0 comments on commit c80c46f

Please sign in to comment.