测试时我应该在 Factory Girl 中还是在规范文件中存根模型吗？

Question

几乎我遇到的每个规范文件最终都会写这样的内容：

  before :each do
    @cimg = Factory.build :cimg_valid
    @cimg.stub(:validate_img).and_return true
    @cimg.stub(:validate_img_url).and_return true
    @cimg.stub(:save_images).and_return true
    @cimg.stub(:process_image).and_return true
    @cimg.stub(:img).and_return true
  end

我的意思是，我从 Factory.build 获得的模型是完全有效的。但是，如果我不存根这些东西，它会将内容保存在文件系统中，并验证我没有测试的东西...

我的意思是，我认为这样做会更干净：

  before :each do
    @cimg = Factory.build :cimg_for_testing_tags
  end

如果工厂内的存根是均匀的可能的。

存根模型的正确方法是什么？

Answer 1

@fkreusch 的答案非常有效，直到您使用新的 RSpec expect() 语法 (3.0+)

将其放入 rails_helper.rb 对我有用：

FactoryBot::SyntaxRunner.class_eval do
  include RSpec::Mocks::ExampleMethods
end

在OP的示例中，您可以现在做：

FactoryBot.define do
  factory :cimg_for_testing_tags do

    ... # Factory attributes

    after(:build) do |cimg|
      allow(cimg).to receive(:validate_img) { true }
    end
  end
end

信用：github.com/printercu，请参阅：https://github.com/thoughtbot/factory_bot/issues/703#issuecomment-83960003< /a>

Answer 2

在最近版本的factory_girl中，你有一个after_build回调，所以我相信你可以像这样定义你的工厂：

FactoryGirl.define do
  factory :cimg_for_testing_tags do

    ... # Factory attributes

    after_build do |cimg|
      cimg.stub(:validate_img).and_return true
    end
  end
end

UPDATE

在factory_girl 3.3.0之后，语法已更改为以下：

FactoryGirl.define do
  factory :cimg_for_testing_tags do

    ... # Factory attributes

    after(:build) do |cimg|
      cimg.stub(:validate_img).and_return true
    end
  end
end

Answer 3

工厂应该生产“现实世界”的对象，因此改变工厂中的行为（即存根）是一种不好的做法（并且容易出错）。

您可以这样做

let(:user) instance_double(User, FactoryGirl.attributes_for(:user))

before do
  allow(user).to receive(:something).and_return('something')
end

，如果您的 before 子句变得太大，您可能需要将其提取到单独的方法或创建一个模拟子类来覆盖您想要存根的方法。

Answer 4

您还可以考虑使用 FactoryGirl#build_stubbed。