From b822545f84611f49ce1c95098767fa3bac9238b3 Mon Sep 17 00:00:00 2001 From: raveneyex Date: Fri, 21 Jun 2019 15:51:01 -0500 Subject: [PATCH] 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: https://github.com/nodejs/node/pull/28341 Reviewed-By: Anna Henningsen Reviewed-By: Rich Trott Reviewed-By: Colin Ihrig Reviewed-By: Yongsheng Zhang Reviewed-By: Ruben Bridgewater Reviewed-By: Trivikram Kamat Reviewed-By: James M Snell --- test/parallel/test-tls-sni-option.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/test/parallel/test-tls-sni-option.js b/test/parallel/test-tls-sni-option.js index f4bc7dc1b2bc1a..9ac5a8c5622de4 100644 --- a/test/parallel/test-tls-sni-option.js +++ b/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 } ); });