nodejs
diff --git a/Collapse file
‎lib/internal/test_runner/test.js‎
Copy file name to clipboardExpand all lines: lib/internal/test_runner/test.js
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎lib/internal/test_runner/test.js‎
Copy file name to clipboardExpand all lines: lib/internal/test_runner/test.js
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎test/fixtures/test-runner/mock-timers-with-timeout.js‎
Copy file name to clipboard
+43Lines changed: 43 additions & 0 deletions b/Collapse file
‎test/fixtures/test-runner/mock-timers-with-timeout.js‎
Copy file name to clipboard
+43Lines changed: 43 additions & 0 deletions
diff --git a/Collapse file
‎test/parallel/test-runner-mock-timers-with-timeout.js‎
Copy file name to clipboard
+14Lines changed: 14 additions & 0 deletions b/Collapse file
‎test/parallel/test-runner-mock-timers-with-timeout.js‎
Copy file name to clipboard
+14Lines changed: 14 additions & 0 deletions
@@ -149,7 +149,7 @@ function stopTest(timeout, signal) {
 
     disposeFunction = () => {
       abortListener[SymbolDispose]();
-      timer[SymbolDispose]();
+      clearTimeout(timer);
     };
   }
 
 
@@ -0,0 +1,43 @@
+'use strict';
+
+// Simulate @sinonjs/fake-timers: patch the timers module BEFORE
+// the test runner is loaded, so the test runner captures the patched
+// versions at import time.
+const nodeTimers = require('node:timers');
+const originalSetTimeout = nodeTimers.setTimeout;
+const originalClearTimeout = nodeTimers.clearTimeout;
+
+const fakeTimers = new Map();
+let nextId = 1;
+
+nodeTimers.setTimeout = (fn, delay, ...args) => {
+  const id = nextId++;
+  const timer = originalSetTimeout(fn, delay, ...args);
+  fakeTimers.set(id, timer);
+  // Sinon fake timers return an object with unref/ref but without
+  // Symbol.dispose, which would cause the test runner to throw.
+  return { id, unref() {}, ref() {} };
+};
+
+nodeTimers.clearTimeout = (id) => {
+  if (id != null && typeof id === 'object') id = id.id;
+  const timer = fakeTimers.get(id);
+  if (timer) {
+    originalClearTimeout(timer);
+    fakeTimers.delete(id);
+  }
+};
+
+// Now load the test runner - it will capture our patched setTimeout/clearTimeout
+const { test } = require('node:test');
+
+test('test with fake timers and timeout', { timeout: 10_000 }, () => {
+  // This test verifies that the test runner works when setTimeout returns
+  // an object without Symbol.dispose (like sinon fake timers).
+  // Previously, the test runner called timer[Symbol.dispose]() which would
+  // throw TypeError on objects returned by fake timer implementations.
+});
+
+// Restore
+nodeTimers.setTimeout = originalSetTimeout;
+nodeTimers.clearTimeout = originalClearTimeout;
@@ -0,0 +1,14 @@
+'use strict';
+require('../common');
+const fixtures = require('../common/fixtures');
+const assert = require('node:assert');
+const { spawnSync } = require('node:child_process');
+const { test } = require('node:test');
+
+test('mock timers do not break test timeout cleanup', async () => {
+  const fixture = fixtures.path('test-runner', 'mock-timers-with-timeout.js');
+  const cp = spawnSync(process.execPath, ['--test', fixture], {
+    timeout: 30_000,
+  });
+  assert.strictEqual(cp.status, 0, `Test failed:\nstdout: ${cp.stdout}\nstderr: ${cp.stderr}`);
+});
Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ function stopTest(timeout, signal) {`
`149`	`149`
`150`	`150`	`disposeFunction = () => {`
`151`	`151`	`abortListener[SymbolDispose]();`
`152`		`- timer[SymbolDispose]();`
	`152`	`+ clearTimeout(timer);`
`153`	`153`	`};`
`154`	`154`	`}`
`155`	`155`