Skip to content

Commit ba5cf7b

Browse files
FieldOfPoppiesjasnell
authored andcommitted
test: switch arguments of assert()
The arguments of the assert were passed in the wrong order (expected, actual). This would have been confusing in case of an error. Changed it to be (actual, expected) PR-URL: #23524 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
1 parent 87eeb6b commit ba5cf7b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/sequential/test-inspector-break-when-eval.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ async function runTests() {
6363
await breakOnLine(session);
6464
await stepOverConsoleStatement(session);
6565
await session.runToCompletion();
66-
assert.strictEqual(0, (await child.expectShutdown()).exitCode);
66+
assert.strictEqual((await child.expectShutdown()).exitCode, 0);
6767
}
6868

6969
runTests();

0 commit comments

Comments
 (0)