Skip to content

Commit

Permalink
test: fix assertion argument order
Browse files Browse the repository at this point in the history
Fix the assertion argument order so that it will report "actual" and
"expected" correctly when the test fails.

Ref: #19263

PR-URL: #19264
Refs: #19263
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
  • Loading branch information
Trott authored and BethGriggs committed Oct 16, 2018
1 parent b67bf38 commit 4bcdc1b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/sequential/test-inspector-stop-profile-after-done.js
Expand Up @@ -24,7 +24,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();
Expand Down

0 comments on commit 4bcdc1b

Please sign in to comment.