Merge pull request #5306 from betagouv/3667-dossiers-groupes-par-type-demarche

groupe les avis par démarche pour les experts

app/controllers/instructeurs/avis_controller.rb

@@ -11,7 +11,13 @@ module Instructeurs
     DONNES_STATUS   = 'donnes'
 
     def index
-      instructeur_avis = current_instructeur.avis.includes(dossier: [:procedure, :user])
+      avis = current_instructeur.avis.includes(dossier: [groupe_instructeur: :procedure])
+      @avis_by_procedure = avis.to_a.group_by(&:procedure)
+    end
+
+    def procedure
+      @procedure = Procedure.find(params[:procedure_id])
+      instructeur_avis = current_instructeur.avis.includes(:dossier).where(dossiers: { groupe_instructeur: GroupeInstructeur.where(procedure: @procedure.id) })
       @avis_a_donner = instructeur_avis.without_answer
       @avis_donnes = instructeur_avis.with_answer
 
@@ -37,7 +43,7 @@ module Instructeurs
     def update
       if @avis.update(avis_params)
         flash.notice = 'Votre réponse est enregistrée.'
-        redirect_to instruction_instructeur_avis_path(@avis)
+        redirect_to instruction_instructeur_avis_path(@avis.procedure, @avis)
       else
         flash.now.alert = @avis.errors.full_messages
         @new_avis = Avis.new
@@ -54,7 +60,7 @@ module Instructeurs
 
       if @commentaire.save
         flash.notice = "Message envoyé"
-        redirect_to messagerie_instructeur_avis_path(avis)
+        redirect_to messagerie_instructeur_avis_path(avis.procedure, avis)
       else
         flash.alert = @commentaire.errors.full_messages
         render :messagerie
@@ -65,7 +71,7 @@ module Instructeurs
       @new_avis = create_avis_from_params(avis.dossier, avis.confidentiel)
 
       if @new_avis.nil?
-        redirect_to instruction_instructeur_avis_path(avis)
+        redirect_to instruction_instructeur_avis_path(avis.procedure, avis)
       else
         set_avis_and_dossier
         render :instruction
@@ -89,6 +95,8 @@ module Instructeurs
     end
 
     def create_instructeur
+      procedure_id = params[:procedure_id]
+      avis_id = params[:id]
       email = params[:email]
       password = params[:user][:password]
 
@@ -99,10 +107,10 @@ module Instructeurs
         sign_in(user)
 
         Avis.link_avis_to_instructeur(user.instructeur)
-        redirect_to url_for(instructeur_avis_index_path)
+        redirect_to url_for(instructeur_all_avis_path)
       else
         flash[:alert] = user.errors.full_messages
-        redirect_to url_for(sign_up_instructeur_avis_path(params[:id], email))
+        redirect_to url_for(sign_up_instructeur_avis_path(procedure_id, avis_id, email))
       end
     end
 
@@ -119,7 +127,7 @@ module Instructeurs
       if current_instructeur.present?
         # a instructeur is authenticated ... lets see if it can view the dossier
 
-        redirect_to instructeur_avis_url(avis)
+        redirect_to instructeur_avis_url(avis.procedure, avis)
       elsif avis.instructeur&.email == params[:email]
         # the avis instructeur has already signed up and it sould sign in
 

app/controllers/instructeurs/procedures_controller.rb

@@ -284,7 +284,7 @@ module Instructeurs
 
     def redirect_to_avis_if_needed
       if current_instructeur.procedures.count == 0 && current_instructeur.avis.count > 0
-        redirect_to instructeur_avis_index_path
+        redirect_to instructeur_all_avis_path
       end
     end
 

app/helpers/dossier_link_helper.rb

@@ -6,7 +6,7 @@ module DossierLinkHelper
       else
         avis = dossier.avis.find_by(instructeur: user)
         if avis.present?
-          instructeur_avis_path(avis)
+          instructeur_avis_path(avis.procedure, avis)
         end
       end
     elsif user.owns_or_invite?(dossier)

app/models/avis.rb

@@ -52,6 +52,10 @@ class Avis < ApplicationRecord
     ]
   end
 
+  def procedure
+    dossier.procedure
+  end
+
   private
 
   def try_to_assign_instructeur

app/views/avis_mailer/avis_invitation.html.haml

