Skip to content
Permalink
Browse files

test: switch the param order in the assertion

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 Jun 21, 2019
1 parent 5367d02 commit b822545f84611f49ce1c95098767fa3bac9238b3
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/parallel/test-tls-sni-option.js
@@ -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 }
);
});

0 comments on commit b822545

Please sign in to comment.
You can’t perform that action at this time.