Merge pull request #8626 from E-L-T/notify-instructeur-when-removed-from-procedure

feat(groupe instructeur mailer): add emailing to removed instructeurs
2023-02-16 10:44:35 +01:00 · 2023-02-16 10:44:35 +01:00 · bfa5f23cb9
commit bfa5f23cb9
parent de11d97df9 4926fbff19
13 changed files with 125 additions and 41 deletions
--- a/app/controllers/administrateurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/administrateurs/groupe_instructeurs_controller.rb
@ -146,15 +146,18 @@ module Administrateurs
        instructeur = groupe_instructeur.instructeurs.find_by(id: instructeur_id)

        if groupe_instructeur.remove(instructeur)
-          flash[:notice] = if procedure.routing_enabled?
-            GroupeInstructeurMailer
-              .remove_instructeurs(groupe_instructeur, [instructeur], current_administrateur.email)
-              .deliver_later
-
+          flash[:notice] = if instructeur.in?(procedure.instructeurs)
            "L’instructeur « #{instructeur.email} » a été retiré du groupe."
          else
            "L’instructeur a bien été désaffecté de la démarche"
          end
+          GroupeInstructeurMailer
+            .notify_removed_instructeur(groupe_instructeur, instructeur, current_administrateur.email)
+            .deliver_later
+
+          GroupeInstructeurMailer
+            .notify_group_when_instructeurs_removed(groupe_instructeur, [instructeur], current_administrateur.email)
+            .deliver_later
        else
          flash[:alert] = if procedure.routing_enabled?
            if instructeur.present?
--- a/app/controllers/instructeurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
@ -35,7 +35,11 @@ module Instructeurs
        if groupe_instructeur.remove(instructeur)
          flash[:notice] = "L’instructeur « #{instructeur.email} » a été retiré du groupe."
          GroupeInstructeurMailer
-            .remove_instructeurs(groupe_instructeur, [instructeur], current_user.email)
+            .notify_removed_instructeur(groupe_instructeur, instructeur, current_user.email)
+            .deliver_later
+
+          GroupeInstructeurMailer
+            .notify_group_when_instructeurs_removed(groupe_instructeur, [instructeur], current_user.email)
            .deliver_later
        else
          flash[:alert] = "L’instructeur « #{instructeur.email} » n’est pas dans le groupe."
--- a/app/graphql/mutations/groupe_instructeur_supprimer_instructeurs.rb
+++ b/app/graphql/mutations/groupe_instructeur_supprimer_instructeurs.rb
@ -17,7 +17,7 @@ module Mutations

      if groupe_instructeur.procedure.routing_enabled? && instructeurs.present?
        GroupeInstructeurMailer
-          .remove_instructeurs(groupe_instructeur, instructeurs, current_administrateur.email)
+          .notify_group_when_instructeurs_removed(groupe_instructeur, instructeurs, current_administrateur.email)
          .deliver_later
      end

--- a/app/mailers/groupe_instructeur_mailer.rb
+++ b/app/mailers/groupe_instructeur_mailer.rb
@ -1,7 +1,7 @@
 class GroupeInstructeurMailer < ApplicationMailer
  layout 'mailers/layout'

-  def remove_instructeurs(group, removed_instructeurs, current_instructeur_email)
+  def notify_group_when_instructeurs_removed(group, removed_instructeurs, current_instructeur_email)
    @removed_instructeur_emails = removed_instructeurs.map(&:email)
    @group = group
    @current_instructeur_email = current_instructeur_email
@ -11,4 +11,17 @@ class GroupeInstructeurMailer < ApplicationMailer
    emails = @group.instructeurs.map(&:email)
    mail(bcc: emails, subject: subject)
  end
+
+  def notify_removed_instructeur(group, removed_instructeur, current_instructeur_email)
+    @group = group
+    @current_instructeur_email = current_instructeur_email
+    @still_assigned_to_procedure = removed_instructeur.in?(group.procedure.instructeurs)
+    subject = if @still_assigned_to_procedure
+      "Vous avez été retiré du groupe \"#{group.label}\" de la démarche \"#{group.procedure.libelle}\""
+    else
+      "Vous avez été désaffecté de la démarche \"#{group.procedure.libelle}\""
+    end
+
+    mail(to: removed_instructeur.email, subject: subject)
+  end
 end
