LG-11383 Update Verify Info Screen

app/views/idv/in_person/verify_info/show.html.erb

@@ -160,7 +160,7 @@ locals:
                 poll_interval_ms: IdentityConfig.store.poll_rate_for_verify_in_seconds * 1000,
               },
             },
-          ).with_content(t('forms.buttons.continue')) %>
+          ).with_content(t('forms.buttons.submit.default')) %>
   </div>
 </div>
 

app/views/idv/verify_info/show.html.erb

@@ -35,6 +35,11 @@ locals:
 <% title t('titles.idv.verify_info') %>
 
 <%= render PageHeadingComponent.new.with_content(t('headings.verify')) %>
+
+<p>
+  <%= t('idv.messages.verify_info') %>
+</p>
+
 <div class='margin-top-4 margin-bottom-2'>
   <div class="grid-row grid-gap grid-gap-2 padding-bottom-1">
     <dl class="grid-col-fill margin-y-0">
@@ -131,7 +136,7 @@ locals:
                 poll_interval_ms: IdentityConfig.store.poll_rate_for_verify_in_seconds * 1000,
               },
             },
-          ).with_content(t('forms.buttons.continue')) %>
+          ).with_content(t('forms.buttons.submit.default')) %>
   </div>
 </div>
 

config/locales/idv/en.yml

@@ -286,6 +286,8 @@ en:
           it.
         no_pii: TEST SITE - Do not use real personal information (demo purposes only) -
           TEST SITE
+      verify_info: We read your information from your ID. Review it and make any
+        updates before submitting for verification.
       verifying: Verifying…
     titles:
       activated: Your identity has already been verified

config/locales/idv/es.yml

@@ -304,6 +304,8 @@ es:
           ellos.
         no_pii: SITIO DE PRUEBA - No utilice información personal real (sólo para
           propósitos de demostración) - SITIO DE PRUEBA
+      verify_info: Leímos sus datos en su identificación. Revíselos y modifíquelos si
+        es necesario antes de enviarlos para la verificación.
       verifying: Verificando…
     titles:
       activated: Ya se verificó tu identidad.

config/locales/idv/fr.yml

@@ -314,6 +314,9 @@ fr:
           y accéder.
         no_pii: SITE DE TEST - N’utilisez pas de véritables données personnelles (il
           s’agit d’une démonstration seulement) - SITE DE TEST
+      verify_info: Nous lisons vos informations à partir de votre pièce d’identité.
+        Passez-les en revue et mettez-les à jour avant de les soumettre pour
+        vérification.
       verifying: Vérification…
     titles:
       activated: Votre identité a déjà été vérifiée

spec/features/idv/doc_auth/redo_document_capture_spec.rb

@@ -77,7 +77,7 @@
 
       visit idv_verify_info_url
 
-      click_idv_continue
+      complete_verify_step
 
       expect(page).to have_current_path(idv_phone_path)
 
@@ -107,7 +107,7 @@
       let(:use_bad_ssn) { true }
 
       it 'shows a troubleshooting option to allow the user to cancel and return to SP' do
