Use Patient::ConsentStatus

This updates the various parts of the code that relied on the
`ConsentOutcome` class to determine the consent status of a patient to
instead use the new `Patient::ConsentStatus` model with the status
cached.

Author: thomasleese

app/components/app_consent_component.html.erb

@@ -1,6 +1,6 @@
 <%= render AppConsentStatusComponent.new(patient_session:, programme:) %>
 
-<% if patient.consent_outcome.no_response?(programme) %>
+<% if consent_status.no_response? %>
   <% if latest_consent_request %>
     <p class="nhsuk-body">
       No-one responded to our requests for consent.

app/components/app_consent_component.rb

@@ -27,10 +27,13 @@ def latest_consent_request
         .first
   end
 
+  def consent_status
+    @consent_status ||= patient.consent_status(programme:)
+  end
+
   def can_send_consent_request?
-    patient.consent_outcome.no_response?(programme) &&
-      patient.send_notifications? && session.open_for_consent? &&
-      patient.parents.any?
+    consent_status.no_response? && patient.send_notifications? &&
+      session.open_for_consent? && patient.parents.any?
   end
 
   def status_colour(consent)

app/components/app_consent_status_component.rb

@@ -9,12 +9,12 @@ def initialize(patient_session:, programme:)
   end
 
   def call
-    case patient.consent_outcome.status[programme]
-    when Patient::ConsentOutcome::GIVEN
+    consent_status = patient.consent_status(programme:)
+    if consent_status.given?
       icon_tick "Consent given", "aqua-green"
-    when Patient::ConsentOutcome::REFUSED
+    elsif consent_status.refused?
       icon_cross "Consent refused", "red"
-    when Patient::ConsentOutcome::CONFLICTS
+    elsif consent_status.conflicts?
       icon_cross "Conflicting consent", "dark-orange"
     end
   end

app/components/app_patient_session_search_result_card_component.rb

@@ -87,11 +87,15 @@ def status_tag
       render AppRegisterStatusTagComponent.new(
                patient_session.register_outcome.status
              )
+    elsif context == :consent
+      statuses =
+        patient_session.programmes.index_with do |programme|
+          patient.consent_status(programme:).status
+        end
+      render AppProgrammeStatusTagsComponent.new(statuses, outcome: :consent)
     else
       outcome =
         case context
-        when :consent
-          patient.consent_outcome
         when :triage
           patient.triage_outcome
         when :outcome

app/components/app_programme_session_table_component.rb

@@ -13,30 +13,40 @@ def initialize(sessions, programme:)
   attr_reader :sessions, :programme
 
   def cohort_count(session:)
-    session.patient_sessions.length.to_s
+    format_number(session.patient_sessions.count)
   end
 
   def number_stat(session:)
-    session.patient_sessions.select { yield it }.length.to_s
+    format_number(session.patient_sessions.select { yield it }.length)
   end
 
   def percentage_stat(session:)
-    total_count = session.patient_sessions.length
-    return nil if total_count.zero?
-
-    count = session.patient_sessions.select { yield it }.length
+    format_percentage(
+      session.patient_sessions.select { yield it }.length,
+      session.patient_sessions.count
+    )
+  end
 
-    number_to_percentage(count / total_count.to_f * 100.0, precision: 0)
+  def no_response_scope(session:)
+    session.patient_sessions.where(
+      Patient::ConsentStatus
+        .for_patient_session
+        .where(programme:)
+        .no_response
+        .arel
+        .exists
+    )
   end
 
   def no_response_count(session:)
-    number_stat(session:) { it.patient.consent_outcome.no_response?(programme) }
+    format_number(no_response_scope(session:).count)
   end
 
   def no_response_percentage(session:)
-    percentage_stat(session:) do
-      it.patient.consent_outcome.no_response?(programme)
-    end
+    format_percentage(
+      no_response_scope(session:).count,
+      session.patient_sessions.count
+    )
   end
 
   def triage_needed_count(session:)
@@ -50,4 +60,12 @@ def vaccinated_count(session:)
   def vaccinated_percentage(session:)
     percentage_stat(session:) { it.session_outcome.vaccinated?(programme) }
   end
+
+  def format_number(count) = count.to_s
+
+  def format_percentage(count, total_count)
+    return nil if total_count.zero?
+
+    number_to_percentage(count / total_count.to_f * 100.0, precision: 0)
+  end
 end

app/components/app_session_actions_component.rb

@@ -32,24 +32,18 @@ def rows
   end
 
   def no_consent_response_row
-    consent_row(
-      text: "No consent response",
-      status: Patient::ConsentOutcome::NO_RESPONSE
-    )
+    consent_row("No consent response", status: "no_response")
   end
 
   def conflicting_consent_row
-    consent_row(
-      text: "Conflicting consent",
-      status: Patient::ConsentOutcome::CONFLICTS
-    )
+    consent_row("Conflicting consent", status: "conflicts")
   end
 
-  def consent_row(text:, status:)
+  def consent_row(text, status:)
     count =
-      patient_sessions.count do
-        it.patient.consent_outcome.status.values_at(*it.programmes).any?(status)
-      end
+      patient_sessions.where(
+        Patient::ConsentStatus.for_patient.where(status:).arel.exists
+      ).count
 
     return nil if count.zero?
 

app/components/app_session_details_summary_component.rb

@@ -32,13 +32,13 @@ def cohort_row
   end
 
   def consent_refused_row
-    status = Patient::ConsentOutcome::REFUSED
     count =
-      patient_sessions.count do
-        it.patient.consent_outcome.status.values_at(*it.programmes).any?(status)
-      end
+      patient_sessions.where(
+        Patient::ConsentStatus.for_patient_session.refused.arel.exists
+      ).count
+
     href =
-      session_consent_path(session, search_form: { consent_status: status })
+      session_consent_path(session, search_form: { consent_status: "refused" })
 
     {
       key: {

app/components/app_simple_status_banner_component.rb

@@ -52,7 +52,7 @@ def description
 
     if patient.triage_outcome.required?(programme)
       reasons = [
-        if patient.triage_outcome.consent_needs_triage?(programme:)
+        if patient.consent_status(programme:).health_answers_require_follow_up?
           I18n.t(
             :consent_needs_triage,
             scope: %i[

app/controllers/consents_controller.rb

@@ -26,7 +26,7 @@ def create
   end
 
   def send_request
-    return unless @patient.consent_outcome.no_response?(@programme)
+    return unless @patient.consent_status(programme: @programme).no_response?
 
     # For programmes that are administered together we should send the consent request together.
     programmes =

app/controllers/sessions/consent_controller.rb

@@ -12,7 +12,7 @@ class Sessions::ConsentController < ApplicationController
   layout "full"
 
   def show
-    @statuses = Patient::ConsentOutcome::STATUSES
+    @statuses = Patient::ConsentStatus.statuses.keys
 
     scope =
       @session.patient_sessions.preload_for_status.in_programmes(