Merge pull request #2361 from betagouv/fix_2359_bad_avis_link

Fix 2359 bad avis link

app/controllers/new_gestionnaire/avis_controller.rb

@@ -96,7 +96,6 @@ module NewGestionnaire
         sign_in(gestionnaire, scope: :gestionnaire)
 
         Avis.link_avis_to_gestionnaire(gestionnaire)
-        avis = Avis.find(params[:id])
         redirect_to url_for(gestionnaire_avis_index_path)
       else
         flash[:alert] = gestionnaire.errors.full_messages

app/views/avis_mailer/avis_invitation.html.haml

@@ -12,7 +12,7 @@
 
 - if @avis.gestionnaire.present?
   %p
-    = link_to "Connectez-vous pour donner votre avis", gestionnaire_dossier_url(@avis.dossier.procedure, @avis.dossier)
+    = link_to "Connectez-vous pour donner votre avis", gestionnaire_avis_url(@avis)
 - else
   %p
     = link_to "Inscrivez-vous pour donner votre avis", sign_up_gestionnaire_avis_url(@avis.id, @avis.email)

spec/mailers/avis_mailer_spec.rb

@@ -9,5 +9,15 @@ RSpec.describe AvisMailer, type: :mailer do
     it { expect(subject.subject).to eq("Donnez votre avis sur le dossier nº #{avis.dossier.id} (#{avis.dossier.procedure.libelle})") }
     it { expect(subject.body).to include("Vous avez été invité par #{avis.claimant.email} à donner votre avis sur le dossier nº #{avis.dossier.id} de la procédure "#{avis.dossier.procedure.libelle}".") }
     it { expect(subject.body).to include(avis.introduction) }
+    it { expect(subject.body).to include(gestionnaire_avis_url(avis)) }
+
+    context 'when the recipient is not already registered' do
+      before do
+        avis.email = 'accompagnateur@email.com'
+        avis.gestionnaire = nil
+      end
+
+      it { expect(subject.body).to include(sign_up_gestionnaire_avis_url(avis.id, avis.email)) }
+    end
   end
 end