search

mfridman / tparse

CLI tool for summarizing go test output. Pipe friendly. CI/CD friendly.
MIT License
948 stars 22 forks source link

Add option to not show summary #90

Open Itarix opened 1 year ago

Itarix commented 1 year ago

Hello,

Thank you to create and maintain this tool!

I think it could be usefull to add an option to hide the summary. Or hide a part of them.

By exemple: show only the summary where the tests are failed.

What do you think?

Thanks !

mfridman commented 1 year ago

Do you mean adding a flag like -fail-only and only display the non-passing tests in both the table and package tables?

Itarix commented 1 year ago

Yes exactly.

In case having big codebase, it's hard to see the failed tests

mfridman commented 1 year ago

Not a bad idea!

mfridman commented 1 year ago

Before implementing this, I'd like to modify the behaviour of the Tests Table to include FAIL tests. Currently, they are omitted from the Tests Table and always displayed in the output preceding the Package Table.

Once those are displayed in that table, we can think about adding a -fail-only flag to limit the output to include only failed tests.

I'm curious @Itarix, if there are NO failed tests, what should the output be?

CleanShot 2023-05-28 at 23 25 36@2x

Itarix commented 1 year ago

If no tests failed, the output can be a simple sentence as : No test failed or something like that.

What do you think about that?