Merge pull request #6480 from NHSDigital/add-pagination-summary-to-vaccination-record-imports

Include a pagination summary in vaccination records imports

Author: stevehook

app/components/app_patient_table_component.html.erb

@@ -1,6 +1,9 @@
 <%= render AppDetailsComponent.new(expander: true, sticky: true, open: @pagy.page > 1) do |c| %>
   <% c.with_summary do %>
     <%= pluralize(@pagy.count, "imported record") %>
+    <% if @pagy.count > @pagy.limit %>
+      (showing <b><%= @pagy.from %></b> to <b><%= @pagy.to %></b>)
+    <% end %>
   <% end %>
   <%= govuk_table(html_attributes: { class: "nhsuk-table-responsive", id: anchor }) do |table| %>
     <% table.with_head do |head| %>

app/components/app_vaccination_record_table_component.html.erb

@@ -2,6 +2,9 @@
 <%= render AppDetailsComponent.new(expander: true, sticky: true, open: @pagy.page > 1) do |c| %>
   <% c.with_summary do %>
     <%= pluralize(@pagy.count, "imported record") %>
+    <% if @pagy.count > @pagy.limit %>
+      (showing <b><%= @pagy.from %></b> to <b><%= @pagy.to %></b>)
+    <% end %>
   <% end %>
   <div class="nhsuk-details__text">
     <%= govuk_table(html_attributes: { class: "nhsuk-table-responsive" }) do |table| %>

spec/components/app_patient_table_component_spec.rb

@@ -26,7 +26,7 @@
         date_of_birth: Date.new(2000, 5, 28),
         address_postcode: "SW1B 1AA"
       )
-    ] + create_list(:patient, 8)
+    ] + create_list(:patient, count - 2)
   end
 
   let(:pagination_result) { pagy_array(all_patients, page: 1) }
@@ -100,6 +100,26 @@
     expect(rendered).not_to have_text("SW1B 1AA")
   end
 
+  it "does not render pagination summary text" do
+    expect(rendered).not_to have_content("showing 1 to 10")
+  end
+
+  context "with paginated results" do
+    let(:count) { 55 }
+
+    it "renders 50 rows of data" do
+      expect(rendered).to have_css(
+        ".nhsuk-table__body .nhsuk-table__row",
+        count: 50,
+        visible: :hidden
+      )
+    end
+
+    it "renders pagination summary text" do
+      expect(rendered).to have_content("showing 1 to 50")
+    end
+  end
+
   context "with a patient not in the cohort" do
     it "doesn't render a link" do
       expect(rendered).not_to have_link("SMITH, John")

spec/components/app_vaccination_record_table_component_spec.rb

@@ -29,8 +29,9 @@
           create(:session, programmes: [programme], date: Date.new(2020, 9, 1))
       )
     ] + create_list(:vaccination_record, 4, programme:) +
-      create_list(:vaccination_record, 5, :not_administered, programme:)
+      create_list(:vaccination_record, count - 5, :not_administered, programme:)
   end
+  let(:count) { 10 }
 
   let(:pagination_result) { pagy_array(all_vaccination_records, page: 1) }
   let(:vaccination_records) { pagination_result[1] }
@@ -71,7 +72,7 @@
   it "renders the rows" do
     expect(rendered).to have_css(
       ".nhsuk-table__body .nhsuk-table__row",
-      count: 10,
+      count: count,
       visible: :hidden
     )
     expect(rendered).to have_css(
@@ -97,6 +98,26 @@
     )
   end
 
+  it "does not render pagination summary text" do
+    expect(rendered).not_to have_content("showing 1 to 10")
+  end
+
+  context "with paginated results" do
+    let(:count) { 55 }
+
+    it "renders 50 rows of data" do
+      expect(rendered).to have_css(
+        ".nhsuk-table__body .nhsuk-table__row",
+        count: 50,
+        visible: :hidden
+      )
+    end
+
+    it "renders pagination summary text" do
+      expect(rendered).to have_content("showing 1 to 50")
+    end
+  end
+
   context "with a vaccination record not performed by the team" do
     before do
       vaccination_records.first.patient.patient_locations.destroy_all