test_runner: add mock-timers support for AbortSignal.timeout

[DeveloperViraj]

Fixes: #60509

This PR completes the fix for making AbortSignal.timeout() compatible with
the test runner's mock timers, and adds a regression test to ensure correct
behavior going forward.

What’s included

1. Mock timers fix

• Added "AbortSignal.timeout" as a discrete mock-timers API target.
• Implemented dedicated store/restore handlers for the original
  AbortSignal.timeout property (no silent failures).
• Integrated the mock implementation into the toFake/toReal paths to match
  the design of other mockable timer APIs.
• Restored documentation comments and internal validation that were
  unintentionally removed earlier.

2. New regression test

• Added test/parallel/test-mock-timers-abortsignal-timeout.js
• Verifies that:
  • enabling mock timers with { apis: ['AbortSignal.timeout'] } works
  • mock.tick() correctly triggers abortion after the expected delay

Notes

I created a fresh branch to ensure a clean diff without unrelated changes and
to fully incorporate the feedback from @Renegade334 and @ErickWendel.

Please let me know if you'd like any adjustments to the test or implementation.
Thanks for the clear guidance and review support

[nodejs-github-bot]

Review requested:

• @nodejs/test_runner

[DeveloperViraj]

Hi @Renegade334 @ErickWendel 👋

I’ve opened a fresh PR with a clean branch so it’s easier to review.
All the requested changes have been applied, including the dedicated API
target for AbortSignal.timeout and the new regression test.

Please let me know if anything needs adjusting, happy to update it!
Thanks again for the guidance. 🙏

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.62%. Comparing base (3330e5c) to head (177ab11).
⚠️ Report is 1054 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #60751      +/-   ##
==========================================
+ Coverage   88.54%   89.62%   +1.08%     
==========================================
  Files         703      706       +3     
  Lines      208252   219181   +10929     
  Branches    40153    41988    +1835     
==========================================
+ Hits       184391   196441   +12050     
+ Misses      15869    14617    -1252     
- Partials     7992     8123     +131     

Files with missing lines	Coverage Δ
lib/internal/test_runner/mock/mock_timers.js	98.76% <100.00%> (+0.07%)	⬆️

... and 438 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[DeveloperViraj]

I've updated both the implementation and the regression test:

Fixed the missing function name & trailing comma issues in mock_timers.js

Cleaned up unused variables

Ensured the mock AbortSignal.timeout() matches Node.js behavior

Updated the test file to avoid restricted syntax and comply with Node’s linting rules

The PR is ready for the next round of review
Let me know if you’d like any additional adjustments!

[Renegade334]

This is looking good! Just a couple of recommended tweaks.

[StefanStojanovic]

LGTM. Make sure you fix issues found by the linter.

[DeveloperViraj]

All requested updates have now been applied.

Summary of changes:

Updated mock_timers.js according to the reviewer’s feedback

Preserved the required comment and applied all suggested code adjustments

Updated the test file to follow the Node.js test style (including loading common first)

Verified everything locally, all lint issues are resolved

@StefanStojanovic @Renegade334
Please let me know if anything else is needed. Thanks for the review and guidance!

[DeveloperViraj]

@Renegade334
Just a quick follow-up, I’ve applied all the requested changes and fixed all lint issues locally.
If you get a moment, could you please take another look and let me know if anything else is needed?
Thanks a lot for your time and guidance!

[DeveloperViraj]

Hi @Renegade334

I’ve addressed the requested changes:

Reverted the EnableOptions typedef to the narrower SupportedApis version

Updated the AbortSignal.timeout test to use the public mock API from node:test

Removed internal imports and the unused variable as suggested

Everything should now align with the review feedback.
Please let me know if anything else needs adjusting.

Stale

[JakobJingleheimer]

@DeveloperViraj I think this is very close to landing. One of the items from René #60751 (comment) I consider not a nit. Could you take a look at it? 🙂

[Renegade334]

Let's get this over the line.

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/72806/