--- a/app/views/groupe_instructeur_mailer/notify_group_when_instructeurs_removed.html.haml
+++ b/app/views/groupe_instructeur_mailer/notify_group_when_instructeurs_removed.html.haml
@ -0,0 +1,11 @@
+%p
+  Bonjour,
+
+%p
+  = t('administrateurs.groupe_instructeurs.notify_group_when_instructeurs_removed.email_body', count: @removed_instructeur_emails.size, emails: @removed_instructeur_emails.join(', '), groupe: @group.label, email: @current_instructeur_email, procedure: @group.procedure.libelle)
+
+%p
+  Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe :
+  = link_to(@group.label, admin_procedure_groupe_instructeur_url(@group.procedure, @group))
+
+= render partial: "layouts/mailers/signature"
--- a/app/views/groupe_instructeur_mailer/notify_removed_instructeur.html.haml
+++ b/app/views/groupe_instructeur_mailer/notify_removed_instructeur.html.haml
@ -0,0 +1,8 @@
+%p
+  Bonjour,
+
+%p
+  - assignment_state = @still_assigned_to_procedure ? 'assigned' : 'unassigned'
+  = t("administrateurs.groupe_instructeurs.notify_removed_instructeur.#{assignment_state}.email_body", groupe: @group.label, email: @current_instructeur_email, procedure: @group.procedure.libelle)
+
+= render partial: "layouts/mailers/signature"
--- a/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml
+++ b/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml
@ -1,11 +0,0 @@
-%p
-  Bonjour,
-
-%p
-  = t('administrateurs.groupe_instructeurs.remove_instructeur.email_body', count: 1, emails: @email, groupe: @group.label, email: @current_instructeur_email, procedure: @group.procedure.libelle)
-
-%p
-  Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe :
-  = link_to(@group.label, admin_procedure_groupe_instructeur_url(@group.procedure, @group))
-
-= render partial: "layouts/mailers/signature"
--- a/app/views/groupe_instructeur_mailer/remove_instructeurs.html.haml
+++ b/app/views/groupe_instructeur_mailer/remove_instructeurs.html.haml
@ -1,11 +0,0 @@
-%p
-  Bonjour,
-
-%p
-  = t('administrateurs.groupe_instructeurs.remove_instructeur.email_body', count: @removed_instructeur_emails.size, emails: @removed_instructeur_emails.join(', '), groupe: @group.label, email: @current_instructeur_email, procedure: @group.procedure.libelle)
-
-%p
-  Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe :
-  = link_to(@group.label, admin_procedure_groupe_instructeur_url(@group.procedure, @group))
-
-= render partial: "layouts/mailers/signature"
--- a/config/locales/views/administrateurs/groupe_instructeurs/en.yml
+++ b/config/locales/views/administrateurs/groupe_instructeurs/en.yml
@ -19,10 +19,15 @@ en:
        email_body:
          one: "The instructor %{emails} was assigned to the group « %{groupe} » by « %{email} », in charge of procedure « %{procedure} »."
          other: "The instructors %{emails} were assigned to the group « %{groupe} » by « %{email} », in charge of procedure « %{procedure} »."
-      remove_instructeur:
+      notify_group_when_instructeurs_removed:
        email_body:
          one: "The instructor %{emails} was removed from the group « %{groupe} » by « %{email} », in charge of procedure « %{procedure} »."
          other: "The instructors %{emails} were removed from the group « %{groupe} » by « %{email} », in charge of procedure « %{procedure} »."
+      notify_removed_instructeur:
+        unassigned:
+          email_body: "You were unassigned from the procedure « %{procedure} » by « %{email} »."
+        assigned:
+          email_body: "You were removed from the group « %{groupe} » by « %{email} », in charge of procedure « %{procedure} »."
      reaffecter_dossiers:
        existing_groupe:
          one: "%{count} group exist"
--- a/config/locales/views/administrateurs/groupe_instructeurs/fr.yml
+++ b/config/locales/views/administrateurs/groupe_instructeurs/fr.yml
@ -25,10 +25,15 @@ fr:
        email_body:
          one: "L’instructeur %{emails} a été affecté au groupe « %{groupe} » par « %{email} », en charge de la démarche « %{procedure} »."
          other: "Les instructeurs %{emails} ont été affectés au groupe « %{groupe} » par « %{email} », en charge de la démarche « %{procedure} »."
-      remove_instructeur:
+      notify_group_when_instructeurs_removed:
        email_body:
          one: "L’instructeur %{emails} a été retiré du groupe « %{groupe} » par « %{email} », en charge de la démarche « %{procedure} »."
          other: "Les instructeurs %{emails} ont été retirés du groupe « %{groupe} » par « %{email} », en charge de la démarche « %{procedure} »."
+      notify_removed_instructeur:
+        assigned:
+          email_body: "Vous avez été retiré du groupe « %{groupe} » par « %{email} », en charge de la démarche « %{procedure} »."
+        unassigned:
+          email_body: "Vous avez été désaffecté de la démarche « %{procedure} » par « %{email} »."
      reaffecter_dossiers:
        existing_groupe:
          one: "%{count} groupe existe"
--- a/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
@ -320,11 +320,22 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
    end

    context 'when there are many instructeurs' do
