Subtly rename admin/instructeurs to admin/assigns

app/controllers/admin/assigns_controller.rb

@@ -1,4 +1,4 @@
-class Admin::InstructeursController < AdminController
+class Admin::AssignsController < AdminController
   include SmartListing::Helper::ControllerExtensions
   helper SmartListing::Helper
 
@@ -12,7 +12,7 @@ class Admin::InstructeursController < AdminController
 
     @instructeurs_assign = smart_listing_create :instructeurs_assign,
       assign_scope,
-      partial: "admin/instructeurs/list_assign",
+      partial: "admin/assigns/list_assign",
       array: true
 
     not_assign_scope = current_administrateur.gestionnaires.where.not(id: assign_scope.ids)
@@ -23,7 +23,7 @@ class Admin::InstructeursController < AdminController
 
     @instructeurs_not_assign = smart_listing_create :instructeurs_not_assign,
       not_assign_scope,
-      partial: "admin/instructeurs/list_not_assign",
+      partial: "admin/assigns/list_not_assign",
       array: true
 
     @gestionnaire ||= Gestionnaire.new
@@ -49,6 +49,6 @@ class Admin::InstructeursController < AdminController
       end
     end
 
-    redirect_to admin_procedure_instructeurs_path, procedure_id: params[:procedure_id]
+    redirect_to admin_procedure_assigns_path, procedure_id: params[:procedure_id]
   end
 end

app/controllers/admin/gestionnaires_controller.rb

@@ -23,7 +23,7 @@ class Admin::GestionnairesController < AdminController
     end
 
     if procedure_id.present?
-      redirect_to admin_procedure_instructeurs_path(procedure_id: procedure_id)
+      redirect_to admin_procedure_assigns_path(procedure_id: procedure_id)
     else
       redirect_to admin_gestionnaires_path
     end

app/views/admin/assigns/_list_assign.html.haml

@@ -9,7 +9,7 @@
     - @instructeurs_assign.each do |instructeur|
       %tr
         %td.col-md-1.col-lg-1.col-sm-1.col-xs-1.col-sm-1.col-xs-1.center
-          = link_to "#{admin_procedure_instructeurs_path(procedure_id: @procedure.id, instructeur_id: instructeur.id, to: Admin::InstructeursController::NOT_ASSIGN)}", class: "btn btn-primary",  'data-method' => 'put' do
+          = link_to "#{admin_procedure_assigns_path(procedure_id: @procedure.id, instructeur_id: instructeur.id, to: Admin::AssignsController::NOT_ASSIGN)}", class: "btn btn-primary",  'data-method' => 'put' do
             .fa.fa-arrow-left
         %td{ style: 'padding-top: 11px; font-size: 15px; text-align: right;' }= instructeur.email
 

app/views/admin/assigns/_list_not_assign.html.haml

@@ -16,7 +16,7 @@
       %tr
         %td.col-xs-11{ style: 'padding-top: 11px; font-size: 15px;' }= instructeur.email
         %td.center
-          = link_to "#{admin_procedure_instructeurs_path(procedure_id: @procedure.id, instructeur_id: instructeur.id, to: Admin::InstructeursController::ASSIGN)}", class: "btn btn-success gestionnaire-affectation", 'data-method' => 'put' do
+          = link_to "#{admin_procedure_assigns_path(procedure_id: @procedure.id, instructeur_id: instructeur.id, to: Admin::AssignsController::ASSIGN)}", class: "btn btn-success instructeur-affectation", 'data-method' => 'put' do
             .fa.fa-arrow-right
 
 

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.gestionnaires.empty?
-                = link_to("des instructeurs", admin_procedure_instructeurs_path(@procedure))
+                = link_to("des instructeurs", admin_procedure_assigns_path(@procedure))
               - if @procedure.gestionnaires.empty? && @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.', admin_procedure_instructeurs_path(@procedure)
+              = link_to 'Cliquez ici.', admin_procedure_assigns_path(@procedure)
 
           %p.alert.alert-info
             Cette démarche n’a pas encore de lien, et n’est pas accessible par le public.

app/views/layouts/left_panels/_left_panel_admin_procedurescontroller_navbar.html.haml

@@ -31,7 +31,7 @@
       .procedure-list-element{ class: ('active' if active == 'Administrateurs') }
         Administrateurs
 
