WIP

Author: Hassanmir92

app/controllers/teacher_interface/application_forms_controller.rb

@@ -11,6 +11,8 @@ class ApplicationFormsController < BaseController
     before_action :load_application_form, except: %i[new create]
     before_action :update_application_and_redirect_if_passport_expired,
                   only: %i[edit update]
+    before_action :update_application_and_redirect_if_any_referee_has_public_email_domain,
+                  only: %i[edit update]
 
     define_history_origin :show
     define_history_reset :show
@@ -120,5 +122,20 @@ def update_application_and_redirect_if_passport_expired
         redirect_to teacher_interface_application_form_path
       end
     end
+
+    def update_application_and_redirect_if_any_referee_has_public_email_domain
+      if requires_private_email_for_referee? &&
+           application_form.work_histories.any?(
+             &:invalid_email_domain_for_contact?
+           )
+        ApplicationFormSectionStatusUpdater.call(application_form:)
+
+        redirect_to teacher_interface_application_form_path
+      end
+    end
+
+    def requires_private_email_for_referee?
+      FeatureFlags::FeatureFlag.active?(:email_domains_for_referees)
+    end
   end
 end

app/view_objects/teacher_interface/application_form_view_object.rb

@@ -234,6 +234,8 @@ def show_work_history_under_induction_banner?
            :requires_preliminary_check,
            :includes_prioritisation_features,
            :work_history_status_completed?,
+           :other_england_work_history_status_completed?,
+           :started_with_private_email_for_referee?,
            to: :application_form
 
   delegate :consent_requests,

app/views/teacher_interface/application_forms/show/_draft.html.erb

@@ -30,6 +30,14 @@
   </div>
 <% end %>
 
+<% if FeatureFlags::FeatureFlag.active?(:email_domains_for_referees) && !view_object.started_with_private_email_for_referee? && (!view_object.work_history_status_completed? || !view_object.other_england_work_history_status_completed?) %>
+  <%= govuk_notification_banner(title_text: "Important") do |notification_banner| %>
+    <% notification_banner.with_heading(text: "Requirements for your references have changed") %>
+
+    <p class="govuk-body">Testing</p>
+  <% end %>
+<% end %>
+
 <% if view_object.show_work_history_under_submission_banner? || view_object.show_work_history_under_induction_banner? %>
   <%= govuk_notification_banner(title_text: "Important") do %>
     <p>You have added at least one teaching role that started before you were qualified as a teacher.</p>

spec/system/teacher_interface/submitting_spec.rb

@@ -110,7 +110,7 @@
     end
   end
 
-  context "when the application required passport as a form of identity proof" do
+  context "when the application requires passport as a form of identity proof" do
     let(:application_form) do
       create(
         :application_form,
@@ -172,6 +172,29 @@
     end
   end
 
+  context "when the application requires private email domain for all referees" do
+    before do
+      FeatureFlags::FeatureFlag.activate(:email_domains_for_referees)
+      application_form.work_histories.last.update!(
+        contact_email: "test@gmail.com",
+        contact_email_domain: "gmail.com",
+      )
+    end
+
+    after { FeatureFlags::FeatureFlag.deactivate(:email_domains_for_referees) }
+
+    it "redirects back to application task list with work history spoke to update needed" do
+      when_i_visit_the(:teacher_application_page)
+      then_i_see_the(:teacher_application_page)
+      and_i_see_the_completed_work_history_task
+
+      when_i_click_check_your_answers
+      then_i_see_the(:teacher_application_page)
+      and_i_see_the_update_needed_work_history_task
+      and_i_see_content_that_referee_requirements_have_changed
+    end
+  end
+
   context "when application has other England work history" do
     let(:application_form) do
       create(
@@ -321,6 +344,18 @@ def and_i_see_the_completed_passport_document_task
     ).to eq("Completed")
   end
 
+  def and_i_see_the_completed_work_history_task
+    expect(
+      teacher_application_page.work_history_task_item.status_tag.text,
+    ).to eq("Completed")
+  end
+
+  def and_i_see_the_update_needed_work_history_task
+    expect(
+      teacher_application_page.work_history_task_item.status_tag.text,
+    ).to eq("Update needed")
+  end
+
   def and_i_see_the_in_progress_passport_document_task
     expect(
       teacher_application_page.passport_document_task_item.status_tag.text,
@@ -333,6 +368,12 @@ def and_i_see_content_that_my_passport_has_expired
     )
   end
 
+  def and_i_see_content_that_referee_requirements_have_changed
+    expect(teacher_application_page).to have_content(
+      "Requirements for your references have changed",
+    )
+  end
+
   def teacher
     @teacher ||= create(:teacher, email: "teacher@example.com")
   end