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: nodejs#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 authored and Michael Scovetta committed Apr 2, 2016
1 parent bb20511 commit d4e4a0d
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 d4e4a0d

Please sign in to comment.