Extract health answers into its own card on the patient session pagei

Moves answers to health questions out of the consent card into a standalone AppPatientSessionHealthAnswersComponent. The card renders as a warning callout when any health answer is "yes", with the heading including a count of yes responses.

Author: murugapl

app/components/app_patient_session_consent_component.html.erb

@@ -50,11 +50,4 @@
     <% end %>
   <% end %>
 
-  <% if show_health_answers? %>
-    <hr class="nhsuk-section-break nhsuk-section-break--visible nhsuk-section-break--l">
-
-    <h5 class="nhsuk-heading-s nhsuk-u-margin-bottom-2">All answers to health questions</h5>
-
-    <%= render AppHealthAnswersSummaryComponent.new(grouped_consents) %>
-  <% end %>
 <% end %>

app/components/app_patient_session_consent_component.rb

@@ -129,10 +129,6 @@ def who_refused
     grouped_consents.find(&:response_refused?)&.who_responded
   end
 
-  def show_health_answers?
-    grouped_consents.any?(&:response_given?)
-  end
-
   def consent_notifications
     patient
       .consent_notifications

app/components/app_patient_session_health_answers_component.rb

@@ -0,0 +1,65 @@
+# frozen_string_literal: true
+
+class AppPatientSessionHealthAnswersComponent < ViewComponent::Base
+  erb_template <<~ERB
+    <% if any_yes_health_answers? %>
+      <%= render AppWarningCalloutComponent.new(heading:, level: 2) do %>
+        <%= render AppHealthAnswersSummaryComponent.new(grouped_consents) %>
+      <% end %>
+    <% else %>
+      <%= render AppCardComponent.new(section: true) do |card| %>
+        <% card.with_heading(level: 2) { heading } %>
+        <%= render AppHealthAnswersSummaryComponent.new(grouped_consents) %>
+      <% end %>
+    <% end %>
+  ERB
+
+  def render? = grouped_consents.any?(&:response_given?)
+
+  def initialize(patient:, session:, programme:)
+    @patient = patient
+    @session = session
+    @programme = programme
+  end
+
+  private
+
+  attr_reader :patient, :session, :programme
+
+  delegate :academic_year, to: :session
+
+  def heading
+    count = yes_health_answers_count
+    if count.positive?
+      "All answers to health questions, including #{count} #{"Yes response".pluralize(count)}"
+    else
+      "All answers to health questions"
+    end
+  end
+
+  def yes_health_answers_count
+    grouped_consents.sum do |consent|
+      consent.health_answers.count(&:response_yes?)
+    end
+  end
+
+  def any_yes_health_answers?
+    grouped_consents.any? do |consent|
+      consent.health_answers.any?(&:response_yes?)
+    end
+  end
+
+  def grouped_consents
+    @grouped_consents ||=
+      ConsentGrouper.call(
+        patient
+          .consents
+          .for_programme(programme)
+          .where(academic_year:)
+          .includes(:consent_form, :parent)
+          .order(created_at: :desc),
+        programme_type: programme.type,
+        academic_year:
+      )
+  end
+end

app/views/patient_sessions/programmes/show.html.erb

@@ -12,6 +12,8 @@
 
     <%= render AppPatientSessionConsentComponent.new(patient: @patient, session: @session, programme: @programme) %>
 
+    <%= render AppPatientSessionHealthAnswersComponent.new(patient: @patient, session: @session, programme: @programme) %>
+
     <%= render AppPatientSessionTriageComponent.new(patient: @patient, session: @session, programme: @programme, current_user:, triage_form: @triage_form) %>
 
     <%= render AppPatientSessionRecordComponent.new(patient: @patient, session: @session, programme: @programme, current_user:, vaccinate_form: @vaccinate_form) %>

spec/components/app_patient_session_health_answers_component_spec.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+describe AppPatientSessionHealthAnswersComponent do
+  subject(:rendered) { render_inline(component) }
+
+  let(:component) { described_class.new(patient:, session:, programme:) }
+
+  let(:programme) { Programme.hpv }
+  let(:session) { create(:session, programmes: [programme]) }
+  let(:patient) { create(:patient, session:) }
+
+  context "without any given consent" do
+    it { expect(rendered.to_html).to be_blank }
+  end
+
+  context "with refused consent" do
+    before { create(:consent, :refused, patient:, programme:) }
+
+    it { expect(rendered.to_html).to be_blank }
+  end
+
+  context "with given consent, all no health answers" do
+    before do
+      create(:consent, :given, :no_contraindications, patient:, programme:)
+    end
+
+    it { should have_css("section", text: "All answers to health questions") }
+    it { should_not have_css(".nhsuk-card--warning") }
+    it { should_not have_text("Yes response") }
+  end
+
+  context "with given consent with 1 yes health answer" do
+    before do
+      create(:consent, :given, :health_question_notes, patient:, programme:)
+    end
+
+    it { should have_css(".nhsuk-card--warning") }
+    it { should have_text("including 1 Yes response") }
+  end
+
+  context "with given consent with multiple yes health answers" do
+    before do
+      create(
+        :consent,
+        :given,
+        patient:,
+        programme:,
+        health_answers: [
+          HealthAnswer.new(question: "Question 1", response: "yes"),
+          HealthAnswer.new(question: "Question 2", response: "yes"),
+          HealthAnswer.new(question: "Question 3", response: "no")
+        ]
+      )
+    end
+
+    it { should have_css(".nhsuk-card--warning") }
+    it { should have_text("including 2 Yes responses") }
+  end
+end