Merge pull request #3241 from DFE-Digital/ey-one-login

[LUPEYALPHA-981] [LUPEYALPHA-994] Add One Login to EY practitioner journey
DFE-Digital · Oct 1, 2024 · aa1116e · aa1116e
2 parents 7cdcff5 + 162d8a4
commit aa1116e
Show file tree

Hide file tree

Showing 7 changed files with 21 additions and 9 deletions.
diff --git a/app/forms/journeys/early_years_payment/practitioner/find_reference_form.rb b/app/forms/journeys/early_years_payment/practitioner/find_reference_form.rb
@@ -11,7 +11,10 @@ class FindReferenceForm < Form
         def save
           return false if invalid?
 
-          journey_session.answers.assign_attributes(reference_number:)
+          journey_session.answers.assign_attributes(
+            reference_number:,
+            start_email: email
+          )
           journey_session.save!
         end
 

diff --git a/app/models/journeys/early_years_payment/practitioner/session_answers.rb b/app/models/journeys/early_years_payment/practitioner/session_answers.rb
@@ -3,6 +3,7 @@ module EarlyYearsPayment
     module Practitioner
       class SessionAnswers < Journeys::SessionAnswers
         attribute :reference_number, :string
+        attribute :start_email, :string
 
         def policy
           Policies::EarlyYearsPayments

diff --git a/app/models/journeys/early_years_payment/practitioner/slug_sequence.rb b/app/models/journeys/early_years_payment/practitioner/slug_sequence.rb
@@ -4,7 +4,7 @@ module Practitioner
       class SlugSequence
         SLUGS = %w[
           find-reference
-          one-login-sign-in
+          sign-in
           how-we-use-your-information
           personal-details
           enter-home-address

diff --git a/app/views/early_years_payment/practitioner/claims/find_reference.html.erb b/app/views/early_years_payment/practitioner/claims/find_reference.html.erb
@@ -5,7 +5,7 @@
     <%= form_with model: @form, url: claim_path(current_journey_routing_name, request.query_parameters), method: :patch, builder: GOVUKDesignSystemFormBuilder::FormBuilder, html: { novalidate: false } do |f| %>
       <%= f.govuk_error_summary %>
 
-      <%= f.hidden_field :email, value: params[:email] %>
+      <%= f.hidden_field :email, value: (params[:email] || journey_session.answers.attributes["start_email"]) %>
 
       <h1 class="govuk-heading-l">
         <%= @form.t(:question) %>

diff --git a/app/views/early_years_payment/practitioner/claims/one_login_sign_in.html.erb b/app/views/early_years_payment/practitioner/claims/one_login_sign_in.html.erb
diff --git a/spec/features/early_years_payment/practitioner/find_reference_spec.rb b/spec/features/early_years_payment/practitioner/find_reference_spec.rb
@@ -33,6 +33,12 @@
     fill_in "Claim reference number", with: claim.reference
     click_button "Submit"
 
-    expect(page).to have_content "login page goes here"
+    expect(page).to have_content "Sign in with GOV.UK One Login"
+    click_link "Back"
+
+    expect(page).to have_content "Track your application"
+    fill_in "Claim reference number", with: claim.reference
+    click_button "Submit"
+    expect(page).to have_content "Sign in with GOV.UK One Login"
   end
 end
diff --git a/spec/features/early_years_payment/practitioner/happy_path_spec.rb b/spec/features/early_years_payment/practitioner/happy_path_spec.rb
@@ -18,7 +18,13 @@
     fill_in "Claim reference number", with: claim.reference
     click_button "Submit"
 
-    expect(page).to have_content "one login page goes here"
+    expect(page).to have_content "Sign in with GOV.UK One Login"
+    click_on "Continue"
+
+    expect(page).to have_content "You have successfully signed in to GOV.UK One Login"
+    click_on "Continue"
+
+    expect(page).to have_content "You have successfully proved your identity with GOV.UK One Login"
     click_on "Continue"
 
     expect(page.title).to have_text("How we’ll use the information you provide")