refactor(siret): move degraded-mode message into InputStatusMessageComponent

The degraded-mode message was rendered directly in SiretComponent,
bypassing the component that owns the a11y contract (fr-messages-group,
aria-describedby). Move it into the :siret branch, keeping it hidden
when the champ feeds a condition since submission is then blocked.

Co-authored-by: mfo <mfo@users.noreply.github.com>

Author: E-L-T

app/components/dsfr/input_status_message_component.rb

@@ -71,7 +71,11 @@ def compute_statut_message
       when TypeDeChamp.type_champs[:siret]
         # TODO: use fetched? after T20251029backfillChampSiretExternalStateTask
         if @champ.etablissement&.as_degraded_mode?
-          nil # the non-blocking banner in SiretComponent explains the situation
+          # When the champ feeds a condition, submission is blocked until the
+          # backfill completes: no "you can continue" message in that case.
+          if !@champ.external_data_required_for_conditions?
+            { state: :info, text: t('.siret.technical_error') }
+          end
         elsif @champ.etablissement && @champ.etablissement.entreprise_capital_social.present?
           { state: :info, text: t('.siret.fetched_with_capital', raison_sociale_or_name: displayable_raison_sociale_or_name(@champ.etablissement), forme_juridique: @champ.etablissement.entreprise_forme_juridique, capital_sociale: pretty_currency(@champ.etablissement.entreprise_capital_social)) }
         elsif @champ.etablissement && @champ.etablissement.entreprise_capital_social.blank?

app/components/dsfr/input_status_message_component/input_status_message_component.en.yml

@@ -5,6 +5,7 @@ en:
     pending: "Search in progress for RNA %{value}…"
     success: "This RNA number is linked to: %{title}, %{address}."
   siret:
+    technical_error: "Automatic verification is temporarily unavailable. You can continue submitting; the information will be retrieved as soon as possible."
     pending: "Search in progress for SIRET %{value}…"
     fetched_with_capital: "%{raison_sociale_or_name} %{forme_juridique} with a share capital of %{capital_sociale}"
     fetched: "%{raison_sociale_or_name} %{forme_juridique}"

app/components/dsfr/input_status_message_component/input_status_message_component.fr.yml

@@ -5,6 +5,7 @@ fr:
     pending: "Recherche en cours pour le RNA %{value}…"
     success: "Ce RNA correspond à : %{title}, %{address}"
   siret:
+    technical_error: "La vérification automatique est temporairement indisponible. Vous pouvez poursuivre le dépôt ; les informations seront récupérées dès que possible."
     pending: "Recherche en cours pour le SIRET %{value}…"
     fetched_with_capital: "%{raison_sociale_or_name} %{forme_juridique} au capital social de %{capital_sociale}"
     fetched: "%{raison_sociale_or_name} %{forme_juridique}"

app/components/editable_champ/siret_component/siret_component.html.haml

@@ -1,4 +1 @@
 = @form.text_field(:external_id, input_opts(value: pretty_siret(@champ.external_id), id: @champ.focusable_input_id, aria: { **labelledby_id_attr, describedby: "#{@champ.describedby_id} #{@champ.error_id(:value)}" }, data: { controller: 'format', format: 'siret' }, required: @champ.required?, class: "width-33-desktop"))
-
-- if @champ.external_data_status_message.present?
-  %p.fr-info-text{ role: 'status' }= t("shared.champs.external_data.#{@champ.external_data_status_message}")

app/models/champs/siret_champ.rb

@@ -19,14 +19,6 @@ def external_data_required_for_conditions?
     dependent_conditions?
   end
 
-  def external_data_status_message
-    return nil if external_data_required_for_conditions?
-    return :technical_error if etablissement&.as_degraded_mode?
-    return :technical_error if external_error? && !external_data_not_found?
-
-    nil
-  end
-
   def focusable_input_id(attribute = :value)
     [input_id, :value].compact.join('-')
   end

config/i18n-tasks.yml

@@ -94,7 +94,6 @@ search:
 ## Consider these keys used:
 ignore_unused:
   - 'errors.format'
-  - 'shared.champs.external_data.*' # dynamic key: t("shared.champs.external_data.#{external_data_status_message}")
   - 'user_mailer.*.subject'
   - 'groupe_gestionnaire_mailer.*.subject'
   - 'invite_mailer.*.subject'

config/locales/shared.en.yml

@@ -28,8 +28,6 @@ en:
       move_down: Move down
 
     champs:
-      external_data:
-        technical_error: "Automatic verification is temporarily unavailable. You can continue submitting; the information will be retrieved as soon as possible."
       drop_down_list:
         other: Enter another option
         other_label: Enter your option

config/locales/shared.fr.yml

@@ -30,8 +30,6 @@ fr:
       move_down: Déplacer vers le bas
 
     champs:
