X-Git-Url: https://git.openstreetmap.org/rails.git/blobdiff_plain/3d55db46ed24d563d82cf42f91fd064e40447c4d..1340fca8f17e4e3cce211e6eafed18cde7f57386:/test/unit/user_test.rb diff --git a/test/unit/user_test.rb b/test/unit/user_test.rb index e537a8433..976a543cf 100644 --- a/test/unit/user_test.rb +++ b/test/unit/user_test.rb @@ -18,23 +18,27 @@ class UserTest < ActiveSupport::TestCase end def test_unique_email - new_user = User.new(:email => users(:normal_user).email, + new_user = User.new({ + :email => users(:normal_user).email, :status => "active", :pass_crypt => Digest::MD5.hexdigest('test'), :display_name => "new user", :data_public => 1, - :description => "desc") + :description => "desc" + }, :without_protection => true) assert !new_user.save assert new_user.errors[:email].include?("has already been taken") end def test_unique_display_name - new_user = User.new(:email => "tester@openstreetmap.org", + new_user = User.new({ + :email => "tester@openstreetmap.org", :status => "pending", :pass_crypt => Digest::MD5.hexdigest('test'), :display_name => users(:normal_user).display_name, :data_public => 1, - :description => "desc") + :description => "desc" + }, :without_protection => true) assert !new_user.save assert new_user.errors[:display_name].include?("has already been taken") end @@ -48,13 +52,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