Skip to content

Commit 15c295a

Browse files
Fishrock123Myles Borins
authored and
Myles Borins
committed
test: use regular timeout times for ARMv8
ARMv8 machines are typically quite fast and likely may not need extended timeout times. PR-URL: #4248 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Rod Vagg <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
1 parent fd250b8 commit 15c295a

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

test/common.js

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -243,10 +243,15 @@ exports.platformTimeout = function(ms) {
243243
if (process.arch !== 'arm')
244244
return ms;
245245

246-
if (process.config.variables.arm_version === '6')
246+
const armv = process.config.variables.arm_version;
247+
248+
if (armv === '6')
247249
return 7 * ms; // ARMv6
248250

249-
return 2 * ms; // ARMv7 and up.
251+
if (armv === '7')
252+
return 2 * ms; // ARMv7
253+
254+
return ms; // ARMv8+
250255
};
251256

252257
var knownGlobals = [setTimeout,

0 commit comments

Comments
 (0)