Hide Records column on imports table if no patients imported

The records count being 0 for imports in review can be misleading. Now that we have separate tabs for imported records and uplaoded files, it is better to hide the Records column entirely.

MAV-2720

Author: murugapl

app/components/app_imports_table_component.html.erb

@@ -11,7 +11,7 @@
         <% row.with_cell(text: "Imported on") %>
         <% row.with_cell(text: "Type") %>
         <% row.with_cell(text: "Status") %>
-        <% row.with_cell(text: "Records", numeric: true) %>
+        <% row.with_cell(text: "Records", numeric: true) unless @uploaded_files %>
       <% end %>
     <% end %>
 
@@ -41,12 +41,10 @@
             <%= render AppImportStatusComponent.new(import: import,
                                                     break_tag: true) %>
           <% end %>
-          <% row.with_cell(numeric: true) do %>
-            <span class="nhsuk-table-responsive__heading">Records</span>
-            <% if import.pending_import? %>
-              …
-            <% else %>
-              <%= import.record_count %>
+          <% if import.processed? || import.partially_processed? %>
+            <% row.with_cell(numeric: true) do %>
+              <span class="nhsuk-table-responsive__heading">Records</span>
+                <%= import.patients.count %>
             <% end %>
           <% end %>
         <% end %>

spec/components/app_imports_table_component_spec.rb

@@ -3,12 +3,13 @@
 describe AppImportsTableComponent do
   subject(:rendered) { render_inline(component) }
 
-  let(:component) { described_class.new(team:, uploaded_files: false) }
+  let(:component) { described_class.new(team:, uploaded_files:) }
 
   let(:programmes) { [Programme.sample] }
   let(:team) { create(:team, programmes:) }
   let(:school) { create(:school, team:, name: "Test School") }
   let(:session) { create(:session, programmes:, location: school) }
+  let(:uploaded_files) { false }
 
   before do
     cohort_imports =
@@ -94,4 +95,29 @@
     expect(rendered).to have_css(".nhsuk-table__cell", text: "1")
     expect(rendered).to have_content("Test School")
   end
+
+  context "when uploaded_files is false" do
+    let(:uploaded_files) { true }
+
+    before do
+      CohortImport.destroy_all
+      ImmunisationImport.destroy_all
+      ClassImport.destroy_all
+      create(:cohort_import, status: "pending_import", team:)
+    end
+
+    it "does not show the Records column header" do
+      rendered =
+        render_inline(described_class.new(team:, uploaded_files: false))
+      expect(rendered).not_to have_css(".nhsuk-table__header", text: "Records")
+    end
+
+    it "does not show record counts" do
+      expect(rendered).to have_css(
+        ".nhsuk-table__body .nhsuk-table__row",
+        count: 1
+      )
+      expect(rendered).not_to have_css(".nhsuk-table__cell", text: /^\d+$/)
+    end
+  end
 end