Dois-je stub le modèle dans Factory girl ou dans le fichier de spec pendant le test?

Presque tous les spec de fichier que j'ai trouvé je poste des trucs comme:

  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

Je veux dire, le modèle que je reçois de l'Usine.la construction est tout à fait valable. Mais si je n'ai pas de stub ce genre de choses, il enregistre des choses dans le système de fichiers, et valide les trucs que je ne suis pas test...

Ce que je veux dire, je pense qu'il serait plus propre de faire quelque chose comme ceci:

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

Si cogner à l'intérieur de l'Usine est encore possible.

Quelle est la bonne façon pour talon le modèle?

InformationsquelleAutor Zequez | 2012-01-30

21

Dans les versions récentes de factory_girl vous avez un after_build de rappel, donc, je crois que vous pourriez définir votre usine comme ceci:
```
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
```
Mise à JOUR

Après factory_girl 3.3.0, la syntaxe a changé à la suite:
```
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
```
- Mais dois-je faire? Ou devrais-je stub dans le fichier spec?
- Je pense que si vous êtes de vous répéter de nombreuses fois, en disposant d'un stub de sens que dans ce cas, assurez-vous que :cimg_for_testing_tags a un parent de l'usine qui n'est pas supprimée pour les situations où vous voulez tester le comportement réel.
- C'est une technique impressionnante. Je me suis trouvé aux prises avec une solution pour ce genre de situations. Merci @fkreusch !
- Cela ne fonctionne plus. La dernière rspec cela va entraîner l'échec "Using 'stub' from rspec-mocks' old ':should' syntax without explicitly enabling the syntax is deprecated." Malheureusement, si vous passez à l'aide de la nouvelle rspec allow() syntaxe vous rencontrez un autre problème, car FactoryGirl ne comprend pas de RSpec méthodes: NoMethodError: undefined method 'allow' for #<FactoryGirl::SyntaxRunner:0x007f95469cc0d8>. Quelqu'un sait-il une solution à ce qui fonctionne avec les dernières versions?
- Vous pouvez faire def cimg.validate_img; true; end et effectivement il écraser sans rspec.
- Il y a un moyen d'utiliser la syntaxe. Voir ma réponse.
InformationsquelleAutor fkreusch
30

@fkreusch réponse fonctionne très bien jusqu'à ce que vous utilisez le nouveau RSpec expect() syntaxe (3.0+)

La mise en rails_helper.rb fonctionne pour moi:
```
FactoryBot::SyntaxRunner.class_eval do
  include RSpec::Mocks::ExampleMethods
end
```
Dans le cas des OP exemple, vous pouvez désormais le faire:
```
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
```
De crédit: github.com/printercu voir: https://github.com/thoughtbot/factory_bot/issues/703#issuecomment-83960003
- Faire ce qui est décrit ici maintenant émet le message d'erreur suivant (FactoryGirl v4.7.0): The use of doubles or partial doubles from rspec-mocks outside of the per-test lifecycle is not supported.
- Je rencontre cette erreur parce que mon rails_helper.rb était en cours d'exécution FactoryGirl.lint dans un before(:suite) bloc, et les talons ne peuvent pas être mises en place. J'ai résolu le problème en les enveloppant de la charpie appel dans RSpec::Mocks.with_temporary_scope { ... }.
- Notez que FactoryGirl est maintenant FactoryBot, de sorte que ces deux noms qui devrait changer dans votre exemple.
InformationsquelleAutor Ho-Sheng Hsiao
3

Vous pouvez également envisager d'utiliser FactoryGirl#build_stubbed.

InformationsquelleAutor Su Zhang
3

Une usine devrait produire "monde réel" des objets, par conséquent, c'est une mauvaise pratique (et erreurs) pour le changement de comportement (c'est à dire stub) dans une usine.

Vous pouvez faire
```
let(:user) instance_double(User, FactoryGirl.attributes_for(:user))

before do
  allow(user).to receive(:something).and_return('something')
end
```
et si votre before clause devient trop grande, vous pouvez extraire vers un autre méthode ou créer une maquette de la classe enfant qui l'emporte sur les méthodes que vous souhaitez stub.

InformationsquelleAutor thisismydesign

Vous devez vous connecter pour publier un commentaire.