fix(TypesDeChamp::ConditionValidator): allow to use types_de_champ_public on condition for types_de_champ_private

app/validators/types_de_champ/condition_validator.rb

@@ -1,16 +1,19 @@
 class TypesDeChamp::ConditionValidator < ActiveModel::EachValidator
   def validate_each(procedure, attribute, types_de_champ)
-    public_tdcs = types_de_champ.to_a
+    return if types_de_champ.empty?
-      .flat_map { _1.repetition? ? procedure.draft_revision.children_of(_1) : _1 }
 
-    public_tdcs
+    tdcs = if attribute == :draft_types_de_champ_private
-      .map.with_index
+      procedure.draft_revision.types_de_champ_for
-      .filter_map { |tdc, i| tdc.condition? ? [tdc, i] : nil }
+    else
-      .map do |tdc, i|
+      procedure.draft_revision.types_de_champ_for(scope: :public)
-        [tdc, tdc.condition.errors(public_tdcs.take(i))]
     end
-      .filter { |_tdc, errors| errors.present? }
+
-      .each do |tdc, _error_hash|
+    tdcs.each_with_index do |tdc, i|
+      next unless tdc.condition?
+
+      errors = tdc.condition.errors(tdcs.take(i))
+      next if errors.blank?
+
       procedure.errors.add(
         attribute,
         procedure.errors.generate_message(attribute, :invalid_condition, { value: tdc.libelle }),

spec/models/procedure_spec.rb

@@ -211,7 +211,7 @@ describe Procedure do
       it { is_expected.to allow_value('text').on(:publication).for(:cadre_juridique) }
 
       context 'with deliberation' do
-        let(:procedure) { build(:procedure, cadre_juridique: nil) }
+        let(:procedure) { build(:procedure, cadre_juridique: nil, revisions: [build(:procedure_revision)]) }
 
         it { expect(procedure.valid?(:publication)).to eq(false) }
 
@@ -420,22 +420,37 @@ describe Procedure do
           procedure.validate(:publication)
           expect(procedure.errors.full_messages_for(:draft_types_de_champ_private)).to include(invalid_drop_down_error_message)
         end
-
-        it 'validates that types de champ private condition works types de champ public and private' do
-        end
       end
 
       context 'when condition on champ private use public champ' do
         include Logic
-        let(:types_de_champ_private) { [{ type: :text, condition: ds_eq(champ_value(1), constant(2)) }] }
+        let(:types_de_champ_public) { [{ type: :decimal_number, stable_id: 1 }] }
-        let(:types_de_champ_public) { [{ type: :number, stable_id: 1 }] }
+        let(:types_de_champ_private) { [{ type: :text, condition: ds_eq(champ_value(1), constant(2)), stable_id: 2 }] }
-
         it 'validate without context' do
-          expect(procedure.validate).to be_truthy
+          procedure.validate
+          expect(procedure.errors.full_messages_for(:draft_types_de_champ_private)).to be_empty
         end
 
-        it 'validate with types_de_champ_private_editor' do
+        it 'validate allows condition' do
-          expect(procedure.validate(:types_de_champ_private_editor)).to be_falsey
+          procedure.validate(:types_de_champ_private_editor)
+          expect(procedure.errors.full_messages_for(:draft_types_de_champ_private)).to be_empty
+        end
+      end
+
+      context 'when condition on champ public use private champ' do
+        include Logic
+        let(:types_de_champ_public) { [{ type: :text, libelle: 'condition', condition: ds_eq(champ_value(1), constant(2)), stable_id: 2 }] }
+        let(:types_de_champ_private) { [{ type: :decimal_number, stable_id: 1 }] }
+        let(:error_on_condition) { "Le champ « condition » a une logique conditionnelle invalide" }
+
+        it 'validate without context' do
+          procedure.validate
+          expect(procedure.errors.full_messages_for(:draft_types_de_champ_public)).to be_empty
+        end
+
+        it 'validate prevent condition' do
+          procedure.validate(:types_de_champ_public_editor)
+          expect(procedure.errors.full_messages_for(:draft_types_de_champ_public)).to include(error_on_condition)
         end
       end
     end

spec/services/procedure_export_service_zip_spec.rb

@@ -39,8 +39,7 @@ describe ProcedureExportService do
             ActiveSupport::Notifications.subscribed(callback, "sql.active_record") do
               subject
             end
-
+            expect(sql_count <= 62).to be_truthy
-            expect(sql_count <= 58).to be_truthy
 
             dossier = dossiers.first