timers: test timeout NaN warning in promises version
Hey, https://github.com/nodejs/node/pull/46678 landed adding a warning when NaN is passed to timers. I noticed a test was missing for timers/promises there (the code works on both paths) - rather than holding that PR I figured we'd land it and I'd PR a test for timers/promises in a separate PR.
cc @jakecastelli
weird the CI bot is not here yet 🤔
Edit: ah CI is in lockdown missed the PSA oops
The subsystem should be test: given it only touch the tests. Removed https://github.com/nodejs/node/labels/semver-major in favor of dont-land labels.
CI: https://ci.nodejs.org/job/node-test-pull-request/60042/
CI: https://ci.nodejs.org/job/node-test-pull-request/60503/
CI: https://ci.nodejs.org/job/node-test-pull-request/60504/
CI: https://ci.nodejs.org/job/node-test-pull-request/60505/
CI: https://ci.nodejs.org/job/node-test-pull-request/60506/
CI: https://ci.nodejs.org/job/node-test-pull-request/60508/
CI: https://ci.nodejs.org/job/node-test-pull-request/60507/
CI: https://ci.nodejs.org/job/node-test-pull-request/60509/
CI: https://ci.nodejs.org/job/node-test-pull-request/60511/
CI: https://ci.nodejs.org/job/node-test-pull-request/60510/
CI: https://ci.nodejs.org/job/node-test-pull-request/60512/
CI: https://ci.nodejs.org/job/node-test-pull-request/60513/
Landed in 259163802c8ab2981da66f36b3c8fa43302869cf