From c5008e42af3d7ed79d39fa59f15c744602e9b235 Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Mon, 24 Jun 2024 16:46:49 +0200
Subject: [PATCH 1/3] chore: disable partial inserts so we can change default
 column values

---
 config/application.rb | 1 +
 1 file changed, 1 insertion(+)

diff --git a/config/application.rb b/config/application.rb
index 638dfa89e..19eaacf06 100644
--- a/config/application.rb
+++ b/config/application.rb
@@ -103,6 +103,7 @@ module TPS
 
     config.active_record.encryption.primary_key = Rails.application.secrets.active_record_encryption.fetch(:primary_key)
     config.active_record.encryption.key_derivation_salt = Rails.application.secrets.active_record_encryption.fetch(:key_derivation_salt)
+    config.active_record.partial_inserts = false
 
     config.exceptions_app = self.routes
 

From 48e5a237e37c568853a8e41eedc3275d26af6e73 Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Tue, 25 Jun 2024 17:17:21 +0200
Subject: [PATCH 2/3] chore(schema): change default types_de_champ#mandatory
 from false to true

---
 app/models/type_de_champ.rb                                | 2 +-
 ...240619205011_change_types_de_champ_mandatory_default.rb | 7 +++++++
 db/schema.rb                                               | 4 ++--
 3 files changed, 10 insertions(+), 3 deletions(-)
 create mode 100644 db/migrate/20240619205011_change_types_de_champ_mandatory_default.rb

diff --git a/app/models/type_de_champ.rb b/app/models/type_de_champ.rb
index cf2c321d8..5ee6bc037 100644
--- a/app/models/type_de_champ.rb
+++ b/app/models/type_de_champ.rb
@@ -261,7 +261,7 @@ class TypeDeChamp < ApplicationRecord
   end
 
   def check_mandatory
-    if non_fillable?
+    if non_fillable? || private?
       self.mandatory = false
     else
       true
diff --git a/db/migrate/20240619205011_change_types_de_champ_mandatory_default.rb b/db/migrate/20240619205011_change_types_de_champ_mandatory_default.rb
new file mode 100644
index 000000000..31a5f2883
--- /dev/null
+++ b/db/migrate/20240619205011_change_types_de_champ_mandatory_default.rb
@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class ChangeTypesDeChampMandatoryDefault < ActiveRecord::Migration[7.0]
+  def change
+    change_column_default :types_de_champ, :mandatory, from: false, to: true
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 28c317cac..8e70f8121 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.0].define(version: 2024_05_27_090508) do
+ActiveRecord::Schema[7.0].define(version: 2024_06_19_205011) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "pg_buffercache"
   enable_extension "pg_stat_statements"
@@ -1122,7 +1122,7 @@ ActiveRecord::Schema[7.0].define(version: 2024_05_27_090508) do
     t.datetime "created_at", precision: nil
     t.text "description"
     t.string "libelle"
-    t.boolean "mandatory", default: false
+    t.boolean "mandatory", default: true
     t.jsonb "options"
     t.boolean "private", default: false, null: false
     t.bigint "stable_id"

From f40b71f450e2dbe8f880ac6464e31389367ba239 Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Tue, 25 Jun 2024 17:17:51 +0200
Subject: [PATCH 3/3] test(type_de_champ): mandatory by default, like schema

