From 782706537de9d416bbb268227d1ab56fabfd292d Mon Sep 17 00:00:00 2001
From: Judith <jpoulvel@student.42.fr>
Date: Tue, 7 Jul 2020 12:03:32 +0200
Subject: [PATCH] Changed the routes to pass all new design admin pages under
 'new_administrateur' scope, except 'services'. As a result, the url of these
 pages will all include '/admin'.

---
 .../admin/instructeurs_controller.rb          |  2 +-
 .../groupe_instructeurs_controller.rb         | 14 ++++----
 .../procedures_controller.rb                  |  2 +-
 app/helpers/procedure_helper.rb               |  4 +--
 app/views/admin/mail_templates/edit.html.haml |  2 +-
 app/views/admin/procedures/show.html.haml     |  4 +--
 .../add_instructeurs.html.haml                |  2 +-
 .../remove_instructeur.html.haml              |  2 +-
 .../send_notifications.html.haml              |  2 +-
 .../groupe_instructeurs/index.html.haml       |  2 +-
 .../groupe_instructeurs/show.html.haml        |  2 +-
 .../procedures/deleted_dossiers.html.haml     |  2 +-
 .../instructeurs/procedures/show.html.haml    |  2 +-
 ...dmin_procedurescontroller_navbar.html.haml | 16 ++++-----
 app/views/manager/procedures/show.html.erb    |  2 +-
 .../new_administrateur/_breadcrumbs.html.haml |  2 +-
 .../new_administrateur/_groups_header.haml    |  2 +-
 .../groupe_instructeurs/index.html.haml       |  6 ++--
 .../reaffecter_dossiers.html.haml             |  4 +--
 .../groupe_instructeurs/show.html.haml        |  2 +-
 .../_administrateur.html.haml                 |  2 +-
 .../procedures/apercu.html.haml               |  4 +--
 config/routes.rb                              | 18 +++++-----
 .../admin/instructeurs_controller_spec.rb     |  2 +-
 .../groupe_instructeurs_controller_spec.rb    | 36 +++++++++----------
 .../procedures_controller_spec.rb             |  2 +-
 .../features/admin/procedure_creation_spec.rb |  6 ++--
 .../new_administrateur/types_de_champ_spec.rb |  2 +-
 spec/features/routing/full_scenario_spec.rb   |  4 +--
 .../send_notifications.html.haml_spec.rb      |  6 ++--
 30 files changed, 79 insertions(+), 79 deletions(-)

diff --git a/app/controllers/admin/instructeurs_controller.rb b/app/controllers/admin/instructeurs_controller.rb
index a3d532234..1f549756c 100644
--- a/app/controllers/admin/instructeurs_controller.rb
+++ b/app/controllers/admin/instructeurs_controller.rb
@@ -22,7 +22,7 @@ class Admin::InstructeursController < AdminController
     end
 
     if procedure_id.present?
-      redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur)
+      redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur)
     else
       redirect_to admin_instructeurs_path
     end
diff --git a/app/controllers/new_administrateur/groupe_instructeurs_controller.rb b/app/controllers/new_administrateur/groupe_instructeurs_controller.rb
index 1e387f621..e90523e76 100644
--- a/app/controllers/new_administrateur/groupe_instructeurs_controller.rb
+++ b/app/controllers/new_administrateur/groupe_instructeurs_controller.rb
@@ -21,7 +21,7 @@ module NewAdministrateur
         .new(label: label, instructeurs: [current_administrateur.instructeur])
 
       if @groupe_instructeur.save
-        redirect_to procedure_groupe_instructeur_path(procedure, @groupe_instructeur),
+        redirect_to admin_procedure_groupe_instructeur_path(procedure, @groupe_instructeur),
           notice: "Le groupe d’instructeurs « #{label} » a été créé."
       else
         @procedure = procedure
@@ -36,7 +36,7 @@ module NewAdministrateur
       @groupe_instructeur = groupe_instructeur
 
       if @groupe_instructeur.update(label: label)
-        redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur),
+        redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur),
           notice: "Le nom est à présent « #{label} »."
       else
         @procedure = procedure
@@ -58,7 +58,7 @@ module NewAdministrateur
         groupe_instructeur.destroy!
         flash[:notice] = "le groupe « #{label} » a été supprimé."
       end
-      redirect_to procedure_groupe_instructeurs_path(procedure)
+      redirect_to admin_procedure_groupe_instructeurs_path(procedure)
     end
 
     def reaffecter_dossiers
@@ -76,7 +76,7 @@ module NewAdministrateur
       end
 
       flash[:notice] = "Les dossiers du groupe « #{groupe_instructeur.label} » ont été réaffectés au groupe « #{target_group.label} »."
-      redirect_to procedure_groupe_instructeurs_path(procedure)
+      redirect_to admin_procedure_groupe_instructeurs_path(procedure)
     end
 
     def add_instructeur
@@ -123,7 +123,7 @@ module NewAdministrateur
         end
       end
 
-      redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur)
+      redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur)
     end
 
     def remove_instructeur
