Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit a43aa0e

Browse filesBrowse files
moe-dizzleMylesBorins
moe-dizzle
authored and
MylesBorins
committed
test: changed test1 of test-vm-timeout.js
test: changed test1 of test-vm-timeout.js so that entire error message would be matched in assert.throw. Before test 1 of test-vm-timeout.js would match any error, now it looks specifically for the error message "Script execution timed out." PR-URL: #11590 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
1 parent 1035967 commit a43aa0e
Copy full SHA for a43aa0e

File tree

Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Collapse file

‎test/parallel/test-vm-timeout.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-vm-timeout.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const vm = require('vm');
66
// Test 1: Timeout of 100ms executing endless loop
77
assert.throws(function() {
88
vm.runInThisContext('while(true) {}', { timeout: 100 });
9-
});
9+
}, /^Error: Script execution timed out\.$/);
1010

1111
// Test 2: Timeout must be >= 0ms
1212
assert.throws(function() {

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.