15 tester la creation dutilisateurs #9
Conversation
| @@ -0,0 +1,51 @@ | |||
| require 'rails_helper' | |||
| require 'dry/transaction' | |||
There was a problem hiding this comment.
Met ce genre de require dans un initializer, que ça s'exécute une fois au lancement de l'app quelque soit l'environnement.
There was a problem hiding this comment.
J'ai require 'rails_helper' directement dans .rspec du coup. Ça résout le problème de l'import dans chaque fichier de test mais j'ai trouvé que ce n'était pas foufou pour les perfs de l'app quand tu as des services externes hors rails (everydayrails/rails-4-1-rspec-3-0#36) mais ce n'est pas notre cas ici donc je ne pense pas que ça dérange.
| it "should send an email to user email" do | ||
| expect { ::Users::CreateTransaction.call(valid_user_attributes) } | ||
| .to change { ActionMailer::Base.deliveries.count }.by(1) | ||
| expect(ActionMailer::Base.deliveries.last.subject).to include('Welcome') |
There was a problem hiding this comment.
Tester qu'un mail est envoyé c'est bien. Tu pourrais aussi tester que la bonne méthode du mailer est appelé.
Tester le contenu du sujet du mail ça sort de la responsabilité de la transaction.
A ajouter : les modifications suite aux merges de 12, 13 et 14 dans master.