test: increase spin for eventloop test on s390

mhdawson · aduh95 · commit 96a38044ee2b · 2025-01-02T15:28:47.000+01:00
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> PR-URL: #56228 Refs: #41286 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
diff --git a/test/sequential/sequential.status b/test/sequential/sequential.status
@@ -48,7 +48,3 @@ test-tls-securepair-client: PASS, FLAKY
 [$arch==arm]
 # https://github.com/nodejs/node/issues/49933
 test-watch-mode-inspect: SKIP
-
-[$arch==s390x]
-# https://github.com/nodejs/node/issues/41286
-test-performance-eventloopdelay: PASS, FLAKY
diff --git a/test/sequential/test-performance-eventloopdelay.js b/test/sequential/test-performance-eventloopdelay.js
@@ -3,6 +3,7 @@
 
 const common = require('../common');
 const assert = require('assert');
+const os = require('os');
 const{
 monitorEventLoopDelay
 } = require('perf_hooks');
@@ -51,9 +52,13 @@ const{sleep } = require('internal/util');
 }
 
{
+ const s390x = os.arch() === 's390x';
 const histogram = monitorEventLoopDelay({resolution: 1 });
 histogram.enable();
 let m = 5;
+ if (s390x){
+ m = m * 2;
+ }
 function spinAWhile(){
 sleep(1000);
 if (--m > 0){

-Original file line number
+Diff line change
 constcommon=require('../common');
 constassert=require('assert');
 +constos=require('os');
 const{
  monitorEventLoopDelay
 }=require('perf_hooks');
+}
+{
 +consts390x=os.arch()==='s390x';
 consthistogram=monitorEventLoopDelay({resolution: 1});
 histogram.enable();
 letm=5;
 +if(s390x){
 +m=m*2;
 +}
 functionspinAWhile(){
 sleep(1000);
 if(--m>0){