search

ci-reporter / ci_reporter

CI::Reporter is an add-on to popular test frameworks that allows you to generate XML reports
MIT License
343 stars 110 forks source link

ci_reporter will kill existing test-unit options on current rake versions #75

Closed rb2k closed 10 years ago

rb2k commented 12 years ago

This line: https://github.com/nicksieger/ci_reporter/blob/master/lib/ci/reporter/rake/test_unit.rb#L12

Will set the TESTOPTS environment variable.

In rake 0.8.7, the only valid way to pass options was the env var TESTOPTS https://github.com/jimweirich/rake/blob/rake-0.8.7/lib/rake/testtask.rb#L123

In rake 0.9.2 you can go crazy with variations of this: https://github.com/jimweirich/rake/blob/rake-0.9.2/lib/rake/testtask.rb#L105

You can also, in both versions of rake, set the options attribute with a string, but this will be ignored if the ENV VAR is set.

I think this means that by including CI reporter options set programmatically or with a different name will be ignored.

shepmaster commented 10 years ago

We are in the process of cleaning out old issues. Specifically, the next version of CI::Reporter is only going to support Test::Unit bundled with the latest version of Ruby 1.9.3, 2.0, and 2.1. If this issue still applies to these versions, please let us know, otherwise we're going to close this issue to focus on current problems. Thanks for your report!

shepmaster commented 10 years ago

Closing due to inactivity.