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: nodejs#19263

PR-URL: nodejs#19264
Refs: nodejs#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 MayaLekova committed May 8, 2018
1 parent fb7552a commit 3f2d485
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 3f2d485

Please sign in to comment.