diff --git a/app/controllers/admin/assigns_controller.rb b/app/controllers/admin/assigns_controller.rb index c295076af..fca3aabc6 100644 --- a/app/controllers/admin/assigns_controller.rb +++ b/app/controllers/admin/assigns_controller.rb @@ -8,7 +8,7 @@ class Admin::AssignsController < AdminController NOT_ASSIGN = 'not_assign' def show - assign_scope = @procedure.instructeurs + assign_scope = @procedure.defaut_groupe_instructeur.instructeurs @instructeurs_assign = smart_listing_create :instructeurs_assign, assign_scope, diff --git a/app/controllers/instructeurs/dossiers_controller.rb b/app/controllers/instructeurs/dossiers_controller.rb index 2f676a371..cdfb3f60a 100644 --- a/app/controllers/instructeurs/dossiers_controller.rb +++ b/app/controllers/instructeurs/dossiers_controller.rb @@ -56,7 +56,7 @@ module Instructeurs @previous_following_instructeurs_emails = previous_followers.pluck(:email) @avis_emails = dossier.avis.includes(:instructeur).map(&:email_to_display) @invites_emails = dossier.invites.map(&:email) - @potential_recipients = procedure.instructeurs.reject { |g| g == current_instructeur } + @potential_recipients = procedure.defaut_groupe_instructeur.instructeurs.reject { |g| g == current_instructeur } end def send_to_instructeurs diff --git a/app/controllers/instructeurs/procedures_controller.rb b/app/controllers/instructeurs/procedures_controller.rb index 71624ab2e..f900a5bf0 100644 --- a/app/controllers/instructeurs/procedures_controller.rb +++ b/app/controllers/instructeurs/procedures_controller.rb @@ -221,7 +221,7 @@ module Instructeurs end def ensure_ownership! - if !procedure.instructeurs.include?(current_instructeur) + if !procedure.defaut_groupe_instructeur.instructeurs.include?(current_instructeur) flash[:alert] = "Vous n'avez pas accès à cette démarche" redirect_to root_path end diff --git a/app/helpers/dossier_link_helper.rb b/app/helpers/dossier_link_helper.rb index 15ad75f12..1a3986771 100644 --- a/app/helpers/dossier_link_helper.rb +++ b/app/helpers/dossier_link_helper.rb @@ -1,7 +1,7 @@ module DossierLinkHelper def dossier_linked_path(user, dossier) if user.is_a?(Instructeur) - if dossier.procedure.instructeurs.include?(user) + if dossier.procedure.defaut_groupe_instructeur.instructeurs.include?(user) instructeur_dossier_path(dossier.procedure, dossier) else avis = dossier.avis.find_by(instructeur: user) diff --git a/spec/controllers/admin/procedures_controller_spec.rb b/spec/controllers/admin/procedures_controller_spec.rb index e32b5ab32..0fe3420d4 100644 --- a/spec/controllers/admin/procedures_controller_spec.rb +++ b/spec/controllers/admin/procedures_controller_spec.rb @@ -218,7 +218,7 @@ describe Admin::ProceduresController, type: :controller do describe "admin can also instruct the procedure as a instructeur" do subject { Procedure.last } - it { expect(subject.instructeurs).to include(instructeur) } + it { expect(subject.defaut_groupe_instructeur.instructeurs).to include(instructeur) } end end end diff --git a/spec/factories/procedure.rb b/spec/factories/procedure.rb index ef845b6d7..39f3583c2 100644 --- a/spec/factories/procedure.rb +++ b/spec/factories/procedure.rb @@ -61,8 +61,8 @@ FactoryBot.define do end trait :with_instructeur do - after(:build) do |procedure, _evaluator| - procedure.instructeurs << create(:instructeur) + after(:create) do |procedure, _evaluator| + procedure.defaut_groupe_instructeur.instructeurs << create(:instructeur) end end diff --git a/spec/features/instructeurs/instructeur_spec.rb b/spec/features/instructeurs/instructeur_spec.rb index ddef26937..d6ce007a5 100644 --- a/spec/features/instructeurs/instructeur_spec.rb +++ b/spec/features/instructeurs/instructeur_spec.rb @@ -171,7 +171,7 @@ feature 'The instructeur part' do scenario 'A instructeur can send a dossier to several instructeurs', js: true do instructeur_2 = FactoryBot.create(:instructeur) instructeur_3 = FactoryBot.create(:instructeur) - procedure.instructeurs << [instructeur_2, instructeur_3] + procedure.defaut_groupe_instructeur.instructeurs << [instructeur_2, instructeur_3] send_dossier = double() expect(InstructeurMailer).to receive(:send_dossier).and_return(send_dossier).twice diff --git a/spec/helpers/dossier_link_helper_spec.rb b/spec/helpers/dossier_link_helper_spec.rb index 7190603a7..2c7db4af1 100644 --- a/spec/helpers/dossier_link_helper_spec.rb +++ b/spec/helpers/dossier_link_helper_spec.rb @@ -18,7 +18,7 @@ describe DossierLinkHelper do let(:dossier) { create(:dossier) } let(:instructeur) { create(:instructeur) } - before { dossier.procedure.instructeurs << instructeur } + before { dossier.procedure.defaut_groupe_instructeur.instructeurs << instructeur } it { expect(helper.dossier_linked_path(instructeur, dossier)).to eq(instructeur_dossier_path(dossier.procedure, dossier)) } end diff --git a/spec/models/procedure_spec.rb b/spec/models/procedure_spec.rb index ab4dad111..0ddc65d61 100644 --- a/spec/models/procedure_spec.rb +++ b/spec/models/procedure_spec.rb @@ -401,7 +401,7 @@ describe Procedure do subject { @procedure } it { expect(subject.parent_procedure).to eq(procedure) } - it { expect(subject.instructeurs.pluck(:email)).to eq([administrateur.email]) } + it { expect(subject.defaut_groupe_instructeur.instructeurs.pluck(:email)).to eq([administrateur.email]) } it 'should duplicate specific objects with different id' do expect(subject.id).not_to eq(procedure.id) diff --git a/spec/views/admin/assigns/show.html.haml_spec.rb b/spec/views/admin/assigns/show.html.haml_spec.rb index 3136d94a7..6c0342922 100644 --- a/spec/views/admin/assigns/show.html.haml_spec.rb +++ b/spec/views/admin/assigns/show.html.haml_spec.rb @@ -4,7 +4,7 @@ describe 'admin/assigns/show.html.haml', type: :view do let(:admin) { create(:administrateur) } let(:procedure) { create :procedure, administrateur: admin } - let(:assign_instructeurs) { procedure.instructeurs } + let(:assign_instructeurs) { procedure.defaut_groupe_instructeur.instructeurs } let(:not_assign_instructeurs) { admin.instructeurs.where.not(id: assign_instructeurs.ids) } before do