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

test: allow for slow hosts in spawnSync() test

Browse Source 
test-child-process-spawnsync-timeout failed from time to time on
Raspberry Pi devices. Use common.platformTimeout() to allow a little
more time to run on resource-constrained hosts.

PR-URL: https://github.com/nodejs/node/pull/10998
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
Rich Trott 2017-01-25 09:15:25 -08:00
parent c74e6fe651
commit b6d2fc9b0f
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
test/parallel/test-child-process-spawnsync-timeout.js
View File

@ -1,11 +1,11 @@
'use strict';
require('../common');
const common = require('../common');
const assert = require('assert');
const spawnSync = require('child_process').spawnSync;
const TIMER = 200;
const SLEEP = 5000;
const SLEEP = common.platformTimeout(5000);
switch (process.argv[2]) {
case 'child':
@ -19,7 +19,6 @@ switch (process.argv[2]) {
const ret = spawnSync(process.execPath, [__filename, 'child'],
{timeout: TIMER});
assert.strictEqual(ret.error.errno, 'ETIMEDOUT');
console.log(ret);
const end = Date.now() - start;
assert(end < SLEEP);
assert(ret.status > 128 || ret.signal);