Hide vaccination tallies table for admins

Author: jhenderson

app/components/app_vaccinations_summary_table_component.html.erb

@@ -1,29 +1,36 @@
-<%= govuk_table do |table| %>
-  <% table.with_head do |head| %>
-    <% head.with_row do |row| %>
-      <% row.with_cell(text: "Vaccine") %>
-      <% row.with_cell(text: "Administered") %>
-      <% row.with_cell(text: "Default batch") %>
-    <% end %>
-  <% end %>
-
-  <% table.with_body do |body| %>
-    <% count_by_vaccine.each do |(vaccine, count)| %>
-      <% body.with_row do |row| %>
-        <% row.with_cell do %>
-          <%= vaccine.brand %>
-        <% end %>
-        <% row.with_cell do %>
-          <%= count %>
+<% if policy(VaccinationRecord).new? %>
+  <div class="nhsuk-u-margin-top-3">
+    <%= render AppDetailsComponent.new(summary: "Your vaccinations today", expander: true) do %>
+      <%= govuk_table do |table| %>
+        <% table.with_head do |head| %>
+          <% head.with_row do |row| %>
+            <% row.with_cell(text: "Vaccine") %>
+            <% row.with_cell(text: "Administered") %>
+            <% row.with_cell(text: "Default batch") %>
+          <% end %>
         <% end %>
-        <% row.with_cell do %>
-          <% batch = default_batch(vaccine) %>
-          <%= batch&.name || "Not set" %>
-          <% if batch %>
-            <%= link_to "Change", batch_session_record_path(session, vaccine.programme, vaccine.method) %>
+
+        <% table.with_body do |body| %>
+          <% count_by_vaccine.each do |(vaccine, count)| %>
+            <% body.with_row do |row| %>
+              <% row.with_cell do %>
+                <%= vaccine.brand %>
+              <% end %>
+              <% row.with_cell do %>
+                <%= count %>
+              <% end %>
+              <% row.with_cell do %>
+                <% batch = default_batch(vaccine) %>
+                <%= batch&.name || "Not set" %>
+                <% if batch %>
+                  <%= link_to "Change", batch_session_record_path(session, vaccine.programme, vaccine.method) %>
+                <% end %>
+              <% end %>
+            <% end %>
           <% end %>
         <% end %>
       <% end %>
     <% end %>
-  <% end %>
+  </div>
 <% end %>
+

app/components/app_vaccinations_summary_table_component.rb

@@ -11,7 +11,7 @@ def initialize(current_user:, session:, request_session:)
 
   attr_reader :session, :request_session, :current_user
 
-  delegate :govuk_table, to: :helpers
+  delegate :govuk_table, :policy, to: :helpers
 
   def count_by_vaccine
     vaccines = session.vaccines.active.includes(:programme).order(:brand)

app/views/sessions/record/show.html.erb

@@ -22,11 +22,7 @@
 
     <div class="app-grid-column-results">
       <%= render AppSessionNeedsReviewWarningComponent.new(session: @session) %>
-      <div class="nhsuk-u-margin-top-3">
-        <%= render AppDetailsComponent.new(summary: "Your vaccinations today", expander: true) do %>
-          <%= render AppVaccinationsSummaryTableComponent.new(current_user:, session: @session, request_session: session) %>
-        <% end %>
-      </div>
+      <%= render AppVaccinationsSummaryTableComponent.new(current_user:, session: @session, request_session: session) %>
       <%= render AppSearchResultsComponent.new(@pagy, label: "children") do %>
         <% @patient_sessions.each do |patient_session| %>
           <%= render AppPatientSessionSearchResultCardComponent.new(

spec/features/tallying_spec.rb

@@ -24,6 +24,12 @@
     then_i_see_my_vaccination_tallies_with_all_zero_values_with_default_batches
   end
 
+  scenario "when an admin is viewing the record tab for a session" do
+    given_a_session_for_hpv_and_flu_is_running_today
+    when_i_visit_the_session_record_tab_as_an_admin
+    then_i_do_not_see_the_vaccination_tallies_table
+  end
+
   def given_a_session_for_hpv_and_flu_is_running_today
     @flu_programme = create(:programme, :flu, vaccines: [])
     @hpv_programme = create(:programme, :hpv, vaccines: [])
@@ -59,6 +65,11 @@ def when_i_visit_the_session_record_tab
     visit session_record_path(@session)
   end
 
+  def when_i_visit_the_session_record_tab_as_an_admin
+    sign_in @user, role: :medical_secretary
+    visit session_record_path(@session)
+  end
+
   def and_click_on_change_default_batch_link
     within ".nhsuk-table" do
       click_on "Change"
@@ -166,4 +177,8 @@ def then_i_see_my_vaccination_tallies_with_all_zero_values_with_default_batches
   def and_i_click_on_the_expander_your_vaccinations_today
     find("span", text: "Your vaccinations today").click
   end
+
+  def then_i_do_not_see_the_vaccination_tallies_table
+    expect(page).to have_no_content("Your vaccinations today")
+  end
 end