Support for Test::Unit 2.1 #18

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
Contributor

amfranz commented Nov 22, 2010

CI_Reporter currently does not work with Test::Unit 2.1 because the code path for creating the mediator, which CI_Reporter hooks into, changed between Test::Unit 2.0 and 2.1.

The attached pull request contains a change to test_unit_loader.rb that allows CI_Reporter to hook into Test::Unit 2.1, without breaking support for older versions of Test::Unit.

Adjusting the logic to install the CI::Reporter::TestUnit mediator so…
… that it will work for newer ruby-test versions (>= 2.1) as well.

Hey, it works again, thanks a lot! :-) I had yet to figure out that it was a version issue.

yairgo commented Feb 14, 2011

will you please pull this in

Owner

nicksieger commented Apr 19, 2011

Is this all that's needed for T::U 2.1? I haven't been using it myself. I am happy to push a new gem with just this change if it's really so simple :)

@nicksieger nicksieger closed this Apr 19, 2011

yairgo commented Apr 19, 2011

I believe so. If you push up a gem I'll test it out

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment