From 9dc11e33544492d5404df1d1e986e1db73cb85c2 Mon Sep 17 00:00:00 2001
From: Arne Schramm <arne.schramm@protonmail.com>
Date: Fri, 12 Oct 2018 10:23:40 -0700
Subject: [PATCH] test: switch arguments of assert()

The arguments of the assert were passed in the wrong order
(expected, actual). This would have been confusing in case
of an error. Changed it to be (actual, expected)

PR-URL: https://github.com/nodejs/node/pull/23524
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
---
 test/sequential/test-inspector-break-when-eval.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/sequential/test-inspector-break-when-eval.js b/test/sequential/test-inspector-break-when-eval.js
index e7529f786a9859..9952650252310e 100644
--- a/test/sequential/test-inspector-break-when-eval.js
+++ b/test/sequential/test-inspector-break-when-eval.js
@@ -63,7 +63,7 @@ async function runTests() {
   await breakOnLine(session);
   await stepOverConsoleStatement(session);
   await session.runToCompletion();
-  assert.strictEqual(0, (await child.expectShutdown()).exitCode);
+  assert.strictEqual((await child.expectShutdown()).exitCode, 0);
 }
 
 runTests();