Skip to content

Commit eab9ced

Browse files
addaleaxMyles Borins
authored and
Myles Borins
committed
test: fix flaky test-vm-timeout
Likely fix the flaky parallel/test-vm-timeout. Increase the outer timeout in the test checking for nested timeouts with `vm` scripts so that its firing won’t interfere with the inner timeout. Fixes: #6727 PR-URL: #7373 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent 3eea551 commit eab9ced

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

test/parallel/test-vm-timeout.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@ assert.throws(function() {
2929
vm.runInNewContext('while(true) {}', context, { timeout: timeout });
3030
}
3131
};
32-
vm.runInNewContext('runInVM(10)', context, { timeout: 100 });
32+
vm.runInNewContext('runInVM(10)', context, { timeout: 10000 });
3333
throw new Error('Test 5 failed');
3434
}, /Script execution timed out./);

0 commit comments

Comments
 (0)