Merge pull request #4746 from nhsuk/offline-spreadsheet-triage-status

Align the triage status between the Mavis UI and the offline spreadsheet

Author: benilovj

app/helpers/triages_helper.rb

@@ -1,7 +1,9 @@
 # frozen_string_literal: true
 
 module TriagesHelper
-  def triage_status_tag(triage)
+  def triage_status_text(triage)
+    return if triage.nil?
+
     status_method =
       if triage.programme.has_multiple_vaccine_methods? &&
            triage.vaccine_method.present?
@@ -10,7 +12,11 @@ def triage_status_tag(triage)
         triage.status
       end
 
-    text = Triage.human_enum_name(:status, status_method)
+    Triage.human_enum_name(:status, status_method)
+  end
+
+  def triage_status_tag(triage)
+    text = triage_status_text(triage)
 
     colour =
       if triage.invalidated?

app/lib/reports/offline_session_exporter.rb

@@ -2,6 +2,7 @@
 
 class Reports::OfflineSessionExporter
   include Reports::ExportFormatters
+  include TriagesHelper
 
   def initialize(session)
     @session = session
@@ -290,7 +291,7 @@ def add_patient_cells(row, patient:, programme:)
         }
       }
     )
-    row[:triage_status] = triage&.status&.humanize
+    row[:triage_status] = triage_status_text(triage)
     row[:triaged_by] = triage&.performed_by&.full_name
     row[:triage_date] = triage&.created_at
     row[:triage_notes] = triage&.notes

spec/factories/patients.rb

@@ -555,6 +555,18 @@
         end
       end
 
+      triages do
+        programmes.map do |programme|
+          association(
+            :triage,
+            :ready_to_vaccinate,
+            :nasal_only,
+            patient: instance,
+            programme:
+          )
+        end
+      end
+
       consent_statuses do
         programmes.map do |programme|
           association(

spec/factories/triage.rb

@@ -59,6 +59,10 @@
       vaccine_method { "injection" }
     end
 
+    trait :nasal_only do
+      vaccine_method { "nasal" }
+    end
+
     trait :do_not_vaccinate do
       status { "do_not_vaccinate" }
       vaccine_method { nil }

spec/features/flu_vaccination_hca_pgd_supply_spec.rb

@@ -55,7 +55,8 @@ def given_a_session_exists
         vaccine: @programme.vaccines.nasal.first
       )
 
-    @nurse = create(:nurse, team: @team)
+    @nurse =
+      create(:nurse, team: @team, given_name: "Supplying", family_name: "Nurse")
     @user = create(:healthcare_assistant, team: @team)
 
     @session =
@@ -140,7 +141,7 @@ def then_i_am_able_to_vaccinate_them_nasal_only
     expect(page).not_to have_content("injected flu instead")
 
     check "I have checked that the above statements are true"
-    select @nurse.full_name
+    select "NURSE, Supplying"
     within all("section")[1] do
       choose "Yes"
     end

spec/lib/reports/offline_session_exporter_spec.rb

@@ -226,6 +226,32 @@ def validation_formula(worksheet:, column_name:, row: 1)
           end
         end
 
+        context "with a triage assessment" do
+          let!(:patient) do
+            create(:patient, :triage_ready_to_vaccinate, session:)
+          end
+
+          it "adds a row with the triage details" do
+            expect(rows.count).to eq(1)
+            expected_status =
+              (
+                if programme.flu?
+                  "Safe to vaccinate with injection"
+                else
+                  "Safe to vaccinate"
+                end
+              )
+            expect(rows.first["TRIAGE_STATUS"]).to eq(expected_status)
+            expect(rows.first["TRIAGED_BY"]).to be_present
+
+            triage = patient.triages.find_by(programme:, academic_year:)
+            expect(Time.zone.parse(rows.first["TRIAGE_DATE"]).to_i).to eq(
+              triage.created_at.to_i
+            )
+            expect(rows.first["TRIAGE_NOTES"]).to eq(triage.notes)
+          end
+        end
+
         context "with a vaccinated patient" do
           before { create(:patient_location, patient:, session:) }
 
@@ -1057,6 +1083,29 @@ def validation_formula(worksheet:, column_name:, row: 1)
     end
 
     include_examples "generates a report"
+
+    context "with a triage assessment for injection only" do
+      subject(:rows) { worksheet_to_hashes(workbook.worksheets[0]) }
+
+      let(:session) { create(:session, programmes: [programme]) }
+      let(:patient) do
+        create(
+          :patient,
+          :consent_given_nasal_triage_safe_to_vaccinate_nasal,
+          session:
+        )
+      end
+      let(:workbook) { RubyXL::Parser.parse_buffer(call) }
+
+      it "adds a row with the triage details" do
+        patient
+
+        expect(rows.count).to eq(1)
+        expect(rows.first["TRIAGE_STATUS"]).to eq(
+          "Safe to vaccinate with nasal spray"
+        )
+      end
+    end
   end
 
   context "HPV programme" do