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 jasnell committed Mar 12, 2018
1 parent c979488 commit da226ea
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
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand Down

0 comments on commit da226ea

Please sign in to comment.