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

Update to Minitest 5 #18

Merged
merged 1 commit into from
Apr 21, 2014
Merged

Update to Minitest 5 #18

merged 1 commit into from
Apr 21, 2014

Conversation

strzibny
Copy link
Contributor

Hi,

this is the first attempt to update the test suite to Minitest 5. We are going to upgrade Ruby on Rails in Fedora 21 and RoR 4.1 uses the new Minitest so we upgraded it as well. We would love to see gems to use the new Minitest not to carry patches or sticking with the old one.

What this PR changes?

  • require 'test/unit' to require 'minitest/autorun'
  • Test::Unit::TestCase to Minitest::Test
  • assert_not_nil to refute_nil
  • adds requirement of Minitest 5 to .gemspec
  • adds assert_nothing_raised, assert_not_same, and assert_block to other custom asserts extending Kernel in test/test_utils.rb. This is kind of backport from older Minitest and we can probably change the tests not to use those at all in future.

Please consider.

@philr philr merged commit 25c0cf6 into tzinfo:master Apr 21, 2014
@philr
Copy link
Member

philr commented Apr 21, 2014

Thanks for submitting this pull request.

I have merged the change.

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.

None yet

2 participants