From e36dc19cc4ed063e6b2a5540600b40936855c357 Mon Sep 17 00:00:00 2001
From: Colin Darie <colin@darie.eu>
Date: Thu, 30 Mar 2023 12:44:43 +0200
Subject: [PATCH] fix(explication): render collapsible text into collapse
 element

---
 .../explication_component.html.haml           |  2 +-
 .../explication_component_spec.rb             | 24 +++++++++++++++++++
 2 files changed, 25 insertions(+), 1 deletion(-)
 create mode 100644 spec/components/editable_champ/explication_component/explication_component_spec.rb

diff --git a/app/components/editable_champ/explication_component/explication_component.html.haml b/app/components/editable_champ/explication_component/explication_component.html.haml
index 07392a30b..bc7ff4573 100644
--- a/app/components/editable_champ/explication_component/explication_component.html.haml
+++ b/app/components/editable_champ/explication_component/explication_component.html.haml
@@ -6,5 +6,5 @@
     - if @champ.collapsible_explanation_enabled? && @champ.collapsible_explanation_text.present?
       %p.fr-my-2w
         %button{ type: "button", "aria-controls": dom_id(@champ, :explanation), "aria-expanded": "false", href: dom_id(@champ, :explanation), class: "fr-btn ft-btn--sm fr-btn--secondary" } Lire plus
-      %p.fr-collapse{ id: dom_id(@champ, :explanation) }
+      .fr-collapse{ id: dom_id(@champ, :explanation) }
         = render SimpleFormatComponent.new(@champ.collapsible_explanation_text, allow_a: true)
diff --git a/spec/components/editable_champ/explication_component/explication_component_spec.rb b/spec/components/editable_champ/explication_component/explication_component_spec.rb
new file mode 100644
index 000000000..023f54c26
--- /dev/null
+++ b/spec/components/editable_champ/explication_component/explication_component_spec.rb
@@ -0,0 +1,24 @@
+describe EditableChamp::ExplicationComponent, type: :component do
+  let(:component) {
+    described_class.new(form: instance_double(ActionView::Helpers::FormBuilder, object_name: "dossier[champs_public_attributes]"), champ:)
+  }
+
+  let(:champ) { create(:champ_explication) }
+
+  describe 'no description' do
+    subject { render_inline(component).to_html }
+
+    it { is_expected.not_to have_button("Lire plus") }
+  end
+
+  describe 'collapsed text is collapsed' do
+    subject { render_inline(component).to_html }
+
+    before do
+      champ.type_de_champ.update!(collapsible_explanation_enabled: "1", collapsible_explanation_text: "hide me")
+    end
+
+    it { is_expected.to have_button("Lire plus") }
+    it { is_expected.to have_selector(".fr-collapse", text: "hide me") }
+  end
+end