X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/001ff5764bcf15aaa179dc5c7844038676536622..e756dbae7f3715561de5688e6503b60fdfe0327d:/test/unit/user_test.rb diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb index 7d5f8dced..843c31905 100644 --- a/test/unit/user_test.rb +++ b/test/unit/user_test.rb @@ -8,13 +8,13 @@ class UserTest < ActiveSupport::TestCase def test_invalid_with_empty_attributes user = User.new assert !user.valid? - assert user.errors.invalid?(:email) - assert user.errors.invalid?(:pass_crypt) - assert user.errors.invalid?(:display_name) - assert user.errors.invalid?(:email) - assert !user.errors.invalid?(:home_lat) - assert !user.errors.invalid?(:home_lon) - assert !user.errors.invalid?(:home_zoom) + assert user.errors[:email].any? + assert user.errors[:pass_crypt].any? + assert user.errors[:display_name].any? + assert user.errors[:email].any? + assert !user.errors[:home_lat].any? + assert !user.errors[:home_lon].any? + assert !user.errors[:home_zoom].any? end def test_unique_email @@ -25,7 +25,7 @@ class UserTest < ActiveSupport::TestCase :data_public => 1, :description => "desc") assert !new_user.save - assert_equal "has already been taken", new_user.errors.on(:email) + assert new_user.errors[:email].include?("has already been taken") end def test_unique_display_name @@ -36,7 +36,7 @@ class UserTest < ActiveSupport::TestCase :data_public => 1, :description => "desc") assert !new_user.save - assert_equal "has already been taken", new_user.errors.on(:display_name) + assert new_user.errors[:display_name].include?("has already been taken") end def test_email_valid @@ -48,13 +48,13 @@ class UserTest < ActiveSupport::TestCase ok.each do |name| user = users(:normal_user) user.email = name - assert user.valid?, user.errors.full_messages + assert user.valid?(:save), user.errors.full_messages end bad.each do |name| user = users(:normal_user) user.email = name - assert !user.valid?, "#{name} is valid when it shouldn't be" + assert user.invalid?(:save), "#{name} is valid when it shouldn't be" end end @@ -93,7 +93,7 @@ class UserTest < ActiveSupport::TestCase user = users(:normal_user) user.display_name = display_name assert !user.valid?, "#{display_name} is valid when it shouldn't be" - assert_equal "is invalid", user.errors.on(:display_name) + assert user.errors[:display_name].include?("is invalid") end end