amelioration(attestation.et.mail): repositionne les messages d'erreur lié à la substitution des balises a proximité du formulaire pour faciliter la comprehension. passage des champs au DSFR

app/assets/stylesheets/dsfr.scss

@@ -3,7 +3,8 @@
 // override default transparent background on inputs & font-size to 16px by default
 input,
 textarea,
-select {
+select,
+.procedure-form__column--form .fr-input { //FIXME when DSFR is DONE
   background: $white;
   font-size: 1em;
 }

app/components/dsfr/input_component.rb

@@ -0,0 +1,76 @@
+class Dsfr::InputComponent < ApplicationComponent
+  def initialize(form:, attribute:, input_type:, opts: {}, required: true)
+    @form = form
+    @attribute = attribute
+    @input_type = input_type
+    @opts = opts
+    @required = required
+  end
+
+  def input_opts
+    @opts[:class] = class_names(map_array_to_hash_with_true(@opts[:class])
+                                  .merge('fr-input': true,
+                                         'fr-mb-0': true,
+                                         'fr-input--error': errors_on_attribute?))
+
+    if errors_on_attribute?
+      @opts = @opts.deep_merge(aria: {
+        describedby: error_message_id,
+                                       invalid: true
+      })
+    end
+    if @required
+      @opts[:required] = true
+    end
+    @opts
+  end
+
+  # add invalid class on input when input is invalid
+  # and and valid on input only if another input is invalid
+  def input_group_class_names
+    class_names('fr-input-group': true,
+                "fr-input-group--error": errors_on_attribute?,
+                "fr-input-group--valid": !errors_on_attribute? && errors_on_another_attribute?)
+  end
+
+  # tried to inline it within the template, but failed miserably with a double render
+  def label
+    label = @form.object.class.human_attribute_name(@attribute)
+
+    if @required
+      label += tag.span(" *", class: 'mandatory')
+    end
+    label
+  end
+
+  def errors_on_attribute?
+    @form.object.errors.has_key?(attribute_or_rich_body)
+  end
+
+  def error_message_id
+    dom_id(@form.object, @attribute)
+  end
+
+  def error_messages
+    @form.object.errors.full_messages_for(attribute_or_rich_body)
+  end
+
+  private
+
+  def errors_on_another_attribute?
+    !@form.object.errors.empty?
+  end
+
+  def attribute_or_rich_body
+    case @input_type
+    when :rich_text_area
+      @attribute.to_s.sub(/\Arich_/, '').to_sym
+    else
+      @attribute
+    end
+  end
+
+  def map_array_to_hash_with_true(array_or_string_or_nil)
+    Array(array_or_string_or_nil).to_h { [_1, true] }
+  end
+end

app/components/dsfr/input_component/input_component.html.haml

@@ -0,0 +1,14 @@
+%div{ class: input_group_class_names }
+  = @form.label @attribute, label.html_safe, class: "fr-label"
+
+  = @form.send(@input_type, @attribute, input_opts)
+
+  - if errors_on_attribute?
+    - if error_messages.size == 1
+      %p.fr-error-text{ id: error_message_id }= error_messages.first
+    - else
+      %div.fr-error-text{ id: error_message_id }
+        %ul
+          - error_messages.map do |error_message|
+            %li= error_message
+

app/controllers/administrateurs/attestation_templates_controller.rb

@@ -14,11 +14,11 @@ module Administrateurs
       @attestation_template = @procedure.attestation_template
 
       if @attestation_template.update(activated_attestation_params)
-        flash.notice = "Le model de l’attestation a bien été modifiée"
+        flash.notice = "Le modèle de l’attestation a bien été modifié"
 
         redirect_to edit_admin_procedure_attestation_template_path(@procedure)
       else
-        flash.now.alert = @attestation_template.errors.full_messages
+        flash.now.alert = "Le modèle de l’attestation contient des erreurs et n'a pas pu être enregistré, veuillez les corriger."
 
         render :edit
       end
@@ -28,7 +28,7 @@ module Administrateurs
       @attestation_template = build_attestation_template(activated_attestation_params)
 
       if @attestation_template.save
-        flash.notice = "Le model de l’attestation a bien été enregistrée"
+        flash.notice = "Le modèle de l’attestation a bien été enregistré"
 
         redirect_to edit_admin_procedure_attestation_template_path(@procedure)
       else

app/controllers/administrateurs/mail_templates_controller.rb

@@ -25,7 +25,7 @@ module Administrateurs
         flash.notice = "Email mis à jour"
         redirect_to edit_admin_procedure_mail_template_path(mail_template.procedure_id, params[:id])
       else
-        flash.now.alert = mail_template.errors.full_messages
+        flash.now.alert = "L’email contient des erreurs et n’a pas pu être enregistré, veuillez les corriger."
         mail_template.rich_body = mail_template.body
 
         @mail_template = mail_template

app/views/administrateurs/attestation_templates/_informations.html.haml

@@ -1,12 +1,6 @@
-= f.label :title do
+= render Dsfr::InputComponent.new(form: f, attribute: :title, input_type: :text_field, opts: {})
-  Titre de l'attestation
-  %span.mandatory *
-= f.text_field :title, class: 'form-control', placeholder: 'Titre de l’attestation'
 
-= f.label :body do
+= render Dsfr::InputComponent.new(form: f, attribute: :body, input_type: :text_area, opts: { rows: '6', placeholder: 'Description de la démarche, destinataires, etc. '})
-  Corps du document
-  %span.mandatory *
-= f.text_area :body, rows: '6', placeholder: 'Description de la démarche, destinataires, etc. ', class: 'form-control'
 
 #tags-table
   %h2.add-tag-title
@@ -40,6 +34,4 @@
   %br
   Dimensions conseillées : au minimum 500 px de largeur ou de hauteur, poids maximum : 0,5 Mo.
 
-= f.label :footer do
+= render Dsfr::InputComponent.new(form: f, attribute: :footer, input_type: :text_field, opts: { maxlength: 190, size: nil }, required: false)
-  Pied de page
-= f.text_field :footer, class: 'form-control', maxlength: 190, size: nil

app/views/administrateurs/mail_templates/_form.html.haml

@@ -1,12 +1,6 @@
-= f.label :subject do
+= render Dsfr::InputComponent.new(form: f, attribute: :subject, input_type: :text_field, opts: {})
-  Objet de l'email
-  %span.mandatory *
-= f.text_field :subject, required: true
 
-= f.label :body do
+= render Dsfr::InputComponent.new(form: f, attribute: :rich_body, input_type: :rich_text_area, opts: { class: 'fr-mb-4vh'})
-  Corps de l'email
-  %span.mandatory *
-= f.rich_text_area :rich_body, required: true, class: "mb-4"
 
 #tags-table{ 'data-controller': 'trix' }
   %h2.add-tag-title

config/locales/models/attestation_template/fr.yml

@@ -1,20 +1,27 @@
 fr:
   activerecord:
+    models:
+      attestation_template: 'Attestation'
+    attributes:
+      attestation_template:
+        title: Titre de l’attestation
+        body: Contenu de l’attestation
+        footer: Pied de page
     errors:
       models:
         attestation_template:
           attributes:
             title:
-              format: Le titre du modèle de l’attestation %{message}
+              format: Le titre de l’attestation %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.
             body:
-              format: Le contenu du modèle de l’attestation %{message}
+              format: Le contenu de l’attestation %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.

config/locales/models/closed_mail/fr.yml

@@ -1,20 +1,24 @@
 fr:
   activerecord:
+    attributes:
+      mails/closed_mail:
+        subject: Objet de l’email
+        rich_body: Corps de l’email
     errors:
       models:
         mails/closed_mail:
           attributes:
             subject:
-              format: Le titre de l’email de notification d’acceptation de dossier %{message}
+              format: L’objet de l’email %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.
             body:
-              format: Le contenu de l’email de notification d’acceptation de dossier %{message}
+              format: Le corps de l’email %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.

config/locales/models/initiated_mail/fr.yml

@@ -1,20 +1,24 @@
 fr:
   activerecord:
+    attributes:
+      mails/initiated_mail:
+        subject: Objet de l’email
+        rich_body: Corps de l’email
     errors:
       models:
         mails/initiated_mail:
           attributes:
             subject:
-              format: Le titre de l’email de notification de passage du dossier en instruction %{message}
+              format: L’objet de l’email %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.
             body:
-              format: Le contenu de l’email de notification de passage du dossier en instruction %{message}
+              format: Le corps de l’email %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publié.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimé.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.

config/locales/models/received_mail/fr.yml

@@ -1,20 +1,24 @@
 fr:
   activerecord:
+    attributes:
+      mails/received_mail:
+        subject: Objet de l’email
+        rich_body: Corps de l’email
     errors:
       models:
         mails/received_mail:
           attributes:
             subject:
-              format: Le titre de l’email de notification de dépot de dossier %{message}
+              format: L’objet de l’email l’email %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.
             body:
-              format: Le contenu de l’email de notification de dépot de dossier %{message}
+              format: Le corps de l’email %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.

config/locales/models/refused_mail/fr.yml

@@ -1,20 +1,24 @@
 fr:
   activerecord:
+    attributes:
+       mails/refused_mail:
+        subject: Objet de l’email
+        rich_body: Corps de l’email
     errors:
       models:
         mails/refused_mail:
           attributes:
             subject:
-              format: Le titre de l’email de notification de refus de dossier %{message}
+              format: L’objet de l’email %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publié
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.
             body:
               format: Le contenu de l’email de notification de refus de dossier %{message}
-              champ_missing: réfère au champ "%{tag}" qui n’existe pas
+              champ_missing: contient la balise "%{tag}" qui n’existe pas, veuillez la supprimer.
-              champ_missing_in_draft_revision: réfère au champ "%{tag}" qui a été supprimé mais la suppression n’est pas encore publiée
+              champ_missing_in_draft_revision: contient la balise "%{tag}" qui a été supprimée mais la suppression n’est pas encore publiée.
-              champ_missing_in_published_revision: réfère au champ "%{tag}" qui n’est pas encore publiée
+              champ_missing_in_published_revision: contient la balise "%{tag}" qui n’est pas encore publiée.
-              champ_missing_in_published_and_draft_revision: réfère au champ "%{tag}" qui a été supprimé
+              champ_missing_in_published_and_draft_revision: contient la balise "%{tag}" qui a été supprimée.
-              champ_missing_in_previous_revision: réfère au champ "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement
+              champ_missing_in_previous_revision: contient la balise "%{tag}" qui n’existe pas sur un des dossiers en cours de traitement.

spec/controllers/administrateurs/attestation_templates_controller_spec.rb

@@ -180,7 +180,7 @@ describe Administrateurs::AttestationTemplatesController, type: :controller do
         expect(procedure.attestation_template.logo.download).to eq(logo2.read)
         expect(procedure.attestation_template.signature.download).to eq(signature2.read)
         expect(response).to redirect_to edit_admin_procedure_attestation_template_path(procedure)
-        expect(flash.notice).to eq("Le model de l’attestation a bien été modifiée")
+        expect(flash.notice).to eq("Le modèle de l’attestation a bien été modifiée")
       end
     end
 
@@ -233,33 +233,32 @@ describe Administrateurs::AttestationTemplatesController, type: :controller do
 
       context 'with invalid tag' do
         let(:body) { 'body --yolo--' }
-
+        it { expect(flash.alert).to eq("Le modèle de l’attestation contient des erreurs et n'a pas pu être enregistré, veuillez les corriger.") }
-        it { expect(flash.alert).to eq(['Le contenu du modèle de l’attestation réfère au champ "yolo" qui n’existe pas']) }
       end
 
       context 'with removed champ' do
+        render_views
         let(:body) { "body --#{removed_type_de_champ.libelle}--" }
-
+        it { expect(response.body).to have_content("Le contenu de l’attestation contient la balise \"#{removed_type_de_champ.libelle}\" qui a été supprimée mais la suppression n’est pas encore publiée.") }
-        it { expect(flash.alert).to eq(["Le contenu du modèle de l’attestation réfère au champ \"#{removed_type_de_champ.libelle}\" qui a été supprimé mais la suppression n’est pas encore publiée"]) }
       end
 
       context 'with removed and published' do
+        render_views
         let(:body) { "body --#{removed_and_published_type_de_champ.libelle}--" }
-
+        it { expect(response.body).to have_content("Le contenu de l’attestation contient la balise \"#{removed_and_published_type_de_champ.libelle}\" qui a été supprimée.") }
-        it { expect(flash.alert).to eq(["Le contenu du modèle de l’attestation réfère au champ \"#{removed_and_published_type_de_champ.libelle}\" qui a été supprimé"]) }
       end
 
       context 'with new champ missing on dossier submitted on previous revision' do
+        render_views
         let(:dossier) { create(:dossier, :en_construction, procedure: procedure, revision: procedure.revisions.first) }
         let(:body) { "body --#{new_type_de_champ.libelle}--" }
-
+        it { expect(response.body).to have_content("Le contenu de l’attestation contient la balise \"#{new_type_de_champ.libelle}\" qui n’existe pas sur un des dossiers en cours de traitement") }
-        it { expect(flash.alert).to eq(["Le contenu du modèle de l’attestation réfère au champ \"#{new_type_de_champ.libelle}\" qui n’existe pas sur un des dossiers en cours de traitement"]) }
       end
 
       context 'with champ on draft' do
+        render_views
         let(:body) { "body --#{draft_type_de_champ.libelle}--" }
-
+        it { expect(response.body).to have_content("Le contenu de l’attestation contient la balise \"#{draft_type_de_champ.libelle}\" qui n’est pas encore publiée") }
-        it { expect(flash.alert).to eq(["Le contenu du modèle de l’attestation réfère au champ \"#{draft_type_de_champ.libelle}\" qui n’est pas encore publié"]) }
       end
     end
   end

spec/controllers/administrateurs/mail_templates_controller_spec.rb

@@ -73,7 +73,7 @@ describe Administrateurs::MailTemplatesController, type: :controller do
 
       it do
         expect(subject.body).not_to eq(mail_body)
-        expect(response.body).to match("Le contenu de l’email de notification de passage du dossier en instruction réfère au champ \"numéro\" qui n’existe pas")
+        expect(response.body).to match("Le corps de l’email contient la balise "numéro" qui n’existe pas, veuillez la supprimer.")
       end
     end
   end

spec/models/mail_template_spec.rb

@@ -38,7 +38,7 @@ describe Mails::InitiatedMail, type: :model do
     context 'template with invalid tag' do
       let(:email_body) { 'foo --numéro du -- bar' }
 
-      it { expect(subject.errors.full_messages).to eq(["Le contenu de l’email de notification de passage du dossier en instruction réfère au champ \"numéro du\" qui n’existe pas"]) }
+      it { expect(subject.errors.full_messages).to eq(["Le corps de l’email contient la balise \"numéro du\" qui n’existe pas, veuillez la supprimer."]) }
     end
 
     context 'template with unpublished tag' do
@@ -48,7 +48,7 @@ describe Mails::InitiatedMail, type: :model do
         procedure.draft_revision.add_type_de_champ(type_champ: :integer_number, libelle: 'age')
       end
 
-      it { expect(subject.errors.full_messages).to eq(["Le contenu de l’email de notification de passage du dossier en instruction réfère au champ \"age\" qui n’est pas encore publié"]) }
+      it { expect(subject.errors.full_messages).to eq(["Le corps de l’email contient la balise \"age\" qui n’est pas encore publié."]) }
     end
 
     context 'template with removed but unpublished tag' do
@@ -58,7 +58,7 @@ describe Mails::InitiatedMail, type: :model do
         procedure.draft_revision.remove_type_de_champ(type_de_champ.stable_id)
       end
 
-      it { expect(subject.errors.full_messages).to eq(["Le contenu de l’email de notification de passage du dossier en instruction réfère au champ \"nom\" qui a été supprimé mais la suppression n’est pas encore publiée"]) }
+      it { expect(subject.errors.full_messages).to eq(["Le corps de l’email contient la balise \"nom\" qui a été supprimé mais la suppression n’est pas encore publiée."]) }
     end
 
     context 'template with removed tag' do
@@ -69,7 +69,7 @@ describe Mails::InitiatedMail, type: :model do
         procedure.publish_revision!
       end
 
-      it { expect(subject.errors.full_messages).to eq(["Le contenu de l’email de notification de passage du dossier en instruction réfère au champ \"nom\" qui a été supprimé"]) }
+      it { expect(subject.errors.full_messages).to eq(["Le corps de l’email contient la balise \"nom\" qui a été supprimé."]) }
     end
 
     context 'template with new tag and old dossier' do
@@ -81,7 +81,7 @@ describe Mails::InitiatedMail, type: :model do
         procedure.publish_revision!
       end
 
-      it { expect(subject.errors.full_messages).to eq(["Le contenu de l’email de notification de passage du dossier en instruction réfère au champ \"age\" qui n’existe pas sur un des dossiers en cours de traitement"]) }
+      it { expect(subject.errors.full_messages).to eq(["Le corps de l’email contient la balise \"age\" qui n’existe pas sur un des dossiers en cours de traitement."]) }
     end
   end
 end

spec/system/administrateurs/procedure_attestation_template_spec.rb

@@ -29,11 +29,12 @@ describe 'As an administrateur, I want to manage the procedure’s attestation',
 
       # now process to enable attestation
       find_attestation_card.click
-      fill_in "Titre de l'attestation", with: 'BOOM'
+      fill_in "Titre de l’attestation", with: 'BOOM'
-      fill_in "Corps du document", with: 'BOOM'
+      fill_in "Contenu de l'attestation", with: 'BOOM'
       find('.toggle-switch-control').click
       click_on 'Enregistrer'
-      page.find(".alert-success", text: "Le model de l’attestation a bien été enregistrée")
+
+      page.find(".alert-success", text: "Le modèle de l’attestation a bien été enregistré")
 
       # check attestation
       visit admin_procedure_path(procedure)
@@ -49,7 +50,7 @@ describe 'As an administrateur, I want to manage the procedure’s attestation',
       find_attestation_card.click
       find('.toggle-switch-control').click
       click_on 'Enregistrer'
-      page.find(".alert-success", text: "Le model de l’attestation a bien été modifiée")
+      page.find(".alert-success", text: "Le modèle de l’attestation a bien été modifié")
 
       # check attestation is now disabled
       visit admin_procedure_path(procedure)