Track session notifications

This updates the session reminders job to track when the notifications
are sent by creating `SessionNotification` instances.

app/jobs/session_reminders_job.rb

@@ -6,7 +6,9 @@ class SessionRemindersJob < ApplicationJob
  def perform
  return unless Flipper.enabled?(:scheduled_emails)
 
- patient_sessions.each do |patient_session|
+ date = Date.tomorrow
+
+ patient_sessions(date).each do |patient_session|
  patient_session.consents_to_send_communication.each do |consent|
  SessionMailer.with(consent:, patient_session:).reminder.deliver_later
  TextDeliveryJob.perform_later(
@@ -17,16 +19,22 @@ def perform
  end
 
  patient_session.update!(reminder_sent_at: Time.zone.now)
+
+ SessionNotification.create!(
+ patient: patient_session.patient,
+ session: patient_session.session,
+ session_date: date
+ )
  end
  end
 
  private
 
- def patient_sessions
+ def patient_sessions(date)
  PatientSession
- .includes(:consents)
+ .includes(:consents, :patient)
  .joins(:session)
- .merge(Session.has_date(Date.tomorrow))
+ .merge(Session.has_date(date))
  .reminder_not_sent
  end
 end

spec/jobs/session_reminders_job_spec.rb

@@ -4,6 +4,7 @@
  subject(:perform_now) { described_class.perform_now }
 
  before { Flipper.enable(:scheduled_emails) }
+ after { Flipper.disable(:scheduled_emails) }
 
  let(:programme) { create(:programme) }
  let(:parents) { create_list(:parent, 2, :recorded) }
@@ -40,6 +41,19 @@
  )
  end
 
+ it "updates the reminder_sent_at attribute for patient sessions" do
+ expect { perform_now }.to(
+ change { patient_session.reload.reminder_sent_at }
+ )
+ end
+
+ it "records a notification" do
+ expect { perform_now }.to change(
+ SessionNotification.where(patient:, session:),
+ :count
+ ).by(1)
+ end
+
  context "when already sent" do
  before { patient_session.update!(reminder_sent_at: Time.zone.now) }
 
@@ -53,12 +67,16 @@
  it "doesn't sent a reminder text" do
  expect { perform_now }.not_to have_enqueued_text(:session_reminder)
  end
- end
 
- it "updates the reminder_sent_at attribute for patient sessions" do
- expect { perform_now }.to(
- change { patient_session.reload.reminder_sent_at }
- )
+ it "doesn't change reminder_sent_at" do
+ expect { perform_now }.not_to(
+ change { patient_session.reload.reminder_sent_at }
+ )
+ end
+
+ it "doesn't record a notification" do
+ expect { perform_now }.not_to change(SessionNotification, :count)
+ end
  end
  end