Merge pull request #3835 from nhsuk/ensure-triage-form-records-correct-vaccine-method

Ensure triage form records correct vaccine method

Author: benilovj

app/forms/triage_form.rb

@@ -73,7 +73,7 @@ def triage_attributes
       performed_by: current_user,
       programme:,
       status:,
-      vaccine_method: "injection"
+      vaccine_method:
     }
   end
 

spec/factories/consents.rb

@@ -84,6 +84,16 @@
       recorded_by
     end
 
+    trait :given_injection do
+      given
+      vaccine_methods { %w[injection] }
+    end
+
+    trait :given_nasal do
+      given
+      vaccine_methods { %w[nasal] }
+    end
+
     trait :self_consent do
       route { "self_consent" }
       parent { nil }

spec/factories/patient_consent_statuses.rb

@@ -31,5 +31,15 @@
       status { "given" }
       vaccine_methods { %w[injection] }
     end
+
+    trait :given_injection_only do
+      status { "given" }
+      vaccine_methods { %w[injection] }
+    end
+
+    trait :given_nasal_only do
+      status { "given" }
+      vaccine_methods { %w[nasal] }
+    end
   end
 end

spec/factories/patient_sessions.rb

@@ -103,6 +103,32 @@
       end
     end
 
+    trait :consent_given_injection_only_triage_needed do
+      patient do
+        association :patient,
+                    :consent_given_injection_only_triage_needed,
+                    performed_by: user,
+                    programmes: session.programmes,
+                    organisation:,
+                    school:,
+                    home_educated:,
+                    year_group:
+      end
+    end
+
+    trait :consent_given_nasal_only_triage_needed do
+      patient do
+        association :patient,
+                    :consent_given_nasal_only_triage_needed,
+                    performed_by: user,
+                    programmes: session.programmes,
+                    organisation:,
+                    school:,
+                    home_educated:,
+                    year_group:
+      end
+    end
+
     trait :consent_refused do
       patient do
         association :patient,

spec/factories/patients.rb

@@ -297,6 +297,82 @@
       end
     end
 
+    trait :consent_given_injection_only_triage_needed do
+      consents do
+        programmes.map do |programme|
+          association(
+            :consent,
+            :given_injection,
+            :from_mum,
+            :health_question_notes,
+            patient: instance,
+            organisation:,
+            programme:
+          )
+        end
+      end
+
+      consent_statuses do
+        programmes.map do |programme|
+          association(
+            :patient_consent_status,
+            :given_injection_only,
+            patient: instance,
+            programme:
+          )
+        end
+      end
+
+      triage_statuses do
+        programmes.map do |programme|
+          association(
+            :patient_triage_status,
+            :required,
+            patient: instance,
+            programme:
+          )
+        end
+      end
+    end
+
+    trait :consent_given_nasal_only_triage_needed do
+      consents do
+        programmes.map do |programme|
+          association(
+            :consent,
+            :given_nasal,
+            :from_mum,
+            :health_question_notes,
+            patient: instance,
+            organisation:,
+            programme:
+          )
+        end
+      end
+
+      consent_statuses do
+        programmes.map do |programme|
+          association(
+            :patient_consent_status,
+            :given_nasal_only,
+            patient: instance,
+            programme:
+          )
+        end
+      end
+
+      triage_statuses do
+        programmes.map do |programme|
+          association(
+            :patient_triage_status,
+            :required,
+            patient: instance,
+            programme:
+          )
+        end
+      end
+    end
+
     trait :consent_refused do
       triage_not_required
 

spec/features/triage_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 describe "Triage" do
-  scenario "nurse can triage a patient" do
+  scenario "nurse can triage a patient for an HPV programme" do
     given_a_programme_with_a_running_session
     and_a_patient_who_needs_triage_exists
     and_a_patient_who_doesnt_need_triage_exists
@@ -28,6 +28,42 @@
     and_vaccination_will_happen_emails_are_sent_to_both_parents
   end
 
