diff --git a/app/controllers/concerns/idv/phone_otp_rate_limitable.rb b/app/controllers/concerns/idv/phone_otp_rate_limitable.rb index 598d0f74ad8..791ea9b4580 100644 --- a/app/controllers/concerns/idv/phone_otp_rate_limitable.rb +++ b/app/controllers/concerns/idv/phone_otp_rate_limitable.rb @@ -3,7 +3,6 @@ module PhoneOtpRateLimitable extend ActiveSupport::Concern included do - before_action :confirm_two_factor_authenticated before_action :handle_locked_out_user end diff --git a/app/controllers/concerns/idv/phone_otp_sendable.rb b/app/controllers/concerns/idv/phone_otp_sendable.rb index 8a657ed53d1..0d6c5c77b4f 100644 --- a/app/controllers/concerns/idv/phone_otp_sendable.rb +++ b/app/controllers/concerns/idv/phone_otp_sendable.rb @@ -3,7 +3,6 @@ module PhoneOtpSendable extend ActiveSupport::Concern included do - before_action :confirm_two_factor_authenticated before_action :handle_locked_out_user end diff --git a/app/controllers/idv/otp_verification_controller.rb b/app/controllers/idv/otp_verification_controller.rb index 57f80b68968..9f690b6bb62 100644 --- a/app/controllers/idv/otp_verification_controller.rb +++ b/app/controllers/idv/otp_verification_controller.rb @@ -4,7 +4,7 @@ class OtpVerificationController < ApplicationController include StepIndicatorConcern include PhoneOtpRateLimitable - # confirm_two_factor_authenticated before action is in PhoneOtpRateLimitable + before_action :confirm_two_factor_authenticated before_action :confirm_step_needed before_action :confirm_otp_sent before_action :set_code diff --git a/app/controllers/idv/resend_otp_controller.rb b/app/controllers/idv/resend_otp_controller.rb index 917f12d3a2b..febd4b4a1ef 100644 --- a/app/controllers/idv/resend_otp_controller.rb +++ b/app/controllers/idv/resend_otp_controller.rb @@ -4,7 +4,7 @@ class ResendOtpController < ApplicationController include PhoneOtpRateLimitable include PhoneOtpSendable - # confirm_two_factor_authenticated before action is in PhoneOtpRateLimitable + before_action :confirm_two_factor_authenticated before_action :confirm_user_phone_confirmation_needed before_action :confirm_user_phone_confirmation_session_started diff --git a/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb b/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb index b4b5d9874c9..e48caf2b801 100644 --- a/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb +++ b/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb @@ -48,6 +48,11 @@ expect(page).to have_current_path(root_url) visit idv_hybrid_mobile_document_capture_url + # Confirm that jumping to Phone page does not cause errors + visit idv_phone_url + expect(page).to have_current_path(root_url) + visit idv_hybrid_mobile_document_capture_url + # Confirm that jumping to Welcome page does not cause errors # This was added for the GettingStarted A/B Test visit idv_welcome_url