@@ -148,13 +148,13 @@ module NewAdministrateur
           end
         end
       end
-      redirect_to procedure_groupe_instructeur_path(procedure, groupe_instructeur)
+      redirect_to admin_procedure_groupe_instructeur_path(procedure, groupe_instructeur)
     end
 
     def update_routing_criteria_name
       procedure.update!(routing_criteria_name: routing_criteria_name)
 
-      redirect_to procedure_groupe_instructeurs_path(procedure),
+      redirect_to admin_procedure_groupe_instructeurs_path(procedure),
         notice: "Le libellé est maintenant « #{procedure.routing_criteria_name} »."
     end
 
diff --git a/app/controllers/new_administrateur/procedures_controller.rb b/app/controllers/new_administrateur/procedures_controller.rb
index a5d266ced..a27895847 100644
--- a/app/controllers/new_administrateur/procedures_controller.rb
+++ b/app/controllers/new_administrateur/procedures_controller.rb
@@ -25,7 +25,7 @@ module NewAdministrateur
         flash.notice = 'Démarche enregistrée.'
         current_administrateur.instructeur.assign_to_procedure(@procedure)
 
-        redirect_to champs_procedure_path(@procedure)
+        redirect_to champs_admin_procedure_path(@procedure)
       end
     end
 
diff --git a/app/helpers/procedure_helper.rb b/app/helpers/procedure_helper.rb
index 922479ec5..8d8362bc8 100644
--- a/app/helpers/procedure_helper.rb
+++ b/app/helpers/procedure_helper.rb
@@ -23,7 +23,7 @@ module ProcedureHelper
       isAnnotation: false,
       typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user),
       typeDeChamps: procedure.types_de_champ.as_json_for_editor,
-      baseUrl: procedure_types_de_champ_path(procedure),
+      baseUrl: admin_procedure_types_de_champ_path(procedure),
       directUploadUrl: rails_direct_uploads_url
     }
   end
@@ -33,7 +33,7 @@ module ProcedureHelper
       isAnnotation: true,
       typeDeChampsTypes: TypeDeChamp.type_de_champ_types_for(procedure, current_user),
       typeDeChamps: procedure.types_de_champ_private.as_json_for_editor,
-      baseUrl: procedure_types_de_champ_path(procedure),
+      baseUrl: admin_procedure_types_de_champ_path(procedure),
       directUploadUrl: rails_direct_uploads_url
     }
   end
diff --git a/app/views/admin/mail_templates/edit.html.haml b/app/views/admin/mail_templates/edit.html.haml
index e515cf7f3..a69a6a0f9 100644
--- a/app/views/admin/mail_templates/edit.html.haml
+++ b/app/views/admin/mail_templates/edit.html.haml
@@ -21,7 +21,7 @@
     .text-right
       = link_to "Annuler", admin_procedure_mail_templates_path(@procedure), class: "btn btn-default"
       = f.submit 'Mettre à jour', class: "btn btn-default btn-success"
-      = link_to "Prévisualiser", preview_procedure_mail_template_path(@procedure, @mail_template.class.const_get(:SLUG)), class: "btn btn-primary", target: "_blank"
+      = link_to "Prévisualiser", preview_admin_procedure_mail_template_path(@procedure, @mail_template.class.const_get(:SLUG)), class: "btn btn-primary", target: "_blank"
 
   .row
     .col-md-12
diff --git a/app/views/admin/procedures/show.html.haml b/app/views/admin/procedures/show.html.haml
index 52fbe51b9..a3d22ac03 100644
--- a/app/views/admin/procedures/show.html.haml
+++ b/app/views/admin/procedures/show.html.haml
@@ -107,7 +107,7 @@
             .alert.alert-info
               Pour pouvoir tester cette démarche, vous devez d’abord lui affecter
               - if @procedure.missing_instructeurs?
-                = link_to("des instructeurs", procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur))
+                = link_to("des instructeurs", admin_procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur))
               - if @procedure.missing_instructeurs? && @procedure.service.nil?
                 et
               - if @procedure.service.nil?
@@ -122,7 +122,7 @@
           - if @procedure.missing_steps.include?(:instructeurs)
             %p.alert.alert-danger
               Vous devez affecter des instructeurs avant de pouvoir publier votre démarche.
-              = link_to 'Cliquez ici.', procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)
+              = link_to 'Cliquez ici.', admin_procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)
 
           %p.alert.alert-info
             Cette démarche n’a pas encore de lien, et n’est pas accessible par le public.
diff --git a/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml b/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml
index b04110592..b6f0dabff 100644
--- a/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml
+++ b/app/views/groupe_instructeur_mailer/add_instructeurs.html.haml
@@ -6,6 +6,6 @@
 
 %p
   Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe :
-  = link_to(@group.label, procedure_groupe_instructeur_url(@group.procedure, @group))
+  = link_to(@group.label, admin_procedure_groupe_instructeur_url(@group.procedure, @group))
 
 = render partial: "layouts/mailers/signature"