---
 .../users/dossiers_controller_spec.rb         |  6 ++--
 spec/factories/type_de_champ.rb               |  2 +-
 spec/helpers/procedure_helper_spec.rb         |  2 +-
 .../batch_update_datetime_values_job_spec.rb  |  4 +++
 .../batch_update_pays_values_job_spec.rb      |  4 +++
 spec/lib/recovery/dossier_life_cycle_spec.rb  |  2 +-
 .../linked_drop_down_list_champ_spec.rb       |  2 +-
 .../concerns/dossier_rebase_concern_spec.rb   | 14 ++++----
 spec/models/dossier_spec.rb                   |  8 ++---
 spec/models/procedure_revision_spec.rb        |  5 +--
 spec/models/procedure_spec.rb                 | 16 +++++----
 ...inked_drop_down_list_type_de_champ_spec.rb | 36 ++++++++++++++-----
 ...demarches_publiques_export_service_spec.rb |  2 +-
 .../pieces_justificatives_service_spec.rb     |  6 ++--
 .../administrateurs/types_de_champ_spec.rb    |  6 ++--
 .../api_particulier/api_particulier_spec.rb   |  8 ++---
 .../routing/rules_full_scenario_spec.rb       |  4 +--
 spec/system/users/brouillon_spec.rb           | 16 ++++-----
 .../users/dossier_ineligibilite_spec.rb       |  4 +--
 19 files changed, 89 insertions(+), 58 deletions(-)

diff --git a/spec/controllers/users/dossiers_controller_spec.rb b/spec/controllers/users/dossiers_controller_spec.rb
index 2a38eb7d7..f09fc4f47 100644
--- a/spec/controllers/users/dossiers_controller_spec.rb
+++ b/spec/controllers/users/dossiers_controller_spec.rb
@@ -399,7 +399,7 @@ describe Users::DossiersController, type: :controller do
   describe '#submit_brouillon' do
     before { sign_in(user) }
     let(:procedure) { create(:procedure, :published, types_de_champ_public:) }
-    let(:types_de_champ_public) { [{ type: :text }] }
+    let(:types_de_champ_public) { [{ type: :text, mandatory: false }] }
     let!(:dossier) { create(:dossier, user:, procedure:) }
     let(:first_champ) { dossier.champs_public.first }
     let(:anchor_to_first_champ) { controller.helpers.link_to first_champ.libelle, brouillon_dossier_path(anchor: first_champ.labelledby_id), class: 'error-anchor' }
@@ -516,7 +516,7 @@ describe Users::DossiersController, type: :controller do
   describe '#submit_en_construction' do
     before { sign_in(user) }
     let(:procedure) { create(:procedure, :published, types_de_champ_public:) }
-    let(:types_de_champ_public) { [{ type: :text }] }
+    let(:types_de_champ_public) { [{ type: :text, mandatory: false }] }
     let(:dossier) { create(:dossier, :en_construction, procedure:, user:) }
     let(:first_champ) { dossier.owner_editing_fork.champs_public.first }
     let(:anchor_to_first_champ) { controller.helpers.link_to I18n.t('views.users.dossiers.fix_champ'), modifier_dossier_path(anchor: first_champ.labelledby_id), class: 'error-anchor' }
@@ -661,7 +661,7 @@ describe Users::DossiersController, type: :controller do
     before { sign_in(user) }
 
     let(:procedure) { create(:procedure, :published, types_de_champ_public:) }
-    let(:types_de_champ_public) { [{}, { type: :piece_justificative }] }
+    let(:types_de_champ_public) { [{}, { type: :piece_justificative, mandatory: false }] }
     let(:dossier) { create(:dossier, user:, procedure:) }
     let(:first_champ) { dossier.champs_public.first }
     let(:piece_justificative_champ) { dossier.champs_public.last }
diff --git a/spec/factories/type_de_champ.rb b/spec/factories/type_de_champ.rb
index d381cdff7..98eb2c840 100644
--- a/spec/factories/type_de_champ.rb
+++ b/spec/factories/type_de_champ.rb
@@ -3,8 +3,8 @@ FactoryBot.define do
     sequence(:libelle) { |n| "Libelle du champ #{n}" }
     sequence(:description) { |n| "description du champ #{n}" }
     type_champ { TypeDeChamp.type_champs.fetch(:text) }
-    mandatory { false }
     add_attribute(:private) { false }
+    mandatory { !private }
 
     transient do
       procedure { nil }
