search

avajs / ava

Node.js test runner that lets you develop with confidence 🚀
MIT License
20.74k stars 1.41k forks source link

Disable timeouts in debug mode #2355

Closed novemberborn closed 4 years ago

novemberborn commented 4 years ago

When using the debug mode in the upcoming v3 release, we should disable any timeouts. It can usually take a while to set up the debugging and it's a shame if AVA's main process then kills the worker.

We should be able to add the check here:

https://github.com/avajs/ava/blob/090884b5040a1d4b02a18939245f7c2305228e00/lib/api.js#L72

If apiOptions.debug is truthy, then we're in debug mode.

novemberborn commented 4 years ago

I suppose we should print a warning in the CLI as well. Like this:

https://github.com/avajs/ava/blob/090884b5040a1d4b02a18939245f7c2305228e00/lib/cli.js#L229

After here, assuming combined.timeout has a value:

https://github.com/avajs/ava/blob/090884b5040a1d4b02a18939245f7c2305228e00/lib/cli.js#L221