Merge branch 'harmonize-actions-for-instructeurs' of github.com:demarches-simplifiees/demarches-simplifiees.fr into harmonize-actions-for-instructeurs

app/assets/stylesheets/dsfr.scss

@@ -56,6 +56,7 @@ fieldset {
   }
 }
 
+// dans le DSFR il est possible d'avoir un bouton seulement avec une icone mais j'ai du surcharger ici pour eviter d'avoir des marges de l'icone. Je n'ai pas bien compris pourquoi
 .fr-btns-group--sm.fr-btns-group--icon-right .fr-btn[class*=" fr-icon-"].icon-only::after {
   margin-left: 0;
   margin-right: 0;

app/views/instructeurs/procedures/_dossier_actions.html.haml

@@ -5,6 +5,9 @@
 - elsif close_to_expiration || Dossier::TERMINE.include?(state)
   %li{ 'data-turbo': 'true' }
     - if close_to_expiration
+      = link_to repasser_en_instruction_instructeur_dossier_path(procedure_id, dossier_id), { class: 'fr-btn fr-btn--secondary fr-icon-edit-line' }.deep_merge!(post_method) do
+        Repasser en instruction
+
       = link_to(repousser_expiration_instructeur_dossier_path(procedure_id, dossier_id), method: :post, class: "fr-btn") do
         = t('instructeurs.dossiers.header.banner.button_delay_expiration')
 

spec/views/instructeur/dossiers/show.html.haml_spec.rb

@@ -108,7 +108,8 @@ describe 'instructeurs/dossiers/show.html.haml', type: :view do
 
     it 'displays the correct actions' do
       expect(subject).to have_text('Conserver un mois de plus')
-      expect(subject).to have_selector('.header-actions ul:first-child .fr-btn', count: 1)
+      expect(subject).to have_link('Repasser en instruction', href: repasser_en_instruction_instructeur_dossier_path(dossier.procedure, dossier))
+      expect(subject).to have_selector('.header-actions ul:first-child .fr-btn', count: 2)
     end
   end