diff --git a/spec/helpers/procedure_helper_spec.rb b/spec/helpers/procedure_helper_spec.rb
index e7f966acd..90acfc3c5 100644
--- a/spec/helpers/procedure_helper_spec.rb
+++ b/spec/helpers/procedure_helper_spec.rb
@@ -37,7 +37,7 @@ RSpec.describe ProcedureHelper, type: :helper do
       let(:procedure) { create(:procedure, types_de_champ_public: [{ type: :yes_no }, { type: :piece_justificative }]) }
 
       it 'rounds up the duration to the minute' do
-        expect(subject).to eq(2)
+        expect(subject).to eq(3)
       end
     end
 
diff --git a/spec/jobs/migrations/batch_update_datetime_values_job_spec.rb b/spec/jobs/migrations/batch_update_datetime_values_job_spec.rb
index 28d012871..c800d97c6 100644
--- a/spec/jobs/migrations/batch_update_datetime_values_job_spec.rb
+++ b/spec/jobs/migrations/batch_update_datetime_values_job_spec.rb
@@ -28,6 +28,10 @@ describe Migrations::BatchUpdateDatetimeValuesJob, type: :job do
   end
 
   context "when the value is a date not convertible to IS8061" do
+    before do
+      datetime_champ.type_de_champ.update!(mandatory: false)
+    end
+
     let!(:datetime_champ) { build(:champ_datetime, value: "blabla") }
 
     subject { described_class.perform_now([datetime_champ.id]) }
diff --git a/spec/jobs/migrations/batch_update_pays_values_job_spec.rb b/spec/jobs/migrations/batch_update_pays_values_job_spec.rb
index 65670e49c..7132a532c 100644
--- a/spec/jobs/migrations/batch_update_pays_values_job_spec.rb
+++ b/spec/jobs/migrations/batch_update_pays_values_job_spec.rb
@@ -12,6 +12,10 @@ describe Migrations::BatchUpdatePaysValuesJob, type: :job do
   end
 
   context "the value is incorrect" do
+    before do
+      pays_champ.type_de_champ.update!(mandatory: false)
+    end
+
     let(:pays_champ) { create(:champ_pays).tap { _1.update_columns(value: 'Incorrect') } }
 
     it 'updates value to nil' do
diff --git a/spec/lib/recovery/dossier_life_cycle_spec.rb b/spec/lib/recovery/dossier_life_cycle_spec.rb
index 8cc25fbbf..daae7c868 100644
--- a/spec/lib/recovery/dossier_life_cycle_spec.rb
+++ b/spec/lib/recovery/dossier_life_cycle_spec.rb
@@ -3,7 +3,7 @@ describe 'Dossier::Recovery::LifeCycle' do
     let(:procedure) do
       create(:procedure,
              types_de_champ_public: [
-               { type: :repetition, children: [{ type: :piece_justificative }] },
+               { type: :repetition, children: [{ type: :piece_justificative }], mandatory: false },
                { type: :carte },
                { type: :siret }
              ])
diff --git a/spec/models/champs/linked_drop_down_list_champ_spec.rb b/spec/models/champs/linked_drop_down_list_champ_spec.rb
index 88fcd9ccb..32c464683 100644
--- a/spec/models/champs/linked_drop_down_list_champ_spec.rb
+++ b/spec/models/champs/linked_drop_down_list_champ_spec.rb
@@ -81,7 +81,7 @@ describe Champs::LinkedDropDownListChamp do
     subject { described_class.new(type_de_champ: type_de_champ) }
 
     context 'when the champ is not mandatory' do
-      let(:type_de_champ) { build(:type_de_champ_linked_drop_down_list, drop_down_list_value: value) }
+      let(:type_de_champ) { build(:type_de_champ_linked_drop_down_list, mandatory: false, drop_down_list_value: value) }
 
       it 'blank is fine' do
         is_expected.not_to be_mandatory_blank
