Commit b5e8413
test: allow for slow hosts in spawnSync() test
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: #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>1 parent 062c851 commit b5e8413
1 file changed
Lines changed: 2 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1,11 +1,11 @@ | |||
| 1 | 1 | 'use strict'; | |
| 2 | - require('../common'); | ||
| 2 | + const common = require('../common'); | ||
| 3 | 3 | const assert = require('assert'); | |
| 4 | 4 | ||
| 5 | 5 | const spawnSync = require('child_process').spawnSync; | |
| 6 | 6 | ||
| 7 | 7 | const TIMER = 200; | |
| 8 | - const SLEEP = 5000; | ||
| 8 | + const SLEEP = common.platformTimeout(5000); | ||
| 9 | 9 | ||
| 10 | 10 | switch (process.argv[2]) { | |
| 11 | 11 | case 'child': | |
@@ -19,7 +19,6 @@ switch (process.argv[2]) { | |||
| 19 | 19 | const ret = spawnSync(process.execPath, [__filename, 'child'], | |
| 20 | 20 | {timeout: TIMER}); | |
| 21 | 21 | assert.strictEqual(ret.error.errno, 'ETIMEDOUT'); | |
| 22 | - console.log(ret); | ||
| 23 | 22 | const end = Date.now() - start; | |
| 24 | 23 | assert(end < SLEEP); | |
| 25 | 24 | assert(ret.status > 128 || ret.signal); | |
0 commit comments