+  scenario "nurse can triage patients for a flu programme with different consent types" do
+    given_a_flu_programme_with_a_running_session
+    and_patients_with_different_flu_consent_types_exist
+
+    when_i_go_to_the_session_triage_tab
+    then_i_see_both_patients_who_need_triage
+
+    # First patient - injection only consent
+    when_i_go_to_the_first_patient_that_needs_triage
+    then_i_see_the_triage_options_for_injection_only_consent
+
+    when_i_record_that_they_need_triage_for_flu
+    then_i_see_the_triage_page
+    and_needs_triage_emails_are_sent_to_both_parents
+
+    when_i_record_that_they_are_safe_to_vaccinate_with_injection
+    then_i_see_the_update_triage_link
+    and_vaccination_will_happen_emails_are_sent_to_both_parents
+    and_the_vaccine_method_is_recorded_as_injection
+
+    # Second patient - nasal only consent
+    when_i_go_to_the_session_triage_tab
+    and_i_go_to_the_second_patient_that_needs_triage
+    then_i_see_the_triage_options_for_nasal_only_consent
+
+    when_i_record_that_they_need_triage_for_flu
+    then_i_see_the_triage_page
+    and_needs_triage_emails_are_sent_to_both_parents
+
+    when_i_go_to_the_second_patient
+    and_i_record_that_they_are_safe_to_vaccinate_with_nasal
+    then_i_see_the_update_triage_link
+    and_vaccination_will_happen_emails_are_sent_to_both_parents
+    and_the_vaccine_method_is_recorded_as_nasal
+  end
+
   def given_a_programme_with_a_running_session
     programmes = [create(:programme, :hpv)]
     @organisation = create(:organisation, :with_one_nurse, programmes:)
@@ -42,6 +78,20 @@ def given_a_programme_with_a_running_session
     @session = create(:session, organisation: @organisation, programmes:)
   end
 
