Add patients to clinic when inviting

When inviting patients to a clinic, we can no longer assume that they
will already be in the clinic due to the previous commits which doesn't
add patients to the community clinic.

Instead we can add patients to the clinic at the point of invitation.

Jira-Issue: MAV-1430

Author: thomasleese

app/controllers/sessions/invite_to_clinic_controller.rb

@@ -3,6 +3,7 @@
 class Sessions::InviteToClinicController < ApplicationController
   before_action :set_session
   before_action :set_generic_clinic_session
+  before_action :set_patient_sessions_to_invite
   before_action :set_invitations_to_send
 
   skip_after_action :verify_policy_scoped
@@ -13,6 +14,11 @@ def edit
 
   def update
     if @session.school?
+      PatientSession.import!(
+        @patient_sessions_to_invite,
+        on_duplicate_key_ignore: true
+      )
+
       SendClinicInitialInvitationsJob.perform_later(
         @generic_clinic_session,
         school: @session.location,
@@ -44,36 +50,51 @@ def set_session
 
   def set_generic_clinic_session
     @generic_clinic_session =
-      (
-        if @session.clinic?
-          @session
-        else
-          @session.team.generic_clinic_session(
-            academic_year: @session.academic_year
-          )
-        end
-      )
+      if @session.clinic?
+        @session
+      else
+        @session.team.generic_clinic_session(
+          academic_year: @session.academic_year
+        )
+      end
   end
 
-  def set_invitations_to_send
+  def set_patient_sessions_to_invite
     session_date = @generic_clinic_session.next_date(include_today: true)
 
-    @invitations_to_send =
-      if @session.school?
-        SendClinicInitialInvitationsJob
-          .new
-          .patient_sessions(
-            @generic_clinic_session,
-            school: @session.location,
-            programmes: @session.programmes.to_a,
-            session_date:
+    if @session.school?
+      patient_sessions_in_school = @session.patient_sessions.includes(:patient)
+
+      patient_sessions_in_clinic =
+        patient_sessions_in_school.map do |patient_session|
+          PatientSession.find_or_initialize_by(
+            patient: patient_session.patient,
+            session: @generic_clinic_session
           )
-          .length
-      else
-        SendClinicSubsequentInvitationsJob
-          .new
-          .patient_sessions(@session, session_date:)
-          .length
-      end
+        end
+
+      programmes = @session.programmes.to_a
+
+      @patient_sessions_to_invite =
+        patient_sessions_in_clinic
+          .reject { it.session_notifications.any? }
+          .select do |patient_session|
+            SendClinicInitialInvitationsJob.new.should_send_notification?(
+              patient_session:,
+              programmes:,
+              session_date:
+            )
+          end
+    else
+      @patient_sessions_to_invite =
+        SendClinicSubsequentInvitationsJob.new.patient_sessions(
+          @session,
+          session_date:
+        )
+    end
+  end
+
+  def set_invitations_to_send
+    @invitations_to_send = @patient_sessions_to_invite.length
   end
 end

app/jobs/send_clinic_initial_invitations_job.rb

@@ -35,12 +35,8 @@ def patient_sessions(session, school:, programmes:, session_date:)
       )
       .where(patient: { school: })
       .reject { it.session_notifications.any? }
-      .select do
-        should_send_notification?(
-          patient_session: it,
-          programmes:,
-          session_date:
-        )
+      .select do |patient_session|
+        should_send_notification?(patient_session:, programmes:, session_date:)
       end
   end
 end

app/jobs/send_clinic_subsequent_invitations_job.rb

@@ -24,8 +24,9 @@ def patient_sessions(session, session_date:)
 
     session
       .patient_sessions
-      .eager_load(:patient)
-      .preload(
+      .joins(:patient)
+      .includes_programmes
+      .includes(
         :session_notifications,
         patient: %i[consents parents vaccination_records]
       )

app/models/team.rb

@@ -82,7 +82,12 @@ def generic_clinic_session(academic_year:)
     location = locations.generic_clinic.first
 
     sessions
-      .includes(:location, :programmes, :session_dates)
+      .includes(
+        :location,
+        :location_programme_year_groups,
+        :programmes,
+        :session_dates
+      )
       .create_with(programmes:)
       .find_or_create_by!(academic_year:, location:)
   end

spec/features/manage_school_sessions_spec.rb

@@ -100,12 +100,6 @@ def given_my_team_is_running_an_hpv_vaccination_programme
       .generic_clinic_session(academic_year: AcademicYear.current)
       .session_dates
       .create!(value: 1.month.from_now.to_date)
-
-    create(
-      :patient_session,
-      patient: @patient,
-      session: @team.generic_clinic_session(academic_year: AcademicYear.current)
-    )
   end
 
   def when_i_go_to_todays_sessions_as_a_nurse