Merge pull request #1954 from alphagov/updte-conditions-for-exit-pages

Update content for routing when exit_pages on

Author: thomasiles

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

@@ -1,6 +1,16 @@
 <% branch_routing_enabled = FeatureService.new(group: form.group).enabled?(:branch_routing) %>
-<% body_text = branch_routing_enabled ? t("routing_page.branch_routing.body_html") : t("routing_page.body_html") %>
-<% hint_text = branch_routing_enabled ? t("routing_page.branch_routing.legend_hint_text") : t("routing_page.legend_hint_text") %>
+<% exit_pages_enabled = FeatureService.new(group: form.group).enabled?(:exit_pages) %>
+
+<% if exit_pages_enabled %>
+  <% body_text = t("routing_page.exit_pages.body_html") %>
+  <% hint_text = t("routing_page.exit_pages.legend_hint_text") %>
+<% elsif branch_routing_enabled %>
+  <% body_text = t("routing_page.branch_routing.body_html") %>
+  <% hint_text = t("routing_page.branch_routing.legend_hint_text") %>
+<% else %>
+  <% body_text = t("routing_page.body_html") %>
+  <% hint_text = t("routing_page.legend_hint_text") %>
+<% end %>
 
 <%= form_with(model: routing_page_input, url: set_routing_page_path(form.id), method: 'POST') do |f| %>
   <% if routing_page_input&.errors.any? %>

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

@@ -3,7 +3,15 @@
 <div class="govuk-grid-row">
   <div class="govuk-grid-column-two-thirds">
     <% branch_routing_enabled = FeatureService.new(group: form.group).enabled?(:branch_routing) %>
-    <% prefix = branch_routing_enabled ? 'branch_routing.' : '' %>
+    <% exit_pages_enabled = FeatureService.new(group: form.group).enabled?(:exit_pages) %>
+
+    <% if exit_pages_enabled %>
+      <% prefix = 'exit_pages.' %>
+    <% elsif branch_routing_enabled %>
+      <% prefix = 'branch_routing.' %>
+    <% else %>
+      <% prefix = '' %>
+    <% end %>
 
     <% if policy(form).can_add_page_routing_conditions? %>
       <%= render partial: "pages/conditions/routing_options", locals: {form:, routing_page_input:} %>

config/i18n-tasks.yml

@@ -132,7 +132,7 @@ ignore_unused:
 - 'errors.*'
 - 'date.formats.*'
 - 'helpers.*'
-- 'routing_page.branch_routing.*'
+- 'routing_page.{branch_routing,exit_pages}.*'
 
 ## Exclude these keys from the `i18n-tasks eq-base' report:
 # ignore_eq_base:

config/locales/en.yml

@@ -1482,6 +1482,32 @@ en:
 
         <p>It’s usually easiest to create all your form’s questions first, then add the routes you need.</p>
     dropdown_default_text: Select a question to start your route from
+    exit_pages:
+      body_html: |
+        <p> You can add a route to a question so if someone selects one specific answer, they’ll be skipped to: </p>
+
+        <ul class="govuk-list govuk-list--bullet">
+          <li> a later question </li>
+          <li> the end of the form </li>
+          <li> an ‘exit page’ to remove them from the form - for example, because they’re not eligible </li>
+        </ul>
+
+        <p> People who select any other answer will continue to the next question and through the rest of the form. </p>
+      legend_hint_text: A route can only start from a question where people select one option from a list. Go back to your questions if you need to edit an existing route.
+      no_remaining_routes_html: |
+        <h2 class="govuk-heading-m">You have no more questions to add a route from</h2>
+        <p>A route can only start from a question where people select one option from a list. If you need to edit an existing route, go back to your questions.</p>
+      routing_requirements_not_met_html: |
+        <h2 class="govuk-heading-m">You cannot add a route yet</h2>
+
+        <p>A route can only start from a question that:</p>
+
+        <ul class="govuk-list govuk-list--bullet">
+          <li>asks people to select only one option from a list</li>
+          <li>has at least one question after it</li>
+        </ul>
+
+        <p>It’s usually easiest to create all your form’s questions first, then add the routes you need.</p>
     legend_hint_text: A route can only start from a question where people select one item from a list. You can only add one route from each question.
     legend_text: Which question do you want to add a route from?
     no_remaining_routes_html: |

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

@@ -34,6 +34,12 @@
     end
   end
 
+  context "when exit pages are enabled", :feature_exit_pages do
+    it "contains content for exit pages" do
+      expect(rendered).to have_text "an ‘exit page’ to remove them from the form - for example, because they’re not eligible ", normalize_ws: true
+    end
+  end
+
   context "when branch routing is not enabled", feature_branch_routing: false do
     it "does not contain content explaining branch routing" do
       expect(rendered).not_to have_text "You can add a route from a question where people can select only one answer from a list."
@@ -107,5 +113,21 @@
         expect(rendered).to have_text(guidance, normalize_ws: true)
       end
     end
+
+    context "when exit_pages is enabled", :feature_exit_pages do
+      it "explains to the user what is required for them to be able to add a new routes" do
+        guidance = Capybara.string(I18n.t("routing_page.exit_pages.routing_requirements_not_met_html")).text(normalize_ws: true)
+        expect(rendered).to have_text(guidance, normalize_ws: true)
+      end
+
+      context "when all qualifying questions have a route" do
+        let(:all_routes_created) { true }
+
+        it "explains to the user that they have created all available routes" do
+          guidance = Capybara.string(I18n.t("routing_page.exit_pages.no_remaining_routes_html")).text(normalize_ws: true)
+          expect(rendered).to have_text(guidance, normalize_ws: true)
+        end
+      end
+    end
   end
 end