diff --git a/spec/models/concerns/dossier_rebase_concern_spec.rb b/spec/models/concerns/dossier_rebase_concern_spec.rb
index 357329898..655e463f6 100644
--- a/spec/models/concerns/dossier_rebase_concern_spec.rb
+++ b/spec/models/concerns/dossier_rebase_concern_spec.rb
@@ -1,6 +1,6 @@
 describe DossierRebaseConcern do
   describe '#can_rebase?' do
-    let(:procedure) { create(:procedure, types_de_champ_public: [{ mandatory: true }, { type: :yes_no }], types_de_champ_private: [{}]) }
+    let(:procedure) { create(:procedure, types_de_champ_public: [{ mandatory: true }, { type: :yes_no, mandatory: false }], types_de_champ_private: [{}]) }
     let(:attestation_template) { procedure.draft_revision.attestation_template.find_or_revise! }
     let(:type_de_champ) { procedure.active_revision.types_de_champ_public.find { |tdc| !tdc.mandatory? } }
     let(:private_type_de_champ) { procedure.active_revision.types_de_champ_private.first }
@@ -26,11 +26,12 @@ describe DossierRebaseConcern do
         dossier
       end
 
-      context 'with added type de champ' do
+      context 'with added non mandatory type de champ' do
         before do
           procedure.draft_revision.add_type_de_champ({
             type_champ: TypeDeChamp.type_champs.fetch(:text),
-            libelle: "Un champ text"
+            libelle: "Un champ text",
+            mandatory: false
           })
           procedure.publish_revision!
           dossier.reload
@@ -125,7 +126,7 @@ describe DossierRebaseConcern do
       end
 
       context 'with type de champ regexp and regexp change' do
-        let(:procedure) { create(:procedure, types_de_champ_public: [{ mandatory: true }, { type: :expression_reguliere }], types_de_champ_private: [{}]) }
+        let(:procedure) { create(:procedure, types_de_champ_public: [{ mandatory: true }, { type: :expression_reguliere, mandatory: false }], types_de_champ_private: [{}]) }
 
         before do
           procedure.draft_revision.find_and_ensure_exclusive_use(type_de_champ.stable_id).update(expression_reguliere: /\d+/)
@@ -162,11 +163,12 @@ describe DossierRebaseConcern do
         dossier
       end
 
-      context 'with added type de champ' do
+      context 'with added non mandatory type de champ' do
         before do
           procedure.draft_revision.add_type_de_champ({
             type_champ: TypeDeChamp.type_champs.fetch(:text),
-            libelle: "Un champ text"
+            libelle: "Un champ text",
+            mandatory: false
           })
           procedure.publish_revision!
           dossier.reload
diff --git a/spec/models/dossier_spec.rb b/spec/models/dossier_spec.rb
index 76fb5414a..7413168e8 100644
--- a/spec/models/dossier_spec.rb
+++ b/spec/models/dossier_spec.rb
@@ -1552,8 +1552,8 @@ describe Dossier, type: :model do
 
     let(:procedure) { create(:procedure, types_de_champ_public: types_de_champ) }
     let(:dossier) { create(:dossier, procedure: procedure) }
-    let(:types_de_champ) { [type_de_champ] }
-    let(:type_de_champ) { {} }
+    let(:types_de_champ) { [type_de_champ].compact }
+    let(:type_de_champ) { nil }
     let(:errors) { dossier.check_mandatory_and_visible_champs }
 
     it 'no mandatory champs' do
@@ -1575,7 +1575,7 @@ describe Dossier, type: :model do
       end
 
       context "conditionaly visible" do
-        let(:types_de_champ) { [{ type: :yes_no, stable_id: 99 }, type_de_champ] }
+        let(:types_de_champ) { [{ type: :yes_no, stable_id: 99, mandatory: false }, type_de_champ] }
         let(:type_de_champ) { { mandatory: true, condition: ds_eq(champ_value(99), constant(true)) } }
 
         it 'should not have errors' do
@@ -1635,7 +1635,7 @@ describe Dossier, type: :model do
 
         context "conditionaly visible" do
           let(:champ_with_error) { dossier.champs_public.second.champs.first }
