Ava: Print pending tests when interrupting AVA, or after a timeout
If you interrupt AVA (Ctrl+C/SIGTERM) while tests are still pending, AVA should print those tests before exiting. Similarly it should print pending tests when --timeout is reached.
This should help debug which tests are taking too long.
Related to #171.
dflupu earned $80.00 by resolving this issue!
- Checkout the Issuehunt explorer to discover more funded issues.
- Need some help from other developers? Add your repositories on Issuehunt to raise funds.
novemberborn
All 20 comments
Great idea!
vadimdemedes
on 1 Mar 2016
:+1:
sindresorhus
on 1 Mar 2016
This would be awesome. 馃憤
radiovisual
on 24 Apr 2016
Would be really useful to use why-is-node-running to also show why tests are still pending.
sindresorhus
on 3 Oct 2016
Just curious, has there been any movement on this issue?
cc: @novemberborn
QuotableWater7
on 22 Mar 2017
@QuotableWater7 nope. Are you interested in taking this on?
novemberborn
on 22 Mar 2017
This would be tremendously useful. Is this still something that is planned @novemberborn?
austinkelleher
on 14 Nov 2017
@austinkelleher most definitely. Would you want to give it a go?
novemberborn
on 14 Nov 2017
Ran into this problem myself - gonna try hacking something together if y'all don't mind. 馃檪
ace-n
on 19 Nov 2017
This will be easier to implement now that #1722 has landed, though perhaps we should implement it in the reporters, in which case it'd be better to wait for https://github.com/avajs/ava/pull/1776.
novemberborn
on 22 Apr 2018
any update on this now https://github.com/avajs/ava/pull/1722 is merged ? Would be very useful 馃槃
vancouverwill
on 24 Jul 2018
@vancouverwill I haven't had much time to work on #1776. I do think it should be implemented in the reporters, but I'm happy with some duplication if that unblocks.
Would you be interested in working on this?
novemberborn
on 26 Jul 2018
@novemberborn hey yeah I would be interested, could you possibly give me some guidelines in terms of code style you would Iike or any goals with it? Thanks
vancouverwill
on 26 Jul 2018
@vancouverwill we use XO for code style.
The reporters should receive events when tests are declared and when they finish, so it'll be possible to keep a list of pending tests (and test files, too). Then in lib/cli there should be a hook for SIGINT & SIGTERM to make the reporters stop listening to incoming events and shut down the workers, and then print the pending tests & test files. The same for --timeout handling.
At least that's the theory, it may be a bit different in practice 馃槃
novemberborn
on 29 Jul 2018
great, i've started to take a look at it, busy with the day job but hopefully I can get more time on the wk 馃槃
vancouverwill
on 2 Aug 2018
can you assign me so no one else ends up having to create duplicate/wasted work?
vancouverwill
on 4 Aug 2018
1886 implements this for timeouts, in the --verbose logger.
We still want to implement this for when the process is interrupted ("killed"), and in the default mini reporter too.
Thank you @vancouverwill for getting us this far!
novemberborn
on 15 Sep 2018
@issuehuntfest has funded $80.00 to this issue. See it on IssueHunt
IssueHuntBot
on 22 Dec 2018
@dflupu has submitted a pull request. See it on IssueHunt
IssueHuntBot
on 27 Jan 2019
@sindresorhus has rewarded $72.00 to @dflupu. See it on IssueHunt
- :moneybag: Total deposit: $80.00
- :tada: Repository reward(0%): $0.00
- :wrench: Service fee(10%): $8.00
IssueHuntBot
on 27 Jan 2019
Related issues
fregante
路
3Comments
dlumma
路
4Comments
gajus
路
5Comments
ivogabe
路
5Comments
ehmicky
路
4Comments
Most helpful comment
Would be really useful to use
why-is-node-runningto also show why tests are still pending.