Merge pull request #6661 from NHSDigital/fix-patient-programme-status-after-consent-request

Update patient status after sending consent requests

Author: jhenderson

app/lib/notifier/patient.rb

@@ -237,6 +237,8 @@ def send_consent_notification(
       SMSDeliveryJob.perform_later(sms_template, **params)
     end
 
+    PatientStatusUpdaterJob.perform_async(patient.id)
+
     consent_notification
   end
 

spec/lib/notifier/patient_spec.rb

@@ -14,7 +14,10 @@
     let(:disease_types) { programmes.flat_map(&:disease_types).uniq.presence }
     let(:programme_types) { programmes.map(&:type) }
     let(:team) { create(:team, programmes:) }
-    let(:session) { create(:session, location:, programmes:, team:) }
+    let(:send_consent_requests_at) { nil }
+    let(:session) do
+      create(:session, location:, programmes:, team:, send_consent_requests_at:)
+    end
     let(:team_location) { session.team_location }
 
     context "with a session" do
@@ -40,6 +43,38 @@
           expect(consent_notification.sent_at).to eq(today)
         end
 
+        context "when the consent request was scheduled for the future" do
+          let(:send_consent_requests_at) { today + 1.day }
+
+          it "updates the programme status after sending the request" do
+            travel_to(today) do
+              PatientStatusUpdater.call(patient:)
+
+              expect(
+                patient.programme_status(
+                  programmes.first,
+                  academic_year: session.academic_year
+                )
+              ).to be_needs_consent_request_scheduled
+
+              notifier.send_consent_request(programmes, session:, sent_by:)
+
+              expect(PatientStatusUpdaterJob).to have_enqueued_sidekiq_job(
+                patient.id
+              )
+
+              PatientStatusUpdaterJob.drain
+
+              expect(
+                patient.programme_status(
+                  programmes.first,
+                  academic_year: session.academic_year
+                ).reload
+              ).to be_needs_consent_no_response
+            end
+          end
+        end
+
         it "enqueues an email per parent" do
           expect { send_consent_request }.to have_delivered_email(
             :consent_school_request_hpv