Merge pull request #4644 from betagouv/bravo_simon

Corrige un bug concernant les destinataires du mail annoncant un nouvel instructeur dans un groupe

app/controllers/new_administrateur/groupe_instructeurs_controller.rb

@@ -69,12 +69,12 @@ module NewAdministrateur
             create_instructeur(instructeur_email)
         end
 
+        groupe_instructeur.instructeurs << instructeurs
+
         GroupeInstructeurMailer
           .add_instructeurs(groupe_instructeur, instructeurs, current_user.email)
           .deliver_later
 
-        groupe_instructeur.instructeurs << instructeurs
-
         flash[:notice] = t('.assignment',
           count: email_to_adds.count,
           value: email_to_adds.join(', '),

app/mailers/groupe_instructeur_mailer.rb

@@ -1,9 +1,9 @@
 class GroupeInstructeurMailer < ApplicationMailer
   layout 'mailers/layout'
 
-  def add_instructeurs(group, instructeurs, current_instructeur_email)
-    @new_instructeur_emails = instructeurs.map(&:email)
-    @group = Procedure.last.defaut_groupe_instructeur
+  def add_instructeurs(group, new_instructeurs, current_instructeur_email)
+    @new_instructeur_emails = new_instructeurs.map(&:email)
+    @group = group
     @current_instructeur_email = current_instructeur_email
 
     subject = "Ajout d’un instructeur dans le groupe \"#{group.label}\""

spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb

@@ -89,6 +89,9 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
     before do
       gi_1_1.instructeurs << instructeur
 
+      allow(GroupeInstructeurMailer).to receive(:add_instructeurs)
+        .and_return(double(deliver_later: true))
+
       post :add_instructeur,
         params: {
           procedure_id: procedure.id,
@@ -103,6 +106,13 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       it { expect(gi_1_1.instructeurs.pluck(:email)).to include(*new_instructeur_emails) }
       it { expect(flash.notice).to be_present }
       it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) }
+      it "calls GroupeInstructeurMailer with the right groupe and instructeurs" do
+        expect(GroupeInstructeurMailer).to have_received(:add_instructeurs).with(
+          gi_1_1,
+          satisfy { |instructeurs| instructeurs.all? { |i| new_instructeur_emails.include?(i.email) } },
+          admin.email
+        )
+      end
     end
 
     context 'of an instructeur already in the group' do

spec/mailers/groupe_instructeur_mailer_spec.rb

@@ -0,0 +1,20 @@
+RSpec.describe GroupeInstructeurMailer, type: :mailer do
+  describe '#add_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
+    end
+    let(:instructeur_1) { create(:instructeur) }
+    let(:instructeur_2) { create(:instructeur) }
+
+    let(:instructeurs) { [instructeur_1, instructeur_2] }
+    let(:current_instructeur_email) { 'toto@email.com' }
+
+    subject { described_class.add_instructeurs(groupe_instructeur, instructeurs, current_instructeur_email) }
+
+    it { expect(subject.body).to include('Bonjour') }
+    it { expect(subject.bcc).to match_array(['int1@g', 'int2@g']) }
+  end
+end