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 <[email protected]> Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Mar 8, 2017 · 5cb1b05 · 5cb1b05
1 parent dc207cd
commit 5cb1b05
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/test/parallel/test-child-process-spawnsync-timeout.js b/test/parallel/test-child-process-spawnsync-timeout.js
@@ -1,11 +1,11 @@
 'use strict';
-require('../common');
-var assert = require('assert');
+const common = require('../common');
+const assert = require('assert');
 
 var spawnSync = require('child_process').spawnSync;
 
-var TIMER = 200;
-var SLEEP = 5000;
+const TIMER = 200;
+const SLEEP = common.platformTimeout(5000);
 
 switch (process.argv[2]) {
   case 'child':
@@ -19,8 +19,7 @@ switch (process.argv[2]) {
     var ret = spawnSync(process.execPath, [__filename, 'child'],
                         {timeout: TIMER});
     assert.strictEqual(ret.error.errno, 'ETIMEDOUT');
-    console.log(ret);
-    var end = Date.now() - start;
+    const end = Date.now() - start;
     assert(end < SLEEP);
     assert(ret.status > 128 || ret.signal);
     break;