From 0934a672fe955e1160bdd71378df2cb6c872e040 Mon Sep 17 00:00:00 2001 From: simon lehericey Date: Wed, 26 Apr 2023 10:52:31 +0200 Subject: [PATCH] fix: defaut_groupe_instructeur for a cloned procedure from another admin without a group with default label --- app/models/procedure.rb | 3 ++- spec/models/procedure_spec.rb | 10 ++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/app/models/procedure.rb b/app/models/procedure.rb index fde527da6..019c0cd09 100644 --- a/app/models/procedure.rb +++ b/app/models/procedure.rb @@ -580,7 +580,8 @@ class Procedure < ApplicationRecord procedure.draft_revision.types_de_champ_public.each { |tdc| tdc.options&.delete(:old_pj) } end - new_defaut_groupe = procedure.groupe_instructeurs.find_by(label: defaut_groupe_instructeur.label) + new_defaut_groupe = procedure.groupe_instructeurs + .find_by(label: defaut_groupe_instructeur.label) || procedure.groupe_instructeurs.first procedure.update!(defaut_groupe_instructeur: new_defaut_groupe) procedure diff --git a/spec/models/procedure_spec.rb b/spec/models/procedure_spec.rb index d40f138c4..a61564470 100644 --- a/spec/models/procedure_spec.rb +++ b/spec/models/procedure_spec.rb @@ -622,6 +622,16 @@ describe Procedure do let(:administrateur) { create(:administrateur) } let(:opendata) { false } + context 'and the procedure does not have a groupe with the defaut label' do + before do + procedure.defaut_groupe_instructeur.update!(label: 'another label') + end + + it "affects the first groupe as the defaut groupe" do + expect(subject.defaut_groupe_instructeur).to eq(subject.groupe_instructeurs.first) + end + end + it 'should not clone service' do expect(subject.service).to eq(nil) end