From d5abbabc4f1db2d4e4ce8c1c98afd4981808074b 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();