Permit recording a Flu vaccination

This refactors the logic related to determining the common delivery
sites and the delivery method when recording a vaccination to allow for
Flu vaccinations to be recorded.

For now this doens't implement the full logic, specifically it doesn't
pick the delivery method according to the consent, but I've left a
`TODO` common so we know where to put this logic in the future.

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/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

@@ -79,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,

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/models/programme_spec.rb

@@ -110,38 +110,8 @@
     end
   end
 
-  describe "#common_delivery_sites" do
-    subject(:common_delivery_sites) { programme.common_delivery_sites }
-
-    context "with a Flu programme" do
-      let(:programme) { build(:programme, :flu) }
-
-      it "raises an error" do
-        expect { common_delivery_sites }.to raise_error(NotImplementedError)
-      end
-    end
-
-    context "with an HPV programme" do
-      let(:programme) { build(:programme, :hpv) }
-
-      it { should eq(%w[left_arm_upper_position right_arm_upper_position]) }
-    end
-
-    context "with an MenACWY programme" do
-      let(:programme) { build(:programme, :menacwy) }
-
-      it { should eq(%w[left_arm_upper_position right_arm_upper_position]) }
-    end
-
-    context "with an Td/IPV programme" do
-      let(:programme) { build(:programme, :td_ipv) }
-
-      it { should eq(%w[left_arm_upper_position right_arm_upper_position]) }
-    end
-  end
-
   describe "#vaccinated_dose_sequence" do
-    subject(:vaccinated_dose_sequence) { programme.vaccinated_dose_sequence }
+    subject { programme.vaccinated_dose_sequence }
 
     context "with a Flu programme" do
       let(:programme) { build(:programme, :flu) }