Merge pull request #6275 from NHSDigital/improved-spec-coverage-for-comms-scenarios

Improved spec coverage for comms scenarios

Author: benilovj

spec/features/mmr_already_had_spec.rb

@@ -91,6 +91,18 @@
     then_i_see_the_mmr_date_instead_of_the_mmr_or_mmrv_page
   end
 
+  scenario "parents are notified when their child's prior vaccination is added to the record" do
+    given_an_mmr_programme_with_a_session
+    and_a_patient_with_a_consenting_parent_is_in_the_session
+    and_the_patients_prior_vaccination_is_recorded
+
+    when_the_status_updater_runs
+
+    when_the_prior_vaccination_notification_is_sent
+    then_the_parent_receives_a_vaccination_already_had_email
+    and_the_parent_receives_a_vaccination_already_had_sms
+  end
+
   scenario "record a patient born after January 2020 as already had their 2nd MMRV dose outside the school session" do
     given_an_mmr_programme_with_a_session
     and_a_patient_is_in_the_session_born_after_january_2020
@@ -431,4 +443,48 @@ def and_it_includes_the_mmrv_programme_and_both_reported_and_performed_at_dates
         "Vaccination given #{@vaccination_date.strftime("%-d %B %Y")}"
     )
   end
+
+  def and_a_patient_with_a_consenting_parent_is_in_the_session
+    @parent = create(:parent, phone_receive_updates: true)
+    @patient =
+      create(
+        :patient,
+        session: @session,
+        date_of_birth: Date.new(2020, 1, 1),
+        parents: [@parent]
+      )
+    create(
+      :consent,
+      :given,
+      patient: @patient,
+      programme: @programme,
+      parent: @parent,
+      team: @session.team
+    )
+  end
+
+  def and_the_patients_prior_vaccination_is_recorded
+    @vaccination_record =
+      create(
+        :vaccination_record,
+        :already_had,
+        :sourced_from_manual_report,
+        patient: @patient,
+        programme: @programme,
+        session: @session,
+        performed_at: 6.months.ago
+      )
+  end
+
+  def when_the_prior_vaccination_notification_is_sent
+    AlreadyHadNotificationSender.call(vaccination_record: @vaccination_record)
+  end
+
+  def then_the_parent_receives_a_vaccination_already_had_email
+    expect_email_to @parent.email, :vaccination_already_had
+  end
+
+  def and_the_parent_receives_a_vaccination_already_had_sms
+    expect_sms_to @parent.phone, :vaccination_already_had, :any
+  end
 end

spec/features/parental_consent_clinic_spec.rb

@@ -69,6 +69,28 @@
     then_the_nurse_should_see_home_schooled
   end
 
+  scenario "submitting a consent form with mismatched contact details warns existing parent" do
+    stub_pds_search_to_return_no_patients
+
+    given_an_hpv_programme_is_underway
+    and_the_patient_has_a_parent_with_known_contact_details
+
+    when_i_go_to_the_consent_form
+    and_i_fill_in_my_childs_name_and_birthday
+    when_i_choose_yes # home educated
+    then_i_see_the_parent_step
+
+    when_i_give_consent
+    and_i_answer_no_to_all_the_medical_questions
+    then_i_can_check_my_answers
+
+    when_i_submit_the_consent_form
+    and_i_refuse_to_answer_questions_on_ethnicity
+    then_i_see_a_confirmation_page_in_clinic
+
+    then_the_existing_parent_receives_a_contact_details_mismatch_warning
+  end
+
   scenario "Child attending a clinic is not in education" do
     stub_pds_search_to_return_no_patients
 
@@ -243,6 +265,7 @@ def then_i_see_a_confirmation_page_in_clinic
     expect(page).not_to have_content("at school")
 
     perform_enqueued_jobs # match consent form with patient
+    expect_email_to "jane@example.com", :consent_confirmation_clinic, :any
   end
 
   def when_the_nurse_checks_the_school_moves
@@ -306,4 +329,28 @@ def and_i_refuse_to_answer_questions_on_ethnicity
     choose "No, skip the ethnicity questions"
     click_on "Continue"
   end
+
+  def and_the_patient_has_a_parent_with_known_contact_details
+    @existing_parent =
+      create(
+        :parent,
+        email: "original@example.com",
+        phone: "07700 900100",
+        phone_receive_updates: true
+      )
+    create(
+      :parent_relationship,
+      :mother,
+      patient: @child,
+      parent: @existing_parent
+    )
+  end
+
+  def then_the_existing_parent_receives_a_contact_details_mismatch_warning
+    perform_enqueued_jobs
+    expect_email_to "original@example.com",
+                    :consent_unknown_contact_details_warning,
+                    :any
+    expect_sms_to "07700 900100", :consent_unknown_contact_details_warning, :any
+  end
 end

spec/features/scheduled_consent_requests_and_reminders_spec.rb

@@ -43,9 +43,16 @@
     ["07700 900000", "07700 900001", "07700 900002", "07700 900003"]
   end
 
