Merge pull request #3679 from nhsuk/prepare-flu-vaccinations

Allow testing Flu programme

Author: thomasleese

app/components/app_vaccinate_form_component.html.erb

@@ -49,15 +49,20 @@
     </h2>
 
     <%= f.govuk_radio_buttons_fieldset :administered, legend: nil do %>
-      <%= f.govuk_radio_button :administered, true, label: { text: "Yes" }, link_errors: true do %>
-        <%= f.govuk_collection_radio_buttons :delivery_site,
-                                             common_delivery_sites_options,
-                                             :value,
-                                             :label,
-                                             legend: {
-                                               text: "Where will the injection be given?",
-                                               size: "s",
-                                             } %>
+      <% if common_delivery_sites_options.length > 1 %>
+        <%= f.govuk_radio_button :administered, true, label: { text: "Yes" }, link_errors: true do %>
+          <%= f.govuk_collection_radio_buttons :delivery_site,
+                                               common_delivery_sites_options,
+                                               :value,
+                                               :label,
+                                               legend: {
+                                                 text: "Where will the injection be given?",
+                                                 size: "s",
+                                               } %>
+        <% end %>
+      <% else %>
+        <%= f.govuk_radio_button :administered, true, label: { text: "Yes" }, link_errors: true %>
+        <%= f.hidden_field :delivery_site, value: common_delivery_sites_options.first.value %>
       <% end %>
       <%= f.govuk_radio_button :administered, false, label: { text: "No" } %>
     <% end %>

app/components/app_vaccinate_form_component.rb

@@ -27,22 +27,37 @@ def url
   end
 
   def delivery_method
-    :intramuscular
+    # TODO: Check which method has been consented to.
+    programme.flu? ? :nasal_spray : :intramuscular
   end
 
   def dose_sequence
     programme.default_dose_sequence
   end
 
+  COMMON_DELIVERY_SITES = {
+    intramuscular: %w[left_arm_upper_position right_arm_upper_position],
+    nasal_spray: %w[nose]
+  }.freeze
+
+  CommonDeliverySite = Struct.new(:value, :label)
+
   def common_delivery_sites_options
-    options =
-      programme.common_delivery_sites.map do
-        OpenStruct.new(
-          value: it,
-          label: VaccinationRecord.human_enum_name(:delivery_site, it)
-        )
+    @common_delivery_sites_options ||=
+      begin
+        options =
+          COMMON_DELIVERY_SITES
+            .fetch(delivery_method)
+            .map do |value|
+              label = VaccinationRecord.human_enum_name(:delivery_site, value)
+              CommonDeliverySite.new(value:, label:)
+            end
+
+        if delivery_method == :intramuscular
+          options << CommonDeliverySite.new(value: "other", label: "Other")
+        end
+
+        options
       end
-
-    options + [OpenStruct.new(value: "other", label: "Other")]
   end
 end

app/components/app_vaccination_record_summary_component.rb

