Skip to content

Commit 23ad927

Browse files
Trotttargos
authored andcommitted
test: add mustCall() to test-inspector-contexts
In test-inspector-contexts, if mainContextPromise is modified such that the `method` argument is changed to something invalid, the test still passes and the second part of the test never runs. Use `common.mustCall()` to cause the test to fail if the second part of the test does not run. Refs: #30519 (comment) PR-URL: #30649 Reviewed-By: Gireesh Punathil <[email protected]> Reviewed-By: Anto Aravinth <[email protected]> Reviewed-By: Denys Otrishko <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
1 parent cac77dd commit 23ad927

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/sequential/test-inspector-contexts.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -155,4 +155,4 @@ async function testBreakpointHit() {
155155
await pausedPromise;
156156
}
157157

158-
testContextCreatedAndDestroyed().then(testBreakpointHit);
158+
testContextCreatedAndDestroyed().then(common.mustCall(testBreakpointHit));

0 commit comments

Comments
 (0)