-          let(:types_de_champ) { [{ type: :yes_no, stable_id: 99 }, type_de_champ] }
+          let(:types_de_champ) { [{ type: :yes_no, stable_id: 99, mandatory: false }, type_de_champ] }
           let(:type_de_champ) { { type: :repetition, mandatory: true, children: [{ mandatory: true }], condition: ds_eq(champ_value(99), constant(true)) } }
 
           it 'should not have errors' do
diff --git a/spec/models/procedure_revision_spec.rb b/spec/models/procedure_revision_spec.rb
index 434d82e44..4b17da875 100644
--- a/spec/models/procedure_revision_spec.rb
+++ b/spec/models/procedure_revision_spec.rb
@@ -439,6 +439,7 @@ describe ProcedureRevision do
         let(:new_tdc) do
           new_draft.add_type_de_champ(
             type_champ: TypeDeChamp.type_champs.fetch(:text),
+            mandatory: false,
             libelle: "Un champ text"
           )
         end
@@ -493,8 +494,8 @@ describe ProcedureRevision do
                 attribute: :mandatory,
                 label: first_tdc.libelle,
                 private: false,
-                from: false,
-                to: true,
+                from: true,
+                to: false,
                 stable_id: first_tdc.stable_id
               }
             ])
diff --git a/spec/models/procedure_spec.rb b/spec/models/procedure_spec.rb
index 971683eb6..ed5f50118 100644
--- a/spec/models/procedure_spec.rb
+++ b/spec/models/procedure_spec.rb
@@ -2,12 +2,14 @@ describe Procedure do
   describe 'mail templates' do
     subject { create(:procedure) }
 
-    it { expect(subject.passer_en_construction_email_template).to be_a(Mails::InitiatedMail) }
-    it { expect(subject.passer_en_instruction_email_template).to be_a(Mails::ReceivedMail) }
-    it { expect(subject.accepter_email_template).to be_a(Mails::ClosedMail) }
-    it { expect(subject.refuser_email_template).to be_a(Mails::RefusedMail) }
-    it { expect(subject.classer_sans_suite_email_template).to be_a(Mails::WithoutContinuationMail) }
-    it { expect(subject.repasser_en_instruction_email_template).to be_a(Mails::ReInstructedMail) }
+    it "returns expected classes" do
+      expect(subject.passer_en_construction_email_template).to be_a(Mails::InitiatedMail)
+      expect(subject.passer_en_instruction_email_template).to be_a(Mails::ReceivedMail)
+      expect(subject.accepter_email_template).to be_a(Mails::ClosedMail)
+      expect(subject.refuser_email_template).to be_a(Mails::RefusedMail)
+      expect(subject.classer_sans_suite_email_template).to be_a(Mails::WithoutContinuationMail)
+      expect(subject.repasser_en_instruction_email_template).to be_a(Mails::ReInstructedMail)
+    end
   end
 
   describe 'compute_dossiers_count' do
@@ -1660,7 +1662,7 @@ describe Procedure do
             children: [
               { libelle: 'Nom', mandatory: true },
               { libelle: 'Prénom', mandatory: true },
-              { libelle: 'Age', type: :integer_number }
+              { libelle: 'Age', type: :integer_number, mandatory: false }
             ]
           }
         ]
diff --git a/spec/models/types_de_champ/linked_drop_down_list_type_de_champ_spec.rb b/spec/models/types_de_champ/linked_drop_down_list_type_de_champ_spec.rb
index 8bab608e5..d9acb8ccb 100644
--- a/spec/models/types_de_champ/linked_drop_down_list_type_de_champ_spec.rb
+++ b/spec/models/types_de_champ/linked_drop_down_list_type_de_champ_spec.rb
@@ -89,17 +89,35 @@ describe TypesDeChamp::LinkedDropDownListTypeDeChamp do
         END_OPTIONS
       end
 
