Merge pull request #6384 from NHSDigital/consent-card-actions

Show consent actions on the summary card

Author: thomasleese

app/components/app_card_component.rb

@@ -41,13 +41,15 @@ class AppCardComponent < ViewComponent::Base
   def initialize(
     colour: nil,
     link_to: nil,
+    clickable: nil,
     feature: false,
     secondary: false,
     compact: false,
     filters: false,
     section: false
   )
     @link_to = link_to
+    @clickable = clickable.nil? ? link_to.present? : clickable
     @colour = colour
     @feature = filters || feature
     @secondary = secondary
@@ -63,7 +65,7 @@ def top_level_tag = @section ? "section" : "div"
   def card_classes
     [
       "nhsuk-card",
-      ("nhsuk-card--clickable" if @link_to.present?),
+      ("nhsuk-card--clickable" if @clickable),
       ("nhsuk-card--feature" if @feature),
       ("nhsuk-card--secondary" if @secondary),
       "app-card",

app/components/app_consent_card_component.rb

@@ -8,7 +8,7 @@ def initialize(consent, session:)
 
   def call
     render AppCardComponent.new(**card_options) do |card|
-      card.with_heading(level: 6) { heading }
+      card.with_heading(level: 6, actions:) { heading }
       render AppConsentSummaryComponent.new(
                consent,
                show_email_address: true,
@@ -28,7 +28,8 @@ def link_to
     session_patient_programme_consent_path(session, patient, programme, consent)
   end
 
-  def card_options = { link_to:, colour: "offset", compact: true }
+  def card_options =
+    { link_to:, clickable: false, colour: "offset", compact: true }
 
   def heading
     if consent.via_self_consent?
@@ -37,4 +38,8 @@ def heading
       "#{consent.name} (#{consent.who_responded})"
     end
   end
+
+  def actions
+    helpers.consent_action_links(consent, session:)
+  end
 end

app/helpers/consents_helper.rb

@@ -96,6 +96,53 @@ def consent_response_tag(consent)
     end
   end
 
+  def consent_action_links(consent, session:)
+    patient = consent.patient
+    programme = consent.programme
+
+    follow_up_link =
+      if consent.can_follow_up?
+        {
+          text: "Follow up",
+          href:
+            follow_up_session_patient_programme_consent_path(
+              session,
+              patient,
+              programme,
+              consent
+            )
+        }
+      end
+    withdraw_link =
+      if consent.can_withdraw?
+        {
+          text: "Withdraw consent",
+          href:
+            withdraw_session_patient_programme_consent_path(
+              session,
+              patient,
+              programme,
+              consent
+            )
+        }
+      end
+    invalidate_link =
+      if consent.can_invalidate?
+        {
+          text: "Mark as invalid",
+          href:
+            invalidate_session_patient_programme_consent_path(
+              session,
+              patient,
+              programme,
+              consent
+            )
+        }
+      end
+
+    [follow_up_link, withdraw_link, invalidate_link].compact
+  end
+
   def consent_parent_name(consentable)
     consentable.parent_full_name.presence ||
       (consentable.respond_to?(:parent) ? consentable.parent&.full_name : nil)

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

@@ -5,25 +5,8 @@
 <%= h1 "Consent response from #{@consent.name}" %>
 
 <%= render AppActionListComponent.new do |action_list| %>
-  <% if @consent.can_follow_up? %>
-    <% action_list.with_item(
-         text: "Follow up",
-         href: follow_up_session_patient_programme_consent_path,
-       ) %>
-  <% end %>
-
-  <% if @consent.can_withdraw? %>
-    <% action_list.with_item(
-         text: "Withdraw consent",
-         href: withdraw_session_patient_programme_consent_path,
-       ) %>
-  <% end %>
-
-  <% if @consent.can_invalidate? %>
-    <% action_list.with_item(
-         text: "Mark as invalid",
-         href: invalidate_session_patient_programme_consent_path,
-       ) %>
+  <% consent_action_links(@consent, session: @session).each do |action| %>
+    <% action_list.with_item(**action) %>
   <% end %>
 <% end %>
 

spec/components/app_consent_card_component_spec.rb

@@ -39,6 +39,50 @@
   it { should have_content("Response") }
   it { should have_content("Consent given") }
 
+  describe "actions" do
+    context "when consent is given" do
+      it { should have_link("Withdraw consent") }
+      it { should have_link("Mark as invalid") }
+      it { should_not have_link("Follow up") }
+    end
+
+    context "when follow-up is requested" do
+      let(:consent) do
+        create(
+          :consent,
+          :follow_up_requested,
+          patient:,
+          parent:,
+          programme:,
+          team:,
+          submitted_at: Time.zone.local(2024, 1, 1)
+        )
+      end
+
+      it { should have_link("Follow up") }
+      it { should have_link("Mark as invalid") }
+      it { should_not have_link("Withdraw consent") }
+    end
+
+    context "when consent is invalidated" do
+      let(:consent) do
+        create(
+          :consent,
+          :invalidated,
+          patient:,
+          parent:,
+          programme:,
+          team:,
+          submitted_at: Time.zone.local(2024, 1, 1)
+        )
+      end
+
+      it { should_not have_link("Follow up") }
+      it { should_not have_link("Withdraw consent") }
+      it { should_not have_link("Mark as invalid") }
+    end
+  end
+
   context "with the flu programme" do
     let(:programme) { Programme.flu }
     let(:consent) { create(:consent, programme:, vaccine_methods: %w[nasal]) }