Procedure: required juridique stuff only if needed

This commit is contained in:
simon lehericey 2018-06-01 10:51:04 +02:00 committed by Frederic Merizen
parent 7eb34053f8
commit 1f0e46a4ff
2 changed files with 7 additions and 1 deletions

View file

@ -357,7 +357,7 @@ class Procedure < ApplicationRecord
end end
def check_juridique def check_juridique
if cadre_juridique.blank? && !deliberation.attached? if juridique_required? && (cadre_juridique.blank? && !deliberation.attached?)
errors.add(:cadre_juridique, " : veuillez remplir le texte de loi ou la délibération") errors.add(:cadre_juridique, " : veuillez remplir le texte de loi ou la délibération")
end end
end end

View file

@ -193,6 +193,12 @@ describe Procedure do
end end
end end
end end
context 'when juridique_required is false' do
let(:procedure) { build(:procedure, juridique_required: false, cadre_juridique: nil) }
it { expect(procedure.valid?).to eq(true) }
end
end end
describe '#types_de_champ_ordered' do describe '#types_de_champ_ordered' do