Procedure has many instructeurs with defaut_groupe_instructeur
This commit is contained in:
parent
ba24958035
commit
b541ef0b41
10 changed files with 11 additions and 11 deletions
|
@ -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,
|
||||
|
|
|
@ -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
|
||||
|
|
|
@ -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
|
||||
|
|
|
@ -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)
|
||||
|
|
|
@ -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
|
||||
|
|
|
@ -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
|
||||
|
||||
|
|
|
@ -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
|
||||
|
|
|
@ -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
|
||||
|
|
|
@ -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)
|
||||
|
|
|
@ -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
|
||||
|
|
Loading…
Reference in a new issue