From c1288fe4320ec1f1e3a29cb7293d2d2faa43d522 Mon Sep 17 00:00:00 2001 From: Shannon Date: Fri, 12 Oct 2018 11:10:26 -0700 Subject: [PATCH] test: fix parameter order passed to strictEqual strictEqual() expects the first argument to be the actual value and the second argument to be the expected value. This fix will correctly label the AssertionError.: PR-URL: https://github.com/nodejs/node/pull/23577 Reviewed-By: Anna Henningsen Reviewed-By: Hitesh Kanwathirtha Reviewed-By: James M Snell Reviewed-By: Ruben Bridgewater Reviewed-By: Colin Ihrig Reviewed-By: Gireesh Punathil --- test/sequential/test-inspector-break-e.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/sequential/test-inspector-break-e.js b/test/sequential/test-inspector-break-e.js index d988c118740..954ce6b8978 100644 --- a/test/sequential/test-inspector-break-e.js +++ b/test/sequential/test-inspector-break-e.js @@ -16,7 +16,7 @@ async function runTests() { ]); await session.waitForBreakOnLine(2, '[eval]'); await session.runToCompletion(); - assert.strictEqual(0, (await instance.expectShutdown()).exitCode); + assert.strictEqual((await instance.expectShutdown()).exitCode, 0); } runTests();