From 38416363a620e93f7e9aa9c532151e5be30e197d Mon Sep 17 00:00:00 2001 From: Rich Trott Date: Fri, 9 Mar 2018 10:05:36 -0800 Subject: [PATCH] test: fix assertion argument order Fix the assertion argument order so that it will report "actual" and "expected" correctly when the test fails. Ref: https://github.com/nodejs/node/issues/19263 --- test/sequential/test-inspector-stop-profile-after-done.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/sequential/test-inspector-stop-profile-after-done.js b/test/sequential/test-inspector-stop-profile-after-done.js index 15764d84860e8c..678f98a556c099 100644 --- a/test/sequential/test-inspector-stop-profile-after-done.js +++ b/test/sequential/test-inspector-stop-profile-after-done.js @@ -25,7 +25,7 @@ async function runTests() { 'Waiting for the debugger to disconnect...'); await session.send({ method: 'Profiler.stop' }); session.disconnect(); - assert.strictEqual(0, (await child.expectShutdown()).exitCode); + assert.strictEqual((await child.expectShutdown()).exitCode, 0); } common.crashOnUnhandledRejection();