779402ec5ftest: changed test2 of test-vm-timeout.js so that entire error message
would be matched in assert.throw.
Before test 2 of test-vm-timeout.js would match any RangeError,
now it looks specifically for the error message
"RangeError: timeout must be a positive number"
test: changed test 3 of test-vm-timeout.js so that entire error message
would be matched in assert.throw.
Before test 3 of test-vm-timeout.js would match any RangeError,
now it looks specifically for the error message
"RangeError: timeout must be a positive number"
PR-URL: https://github.com/nodejs/node/pull/13453
Refs: https://github.com/nodejs/node/issues/13454
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Yuta Hiroto <hello@about-hiroppy.com>
57 lines
1.8 KiB
JavaScript
57 lines
1.8 KiB
JavaScript
'use strict';
|
|
require('../common');
|
|
const assert = require('assert');
|
|
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() {
|
|
vm.runInThisContext('', { timeout: -1 });
|
|
}, /^RangeError: timeout must be a positive number$/);
|
|
|
|
// Test 3: Timeout of 0ms
|
|
assert.throws(function() {
|
|
vm.runInThisContext('', { timeout: 0 });
|
|
}, /^RangeError: timeout must be a positive number$/);
|
|
|
|
// Test 4: Timeout of 1000ms, script finishes first
|
|
vm.runInThisContext('', { timeout: 1000 });
|
|
|
|
// Test 5: Nested vm timeouts, inner timeout propagates out
|
|
assert.throws(function() {
|
|
const context = {
|
|
log: console.log,
|
|
runInVM: function(timeout) {
|
|
vm.runInNewContext('while(true) {}', context, { timeout: timeout });
|
|
}
|
|
};
|
|
vm.runInNewContext('runInVM(10)', context, { timeout: 10000 });
|
|
throw new Error('Test 5 failed');
|
|
}, /Script execution timed out./);
|
|
|
|
// Test 6: Nested vm timeouts, outer timeout is shorter and fires first.
|
|
assert.throws(function() {
|
|
const context = {
|
|
runInVM: function(timeout) {
|
|
vm.runInNewContext('while(true) {}', context, { timeout: timeout });
|
|
}
|
|
};
|
|
vm.runInNewContext('runInVM(10000)', context, { timeout: 100 });
|
|
throw new Error('Test 6 failed');
|
|
}, /Script execution timed out./);
|
|
|
|
// Test 7: Nested vm timeouts, inner script throws an error.
|
|
assert.throws(function() {
|
|
const context = {
|
|
runInVM: function(timeout) {
|
|
vm.runInNewContext('throw new Error(\'foobar\')', context, {
|
|
timeout: timeout
|
|
});
|
|
}
|
|
};
|
|
vm.runInNewContext('runInVM(10000)', context, { timeout: 100000 });
|
|
}, /foobar/);
|