+  let(:mmrv_parent_emails) do
+    %w[parent1.child3@example.com parent2.child3@example.com]
+  end
+
+  let(:mmrv_parent_phones) { ["07700 900004", "07700 900005"] }
+
   scenario "Consent requests and reminders are sent automatically" do
     given_my_team_is_running_all_vaccination_programmes
     and_one_unscheduled_session_exists_with_two_children_and_two_parents_each
+    and_an_mmrv_eligible_child_with_two_parents_exists
     and_i_am_signed_in
 
     when_i_go_to_my_team_page
@@ -75,7 +82,7 @@ def given_my_team_is_running_all_vaccination_programmes
       Programme.td_ipv
     ]
     @team = create(:team, :with_one_nurse, :with_generic_clinic, programmes:)
-    @location = create(:school, :secondary, team: @team)
+    @location = create(:school, team: @team)
     @session =
       create(
         :session,
@@ -116,6 +123,33 @@ def and_one_unscheduled_session_exists_with_two_children_and_two_parents_each
     end
   end
 
+  def and_an_mmrv_eligible_child_with_two_parents_exists
+    parents = [
+      create(
+        :parent,
+        email: "parent1.child3@example.com",
+        phone: "07700900004",
+        phone_receive_updates: true
+      ),
+      create(
+        :parent,
+        email: "parent2.child3@example.com",
+        phone: "07700900005",
+        phone_receive_updates: true
+      )
+    ]
+    # Child3 is born after the MMRV eligibility cutoff (2020-01-01) so they
+    # receive MMRV consent request and reminder emails.
+    create(
+      :patient,
+      date_of_birth: Date.new(2020, 6, 1),
+      session: @session,
+      given_name: "Child3",
+      family_name: "Test",
+      parents:
+    )
+  end
+
   def and_i_am_signed_in
     sign_in @user
   end
@@ -205,6 +239,16 @@ def then_all_four_parents_received_all_programme_consent_requests
     parent_phones.each do |phone|
       expect_sms_to(phone, :consent_school_request, :any)
     end
+
+    mmrv_parent_emails.each do |email|
+      expect(email_deliveries).to include(
+        matching_notify_email(to: email, template: :consent_school_request_mmrv)
+      )
+    end
+
+    mmrv_parent_phones.each do |phone|
+      expect_sms_to(phone, :consent_school_request, :any)
+    end
   end
 
   def then_all_four_parents_received_all_programme_initial_reminders
@@ -223,6 +267,19 @@ def then_all_four_parents_received_all_programme_initial_reminders
     parent_phones.each do |phone|
       expect_sms_to(phone, :consent_school_reminder, :any)
     end
+
+    mmrv_parent_emails.each do |email|
+      expect(email_deliveries).to include(
+        matching_notify_email(
+          to: email,
+          template: :consent_school_initial_reminder_mmrv
+        )
+      )
+    end
+
+    mmrv_parent_phones.each do |phone|
+      expect_sms_to(phone, :consent_school_reminder, :any)
+    end
   end
 
   def then_all_four_parents_received_all_programme_subsequent_reminders
@@ -241,5 +298,18 @@ def then_all_four_parents_received_all_programme_subsequent_reminders
     parent_phones.each do |phone|
       expect_sms_to(phone, :consent_school_reminder, :any)
     end
+
+    mmrv_parent_emails.each do |email|
+      expect(email_deliveries).to include(
+        matching_notify_email(
+          to: email,
+          template: :consent_school_subsequent_reminder_mmrv
+        )
+      )
+    end
+
+    mmrv_parent_phones.each do |phone|
+      expect_sms_to(phone, :consent_school_reminder, :any)
+    end
   end
 end

spec/features/session_school_reminder_spec.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+describe "Session school reminders" do
+  scenario "parents with consent given receive a reminder the day before a session" do
+    given_a_school_session_is_scheduled_for_tomorrow
+    and_a_patient_has_given_consent
+
+    when_school_session_reminders_are_sent
+
+    then_the_consenting_parent_receives_an_email_reminder
+    and_the_consenting_parent_receives_an_sms_reminder
+  end
+
+  def given_a_school_session_is_scheduled_for_tomorrow
+    programme = Programme.hpv
+    @team = create(:team, programmes: [programme])
+    school = create(:school, team: @team)
+    @session =
+      create(
+        :session,
+        date: Date.tomorrow,
+        team: @team,
+        programmes: [programme],
+        location: school
+      )
+  end
+
+  def and_a_patient_has_given_consent
+    @patient =
+      create(
+        :patient,
+        :consent_given_triage_not_needed,
+        session: @session,
+        team: @team,
+        programmes: @session.programmes
+      )
+    @parent = @patient.consents.first.parent
+  end
+
+  def when_school_session_reminders_are_sent
+    EnqueueSchoolSessionRemindersJob.perform_now
+    perform_enqueued_jobs
+    Sidekiq::Job.drain_all
+  end
+
+  def then_the_consenting_parent_receives_an_email_reminder
+    expect_email_to @parent.email, :session_school_reminder
+  end
+
+  def and_the_consenting_parent_receives_an_sms_reminder
+    expect_sms_to @parent.phone, :session_school_reminder, :any
+  end
+end