1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

test: changed test1 of test-vm-timeout.js

Browse Source 
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: https://github.com/nodejs/node/pull/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>
This commit is contained in:
maurice_hayward 2017-02-27 16:37:05 -05:00 committed by Anna Henningsen
parent 531de63c14
commit 94d1c8d1b0
No known key found for this signature in database
GPG Key ID: D8B9F5AEAE84E4CF
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/parallel/test-vm-timeout.js
View File

@ -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() {