diff --git a/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml b/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml
index 15a3f15a7..6a8e1cc94 100644
--- a/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml
+++ b/app/views/groupe_instructeur_mailer/remove_instructeur.html.haml
@@ -6,6 +6,6 @@
 
 %p
   Cliquez sur le lien ci-dessous pour voir la liste des instructeurs de ce groupe :
-  = link_to(@group.label, procedure_groupe_instructeur_url(@group.procedure, @group))
+  = link_to(@group.label, admin_procedure_groupe_instructeur_url(@group.procedure, @group))
 
 = render partial: "layouts/mailers/signature"
diff --git a/app/views/instructeur_mailer/send_notifications.html.haml b/app/views/instructeur_mailer/send_notifications.html.haml
index 04a44c9fe..7cd1a1dcd 100644
--- a/app/views/instructeur_mailer/send_notifications.html.haml
+++ b/app/views/instructeur_mailer/send_notifications.html.haml
@@ -9,7 +9,7 @@
 %ul
   - @data.each do |datum|
     %li
-      = link_to(datum[:procedure_libelle], procedure_url(datum[:procedure_id]))
+      = link_to(datum[:procedure_libelle], instructeur_procedure_url(datum[:procedure_id]))
       - if datum[:nb_en_construction] > 0
         %br
         #{datum[:nb_en_construction]} #{'dossier'.pluralize(datum[:nb_en_construction])} en construction
diff --git a/app/views/instructeurs/groupe_instructeurs/index.html.haml b/app/views/instructeurs/groupe_instructeurs/index.html.haml
index 97e62c876..009e150fb 100644
--- a/app/views/instructeurs/groupe_instructeurs/index.html.haml
+++ b/app/views/instructeurs/groupe_instructeurs/index.html.haml
@@ -1,7 +1,7 @@
 - content_for(:title, "Notifications pour #{@procedure.libelle}")
 
 = render partial: 'new_administrateur/breadcrumbs',
