Skip to content

Commit

Permalink
test: use regular timeout times for ARMv8
Browse files Browse the repository at this point in the history
ARMv8 machines are typically quite fast and likely may not need
extended timeout times.

PR-URL: #4248
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
Fishrock123 committed Dec 14, 2015
1 parent 671347c commit 668449a
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions test/common.js
Original file line number Diff line number Diff line change
Expand Up @@ -245,10 +245,15 @@ exports.platformTimeout = function(ms) {
if (process.arch !== 'arm')
return ms;

if (process.config.variables.arm_version === '6')
const armv = process.config.variables.arm_version;

if (armv === '6')
return 7 * ms; // ARMv6

return 2 * ms; // ARMv7 and up.
if (armv === '7')
return 2 * ms; // ARMv7

return ms; // ARMv8+
};

var knownGlobals = [setTimeout,
Expand Down

0 comments on commit 668449a

Please sign in to comment.