Skip to content

Output in watch mode should be less verbose #670

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
ben-eb opened this issue Mar 22, 2016 · 5 comments
Closed

Output in watch mode should be less verbose #670

ben-eb opened this issue Mar 22, 2016 · 5 comments
Assignees
Labels
enhancement new functionality

Comments

@ben-eb
Copy link
Contributor

ben-eb commented Mar 22, 2016

Description

I like the watch mode so far, but I think its verbosity could be reduced. Using the default reporter, the output is this:

screen shot 2016-03-22 at 10 51 11

A good solution to this would be to print less empty lines; but my ideal solution would be that these messages should not show at all. Instead, it would be really great if it showed simply a wait cursor, and only printed a message when a test fails.

Environment

Node.js v5.4.1
darwin 15.0.0
@sindresorhus
Copy link
Member

@novemberborn Any reason we can't just overwrite the output like we do in the mini-reporter?

@jamestalmage
Copy link
Contributor

A good solution to this would be to print less empty lines;

👍

my ideal solution would be that these messages should not show at all

👎 It is good to know it is still doing something (a simple spinner isn't reassuring enough). Also, you can technically also use this with the verbose reporter (--watch --verbose).

any reason we can't just overwrite the output

👍 for doing that with the mini reporter. Verbose should just be an endless scroll.

We should also include some indicator that another test run has occurred. For small test suites it's already so fast that if you blink you might miss it (and you can't tell one 12 passed line from the previous). I am thinking maybe a timestamp (light gray):

[10:51:03] 12 passed

@sindresorhus
Copy link
Member

Timestamp would indeed be good for both default and verbose mode.

@sindresorhus sindresorhus added enhancement new functionality accepted labels Mar 22, 2016
@novemberborn
Copy link
Member

👍 to overwrite the output, and the timestamp.

@ben-eb thanks for using watch mode!

@jfmengels
Copy link
Contributor

Food for thought: In Mocha's watch mode, we get the time the tests took to run.
image
I like this since you have early feedback about whether your tests are fast or not, since Mocha re-runs all tests every time. In AVA, only the tests for the saved file are re-run, but it might be a nice addition too.
I usually look at the time it took to see whether tests have re-run. Look at previous run time, save file, compare with run time, if it's different, it has run. 👍 on either the timestamp, the run time or both.

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

No branches or pull requests

5 participants