-  locals: { steps: [link_to(@procedure.libelle, procedure_path(@procedure)),
+  locals: { steps: [link_to(@procedure.libelle, instructeur_procedure_path(@procedure)),
                     'Groupes d’instructeurs'] }
 
 .container.groupe-instructeur
diff --git a/app/views/instructeurs/groupe_instructeurs/show.html.haml b/app/views/instructeurs/groupe_instructeurs/show.html.haml
index 1fcc742ef..f4faaef80 100644
--- a/app/views/instructeurs/groupe_instructeurs/show.html.haml
+++ b/app/views/instructeurs/groupe_instructeurs/show.html.haml
@@ -1,7 +1,7 @@
 - content_for(:title, "Instructeurs du groupe #{@groupe_instructeur.label}")
 
 = render partial: 'new_administrateur/breadcrumbs',
-  locals: { steps: [link_to(@procedure.libelle, procedure_path(@procedure)),
+  locals: { steps: [link_to(@procedure.libelle, instructeur_procedure_path(@procedure)),
                     link_to('Groupes d’instructeurs', instructeur_groupes_path(@procedure)),
                     @groupe_instructeur.label] }
 
diff --git a/app/views/instructeurs/procedures/deleted_dossiers.html.haml b/app/views/instructeurs/procedures/deleted_dossiers.html.haml
index 9063e18e2..65a2413a6 100644
--- a/app/views/instructeurs/procedures/deleted_dossiers.html.haml
+++ b/app/views/instructeurs/procedures/deleted_dossiers.html.haml
@@ -16,7 +16,7 @@
         - if @procedure.routee?
           |
           - if current_administrateur.present? && current_administrateur.owns?(@procedure)
-            = link_to 'instructeurs', procedure_groupe_instructeurs_path(@procedure), class: 'header-link'
+            = link_to 'instructeurs', admin_procedure_groupe_instructeurs_path(@procedure), class: 'header-link'
           - else
             = link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link'
 
diff --git a/app/views/instructeurs/procedures/show.html.haml b/app/views/instructeurs/procedures/show.html.haml
index 2658d6a9b..948338ee2 100644
--- a/app/views/instructeurs/procedures/show.html.haml
+++ b/app/views/instructeurs/procedures/show.html.haml
@@ -16,7 +16,7 @@
         - if @procedure.routee?
           |
           - if current_administrateur.present? && current_administrateur.owns?(@procedure)
-            = link_to 'instructeurs', procedure_groupe_instructeurs_path(@procedure), class: 'header-link'
+            = link_to 'instructeurs', admin_procedure_groupe_instructeurs_path(@procedure), class: 'header-link'
           - else
             = link_to 'instructeurs', instructeur_groupes_path(@procedure), class: 'header-link'
 
diff --git a/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml b/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml
index 6ef6c05ea..a262889dc 100644
--- a/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml
+++ b/app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml
@@ -26,33 +26,33 @@
         - if @procedure.missing_steps.include?(:service)
           %p.missing-steps (à compléter)
 
-    %a#onglet-administrateurs{ href: url_for(procedure_administrateurs_path(@procedure)) }
+    %a#onglet-administrateurs{ href: url_for(admin_procedure_administrateurs_path(@procedure)) }
       .procedure-list-element
         Administrateurs
 
     - if !feature_enabled?(:administrateur_routage)
-      %a#onglet-instructeurs{ href: url_for(procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)) }
+      %a#onglet-instructeurs{ href: url_for(admin_procedure_groupe_instructeur_path(@procedure, @procedure.defaut_groupe_instructeur)) }
         .procedure-list-element{ class: ('active' if active == 'Instructeurs') }
           Instructeurs
           - if @procedure.missing_steps.include?(:instructeurs)
             %p.missing-steps (à compléter)
 
     - if feature_enabled?(:administrateur_routage)
-      %a#onglet-instructeurs{ href: url_for(procedure_groupe_instructeurs_path(@procedure)) }
+      %a#onglet-instructeurs{ href: url_for(admin_procedure_groupe_instructeurs_path(@procedure)) }
         .procedure-list-element
           Groupe d'instructeurs
 
     - if !@procedure.locked?
-      %a#onglet-champs{ href: champs_procedure_path(@procedure) }
+      %a#onglet-champs{ href: champs_admin_procedure_path(@procedure) }
         .procedure-list-element{ class: ('active' if active == 'Champs') }
           Champs
 
     - if !@procedure.locked?
-      %a#onglet-private-champs{ href: annotations_procedure_path(@procedure) }
+      %a#onglet-private-champs{ href: annotations_admin_procedure_path(@procedure) }
         .procedure-list-element{ class: ('active' if active == 'Annotations privées') }
           Annotations privées
 
-    %a#onglet-preview{ href: url_for(apercu_procedure_path(@procedure)), target: "_blank", rel: "noopener" }
+    %a#onglet-preview{ href: url_for(apercu_admin_procedure_path(@procedure)), target: "_blank", rel: "noopener" }
       .procedure-list-element{ class: ('active' if active == 'Prévisualisation') }
         Prévisualisation
 
@@ -64,11 +64,11 @@
       .procedure-list-element{ class: ('active' if active == 'Attestation') }
         Attestation
 
-    %a#onglet-description{ href: url_for(admin_procedure_monavis_path(@procedure)) }
+    %a#onglet-description{ href: url_for(monavis_admin_procedure_path(@procedure)) }
       .procedure-list-element{ class: ('active' if active == 'MonAvis') }
         MonAvis
 
-    %a#onglet-description{ href: url_for(admin_procedure_jeton_path(@procedure)) }
+    %a#onglet-description{ href: url_for(jeton_admin_procedure_path(@procedure)) }
       .procedure-list-element
         Jeton
 
diff --git a/app/views/manager/procedures/show.html.erb b/app/views/manager/procedures/show.html.erb
index cae25c6be..da0724ee8 100644
--- a/app/views/manager/procedures/show.html.erb
+++ b/app/views/manager/procedures/show.html.erb
@@ -36,7 +36,7 @@ as well as a link to its edit page.
   </div>
 
   <div>
-    <%= link_to 'aperçu', apercu_procedure_path(procedure), class: 'button' %>
+    <%= link_to 'aperçu', apercu_admin_procedure_path(procedure), class: 'button' %>
 
     <% if !procedure.whitelisted? %>
       <%= link_to 'whitelister', whitelist_manager_procedure_path(procedure), method: :post, class: 'button' %>
diff --git a/app/views/new_administrateur/_breadcrumbs.html.haml b/app/views/new_administrateur/_breadcrumbs.html.haml
index eaff46f36..b8dc5d94c 100644
--- a/app/views/new_administrateur/_breadcrumbs.html.haml
+++ b/app/views/new_administrateur/_breadcrumbs.html.haml
@@ -4,5 +4,5 @@
       - steps.each do |step|
         %li= step
     - if defined?(preview) && preview
-      = link_to "Prévisualiser le formulaire", apercu_procedure_path(@procedure), target: "_blank", rel: "noopener", class: 'button'
+      = link_to "Prévisualiser le formulaire", apercu_admin_procedure_path(@procedure), target: "_blank", rel: "noopener", class: 'button'
       = link_to "Continuer >", admin_procedure_path(@procedure), title: 'Vous pourrez revenir ici par la suite', class: 'button accepted'
diff --git a/app/views/new_administrateur/_groups_header.haml b/app/views/new_administrateur/_groups_header.haml
index e300d6596..547a8d9ed 100644
--- a/app/views/new_administrateur/_groups_header.haml
+++ b/app/views/new_administrateur/_groups_header.haml
@@ -1,7 +1,7 @@
 %h1 Groupe « #{@groupe_instructeur.label} »
 .card.mt-2
   = form_for @groupe_instructeur,
-    url: procedure_groupe_instructeur_path(@procedure, @groupe_instructeur),
+    url: admin_procedure_groupe_instructeur_path(@procedure, @groupe_instructeur),
     html: { class: 'form' } do |f|
 
     = f.label :label, 'Nom du groupe'
diff --git a/app/views/new_administrateur/groupe_instructeurs/index.html.haml b/app/views/new_administrateur/groupe_instructeurs/index.html.haml
index 0e7923121..ed198b774 100644
--- a/app/views/new_administrateur/groupe_instructeurs/index.html.haml
+++ b/app/views/new_administrateur/groupe_instructeurs/index.html.haml
@@ -33,16 +33,16 @@
         - @groupes_instructeurs.each do |group|
           %tr
             %td= group.label
-            %td.actions= link_to "voir", procedure_groupe_instructeur_path(@procedure, group)
+            %td.actions= link_to "voir", admin_procedure_groupe_instructeur_path(@procedure, group)
             - if @groupes_instructeurs.many?
               - if group.dossiers.empty?
                 %td.actions
-                  = link_to procedure_groupe_instructeur_path(@procedure, group), { method: :delete, class: 'button', data: { confirm: "Êtes-vous sûr de vouloir supprimer le groupe « #{group.label} » ?" }} do
+                  = link_to admin_procedure_groupe_instructeur_path(@procedure, group), { method: :delete, class: 'button', data: { confirm: "Êtes-vous sûr de vouloir supprimer le groupe « #{group.label} » ?" }} do
                     %span.icon.delete
                     supprimer ce groupe
               - else
                 %td.actions
-                  = link_to reaffecter_dossiers_procedure_groupe_instructeur_path(@procedure, group), class: 'button', title:'Réaffecter les dossiers à un autre groupe afin de pouvoir le supprimer' do
+                  = link_to reaffecter_dossiers_admin_procedure_groupe_instructeur_path(@procedure, group), class: 'button', title:'Réaffecter les dossiers à un autre groupe afin de pouvoir le supprimer' do
                     %span.icon.follow
                     déplacer les dossiers
     = paginate @groupes_instructeurs
diff --git a/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml b/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml
index 8e7866162..eae251882 100644
--- a/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml
+++ b/app/views/new_administrateur/groupe_instructeurs/reaffecter_dossiers.html.haml
@@ -1,7 +1,7 @@
 = render partial: 'new_administrateur/breadcrumbs',
   locals: { steps: [link_to('Démarches', admin_procedures_path),
                     link_to(@procedure.libelle, admin_procedure_path(@procedure)),
-                    link_to('Groupes d’instructeurs', procedure_groupe_instructeurs_path(@procedure)),
+                    link_to('Groupes d’instructeurs', admin_procedure_groupe_instructeurs_path(@procedure)),
                     @groupe_instructeur.label] }
 
 .container.groupe-instructeur
@@ -20,7 +20,7 @@
           %tr
             %td= group.label
             %td.actions= button_to 'Réaffecter les dossiers à ce groupe',
-              reaffecter_procedure_groupe_instructeur_path(:target_group => group),
+              reaffecter_admin_procedure_groupe_instructeur_path(:target_group => group),
               { class: 'button',
                 data: { confirm: "Êtes-vous sûr de vouloir réaffecter les dossiers du groupe « #{@groupe_instructeur.label} » vers le groupe  « #{group.label} » ?" } }
 
diff --git a/app/views/new_administrateur/groupe_instructeurs/show.html.haml b/app/views/new_administrateur/groupe_instructeurs/show.html.haml
index f6a8af25d..c026fa1c9 100644
--- a/app/views/new_administrateur/groupe_instructeurs/show.html.haml
+++ b/app/views/new_administrateur/groupe_instructeurs/show.html.haml
@@ -3,7 +3,7 @@
   = render partial: 'new_administrateur/breadcrumbs',
     locals: { steps: [link_to('Démarches', admin_procedures_path),
                       link_to(@procedure.libelle, admin_procedure_path(@procedure)),
-                      link_to('Groupes d’instructeurs', procedure_groupe_instructeurs_path(@procedure)),
+                      link_to('Groupes d’instructeurs', admin_procedure_groupe_instructeurs_path(@procedure)),
                       @groupe_instructeur.label] }
 - else
   = render partial: 'new_administrateur/breadcrumbs',
diff --git a/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml b/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml
index 9277ae0ad..ca8fb3310 100644
--- a/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml
+++ b/app/views/new_administrateur/procedure_administrateurs/_administrateur.html.haml
@@ -7,7 +7,7 @@
       C’est vous !
     - else
       = link_to 'Retirer',
-        [@procedure, administrateur],
+        admin_procedure_administrateur_path(@procedure, administrateur),
         method: :delete,
         'data-confirm': "Retirer « #{administrateur.email} » des administrateurs de « #{@procedure.libelle} » ?",
         remote: true
diff --git a/app/views/new_administrateur/procedures/apercu.html.haml b/app/views/new_administrateur/procedures/apercu.html.haml
index 8b09de4e9..857ea4462 100644
--- a/app/views/new_administrateur/procedures/apercu.html.haml
+++ b/app/views/new_administrateur/procedures/apercu.html.haml
@@ -4,11 +4,11 @@
 
     %ul.tabs
       = tab_item('le dossier',
-        apercu_procedure_path(@dossier.procedure, tab: 'dossier'),
+        apercu_admin_procedure_path(@dossier.procedure, tab: 'dossier'),
         active: @tab == 'dossier')
       - if @dossier.champs_private.size > 0
         = tab_item('les annotations privées',
-          apercu_procedure_path(@dossier.procedure, tab: 'annotations-privees'),
+          apercu_admin_procedure_path(@dossier.procedure, tab: 'annotations-privees'),
           active: @tab == 'annotations-privees')
 
 - if @tab == 'dossier'
diff --git a/config/routes.rb b/config/routes.rb
index 67801ac80..cb4ce92f1 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -170,12 +170,6 @@ Rails.application.routes.draw do
 
   # order matters: we don't want those routes to match /admin/procedures/:id
   get 'admin/procedures/new' => 'new_administrateur/procedures#new', as: :new_admin_procedure
-  get 'admin/procedures/:id/edit' => 'new_administrateur/procedures#edit', as: :edit_admin_procedure
-  post 'admin/procedures' => 'new_administrateur/procedures#create'
-  get 'admin/procedures/:id/monavis' => 'new_administrateur/procedures#monavis', as: :admin_procedure_monavis
-  patch 'admin/procedures/:id/monavis' => 'new_administrateur/procedures#update_monavis', as: :update_monavis
-  get 'admin/procedures/:id/jeton' => 'new_administrateur/procedures#jeton', as: :admin_procedure_jeton
-  patch 'admin/procedures/:id/jeton' => 'new_administrateur/procedures#update_jeton', as: :update_jeton
 
   namespace :admin do
     get 'activate' => '/administrateurs/activate#new'
@@ -183,7 +177,7 @@ Rails.application.routes.draw do
     get 'procedures/archived' => 'procedures#archived'
     get 'procedures/draft' => 'procedures#draft'
 
-    resources :procedures, except: [:new, :edit, :update] do
+    resources :procedures, only: [:index, :show, :destroy] do
       collection do
         get 'new_from_existing' => 'procedures#new_from_existing', as: :new_from_existing
       end
@@ -375,12 +369,16 @@ Rails.application.routes.draw do
   # Administrateur
   #
 
-  scope module: 'new_administrateur' do
-    resources :procedures, only: [:update, :new] do
+  namespace :admin, module: 'new_administrateur' do
+    resources :procedures, except: [:index, :show, :destroy] do
       member do
         get 'apercu'
         get 'champs'
         get 'annotations'
+        get 'monavis'
+        patch 'update_monavis'
+        get 'jeton'
+        patch 'update_jeton'
       end
 
       resources :groupe_instructeurs, only: [:index, :show, :create, :update, :destroy] do
@@ -408,7 +406,9 @@ Rails.application.routes.draw do
         get 'preview', on: :member
       end
     end
+  end
 
+  scope module: 'new_administrateur' do
     resources :services, except: [:show] do
       collection do
         patch 'add_to_procedure'
diff --git a/spec/controllers/admin/instructeurs_controller_spec.rb b/spec/controllers/admin/instructeurs_controller_spec.rb
index ccfe14063..3bf695cdf 100644
--- a/spec/controllers/admin/instructeurs_controller_spec.rb
+++ b/spec/controllers/admin/instructeurs_controller_spec.rb
@@ -43,7 +43,7 @@ describe Admin::InstructeursController, type: :controller do
         let(:procedure) { create :procedure }
         let(:procedure_id) { procedure.id }
         it { expect(response.status).to eq(302) }
-        it { expect(response).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
+        it { expect(response).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
       end
 
       describe 'Instructeur attributs in database' do
diff --git a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
index 9e7c1b0e7..a4bf90d06 100644
--- a/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
+++ b/spec/controllers/new_administrateur/groupe_instructeurs_controller_spec.rb
@@ -62,7 +62,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       let(:label) { "nouveau_groupe" }
 
       it { expect(flash.notice).to be_present }
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, procedure.groupe_instructeurs.last)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, procedure.groupe_instructeurs.last)) }
       it { expect(procedure.groupe_instructeurs.count).to eq(2) }
     end
 
@@ -88,7 +88,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       before { delete_group gi_1_1 }
 
       it { expect(flash.alert).to be_present }
-      it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
       it { expect(procedure.groupe_instructeurs.count).to eq(1) }
     end
 
@@ -99,7 +99,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
         before { delete_group gi_1_2 }
         it { expect(flash.notice).to be_present }
         it { expect(procedure.groupe_instructeurs.count).to eq(1) }
-        it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) }
+        it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
       end
 
       context 'of a group with dossiers, that cannot be deleted' do
