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

allow specifying separate outputs for reporters #669

Draft
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

srnwk
Copy link

@srnwk srnwk commented Jul 7, 2022

Description

Extends the command-line arguments to allow specifying a filter of reporters
to which the output filename applies: --out=junit[junit-report.xml],xml[report.xml],console[log.txt]
The old syntax --out=filename still works, being treated like --out=*[filename].
This allows sending for example JUnit output to a file, and still having the console reporter output to the console: -r=junit,console -o=junit[report.xml] (since no output file has a matcher that matches the console reporter, it uses its default, stdout).

(Should there be an explicit way to specify stdout as well? Possibly using - as the filename, like many *nix tools do?)

TODO:

  • update documentation
  • API for setting these from main
  • tests, coverage

GitHub Issues

See #627

@srnwk srnwk force-pushed the separate-reporter-outputs branch from 636404d to 33b7747 Compare July 7, 2022 13:30
@codecov
Copy link

codecov bot commented Jul 7, 2022

Codecov Report

Merging #669 (b0b48ca) into dev (b7c21ec) will decrease coverage by 0.03%.
The diff coverage is 81.48%.

@@            Coverage Diff             @@
##              dev     #669      +/-   ##
==========================================
- Coverage   92.19%   92.16%   -0.04%     
==========================================
  Files           2        2              
  Lines        2127     2156      +29     
==========================================
+ Hits         1961     1987      +26     
- Misses        166      169       +3     
Impacted Files Coverage Δ
doctest/parts/doctest.cpp 91.67% <80.76%> (-0.03%) ⬇️
doctest/parts/doctest_fwd.h 97.70% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7c21ec...b0b48ca. Read the comment docs.

@srnwk srnwk force-pushed the separate-reporter-outputs branch from 38209c6 to b0b48ca Compare July 7, 2022 15:46
@nickdademo
Copy link

Very nice! Looking forward to this one ;)

@beyse
Copy link

beyse commented Oct 19, 2022

Also looking forward to it

@vitaliy-ostapchuk93
Copy link

any updates on this? would be really great to use 💯

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

Successfully merging this pull request may close these issues.

None yet

4 participants