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

fix(dossier): do not try to rebase if procedure is not published

Browse source
This commit is contained in:
Paul Chavard 2023-02-01 17:33:23 +01:00
parent fc31ef6da3
commit 2971100081
2 changed files with 13 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/models/concerns/dossier_rebase_concern.rb
View file

@ -14,12 +14,12 @@ module DossierRebaseConcern
end end
def can_rebase? def can_rebase?
revision != procedure.published_revision && procedure.published_revision.present? && revision != procedure.published_revision &&
(brouillon? || accepted_en_construction_changes? || accepted_en_instruction_changes?) (brouillon? || accepted_en_construction_changes? || accepted_en_instruction_changes?)
end end
def pending_changes def pending_changes
revision.compare(procedure.published_revision) procedure.published_revision.present? ? revision.compare(procedure.published_revision) : []
end end
def can_rebase_mandatory_change?(stable_id) def can_rebase_mandatory_change?(stable_id)

11
spec/models/concern/dossier_rebase_concern_spec.rb
View file

@ -6,6 +6,17 @@ describe DossierRebaseConcern do
let(:private_type_de_champ) { procedure.active_revision.types_de_champ_private.first } let(:private_type_de_champ) { procedure.active_revision.types_de_champ_private.first }
let(:mandatory_type_de_champ) { procedure.active_revision.types_de_champ_public.find(&:mandatory?) } let(:mandatory_type_de_champ) { procedure.active_revision.types_de_champ_public.find(&:mandatory?) }
context 'on unpublished procedure' do
context 'en_construction' do
let(:dossier) { create(:dossier, :en_construction, procedure: procedure) }
it 'should be false' do
expect(dossier.pending_changes).to be_empty
expect(dossier.can_rebase?).to be_falsey
end
end
end
context 'en_construction' do context 'en_construction' do
let(:dossier) { create(:dossier, :en_construction, procedure: procedure) } let(:dossier) { create(:dossier, :en_construction, procedure: procedure) }