Skip to content

Conversation

@agrimm
Copy link
Contributor

@agrimm agrimm commented Aug 22, 2013

I wasn't able to run unit tests on Ruby 2.0. I came across http://stackoverflow.com/questions/17461245/what-does-this-minitestunittestcase-warning-mean , suggesting that the error was due to using minitest 5.0 when the code was written for a previous version.

Without this change, no tests at all get run on Ruby 2.0.

Edit: actually, looks like the problem does not involve Ruby 2.0. It looks like it's just which version of minitest is being used.

@cfis
Copy link
Member

cfis commented Aug 23, 2013

Thanks for the report. Would it be possible to detect which version of minitest is installed and do the right thing? I'd rather not lock users into an older version of minitest if we don't have to.

@agrimm
Copy link
Contributor Author

agrimm commented Aug 25, 2013

It's only a development dependency, not a runtime dependency. So it'd only affect developers, not users.

skaes pushed a commit that referenced this pull request Nov 26, 2013
Specify version of minitest. Tests do not run on 2.0 otherwise.
@skaes skaes merged commit 2a19dc6 into ruby-prof:master Nov 26, 2013
@skaes
Copy link
Member

skaes commented Nov 26, 2013

Indeed. We currently require minitest < 5.0. Thx for the help.

@agrimm agrimm deleted the get_tests_running branch November 26, 2013 08:25
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

Successfully merging this pull request may close these issues.

3 participants