-      it do
-        expect(subject.secondary_options).to eq(
-          {
-            '' => [],
-            'Primary 1' => ['', 'secondary 1.1', 'secondary 1.2'],
-            'Primary 2' => ['', 'secondary 2.1', 'secondary 2.2', 'secondary 2.3']
-          }
-        )
+      context "mandatory tdc" do
+        it do
+          expect(subject.secondary_options).to eq(
+            {
+              '' => [],
+              'Primary 1' => ['secondary 1.1', 'secondary 1.2'],
+              'Primary 2' => ['secondary 2.1', 'secondary 2.2', 'secondary 2.3']
+            }
+          )
+        end
+
+        it { expect(subject.primary_options).to eq(['Primary 1', 'Primary 2']) }
       end
 
-      it { expect(subject.primary_options).to eq(['', 'Primary 1', 'Primary 2']) }
+      context "not mandatory" do
+        let(:type_de_champ) { build(:type_de_champ_linked_drop_down_list, drop_down_list_value: menu_options, mandatory: false) }
+
+        it do
+          expect(subject.secondary_options).to eq(
+            {
+              '' => [],
+              'Primary 1' => ['', 'secondary 1.1', 'secondary 1.2'],
+              'Primary 2' => ['', 'secondary 2.1', 'secondary 2.2', 'secondary 2.3']
+            }
+          )
+        end
+
+        it { expect(subject.primary_options).to eq(['', 'Primary 1', 'Primary 2']) }
+      end
     end
   end
 end
diff --git a/spec/services/demarches_publiques_export_service_spec.rb b/spec/services/demarches_publiques_export_service_spec.rb
index f29dc79ad..9e63003a4 100644
--- a/spec/services/demarches_publiques_export_service_spec.rb
+++ b/spec/services/demarches_publiques_export_service_spec.rb
@@ -33,7 +33,7 @@ describe DemarchesPubliquesExportService do
             {
               description: procedure.active_revision.types_de_champ_public.first.description,
               label: procedure.active_revision.types_de_champ_public.first.libelle,
-              required: false,
+              required: true,
               __typename: "TextChampDescriptor"
             }
           ]
diff --git a/spec/services/pieces_justificatives_service_spec.rb b/spec/services/pieces_justificatives_service_spec.rb
index 341cb33ad..5d731e76a 100644
--- a/spec/services/pieces_justificatives_service_spec.rb
+++ b/spec/services/pieces_justificatives_service_spec.rb
@@ -1,6 +1,6 @@
 describe PiecesJustificativesService do
   describe 'pjs_for_champs' do
-    let(:procedure) { create(:procedure, types_de_champ_public: [{ type: :piece_justificative }, { type: :repetition, children: [{ type: :piece_justificative }] }]) }
+    let(:procedure) { create(:procedure, types_de_champ_public: [{ type: :piece_justificative, mandatory: false }, { type: :repetition, mandatory: false, children: [{ type: :piece_justificative, mandatory: false }] }]) }
     let(:dossier) { create(:dossier, procedure: procedure) }
     let(:dossiers) { Dossier.where(id: dossier.id) }
     let(:witness) { create(:dossier, procedure: procedure) }
@@ -477,8 +477,8 @@ describe PiecesJustificativesService do
     let(:user_profile) { build(:administrateur) }
     let(:types_de_champ_public) do
       [
-        { type: :repetition, children: [{ type: :piece_justificative }] },
-        { type: :repetition, children: [{ type: :piece_justificative }, { type: :piece_justificative }] }
+        { type: :repetition, mandatory: false, children: [{ type: :piece_justificative }] },
+        { type: :repetition, mandatory: false, children: [{ type: :piece_justificative }, { type: :piece_justificative }] }
       ]
     end
 
