Cache navigation header counts

This updates `TeamCachedCounts` to cache the values in the header to
ensure a more performant page load instead of needing to query the
latest information from the database each time.

Jira-Issue: MAV-2006

Author: thomasleese

app/controllers/parent_interface/consent_forms_controller.rb

@@ -55,6 +55,8 @@ def record
 
       session.delete(:consent_form_id)
 
+      TeamCachedCounts.new(@team).reset_unmatched_consent_responses!
+
       send_consent_form_confirmation(@consent_form)
 
       ConsentFormMatchingJob.perform_later(@consent_form)

app/jobs/commit_patient_changesets_job.rb

@@ -29,6 +29,8 @@ def perform(import)
 
       import.postprocess_rows!
 
+      reset_counts(import)
+
       import.update_columns(
         processed_at: Time.zone.now,
         status: :processed,
@@ -165,4 +167,10 @@ def has_auto_confirmable_school_move?(school_move, import)
         academic_year: import.academic_year
       ) || school_move.patient.archived?(team: import.team)
   end
+
+  def reset_counts(import)
+    cached_counts = TeamCachedCounts.new(import.team)
+    cached_counts.reset_import_issues!
+    cached_counts.reset_school_moves!
+  end
 end

app/jobs/consent_form_matching_job.rb

@@ -59,6 +59,8 @@ def match_with_exact_nhs_number
     patient.update_from_pds!(pds_patient)
     send_parental_contact_warning_if_needed(patient, @consent_form)
     @consent_form.match_with_patient!(patient, current_user: nil)
+    reset_counts
+    true
   end
 
   def session_patients
@@ -95,5 +97,10 @@ def match_patient(patient)
 
     send_parental_contact_warning_if_needed(patient, @consent_form)
     @consent_form.match_with_patient!(patient, current_user: nil)
+    reset_counts
+  end
+
+  def reset_counts
+    TeamCachedCounts.new(@consent_form.team).reset_unmatched_consent_responses!
   end
 end

app/lib/team_cached_counts.rb

@@ -8,45 +8,75 @@ def initialize(team)
   def import_issues
     return nil if current_user.nil?
 
-    vaccination_records_with_issues =
-      VaccinationRecordPolicy::Scope
-        .new(current_user, VaccinationRecord)
-        .resolve
-        .with_pending_changes
-        .pluck(:patient_id)
+    Rails
+      .cache
+      .fetch(import_issues_key) do
+        vaccination_records_with_issues =
+          VaccinationRecordPolicy::Scope
+            .new(current_user, VaccinationRecord)
+            .resolve
+            .with_pending_changes
+            .pluck(:patient_id)
 
-    patients_with_issues =
-      PatientPolicy::Scope
-        .new(current_user, Patient)
-        .resolve
-        .with_pending_changes
-        .pluck(:id)
+        patients_with_issues =
+          PatientPolicy::Scope
+            .new(current_user, Patient)
+            .resolve
+            .with_pending_changes
+            .pluck(:id)
 
-    (vaccination_records_with_issues + patients_with_issues).uniq.length
+        (vaccination_records_with_issues + patients_with_issues).uniq.length
+      end
+  end
+
+  def reset_import_issues!
+    Rails.cache.delete(import_issues_key)
   end
 
   def school_moves
     return nil if current_user.nil?
 
-    SchoolMovePolicy::Scope.new(current_user, SchoolMove).resolve.count
+    Rails
+      .cache
+      .fetch(school_moves_key) do
+        SchoolMovePolicy::Scope.new(current_user, SchoolMove).resolve.count
+      end
+  end
+
+  def reset_school_moves!
+    Rails.cache.delete(school_moves_key)
   end
 
   def unmatched_consent_responses
     return nil if current_user.nil?
 
-    ConsentFormPolicy::Scope
-      .new(current_user, ConsentForm)
-      .resolve
-      .unmatched
-      .recorded
-      .not_archived
-      .count
+    Rails
+      .cache
+      .fetch(unmatched_consent_responses_key) do
+        ConsentFormPolicy::Scope
+          .new(current_user, ConsentForm)
+          .resolve
+          .unmatched
+          .recorded
+          .not_archived
+          .count
+      end
+  end
+
+  def reset_unmatched_consent_responses!
+    Rails.cache.delete(unmatched_consent_responses_key)
   end
 
   private
 
   attr_reader :team
 
+  def import_issues_key = cache_key("import-issues")
+
+  def school_moves_key = cache_key("school-moves")
+
+  def unmatched_consent_responses_key = cache_key("unmatched-consent-responses")
+
   def current_user
     # We can't use the policy_scope helper here as we're not in a controller.
     # Instead, we can mock what a `User` looks like from the perspective of a
@@ -56,4 +86,6 @@ def current_user
         OpenStruct.new(selected_team: team, selected_organisation: organisation)
       end
   end
+
+  def cache_key(type) = "cached-counts/#{type}/#{team.id}"
 end