Skip to content

Commit f9562c9

Browse files
committed
test: increase spin for eventloop test on s390
It was excluded as it was failing intermittently. Likely that s390 was just so fast times were rounded down to 0. Increase the spin time on s390x only. Signed-off-by: Michael Dawson <midawson@redhat.com>
1 parent 142adef commit f9562c9

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

test/sequential/sequential.status

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,6 @@ test-tls-securepair-client: PASS, FLAKY
4949
# https://github.com/nodejs/node/issues/49933
5050
test-watch-mode-inspect: SKIP
5151

52-
[$arch==s390x]
53-
# https://github.com/nodejs/node/issues/41286
54-
test-performance-eventloopdelay: PASS, FLAKY
55-
5652
[$system==linux && $arch==ppc64]
5753
# https://github.com/nodejs/node/issues/50740
5854
test-single-executable-application-assets-raw: PASS, FLAKY

test/sequential/test-performance-eventloopdelay.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33

44
const common = require('../common');
55
const assert = require('assert');
6+
const os = require('os');
67
const {
78
monitorEventLoopDelay
89
} = require('perf_hooks');
@@ -51,9 +52,13 @@ const { sleep } = require('internal/util');
5152
}
5253

5354
{
55+
const s390x = os.arch() === 's390x';
5456
const histogram = monitorEventLoopDelay({ resolution: 1 });
5557
histogram.enable();
5658
let m = 5;
59+
if (s390x) {
60+
m = m * 2;
61+
}
5762
function spinAWhile() {
5863
sleep(1000);
5964
if (--m > 0) {

0 commit comments

Comments
 (0)