Skip to content

Commit

Permalink
test: fix assertion order test-tls-server-verify
Browse files Browse the repository at this point in the history
PR-URL: #23549
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
  • Loading branch information
Carolina Pinzon authored and jasnell committed Oct 17, 2018
1 parent 3ad2267 commit b041922
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-tls-server-verify.js
Original file line number Diff line number Diff line change
Expand Up @@ -226,14 +226,14 @@ function runClient(prefix, port, options, cb) {
client.on('exit', function(code) {
if (options.shouldReject) {
assert.strictEqual(
true, rejected,
rejected, true,
`${prefix}${options.name} NOT rejected, but should have been`);
} else {
assert.strictEqual(
false, rejected,
rejected, false,
`${prefix}${options.name} rejected, but should NOT have been`);
assert.strictEqual(
options.shouldAuth, authed,
authed, options.shouldAuth,
`${prefix}${options.name} authed is ${authed} but should have been ${
options.shouldAuth}`);
}
Expand Down

0 comments on commit b041922

Please sign in to comment.