@@ -108,7 +108,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
 
         it { expect(flash.alert).to be_present }
         it { expect(procedure.groupe_instructeurs.count).to eq(2) }
-        it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) }
+        it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
       end
     end
   end
@@ -125,7 +125,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
         }
     end
     def reaffecter_url(group)
-      reaffecter_procedure_groupe_instructeur_path(:id => gi_1_2,
+      reaffecter_admin_procedure_groupe_instructeur_path(:id => gi_1_2,
                                                     :target_group => group)
     end
 
@@ -155,7 +155,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
         dossier12.reload
       end
 
-      it { expect(response).to redirect_to(procedure_groupe_instructeurs_path(procedure)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
       it { expect(gi_1_1.dossiers.with_discarded.count).to be(0) }
       it { expect(gi_1_2.dossiers.with_discarded.count).to be(2) }
       it { expect(gi_1_2.dossiers.last.id).to be(dossier12.id) }
@@ -193,7 +193,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
     end
 
     it { expect(gi_1_1.reload.label).to eq(new_name) }
-    it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) }
+    it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_1)) }
     it { expect(flash.notice).to be_present }
 
     context 'when the name is already taken' do
@@ -215,7 +215,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       it { expect(response.status).to eq(200) }
       it { expect(subject.request.flash[:alert]).to be_nil }
       it { expect(subject.request.flash[:notice]).to be_present }
