Use AppConsentCardComponent

This updates the `AppPatientSessionConsentComponent` to use the new
`AppConsentCardComponent`, matching the latest designs in the prototype.
This also allows us to remove the `AppConsentTableComponent` as its no
longer in use.

Jira-Issue: MAV-1474

Author: thomasleese

app/components/app_consent_table_component.html.erb

@@ -38,13 +38,19 @@
     <%= render AppGillickAssessmentComponent.new(patient_session:, programme:) %>
   <% end %>
 
-  <%= render AppConsentTableComponent.new(patient_session:, programme:) %>
+  <% if consents.any? %>
+    <h3 class="nhsuk-heading-s">Consent responses</h3>
+
+    <% consents.each do |consent| %>
+      <%= render AppConsentCardComponent.new(consent, session:) %>
+    <% end %>
+  <% end %>
 
   <% if show_health_answers? %>
     <hr class="nhsuk-section-break nhsuk-section-break--visible nhsuk-section-break--l">
 
     <h3 class="nhsuk-heading-s">All answers to health questions</h3>
 
-    <%= render AppHealthAnswersSummaryComponent.new(consents) %>
+    <%= render AppHealthAnswersSummaryComponent.new(grouped_consents) %>
   <% end %>
 <% end %>

app/components/app_consent_table_component.rb

@@ -32,6 +32,15 @@ def latest_consent_request
         .first
   end
 
+  def consents
+    @consents ||=
+      patient
+        .consents
+        .where(programme:)
+        .includes(:consent_form, :parent, :programme)
+        .order(created_at: :desc)
+  end
+
   def consent_status
     @consent_status ||= patient.consent_status(programme:)
   end
@@ -45,16 +54,16 @@ def can_send_consent_request?
       session.open_for_consent? && patient.parents.any?
   end
 
-  def who_refused
-    consents.find(&:response_refused?)&.who_responded
+  def grouped_consents
+    @grouped_consents ||= ConsentGrouper.call(consents, programme:)
   end
 
-  def consents
-    @consents ||= ConsentGrouper.call(patient.consents, programme:)
+  def who_refused
+    grouped_consents.find(&:response_refused?)&.who_responded
   end
 
   def show_health_answers?
-    consents.any?(&:response_given?)
+    grouped_consents.any?(&:response_given?)
   end
 
   delegate :status, to: :consent_status

app/components/app_patient_session_consent_component.html.erb

@@ -45,9 +45,9 @@
     before { create(:patient_consent_status, :refused, patient:, programme:) }
 
     it { should have_css(".app-card--red", text: "Consent refused") }
-    it { should have_css("tr", text: /#{consent.parent.full_name}/) }
-    it { should have_css("tr", text: /#{consent.parent_relationship.label}/) }
-    it { should have_css("table tr", text: /Consent refused/) }
+    it { should have_content(consent.parent.full_name) }
+    it { should have_content(consent.parent_relationship.label) }
+    it { should have_content("Consent refused") }
     it { should_not have_css("details", text: "Responses to health questions") }
   end
 

app/components/app_patient_session_consent_component.rb

@@ -268,14 +268,12 @@ def when_the_nurse_views_the_childs_record
 
   def then_they_see_that_the_child_has_consent_from_themselves
     expect(page).to have_content("Consent given")
-    expect(page).to have_content(
-      "#{@patient.full_name} Child (Gillick competent)"
-    )
+    expect(page).to have_content("Child (Gillick competent)")
   end
 
   def then_they_see_that_the_child_has_consent_from_the_parent
     expect(page).to have_content("Consent given")
-    expect(page).to have_content("Decision #{@parent.full_name}")
+    expect(page).to have_content(@parent.full_name)
   end
 
   def and_the_child_should_be_safe_to_vaccinate