From d7811fca402de896d3658bc03ab1a8a36de7fa41 Mon Sep 17 00:00:00 2001
From: Mathieu Magnin <mathieu@magnin.sh>
Date: Tue, 10 Sep 2024 16:15:30 +0200
Subject: [PATCH] [#10751] Only send the first invitation mail to the
 instructeur if email is not verified

---
 .../groupe_instructeurs_controller.rb         |  3 ++-
 .../groupe_instructeurs_controller_spec.rb    | 26 +++++++++++++++++++
 spec/factories/instructeur.rb                 |  8 +++++-
 3 files changed, 35 insertions(+), 2 deletions(-)

diff --git a/app/controllers/instructeurs/groupe_instructeurs_controller.rb b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
index 7fbf5dbd2..3c6fccf22 100644
--- a/app/controllers/instructeurs/groupe_instructeurs_controller.rb
+++ b/app/controllers/instructeurs/groupe_instructeurs_controller.rb
@@ -50,9 +50,10 @@ module Instructeurs
           GroupeInstructeurMailer
             .notify_added_instructeurs(groupe_instructeur, [instructeur], current_user.email)
             .deliver_later
-        else
+        elsif instructeur.previously_new_record?
           InstructeurMailer.confirm_and_notify_added_instructeur(instructeur, groupe_instructeur, current_user.email).deliver_later
         end
+        # else instructeur already exists and email is not verified, so do not spam them
       end
 
       redirect_to instructeur_groupe_path(procedure, groupe_instructeur)
diff --git a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
index 7e52d38e5..5187da647 100644
--- a/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/instructeurs/groupe_instructeurs_controller_spec.rb
@@ -78,6 +78,11 @@ describe Instructeurs::GroupeInstructeursController, type: :controller do
   end
 
   describe '#add_instructeur' do
+    before do
+      allow(InstructeurMailer).to receive(:confirm_and_notify_added_instructeur).and_call_original
+      allow(GroupeInstructeurMailer).to receive(:notify_added_instructeurs).and_call_original
+    end
+
     subject do
       post :add_instructeur,
         params: {
@@ -95,6 +100,23 @@ describe Instructeurs::GroupeInstructeursController, type: :controller do
         expect(gi_1_2.instructeurs.map(&:email)).to include(new_instructeur_email)
         expect(flash.notice).to be_present
         expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_2))
+        expect(InstructeurMailer).to have_received(:confirm_and_notify_added_instructeur).with(instance_of(Instructeur), gi_1_2, anything)
+        expect(GroupeInstructeurMailer).not_to have_received(:notify_added_instructeurs)
+      end
+    end
+
+    context 'of an instructeur already known and with email verified' do
+      let(:known_instructeur) { create(:instructeur, :email_verified) }
+      let(:new_instructeur_email) { known_instructeur.email }
+
+      before { subject }
+
+      it "works" do
+        expect(gi_1_2.instructeurs.map(&:email)).to include(new_instructeur_email)
+        expect(flash.notice).to be_present
+        expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_2))
+        expect(InstructeurMailer).not_to have_received(:confirm_and_notify_added_instructeur)
+        expect(GroupeInstructeurMailer).to have_received(:notify_added_instructeurs)
       end
     end
 
@@ -105,6 +127,8 @@ describe Instructeurs::GroupeInstructeursController, type: :controller do
       it "works" do
         expect(flash.alert).to be_present
         expect(response).to redirect_to(instructeur_groupe_path(procedure, gi_1_2))
+        expect(InstructeurMailer).not_to have_received(:confirm_and_notify_added_instructeur)
+        expect(GroupeInstructeurMailer).not_to have_received(:notify_added_instructeurs)
       end
     end
 
@@ -114,6 +138,8 @@ describe Instructeurs::GroupeInstructeursController, type: :controller do
       it "works" do
         expect { subject }.not_to enqueue_email
         expect(flash.alert).to include(new_instructeur_email)
+        expect(InstructeurMailer).not_to have_received(:confirm_and_notify_added_instructeur)
+        expect(GroupeInstructeurMailer).not_to have_received(:notify_added_instructeurs)
       end
     end
   end
diff --git a/spec/factories/instructeur.rb b/spec/factories/instructeur.rb
index a26627619..31d9e8b02 100644
--- a/spec/factories/instructeur.rb
+++ b/spec/factories/instructeur.rb
@@ -10,7 +10,13 @@ FactoryBot.define do
 
     transient do
       email { generate(:instructeur_email) }
-      password { 'somethingverycomplated!' }
+      password { '{my-%s3cure[]-p4$$w0rd' }
+    end
+
+    trait :email_verified do
+      after(:create) do |instructeur|
+        instructeur.user.update(email_verified_at: Time.zone.now)
+      end
     end
 
     trait :with_agent_connect_information do