-      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
+      it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
     end
 
     context 'when there is at least one bad email' do
@@ -223,13 +223,13 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       it { expect(response.status).to eq(200) }
       it { expect(subject.request.flash[:alert]).to be_present }
       it { expect(subject.request.flash[:notice]).to be_present }
-      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
+      it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
     end
 
     context 'when the admin wants to assign an instructor who is already assigned on this procedure' do
       let(:emails) { ['instructeur_1@ministere_a.gouv.fr'] }
       it { expect(subject.request.flash[:alert]).to be_present }
-      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
+      it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, procedure.defaut_groupe_instructeur) }
     end
   end
 
@@ -256,7 +256,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
 
       it { expect(gi_1_2.instructeurs.pluck(:email)).to include(*new_instructeur_emails) }
       it { expect(flash.notice).to be_present }
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
       it { expect(procedure.routee?).to be_truthy }
       it "calls GroupeInstructeurMailer with the right groupe and instructeurs" do
         expect(GroupeInstructeurMailer).to have_received(:add_instructeurs).with(
@@ -270,20 +270,20 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
     context 'of an instructeur already in the group' do
       let(:new_instructeur_emails) { [instructeur.email] }
 
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
     end
 
     context 'of badly formed email' do
       let(:new_instructeur_emails) { ['badly_formed_email'] }
 
       it { expect(flash.alert).to be_present }
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
     end
 
     context 'of an empty string' do
       let(:new_instructeur_emails) { '' }
 
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_2)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_2)) }
     end
   end
 
