Skip to content

Commit da7f3a9

Browse files
FieldOfPoppiesaddaleax
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 913e40e commit da7f3a9

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

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

Lines changed: 1 addition & 1 deletion
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)