DGNum/demarches-normaliennes
13
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

chore(groupe_instructeurs): do not reaffect bulk messages anymore

Browse source
This commit is contained in:
Eric Leroy-Terquem 2024-03-22 14:56:54 +01:00
parent 60f7e02429
commit 773de2dbf2
2 changed files with 0 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
app/controllers/administrateurs/groupe_instructeurs_controller.rb
View file

@ -199,19 +199,8 @@ module Administrateurs
.without_group(@groupe_instructeur)
end
def reaffecter_bulk_messages(target_group)
bulk_messages = BulkMessage.joins(:groupe_instructeurs).where(groupe_instructeurs: { id: groupe_instructeur.id })
bulk_messages.each do |bulk_message|
bulk_message.groupe_instructeurs.delete(groupe_instructeur)
if !bulk_message.groupe_instructeur_ids.include?(target_group.id)
bulk_message.groupe_instructeurs << target_group
end
end
end
def reaffecter
target_group = procedure.groupe_instructeurs.find(params[:target_group])
reaffecter_bulk_messages(target_group)
groupe_instructeur.dossiers.find_each do |dossier|
dossier.assign_to_groupe_instructeur(target_group, DossierAssignment.modes.fetch(:manual), current_administrateur)
end

5
spec/controllers/administrateurs/groupe_instructeurs_controller_spec.rb
View file

@ -183,7 +183,6 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
let!(:gi_1_3) { create(:groupe_instructeur, label: 'groupe instructeur 3', procedure: procedure) }
let!(:dossier12) { create(:dossier, :en_construction, :with_individual, procedure: procedure, groupe_instructeur: gi_1_1) }
let!(:instructeur) { create(:instructeur) }
let!(:bulk_message) { BulkMessage.create(dossier_count: 2, dossier_state: "en_construction", body: "hello", sent_at: Time.zone.now, groupe_instructeurs: [gi_1_1, gi_1_3], instructeur: instructeur, procedure: procedure) }
describe 'when the new group is a group of the procedure' do
before do
@ -194,7 +193,6 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
target_group: gi_1_2.id
}
dossier12.reload
bulk_message.reload
end
it { expect(response).to redirect_to(admin_procedure_groupe_instructeurs_path(procedure)) }
@ -203,7 +201,6 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
it { expect(dossier12.dossier_assignment.dossier_id).to be(dossier12.id) }
it { expect(dossier12.dossier_assignment.groupe_instructeur_id).to be(gi_1_2.id) }
it { expect(dossier12.dossier_assignment.assigned_by).to eq(admin.email) }
it { expect(bulk_message.groupe_instructeurs).to contain_exactly(gi_1_2, gi_1_3) }
end
describe 'when the target group is not a possible group' do
@ -218,11 +215,9 @@ describe Administrateurs::GroupeInstructeursController, type: :controller do
}
before do
dossier12.reload
bulk_message.reload
end
it { expect { subject }.to raise_error(ActiveRecord::RecordNotFound) }
it { expect(bulk_message.groupe_instructeurs).to match_array([gi_1_1, gi_1_3]) }
end
end