From 45529a5198cde52b0535a44af0e34ee005149a19 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 567ef7911c446d..5abaca8091b897 100644 --- a/test/sequential/test-inspector-break-e.js +++ b/test/sequential/test-inspector-break-e.js @@ -17,7 +17,7 @@ async function runTests() { ]); await session.waitForBreakOnLine(0, '[eval]'); await session.runToCompletion(); - assert.strictEqual(0, (await instance.expectShutdown()).exitCode); + assert.strictEqual((await instance.expectShutdown()).exitCode, 0); } runTests();