Trigger changes to notify_parents when consents are changed

This will cause an update on all related vaccination records if a relevant consent is:
- invalidated
- created (it is currently not possible to create a consent form after a vaccination record has been created, so this code is currently not used. The code has been included in case this becomes possible in the future)

Author: alistair-white-horne-tng

app/controllers/draft_consents_controller.rb

@@ -68,6 +68,8 @@ def handle_confirm
 
       @consent.save!
 
+      @consent.update_vaccination_records_no_notify!
+
       StatusUpdater.call(patient: @patient)
     end
 

app/controllers/patient_sessions/consents_controller.rb

@@ -65,6 +65,7 @@ def update_withdraw
     if @consent.valid?
       ActiveRecord::Base.transaction do
         @consent.save!
+
         update_patient_status
       end
 
@@ -84,6 +85,9 @@ def update_invalidate
     if @consent.valid?
       ActiveRecord::Base.transaction do
         @consent.save!
+
+        @consent.update_vaccination_records_no_notify!
+
         update_patient_status
       end
 

app/models/consent.rb

@@ -212,6 +212,17 @@ def notes_required?
       )
   end
 
+  def update_vaccination_records_no_notify!
+    vaccination_records = VaccinationRecord.where(patient:, programme:)
+
+    vaccination_records.find_each do |vaccination_record|
+      vaccination_record.update!(
+        notify_parents:
+          VaccinationNotificationCriteria.call(vaccination_record:)
+      )
+    end
+  end
+
   class ConsentFormNotRecorded < StandardError
   end
 end

spec/features/invalidate_consent_spec.rb

@@ -50,6 +50,23 @@
     and_i_am_not_able_to_record_a_vaccination
   end
 
+  scenario "Given self consent, and requested parents aren't notified, and vaccinated, then invalidate" do
+    given_i_am_signed_in
+    and_self_consent_has_been_given
+    and_patient_has_been_vaccinated_with_no_parent_notification
+
+    when_i_go_to_the_patient
+    then_i_see_the_self_consent
+
+    when_i_click_on_the_self_consent
+    and_i_click_invalidate_consent
+
+    when_i_fill_in_the_notes
+    and_i_click_invalidate_consent
+    then_i_see_the_consent_has_been_invalidated
+    and_the_vaccination_record_is_updated_to_notify_parents
+  end
+
   def given_i_am_signed_in
     @programme = create(:programme, :hpv)
     team = create(:team, :with_one_nurse, programmes: [@programme])
@@ -120,9 +137,15 @@ def when_i_fill_in_the_notes
 
   def then_i_see_the_consent_has_been_invalidated
     expect(page).to have_content("Invalid")
-    expect(page).to have_content(
-      "Consent response from #{@parent.full_name} marked as invalid"
-    )
+    if @parent
+      expect(page).to have_content(
+        "Consent response from #{@parent.full_name} marked as invalid"
+      )
+    else
+      expect(page).to have_content(
+        "Consent response from #{@patient.full_name} marked as invalid"
+      )
+    end
   end
 
   def and_i_cant_mark_as_invalid
@@ -137,4 +160,44 @@ def and_i_am_not_able_to_record_a_vaccination
     expect(page).to have_content("No response")
     expect(page).not_to have_content("ready for their HPV vaccination?")
   end
+
+  def and_self_consent_has_been_given
+    @consent =
+      create(
+        :consent,
+        :given,
+        :self_consent,
+        patient: @patient,
+        programme: @programme
+      )
+    create(
+      :patient_consent_status,
+      :given,
+      patient: @patient,
+      programme: @programme
+    )
+  end
+
+  def and_patient_has_been_vaccinated_with_no_parent_notification
+    @vaccination_record =
+      create(
+        :vaccination_record,
+        patient: @patient,
+        programme: @programme,
+        session: @session,
+        notify_parents: false
+      )
+  end
+
+  def then_i_see_the_self_consent
+    expect(page).to have_content("Child (Gillick competent)")
+  end
+
+  def when_i_click_on_the_self_consent
+    click_on "Child (Gillick competent)"
+  end
+
+  def and_the_vaccination_record_is_updated_to_notify_parents
+    expect(@vaccination_record.reload.notify_parents).to be true
+  end
 end