@@ -1,5 +1,5 @@
 - content_for(:title, 'Invitation à donner votre avis')
-- avis_link = @avis.instructeur.present? ? instructeur_avis_url(@avis) : sign_up_instructeur_avis_url(@avis.id, @avis.email)
+- avis_link = @avis.instructeur.present? ? instructeur_avis_url(@avis.procedure, @avis) : sign_up_instructeur_avis_url(@avis.procedure, @avis.id, @avis.email)
 
 - content_for(:footer) do
   Merci de ne pas répondre à cet email. Donnez votre avis
@@ -15,7 +15,7 @@
   Vous avez été invité par
   %strong= @avis.claimant.email
   = "à donner votre avis sur le dossier nº #{@avis.dossier.id} de la démarche :"
-  %strong= @avis.dossier.procedure.libelle
+  %strong= @avis.procedure.libelle
 
 %p
   = "#{@avis.claimant.email} vous a écrit :"

app/views/instructeurs/avis/_header.html.haml

@@ -1,10 +1,11 @@
 .sub-header
   .container
     %ul.breadcrumbs
-      %li= link_to('Avis', instructeur_avis_index_path)
+      %li= link_to('Avis', instructeur_all_avis_path)
-      %li= "#{dossier.procedure.libelle}, dossier nº #{dossier.id}"
+      %li= link_to(dossier.procedure.libelle, procedure_instructeur_avis_index_path(avis.procedure))
+      %li= link_to("Dossier nº #{dossier.id}", instructeur_avis_path(avis.procedure, avis))
 
     %ul.tabs
-      = dynamic_tab_item('Demande', instructeur_avis_path(avis))
+      = dynamic_tab_item('Demande', instructeur_avis_path(avis.procedure, avis))
-      = dynamic_tab_item('Avis', instruction_instructeur_avis_path(avis), notification: avis.answer.blank?)
+      = dynamic_tab_item('Avis', instruction_instructeur_avis_path(avis.procedure, avis), notification: avis.answer.blank?)
-      = dynamic_tab_item('Messagerie', messagerie_instructeur_avis_path(avis))
+      = dynamic_tab_item('Messagerie', messagerie_instructeur_avis_path(avis.procedure, avis))

app/views/instructeurs/avis/index.html.haml

@@ -1,39 +1,41 @@
-- avis_statut = (@statut == Instructeurs::AvisController::A_DONNER_STATUS) ? 'à donner' : 'rendus'
+- content_for(:title, "Avis")
-- content_for(:title, "Avis #{avis_statut}")
-
-.sub-header
-  .container.flex
-    .width-100
-      %h1.tab-title Avis
-      %ul.tabs
-        = tab_item('avis à donner',
-          instructeur_avis_index_path(statut: Instructeurs::AvisController::A_DONNER_STATUS),
-          active: @statut == Instructeurs::AvisController::A_DONNER_STATUS,
-          badge: @avis_a_donner.count,
-          notification: @avis_a_donner.any?)
-
-        = tab_item("avis #{'donné'.pluralize(@avis_donnes.count)}",
-          instructeur_avis_index_path(statut: Instructeurs::AvisController::DONNES_STATUS),
-          active: @statut == Instructeurs::AvisController::DONNES_STATUS,
-          badge: @avis_donnes.count)
 
 .container