@@ -306,7 +306,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
 
       it { expect(gi_1_1.instructeurs).to include(instructeur) }
       it { expect(gi_1_1.reload.instructeurs.count).to eq(1) }
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_1)) }
     end
 
     context 'when there is only one instructeur' do
@@ -318,7 +318,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       it { expect(gi_1_1.instructeurs).to include(instructeur) }
       it { expect(gi_1_1.instructeurs.count).to eq(1) }
       it { expect(flash.alert).to eq('Suppression impossible : il doit y avoir au moins un instructeur dans le groupe') }
-      it { expect(response).to redirect_to(procedure_groupe_instructeur_path(procedure, gi_1_1)) }
+      it { expect(response).to redirect_to(admin_procedure_groupe_instructeur_path(procedure, gi_1_1)) }
     end
   end
 
@@ -337,7 +337,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       it { expect(subject.request.flash[:notice]).to be_present }
       it { expect(subject.request.flash[:alert]).to be_nil }
       it { expect(response.status).to eq(302) }
-      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, gi_1_1) }
+      it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, gi_1_1) }
     end
 
     context 'when the instructor is not assigned to the procedure' do
@@ -345,7 +345,7 @@ describe NewAdministrateur::GroupeInstructeursController, type: :controller do
       it { expect(subject.request.flash[:alert]).to be_present }
       it { expect(subject.request.flash[:notice]).to be_nil }
       it { expect(response.status).to eq(302) }
-      it { expect(subject).to redirect_to procedure_groupe_instructeur_path(procedure, gi_1_1) }
+      it { expect(subject).to redirect_to admin_procedure_groupe_instructeur_path(procedure, gi_1_1) }
     end
   end
 
diff --git a/spec/controllers/new_administrateur/procedures_controller_spec.rb b/spec/controllers/new_administrateur/procedures_controller_spec.rb
index 19c9f30fb..05f8ccf37 100644
--- a/spec/controllers/new_administrateur/procedures_controller_spec.rb
+++ b/spec/controllers/new_administrateur/procedures_controller_spec.rb
@@ -102,7 +102,7 @@ describe NewAdministrateur::ProceduresController, type: :controller do
           it { expect(subject.duree_conservation_dossiers_hors_ds).to eq(duree_conservation_dossiers_hors_ds) }
         end
 