-    %a#onglet-instructeurs{ href: url_for(admin_procedure_instructeurs_path(@procedure)) }
+    %a#onglet-instructeurs{ href: url_for(admin_procedure_assigns_path(@procedure)) }
       .procedure-list-element{ class: ('active' if active == 'Instructeurs') }
         Instructeurs
         - if @procedure.missing_steps.include?(:instructeurs)

config/routes.rb

@@ -204,7 +204,7 @@ Rails.application.routes.draw do
       get 'monavis' => 'procedures#monavis', as: :monavis
       patch 'monavis' => 'procedures#update_monavis', as: :update_monavis
 
-      resource :instructeurs, only: [:show, :update]
+      resource :assigns, only: [:show, :update], path: 'instructeurs'
 
       resource :attestation_template, only: [:edit, :update, :create]
 
@@ -218,7 +218,7 @@ Rails.application.routes.draw do
       delete 'attestation_template/signature' => 'attestation_templates#delete_signature'
     end
 
-    namespace :instructeurs do
+    namespace :assigns do
       get 'show' # delete after fixed tests admin/instructeurs/show_spec without this line
     end
 

spec/controllers/admin/assigns_controller_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe Admin::InstructeursController, type: :controller do
+describe Admin::AssignsController, type: :controller do
   let(:admin) { create(:administrateur) }
   let(:procedure) { create :procedure, administrateur: admin }
   let(:gestionnaire) { create :gestionnaire, administrateurs: [admin] }
@@ -17,7 +17,7 @@ describe Admin::InstructeursController, type: :controller do
   describe 'PUT #update' do
     subject { put :update, params: { instructeur_id: gestionnaire.id, procedure_id: procedure.id, to: 'assign' } }
 
-    it { expect(subject).to redirect_to admin_procedure_instructeurs_path(procedure_id: procedure.id) }
+    it { expect(subject).to redirect_to admin_procedure_assigns_path(procedure_id: procedure.id) }
 
     context 'when assignement is valid' do
       before do

spec/controllers/admin/gestionnaires_controller_spec.rb

@@ -46,7 +46,7 @@ describe Admin::GestionnairesController, type: :controller do
         let(:procedure_id) { procedure.id }
 
         it { expect(response.status).to eq(302) }
-        it { expect(response).to redirect_to admin_procedure_instructeurs_path(procedure_id: procedure_id) }
+        it { expect(response).to redirect_to admin_procedure_assigns_path(procedure_id: procedure_id) }
       end
 
       describe 'Gestionnaire attributs in database' do

spec/views/admin/assigns/show.html.haml_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'admin/instructeurs/show.html.haml', type: :view do
+describe 'admin/assigns/show.html.haml', type: :view do
   let(:admin) { create(:administrateur) }
   let(:procedure) { create :procedure, administrateur: admin }
 
@@ -12,13 +12,13 @@ describe 'admin/instructeurs/show.html.haml', type: :view do
     assign(:gestionnaire, Gestionnaire.new)
 
     assign(:instructeurs_assign, (smart_listing_create :instructeurs_assign,
-      assign_gestionnaires,
-      partial: "admin/instructeurs/list_assign",
+      assign_instructeurs,
+      partial: "admin/assigns/list_assign",
       array: true))
 
     assign(:instructeurs_not_assign, (smart_listing_create :instructeurs_not_assign,
-      not_assign_gestionnaires,
-      partial: "admin/instructeurs/list_not_assign",
+      not_assign_instructeurs,
+      partial: "admin/assigns/list_not_assign",
       array: true))
   end
 
@@ -43,13 +43,13 @@ describe 'admin/instructeurs/show.html.haml', type: :view do
       assign_gestionnaires.reload
 
       assign(:instructeurs_assign, (smart_listing_create :instructeurs_assign,
-        assign_gestionnaires,
-        partial: "admin/instructeurs/list_assign",
+        assign_instructeurs,
+        partial: "admin/assigns/list_assign",
         array: true))
 
       assign(:instructeurs_not_assign, (smart_listing_create :instructeurs_not_assign,
-        not_assign_gestionnaires,
-        partial: "admin/instructeurs/list_not_assign",
+        not_assign_instructeurs,
+        partial: "admin/assigns/list_not_assign",
         array: true))
 
       render