Amend AppPatientVaccinationTableComponent to show/hide extra details

The national reporting teams uses the same vaccination table but requires more details on vaccination records like
location and source, whereas full-fat Mavis doesn't need so much information. The new `show_details` arg controls
whether to show or hide this info.

Author: murugapl

app/components/app_patient_vaccination_table_component.html.erb

@@ -1,13 +1,13 @@
 <% if vaccination_records.present? %>
   <%= govuk_table(html_attributes: { class: "nhsuk-table-responsive" }) do |table| %>
-    <% table.with_caption(text: "Vaccination records", size: "s") if show_caption %>
+    <% table.with_caption(text: "Vaccination outcomes", size: "s") if show_caption %>
 
     <% table.with_head do |head| %>
       <% head.with_row do |row| %>
         <% row.with_cell(text: "Date") %>
-        <% row.with_cell(text: "Location") %>
+        <% row.with_cell(text: "Location") if show_details %>
         <% row.with_cell(text: "Programme") if show_programme %>
-        <% row.with_cell(text: "Source") %>
+        <% row.with_cell(text: "Source") if show_details %>
         <% row.with_cell(text: "Outcome") %>
       <% end %>
     <% end %>
@@ -17,19 +17,21 @@
         <% body.with_row do |row| %>
           <% row.with_cell do %>
             <span class="nhsuk-table-responsive__heading">Date</span>
-            <%= link_to vaccination_record.performed_at.to_date.to_fs(:long),
+            <%= link_to vaccination_record.performed_at.to_fs(:long),
                         vaccination_record_path(vaccination_record) %>
           <% end %>
 
-          <% row.with_cell do %>
-            <span class="nhsuk-table-responsive__heading">Location</span>
-            <%= helpers.vaccination_record_location(vaccination_record) %>
+          <% if show_details %>
+            <% row.with_cell do %>
+              <span class="nhsuk-table-responsive__heading">Location</span>
+              <%= helpers.vaccination_record_location(vaccination_record) %>
 
-            <% if (location = vaccination_record.location) && location.has_address? %>
-              <br />
-              <span class="nhsuk-u-secondary-text-colour">
-                <%= helpers.format_address_single_line(location) %>
-              </span>
+              <% if (location = vaccination_record.location) && location.has_address? %>
+                <br />
+                <span class="nhsuk-u-secondary-text-colour">
+                  <%= helpers.format_address_single_line(location) %>
+                </span>
+              <% end %>
             <% end %>
           <% end %>
 
@@ -40,9 +42,11 @@
             <% end %>
           <% end %>
 
-          <% row.with_cell do %>
-            <span class="nhsuk-table-responsive__heading">Source</span>
-            <%= vaccination_record_source(vaccination_record) %>
+          <% if show_details %>
+            <% row.with_cell do %>
+              <span class="nhsuk-table-responsive__heading">Source</span>
+              <%= vaccination_record_source(vaccination_record) %>
+            <% end %>
           <% end %>
 
           <% row.with_cell do %>

app/components/app_patient_vaccination_table_component.rb

@@ -1,18 +1,25 @@
 # frozen_string_literal: true
 
 class AppPatientVaccinationTableComponent < ViewComponent::Base
-  def initialize(patient, academic_year:, programme: nil, show_caption: false)
+  def initialize(
+    patient,
+    academic_year:,
+    programme: nil,
+    show_caption: false,
+    show_details: true
+  )
     @patient = patient
     @academic_year = academic_year
     @programme = programme
     @show_caption = show_caption
+    @show_details = show_details
   end
 
   private
 
   delegate :govuk_table, :vaccination_record_source, to: :helpers
 
-  attr_reader :patient, :academic_year, :programme, :show_caption
+  attr_reader :patient, :academic_year, :programme, :show_caption, :show_details
 
   def show_programme = programme.nil?
 

spec/components/app_patient_vaccination_table_component_spec.rb

@@ -4,13 +4,20 @@
   subject { render_inline(component) }
 
   let(:component) do
-    described_class.new(patient, academic_year:, programme:, show_caption:)
+    described_class.new(
+      patient,
+      academic_year:,
+      programme:,
+      show_caption:,
+      show_details:
+    )
   end
 
   let(:patient) { create(:patient) }
   let(:academic_year) { 2023 }
   let(:programme) { nil }
   let(:show_caption) { false }
+  let(:show_details) { true }
 
   it { should have_content("No vaccinations") }
 
@@ -50,6 +57,15 @@
     it { should have_content("Vaccinated") }
     it { should have_content("HPV") }
 
+    context "when show_details is false" do
+      let(:show_details) { false }
+
+      it { should have_link("1 January 2024") }
+      it { should have_content("Vaccinated") }
+      it { should_not have_content("Test School") }
+      it { should_not have_content("Waterloo Road, London, SE1 8TY") }
+    end
+
     context "when showing records from a specific programme" do
       let(:programme) { vaccination_record_programme }
 

spec/features/hpv_vaccination_administered_spec.rb

@@ -274,7 +274,7 @@ def then_i_see_that_the_status_is_vaccinated
   end
 
   def and_i_see_the_vaccination_details
-    expect(page).to have_content("Vaccination records")
+    expect(page).to have_content("Vaccination outcomes")
     click_on Date.current.to_fs(:long)
 
     expect(page).to have_content("Vaccination details")

spec/features/menacwy_vaccination_administered_spec.rb

@@ -201,7 +201,7 @@ def then_i_see_that_the_status_is_vaccinated
   end
 
   def and_i_see_the_vaccination_details
-    expect(page).to have_content("Vaccination records")
+    expect(page).to have_content("Vaccination outcomes")
     click_on Date.current.to_fs(:long)
 
     expect(page).to have_content("Vaccination details")

spec/features/td_ipv_vaccination_administered_spec.rb

@@ -201,7 +201,7 @@ def then_i_see_that_the_status_is_vaccinated
   end
 
   def and_i_see_the_vaccination_details
-    expect(page).to have_content("Vaccination records")
+    expect(page).to have_content("Vaccination outcomes")
     click_on Date.current.to_fs(:long)
 
     expect(page).to have_content("Vaccination details")