diff --git a/spec/system/administrateurs/types_de_champ_spec.rb b/spec/system/administrateurs/types_de_champ_spec.rb
index c63335e8a..9f06e0356 100644
--- a/spec/system/administrateurs/types_de_champ_spec.rb
+++ b/spec/system/administrateurs/types_de_champ_spec.rb
@@ -177,11 +177,11 @@ describe 'As an administrateur I can edit types de champ', js: true do
       # It displays the estimate when adding a new champ
       add_champ
       select('Pièce justificative', from: 'Type de champ')
-      expect(page).to have_content('Durée de remplissage estimée : 2 min')
+      expect(page).to have_content('Durée de remplissage estimée : 3 min')
 
       # It updates the estimate when updating the champ
-      check 'Champ obligatoire'
-      expect(page).to have_content('Durée de remplissage estimée : 3 min')
+      uncheck 'Champ obligatoire'
+      expect(page).to have_content('Durée de remplissage estimée : 2 min')
 
       # It updates the estimate when removing the champ
       page.accept_alert do
diff --git a/spec/system/api_particulier/api_particulier_spec.rb b/spec/system/api_particulier/api_particulier_spec.rb
index aa06327db..0a9b8f81b 100644
--- a/spec/system/api_particulier/api_particulier_spec.rb
+++ b/spec/system/api_particulier/api_particulier_spec.rb
@@ -252,10 +252,10 @@ describe 'fetch API Particulier Data', js: true do
              instructeurs: [instructeur],
              api_particulier_sources: expected_sources,
              api_particulier_token: api_particulier_token).tap do |p|
-               p.active_revision.add_type_de_champ(type_champ: :cnaf, libelle: 'cnaf')
-               p.active_revision.add_type_de_champ(type_champ: :dgfip, libelle: 'dgfip')
-               p.active_revision.add_type_de_champ(type_champ: :pole_emploi, libelle: 'pole_emploi')
-               p.active_revision.add_type_de_champ(type_champ: :mesri, libelle: 'mesri')
+               p.active_revision.add_type_de_champ(type_champ: :cnaf, libelle: 'cnaf', mandatory: false)
+               p.active_revision.add_type_de_champ(type_champ: :dgfip, libelle: 'dgfip', mandatory: false)
+               p.active_revision.add_type_de_champ(type_champ: :pole_emploi, libelle: 'pole_emploi', mandatory: false)
+               p.active_revision.add_type_de_champ(type_champ: :mesri, libelle: 'mesri', mandatory: false)
              end
     end
 
diff --git a/spec/system/routing/rules_full_scenario_spec.rb b/spec/system/routing/rules_full_scenario_spec.rb
index 90c0cdf49..41fa7f54e 100644
--- a/spec/system/routing/rules_full_scenario_spec.rb
+++ b/spec/system/routing/rules_full_scenario_spec.rb
@@ -3,8 +3,8 @@ describe 'The routing with rules', js: true do
 
   let(:procedure) do
     create(:procedure, :with_service, :for_individual, :with_zone, types_de_champ_public: [
-      { type: :text, libelle: 'un premier champ text' },
-      { type: :drop_down_list, libelle: 'Spécialité', options: ["", "littéraire", "scientifique", "artistique"] }
+      { type: :text, libelle: 'un premier champ text', mandatory: false },
+      { type: :drop_down_list, libelle: 'Spécialité', options: ["", "littéraire", "scientifique", "artistique"], mandatory: false }
     ])
   end
   let(:administrateur) { create(:administrateur, procedures: [procedure]) }
