[#1110] use modern rather than legacy tags in default mail templates

app/models/mails/closed_mail.rb

@@ -7,7 +7,7 @@ module Mails
     SLUG = "closed_mail"
     TEMPLATE_NAME = "mails/closed_mail"
     DISPLAYED_NAME = "Accusé d'acceptation"
-    DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a été accepté'
+    DEFAULT_OBJECT = 'Votre dossier TPS nº --numéro du dossier-- a été accepté'
     IS_DOSSIER_TERMINE = true
   end
 end

app/models/mails/initiated_mail.rb

@@ -7,7 +7,7 @@ module Mails
     SLUG = "initiated_mail"
     TEMPLATE_NAME = "mails/initiated_mail"
     DISPLAYED_NAME = 'Accusé de réception'
-    DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a bien été reçu'
+    DEFAULT_OBJECT = 'Votre dossier TPS nº --numéro du dossier-- a bien été reçu'
     IS_DOSSIER_TERMINE = false
   end
 end

app/models/mails/received_mail.rb

@@ -7,7 +7,7 @@ module Mails
     SLUG = "received_mail"
     TEMPLATE_NAME = "mails/received_mail"
     DISPLAYED_NAME = 'Accusé de passage en instruction'
-    DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- va être instruit'
+    DEFAULT_OBJECT = 'Votre dossier TPS nº --numéro du dossier-- va être instruit'
     IS_DOSSIER_TERMINE = false
   end
 end

app/models/mails/refused_mail.rb

@@ -7,7 +7,7 @@ module Mails
     SLUG = "refused_mail"
     TEMPLATE_NAME = "mails/refused_mail"
     DISPLAYED_NAME = 'Accusé de rejet du dossier'
-    DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a été refusé'
+    DEFAULT_OBJECT = 'Votre dossier TPS nº --numéro du dossier-- a été refusé'
     IS_DOSSIER_TERMINE = true
   end
 end

app/models/mails/without_continuation_mail.rb

@@ -7,7 +7,7 @@ module Mails
     SLUG = "without_continuation"
     TEMPLATE_NAME = "mails/without_continuation_mail"
     DISPLAYED_NAME = 'Accusé de classement sans suite'
-    DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a été classé sans suite'
+    DEFAULT_OBJECT = 'Votre dossier TPS nº --numéro du dossier-- a été classé sans suite'
     IS_DOSSIER_TERMINE = true
   end
 end

app/views/mails/closed_mail.html.haml

@@ -1,10 +1,10 @@
 Bonjour,
 %br
 %br
-Votre dossier nº --numero_dossier-- a été accepté le --date_de_decision--.
+Votre dossier nº --numéro du dossier-- a été accepté le --date de décision--.
 %br
 %br
-A tout moment, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier--
+A tout moment, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien dossier--
 %br
 %br
 Bonne journée,

app/views/mails/initiated_mail.html.haml

@@ -1,10 +1,10 @@
 Bonjour,
 %br
 %br
-Votre administration vous confirme la bonne réception de votre dossier nº --numero_dossier--.
+Votre administration vous confirme la bonne réception de votre dossier nº --numéro du dossier--.
 %br
 %br
-A tout moment, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier--
+A tout moment, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien dossier--
 %br
 %br
 Bonne journée,

app/views/mails/received_mail.html.haml

@@ -1,7 +1,7 @@
 Bonjour,
 %br
 %br
-Votre administration vous confirme la bonne réception de votre dossier nº --numero_dossier--. Celui-ci sera instruit dans le délai légal déclaré par votre interlocuteur.
+Votre administration vous confirme la bonne réception de votre dossier nº --numéro du dossier--. Celui-ci sera instruit dans le délai légal déclaré par votre interlocuteur.
 %br
 %br
 Bonne journée,

app/views/mails/refused_mail.html.haml

@@ -1,10 +1,10 @@
 Bonjour,
 %br
 %br
-Votre dossier nº --numero_dossier-- a été refusé le --date_de_decision--.
+Votre dossier nº --numéro du dossier-- a été refusé le --date de décision--.
 %br
 %br
-Pour en savoir plus sur le motif du refus, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier--
+Pour en savoir plus sur le motif du refus, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien dossier--
 %br
 %br
 Bonne journée,

app/views/mails/without_continuation_mail.html.haml

@@ -1,10 +1,10 @@
 Bonjour,
 %br
 %br
-Votre dossier nº --numero_dossier-- a été classé sans suite le --date_de_decision--.
+Votre dossier nº --numéro du dossier-- a été classé sans suite le --date de décision--.
 %br
 %br
-Pour en savoir plus sur les raisons de ce classement sans suite, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier--
+Pour en savoir plus sur les raisons de ce classement sans suite, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien dossier--
 %br
 %br
 Bonne journée,

spec/factories/mail_templates.rb

@@ -8,8 +8,8 @@ FactoryGirl.define do
     factory :dossier_refused_mail_template, class: Mails::RefusedMail
 
     factory :dossier_en_instruction_mail_template, class: Mails::InitiatedMail do
-      object "[TPS] Accusé de réception pour votre dossier nº --numero_dossier--"
+      object "[TPS] Accusé de réception pour votre dossier nº --numéro du dossier--"
-      body "Votre administration vous confirme la bonne réception de votre dossier nº --numero_dossier--"
+      body "Votre administration vous confirme la bonne réception de votre dossier nº --numéro du dossier--"
     end
   end
 end

spec/models/concern/mail_template_concern_spec.rb

@@ -15,14 +15,14 @@ describe MailTemplateConcern do
     end
 
     describe 'with one token to replace' do
-      let(:template) { '[TPS] Dossier : --numero_dossier--' }
+      let(:template) { '[TPS] Dossier : --numéro du dossier--' }
       it do
         is_expected.to eq("[TPS] Dossier : #{dossier.id}")
       end
     end
 
     describe 'with multiples tokens to replace' do
-      let(:template) { '[TPS] --numero_dossier-- --libelle_procedure-- --lien_dossier--' }
+      let(:template) { '[TPS] --numéro du dossier-- --libellé procédure-- --lien dossier--' }
       it do
         expected =
           "[TPS] #{dossier.id} #{dossier.procedure.libelle} " +
@@ -48,10 +48,10 @@ describe MailTemplateConcern do
   end
 
   describe '.replace_tags' do
+    before { initiated_mail.body = "n --numéro du dossier--" }
     it "avoids side effects" do
-      subject = "n --numero_dossier--"
+      expect(initiated_mail.body_for_dossier(dossier)).to eq("n #{dossier.id}")
-      expect(initiated_mail.replace_tags(subject, dossier)).to eq("n #{dossier.id}")
+      expect(initiated_mail.body_for_dossier(dossier2)).to eq("n #{dossier2.id}")
-      expect(initiated_mail.replace_tags(subject, dossier2)).to eq("n #{dossier2.id}")
     end
   end
 end