-        it { is_expected.to redirect_to(champs_procedure_path(Procedure.last)) }
+        it { is_expected.to redirect_to(champs_admin_procedure_path(Procedure.last)) }
         it { expect(flash[:notice]).to be_present }
       end
 
diff --git a/spec/features/admin/procedure_creation_spec.rb b/spec/features/admin/procedure_creation_spec.rb
index 2f33cf8ab..2850d48bf 100644
--- a/spec/features/admin/procedure_creation_spec.rb
+++ b/spec/features/admin/procedure_creation_spec.rb
@@ -45,7 +45,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
         fill_in_dummy_procedure_details
         click_on 'Créer la démarche'
 
-        expect(page).to have_current_path(champs_procedure_path(Procedure.last))
+        expect(page).to have_current_path(champs_admin_procedure_path(Procedure.last))
       end
     end
   end
@@ -66,7 +66,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
 
     scenario 'Add champ, add file, visualize them in procedure preview' do
       page.refresh
-      expect(page).to have_current_path(champs_procedure_path(Procedure.last))
+      expect(page).to have_current_path(champs_admin_procedure_path(Procedure.last))
 
       add_champ(remove_flash_message: true)
       fill_in 'champ-0-libelle', with: 'libelle de champ'
@@ -80,7 +80,7 @@ feature 'As an administrateur I wanna create a new procedure', js: true do
 
       preview_window = window_opened_by { click_on 'onglet-preview' }
       within_window(preview_window) do
-        expect(page).to have_current_path(apercu_procedure_path(Procedure.last))
+        expect(page).to have_current_path(apercu_admin_procedure_path(Procedure.last))
         expect(page).to have_field('libelle de champ')
       end
     end
diff --git a/spec/features/new_administrateur/types_de_champ_spec.rb b/spec/features/new_administrateur/types_de_champ_spec.rb
index c079bb375..c3c5fb77a 100644
--- a/spec/features/new_administrateur/types_de_champ_spec.rb
+++ b/spec/features/new_administrateur/types_de_champ_spec.rb
@@ -4,7 +4,7 @@ feature 'As an administrateur I can edit types de champ', js: true do
 
   before do
     login_as administrateur.user, scope: :user
-    visit champs_procedure_path(procedure)
+    visit champs_admin_procedure_path(procedure)
   end
 
   it "Add a new champ" do
diff --git a/spec/features/routing/full_scenario_spec.rb b/spec/features/routing/full_scenario_spec.rb
index 08c913e0b..1f5a6213a 100644
--- a/spec/features/routing/full_scenario_spec.rb
+++ b/spec/features/routing/full_scenario_spec.rb
@@ -147,7 +147,7 @@ feature 'The routing', js: true do
 
     # the instructeurs who belong to scientifique AND litteraire groups manage scientifique and litterraire dossiers
     register_instructeur_and_log_in(superwoman.email)
-    visit procedure_path(procedure, params: { statut: 'tous' })
+    visit instructeur_procedure_path(procedure, params: { statut: 'tous' })
     expect(page).to have_text(litteraire_user.email)
     expect(page).to have_text(scientifique_user.email)
 
@@ -155,7 +155,7 @@ feature 'The routing', js: true do
     click_on scientifique_user.email
     click_on 'Suivre le dossier'
 
-    visit procedure_path(procedure, params: { statut: 'tous' })
+    visit instructeur_procedure_path(procedure, params: { statut: 'tous' })
     click_on litteraire_user.email
     click_on 'Suivre le dossier'
     log_out
diff --git a/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb b/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb
index c32a37d58..36dc08cbc 100644
--- a/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb
+++ b/spec/views/instructeur_mailer/send_notifications.html.haml_spec.rb
@@ -21,7 +21,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do
       ]
     end
 
-    it { expect(rendered).to have_link('une superbe démarche', href: procedure_url(213)) }
+    it { expect(rendered).to have_link('une superbe démarche', href: instructeur_procedure_url(213)) }
     it { expect(rendered).to have_text('une superbe démarche') }
     it { expect(rendered).to have_text('1 dossier en construction') }
     it { expect(rendered).not_to have_text('notification') }
@@ -41,7 +41,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do
       ]
     end
 
-    it { expect(rendered).to have_link('une superbe démarche', href: procedure_url(213)) }
+    it { expect(rendered).to have_link('une superbe démarche', href: instructeur_procedure_url(213)) }
     it { expect(rendered).to have_text('une superbe démarche') }
     it { expect(rendered).to have_text('1 dossier') }
     it { expect(rendered).not_to have_text('notification') }
@@ -63,7 +63,7 @@ describe 'instructeur_mailer/send_notifications.html.haml', type: :view do
       ]
     end
 
-    it { expect(rendered).to have_link('une superbe démarche', href: procedure_url(213)) }
+    it { expect(rendered).to have_link('une superbe démarche', href: instructeur_procedure_url(213)) }
     it { expect(rendered).to have_text('une superbe démarche') }
     it { expect(rendered).to have_text('1 dossier') }
     it { expect(rendered).not_to have_text('notification') }