Skip to content
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

Default reporter #18

Closed
sindresorhus opened this issue May 3, 2015 · 13 comments · Fixed by #359
Closed

Default reporter #18

sindresorhus opened this issue May 3, 2015 · 13 comments · Fixed by #359
Assignees
Labels
enhancement new functionality

Comments

@sindresorhus
Copy link
Member

I think the default reporter should be super minimal one. It should only log out detailed test info if a test fails, otherwise just 1 test passed.

It could have a live counter, so while tests are running the line is dynamically updated with 3/12 tests passed, and when done rewritten to be 12 tests passed. That way it can be live, but also minimal.

Can use log-update.

@sindresorhus sindresorhus self-assigned this May 3, 2015
@sindresorhus sindresorhus added the enhancement new functionality label May 3, 2015
@SamVerschueren
Copy link
Contributor

Agree on this one, but should be implemented after #27 I guess. If people want the full blown output, they can then enable another reporter.

@vadimdemedes
Copy link
Contributor

Yeah, reporters are an important feature to implement. But I think we're not "there" yet, there are some issues that have a much higher priority.

But once those are resolved, I am 100% on board with implementing custom reporters.

@vadimdemedes
Copy link
Contributor

@sindresorhus do you still want to work on this? I'd like to assign this to myself :)

@sindresorhus
Copy link
Member Author

Go ahead ;)

@alubbe
Copy link

alubbe commented Nov 25, 2015

Just some quick feedback from an early user that it would be awesome to have custom reporters :)

@sindresorhus
Copy link
Member Author

@alubbe That will come when we add TAP support #27. You can then use any TAP-reporter.

@alubbe
Copy link

alubbe commented Nov 26, 2015

Awesome, I just hope that the concurrency doesn't end up posing a problem for TAP.

@jamestalmage
Copy link
Contributor

We likely will have to buffer and output one whole test at a time.

@vadimdemedes
Copy link
Contributor

What do you think about this?

demo

@sindresorhus
Copy link
Member Author

@vdemedes ❤️ Exactly what I was thinking.

@jamestalmage
Copy link
Contributor

👍

2 similar comments
@SamVerschueren
Copy link
Contributor

👍

@ariporad
Copy link
Contributor

👍

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

Successfully merging a pull request may close this issue.

6 participants