-      before { remove_instructeur(admin.instructeur) }
+      before do
+        allow(GroupeInstructeurMailer).to receive(:notify_removed_instructeur)
+          .and_return(double(deliver_later: true))
+        remove_instructeur(admin.instructeur)
+      end

      it { expect(gi_1_1.instructeurs).to include(instructeur) }
      it { expect(gi_1_1.reload.instructeurs.count).to eq(1) }
      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_1)) }
+      it "calls GroupeInstructeurMailer with the right groupe and instructeur" do
+        expect(GroupeInstructeurMailer).to have_received(:notify_removed_instructeur).with(
+          gi_1_1,
+          admin.instructeur,
+          admin.email
+        )
+      end
    end

    context 'when there is only one instructeur' do
--- a/spec/mailers/groupe_instructeur_mailer_spec.rb
+++ b/spec/mailers/groupe_instructeur_mailer_spec.rb
@ -1,5 +1,5 @@
 RSpec.describe GroupeInstructeurMailer, type: :mailer do
-  describe '#remove_instructeurs' do
+  describe '#notify_group_when_instructeurs_removed' do
    let(:groupe_instructeur) do
      gi = GroupeInstructeur.create(label: 'gi1', procedure: create(:procedure))
      gi.instructeurs << create(:instructeur, email: 'int1@g')
@ -7,15 +7,53 @@ RSpec.describe GroupeInstructeurMailer, type: :mailer do
      gi.instructeurs << instructeurs_to_remove
      gi
    end
-    let(:instructeur_1) { create(:instructeur, email: 'int3@g') }
-    let(:instructeur_2) { create(:instructeur, email: 'int4@g') }
+    let(:instructeur_3) { create(:instructeur, email: 'int3@g') }
+    let(:instructeur_4) { create(:instructeur, email: 'int4@g') }

-    let(:instructeurs_to_remove) { [instructeur_1, instructeur_2] }
+    let(:instructeurs_to_remove) { [instructeur_3, instructeur_4] }
    let(:current_instructeur_email) { 'toto@email.com' }

-    subject { described_class.remove_instructeurs(groupe_instructeur, instructeurs_to_remove, current_instructeur_email) }
+    subject { described_class.notify_group_when_instructeurs_removed(groupe_instructeur, instructeurs_to_remove, current_instructeur_email) }
+
+    before { instructeurs_to_remove.each { groupe_instructeur.remove(_1) } }

    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']) }
+    it { expect(subject.bcc).to match_array(['int1@g', 'int2@g']) }
+  end
+
+  describe '#notify_removed_instructeur' do
+    let(:procedure) { create(:procedure) }
+    let(:groupe_instructeur) do
+      gi = GroupeInstructeur.create(label: 'gi1', procedure: procedure)
+      gi.instructeurs << create(:instructeur, email: 'int1@g')
+      gi.instructeurs << create(:instructeur, email: 'int2@g')
+      gi.instructeurs << instructeur_to_remove
+      gi
+    end
+    let(:instructeur_to_remove) { create(:instructeur, email: 'int3@g') }
+
+    let(:current_instructeur_email) { 'toto@email.com' }
+
+    subject { described_class.notify_removed_instructeur(groupe_instructeur, instructeur_to_remove, current_instructeur_email) }
+
+    before { groupe_instructeur.remove(instructeur_to_remove) }
+
+    context 'when instructeur is fully removed form procedure' do
+      it { expect(subject.body).to include('Vous avez été désaffecté de la démarche') }
+      it { expect(subject.to).to include('int3@g') }
+      it { expect(subject.to).not_to include('int1@g', 'int2@g') }
+    end
+
+    context 'when instructeur is removed from one group but still affected to procedure' do
+      let!(:groupe_instructeur_2) do
+        gi2 = GroupeInstructeur.create(label: 'gi2', procedure: procedure)
+        gi2.instructeurs << instructeur_to_remove
+        gi2
+      end
+
+      it { expect(subject.body).to include('Vous avez été retiré du groupe « gi1 » par « toto@email.com »') }
+      it { expect(subject.to).to include('int3@g') }
+      it { expect(subject.to).not_to include('int1@g', 'int2@g') }
+    end
  end
 end
--- a/spec/mailers/previews/groupe_instructeur_mailer_preview.rb
+++ b/spec/mailers/previews/groupe_instructeur_mailer_preview.rb
@ -1,9 +1,17 @@
 class GroupeInstructeurMailerPreview < ActionMailer::Preview
-  def remove_instructeurs
+  def notify_group_when_instructeurs_removed
    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.remove_instructeurs(groupe, instructeurs, current_instructeur_email)
+    GroupeInstructeurMailer.notify_group_when_instructeurs_removed(groupe, instructeurs, current_instructeur_email)
+  end
+
+  def notify_removed_instructeur
+    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'
+    instructeur = Instructeur.last
+    GroupeInstructeurMailer.notify_removed_instructeur(groupe, instructeur, current_instructeur_email)
  end
 end