Skip to content

Commit 842c8fd

Browse files
FieldOfPoppiesBridgeAR
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: nodejs#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 d01a3d7 commit 842c8fd

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)