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.

Sign up for GitHub

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

Jump to bottom

The output of --list-* options should obey -o flag #2061

Closed
horenmar opened this issue Oct 20, 2020 · 0 comments
Closed

The output of --list-* options should obey -o flag #2061

horenmar opened this issue Oct 20, 2020 · 0 comments
Labels
Feature Request
Milestone
3.0

Comments

@horenmar
Copy link
Member

Description
Currently, the output of --list-* options writes to stdout through Catch::cout(). It should obey the -o flag instead.

Additional context
Because the default implementation of --list* flags is done in the IStreamingReporter interface, we need to extend it to contain the stream it should write to, e.g. by giving it a ReporterConfig member and initializing it properly.
@horenmar horenmar added this to the 3.0 milestone Nov 2, 2020
horenmar added a commit that referenced this issue Jan 16, 2021
@horenmar
Hold reporter's IConfig instance in the interface
74f2f4b 
Previously, every base derived from the IStreamingReporter had
its own `IConfig const* m_config` member, so this just centralizes
the handling thereof.

Part of #2061
@horenmar horenmar mentioned this issue Jan 26, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Feature Request
Projects
None yet
Milestone
3.0
Development

No branches or pull requests
1 participant
@horenmar