-  - if @avis.present?
+  %h1.page-title Avis
-    %table.table.dossiers-table.hoverable
+
-      %thead
+  %ul.procedure-list
-        %tr
+    - @avis_by_procedure.each do |p, procedure_avis|
-          %th.number-col Nº dossier
+      %li.procedure-item.flex.align-start
-          %th Demandeur
+        = link_to(procedure_instructeur_avis_index_path(p)) do
-          %th Démarche
+          .flex
-      %tbody
+
-        - @avis.each do |avis|
+            .procedure-logo{ style: "background-image: url(#{p.logo_url})" }
-          %tr
+
-            %td.number-col
+            .procedure-details
-              = link_to(instructeur_avis_path(avis), class: 'cell-link') do
+              %p.procedure-title
-                %span.icon.folder
+                = procedure_libelle p
-                #{avis.dossier.id}
+              %ul.procedure-stats.flex
-            %td= link_to(avis.dossier.user.email, instructeur_avis_path(avis), class: 'cell-link')
+                %li
-            %td= link_to(avis.dossier.procedure.libelle, instructeur_avis_path(avis), class: 'cell-link')
+                  %object
-    = paginate(@avis)
+                    = link_to(procedure_instructeur_avis_index_path(p, statut: Instructeurs::AvisController::A_DONNER_STATUS)) do
-  - else
+                      - without_answer_count = procedure_avis.select { |a| a.answer.nil? }.size
-    %h2.empty-text Aucun avis
+                      - if without_answer_count > 0
+                        %span.notifications{ 'aria-label': "notifications" }
+                      .stats-number
+                        = without_answer_count
+                      .stats-legend
+                        avis à donner
+                %li
+                  %object
+                    = link_to(procedure_instructeur_avis_index_path(p, statut: Instructeurs::AvisController::DONNES_STATUS)) do
+                      - with_answer_count = procedure_avis.select { |a| a.answer.present? }.size
+                      .stats-number= with_answer_count
+                      .stats-legend
+                        = pluralize(with_answer_count, "avis donné")
+
+            - if p.close?
+              .procedure-status
+                %span.label Close
+            - elsif p.depubliee?
+              .procedure-status
+                %span.label Dépubliée

app/views/instructeurs/avis/instruction.html.haml

@@ -15,7 +15,7 @@
       = render partial: 'shared/attachment/show', locals: { attachment: @avis.introduction_file.attachment }
     %br/
 
