kaocha icon indicating copy to clipboard operation
kaocha copied to clipboard

Draft: Add summary reporter

Open NoahTheDuke opened this issue 2 years ago • 4 comments

Hey friends,

I'm interested in a reporter that does not print anything except for the summary at the end. I would ideally also like a leading "Starting test run" so I know it's working, but there doesn't seem to be support for that at the moment, so I don't mind excluding that.

This is pretty simple and I don't want to put a ton of work into it if there's no interest so here's the minimum working example. If there is tho, I'll write up all of the docs and changelog necessary. I couldn't find where the reporter symbols (the vectors) are tested, which is why I didn't write any tests for this PR.

Thanks so much!

NoahTheDuke avatar Aug 09 '22 17:08 NoahTheDuke

You're looking for kaocha.report/result

plexus avatar Aug 09 '22 17:08 plexus

Codecov Report

Merging #304 (3d7b362) into main (f97d9fd) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #304      +/-   ##
==========================================
+ Coverage   75.53%   75.55%   +0.01%     
==========================================
  Files          51       51              
  Lines        2714     2716       +2     
  Branches      252      252              
==========================================
+ Hits         2050     2052       +2     
  Misses        506      506              
  Partials      158      158              
Flag Coverage Δ
integration 57.06% <100.00%> (+0.03%) :arrow_up:
unit 69.55% <100.00%> (+0.02%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/kaocha/report.clj 84.86% <100.00%> (+0.12%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us.

codecov[bot] avatar Aug 09 '22 17:08 codecov[bot]

You're looking for kaocha.report/result

I don't see it referenced anywhere in the docs. Would you accept a PR mentioning it?

NoahTheDuke avatar Aug 10 '22 01:08 NoahTheDuke

Yeah sure, that would be great. It's mostly used as part of other reporters, I guess I never imagined people using it by itself but it makes sense.

plexus avatar Aug 10 '22 11:08 plexus

@NoahTheDuke I think I'll close this PR so you can open a separate one with the do change. Or are you planning to repurpose this PR?

alysbrooks avatar Oct 21 '22 17:10 alysbrooks

Sorry, been busy at work. I'll open a new PR with the documentation.

NoahTheDuke avatar Oct 21 '22 18:10 NoahTheDuke