spec/models/consent_spec.rb

@@ -345,4 +345,113 @@
       expect(consents).not_to include(consent_next_year)
     end
   end
+
+  describe "#update_vaccination_records_no_notify" do
+    let(:patient) { create(:patient) }
+    let(:programme) { create(:programme, :hpv) }
+    let(:consent) { create(:consent, patient:, programme:) }
+
+    context "when vaccination records exist for the patient and programme" do
+      let!(:first_vaccination_record) do
+        create(:vaccination_record, patient:, programme:, notify_parents: false)
+      end
+      let!(:second_vaccination_record) do
+        create(:vaccination_record, patient:, programme:, notify_parents: true)
+      end
+      let!(:other_patient_record) do
+        create(
+          :vaccination_record,
+          patient: create(:patient),
+          programme:,
+          notify_parents: false
+        )
+      end
+      let!(:other_programme_record) do
+        create(
+          :vaccination_record,
+          patient:,
+          programme: create(:programme, :flu),
+          notify_parents: false
+        )
+      end
+
+      before do
+        allow(VaccinationNotificationCriteria).to receive(:call).with(
+          vaccination_record: first_vaccination_record
+        ).and_return(true)
+        allow(VaccinationNotificationCriteria).to receive(:call).with(
+          vaccination_record: second_vaccination_record
+        ).and_return(false)
+      end
+
+      it "updates notify_parents for matching vaccination records" do
+        expect { consent.update_vaccination_records_no_notify! }.to change {
+          first_vaccination_record.reload.notify_parents
+        }.from(false).to(true).and change {
+                second_vaccination_record.reload.notify_parents
+              }.from(true).to(false)
+      end
+
+      it "does not update vaccination records for other patients" do
+        expect { consent.update_vaccination_records_no_notify! }.not_to(
+          change { other_patient_record.reload.notify_parents }
+        )
+      end
+
+      it "does not update vaccination records for other programmes" do
+        expect { consent.update_vaccination_records_no_notify! }.not_to(
+          change { other_programme_record.reload.notify_parents }
+        )
+      end
+
+      it "calls VaccinationNotificationCriteria for each matching record" do
+        consent.update_vaccination_records_no_notify!
+
+        expect(VaccinationNotificationCriteria).to have_received(:call).with(
+          vaccination_record: first_vaccination_record
+        )
+        expect(VaccinationNotificationCriteria).to have_received(:call).with(
+          vaccination_record: second_vaccination_record
+        )
+      end
+    end
+
+    context "with multiple vaccination records and mixed results" do
+      let!(:vaccination_records) do
+        Array.new(3) do
+          create(
+            :vaccination_record,
+            patient:,
+            programme:,
+            notify_parents: false
+          )
+        end
+      end
+
+      before do
+        allow(VaccinationNotificationCriteria).to receive(:call).with(
+          vaccination_record: vaccination_records[0]
+        ).and_return(true)
+        allow(VaccinationNotificationCriteria).to receive(:call).with(
+          vaccination_record: vaccination_records[1]
+        ).and_return(nil)
+        allow(VaccinationNotificationCriteria).to receive(:call).with(
+          vaccination_record: vaccination_records[2]
+        ).and_return(false)
+      end
+
+      it "updates each vaccination record according to the criteria result" do
+        expect { consent.update_vaccination_records_no_notify! }.to change {
+          vaccination_records[0].reload.notify_parents
+        }.from(false).to(true).and change {
+                vaccination_records[1].reload.notify_parents
+              }
+                .from(false)
+                .to(nil)
+                .and(
+                  not_change { vaccination_records[2].reload.notify_parents }
+                )
+      end
+    end
+  end
 end