-    = form_for @avis, url: instructeur_avis_path(@avis), html: { class: 'form' } do |f|
+    = form_for @avis, url: instructeur_avis_path(@avis.procedure, @avis), html: { class: 'form' } do |f|
       = f.text_area :answer, rows: 3, placeholder: 'Votre avis', required: true
       = render 'shared/attachment/edit',
         { form: f,
@@ -35,7 +35,7 @@
           = f.submit 'Envoyer votre avis', class: 'button send'
 
   - if !@dossier.termine?
-    = render partial: "instructeurs/shared/avis/form", locals: { url: avis_instructeur_avis_path(@avis), linked_dossiers: @dossier.linked_dossiers_for(current_instructeur), must_be_confidentiel: @avis.confidentiel?, avis: @new_avis }
+    = render partial: "instructeurs/shared/avis/form", locals: { url: avis_instructeur_avis_path(@avis.procedure, @avis), linked_dossiers: @dossier.linked_dossiers_for(current_instructeur), must_be_confidentiel: @avis.confidentiel?, avis: @new_avis }
 
   - if @dossier.avis_for(current_instructeur).present?
     = render partial: 'instructeurs/shared/avis/list', locals: { avis: @dossier.avis_for(current_instructeur), avis_seen_at: nil }

app/views/instructeurs/avis/procedure.html.haml

@@ -0,0 +1,45 @@
+- avis_statut = (@statut == Instructeurs::AvisController::A_DONNER_STATUS) ? 'à donner' : 'rendus'
+- content_for(:title, "Avis #{avis_statut}")
+
+#procedure-show
+  .sub-header
+    .container.flex
+
+      .procedure-logo{ style: "background-image: url(#{@procedure.logo_url})",
+        role: 'img', 'aria-label': "logo de la démarche #{@procedure.libelle}" }
+
+      .procedure-header
+        %h1= procedure_libelle @procedure
+
+        %ul.tabs
+          = tab_item('avis à donner',
+            procedure_instructeur_avis_index_path(statut: Instructeurs::AvisController::A_DONNER_STATUS),
+            active: @statut == Instructeurs::AvisController::A_DONNER_STATUS,
+            badge: @avis_a_donner.count,
+            notification: @avis_a_donner.any?)
+
+          = tab_item("avis #{'donné'.pluralize(@avis_donnes.count)}",
+            procedure_instructeur_avis_index_path(statut: Instructeurs::AvisController::DONNES_STATUS),
+            active: @statut == Instructeurs::AvisController::DONNES_STATUS,
+            badge: @avis_donnes.count)
+
+.container
+  - if @avis.present?
+    %table.table.dossiers-table.hoverable
+      %thead
+        %tr
+          %th.number-col Nº dossier
+          %th Demandeur
+          %th Démarche
+      %tbody
+        - @avis.each do |avis|
+          %tr
+            %td.number-col
+              = link_to(instructeur_avis_path(avis.procedure, avis), class: 'cell-link') do
+                %span.icon.folder
+                #{avis.dossier.id}
+            %td= link_to(avis.dossier.user.email, instructeur_avis_path(avis.procedure, avis), class: 'cell-link')
+            %td= link_to(avis.procedure.libelle, instructeur_avis_path(avis.procedure, avis), class: 'cell-link')
+    = paginate(@avis)
+  - else
+    %h2.empty-text Aucun avis

app/views/layouts/_new_header.haml

@@ -29,10 +29,10 @@
         %ul.header-tabs
           - if current_instructeur.procedures.count > 0
             %li
-              = active_link_to "Démarches", instructeur_procedures_path, active: :inclusive, class: 'tab-link'
+              = active_link_to "Démarches", instructeur_procedures_path, active: /^((?!avis).)*$/, class: 'tab-link'
           - if current_instructeur.avis.count > 0
             %li
-              = active_link_to instructeur_avis_index_path, active: :inclusive, class: 'tab-link' do
+              = active_link_to instructeur_all_avis_path, active: /avis/, class: 'tab-link' do
                 Avis
                 - avis_counter = current_instructeur.avis.without_answer.count
                 - if avis_counter > 0

config/routes.rb

@@ -300,6 +300,13 @@ Rails.application.routes.draw do
   #
 
   scope module: 'instructeurs', as: 'instructeur' do
+    get 'avis', to: 'avis#index', as: 'all_avis'
+
+    # this redirections are ephemeral, to ensure that emails sent to experts before are still valid
+    # TODO : they will be removed in September, 2020
+    get 'avis/:id', to: redirect('/procedures/old/avis/%{id}')
+    get 'avis/:id/sign_up/email/:email', to: redirect("/procedures/old/avis/%{id}/sign_up/email/%{email}"), constraints: { email: /.*/ }
+
     resources :procedures, only: [:index, :show], param: :procedure_id do
       member do
         resources :groupes, only: [:index, :show], controller: 'groupe_instructeurs' do
@@ -309,6 +316,20 @@ Rails.application.routes.draw do
           end
         end
 
+        resources :avis, only: [:show, :update] do
+          get '', action: 'procedure', on: :collection, as: :procedure
+          member do
+            get 'instruction'
+            get 'messagerie'
+            post 'commentaire' => 'avis#create_commentaire'
+            post 'avis' => 'avis#create_avis'
+            get 'bilans_bdf'
+
+            get 'sign_up/email/:email' => 'avis#sign_up', constraints: { email: /.*/ }, as: 'sign_up'
+            post 'sign_up/email/:email' => 'avis#create_instructeur', constraints: { email: /.*/ }
+          end
+        end
+
         patch 'update_displayed_fields'
         get 'update_sort/:table/:column' => 'procedures#update_sort', as: 'update_sort'
         post 'add_filter'
@@ -347,18 +368,6 @@ Rails.application.routes.draw do
         end
       end
     end
-    resources :avis, only: [:index, :show, :update] do
-      member do
-        get 'instruction'
-        get 'messagerie'
-        post 'commentaire' => 'avis#create_commentaire'
-        post 'avis' => 'avis#create_avis'
-        get 'bilans_bdf'
-
-        get 'sign_up/email/:email' => 'avis#sign_up', constraints: { email: /.*/ }, as: 'sign_up'
-        post 'sign_up/email/:email' => 'avis#create_instructeur', constraints: { email: /.*/ }
-      end
-    end
     get "recherche" => "recherche#index"
   end
 

spec/controllers/instructeurs/avis_controller_spec.rb

@@ -4,7 +4,8 @@ describe Instructeurs::AvisController, type: :controller do
 
     let(:claimant) { create(:instructeur) }
     let(:instructeur) { create(:instructeur) }
-    let(:procedure) { create(:procedure, :published, instructeurs: [instructeur]) }
+    let(:procedure) { create(:procedure, :published, instructeurs: [claimant]) }
+    let(:another_procedure) { create(:procedure, :published, instructeurs: [claimant]) }
     let(:dossier) { create(:dossier, :en_construction, procedure: procedure) }
     let!(:avis_without_answer) { Avis.create(dossier: dossier, claimant: claimant, instructeur: instructeur) }
     let!(:avis_with_answer) { Avis.create(dossier: dossier, claimant: claimant, instructeur: instructeur, answer: 'yop') }
@@ -14,20 +15,28 @@ describe Instructeurs::AvisController, type: :controller do
     describe '#index' do
       before { get :index }
 
+      it { expect(response).to have_http_status(:success) }
+      it { expect(assigns(:avis_by_procedure).flatten).to include(procedure) }
+      it { expect(assigns(:avis_by_procedure).flatten).not_to include(another_procedure) }
+    end
+
+    describe '#procedure' do
+      before { get :procedure, params: { procedure_id: procedure.id } }
+
       it { expect(response).to have_http_status(:success) }
       it { expect(assigns(:avis_a_donner)).to match([avis_without_answer]) }
       it { expect(assigns(:avis_donnes)).to match([avis_with_answer]) }
       it { expect(assigns(:statut)).to eq('a-donner') }
 
       context 'with a statut equal to donnes' do
-        before { get :index, params: { statut: 'donnes' } }
+        before { get :procedure, params: { statut: 'donnes', procedure_id: procedure.id } }
 
         it { expect(assigns(:statut)).to eq('donnes') }
       end
     end
 
     describe '#show' do
-      before { get :show, params: { id: avis_without_answer.id } }
+      before { get :show, params: { id: avis_without_answer.id, procedure_id: procedure.id } }
 
       it { expect(response).to have_http_status(:success) }
       it { expect(assigns(:avis)).to eq(avis_without_answer) }
@@ -35,7 +44,7 @@ describe Instructeurs::AvisController, type: :controller do
     end
 
     describe '#instruction' do
-      before { get :instruction, params: { id: avis_without_answer.id } }
+      before { get :instruction, params: { id: avis_without_answer.id, procedure_id: procedure.id } }
 
       it { expect(response).to have_http_status(:success) }
       it { expect(assigns(:avis)).to eq(avis_without_answer) }
@@ -43,7 +52,7 @@ describe Instructeurs::AvisController, type: :controller do
     end
 
     describe '#messagerie' do
-      before { get :messagerie, params: { id: avis_without_answer.id } }
+      before { get :messagerie, params: { id: avis_without_answer.id, procedure_id: procedure.id } }
 
       it { expect(response).to have_http_status(:success) }
       it { expect(assigns(:avis)).to eq(avis_without_answer) }
@@ -51,7 +60,7 @@ describe Instructeurs::AvisController, type: :controller do
     end
 
     describe '#bilans_bdf' do
-      before { get :bilans_bdf, params: { id: avis_without_answer.id } }
+      before { get :bilans_bdf, params: { id: avis_without_answer.id, procedure_id: procedure.id } }
 
       it { expect(response).to redirect_to(instructeur_avis_path(avis_without_answer)) }
     end
@@ -59,12 +68,12 @@ describe Instructeurs::AvisController, type: :controller do
     describe '#update' do
       describe 'without attachment' do
         before do
-          patch :update, params: { id: avis_without_answer.id, avis: { answer: 'answer' } }
+          patch :update, params: { id: avis_without_answer.id, procedure_id: procedure.id, avis: { answer: 'answer' } }
           avis_without_answer.reload
         end
 
         it 'should be ok' do
-          expect(response).to redirect_to(instruction_instructeur_avis_path(avis_without_answer))
+          expect(response).to redirect_to(instruction_instructeur_avis_path(avis_without_answer.procedure, avis_without_answer))
           expect(avis_without_answer.answer).to eq('answer')
           expect(avis_without_answer.piece_justificative_file).to_not be_attached
           expect(flash.notice).to eq('Votre réponse est enregistrée.')
@@ -78,13 +87,13 @@ describe Instructeurs::AvisController, type: :controller do
         before do
           expect(ClamavService).to receive(:safe_file?).and_return(true)
           perform_enqueued_jobs do
-            post :update, params: { id: avis_without_answer.id, avis: { answer: 'answer', piece_justificative_file: file } }
+            post :update, params: { id: avis_without_answer.id, procedure_id: procedure.id, avis: { answer: 'answer', piece_justificative_file: file } }
           end
           avis_without_answer.reload
         end
 
         it 'should be ok' do
-          expect(response).to redirect_to(instruction_instructeur_avis_path(avis_without_answer))
+          expect(response).to redirect_to(instruction_instructeur_avis_path(avis_without_answer.procedure, avis_without_answer))
           expect(avis_without_answer.answer).to eq('answer')
           expect(avis_without_answer.piece_justificative_file).to be_attached
           expect(flash.notice).to eq('Votre réponse est enregistrée.')
@@ -96,7 +105,7 @@ describe Instructeurs::AvisController, type: :controller do
       let(:file) { nil }
       let(:scan_result) { true }
 
-      subject { post :create_commentaire, params: { id: avis_without_answer.id, commentaire: { body: 'commentaire body', piece_jointe: file } } }
+      subject { post :create_commentaire, params: { id: avis_without_answer.id, procedure_id: procedure.id, commentaire: { body: 'commentaire body', piece_jointe: file } } }
 
       before do
         allow(ClamavService).to receive(:safe_file?).and_return(scan_result)
@@ -105,7 +114,7 @@ describe Instructeurs::AvisController, type: :controller do
       it do
         subject
 
-        expect(response).to redirect_to(messagerie_instructeur_avis_path(avis_without_answer))
+        expect(response).to redirect_to(messagerie_instructeur_avis_path(avis_without_answer.procedure, avis_without_answer))
         expect(dossier.commentaires.map(&:body)).to match(['commentaire body'])
       end
 
@@ -131,7 +140,7 @@ describe Instructeurs::AvisController, type: :controller do
 
       before do
         @introduction_file = Rack::Test::UploadedFile.new("./spec/fixtures/files/piece_justificative_0.pdf", 'application/pdf')
-        post :create_avis, params: { id: previous_avis.id, avis: { emails: emails, introduction: intro, confidentiel: asked_confidentiel, invite_linked_dossiers: invite_linked_dossiers, introduction_file: @introduction_file } }
+        post :create_avis, params: { id: previous_avis.id, procedure_id: procedure.id, avis: { emails: emails, introduction: intro, confidentiel: asked_confidentiel, invite_linked_dossiers: invite_linked_dossiers, introduction_file: @introduction_file } }
         created_avis.reload
       end
 
@@ -178,7 +187,7 @@ describe Instructeurs::AvisController, type: :controller do
           it { expect(created_avis.introduction).to eq(intro) }
           it { expect(created_avis.dossier).to eq(previous_avis.dossier) }
           it { expect(created_avis.claimant).to eq(instructeur) }
-          it { expect(response).to redirect_to(instruction_instructeur_avis_path(previous_avis)) }
+          it { expect(response).to redirect_to(instruction_instructeur_avis_path(previous_avis.procedure, previous_avis)) }
         end
 
         context 'when the user asked for a confidentiel avis' do
@@ -255,6 +264,7 @@ describe Instructeurs::AvisController, type: :controller do
     describe '#sign_up' do
       let(:invited_email) { 'invited@avis.com' }
       let(:dossier) { create(:dossier) }
+      let(:procedure) { dossier.procedure }
       let!(:avis) { create(:avis, email: invited_email, dossier: dossier) }
       let(:invitations_email) { true }
 
@@ -263,7 +273,7 @@ describe Instructeurs::AvisController, type: :controller do
           expect(Avis).to receive(:avis_exists_and_email_belongs_to_avis?)
             .with(avis.id.to_s, invited_email)
             .and_return(invitations_email)
-          get :sign_up, params: { id: avis.id, email: invited_email }
+          get :sign_up, params: { id: avis.id, procedure_id: procedure.id, email: invited_email }
         end
 
         context 'when the email belongs to the invitation' do
@@ -286,15 +296,15 @@ describe Instructeurs::AvisController, type: :controller do
         context 'when the instructeur is authenticated' do
           before do
             sign_in(instructeur.user)
-            get :sign_up, params: { id: avis.id, email: invited_email }
+            get :sign_up, params: { id: avis.id, procedure_id: procedure.id, email: invited_email }
           end
 
-          it { is_expected.to redirect_to instructeur_avis_url(avis) }
+          it { is_expected.to redirect_to instructeur_avis_url(avis.procedure, avis) }
         end
 
         context 'when the instructeur is not authenticated' do
           before do
-            get :sign_up, params: { id: avis.id, email: invited_email }
+            get :sign_up, params: { id: avis.id, procedure_id: procedure.id, email: invited_email }
           end
 
           it { is_expected.to redirect_to new_user_session_url }
@@ -307,11 +317,11 @@ describe Instructeurs::AvisController, type: :controller do
 
         before do
           sign_in(instructeur.user)
-          get :sign_up, params: { id: avis.id, email: invited_email }
+          get :sign_up, params: { id: avis.id, procedure_id: procedure.id, email: invited_email }
         end
 
         # redirected to dossier but then the instructeur gonna be banished !
-        it { is_expected.to redirect_to instructeur_avis_url(avis) }
+        it { is_expected.to redirect_to instructeur_avis_url(avis.procedure, avis) }
       end
     end
 
@@ -320,6 +330,7 @@ describe Instructeurs::AvisController, type: :controller do
       let!(:existing_user) { create(:user, email: existing_user_mail) }
       let(:invited_email) { 'invited@avis.com' }
       let(:dossier) { create(:dossier) }
+      let(:procedure) { dossier.procedure }
       let!(:avis) { create(:avis, email: invited_email, dossier: dossier) }
       let(:avis_id) { avis.id }
       let(:password) { 'démarches-simplifiées-pwd' }
@@ -334,6 +345,7 @@ describe Instructeurs::AvisController, type: :controller do
 
         post :create_instructeur, params: {
           id: avis_id,
+          procedure_id: procedure.id,
           email: invited_email,
           user: {
             password: password
@@ -355,7 +367,7 @@ describe Instructeurs::AvisController, type: :controller do
           it { expect(Avis).to have_received(:link_avis_to_instructeur) }
 
           it { expect(subject.current_instructeur).to eq(created_instructeur) }
-          it { is_expected.to redirect_to instructeur_avis_index_path }
+          it { is_expected.to redirect_to instructeur_all_avis_path }
 
           it 'creates a corresponding user account for the email' do
             user = User.find_by(email: invited_email)
@@ -375,7 +387,7 @@ describe Instructeurs::AvisController, type: :controller do
           let(:password) { '' }
 
           it { expect(created_instructeur).to be_nil }
-          it { is_expected.to redirect_to sign_up_instructeur_avis_path(avis_id, invited_email) }
+          it { is_expected.to redirect_to sign_up_instructeur_avis_path(procedure.id, avis_id, invited_email) }
           it { expect(flash.alert).to eq(['Le mot de passe doit être rempli']) }
         end
       end

spec/controllers/instructeurs/procedures_controller_spec.rb

@@ -76,7 +76,7 @@ describe Instructeurs::ProceduresController, type: :controller do
       end
 
       it "redirects avis" do
-        expect(@controller).to have_received(:redirect_to).with(instructeur_avis_index_path)
+        expect(@controller).to have_received(:redirect_to).with(instructeur_all_avis_path)
       end
     end
   end

spec/features/instructeurs/expert_spec.rb

@@ -42,7 +42,7 @@ feature 'Inviting an expert:' do
 
       invitation_email = open_email('expert2@exemple.fr')
       avis = Avis.find_by(email: 'expert2@exemple.fr', dossier: dossier)
-      sign_up_link = sign_up_instructeur_avis_path(avis.id, avis.email)
+      sign_up_link = sign_up_instructeur_avis_path(avis.dossier.procedure, avis, avis.email)
       expect(invitation_email.body).to include(sign_up_link)
     end
 
@@ -71,14 +71,14 @@ feature 'Inviting an expert:' do
       let(:avis_email) { 'not-signed-up-expert@exemple.fr' }
 
       scenario 'I can sign up' do
-        visit sign_up_instructeur_avis_path(avis.id, avis_email)
+        visit sign_up_instructeur_avis_path(avis.dossier.procedure, avis, avis_email)
 
         expect(page).to have_field('Email', with: avis_email, disabled: true)
         fill_in 'Mot de passe', with: 'This is a very complicated password !'
         click_on 'Créer un compte'
 
-        expect(page).to have_current_path(instructeur_avis_index_path)
+        expect(page).to have_current_path(instructeur_all_avis_path)
-        expect(page).to have_text('avis à donner 1')
+        expect(page).to have_text('1 avis à donner')
       end
     end
 
@@ -86,13 +86,13 @@ feature 'Inviting an expert:' do
       let(:avis_email) { expert.email }
 
       scenario 'I can sign in' do
-        visit sign_up_instructeur_avis_path(avis.id, avis_email)
+        visit sign_up_instructeur_avis_path(avis.dossier.procedure, avis, avis_email)
 
         expect(page).to have_current_path(new_user_session_path)
         sign_in_with(expert.email, expert_password)
 
-        expect(page).to have_current_path(instructeur_avis_index_path)
+        expect(page).to have_current_path(instructeur_all_avis_path)
-        expect(page).to have_text('avis à donner 1')
+        expect(page).to have_text('1 avis à donner')
       end
     end
 
@@ -100,10 +100,11 @@ feature 'Inviting an expert:' do
       avis # create avis
       login_as expert.user, scope: :user
 
-      visit instructeur_avis_index_path
+      visit instructeur_all_avis_path
-      expect(page).to have_text('avis à donner 1')
+      expect(page).to have_text('1 avis à donner')
-      expect(page).to have_text('avis donnés 0')
+      expect(page).to have_text('0 avis donnés')
 
+      click_on '1 avis à donner'
       click_on avis.dossier.user.email
 
       within('.tabs') { click_on 'Avis' }
@@ -119,8 +120,8 @@ feature 'Inviting an expert:' do
       expect(page).to have_content('RIB.pdf')
 
       within('.new-header') { click_on 'Avis' }
-      expect(page).to have_text('avis à donner 0')
+      expect(page).to have_text('0 avis à donner')
-      expect(page).to have_text('avis donné 1')
+      expect(page).to have_text('1 avis donné')
     end
 
     # TODO

spec/features/instructeurs/instruction_spec.rb

@@ -159,12 +159,6 @@ feature 'Instructing a dossier:' do
     click_on 'Demander un avis'
   end
 
-  def test_mail(to, content)
-    mail = ActionMailer::Base.deliveries.last
-    expect(mail.to).to match([to])
-    expect(mail.body.parts.map(&:to_s)).to all(include(content))
-  end
-
   def test_statut_bar(a_suivre: 0, suivi: 0, traite: 0, tous_les_dossiers: 0, archive: 0)
     texts = [
       "à suivre #{a_suivre}",
@@ -177,13 +171,6 @@ feature 'Instructing a dossier:' do
     texts.each { |text| expect(page).to have_text(text) }
   end
 
-  def avis_sign_up(avis, email)
-    visit sign_up_instructeur_avis_path(avis, email)
-    fill_in 'user_password', with: 'démarches-simplifiées-pwd'
-    click_on 'Créer un compte'
-    expect(page).to have_current_path(instructeur_avis_index_path)
-  end
-
   def dossier_present?(id, statut)
     within(:css, '.dossiers-table') do
       expect(page).to have_text(id)

spec/helpers/dossier_link_helper_spec.rb

@@ -29,7 +29,7 @@ describe DossierLinkHelper do
       let(:instructeur) { create(:instructeur) }
       let!(:avis) { create(:avis, dossier: dossier, instructeur: instructeur) }
 
-      it { expect(helper.dossier_linked_path(instructeur, dossier)).to eq(instructeur_avis_path(avis)) }
+      it { expect(helper.dossier_linked_path(instructeur, dossier)).to eq(instructeur_avis_path(avis.dossier.procedure, avis)) }
     end
 
     context "when access as user" do

spec/mailers/avis_mailer_spec.rb

@@ -7,7 +7,7 @@ RSpec.describe AvisMailer, type: :mailer do
     it { expect(subject.subject).to eq("Donnez votre avis sur le dossier nº #{avis.dossier.id} (#{avis.dossier.procedure.libelle})") }
     it { expect(subject.body).to have_text("Vous avez été invité par\r\n#{avis.claimant.email}\r\nà donner votre avis sur le dossier nº #{avis.dossier.id} de la démarche :\r\n#{avis.dossier.procedure.libelle}") }
     it { expect(subject.body).to include(avis.introduction) }
-    it { expect(subject.body).to include(instructeur_avis_url(avis)) }
+    it { expect(subject.body).to include(instructeur_avis_url(avis.dossier.procedure.id, avis)) }
 
     context 'when the recipient is not already registered' do
       before do
@@ -15,7 +15,7 @@ RSpec.describe AvisMailer, type: :mailer do
         avis.instructeur = nil
       end
 
-      it { expect(subject.body).to include(sign_up_instructeur_avis_url(avis.id, avis.email)) }
+      it { expect(subject.body).to include(sign_up_instructeur_avis_url(avis.dossier.procedure.id, avis.id, avis.email)) }
     end
   end
 end