@@ -16,6 +16,7 @@ def initialize(
 
     @batch = vaccination_record.batch
     @patient = vaccination_record.patient
+    @programme = vaccination_record.programme
     @vaccine = vaccination_record.vaccine
   end
 
@@ -218,10 +219,7 @@ def outcome_value
   end
 
   def programme_value
-    highlight_if(
-      @vaccination_record.programme.name,
-      @vaccination_record.programme_id_changed?
-    )
+    highlight_if(@programme.name, @vaccination_record.programme_id_changed?)
   end
 
   def vaccine_value
@@ -318,7 +316,7 @@ def dose_number_value
   end
 
   def dose_number
-    return nil if @vaccine&.seasonal?
+    return nil if @programme.seasonal?
 
     dose_sequence = @vaccination_record.dose_sequence
 

app/controllers/draft_vaccination_records_controller.rb

@@ -199,7 +199,7 @@ def set_batches
 
     method =
       if @draft_vaccination_record.delivery_method == "nasal_spray"
-        "nasal_spray"
+        "nasal"
       else
         "injection"
       end

app/models/programme.rb

@@ -46,18 +46,16 @@ class Programme < ApplicationRecord
        { flu: "flu", hpv: "hpv", menacwy: "menacwy", td_ipv: "td_ipv" },
        validate: true
 
-  def to_param
-    type
-  end
-
-  def doubles?
-    menacwy? || td_ipv?
-  end
+  def to_param = type
 
   def name
     human_enum_name(:type)
   end
 
+  def doubles? = menacwy? || td_ipv?
+
+  def seasonal? = flu?
+
   YEAR_GROUPS_BY_TYPE = {
     "flu" => (0..11).to_a,
     "hpv" => (8..11).to_a,
@@ -81,15 +79,6 @@ def available_delivery_sites
     vaccines.flat_map(&:available_delivery_sites).uniq
   end
 
-  def common_delivery_sites
-    if hpv? || menacwy? || td_ipv?
-      %w[left_arm_upper_position right_arm_upper_position]
-    else
-      raise NotImplementedError,
-            "Common delivery sites not implemented for #{type} vaccines."
-    end
-  end
-
   DOSE_SEQUENCES = {
     "flu" => 1,
     "hpv" => 1,

app/models/vaccine.rb

@@ -51,22 +51,19 @@ class Vaccine < ApplicationRecord
 
   delegate :first_health_question, to: :health_questions
 
-  def active?
-    !discontinued
-  end
-
-  def contains_gelatine?
-    programme.flu? && nasal?
-  end
+  def active? = !discontinued
 
-  def seasonal?
-    programme.flu?
-  end
+  def contains_gelatine? = programme.flu? && nasal?
 
   AVAILABLE_DELIVERY_SITES = {
-    "injection" =>
-      VaccinationRecord.delivery_sites.keys -
-        %w[left_buttock right_buttock nose],
+    "injection" => %w[
+      left_arm_upper_position
+      left_arm_lower_position
+      right_arm_upper_position
+      right_arm_lower_position
+      left_thigh
+      right_thigh
+    ],
     "nasal" => %w[nose]
   }.freeze
 

db/seeds.rb

@@ -19,16 +19,14 @@ def create_gp_practices
   FactoryBot.create_list(:gp_practice, 30)
 end
 
-def create_organisation(ods_code:, programme_types: %w[hpv menacwy td_ipv])
+def create_organisation(ods_code:)
   organisation =
     Organisation.find_by(ods_code:) ||
       FactoryBot.create(:organisation, :with_generic_clinic, ods_code:)
 
-  programme_types
-    .map { Programme.find_by!(type: it) }
-    .each do |programme|
-      FactoryBot.create(:organisation_programme, organisation:, programme:)
-    end
+  Programme.all.find_each do |programme|
+    FactoryBot.create(:organisation_programme, organisation:, programme:)
+  end
 
   organisation
 end
@@ -85,14 +83,10 @@ def create_session(
 )
   year_groups ||= programmes.flat_map(&:year_groups).uniq
 
-  programmes.each do |programme|
-    FactoryBot.create_list(
-      :batch,
-      3,
-      organisation:,
-      vaccine: programme.vaccines.active.first
-    )
-  end
+  Vaccine
+    .active
+    .where(programme: programmes)
+    .find_each { |vaccine| FactoryBot.create(:batch, organisation:, vaccine:) }
 
   location = FactoryBot.create(:school, organisation:, year_groups:)
   date = completed ? 1.week.ago.to_date : Date.current
@@ -232,10 +226,15 @@ def create_school_moves(organisation)
 end
 
 def create_organisation_sessions(user, organisation)
+  flu = Programme.find_by!(type: "flu")
   hpv = Programme.find_by!(type: "hpv")
   menacwy = Programme.find_by!(type: "menacwy")
   td_ipv = Programme.find_by!(type: "td_ipv")
 
+  # Flu-only sessions
+  create_session(user, organisation, programmes: [flu], completed: false)
+  create_session(user, organisation, programmes: [hpv], completed: true)
+
   # HPV-only sessions
   create_session(user, organisation, programmes: [hpv], completed: false)
   create_session(user, organisation, programmes: [hpv], completed: true)

spec/components/app_vaccinate_form_component_spec.rb

@@ -1,11 +1,8 @@
 # frozen_string_literal: true
 
 describe AppVaccinateFormComponent do
-  subject { render_inline(component) }
-
-  let(:heading) { "A Heading" }
-  let(:body) { "A Body" }
-  let(:programmes) { [create(:programme, :hpv)] }
+  let(:programme) { create(:programme) }
+  let(:programmes) { [programme] }
   let(:session) { create(:session, :today, programmes:) }
   let(:patient) do
     create(
@@ -19,21 +16,12 @@
     create(:patient_session, :in_attendance, programmes:, patient:, session:)
   end
 
-  let(:vaccinate_form) do
-    VaccinateForm.new(patient_session:, programme: programmes.first)
-  end
+  let(:vaccinate_form) { VaccinateForm.new(patient_session:, programme:) }
 
   let(:component) { described_class.new(vaccinate_form) }
 
   before { patient_session.strict_loading!(false) }
 
-  it { should have_css(".nhsuk-card") }
-
-  it { should have_heading("Is Hari ready for their HPV vaccination?") }
-
-  it { should have_field("Yes") }
-  it { should have_field("No") }
-
   describe "#render?" do
     subject(:render) { component.render? }
 
@@ -53,4 +41,38 @@
       it { should be(false) }
     end
   end
+
+  describe "rendered content" do
+    subject { render_inline(component) }
+
+    it { should have_css(".nhsuk-card") }
+
+    context "with a Flu programme" do
+      let(:programme) { create(:programme, :flu) }
+
+      it { should have_heading("Is Hari ready for their Flu vaccination?") }
+
+      it { should have_field("Yes") }
+      it { should have_field("No") }
+
+      it { should_not have_field("Left arm (upper position)") }
+      it { should_not have_field("Right arm (upper position)") }
+      it { should_not have_field("Nose") }
+      it { should_not have_field("Other") }
+    end
+
+    context "with an HPV programme" do
+      let(:programme) { create(:programme, :hpv) }
+
+      it { should have_heading("Is Hari ready for their HPV vaccination?") }
+
+      it { should have_field("Yes") }
+      it { should have_field("No") }
+
+      it { should have_field("Left arm (upper position)") }
+      it { should have_field("Right arm (upper position)") }
+      it { should_not have_field("Nose") }
+      it { should have_field("Other") }
+    end
+  end
 end

spec/factories/programmes.rb

@@ -39,15 +39,7 @@
       vaccines do
         [
           association(:vaccine, :adjuvanted_quadrivalent, programme: instance),
-          association(:vaccine, :cell_quadrivalent, programme: instance),
-          association(:vaccine, :fluenz_tetra, programme: instance),
-          association(:vaccine, :quadrivalent_influenza, programme: instance),
-          association(
-            :vaccine,
-            :quadrivalent_influvac_tetra,
-            programme: instance
-          ),
-          association(:vaccine, :supemtek, programme: instance)
+          association(:vaccine, :fluenz_tetra, programme: instance)
         ]
       end
     end

spec/factories/vaccination_records.rb

@@ -72,7 +72,7 @@
 
     vaccine do
       if session
-        programme.vaccines.active.first || association(:vaccine, programme:)
+        programme.vaccines.active.sample || association(:vaccine, programme:)
       end
     end