test_runner: fix mock timer setInterval tests

Some tests relied on the custom return value and were broken by the
change to always return the value provided by the caller.

Author: mika-fischer

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

@@ -667,10 +667,11 @@ describe('Mock Timers Test Suite', () => {
 
         const expectedIterations = 5;
         const interval = 1000;
-        const startedAt = Date.now();
+        let time = 0;
         async function run() {
           const times = [];
-          for await (const time of nodeTimersPromises.setInterval(interval, startedAt)) {
+          for await (const _ of nodeTimersPromises.setInterval(interval)) { // eslint-disable-line no-unused-vars
+            time += interval;
             times.push(time);
             if (times.length === expectedIterations) break;
 
@@ -688,7 +689,7 @@ describe('Mock Timers Test Suite', () => {
         const timeResults = await r;
         assert.strictEqual(timeResults.length, expectedIterations);
         for (let it = 1; it < expectedIterations; it++) {
-          assert.strictEqual(timeResults[it - 1], startedAt + (interval * it));
+          assert.strictEqual(timeResults[it - 1], interval * it);
         }
       });
 
@@ -711,7 +712,7 @@ describe('Mock Timers Test Suite', () => {
 
         const interval = 100;
         const abortController = new AbortController();
-        const intervalIterator = nodeTimersPromises.setInterval(interval, Date.now(), {
+        const intervalIterator = nodeTimersPromises.setInterval(interval, undefined, {
           signal: abortController.signal
         });
 
@@ -723,8 +724,6 @@ describe('Mock Timers Test Suite', () => {
         t.mock.timers.tick(interval);
 
         const firstResult = await first;
-        // Interval * 2 because value can be a little bit greater than interval
-        assert.ok(firstResult.value < Date.now() + interval * 2);
         assert.strictEqual(firstResult.done, false);
 
         await assert.rejects(() => second, {
@@ -757,13 +756,12 @@ describe('Mock Timers Test Suite', () => {
         const signal = controller.signal;
         const interval = 200;
         const expectedIterations = 2;
-        const startedAt = Date.now();
-        const timeResults = [];
+        let numIterations = 0;
         async function run() {
-          const it = nodeTimersPromises.setInterval(interval, startedAt, { signal });
-          for await (const time of it) {
-            timeResults.push(time);
-            if (timeResults.length === 5) break;
+          const it = nodeTimersPromises.setInterval(interval, undefined, { signal });
+          for await (const _ of it) { // eslint-disable-line no-unused-vars
+            numIterations += 1;
+            if (numIterations === 5) break;
           }
         }
 
@@ -779,11 +777,7 @@ describe('Mock Timers Test Suite', () => {
         await assert.rejects(() => r, {
           name: 'AbortError',
         });
-        assert.strictEqual(timeResults.length, expectedIterations);
-
-        for (let it = 1; it < expectedIterations; it++) {
-          assert.strictEqual(timeResults[it - 1], startedAt + (interval * it));
-        }
+        assert.strictEqual(numIterations, expectedIterations);
 
       });