Fake timers with a delay of NaN never get run

[corhere]

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Timers with a timeout of NaN can never get run when fake timers are in use.

If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.
https://repl.it/repls/MedicalSolidLines

jest.useFakeTimers();

it('works with fake timers', (done) => {
  setTimeout(done, NaN);
  jest.runAllTimers();
});

it('works with real timers', (done) => {
  jest.useRealTimers();
  setTimeout(done, NaN);
});

What is the expected behavior?
setTimeout(callback, NaN) is treated the same as setTimeout(callback, 0) when fake timers are in use. This matches the behavior specified by the HTML Living Standard.
https://html.spec.whatwg.org/multipage/webappapis.html#windoworworkerglobalscope-mixin:dom-settimeout
https://heycam.github.io/webidl/#abstract-opdef-converttoint

[SimenB]

Nice catch! Should be easy enough to fix, just need to expand handling here: https://github.com/facebook/jest/blob/1a51672c7597d900953a0b95e4430e65eaa04259/packages/jest-util/src/fake_timers.js#L489-L491

[keithkml]

Shouldn't this issue be closed if #5966 was merged?

[SimenB]

Indeed!

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.