2017-01-12 12 views
0

Я следую учебник и до сих пор мои тесты были успешными, пока я не написал последнюю проверку на уникальность адресов электронной почтыОшибка проверки дубликатов пользователей?

тест/модель/user_test.rb

require 'test_helper' 

class UserTest < ActiveSupport::TestCase 

. 
. 
. 


    test "email addresses should be unique" do 
    duplicate_user = @user.dup 
    duplicate_user.email = @user.email.upcase 
    @user.save 
    assert_not duplicate_user.valid? 
    end 

модели сами по себе включает в себя следующие проверки.

приложения/модель/user.rb

class User < ApplicationRecord 
    before_save { self.email = email.downcase! } 
    validates :name, presence: true, length: { maximum: 50 } 
    VALID_EMAIL_REGEX = /\A[\w+\-.][email protected][a-z\d\-]+(\.[a-z\d\-]+)*\.[a-z]+\z/i 
    validates :email, presence: true, length: { maximum: 255 }, 
        format: { with: VALID_EMAIL_REGEX }, 
        uniqueness: { case_sensitive: false } 
        has_secure_password 
    validates :password, presence: true, length: { minimum: 6 } 
end 

с последним тестом я получаю ошибку булева

" test_email_addresses_should_be_unique#UserTest (0.87s) 
     Expected true to be nil or false" 

я установить уникальность валидацию опускает чувствительность к регистру. Следовательно, не должен ли дублирующий пользователь быть недействительным?

полный тест: теста/модель/user_test.rb

require 'test_helper' 

class UserTest < ActiveSupport::TestCase 

    def setup 
    @user = User.create(name: "Example User", email: "[email protected]", 
        password: "foobar", password_confirmation: "foobar") 
    end 

    test "should be valid" do 
    assert @user.valid? 
    end 

    test "name should not be too long" do 
    @user.name = "a" * 51 
    assert_not @user.valid? 
    end 

    test "email should not be too long" do 
    @user.email = "a" * 244 + "@example.com" 
    assert_not @user.valid? 
    end 

    test "email validation should accept valid addresses" do 
    valid_addresses = %w[[email protected] [email protected] [email protected] 
         [email protected] [email protected]] 
    valid_addresses.each do |valid_address| 
     @user.email = valid_address 
     assert @user.valid?, "#{valid_address.inspect} should be valid" 
    end 
    end 
    test "email validation should reject invalid addresses" do 
    invalid_addresses = %w[[email protected],com user_at_foo.org [email protected] 
          [email protected]_baz.com [email protected]+baz.com] 
    invalid_addresses.each do |invalid_address| 
     @user.email = invalid_address 
     assert_not @user.valid?, "#{invalid_address.inspect} should be invalid" 
    end 
    end 

    test "email addresses should be unique" do 
    duplicate_user = @user.dup 
    duplicate_user.email = @user.email.upcase 
    @user.save! 
    assert_not duplicate_user.valid? 
    end 
+0

Что такое '@ user'? Является ли сохранение пользователя в базе данных? Вы дважды проверяли, существует ли пользователь? – spickermann

+0

Только что обновлено. Да, пользователь сохраняется в базе данных. – MisterCal

+0

Проверка уникальности учитывает только тех пользователей, которые уже были сохранены в базе данных. Использование '@user = User.create (...)' в вашей 'setup' должно сделать ваш тест зеленым. Дайте мне знать, если это помогло :) – Raffael

ответ

2

Я попробовал ваш код на моей машине, и это, кажется, проблема заключается в before_save обратного вызова.

Вариант bang downcase! изменяет сама строку, и если были произведены какие-либо изменения (то есть любые символы в нижней части) возвращает результат, в противном случае возвращается nil.

В то время как downcase возвращает копию строки со всеми символами в нижнем регистре. Таким образом, вы можете использовать один из следующих подходов: before_save { self.email = email.downcase } или before_save { email.downcase! }.

+0

Работал для меня! благодаря – MisterCal