New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Set test.meta.debugMode = true
when timeout has been disabled for debugging
#3152
Comments
I like it! @bitjson are you game to implement this? Let me know if you need some pointers. |
@novemberborn I'd be happy to try! Though I'll need help with testing the feature. I tried hacking something together initially, but I probably need a better understanding of the codebase. Could you point me to the location where |
Hi @bitjson, I think the Lines 71 to 77 in c749348
It's then made available through the CJS and ESM entrypoints and some indirection.
Line 78 in c749348
The Line 186 in c749348
|
I'm using AVA to test a program that interacts with a Postgres database and has timing-related tests (code here).
I'm analyzing the
stdout
from a child process to verify it's producing the expected results. Results can sometimes come in unexpected orders, so I need close control over how long to wait for each, and timers run in parallel, sot.timeout()
isn't really a good fit.It's easy enough to manage the timers myself, but one inconvenience is that I need to manually disable them when debugging the tests. (AVA logs
⚠ The timeout option has been disabled to help with debugging.
which is perfect, but my timers still interfere with debugging.)Also, my tests can't inspect whether or not the inspector is active for some reason (probably something about them being in separate processes?) so I can't use the same code as AVA here:
ava/lib/cli.js
Lines 122 to 133 in 639b905
It would be very helpful if AVA somehow exposed this boolean to my tests so I can have them deliberately behave differently in debug mode. Maybe by adding it to
test.meta
? E.g.:The text was updated successfully, but these errors were encountered: