Show immunisation import warning callout

If there are duplicate records or not administered records, we want to
include this information in a warning callout at the top of the page.

The warning callout component is part of the NHS design system, but not
part of GOV.UK's so we don't have a component in govuk-components for
it. As this is the first time we're using it, I decided against creating
a component for it, but we might want to do add one in the future.

Author: thomasleese

app/controllers/immunisation_imports_controller.rb

@@ -44,23 +44,6 @@ def create
       return
     end
 
-    flash[
-      :success
-    ] = "#{@immunisation_import.new_record_count} vaccinations uploaded"
-
-    # TODO: Move to "Check and confirm" page
-    if (duplicate_count = @immunisation_import.exact_duplicate_record_count) > 1
-      flash[
-        :info
-      ] = "#{duplicate_count} previously uploaded records were omitted"
-    end
-
-    if (ignored_count = @immunisation_import.not_administered_record_count) > 1
-      flash[
-        :info
-      ] = "#{ignored_count} records for children who were not vaccinated were omitted"
-    end
-
     redirect_to edit_campaign_immunisation_import_path(
                   @campaign,
                   @immunisation_import

app/views/immunisation_imports/edit.html.erb

@@ -10,6 +10,25 @@
 <span class="nhsuk-caption-l"><%= @campaign.name %></span>
 <%= h1 title, page_title: "#{@campaign.name} – #{title}" %>
 
+<% if @immunisation_import.exact_duplicate_record_count > 1 || @immunisation_import.not_administered_record_count > 1 %>
+  <div class="nhsuk-warning-callout">
+    <h3 class="nhsuk-warning-callout__label">
+      <span class="nhsuk-u-visually-hidden">Important: </span>
+      Uploaded records will differ from the file
+    </h3>
+
+    <ul class="nhsuk-list nhsuk-list--bullet">
+      <% if @immunisation_import.exact_duplicate_record_count > 1 %>
+        <li><%= @immunisation_import.exact_duplicate_record_count %> previously uploaded records were omitted</li>
+      <% end %>
+
+      <% if @immunisation_import.not_administered_record_count > 1 %>
+        <li><%= @immunisation_import.not_administered_record_count %> records for children who were not vaccinated were omitted</li>
+      <% end %>
+    </ul>
+  </div>
+<% end %>
+
 <%= govuk_button_to "Upload records", campaign_immunisation_import_path(
       @campaign,
       @immunisation_import

spec/features/immunisation_imports_upload_spec.rb

@@ -22,7 +22,7 @@
     and_i_go_back_to_the_upload_page
 
     when_i_upload_a_valid_file
-    then_i_should_see_the_success_banner
+    then_i_should_see_the_success_heading
     and_i_should_see_the_vaccination_records
 
     when_i_click_on_upload_records
@@ -114,8 +114,8 @@ def when_i_upload_a_valid_file
     click_on "Continue"
   end
 
-  def then_i_should_see_the_success_banner
-    expect(page).to have_content("7 vaccinations uploaded")
+  def then_i_should_see_the_success_heading
+    expect(page).to have_content("7 new vaccination records")
   end
 
   def and_i_should_see_the_vaccination_records

spec/models/immunisation_import_spec.rb

@@ -191,15 +191,11 @@
       end
 
       it "stores statistics on the import" do
-        expect { process! }.to change(
-          immunisation_import,
-          :exact_duplicate_record_count
-        ).to(0).and change(immunisation_import, :new_record_count).to(
-                7
-              ).and change(
-                      immunisation_import,
-                      :not_administered_record_count
-                    ).to(0)
+        # stree-ignore
+        expect { process! }
+          .to change(immunisation_import, :exact_duplicate_record_count).to(0)
+          .and change(immunisation_import, :new_record_count).to(7)
+          .and change(immunisation_import, :not_administered_record_count).to(0)
       end
 
       it "ignores and counts duplicate records" do