-      external_data:
-        technical_error: "La vérification automatique est temporairement indisponible. Vous pouvez poursuivre le dépôt ; les informations seront récupérées dès que possible."
       rna:
         association:
           data_fetched: "Ce RNA correspond à : %{title}, %{address}"

spec/components/input_status_message_component_spec.rb

@@ -250,7 +250,7 @@
           )
         end
 
-        it "does not render a warning message (the non-blocking info banner already covers it)" do
+        it "does not render a warning message (submission is not blocked)" do
           expect(subject).not_to have_css(".fr-message--warning")
         end
       end
@@ -260,10 +260,20 @@
           champ.update!(value: '44011762001530', external_id: '44011762001530', external_state: 'fetched', etablissement: Etablissement.new(siret: '44011762001530'))
         end
 
-        it "does not render a fetched message (the non-blocking info banner covers it)" do
-          expect(subject).not_to have_css(".fr-message--info")
+        it "renders an info message explaining verification is postponed" do
+          expect(subject).to have_css(".fr-message--info", text: "La vérification automatique est temporairement indisponible")
           expect(subject).not_to have_text("Non-diffusible")
         end
+
+        context "when the champ feeds a condition (submission is blocked until backfill)" do
+          before do
+            allow(champ).to receive(:external_data_required_for_conditions?).and_return(true)
+          end
+
+          it "does not render the 'you can continue' message" do
+            expect(subject).not_to have_css(".fr-message--info")
+          end
+        end
       end
 
       context "when etablissement is non-diffusible (diffusable_commercialement: false)" do

spec/models/champs/siret_champ_spec.rb

@@ -292,81 +292,6 @@
     end
   end
 
-  describe '#external_data_status_message' do
-    let(:procedure) { create(:procedure, types_de_champ_public: [{ type: :siret }]) }
-    let(:dossier) { create(:dossier, procedure:) }
-    let(:champ) { dossier.champs.first }
-
-    context 'when fetched with a degraded-mode etablissement (API Entreprise was down)' do
-      before do
-        champ.update!(external_id: '44011762001530', external_state: 'fetched', etablissement: Etablissement.new(siret: '44011762001530'))
-      end
-
-      it 'returns :technical_error so the user knows verification is postponed' do
-        expect(champ.external_data_status_message).to eq(:technical_error)
-      end
-    end
-
-    context 'when fetched with a complete etablissement' do
-      before do
-        champ.update!(external_id: '44011762001530', external_state: 'fetched', etablissement: create(:etablissement))
-      end
-
-      it 'returns nil' do
-        expect(champ.external_data_status_message).to be_nil
-      end
-    end
-  end
-
-  describe '#external_data_status_message (unit)' do
-    let(:champ) { Champs::SiretChamp.new }
-
-    it 'returns nil when fetched' do
-      allow(champ).to receive_messages(pending?: false, external_error?: false, external_data_required_for_conditions?: false)
-      expect(champ.external_data_status_message).to be_nil
-    end
-
-    it 'returns nil when pending (message already handled by InputStatusMessageComponent)' do
-      allow(champ).to receive_messages(pending?: true, external_error?: false, external_data_required_for_conditions?: false)
-      expect(champ.external_data_status_message).to be_nil
-    end
-
-    it 'returns :technical_error on external_error with a non-404 code' do
-      allow(champ).to receive_messages(
-        pending?: false, external_error?: true,
-        external_data_required_for_conditions?: false,
-        fetch_external_data_exceptions: [ExternalDataException.new(error: 'x', code: 503)]
-      )
-      expect(champ.external_data_status_message).to eq(:technical_error)
-    end
-
-    it 'returns nil on a 404 (handled by the validation error)' do
-      allow(champ).to receive_messages(
-        pending?: false, external_error?: true,
-        external_data_required_for_conditions?: false,
-        fetch_external_data_exceptions: [ExternalDataException.new(error: 'x', code: 404)]
-      )
-      expect(champ.external_data_status_message).to be_nil
-    end
-
-    it 'returns nil when a technical retry was followed by a 404 (exceptions accumulate)' do
-      allow(champ).to receive_messages(
-        pending?: false, external_error?: true,
-        external_data_required_for_conditions?: false,
-        fetch_external_data_exceptions: [
-          ExternalDataException.new(error: 'boom', code: 503),
-          ExternalDataException.new(error: 'NF', code: 404),
-        ]
-      )
-      expect(champ.external_data_status_message).to be_nil
-    end
-
-    it 'returns nil when external_data_required_for_conditions? is true' do
-      allow(champ).to receive_messages(external_data_required_for_conditions?: true)
-      expect(champ.external_data_status_message).to be_nil
-    end
-  end
-
   describe '#handle_exhausted_external_data_retries!' do
     let(:procedure) { create(:procedure, types_de_champ_public: [{ type: :siret }]) }
     let(:dossier) { create(:dossier, procedure:) }