Replace homemade assert_not matcher in favor of refute.

This commit is contained in:
Lucas Mazza
2016-05-03 13:57:10 -03:00
parent 031351224a
commit 536279b05b
21 changed files with 144 additions and 148 deletions

View File

@@ -69,8 +69,8 @@ class DeviseTest < ActiveSupport::TestCase
test 'add new module using the helper method' do
assert_nothing_raised(Exception) { Devise.add_module(:coconut) }
assert_equal 1, Devise::ALL.select { |v| v == :coconut }.size
assert_not Devise::STRATEGIES.include?(:coconut)
assert_not defined?(Devise::Models::Coconut)
refute Devise::STRATEGIES.include?(:coconut)
refute defined?(Devise::Models::Coconut)
Devise::ALL.delete(:coconut)
assert_nothing_raised(Exception) { Devise.add_module(:banana, strategy: :fruits) }
@@ -86,11 +86,11 @@ class DeviseTest < ActiveSupport::TestCase
test 'should complain when comparing empty or different sized passes' do
[nil, ""].each do |empty|
assert_not Devise.secure_compare(empty, "something")
assert_not Devise.secure_compare("something", empty)
assert_not Devise.secure_compare(empty, empty)
refute Devise.secure_compare(empty, "something")
refute Devise.secure_compare("something", empty)
refute Devise.secure_compare(empty, empty)
end
assert_not Devise.secure_compare("size_1", "size_four")
refute Devise.secure_compare("size_1", "size_four")
end
test 'Devise.email_regexp should match valid email addresses' do