mailer: improve notify_new_answer wording

And also insert the reply email address directly in the email body.

app/views/dossier_mailer/notify_new_answer.html.haml

@@ -7,7 +7,10 @@
   %p= t('.body_draft_html', libelle_demarche: @dossier.procedure.libelle)
   %p{ style: "padding: 8px; color: #333333; background-color: #EEEEEE; font-size: 14px;" }
     = @body
-  %p= t('.contact')
+  - if @service&.email.present?
+    %p= t('.contact_html', email: @service.email)
+  - else
+    %p= t('.contact_no_email')
   = round_button(t('.access_file'), dossier_url(@dossier), :primary)
 - else
   %p= t('.body_html', libelle_demarche: @dossier.procedure.libelle)

config/locales/views/dossier_mailer/notify_new_answer/en.yml

@@ -8,6 +8,7 @@ en:
         To read the message and answer it, select the following link:
       body_draft_html: |
         You received <b>a new message</b> from the service in charge of reviewing the file you started a draft for on the procedure « %{libelle_demarche} ».
-      contact: If you chose to contact the service, please use the email available below in the page.
+      contact_html: "If you chose to contact the service, please send an email directly to this address: <a href=\"mailto:%{email}\">%{email}</a>"
+      contact_no_email: If you chose to contact the service, please use the contact infos available below.
       access_message: Read the message
       access_file: Open file

config/locales/views/dossier_mailer/notify_new_answer/fr.yml

@@ -3,11 +3,12 @@ fr:
     notify_new_answer:
       subject: Nouveau message pour votre dossier nº %{dossier_id} « %{libelle_demarche} »
       body_html: |
-        Vous avez reçu un <b>nouveau message</b> de la part du service en charge de votre dossier sur la démarche « %{libelle_demarche} ».
+        Vous avez reçu un <b>nouveau message</b> de la part de l’administration en charge de votre dossier sur la démarche « %{libelle_demarche} ».
       link: |
         Pour consulter le message et y répondre, cliquez sur le bouton ci-dessous :
       body_draft_html: |
-        Vous avez reçu un <b>nouveau message</b> du service pour lequel votre dossier est en brouillon pour la démarche « %{libelle_demarche} ».
+        Vous avez reçu un <b>nouveau message</b> de l’administration, au sujet de votre dossier en brouillon pour la démarche « %{libelle_demarche} ».
-      contact: Si vous souhaitez contacter le service, merci de le faire directement à l'aide de l'email en bas de page.
+      contact_html: "Si vous souhaitez répondre à ce message, contactez directement l’administration à l’adresse suivante : <a href=\"mailto:%{email}\">%{email}</a>"
+      contact_no_email: Si vous souhaitez répondre à ce message, contactez directement l’administration à l’aide des coordonnées en bas de cet email.
       access_message: Lire le message
       access_file: Voir le dossier

spec/mailers/dossier_mailer_spec.rb

@@ -27,15 +27,23 @@ RSpec.describe DossierMailer, type: :mailer do
   end
 
   describe '.notify_new_answer with dossier brouillon' do
-    let(:dossier) { create(:dossier, procedure: create(:simple_procedure)) }
+    let(:service) { build(:service) }
+    let(:procedure) { create(:simple_procedure, service: service) }
+    let(:dossier) { create(:dossier, procedure: procedure) }
     let(:commentaire) { create(:commentaire, dossier: dossier) }
     subject { described_class.with(commentaire: commentaire).notify_new_answer }
 
     it { expect(subject.subject).to include("Nouveau message") }
     it { expect(subject.subject).to include(dossier.id.to_s) }
+    it { expect(subject.body).to include(dossier.procedure.service.email) }
     it { expect(subject.body).not_to include(messagerie_dossier_url(dossier)) }
 
     it_behaves_like 'a dossier notification'
+
+    context 'when there is no associated service' do
+      let(:service) { nil }
+      it { expect { subject }.not_to raise_error }
+    end
   end
 
   describe '.notify_new_answer with dossier en construction' do