Skip to content

Commit

Permalink
test: changed test1 of test-vm-timeout.js
Browse files Browse the repository at this point in the history
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 <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
  • Loading branch information
moe-dizzle authored and MylesBorins committed Apr 17, 2017
1 parent c283682 commit 9ecc2c9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/parallel/test-vm-timeout.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ const vm = require('vm');
// Test 1: Timeout of 100ms executing endless loop
assert.throws(function() {
vm.runInThisContext('while(true) {}', { timeout: 100 });
});
}, /^Error: Script execution timed out\.$/);

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

0 comments on commit 9ecc2c9

Please sign in to comment.