Merge pull request #2042 from alphagov/348-add-a-banner-notification-warning-to-the-confirmation-page-when-deleting-a-route-that-goes-to-an-exit-page

Add notification banner when deleting route with exit page

Author: SamJamCul

app/views/pages/conditions/delete.html.erb

@@ -10,6 +10,10 @@
         <%= govuk_notification_banner(title_text: t("banner.default.title")) do |banner| %>
           <% banner.with_heading(text: t(".any_other_answer_warning")) %>
         <% end %>
+      <% elsif delete_condition_input.record.exit_page? %>
+        <%= govuk_notification_banner(title_text: t("banner.default.title")) do |banner| %>
+          <% banner.with_heading(text: t(".exit_page_warning")) %>
+        <% end %>
       <% end %>
 
       <h1 class="govuk-heading-l">
@@ -28,7 +32,7 @@
         end;
         summary_list.with_row do |row|
           row.with_key { t(".skip_to_key") }
-          row.with_value { delete_condition_input.goto_page_question_text }
+          row.with_value { delete_condition_input.record.exit_page? ? delete_condition_input.record.exit_page_heading : delete_condition_input.goto_page_question_text }
         end;
 
       end %>

config/locales/en.yml

@@ -1204,6 +1204,7 @@ en:
         answered_as_key: is answered as
         any_other_answer_warning: If you delete this route, the route for any other answer will also be deleted
         delete_condition_legend: Are you sure you want to delete this route?
+        exit_page_warning: If you delete this route, the exit page it goes to will also be deleted
         skip_to_key: skip the person to
       edit:
         back_link: Back to question %{question_number}’s routes

spec/views/pages/conditions/delete.html.erb_spec.rb

@@ -93,4 +93,16 @@
       end
     end
   end
+
+  context "when the condition has an exit page" do
+    let(:condition) { build :condition, :with_exit_page, id: 1, routing_page_id: pages.first.id, check_page_id: pages.first.id }
+
+    it "renders a warning about the exit page" do
+      expect(rendered).to have_css ".govuk-notification-banner", text: "If you delete this route, the exit page it goes to will also be deleted"
+    end
+
+    it "shows the exit page heading in the summary list" do
+      expect(rendered).to have_css(".govuk-summary-list__value", text: condition.exit_page_heading)
+    end
+  end
 end