Skip to content

Commit

Permalink
test: switch the param order in the assertion
Browse files Browse the repository at this point in the history
Switch the param order in the assertion at line 140
so that they are in the correct order (actual, expected)

PR-URL: #28341
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
raveneyex authored and targos committed Jul 2, 2019
1 parent 5367d02 commit b822545
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-tls-sni-option.js
Original file line number Diff line number Diff line change
Expand Up @@ -138,8 +138,8 @@ test({
function test(options, clientResult, serverResult, clientError, serverError) {
const server = tls.createServer(serverOptions, (c) => {
assert.deepStrictEqual(
{ sni: c.servername, authorized: c.authorized },
serverResult
serverResult,
{ sni: c.servername, authorized: c.authorized }
);
});

Expand Down

0 comments on commit b822545

Please sign in to comment.