mirror of
https://github.com/nodejs/node.git
synced 2025-05-06 01:43:00 +00:00
efc31503b3
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: https://github.com/nodejs/node/issues/6727 PR-URL: https://github.com/nodejs/node/pull/7373 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
35 lines
950 B
JavaScript
35 lines
950 B
JavaScript
'use strict';
|
|
require('../common');
|
|
var assert = require('assert');
|
|
var vm = require('vm');
|
|
|
|
// Test 1: Timeout of 100ms executing endless loop
|
|
assert.throws(function() {
|
|
vm.runInThisContext('while(true) {}', { timeout: 100 });
|
|
});
|
|
|
|
// Test 2: Timeout must be >= 0ms
|
|
assert.throws(function() {
|
|
vm.runInThisContext('', { timeout: -1 });
|
|
}, RangeError);
|
|
|
|
// Test 3: Timeout of 0ms
|
|
assert.throws(function() {
|
|
vm.runInThisContext('', { timeout: 0 });
|
|
}, RangeError);
|
|
|
|
// Test 4: Timeout of 1000ms, script finishes first
|
|
vm.runInThisContext('', { timeout: 1000 });
|
|
|
|
// Test 5: Nested vm timeouts, inner timeout propagates out
|
|
assert.throws(function() {
|
|
var 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./);
|