Refactor: extract patient attendance validation

This validation is done on both draft and full vaccination record so it
would be better to extract it.

Author: MartinVanIJcken

app/models/concerns/patient_attendance_validation.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module PatientAttendanceValidation
+  extend ActiveSupport::Concern
+
+  private
+
+  def patient_attended_session
+    return if session.nil? || patient.nil?
+
+    unless patient.attending?(session:) || patient.completed?(session:)
+      date = performed_at.to_date
+      errors.add(:session_id, :patient_not_attending,
+                 patient_full_name: patient.full_name,
+                 display_date: date.today? ? "Today (#{date.to_fs(:long)})" : date.to_fs(:long),
+                 location_name: location[:name]
+      )
+    end
+  end
+end

app/models/draft_vaccination_record.rb

@@ -4,6 +4,7 @@ class DraftVaccinationRecord
   include RequestSessionPersistable
   include EditableWrapper
   include HasDoseVolume
+  include PatientAttendanceValidation
   include VaccinationRecordPerformedByConcern
   include WizardStepConcern
 
@@ -213,16 +214,5 @@ def can_change_outcome?
     outcome != "already_had" || editing? || session.nil? || session.today?
   end
 
-  def patient_attended_session
-    return if patient.nil? || session.nil?
 
-    unless patient.attending?(session:) || patient.completed?(session:)
-      date = performed_at.to_date
-      errors.add(:session_id,:patient_not_attending,
-                 patient_full_name: patient.full_name,
-                 display_date: date.today? ? "Today (#{date.to_fs(:long)})" : date.to_fs(:long),
-                 location_name: location[:name]
-      )
-    end
-  end
 end

app/models/vaccination_record.rb

@@ -52,6 +52,7 @@
 class VaccinationRecord < ApplicationRecord
   include Discard::Model
   include HasDoseVolume
+  include PatientAttendanceValidation
   include PendingChangesConcern
   include VaccinationRecordPerformedByConcern
 
@@ -186,18 +187,7 @@ def requires_location_name?
     session.nil? || location&.generic_clinic?
   end
 
-  def patient_attended_session
-    return if session.nil? || patient.nil?
-
-    unless patient.attending?(session:) || patient.completed?(session:)
-      date = performed_at.to_date
-      errors.add(:session_id,:patient_not_attending,
-                 patient_full_name: patient.full_name,
-                 display_date: date.today? ? "Today (#{date.to_fs(:long)})" : date.to_fs(:long),
-                 location_name: location[:name]
-      )
-    end
-  end
+
 
   delegate :maximum_dose_sequence, to: :programme
 end