FactoryGirl 的 RSpec 唯一性电子邮件测试失败

Question

编辑

使用问题的答案，我将测试更改为正确测试并通过的以下测试..

describe "when email is already taken" do
  let(:user_with_same_email) { @user.dup }
  before do
    user_with_same_email.email.upcase!
    user_with_same_email.save
  end

  it { user_with_same_email.should_not be_valid }
end

注意：不使用let(:user_with_same_email) { @user.dup } 会使测试失败，因为如果变量user_with_same_email 只是在before 块中重复，就像此问题的所选答案一样。

---

我有一个User 模型和一个user_spec.rb 测试文件，它针对User 模型属性进行了各种验证。

之前我在user_spec.rb 文件的顶部写了以下内容来测试User 模型：

describe User do

  before do
  @user = User.new(name: "Example User", email: "user@example.com",
                 password: "foobar88", password_confirmation: "foobar88")
end
...

我想将此模型创建移至FactoryGirl，因此我创建了一个factories.rb 文件：

FactoryGirl.define do
  factory :user do
    name "foo"
    email { "#{name}@example.com" }
    password "foobar99"
    password_confirmation "foobar99"
  end
end

然后我更改了我的user_spec.rb：

describe User do

  before do
    @user = FactoryGirl.create(:user)
  end
...

现在每个测试都像以前一样通过，除了一个：

describe "when email is already taken" do
  before do
    user_with_same_email = @user.dup
    user_with_same_email.email = @user.email.upcase
    user_with_same_email.save
  end

  it { should_not be_valid }
end

现在除非 `FactoryGirl 跳过我的电子邮件唯一性验证，否则我无法弄清楚这里出了什么问题。

我的User模型验证码：

class User < ActiveRecord::Base

  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i unless const_defined?(:VALID_EMAIL_REGEX)

  has_secure_password
  attr_accessible :name, :email, :password, :password_confirmation

  has_many :programs

  before_save { self.email.downcase! }

  validates :name, presence: true, length: { maximum: 50 }
  validates :email, presence: true, format: { with: VALID_EMAIL_REGEX },
                    uniqueness: { case_sensitive: false }

Answer 1

问题是当你说它 { should_not be_valid } 时，RSpec 会检查主题。在这种情况下，主题是 User.new（您在顶部有“描述用户”，因此除非您指定其他内容，否则这是默认设置）。

您想改为检查 user_with_same_email 的有效性。

编辑： 试试这个，我认为它可能会起作用：

describe "when email is already taken" do
  before do
    @user_with_same_email = @user.dup
    @user_with_same_email.email = @user.email.upcase
    @user_with_same_email.save
  end

  it { @user_with_same_email.should_not be_valid }
end

Answer 2

看起来您可能正在使用（或引用）Michael Hartl 的Rails Tutorial。这是我的代码对你正在做的事情的样子，所以我希望它可以使用：

spec/models/user_spec.rb

describe User do

  let(:user) { valid_user }
  subject { user }

  # ...

  context "when email address is already taken" do
    before { save_user(user) }
    it { should_not be_valid }
  end

  # ...
end

spec/support/utilities.rb（创建特定用户）

def valid_user
  User.new(name:     "Example User", 
           email:    "user@example.com",
           password: "foobar", 
           password_confirmation: "foobar")
end

# ...

def save_user(user)
  user_with_same_email = user.dup
  user_with_same_email.email.upcase!
  user_with_same_email.save
end

供参考：spec/factories.rb（仅创建任何旧的随机用户）

FactoryGirl.define do
  factory :user do
    sequence(:name)  { |n| "Person #{n}" }
    sequence(:email) { |n| "person_#{n}@example.com" }
    password "foobar"
    password_confirmation "foobar"

    # ...
  end
  # ...
end

更新：在this StackOverflow answer 找到了您正在寻找的答案，概述了同样的问题。我也用我的代码对其进行了测试，它对我有用。

更新 2：也更改了我的代码，在我想要用户但不希望将其保存到数据库中时使用 FactoryGirl.build。 This StackOverflow answer 帮助我理解了。

spec/models/user_spec.rb

describe User do

  let(:user) { FactoryGirl.create(:user) }
  subject { user }

  # ...

  context "when email address is already taken" do
    let(:user_with_same_email) do
      FactoryGirl.build(:user, email: user.email)
    end

    subject { user_with_same_email }

    before do
      user_with_same_email.email.upcase!
      user_with_same_email.save
    end

    it { should_not be_valid }
  end
  # ...
end

感谢您提出这个问题。让我深思熟虑，并在我自己的代码中进行一些重构。