lib: test_runner#mock:timers respeced timeout_max behaviour

PR-URL: #55375 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Claudio Wunder <[email protected]>

Author: badkeyy

lib/internal/test_runner/mock/mock_timers.js

@@ -38,6 +38,8 @@ const{
 },
 }=require('internal/errors');
 
+const{TIMEOUT_MAX}=require('internal/timers');
+
 constPriorityQueue=require('internal/priority_queue');
 constnodeTimers=require('timers');
 constnodeTimersPromises=require('timers/promises');
@@ -288,6 +290,10 @@ class MockTimers{
 }
 
  #createTimer(isInterval,callback,delay, ...args){
+if(delay>TIMEOUT_MAX){
+delay=1;
+}
+
 consttimerId=this.#currentTimer++;
 constopts={
 __proto__: null,

test/parallel/test-runner-mock-timers.js

@@ -251,6 +251,22 @@ describe('Mock Timers Test Suite', () =>{
 done();
 }),timeout);
 });
+
+it('should change timeout to 1ms when it is >= 2 ** 31',(t)=>{
+t.mock.timers.enable({apis: ['setTimeout']});
+constfn=t.mock.fn();
+global.setTimeout(fn,2**31);
+t.mock.timers.tick(1);
+assert.strictEqual(fn.mock.callCount(),1);
+});
+
+it('should change the delay to one if timeout < 0',(t)=>{
+t.mock.timers.enable({apis: ['setTimeout']});
+constfn=t.mock.fn();
+global.setTimeout(fn,-1);
+t.mock.timers.tick(1);
+assert.strictEqual(fn.mock.callCount(),1);
+});
 });
 
 describe('clearTimeout Suite',()=>{