-        click_idv_continue
+        complete_verify_step
         expect(page).to have_link(
           t('links.cancel'),
           href: idv_cancel_path(step: :invalid_session),

spec/features/idv/doc_auth/verify_info_step_spec.rb

@@ -48,7 +48,7 @@
       expect(page).to have_content('12345')
       expect(page).to have_content('Apt 3E')
 
-      click_idv_continue
+      complete_verify_step
 
       expect(fake_analytics).to have_logged_event(
         'IdV: doc auth verify proofing results',
@@ -90,7 +90,7 @@
         **fake_pii_details,
         ssn: DocAuthHelper::GOOD_SSN,
       )
-      click_idv_continue
+      complete_verify_step
 
       expect(fake_analytics).to have_logged_event(
         'IdV: doc auth verify proofing results',
@@ -108,7 +108,7 @@
     )
     fill_out_ssn_form_with_ssn_that_fails_resolution
     click_idv_continue
-    click_idv_continue
+    complete_verify_step
 
     expect(page).to have_current_path(idv_session_errors_warning_path)
     expect_step_indicator_current_step(t('step_indicator.flows.idv.verify_info'))
@@ -127,7 +127,7 @@
     fill_out_ssn_form_with_ssn_that_raises_exception
 
     click_idv_continue
-    click_idv_continue
+    complete_verify_step
 
     expect(fake_analytics).to have_logged_event(
       'IdV: doc auth exception visited',
@@ -157,13 +157,13 @@
         with({ limiter_context: 'single-session' })
 
       (max_resolution_attempts - 2).times do
-        click_idv_continue
+        complete_verify_step
         expect(page).to have_current_path(idv_session_errors_warning_path)
         click_try_again
       end
 
       # Check that last attempt shows correct warning text
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_current_path(idv_session_errors_warning_path)
       expect(page).to have_content(
         strip_tags(
@@ -172,7 +172,7 @@
       )
       click_try_again
 
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_current_path(idv_session_errors_failure_path)
       expect(page).not_to have_css('.step-indicator__step--current', text: text, wait: 5)
       expect(fake_analytics).to have_logged_event(
@@ -187,7 +187,7 @@
       travel_to(IdentityConfig.store.idv_attempt_window_in_hours.hours.from_now + 1) do
         sign_in_and_2fa_user(user)
         complete_doc_auth_steps_before_verify_step
-        click_idv_continue
+        complete_verify_step
 
         expect(page).to have_current_path(idv_phone_path)
         expect(RateLimiter.new(user: user, rate_limit_type: :idv_resolution)).to be_limited
@@ -215,7 +215,7 @@
       fill_out_ssn_form_with_ssn_that_fails_resolution
       click_idv_continue
       (max_ssn_attempts - 1).times do
-        click_idv_continue
+        complete_verify_step
         expect(page).to have_current_path(idv_session_errors_warning_path)
         click_try_again
       end
@@ -224,7 +224,7 @@
     it 'rate limits ssn and continues when it expires' do
       expect(fake_attempts_tracker).to receive(:idv_verification_rate_limited).at_least(1).times.
         with({ limiter_context: 'multi-session' })
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_current_path(idv_session_errors_ssn_failure_path)
       expect(fake_analytics).to have_logged_event(
         'Rate Limit Reached',
@@ -238,7 +238,7 @@
       travel_to(IdentityConfig.store.idv_attempt_window_in_hours.hours.from_now + 1) do
         sign_in_and_2fa_user(user)
         complete_doc_auth_steps_before_verify_step
-        click_idv_continue
+        complete_verify_step
 
         expect(page).to have_current_path(idv_phone_path)
       end
@@ -256,7 +256,7 @@
 
       fill_in t('idv.form.ssn_label'), with: '900456789'
       click_button t('forms.buttons.submit.update')
-      click_idv_continue
+      complete_verify_step
 
       expect(page).to have_current_path(idv_phone_path)
       expect(fake_analytics).not_to have_logged_event(
@@ -297,7 +297,7 @@
 
         sign_in_and_2fa_user(user)
         complete_doc_auth_steps_before_verify_step
-        click_idv_continue
+        complete_verify_step
 
         expect(DocAuthLog.find_by(user_id: user.id).aamva).not_to be_nil
       end
@@ -321,7 +321,7 @@
 
         sign_in_and_2fa_user(user)
         complete_doc_auth_steps_before_verify_step
-        click_idv_continue
+        complete_verify_step
 
         expect(DocAuthLog.find_by(user_id: user.id).aamva).to be_nil
       end
@@ -344,7 +344,7 @@
         visit_idp_from_sp_with_ial1(:oidc)
         sign_in_and_2fa_user(user)
         complete_doc_auth_steps_before_verify_step
-        click_idv_continue
+        complete_verify_step
 
         expect(DocAuthLog.find_by(user_id: user.id).aamva).to be_nil
       end
@@ -359,12 +359,12 @@
       allow(DocumentCaptureSession).to receive(:find_by).
         and_return(nil)
 
-      click_idv_continue
+      complete_verify_step
       expect(fake_analytics).to have_logged_event('IdV: proofing resolution result missing')
       expect(page).to have_content(t('idv.failure.timeout'))
       expect(page).to have_current_path(idv_verify_info_path)
       allow(DocumentCaptureSession).to receive(:find_by).and_call_original
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_current_path(idv_phone_path)
     end
 
@@ -381,7 +381,7 @@
       allow(DocumentCaptureSession).to receive(:find_by).
         and_return(nil)
 
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_content(t('idv.failure.timeout'))
       expect(page).to have_current_path(idv_verify_info_path)
       allow(DocumentCaptureSession).to receive(:find_by).and_call_original
@@ -396,11 +396,11 @@
       allow(DocumentCaptureSession).to receive(:find_by).
         and_return(nil)
 
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_content(t('idv.failure.timeout'))
       expect(page).to have_current_path(idv_verify_info_path)
       allow(DocumentCaptureSession).to receive(:find_by).and_call_original
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_current_path(idv_phone_path)
     end
   end
@@ -418,7 +418,7 @@
     end
 
     it 'redirects to the gpo page when continuing from verify info page' do
-      click_idv_continue
+      complete_verify_step
       expect(page).to have_current_path(idv_request_letter_path)
 
       click_on 'Cancel'

spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb

@@ -79,7 +79,7 @@
       click_idv_continue
 
       expect(page).to have_content(t('headings.verify'))
-      click_idv_continue
+      complete_verify_step
 
       prefilled_phone = page.find(id: 'idv_phone_form_phone').value
 
@@ -252,7 +252,7 @@
         expect(page).to have_text(Idp::Constants::MOCK_IDV_APPLICANT[:last_name])
         expect(page).to have_text(Idp::Constants::MOCK_IDV_APPLICANT[:address1])
 
-        click_idv_continue
+        complete_verify_step
       end
     end
   end
@@ -308,7 +308,7 @@
         expect(page).to have_text(Idp::Constants::MOCK_IDV_APPLICANT[:last_name])
         expect(page).to have_text(Idp::Constants::MOCK_IDV_APPLICANT[:address1])
 
-        click_idv_continue
+        complete_verify_step
       end
     end
   end
@@ -342,7 +342,7 @@
       click_idv_continue
 
       expect(page).to have_content(t('headings.verify'))
-      click_idv_continue
+      complete_verify_step
 
       prefilled_phone = page.find(id: 'idv_phone_form_phone').value
 

spec/features/idv/steps/in_person/verify_info_spec.rb

@@ -145,7 +145,7 @@
     complete_state_id_step(user)
     fill_out_ssn_form_with_ssn_that_fails_resolution
     click_idv_continue
-    click_idv_continue
+    complete_verify_step(user)
 
     expect(page).to have_current_path(idv_session_errors_warning_path(flow: 'in_person'))
     click_on t('idv.failure.button.warning')
@@ -161,7 +161,7 @@
     complete_location_step(user)
     complete_state_id_step(user)
     complete_ssn_step(user)
-    click_idv_continue
+    complete_verify_step(user)
 
     expect(page).to have_content(t('titles.idv.phone'))
     expect(fake_analytics).to have_logged_event(

spec/features/idv/threat_metrix_pending_spec.rb

@@ -30,7 +30,7 @@
     complete_doc_auth_steps_before_ssn_step
     select 'Reject', from: :mock_profiling_result
     complete_ssn_step
-    click_idv_continue
+    complete_verify_step
     complete_phone_step(user)
     complete_enter_password_step(user)
     acknowledge_and_confirm_personal_key
@@ -117,7 +117,7 @@
       complete_doc_auth_steps_before_ssn_step
       select 'No Result', from: :mock_profiling_result
       complete_ssn_step
-      click_idv_continue
+      complete_verify_step
 
       expect(page).to have_content(t('idv.failure.sessions.exception'))
       expect(page).to have_current_path(idv_session_errors_exception_path)

spec/support/features/doc_auth_helper.rb

@@ -134,7 +134,7 @@ def complete_doc_auth_steps_before_verify_step(expect_accessible: false)
   end
 
   def complete_verify_step
-    click_idv_continue
+    click_idv_submit_default
   end
 
   def complete_doc_auth_steps_before_address_step(expect_accessible: false)
@@ -315,7 +315,7 @@ def complete_all_idv_steps_with(threatmetrix:)
     complete_doc_auth_steps_before_ssn_step
     select threatmetrix, from: :mock_profiling_result
     complete_ssn_step
-    click_idv_continue
+    complete_verify_step
     complete_phone_step(user)
     complete_enter_password_step(user)
     acknowledge_and_confirm_personal_key

spec/support/features/idv_helper.rb

@@ -39,6 +39,10 @@ def click_idv_continue
     click_spinner_button_and_wait t('forms.buttons.continue')
   end
 
+  def click_idv_submit_default
+    click_spinner_button_and_wait t('forms.buttons.submit.default')
+  end
+
   def click_idv_update
     click_on t('forms.buttons.submit.update')
   end

spec/support/features/in_person_helper.rb

@@ -136,7 +136,7 @@ def complete_ssn_step(_user = nil)
   end
 
   def complete_verify_step(_user = nil)
-    click_idv_continue
+    click_idv_submit_default
   end
 
   def complete_all_in_person_proofing_steps(user = user_with_2fa, same_address_as_id: true)