rspec_junit_formatter icon indicating copy to clipboard operation
rspec_junit_formatter copied to clipboard

Show detail for failing examples using aggregate_failures

Open jasoncodes opened this issue 5 years ago • 9 comments

Fixes #56

Example

Before

After

Note

There is a minor change to how backtraces are formatted. It’s now the same as what rspec shows when ran from directly:

Before

After

jasoncodes avatar Oct 18 '19 04:10 jasoncodes

I am waiting for review and merge from owner

xuantuan58 avatar Oct 24 '19 09:10 xuantuan58

Sorry that this hasn't had attention in a while, @jasoncodes.

I really like this idea, but I'm very concerned that the parsing code seems to be a bit brittle and may break if future rspecs change output formats slightly. I'm not sure what the best answer here, which is why I haven't really jumped into it. 🤔

sj26 avatar Apr 02 '20 06:04 sj26

What if I made the guards less strict so instead of raising on an unexpected format, they instead silently continue? This should at worst output a bit of duplication instead of erroring.

I’m not sure how to best add coverage for that though. Conditionally stub out fully_formatted_lines so it does something different than expected? Not sure if it’d be worth the effort.

jasoncodes avatar Apr 05 '20 06:04 jasoncodes

@sj26 Sorry to bother, but do you have any further opinion on this?

taylorthurlow avatar Oct 20 '20 16:10 taylorthurlow

@sj26 Any chance of an update?

taylorthurlow avatar May 11 '21 16:05 taylorthurlow

Right now we're running on a fork based on this PR. I too would appreciate any opinions on a way forward. Thanks!

dschweisguth avatar Nov 02 '22 02:11 dschweisguth

Any updates on this guys, would really love to see it going in ?

dimitarizpitaime avatar Feb 20 '23 09:02 dimitarizpitaime

This would be a great addition to merge and then cut a release...

krasnoukhov avatar Jul 26 '23 12:07 krasnoukhov

@sj26 any chance we can get this one on a new release? :)

Drowze avatar Mar 27 '24 11:03 Drowze