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

Feature request: Quieter CLI reporter #952

Open
mlambley opened this issue Jan 18, 2018 · 3 comments
Open

Feature request: Quieter CLI reporter #952

mlambley opened this issue Jan 18, 2018 · 3 comments
Labels
Epic: CLI reporter Dredd's default reporter

Comments

@mlambley
Copy link

I'm not sure whether this will be its own feature or part of #765 so I'm just logging it here for now.

In #765 there are separate sections for Progress and List of Failures.

What I'd like is to have the option to hide the List of Failures section from the CLI reporter. The reason is that my expected/actual json is quite long, and when dumped into my terminal it destroys all meaningful information there.

What I'd prefer is to be able to use both the html and cli reporters, have the failures dumped into the html reporter (as it is now) but make the cli reporter only show the Progress and not the List of Failures.

I had a few thoughts. One is to create a new reporter with the failure data omitted. I created one locally and can submit a PR if you wish.

Another is to modify the parameters in the cli-reporter.js - such as changing the variable inlineErrors to displayErrors which would be either none, inLine or atEnd. But since this would be a breaking change it might be better to incorporate it as part of #765.

Another is to have a means whereby a dev can create their own custom reporters. This might involve configuring a directory for the system to scan and dynamically load all reporter js files in it. I am yet to do this but could develop and submit a PR for this if you wish. However since you are considering switching to TAP this may be redundant.

@honzajavorek
Copy link
Contributor

Hi, wouldn't the built-in dot reporter combined with html/apiary reporter solve your issue?

@mlambley
Copy link
Author

Dot reporter still displays failed tests anyway, so the issue of my long json cluttering up the terminal isn't abated by that.

Also, I like seeing pass/fail/skip next to each action/url/duration. I still want the CLI reporter, just without the clutter.

@mlambley
Copy link
Author

I've gone ahead and created a PR for the least intrusive option. Please give feedback. Thank you.

@honzajavorek honzajavorek added the Epic: CLI reporter Dredd's default reporter label Jul 16, 2018
@honzajavorek honzajavorek modified the milestone: Improve CLI output Jan 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic: CLI reporter Dredd's default reporter
Projects
None yet
Development

No branches or pull requests

2 participants