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

New feature: use multiple reporters #169

Closed
wichert opened this issue May 13, 2013 · 10 comments
Closed

New feature: use multiple reporters #169

wichert opened this issue May 13, 2013 · 10 comments

Comments

@wichert
Copy link
Contributor

wichert commented May 13, 2013

I would like to be able to use both the standard reporter and the junit reporter at the same time. I think my use cases is a common one: I want human readable output on stdout so the emails jenkins sends me are useful, but I want the junit XML output for tracking purposes.

Looking at other test runners I use they handle this by always reporting to stdout while an XML output option creates one or more XML files in junit format in addition to the normal stdout output.

@philsquared
Copy link
Collaborator

Oddly enough the very first versions of Catch did support multiple reporters.
It was causing a lot of out-of-proportion (at the time) complexity and after some thought (and discussion with peers) decided that it was not something that was really useful, so I took it out in the name of simplicity.

At this point I think it could be supported with fairly low complexity, so I'm going to look at it again.
It might have to wait a bit, though, as I have quite a lot on my queue at the moment.

@michaelbprice
Copy link

@philsquared, we are working on a tool that allows people to more easily migrate between test frameworks without having to rewrite a whole bunch of existing test code. Having the ability to have multiple reporters registered during a single call to Session::run would be incredibly valuable.

It looks like this issue is quite old at this point. If there were a pull request that implemented this feature (we almost hacked one up that worked around the framework limitation), would you consider it for inclusion?

@philsquared
Copy link
Collaborator

Catch does actually now support multiple reporters

@philsquared philsquared added the Resolved - pending review Issue waiting for feedback from the original author label Jan 9, 2017
@horenmar horenmar removed the Resolved - pending review Issue waiting for feedback from the original author label Feb 6, 2017
@Reonaydo
Copy link

Reonaydo commented May 4, 2017

Catch does actually now support multiple reporters

How it can be used? How can I set write "junit" into file and "console" to stdout ?

@philsquared
Copy link
Collaborator

I was probably a bit premature in announcing that.
Technically Catch does support multiple reporters - but not in any useful way yet. Or rather it supports them for the purpose of Listeners. As a side effect you can specify multiple reporters, but there is currently no way to specify different locations for them to write to - so they all go either to the console or to the same file. Like I said, not useful.
It shouldn't be a huge job to address that but it's just not been high enough priority yet.
Hopefully will get to it at some point.

@LesserBabka
Copy link

I would also appreciate this functionality (to be useful).

@fbradyirl
Copy link

It would be great to see this implemented.

@cjw296
Copy link

cjw296 commented Feb 25, 2021

Would still be great to see this, or are there other solutions (nested reported or some such?)

@Viatorus
Copy link

@philsquared Can we please reopen this issues?

@horenmar
Copy link
Member

@Viatorus #1712

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

No branches or pull requests

9 participants