Skip to content

Show last run time in humanized format #976

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

Closed
tusharmath opened this issue Jul 22, 2016 · 9 comments
Closed

Show last run time in humanized format #976

tusharmath opened this issue Jul 22, 2016 · 9 comments
Labels
enhancement new functionality question

Comments

@tusharmath
Copy link

As of now (version 0.15.2) ava cli shows the last run time as follows —
2 passed [14:00:46]

Though useful, its not very clear unless someone keeps checking what the current time is and then match it with what's displayed. It would be great if it could say something like —

  • a few seconds ago
  • a minute ago
  • 5 minutes ago

and so on.

@vadimdemedes
Copy link
Contributor

I agree, something like a minute ago would be faster to understand when was the last run. Thanks for the suggestion, @tusharmath, let's see what other people think!

@vadimdemedes vadimdemedes added enhancement new functionality question labels Jul 24, 2016
@novemberborn
Copy link
Member

Wouldn't we have to continuously update the timestamp? (I assume this is to do with the watcher, right?)

@vadimdemedes
Copy link
Contributor

@novemberborn Good point :)

@novemberborn
Copy link
Member

I agree having a relative timestamp would be nice, but we'd have to keep it in sync which is a bit more work.

@tusharmath
Copy link
Author

tusharmath commented Jul 25, 2016

WE can simple put an interval for 5 seconds initially then then change it to minutes once the last run goes beyond 1 min and then 15 mins (or something else)

@vadimdemedes
Copy link
Contributor

That would involve rewriting all the previous output, which is a nightmare.

@simonbuchan
Copy link

Is there any reason ava --watch leaves the previous errors up? This would be out of scope for this issue, but ideally it would re-write to show only the still failing tests, so if that was done then there is only one timestamp to worry about here.

@novemberborn
Copy link
Member

@simonbuchan

Is there any reason ava --watch leaves the previous errors up? This would be out of scope for this issue, but ideally it would re-write to show only the still failing tests

The reporter buffer gets reset between test runs, so it can't clear earlier output. TBH I'm also not sure how terminals respond if you remove more lines than are currently displayed.


#939 implies a different approach to how we output logs, which could help with this feature request.

@novemberborn
Copy link
Member

#939 is closed for now. Consequently I don't think we can make changes here anytime soon, so closing this as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement new functionality question
Projects
None yet
Development

No branches or pull requests

4 participants