From 22089efa126711e2bcffb33e5581afa14a7ec959 Mon Sep 17 00:00:00 2001 From: Eric Leroy-Terquem Date: Mon, 6 Feb 2023 16:11:40 +0100 Subject: [PATCH] test(groupe instructeurs): add test and preview when removing instructeurs --- spec/mailers/groupe_instructeur_mailer_spec.rb | 15 ++++++++------- .../previews/groupe_instructeur_mailer_preview.rb | 4 ++-- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/spec/mailers/groupe_instructeur_mailer_spec.rb b/spec/mailers/groupe_instructeur_mailer_spec.rb index ba9f82711..9157f03ef 100644 --- a/spec/mailers/groupe_instructeur_mailer_spec.rb +++ b/spec/mailers/groupe_instructeur_mailer_spec.rb @@ -1,20 +1,21 @@ RSpec.describe GroupeInstructeurMailer, type: :mailer do - describe '#add_instructeurs' do + describe '#remove_instructeurs' do let(:groupe_instructeur) do gi = GroupeInstructeur.create(label: 'gi1', procedure: create(:procedure)) gi.instructeurs << create(:instructeur, email: 'int1@g') gi.instructeurs << create(:instructeur, email: 'int2@g') + gi.instructeurs << instructeurs_to_remove gi end - let(:instructeur_1) { create(:instructeur) } - let(:instructeur_2) { create(:instructeur) } + let(:instructeur_1) { create(:instructeur, email: 'int3@g') } + let(:instructeur_2) { create(:instructeur, email: 'int4@g') } - let(:instructeurs) { [instructeur_1, instructeur_2] } + let(:instructeurs_to_remove) { [instructeur_1, instructeur_2] } let(:current_instructeur_email) { 'toto@email.com' } - subject { described_class.add_instructeurs(groupe_instructeur, instructeurs, current_instructeur_email) } + subject { described_class.remove_instructeurs(groupe_instructeur, instructeurs_to_remove, current_instructeur_email) } - it { expect(subject.body).to include('Bonjour') } - it { expect(subject.bcc).to match_array(['int1@g', 'int2@g']) } + it { expect(subject.body).to include('Les instructeurs int3@g, int4@g ont été retirés du groupe') } + it { expect(subject.bcc).to match_array(['int1@g', 'int2@g', 'int3@g', 'int4@g']) } end end diff --git a/spec/mailers/previews/groupe_instructeur_mailer_preview.rb b/spec/mailers/previews/groupe_instructeur_mailer_preview.rb index f8366a7b2..1b6c006a1 100644 --- a/spec/mailers/previews/groupe_instructeur_mailer_preview.rb +++ b/spec/mailers/previews/groupe_instructeur_mailer_preview.rb @@ -1,9 +1,9 @@ class GroupeInstructeurMailerPreview < ActionMailer::Preview - def add_instructeurs + def remove_instructeurs procedure = Procedure.new(id: 1, libelle: 'une superbe procedure') groupe = GroupeInstructeur.new(id: 1, label: 'Val-De-Marne', procedure:) current_instructeur_email = 'admin@dgfip.com' instructeurs = Instructeur.limit(2) - GroupeInstructeurMailer.add_instructeurs(groupe, instructeurs, current_instructeur_email) + GroupeInstructeurMailer.remove_instructeurs(groupe, instructeurs, current_instructeur_email) end end