diff --git a/spec/system/users/brouillon_spec.rb b/spec/system/users/brouillon_spec.rb
index 97498b83b..6f5a65135 100644
--- a/spec/system/users/brouillon_spec.rb
+++ b/spec/system/users/brouillon_spec.rb
@@ -410,7 +410,7 @@ describe 'The user' do
       let(:procedure) do
         create(:procedure, :published, :for_individual,
           types_de_champ_public: [
-            { type: :integer_number, libelle: 'age', stable_id: },
+            { type: :integer_number, libelle: 'age', mandatory: false, stable_id: },
             {
               type: :repetition, libelle: 'repetition', condition:, children: [
                 { type: :text, libelle: 'nom', mandatory: true }
@@ -439,7 +439,7 @@ describe 'The user' do
       let(:procedure) do
         create(:procedure, :published, :for_individual,
           types_de_champ_public: [
-            { type: :checkbox, libelle: 'champ_a', stable_id: a_stable_id },
+            { type: :checkbox, libelle: 'champ_a', mandatory: false, stable_id: a_stable_id },
             {
               type: :repetition, libelle: 'repetition', mandatory: true, children: [
                 { type: :checkbox, libelle: 'champ_b', stable_id: b_stable_id },
@@ -476,7 +476,7 @@ describe 'The user' do
       let(:procedure) do
         create(:procedure, :published, :for_individual,
           types_de_champ_public: [
-            { type: :integer_number, libelle: 'age', stable_id: },
+            { type: :integer_number, libelle: 'age', mandatory: false, stable_id: },
             { type: :text, libelle: 'nom', mandatory: true, condition: }
           ])
       end
@@ -514,11 +514,11 @@ describe 'The user' do
       let(:procedure) do
         create(:procedure, :published, :for_individual,
           types_de_champ_public: [
-            { type: :integer_number, libelle: 'age du candidat', stable_id: age_stable_id },
-            { type: :yes_no, libelle: 'permis de conduire', stable_id: permis_stable_id, condition: permis_condition },
-            { type: :header_section, libelle: 'info voiture', condition: permis_condition },
-            { type: :integer_number, libelle: 'tonnage', stable_id: tonnage_stable_id, condition: tonnage_condition },
-            { type: :text, libelle: 'parking', condition: parking_condition }
+            { type: :integer_number, libelle: 'age du candidat', stable_id: age_stable_id, mandatory: false },
+            { type: :yes_no, libelle: 'permis de conduire', stable_id: permis_stable_id, condition: permis_condition, mandatory: false },
+            { type: :header_section, libelle: 'info voiture', condition: permis_condition, mandatory: false },
+            { type: :integer_number, libelle: 'tonnage', stable_id: tonnage_stable_id, condition: tonnage_condition, mandatory: false },
+            { type: :text, libelle: 'parking', condition: parking_condition, mandatory: false }
           ])
       end
 
diff --git a/spec/system/users/dossier_ineligibilite_spec.rb b/spec/system/users/dossier_ineligibilite_spec.rb
index 9b9fa9de6..045b77983 100644
--- a/spec/system/users/dossier_ineligibilite_spec.rb
+++ b/spec/system/users/dossier_ineligibilite_spec.rb
@@ -64,7 +64,7 @@ describe 'Dossier Inéligibilité', js: true do
   end
 
   describe 'ineligibilite_rules with a Or' do
-    let(:types_de_champ_public) { [{ type: :yes_no, libelle: 'l1' }, { type: :drop_down_list, libelle: 'l2', options: ['Paris', 'Marseille'] }] }
+    let(:types_de_champ_public) { [{ type: :yes_no, libelle: 'l1' }, { type: :drop_down_list, libelle: 'l2', options: ['Paris', 'Marseille'], mandatory: false }] }
     let(:ineligibilite_rules) do
       ds_or([
         ds_eq(champ_value(first_tdc.stable_id), constant(true)),
@@ -134,7 +134,7 @@ describe 'Dossier Inéligibilité', js: true do
   end
 
   describe 'ineligibilite_rules with a And and all visible champs' do
-    let(:types_de_champ_public) { [{ type: :yes_no, libelle: 'l1' }, { type: :drop_down_list, libelle: 'l2', options: ['Paris', 'Marseille'] }] }
+    let(:types_de_champ_public) { [{ type: :yes_no, libelle: 'l1' }, { type: :drop_down_list, libelle: 'l2', options: ['Paris', 'Marseille'], mandatory: false }] }
     let(:ineligibilite_rules) do
       ds_and([
         ds_eq(champ_value(first_tdc.stable_id), constant(true)),