Set session outcome to absent from session (#3238)

If the patient's attendance has been marked as absent, the session
outcome for that day should show as "absent from session".

Currently, the only way for a patient outcome to be "absent from
session" would be if the vaccination was recorded as not administered
for that reason, however we don't present "Absent from session" as one
of the options available to the nurses. Instead, the expectation is that
they would use the registration status.

In theory, you could import a vaccination record with this status, so we
still need to support that, but it's not the usual way this would work.

Potentially we could remove "absent from school" and "absent from
session" as vaccination record outcomes.

Author: thomasleese

app/components/app_outcome_banner_component.rb

@@ -61,6 +61,10 @@ def triage
     @triage ||= patient.triage_outcome.latest[programme]
   end
 
+  def session_attendance
+    @session_attendance ||= patient_session.register_outcome.latest
+  end
+
   def show_location?
     # location only makes sense if an attempt to vaccinate on site was made
     vaccination_record.present?
@@ -87,6 +91,8 @@ def reason_do_not_vaccinate
     key =
       if vaccination_record&.not_administered?
         "patient_session_statuses.unable_to_vaccinate.banner_explanation.#{vaccination_record.outcome}"
+      elsif session_attendance&.attending == false
+        "patient_session_statuses.unable_to_vaccinate.banner_explanation.absent_from_session"
       else
         "patient_session_statuses.unable_to_vaccinate.banner_explanation.#{status}"
       end
@@ -102,7 +108,7 @@ def clinician_name
   end
 
   def clinician
-    @clinician ||= (vaccination_record || triage).performed_by
+    @clinician ||= (vaccination_record || triage)&.performed_by
   end
 
   def location
@@ -125,7 +131,9 @@ def date_summary
   end
 
   def last_action_time
-    @last_action_time ||= vaccination_record&.performed_at || triage&.created_at
+    @last_action_time ||=
+      vaccination_record&.performed_at || triage&.created_at ||
+        session_attendance&.created_at
   end
 
   def heading

app/models/patient_session/register_outcome.rb

@@ -56,6 +56,6 @@ def session_date
   end
 
   def all_programmes_have_outcome?
-    programmes.none? { session_outcome.none_yet?(it) }
+    programmes.none? { session_outcome.latest[it].nil? }
   end
 end

app/models/patient_session/session_outcome.rb

@@ -47,7 +47,11 @@ def latest
 
   attr_reader :patient_session
 
-  delegate :patient, :session, :programmes, to: :patient_session
+  delegate :patient,
+           :programmes,
+           :register_outcome,
+           :session,
+           to: :patient_session
   delegate :consent_outcome, :triage_outcome, to: :patient
 
   def programme_status(programme)
@@ -57,6 +61,8 @@ def programme_status(programme)
       REFUSED
     elsif triage_outcome.do_not_vaccinate?(programme)
       HAD_CONTRAINDICATIONS
+    elsif register_outcome.not_attending?
+      ABSENT_FROM_SESSION
     else
       NONE_YET
     end

spec/features/manage_attendance_spec.rb

@@ -17,6 +17,9 @@
     when_i_register_a_patient_as_absent
     then_i_see_the_absent_flash
 
+    when_i_go_to_the_session_outcomes
+    then_i_see_a_patient_is_absent
+
     when_i_go_to_a_patient
     then_the_patient_is_not_registered_yet
     and_i_am_not_able_to_vaccinate
@@ -87,7 +90,21 @@ def when_i_register_a_patient_as_absent
     click_button "Absent", match: :first
   end
 
+  def when_i_go_to_the_session_outcomes
+    click_on "Session outcomes"
+  end
+
+  def then_i_see_a_patient_is_absent
+    choose "Absent from session"
+    click_on "Update results"
+
+    expect(page).to have_content("Showing 1 to 1 of 1 children")
+  end
+
   def when_i_go_to_a_patient
+    choose "Any"
+    click_on "Update results"
+
     click_link PatientSession
                  .where
                  .missing(:session_attendances)

spec/models/patient_session/session_outcome_spec.rb

@@ -56,6 +56,12 @@
 
       it { should be(described_class::HAD_CONTRAINDICATIONS) }
     end
+
+    context "when not attending the session" do
+      before { create(:session_attendance, :absent, patient_session:) }
+
+      it { should be(described_class::ABSENT_FROM_SESSION) }
+    end
   end
 
   describe "#all" do