Update session vaccinations given card

This updates the card showing the vaccinations that were given at the
session to match the latest designs in the prototype.

Jira-Issue: MAV-2645

Author: thomasleese

app/components/app_session_buttons_component.rb

@@ -4,7 +4,7 @@ class AppSessionButtonsComponent < ViewComponent::Base
   erb_template <<-ERB
     <div class="nhsuk-button-group">
       <% if policy(session).edit? %>
-        <%= govuk_button_link_to edit_button_text, edit_session_path(session), secondary: true %>
+        <%= govuk_button_link_to "Edit session", edit_session_path(session), secondary: true %>
 
         <%= link_to "Record offline", session_path(session, format: :xlsx) %>
 
@@ -26,8 +26,4 @@ def initialize(session)
   attr_reader :session
 
   delegate :policy, :govuk_button_link_to, to: :helpers
-
-  def edit_button_text
-    session.dates.empty? ? "Schedule sessions" : "Edit session"
-  end
 end

app/components/app_session_dates_component.rb

@@ -9,6 +9,7 @@ class AppSessionDetailsComponent < ViewComponent::Base
             session,
             patient_count: session.patients.count,
             show_consent_forms: true,
+            show_dates: true,
             show_location: true,
             show_status: true,
           ) %>

app/components/app_session_dates_table_component.html.erb

@@ -5,7 +5,7 @@ class AppSessionOverviewComponent < ViewComponent::Base
     <%= render AppSessionStatsComponent.new(session) %>
 
     <section>
-      <%= render AppSessionDatesComponent.new(session) %>
+      <%= render AppSessionVaccinationsComponent.new(session) %>
     </section>
 
     <section>
@@ -17,9 +17,7 @@ class AppSessionOverviewComponent < ViewComponent::Base
     </section>
 
     <section>
-      <% if Flipper.enabled?(:schools_and_sessions) %>
-        <%= render AppSessionButtonsComponent.new(session) %>
-      <% end %>
+      <%= render AppSessionButtonsComponent.new(session) %>
     </section>
   ERB
 

app/components/app_session_details_component.rb

@@ -1,34 +1,62 @@
 # frozen_string_literal: true
 
-class AppSessionDatesTableComponent < ViewComponent::Base
+class AppSessionVaccinationsComponent < ViewComponent::Base
+  erb_template <<-ERB
+    <%= render AppCardComponent.new do |card| %>
+      <% card.with_heading(level: 3) { "Vaccinations given in this session" } %>
+
+      <%= govuk_table(html_attributes: { class: "nhsuk-table-responsive" }) do |table| %>
+        <% table.with_head do |head| %>
+          <% head.with_row do |row| %>
+            <% row.with_cell(text: "Session date") %>
+            <% session_column_names.each do |column| %>
+              <% row.with_cell(text: column, numeric: true) %>
+            <% end %>
+          <% end %>
+        <% end %>
+
+        <% table.with_body do |body| %>
+          <% rows.each_with_index do |row_data| %>
+            <% body.with_row do |row| %>
+              <% row.with_cell(text: row_data[:label]) %>
+              <% row_data[:tallies].each do |tally| %>
+                <% row.with_cell(text: tally.to_s, numeric: true) %>
+              <% end %>
+            <% end %>
+          <% end %>
+        <% end %>
+      <% end %>
+    <% end %>
+  ERB
+
   def initialize(session)
     @session = session
   end
 
-  def session_column_names
-    @session_column_names ||= build_column_names
-  end
-
-  def rows = session_dates_rows + [total_row]
+  def render? = session.started?
 
   private
 
   attr_reader :session
 
-  delegate :govuk_table, to: :helpers
   delegate :programmes, to: :session
 
-  def build_column_names
-    programmes.flat_map do |programme|
-      if programme.has_multiple_vaccine_methods?
-        base_name = programme.name_in_sentence.titlecase
-        ["#{base_name} (nasal spray)", "#{base_name} (injection)"]
-      else
-        programme.name_in_sentence
+  delegate :govuk_table, to: :helpers
+
+  def session_column_names
+    @session_column_names ||=
+      programmes.flat_map do |programme|
+        if programme.has_multiple_vaccine_methods?
+          base_name = programme.name_in_sentence.titlecase
+          ["#{base_name} (nasal spray)", "#{base_name} (injection)"]
+        else
+          programme.name_in_sentence
+        end
       end
-    end
   end
 
+  def rows = session_dates_rows + [total_row]
+
   def session_dates_rows
     @session_dates_rows ||=
       session.dates.map do |date|

app/components/app_session_overview_component.rb

@@ -363,19 +363,5 @@
         end
       end
     end
-
-    context "when session has not started" do
-      let(:dates) { [Date.new(2025, 1, 20)] }
-
-      it "renders the scheduled dates list instead of the table" do
-        expect(rendered).to have_content("Monday, 20 January 2025")
-        expect(rendered).not_to have_content("Vaccinations given")
-        expect(rendered).not_to have_content("Session date")
-      end
-
-      it "displays the consent period information" do
-        expect(rendered).to have_content("Consent period")
-      end
-    end
   end
 end

…nts/app_session_dates_table_component.rb …ts/app_session_vaccinations_component.rbapp/components/app_session_dates_table_component.rb renamed to app/components/app_session_vaccinations_component.rb app/components/app_session_dates_table_component.rb renamed to app/components/app_session_vaccinations_component.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-describe AppSessionDatesTableComponent do
+describe AppSessionVaccinationsComponent do
   subject(:rendered) { render_inline(component) }
 
   let(:hpv_programme) { Programme.hpv }

spec/components/app_session_overview_component_spec.rb

@@ -149,7 +149,7 @@ def when_i_start_creating_a_new_session_by_choosing_school_and_time
     choose "Unscheduled"
     click_on "Update results"
     click_on "Pilot School"
-    click_on "Schedule sessions"
+    click_on "Edit session"
 
     click_on "Add session dates"
     expect(page).to have_content("When will sessions be held?")

…pp_session_dates_table_component_spec.rb …p_session_vaccinations_component_spec.rbspec/components/app_session_dates_table_component_spec.rb renamed to spec/components/app_session_vaccinations_component_spec.rb spec/components/app_session_dates_table_component_spec.rb renamed to spec/components/app_session_vaccinations_component_spec.rb

@@ -53,7 +53,7 @@ def when_i_visit_the_session_overview_page
   end
 
   def and_i_click_on_schedule_sessions
-    click_on "Schedule sessions"
+    click_on "Edit session"
   end
 
   def and_i_click_on_add_session_dates