Don't invalidate self-consent if vaccinated

If the patient was vaccinated after getting self-consent, we shouldn't
invalidate that consent as it was valid when the vaccination was given.
A future enhancement might be to add a concept of "lifespan" to
consents, to capture exactly at which point they are invalid.

At the moment, the nurses see a patient that was vaccinated, with
supposedly no valid consent, which is not accurate.

Author: thomasleese

app/jobs/invalidate_self_consents_job.rb

@@ -4,29 +4,38 @@ class InvalidateSelfConsentsJob < ApplicationJob
   queue_as :consents
 
   def perform
-    team_ids = Team.pluck(:id)
-    programme_ids = Programme.pluck(:id)
+    academic_year = AcademicYear.current
+
+    Programme.find_each do |programme|
+      patients =
+        Patient.has_vaccination_status(
+          %i[none_yet could_not_vaccinate],
+          programme:,
+          academic_year:
+        )
+
+      Team.find_each do |team|
+        # FIXME: Filter on academic year.
 
-    team_ids
-      .product(programme_ids)
-      .each do |team_id, programme_id|
         consents =
           Consent
             .via_self_consent
-            .where(team_id:, programme_id:)
+            .where(team:, programme:)
+            .where(patient: patients)
             .where("created_at < ?", Date.current.beginning_of_day)
             .not_withdrawn
 
         triages =
           Triage
-            .where(team_id:, programme_id:)
-            .where("created_at < ?", Date.current.beginning_of_day)
+            .where(team:, programme:)
             .where(patient_id: consents.pluck(:patient_id))
+            .where("created_at < ?", Date.current.beginning_of_day)
 
         ActiveRecord::Base.transaction do
           consents.invalidate_all
           triages.invalidate_all
         end
       end
+    end
   end
 end

spec/jobs/invalidate_self_consents_job_spec.rb

@@ -3,22 +3,22 @@
 describe InvalidateSelfConsentsJob do
   subject(:perform_now) { described_class.perform_now }
 
+  let(:patient) { consent.patient }
+  let(:programme) { consent.programme }
+  let(:team) { consent.team }
+
   context "with parental consent from yesterday" do
     let(:consent) { create(:consent, created_at: 1.day.ago) }
 
+    before { create(:patient_vaccination_status, patient:, programme:) }
+
     it "does not invalidate the consent" do
       expect { perform_now }.not_to(change { consent.reload.invalidated? })
     end
 
     context "with triage" do
       let(:triage) do
-        create(
-          :triage,
-          created_at: 1.day.ago,
-          team: consent.team,
-          programme: consent.programme,
-          patient: consent.patient
-        )
+        create(:triage, created_at: 1.day.ago, team:, programme:, patient:)
       end
 
       it "does not invalidate the triage" do
@@ -30,19 +30,14 @@
   context "with parental consent from today" do
     let(:consent) { create(:consent) }
 
+    before { create(:patient_vaccination_status, patient:, programme:) }
+
     it "does not invalidate the consent" do
       expect { perform_now }.not_to(change { consent.reload.invalidated? })
     end
 
     context "with triage" do
-      let(:triage) do
-        create(
-          :triage,
-          team: consent.team,
-          programme: consent.programme,
-          patient: consent.patient
-        )
-      end
+      let(:triage) { create(:triage, team:, programme:, patient:) }
 
       it "does not invalidate the triage" do
         expect { perform_now }.not_to(change { triage.reload.invalidated? })
@@ -53,6 +48,8 @@
   context "with self-consent from yesterday" do
     let(:consent) { create(:consent, :self_consent, created_at: 1.day.ago) }
 
+    before { create(:patient_vaccination_status, patient:, programme:) }
+
     it "invalidates the consent" do
       expect { perform_now }.to change { consent.reload.invalidated? }.from(
         false
@@ -61,13 +58,7 @@
 
     context "with triage" do
       let(:triage) do
-        create(
-          :triage,
-          created_at: 1.day.ago,
-          team: consent.team,
-          programme: consent.programme,
-          patient: consent.patient
-        )
+        create(:triage, created_at: 1.day.ago, team:, programme:, patient:)
       end
 
       it "invalidates the triage" do
@@ -76,24 +67,47 @@
         ).to(true)
       end
     end
+
+    context "if the patient was vaccinated" do
+      before do
+        create(
+          :vaccination_record,
+          team:,
+          programme:,
+          patient:,
+          created_at: 1.day.ago
+        )
+
+        patient.vaccination_statuses.update_all(status: :vaccinated)
+      end
+
+      it "does not invalidate the consent" do
+        expect { perform_now }.not_to(change { consent.reload.invalidated? })
+      end
+
+      context "with triage" do
+        let(:triage) do
+          create(:triage, created_at: 1.day.ago, team:, programme:, patient:)
+        end
+
+        it "does not invalidate the triage" do
+          expect { perform_now }.not_to(change { triage.reload.invalidated? })
+        end
+      end
+    end
   end
 
   context "with self-consent from today" do
     let(:consent) { create(:consent, :self_consent) }
 
+    before { create(:patient_vaccination_status, patient:, programme:) }
+
     it "does not invalidate the consent" do
       expect { perform_now }.not_to(change { consent.reload.invalidated? })
     end
 
     context "with triage" do
-      let(:triage) do
-        create(
-          :triage,
-          team: consent.team,
-          programme: consent.programme,
-          patient: consent.patient
-        )
-      end
+      let(:triage) { create(:triage, team:, programme:, patient:) }
 
       it "does not invalidate the triage" do
         expect { perform_now }.not_to(change { triage.reload.invalidated? })
@@ -129,6 +143,11 @@
       )
     end
 
+    before do
+      create(:patient_vaccination_status, patient:, programme: self_programme)
+      create(:patient_vaccination_status, patient:, programme: parent_programme)
+    end
+
     it "does not invalidate the parent consent" do
       expect { perform_now }.not_to(
         change { parent_consent.reload.invalidated? }