¿Debo colgar el modelo en Factory girl o en el archivo de especificaciones durante la prueba?

Question

Casi cada archivo de especificaciones llego al otro lado me acaban de escribir cosas como:

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 

Es decir, el modelo que recibo de Factory.build es completamente válido. Pero si yo no golpeo eso se ahorra cosas en el sistema de archivos, y valida cosas que no estoy probando ...

Lo que quiero decir, yo creo que sería más limpio que hacer algo como esto:

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

Si es posible anular dentro de la fábrica.

¿Cuál es la forma correcta de colgar el modelo?

Answer 1

En versiones recientes de factory_girl tiene una devolución de llamada after_build, por lo que cree que podría definir su fábrica de la siguiente manera:

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 

ACTUALIZACIÓN

Después factory_girl 3.3.0, la sintaxis ha cambiado para seguir:

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 2

También podría considerar el uso de FactoryGirl#build_stubbed.

Answer 3

@ respuesta de fkreusch funciona muy bien hasta que utilice la nueva sintaxis RSpec expect() (3.0+)

Su puesta en rails_helper.rb funciona para mí:

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

En el ejemplo de la OP, ahora se puede hacer:

FactoryGirl.define do 
    factory :cimg_for_testing_tags do 

    ... # Factory attributes 

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

crédito: github.com/printercu, consulte: https://github.com/thoughtbot/factory_girl/issues/703#issuecomment-83960003

Answer 4

Una fábrica debe producir objetos del "mundo real", por lo tanto, es una mala práctica (y propensa a errores) cambiar el comportamiento (es decir, stub) en una fábrica.

Usted puede hacer

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

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

y si su cláusula de before se hace demasiado grande es posible que desee extraerlo a un método separado o crear una clase hija maqueta que anula métodos que desee trozo.