+  def given_a_flu_programme_with_a_running_session
+    programmes = [create(:programme, :flu)]
+    @organisation = create(:organisation, :with_one_nurse, programmes:)
+
+    @batch =
+      create(
+        :batch,
+        organisation: @organisation,
+        vaccine: programmes.first.vaccines.first
+      )
+
+    @session = create(:session, organisation: @organisation, programmes:)
+  end
+
   def and_a_patient_who_needs_triage_exists
     @patient_triage_needed =
       create(
@@ -62,6 +112,25 @@ def and_a_patient_who_needs_triage_exists
     @patient_triage_needed.reload # Make sure both consents are accessible
   end
 
+  def and_patients_with_different_flu_consent_types_exist
+    @patient_injection_only =
+      create(
+        :patient_session,
+        :consent_given_injection_only_triage_needed,
+        session: @session
+      ).patient
+
+    @patient_nasal_only =
+      create(
+        :patient_session,
+        :consent_given_nasal_only_triage_needed,
+        session: @session
+      ).patient
+
+    @patient_injection_only.reload
+    @patient_nasal_only.reload
+  end
+
   def and_a_patient_who_doesnt_need_triage_exists
     @patient_triage_not_needed =
       create(
@@ -80,6 +149,11 @@ def then_i_see_the_patient_who_needs_triage
     expect(page).to have_content(@patient_triage_needed.full_name)
   end
 
+  def then_i_see_both_patients_who_need_triage
+    expect(page).to have_content(@patient_injection_only.full_name)
+    expect(page).to have_content(@patient_nasal_only.full_name)
+  end
+
   def and_i_dont_see_the_patient_who_doesnt_need_triage
     expect(page).not_to have_content(@patient_triage_not_needed.full_name)
   end
@@ -90,10 +164,6 @@ def when_i_go_to_the_patient_that_needs_triage
     click_link @patient_triage_needed.full_name
   end
 
-  def when_i_go_to_the_patient
-    click_link @patient_triage_needed.full_name, match: :first
-  end
-
   def then_i_see_the_triage_options
     expect(page).to have_selector :heading, "Is it safe to vaccinate"
   end
@@ -103,12 +173,27 @@ def when_i_record_that_they_need_triage
     click_button "Save triage"
   end
 
+  def when_i_record_that_they_need_triage_for_flu
+    choose "No, keep in triage"
+    click_button "Save triage"
+  end
+
   def when_i_record_that_they_are_safe_to_vaccinate
     click_link "Update triage"
     choose "Yes, it’s safe to vaccinate"
     click_button "Save triage"
   end
 
+  def when_i_record_that_they_are_safe_to_vaccinate_with_injection
+    choose "Yes, it’s safe to vaccinate with injected vaccine"
+    click_button "Save triage"
+  end
+
+  def and_i_record_that_they_are_safe_to_vaccinate_with_nasal
+    choose "Yes, it’s safe to vaccinate with nasal spray"
+    click_button "Save triage"
+  end
+
   def when_i_do_not_vaccinate
     choose "No, do not vaccinate"
     click_button "Save triage"
@@ -126,21 +211,84 @@ def then_i_see_the_update_triage_link
     expect(page).to have_link "Update triage"
   end
 
+  def then_i_see_the_triage_page
+    expect(page).to have_selector :heading, "Is it safe to vaccinate"
+  end
+
   def and_needs_triage_emails_are_sent_to_both_parents
-    @patient_triage_needed.parents.each do |parent|
+    current_patient =
+      @patient_triage_needed || @patient_injection_only || @patient_nasal_only
+    current_patient.parents.each do |parent|
       expect_email_to parent.email, :consent_confirmation_triage, :any
     end
   end
 
   def and_vaccination_wont_happen_emails_are_sent_to_both_parents
-    @patient_triage_needed.parents.each do |parent|
+    current_patient =
+      @patient_triage_needed || @patient_injection_only || @patient_nasal_only
+    current_patient.parents.each do |parent|
       expect_email_to parent.email, :triage_vaccination_wont_happen, :any
     end
   end
 
   def and_vaccination_will_happen_emails_are_sent_to_both_parents
-    @patient_triage_needed.parents.each do |parent|
+    current_patient =
+      @patient_triage_needed || @patient_injection_only || @patient_nasal_only
+    current_patient.parents.each do |parent|
       expect_email_to parent.email, :triage_vaccination_will_happen, :any
     end
   end
+
+  def and_the_vaccine_method_is_recorded_as_injection
+    current_patient =
+      @patient_triage_needed || @patient_injection_only || @patient_nasal_only
+    triage = current_patient.triages.last
+    expect(triage.vaccine_method).to eq("injection")
+  end
+
+  def when_i_go_to_the_first_patient_that_needs_triage
+    choose "Needs triage"
+    click_on "Update results"
+    click_link @patient_injection_only.full_name
+  end
+
+  def then_i_see_the_triage_options_for_injection_only_consent
+    expect(page).to have_selector :heading,
+                  "Is it safe to vaccinate #{@patient_injection_only.given_name}?"
+    expect(page).to have_content(
+      "The parent has consented to the injected vaccine only"
+    )
+    expect(page).to have_field(
+      "Yes, it’s safe to vaccinate with injected vaccine",
+      type: "radio"
+    )
+  end
+
+  def and_i_go_to_the_second_patient_that_needs_triage
+    choose "Needs triage"
+    click_on "Update results"
+    click_link @patient_nasal_only.full_name
+  end
+
+  def then_i_see_the_triage_options_for_nasal_only_consent
+    expect(page).to have_selector :heading,
+                  "Is it safe to vaccinate #{@patient_nasal_only.given_name}?"
+    expect(page).to have_content(
+      "The parent has consented to the nasal spray only"
+    )
+    expect(page).to have_field(
+      "Yes, it’s safe to vaccinate with nasal spray",
+      type: "radio"
+    )
+  end
+
+  def when_i_go_to_the_second_patient
+    click_link "Triage"
+    click_link @patient_nasal_only.full_name, match: :first
+  end
+
+  def and_the_vaccine_method_is_recorded_as_nasal
+    triage = @patient_nasal_only.triages.last
+    expect(triage.vaccine_method).to eq("nasal")
+  end
 end