diff --git a/app/helpers/opt_in_helper.rb b/app/helpers/opt_in_helper.rb index 5db0323aa5c..f33694dd0d5 100644 --- a/app/helpers/opt_in_helper.rb +++ b/app/helpers/opt_in_helper.rb @@ -4,8 +4,8 @@ module OptInHelper def opt_in_analytics_properties if IdentityConfig.store.in_person_proofing_opt_in_enabled { opted_in_to_in_person_proofing: idv_session.opted_in_to_in_person_proofing } - else - {} - end + else + { opted_in_to_in_person_proofing: nil } + end end end diff --git a/app/services/analytics_events.rb b/app/services/analytics_events.rb index 58ae3174437..714d55e1f0c 100644 --- a/app/services/analytics_events.rb +++ b/app/services/analytics_events.rb @@ -406,12 +406,16 @@ def create_new_device_alert_job_emails_sent(count:, **extra) end # @param [String] message the warning + # @param [Array] unknown_alerts Names of alerts not recognized by our code + # @param [Hash] response_info Response payload # Logged when there is a non-user-facing error in the doc auth process, such as an unrecognized # field from a vendor - def doc_auth_warning(message: nil, **extra) + def doc_auth_warning(message: nil, unknown_alerts: nil, response_info: nil, **extra) track_event( 'Doc Auth Warning', - message: message, + message:, + unknown_alerts:, + response_info:, **extra, ) end @@ -1098,11 +1102,13 @@ def idv_consent_checkbox_toggled(checked:, **extra) # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # SDK upgrades # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_agreement_submitted( success:, errors:, step:, analytics_id:, + opted_in_to_in_person_proofing: nil, error_details: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, @@ -1117,6 +1123,7 @@ def idv_doc_auth_agreement_submitted( analytics_id:, acuant_sdk_upgrade_ab_test_bucket:, skip_hybrid_handoff:, + opted_in_to_in_person_proofing:, **extra, ) end @@ -1127,9 +1134,11 @@ def idv_doc_auth_agreement_submitted( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # SDK upgrades # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_agreement_visited( step:, analytics_id:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, **extra @@ -1140,6 +1149,7 @@ def idv_doc_auth_agreement_visited( analytics_id:, acuant_sdk_upgrade_ab_test_bucket:, skip_hybrid_handoff:, + opted_in_to_in_person_proofing:, **extra, ) end @@ -1179,6 +1189,7 @@ def idv_doc_auth_capture_complete_visited( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # SDK upgrades # @param [Boolean] stored_result_present Whether a stored result was present + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_document_capture_submitted( success:, errors:, @@ -1187,6 +1198,7 @@ def idv_doc_auth_document_capture_submitted( liveness_checking_required:, selfie_check_required:, flow_path:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, redo_document_capture: nil, skip_hybrid_handoff: nil, @@ -1205,6 +1217,7 @@ def idv_doc_auth_document_capture_submitted( selfie_check_required:, acuant_sdk_upgrade_ab_test_bucket:, flow_path:, + opted_in_to_in_person_proofing:, stored_result_present:, **extra, ) @@ -1221,12 +1234,14 @@ def idv_doc_auth_document_capture_submitted( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # SDK upgrades # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_document_capture_visited( step:, analytics_id:, liveness_checking_required:, selfie_check_required:, flow_path:, + opted_in_to_in_person_proofing: nil, redo_document_capture: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, @@ -1241,6 +1256,7 @@ def idv_doc_auth_document_capture_visited( skip_hybrid_handoff:, liveness_checking_required:, selfie_check_required:, + opted_in_to_in_person_proofing:, acuant_sdk_upgrade_ab_test_bucket:, **extra, ) @@ -1275,12 +1291,14 @@ def idv_doc_auth_failed_image_resubmitted(side:, **extra) # @param [String] analytics_id Current IdV flow identifier # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [String] selection Selection form parameter + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_how_to_verify_submitted( success:, errors:, step:, analytics_id:, skip_hybrid_handoff:, + opted_in_to_in_person_proofing: nil, selection: nil, error_details: nil, **extra @@ -1294,6 +1312,7 @@ def idv_doc_auth_how_to_verify_submitted( analytics_id:, skip_hybrid_handoff:, selection:, + opted_in_to_in_person_proofing:, **extra, ) end @@ -1301,10 +1320,12 @@ def idv_doc_auth_how_to_verify_submitted( # @param [String] step Current IdV step # @param [String] analytics_id Current IdV flow identifier # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_how_to_verify_visited( step:, analytics_id:, skip_hybrid_handoff:, + opted_in_to_in_person_proofing: nil, **extra ) track_event( @@ -1312,6 +1333,7 @@ def idv_doc_auth_how_to_verify_visited( step:, analytics_id:, skip_hybrid_handoff:, + opted_in_to_in_person_proofing:, **extra, ) end @@ -1333,6 +1355,7 @@ def idv_doc_auth_how_to_verify_visited( # SDK upgrades # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [Hash] telephony_response Response from Telephony gem + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_hybrid_handoff_submitted( success:, errors:, @@ -1342,6 +1365,7 @@ def idv_doc_auth_hybrid_handoff_submitted( selfie_check_required:, destination:, flow_path:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, telephony_response: nil, @@ -1357,6 +1381,7 @@ def idv_doc_auth_hybrid_handoff_submitted( skip_hybrid_handoff:, selfie_check_required:, acuant_sdk_upgrade_ab_test_bucket:, + opted_in_to_in_person_proofing:, destination:, flow_path:, telephony_response:, @@ -1374,11 +1399,13 @@ def idv_doc_auth_hybrid_handoff_submitted( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # SDK upgrades # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_hybrid_handoff_visited( step:, analytics_id:, redo_document_capture:, selfie_check_required:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, **extra @@ -1389,6 +1416,7 @@ def idv_doc_auth_hybrid_handoff_visited( analytics_id:, redo_document_capture:, skip_hybrid_handoff:, + opted_in_to_in_person_proofing:, selfie_check_required:, acuant_sdk_upgrade_ab_test_bucket:, **extra, @@ -1398,13 +1426,24 @@ def idv_doc_auth_hybrid_handoff_visited( # @param [String] step Current IdV step # @param [String] analytics_id Current IdV flow identifier # @param ["hybrid","standard"] flow_path Document capture user flow + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # @identity.idp.previous_event_name IdV: doc auth send_link submitted - def idv_doc_auth_link_sent_submitted(step:, analytics_id:, flow_path:, **extra) + def idv_doc_auth_link_sent_submitted( + step:, + analytics_id:, + flow_path:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, + **extra + ) track_event( 'IdV: doc auth link_sent submitted', step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing:, + skip_hybrid_handoff:, **extra, ) end @@ -1412,12 +1451,23 @@ def idv_doc_auth_link_sent_submitted(step:, analytics_id:, flow_path:, **extra) # @param [String] step Current IdV step # @param [String] analytics_id Current IdV flow identifier # @param ["hybrid","standard"] flow_path Document capture user flow - def idv_doc_auth_link_sent_visited(step:, analytics_id:, flow_path:, **extra) + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing + def idv_doc_auth_link_sent_visited( + step:, + analytics_id:, + flow_path:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, + **extra + ) track_event( 'IdV: doc auth link_sent visited', step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing:, + skip_hybrid_handoff:, **extra, ) end @@ -1426,10 +1476,14 @@ def idv_doc_auth_link_sent_visited(step:, analytics_id:, flow_path:, **extra) # @param [String] analytics_id Current IdV flow identifier # @param ["hybrid","standard"] flow_path Document capture user flow # @param [Boolean] same_address_as_id + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_redo_ssn_submitted( step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, same_address_as_id: nil, **extra ) @@ -1438,6 +1492,8 @@ def idv_doc_auth_redo_ssn_submitted( step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing:, + skip_hybrid_handoff:, same_address_as_id:, **extra, ) @@ -1478,12 +1534,14 @@ def idv_doc_auth_socure_webhook_received( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [Boolean] same_address_as_id + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_ssn_submitted( success:, errors:, step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing: nil, error_details: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, @@ -1500,6 +1558,7 @@ def idv_doc_auth_ssn_submitted( skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, flow_path:, + opted_in_to_in_person_proofing:, same_address_as_id:, **extra, ) @@ -1513,10 +1572,12 @@ def idv_doc_auth_ssn_submitted( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [Boolean] same_address_as_id + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_ssn_visited( step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, same_address_as_id: nil, @@ -1529,6 +1590,7 @@ def idv_doc_auth_ssn_visited( skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, flow_path:, + opted_in_to_in_person_proofing:, same_address_as_id:, **extra, ) @@ -1625,6 +1687,8 @@ def idv_doc_auth_submitted_image_upload_form( # @param [String] workflow LexisNexis TrueID workflow # @param [String] birth_year Birth year from document # @param [Integer] issue_year Year document was issued + # @param [Hash] failed_image_fingerprints Hash of document field with an array of failed image + # fingerprints for that field. # @param [Integer] selfie_attempts number of selfie attempts the user currently has processed # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # SDK upgrades @@ -1656,6 +1720,7 @@ def idv_doc_auth_submitted_image_upload_vendor( flow_path:, liveness_checking_required:, issue_year:, + failed_image_fingerprints: nil, billed: nil, doc_auth_result: nil, vendor_request_time_in_ms: nil, @@ -1733,6 +1798,7 @@ def idv_doc_auth_submitted_image_upload_vendor( workflow:, birth_year:, issue_year:, + failed_image_fingerprints:, selfie_attempts:, acuant_sdk_upgrade_ab_test_bucket:, liveness_enabled:, @@ -1915,10 +1981,12 @@ def idv_doc_auth_verify_proofing_results( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [Boolean] same_address_as_id + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_verify_submitted( step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, same_address_as_id: nil, @@ -1931,6 +1999,7 @@ def idv_doc_auth_verify_submitted( skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, flow_path:, + opted_in_to_in_person_proofing:, same_address_as_id:, **extra, ) @@ -1944,10 +2013,12 @@ def idv_doc_auth_verify_submitted( # @param [String] acuant_sdk_upgrade_ab_test_bucket A/B test bucket for Acuant document capture # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [Boolean] same_address_as_id + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_doc_auth_verify_visited( step:, analytics_id:, flow_path:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, same_address_as_id: nil, @@ -1960,6 +2031,7 @@ def idv_doc_auth_verify_visited( skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, flow_path:, + opted_in_to_in_person_proofing:, same_address_as_id:, **extra, ) @@ -1981,11 +2053,19 @@ def idv_doc_auth_warning_visited(step_name:, remaining_submit_attempts:, **extra # @param [String] step Current IdV step # @param [String] analytics_id Current IdV flow identifier # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active - def idv_doc_auth_welcome_submitted(step:, analytics_id:, skip_hybrid_handoff: nil, **extra) + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing + def idv_doc_auth_welcome_submitted( + step:, + analytics_id:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, + **extra + ) track_event( 'IdV: doc auth welcome submitted', step:, analytics_id:, + opted_in_to_in_person_proofing:, skip_hybrid_handoff:, **extra, ) @@ -1995,12 +2075,20 @@ def idv_doc_auth_welcome_submitted(step:, analytics_id:, skip_hybrid_handoff: ni # @param [String] step Current IdV step # @param [String] analytics_id Current IdV flow identifier # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active - def idv_doc_auth_welcome_visited(step:, analytics_id:, skip_hybrid_handoff: nil, **extra) + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing + def idv_doc_auth_welcome_visited( + step:, + analytics_id:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, + **extra + ) track_event( 'IdV: doc auth welcome visited', step:, analytics_id:, skip_hybrid_handoff:, + opted_in_to_in_person_proofing:, **extra, ) end @@ -2025,6 +2113,7 @@ def idv_doc_auth_welcome_visited(step:, analytics_id:, skip_hybrid_handoff: nil, # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. # @param [Integer,nil] proofing_workflow_time_in_seconds The time since starting proofing + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # @identity.idp.previous_event_name IdV: review info visited def idv_enter_password_submitted( success:, @@ -2032,6 +2121,7 @@ def idv_enter_password_submitted( fraud_rejection:, gpo_verification_pending:, in_person_verification_pending:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, deactivation_reason: nil, @@ -2050,6 +2140,7 @@ def idv_enter_password_submitted( in_person_verification_pending:, skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, + opted_in_to_in_person_proofing:, fraud_rejection:, proofing_components:, active_profile_idv_level:, @@ -2073,9 +2164,11 @@ def idv_enter_password_submitted( # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. # used to verify the user's identity + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # User visited IDV password confirm page # @identity.idp.previous_event_name IdV: review info visited def idv_enter_password_visited( + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, proofing_components: nil, @@ -2086,6 +2179,7 @@ def idv_enter_password_visited( ) track_event( :idv_enter_password_visited, + opted_in_to_in_person_proofing:, skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, address_verification_method:, @@ -2116,6 +2210,7 @@ def idv_enter_password_visited( # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. # @param [Array,nil] profile_history Array of user's profiles (oldest to newest). # @param [Integer,nil] proofing_workflow_time_in_seconds The time since starting proofing + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # @see Reporting::IdentityVerificationReport#query This event is used by the identity verification # report. Changes here should be reflected there. # Tracks the last step of IDV, indicates the user successfully proofed @@ -2125,6 +2220,7 @@ def idv_final( fraud_rejection:, gpo_verification_pending:, in_person_verification_pending:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, deactivation_reason: nil, @@ -2142,6 +2238,7 @@ def idv_final( fraud_rejection:, gpo_verification_pending:, in_person_verification_pending:, + opted_in_to_in_person_proofing:, skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, deactivation_reason:, @@ -2343,6 +2440,8 @@ def idv_front_image_clicked( # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # GPO letter was enqueued and the time at which it was enqueued def idv_gpo_address_letter_enqueued( enqueued_at:, @@ -2350,6 +2449,8 @@ def idv_gpo_address_letter_enqueued( first_letter_requested_at:, hours_since_first_letter:, phone_step_attempts:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, proofing_components: nil, active_profile_idv_level: nil, pending_profile_idv_level: nil, @@ -2357,14 +2458,16 @@ def idv_gpo_address_letter_enqueued( ) track_event( 'IdV: USPS address letter enqueued', - enqueued_at: enqueued_at, - resend: resend, - first_letter_requested_at: first_letter_requested_at, - hours_since_first_letter: hours_since_first_letter, - phone_step_attempts: phone_step_attempts, - proofing_components: proofing_components, - active_profile_idv_level: active_profile_idv_level, - pending_profile_idv_level: pending_profile_idv_level, + enqueued_at:, + resend:, + first_letter_requested_at:, + hours_since_first_letter:, + phone_step_attempts:, + opted_in_to_in_person_proofing:, + skip_hybrid_handoff:, + proofing_components:, + active_profile_idv_level:, + pending_profile_idv_level:, **extra, ) end @@ -2384,12 +2487,16 @@ def idv_gpo_address_letter_enqueued( # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # GPO letter was requested def idv_gpo_address_letter_requested( resend:, first_letter_requested_at:, hours_since_first_letter:, phone_step_attempts:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, proofing_components: nil, active_profile_idv_level: nil, pending_profile_idv_level: nil, @@ -2397,13 +2504,15 @@ def idv_gpo_address_letter_requested( ) track_event( 'IdV: USPS address letter requested', - resend: resend, + resend:, first_letter_requested_at:, hours_since_first_letter:, phone_step_attempts:, - proofing_components: proofing_components, - active_profile_idv_level: active_profile_idv_level, - pending_profile_idv_level: pending_profile_idv_level, + opted_in_to_in_person_proofing:, + skip_hybrid_handoff:, + proofing_components:, + active_profile_idv_level:, + pending_profile_idv_level:, **extra, ) end @@ -2520,7 +2629,7 @@ def idv_in_person_email_reminder_job_exception( def idv_in_person_location_submitted( selected_location:, flow_path:, - opted_in_to_in_person_proofing:, + opted_in_to_in_person_proofing: nil, **extra ) track_event( @@ -2629,11 +2738,13 @@ def idv_in_person_prepare_visited(flow_path:, opted_in_to_in_person_proofing:, * # @param [String] analytics_id # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [Boolean] same_address_as_id + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # address page visited def idv_in_person_proofing_address_visited( flow_path:, step:, analytics_id:, + opted_in_to_in_person_proofing: nil, skip_hybrid_handoff: nil, same_address_as_id: nil, **extra @@ -2643,6 +2754,7 @@ def idv_in_person_proofing_address_visited( flow_path:, step:, analytics_id:, + opted_in_to_in_person_proofing:, skip_hybrid_handoff:, same_address_as_id:, **extra, @@ -2756,6 +2868,7 @@ def idv_in_person_proofing_nontransliterable_characters_submitted( # @param [Hash] errors Errors resulting from form validation # @param [Hash] error_details Details for errors that occurred in unsuccessful submission # @param [Boolean] same_address_as_id + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # User submitted state id on redo state id page def idv_in_person_proofing_redo_state_id_submitted( success:, @@ -2765,6 +2878,7 @@ def idv_in_person_proofing_redo_state_id_submitted( step: nil, analytics_id: nil, same_address_as_id: nil, + opted_in_to_in_person_proofing: nil, **extra ) track_event( @@ -2776,6 +2890,7 @@ def idv_in_person_proofing_redo_state_id_submitted( errors:, error_details:, same_address_as_id:, + opted_in_to_in_person_proofing:, **extra, ) end @@ -2789,6 +2904,7 @@ def idv_in_person_proofing_redo_state_id_submitted( # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # @param [Boolean, nil] same_address_as_id # @param [String] current_address_zip_code ZIP code of given address + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing def idv_in_person_proofing_residential_address_submitted( success:, errors:, @@ -2796,6 +2912,7 @@ def idv_in_person_proofing_residential_address_submitted( step:, analytics_id:, current_address_zip_code:, + opted_in_to_in_person_proofing: nil, error_details: nil, skip_hybrid_handoff: nil, same_address_as_id: nil, @@ -2809,6 +2926,7 @@ def idv_in_person_proofing_residential_address_submitted( step:, analytics_id:, current_address_zip_code:, + opted_in_to_in_person_proofing:, error_details:, skip_hybrid_handoff:, same_address_as_id:, @@ -2826,6 +2944,7 @@ def idv_in_person_proofing_residential_address_submitted( # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # @param [String] birth_year Birth year from document # @param [String] document_zip_code ZIP code from document + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # User submitted state id def idv_in_person_proofing_state_id_submitted( success:, @@ -2835,6 +2954,7 @@ def idv_in_person_proofing_state_id_submitted( analytics_id:, birth_year:, document_zip_code:, + skip_hybrid_handoff: nil, error_details: nil, same_address_as_id: nil, opted_in_to_in_person_proofing: nil, @@ -2850,6 +2970,7 @@ def idv_in_person_proofing_state_id_submitted( error_details:, birth_year:, document_zip_code:, + skip_hybrid_handoff:, same_address_as_id:, opted_in_to_in_person_proofing:, **extra, @@ -2861,12 +2982,14 @@ def idv_in_person_proofing_state_id_submitted( # @param [String] analytics_id # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # @param [Boolean] same_address_as_id + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active # State id page visited def idv_in_person_proofing_state_id_visited( flow_path: nil, step: nil, analytics_id: nil, opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, same_address_as_id: nil, **extra ) @@ -2876,6 +2999,7 @@ def idv_in_person_proofing_state_id_visited( step:, analytics_id:, opted_in_to_in_person_proofing:, + skip_hybrid_handoff:, same_address_as_id:, **extra, ) @@ -2899,16 +3023,21 @@ def idv_in_person_ready_to_verify_sp_link_clicked(**extra) # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # The user visited the "ready to verify" page for the in person proofing flow - def idv_in_person_ready_to_verify_visit(proofing_components: nil, - active_profile_idv_level: nil, - pending_profile_idv_level: nil, - **extra) + def idv_in_person_ready_to_verify_visit( + opted_in_to_in_person_proofing: nil, + proofing_components: nil, + active_profile_idv_level: nil, + pending_profile_idv_level: nil, + **extra + ) track_event( 'IdV: in person ready to verify visited', - proofing_components: proofing_components, - active_profile_idv_level: active_profile_idv_level, - pending_profile_idv_level: pending_profile_idv_level, + opted_in_to_in_person_proofing:, + proofing_components:, + active_profile_idv_level:, + pending_profile_idv_level:, **extra, ) end @@ -3589,8 +3718,10 @@ def idv_personal_key_submitted( # @param [Boolean] encrypted_profiles_missing True if user's session had no encrypted pii # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # User visited IDV personal key page def idv_personal_key_visited( + opted_in_to_in_person_proofing: nil, proofing_components: nil, address_verification_method: nil, in_person_verification_pending: nil, @@ -3601,12 +3732,13 @@ def idv_personal_key_visited( ) track_event( 'IdV: personal key visited', - proofing_components: proofing_components, - address_verification_method: address_verification_method, - in_person_verification_pending: in_person_verification_pending, - encrypted_profiles_missing: encrypted_profiles_missing, - active_profile_idv_level: active_profile_idv_level, - pending_profile_idv_level: pending_profile_idv_level, + opted_in_to_in_person_proofing:, + proofing_components:, + address_verification_method:, + in_person_verification_pending:, + encrypted_profiles_missing:, + active_profile_idv_level:, + pending_profile_idv_level:, **extra, ) end @@ -3632,6 +3764,7 @@ def idv_personal_key_visited( # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # The user submitted their phone on the phone confirmation page def idv_phone_confirmation_form_submitted( success:, @@ -3642,6 +3775,7 @@ def idv_phone_confirmation_form_submitted( carrier:, country_code:, area_code:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, error_details: nil, @@ -3660,6 +3794,7 @@ def idv_phone_confirmation_form_submitted( carrier:, country_code:, area_code:, + opted_in_to_in_person_proofing:, skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, otp_delivery_preference:, @@ -3880,6 +4015,7 @@ def idv_phone_confirmation_otp_sent( # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # When a user attempts to confirm possession of a new phone number during the IDV process def idv_phone_confirmation_otp_submitted( success:, @@ -3889,6 +4025,7 @@ def idv_phone_confirmation_otp_submitted( otp_delivery_preference:, second_factor_attempts_count:, second_factor_locked_at:, + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, error_details: nil, @@ -3907,6 +4044,7 @@ def idv_phone_confirmation_otp_submitted( otp_delivery_preference:, second_factor_attempts_count:, second_factor_locked_at:, + opted_in_to_in_person_proofing:, skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, proofing_components:, @@ -3962,6 +4100,7 @@ def idv_phone_confirmation_otp_visit( # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # The vendor finished the process of confirming the users phone def idv_phone_confirmation_vendor_submitted( success:, @@ -3972,6 +4111,7 @@ def idv_phone_confirmation_vendor_submitted( phone_fingerprint:, new_phone_added:, hybrid_handoff_phone_used:, + opted_in_to_in_person_proofing: nil, error_details: nil, proofing_components: nil, active_profile_idv_level: nil, @@ -3989,6 +4129,7 @@ def idv_phone_confirmation_vendor_submitted( phone_fingerprint:, new_phone_added:, hybrid_handoff_phone_used:, + opted_in_to_in_person_proofing:, proofing_components:, active_profile_idv_level:, pending_profile_idv_level:, @@ -4010,9 +4151,13 @@ def idv_phone_confirmation_vendor_submitted( # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] skip_hybrid_handoff Whether skipped hybrid handoff A/B test is active + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # When a user gets an error during the phone finder flow of IDV def idv_phone_error_visited( type:, + opted_in_to_in_person_proofing: nil, + skip_hybrid_handoff: nil, proofing_components: nil, limiter_expires_at: nil, remaining_submit_attempts: nil, @@ -4023,6 +4168,8 @@ def idv_phone_error_visited( track_event( 'IdV: phone error visited', type:, + opted_in_to_in_person_proofing:, + skip_hybrid_handoff:, proofing_components:, limiter_expires_at:, remaining_submit_attempts:, @@ -4044,8 +4191,10 @@ def idv_phone_error_visited( # @option proofing_components [String,nil] 'threatmetrix_review_status' TMX decision on the user # @param [String,nil] active_profile_idv_level ID verification level of user's active profile. # @param [String,nil] pending_profile_idv_level ID verification level of user's pending profile. + # @param [Boolean] opted_in_to_in_person_proofing User opted into in person proofing # User visited idv phone of record def idv_phone_of_record_visited( + opted_in_to_in_person_proofing: nil, acuant_sdk_upgrade_ab_test_bucket: nil, skip_hybrid_handoff: nil, proofing_components: nil, @@ -4055,6 +4204,7 @@ def idv_phone_of_record_visited( ) track_event( 'IdV: phone of record visited', + opted_in_to_in_person_proofing:, skip_hybrid_handoff:, acuant_sdk_upgrade_ab_test_bucket:, proofing_components:, diff --git a/spec/controllers/idv/agreement_controller_spec.rb b/spec/controllers/idv/agreement_controller_spec.rb index fb47293df1b..d4905310069 100644 --- a/spec/controllers/idv/agreement_controller_spec.rb +++ b/spec/controllers/idv/agreement_controller_spec.rb @@ -1,20 +1,14 @@ require 'rails_helper' -RSpec.describe Idv::AgreementController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::AgreementController do include FlowPolicyHelper let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_sign_in(user) stub_up_to(:welcome, idv_session: subject.idv_session) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -45,7 +39,7 @@ { step: 'agreement', analytics_id: 'Doc Auth', - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -109,7 +103,7 @@ errors: {}, step: 'agreement', analytics_id: 'Doc Auth', - }.merge(ab_test_args) + } end let(:skip_hybrid_handoff) { nil } diff --git a/spec/controllers/idv/by_mail/request_letter_controller_spec.rb b/spec/controllers/idv/by_mail/request_letter_controller_spec.rb index 357fa45596d..81e58922453 100644 --- a/spec/controllers/idv/by_mail/request_letter_controller_spec.rb +++ b/spec/controllers/idv/by_mail/request_letter_controller_spec.rb @@ -1,16 +1,10 @@ require 'rails_helper' -RSpec.describe Idv::ByMail::RequestLetterController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::ByMail::RequestLetterController do let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -104,7 +98,6 @@ resend: false, phone_step_attempts: 1, hours_since_first_letter: 0, - **ab_test_args, ), ) end diff --git a/spec/controllers/idv/by_mail/resend_letter_controller_spec.rb b/spec/controllers/idv/by_mail/resend_letter_controller_spec.rb index 79a91b15d64..5ca562aa876 100644 --- a/spec/controllers/idv/by_mail/resend_letter_controller_spec.rb +++ b/spec/controllers/idv/by_mail/resend_letter_controller_spec.rb @@ -1,17 +1,11 @@ require 'rails_helper' -RSpec.describe Idv::ByMail::ResendLetterController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::ByMail::ResendLetterController do let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_sign_in(user) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#new' do diff --git a/spec/controllers/idv/document_capture_controller_spec.rb b/spec/controllers/idv/document_capture_controller_spec.rb index 0eef9736157..41529df2ddb 100644 --- a/spec/controllers/idv/document_capture_controller_spec.rb +++ b/spec/controllers/idv/document_capture_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::DocumentCaptureController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::DocumentCaptureController do include FlowPolicyHelper let(:document_capture_session_requested_at) { Time.zone.now } @@ -17,10 +16,6 @@ let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - # selfie related test flags let(:sp_selfie_enabled) { false } let(:flow_path) { 'standard' } @@ -37,7 +32,6 @@ allow(controller).to receive(:resolved_authn_context_result). and_return(resolved_authn_context) subject.idv_session.flow_path = flow_path - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -110,7 +104,7 @@ step: 'document_capture', liveness_checking_required: false, selfie_check_required: sp_selfie_enabled, - }.merge(ab_test_args) + } end it 'has non-nil presenter' do @@ -302,7 +296,7 @@ step: 'document_capture', liveness_checking_required: false, selfie_check_required: sp_selfie_enabled, - }.merge(ab_test_args) + } end let(:result) { { success: true, errors: {} } } diff --git a/spec/controllers/idv/enter_password_controller_spec.rb b/spec/controllers/idv/enter_password_controller_spec.rb index 86e8ec2057e..145d455324b 100644 --- a/spec/controllers/idv/enter_password_controller_spec.rb +++ b/spec/controllers/idv/enter_password_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::EnterPasswordController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::EnterPasswordController do include UspsIppHelper let(:user) do @@ -18,15 +17,10 @@ subject.idv_session end - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_analytics stub_sign_in(user) allow(IdentityConfig.store).to receive(:usps_mock_fallback).and_return(false) - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) subject.idv_session.welcome_visited = true subject.idv_session.idv_consent_given_at = Time.zone.now subject.idv_session.proofing_started_at = 5.minutes.ago.iso8601 @@ -279,7 +273,6 @@ def show fraud_rejection: false, gpo_verification_pending: false, in_person_verification_pending: false, - **ab_test_args, ), ) end @@ -297,7 +290,6 @@ def show gpo_verification_pending: false, in_person_verification_pending: false, proofing_workflow_time_in_seconds: 5.minutes.to_i, - **ab_test_args, ), ) expect(@analytics).to have_logged_event( @@ -848,7 +840,6 @@ def show fraud_rejection: false, gpo_verification_pending: false, in_person_verification_pending: false, - **ab_test_args, ), ) expect(@analytics).to have_logged_event( @@ -859,7 +850,6 @@ def show fraud_rejection: false, gpo_verification_pending: false, in_person_verification_pending: false, - **ab_test_args, ), ) end @@ -916,7 +906,6 @@ def show phone_step_attempts: 1, first_letter_requested_at: subject.idv_session.profile.gpo_verification_pending_at, hours_since_first_letter: 0, - **ab_test_args, ), ) end @@ -936,7 +925,6 @@ def show phone_step_attempts: RateLimiter.max_attempts(rate_limit_type), first_letter_requested_at: subject.idv_session.profile.gpo_verification_pending_at, hours_since_first_letter: 0, - **ab_test_args, ), ) end diff --git a/spec/controllers/idv/how_to_verify_controller_spec.rb b/spec/controllers/idv/how_to_verify_controller_spec.rb index 63d9f0100c8..79a87fd0123 100644 --- a/spec/controllers/idv/how_to_verify_controller_spec.rb +++ b/spec/controllers/idv/how_to_verify_controller_spec.rb @@ -1,12 +1,8 @@ require 'rails_helper' -RSpec.describe Idv::HowToVerifyController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::HowToVerifyController do let(:user) { create(:user) } let(:enabled) { true } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end let(:service_provider) do create(:service_provider, :active, :in_person_proofing_enabled) end @@ -16,7 +12,6 @@ allow(IdentityConfig.store).to receive(:in_person_proofing_enabled) { true } stub_sign_in(user) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) allow(subject.idv_session).to receive(:service_provider).and_return(service_provider) subject.idv_session.welcome_visited = true subject.idv_session.idv_consent_given_at = Time.zone.now @@ -115,7 +110,7 @@ { step: 'how_to_verify', analytics_id: 'Doc Auth', - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -184,7 +179,7 @@ error_details: { selection: { blank: true } }, errors: { selection: ['Select a way to verify your identity.'] }, success: false, - }.merge(ab_test_args) + } end let(:params) { nil } @@ -204,7 +199,7 @@ error_details: { selection: { inclusion: true } }, errors: { selection: ['Select a way to verify your identity.'] }, success: false, - }.merge(ab_test_args) + } end it_behaves_like 'invalid form submissions' @@ -219,7 +214,7 @@ errors: {}, success: true, selection:, - }.merge(ab_test_args) + } end it 'sets skip doc auth on idv session to false and redirects to hybrid handoff' do put :update, params: params @@ -245,7 +240,7 @@ errors: {}, success: true, selection:, - }.merge(ab_test_args) + } end it 'sets skip doc auth on idv session to true and redirects to document capture' do put :update, params: params diff --git a/spec/controllers/idv/hybrid_handoff_controller_spec.rb b/spec/controllers/idv/hybrid_handoff_controller_spec.rb index c4a9ab245cf..65d5c6c9300 100644 --- a/spec/controllers/idv/hybrid_handoff_controller_spec.rb +++ b/spec/controllers/idv/hybrid_handoff_controller_spec.rb @@ -1,14 +1,10 @@ require 'rails_helper' -RSpec.describe Idv::HybridHandoffController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::HybridHandoffController do include FlowPolicyHelper let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end let(:service_provider) do create(:service_provider, :active, :in_person_proofing_enabled) end @@ -22,7 +18,6 @@ stub_sign_in(user) stub_up_to(:agreement, idv_session: subject.idv_session) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) allow(subject.idv_session).to receive(:service_provider).and_return(service_provider) resolved_authn_context_result = sp_selfie_enabled ? @@ -67,7 +62,7 @@ step: 'hybrid_handoff', analytics_id: 'Doc Auth', selfie_check_required: sp_selfie_enabled, - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -306,7 +301,7 @@ request_id: 'fake-message-request-id', success: true, }, - }.merge(ab_test_args) + } end let(:params) do @@ -356,7 +351,7 @@ step: 'hybrid_handoff', analytics_id: 'Doc Auth', selfie_check_required: sp_selfie_enabled, - }.merge(ab_test_args) + } end let(:params) do diff --git a/spec/controllers/idv/hybrid_mobile/capture_complete_controller_spec.rb b/spec/controllers/idv/hybrid_mobile/capture_complete_controller_spec.rb index edbb9fefdc4..92cd64ad58f 100644 --- a/spec/controllers/idv/hybrid_mobile/capture_complete_controller_spec.rb +++ b/spec/controllers/idv/hybrid_mobile/capture_complete_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::HybridMobile::CaptureCompleteController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::HybridMobile::CaptureCompleteController do let(:user) { create(:user) } let!(:document_capture_session) do @@ -21,17 +20,12 @@ ) end - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do session[:doc_capture_user_id] = user&.id session[:document_capture_session_uuid] = document_capture_session_uuid stub_analytics allow(subject).to receive(:confirm_document_capture_session_complete). and_return(true) - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe 'before_actions' do @@ -51,7 +45,7 @@ flow_path: 'hybrid', step: 'capture_complete', liveness_checking_required: false, - }.merge(ab_test_args) + } end it 'renders the show template' do diff --git a/spec/controllers/idv/hybrid_mobile/document_capture_controller_spec.rb b/spec/controllers/idv/hybrid_mobile/document_capture_controller_spec.rb index 35b601cc796..3f58041aa49 100644 --- a/spec/controllers/idv/hybrid_mobile/document_capture_controller_spec.rb +++ b/spec/controllers/idv/hybrid_mobile/document_capture_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::HybridMobile::DocumentCaptureController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::HybridMobile::DocumentCaptureController do let(:user) { create(:user) } let!(:document_capture_session) do @@ -17,17 +16,11 @@ let(:document_capture_session_result_captured_at) { Time.zone.now + 1.second } let(:document_capture_session_result_success) { true } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_analytics session[:doc_capture_user_id] = user&.id session[:document_capture_session_uuid] = document_capture_session_uuid - - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe 'before_actions' do @@ -59,7 +52,7 @@ step: 'document_capture', selfie_check_required: false, liveness_checking_required: boolean, - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -182,7 +175,7 @@ step: 'document_capture', liveness_checking_required: false, selfie_check_required: boolean, - }.merge(ab_test_args) + } end before do diff --git a/spec/controllers/idv/in_person/ssn_controller_spec.rb b/spec/controllers/idv/in_person/ssn_controller_spec.rb index c44ada2bfa6..b8d9d01ded8 100644 --- a/spec/controllers/idv/in_person/ssn_controller_spec.rb +++ b/spec/controllers/idv/in_person/ssn_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::InPerson::SsnController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::InPerson::SsnController do let(:pii_from_user) { Idp::Constants::MOCK_IDV_APPLICANT_SAME_ADDRESS_AS_ID_WITH_NO_SSN.dup } let(:flow_session) do @@ -12,15 +11,10 @@ let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_sign_in(user) subject.user_session['idv/in_person'] = flow_session stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) subject.idv_session.flow_path = 'standard' end @@ -49,7 +43,7 @@ flow_path: 'standard', step: 'ssn', same_address_as_id: true, - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -119,7 +113,7 @@ success: true, errors: {}, same_address_as_id: true, - }.merge(ab_test_args) + } end it 'sends analytics_submitted event' do @@ -163,7 +157,7 @@ }, error_details: { ssn: { invalid: true } }, same_address_as_id: true, - }.merge(ab_test_args) + } end render_views diff --git a/spec/controllers/idv/in_person/state_id_controller_spec.rb b/spec/controllers/idv/in_person/state_id_controller_spec.rb index 6c81162a1ba..1b195113fca 100644 --- a/spec/controllers/idv/in_person/state_id_controller_spec.rb +++ b/spec/controllers/idv/in_person/state_id_controller_spec.rb @@ -1,17 +1,12 @@ require 'rails_helper' -RSpec.describe Idv::InPerson::StateIdController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::InPerson::StateIdController do include FlowPolicyHelper include InPersonHelper let(:user) { build(:user) } let(:enrollment) { InPersonEnrollment.new } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do allow(IdentityConfig.store).to receive(:in_person_state_id_controller_enabled). and_return(true) @@ -23,7 +18,6 @@ subject.user_session['idv/in_person'] = { pii_from_user: {} } subject.idv_session.ssn = nil # This made specs pass. Might need more investigation. stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe 'before_actions' do @@ -77,7 +71,7 @@ analytics_id: 'In Person Proofing', flow_path: 'standard', step: 'state_id', - }.merge(ab_test_args) + } end it 'has non-nil presenter' do @@ -183,7 +177,7 @@ same_address_as_id: true, birth_year: dob[:year], document_zip_code: identity_doc_zipcode&.slice(0, 5), - }.merge(ab_test_args) + } end it 'logs idv_in_person_proofing_state_id_submitted' do diff --git a/spec/controllers/idv/in_person/verify_info_controller_spec.rb b/spec/controllers/idv/in_person/verify_info_controller_spec.rb index b473e2d420b..ebea673e80c 100644 --- a/spec/controllers/idv/in_person/verify_info_controller_spec.rb +++ b/spec/controllers/idv/in_person/verify_info_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::InPerson::VerifyInfoController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::InPerson::VerifyInfoController do let(:pii_from_user) { Idp::Constants::MOCK_IDV_APPLICANT_SAME_ADDRESS_AS_ID.dup } let(:flow_session) do { pii_from_user: pii_from_user } @@ -10,16 +9,11 @@ let(:user) { create(:user, :with_phone, with: { phone: '+1 (415) 555-0130' }) } let(:service_provider) { create(:service_provider) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_sign_in(user) subject.idv_session.flow_path = 'standard' subject.idv_session.ssn = Idp::Constants::MOCK_IDV_APPLICANT_SAME_ADDRESS_AS_ID[:ssn] subject.user_session['idv/in_person'] = flow_session - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -79,7 +73,7 @@ flow_path: 'standard', step: 'verify', same_address_as_id: true, - }.merge(ab_test_args), + }, ) end @@ -139,7 +133,7 @@ flow_path: 'standard', step: 'verify', same_address_as_id: true, - }.merge(ab_test_args), + }, ), ) end diff --git a/spec/controllers/idv/link_sent_controller_spec.rb b/spec/controllers/idv/link_sent_controller_spec.rb index 660622b51fd..fea657a0e01 100644 --- a/spec/controllers/idv/link_sent_controller_spec.rb +++ b/spec/controllers/idv/link_sent_controller_spec.rb @@ -1,20 +1,14 @@ require 'rails_helper' -RSpec.describe Idv::LinkSentController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::LinkSentController do let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_sign_in(user) subject.idv_session.welcome_visited = true subject.idv_session.idv_consent_given_at = Time.zone.now subject.idv_session.flow_path = 'hybrid' stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -53,7 +47,7 @@ analytics_id: 'Doc Auth', flow_path: 'hybrid', step: 'link_sent', - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -115,7 +109,7 @@ analytics_id: 'Doc Auth', flow_path: 'hybrid', step: 'link_sent', - }.merge(ab_test_args) + } end it 'invalidates future steps' do diff --git a/spec/controllers/idv/otp_verification_controller_spec.rb b/spec/controllers/idv/otp_verification_controller_spec.rb index 586a83d47a3..ddd3833539a 100644 --- a/spec/controllers/idv/otp_verification_controller_spec.rb +++ b/spec/controllers/idv/otp_verification_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::OtpVerificationController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::OtpVerificationController do let(:user) { create(:user) } let(:phone) { '2255555000' } @@ -19,13 +18,9 @@ sent_at: phone_confirmation_otp_sent_at, ) end - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end before do stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) sign_in(user) stub_verify_steps_one_and_two(user) @@ -169,7 +164,6 @@ code_matches: true, otp_delivery_preference: :sms, second_factor_attempts_count: 0, - **ab_test_args, ), ) end diff --git a/spec/controllers/idv/phone_controller_spec.rb b/spec/controllers/idv/phone_controller_spec.rb index fdc92ebcea3..664a4dee1c9 100644 --- a/spec/controllers/idv/phone_controller_spec.rb +++ b/spec/controllers/idv/phone_controller_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::PhoneController, allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::PhoneController do include FlowPolicyHelper let(:max_attempts) { RateLimiter.max_attempts(:proof_address) } @@ -241,13 +241,7 @@ with: { phone: '+1 (415) 555-0130' } ) end - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) - end context 'when form is invalid' do let(:improbable_phone_message) { t('errors.messages.improbable_phone') } let(:improbable_otp_message) { 'is not included in the list' } @@ -309,7 +303,6 @@ phone_type: :mobile, otp_delivery_preference: '🎷', types: [], - **ab_test_args, ), ) @@ -351,7 +344,6 @@ phone_type: :mobile, otp_delivery_preference: 'sms', types: [:fixed_or_mobile], - **ab_test_args, ), ) end diff --git a/spec/controllers/idv/phone_errors_controller_spec.rb b/spec/controllers/idv/phone_errors_controller_spec.rb index 14e9003ff03..85f45b5df22 100644 --- a/spec/controllers/idv/phone_errors_controller_spec.rb +++ b/spec/controllers/idv/phone_errors_controller_spec.rb @@ -1,11 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::PhoneErrorsController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - +RSpec.describe Idv::PhoneErrorsController do describe '#step_info' do it 'returns a valid StepInfo object' do expect(Idv::PhoneErrorsController.step_info).to be_valid @@ -15,7 +10,6 @@ before do allow(subject).to receive(:remaining_submit_attempts).and_return(5) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) if user stub_sign_in(user) @@ -159,7 +153,6 @@ 'IdV: phone error visited', type: action, remaining_submit_attempts: 4, - **ab_test_args, ) end end @@ -212,7 +205,6 @@ 'IdV: phone error visited', type: action, remaining_submit_attempts: 4, - **ab_test_args, ) end end @@ -244,7 +236,6 @@ 'IdV: phone error visited', type: action, limiter_expires_at: attempted_at + rate_limit_window, - **ab_test_args, ) end end diff --git a/spec/controllers/idv/ssn_controller_spec.rb b/spec/controllers/idv/ssn_controller_spec.rb index da514b54abd..d4e32ca1373 100644 --- a/spec/controllers/idv/ssn_controller_spec.rb +++ b/spec/controllers/idv/ssn_controller_spec.rb @@ -1,21 +1,16 @@ require 'rails_helper' -RSpec.describe Idv::SsnController, allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::SsnController do include FlowPolicyHelper let(:ssn) { Idp::Constants::MOCK_IDV_APPLICANT_WITH_SSN[:ssn] } let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_sign_in(user) stub_up_to(:document_capture, idv_session: subject.idv_session) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -54,7 +49,7 @@ analytics_id: 'Doc Auth', flow_path: 'standard', step: 'ssn', - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -131,7 +126,7 @@ step: 'ssn', success: true, errors: {}, - }.merge(ab_test_args) + } end it 'updates idv_session.ssn' do @@ -189,7 +184,7 @@ ssn: [t('idv.errors.pattern_mismatch.ssn')], }, error_details: { ssn: { invalid: true } }, - }.merge(ab_test_args) + } end render_views diff --git a/spec/controllers/idv/verify_info_controller_spec.rb b/spec/controllers/idv/verify_info_controller_spec.rb index 166ff64935d..67107ec790e 100644 --- a/spec/controllers/idv/verify_info_controller_spec.rb +++ b/spec/controllers/idv/verify_info_controller_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe Idv::VerifyInfoController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::VerifyInfoController do include FlowPolicyHelper let(:user) { create(:user) } @@ -10,18 +9,13 @@ analytics_id: 'Doc Auth', flow_path: 'standard', step: 'verify', - }.merge(ab_test_args) - end - - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } + } end before do stub_sign_in(user) stub_up_to(:ssn, idv_session: subject.idv_session) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -62,7 +56,7 @@ analytics_id: 'Doc Auth', flow_path: 'standard', step: 'verify', - }.merge(ab_test_args), + }, ) end @@ -300,7 +294,7 @@ analytics_id: 'Doc Auth', flow_path: 'standard', step: 'verify', - }.merge(ab_test_args), + }, ), ) end diff --git a/spec/controllers/idv/welcome_controller_spec.rb b/spec/controllers/idv/welcome_controller_spec.rb index 7dc2a51d62e..793e1eebfe7 100644 --- a/spec/controllers/idv/welcome_controller_spec.rb +++ b/spec/controllers/idv/welcome_controller_spec.rb @@ -1,17 +1,11 @@ require 'rails_helper' -RSpec.describe Idv::WelcomeController, - allowed_extra_analytics: [:sample_bucket1, :sample_bucket2] do +RSpec.describe Idv::WelcomeController do let(:user) { create(:user) } - let(:ab_test_args) do - { sample_bucket1: :sample_value1, sample_bucket2: :sample_value2 } - end - before do stub_sign_in(user) stub_analytics - allow(subject).to receive(:ab_test_analytics_buckets).and_return(ab_test_args) end describe '#step_info' do @@ -42,7 +36,7 @@ { step: 'welcome', analytics_id: 'Doc Auth', - }.merge(ab_test_args) + } end it 'renders the show template' do @@ -122,7 +116,7 @@ { step: 'welcome', analytics_id: 'Doc Auth', - }.merge(ab_test_args) + } end it 'sends analytics_submitted event' do diff --git a/spec/features/accessibility/idv_pages_spec.rb b/spec/features/accessibility/idv_pages_spec.rb index 85087f1fe33..d5d92c3374a 100644 --- a/spec/features/accessibility/idv_pages_spec.rb +++ b/spec/features/accessibility/idv_pages_spec.rb @@ -1,7 +1,7 @@ require 'rails_helper' require 'axe-rspec' -RSpec.feature 'Accessibility on IDV pages', :js, allowed_extra_analytics: [:*] do +RSpec.feature 'Accessibility on IDV pages', :js do describe 'IDV pages' do include IdvStepHelper diff --git a/spec/features/idv/analytics_spec.rb b/spec/features/idv/analytics_spec.rb index 3f2624a5bb0..465b3de5145 100644 --- a/spec/features/idv/analytics_spec.rb +++ b/spec/features/idv/analytics_spec.rb @@ -1,7 +1,7 @@ require 'rails_helper' require 'csv' -RSpec.feature 'Analytics Regression', js: true, allowed_extra_analytics: [:*] do +RSpec.feature 'Analytics Regression', :js do include IdvStepHelper include InPersonHelper diff --git a/spec/features/idv/clearing_and_restarting_spec.rb b/spec/features/idv/clearing_and_restarting_spec.rb index a7e5915c625..8e752b26ee4 100644 --- a/spec/features/idv/clearing_and_restarting_spec.rb +++ b/spec/features/idv/clearing_and_restarting_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.describe 'clearing IdV and restarting', allowed_extra_analytics: [:*] do +RSpec.describe 'clearing IdV and restarting' do include IdvStepHelper let(:user) { user_with_2fa } diff --git a/spec/features/idv/doc_auth/address_step_spec.rb b/spec/features/idv/doc_auth/address_step_spec.rb index 0fa48a3299b..ed8969a8409 100644 --- a/spec/features/idv/doc_auth/address_step_spec.rb +++ b/spec/features/idv/doc_auth/address_step_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'doc auth verify step', :js, allowed_extra_analytics: [:*] do +RSpec.feature 'doc auth verify step', :js do include IdvStepHelper include DocAuthHelper diff --git a/spec/features/idv/doc_auth/document_capture_spec.rb b/spec/features/idv/doc_auth/document_capture_spec.rb index 2d89db5ae1c..b6b4746625c 100644 --- a/spec/features/idv/doc_auth/document_capture_spec.rb +++ b/spec/features/idv/doc_auth/document_capture_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'document capture step', :js, allowed_extra_analytics: [:*] do +RSpec.feature 'document capture step', :js do include IdvStepHelper include DocAuthHelper include DocCaptureHelper @@ -1282,7 +1282,7 @@ def costing_for(cost_type) end end -RSpec.feature 'direct access to IPP on desktop', :js, allowed_extra_analytics: [:*] do +RSpec.feature 'direct access to IPP on desktop', :js do include IdvStepHelper include DocAuthHelper context 'direct access to IPP before handoff page' do diff --git a/spec/features/idv/doc_auth/how_to_verify_spec.rb b/spec/features/idv/doc_auth/how_to_verify_spec.rb index 10600f5e5b8..5de62644584 100644 --- a/spec/features/idv/doc_auth/how_to_verify_spec.rb +++ b/spec/features/idv/doc_auth/how_to_verify_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'how to verify step', js: true, allowed_extra_analytics: [:*] do +RSpec.feature 'how to verify step', js: true do include IdvHelper include DocAuthHelper diff --git a/spec/features/idv/doc_auth/link_sent_spec.rb b/spec/features/idv/doc_auth/link_sent_spec.rb index a8ffd86fa08..82a5b0e76cc 100644 --- a/spec/features/idv/doc_auth/link_sent_spec.rb +++ b/spec/features/idv/doc_auth/link_sent_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'doc auth link sent step', allowed_extra_analytics: [:*] do +RSpec.feature 'doc auth link sent step' do include IdvStepHelper include DocAuthHelper include DocCaptureHelper diff --git a/spec/features/idv/hybrid_mobile/entry_spec.rb b/spec/features/idv/hybrid_mobile/entry_spec.rb index 1bafd59bf40..9b252f78109 100644 --- a/spec/features/idv/hybrid_mobile/entry_spec.rb +++ b/spec/features/idv/hybrid_mobile/entry_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'mobile hybrid flow entry', js: true, allowed_extra_analytics: [:*] do +RSpec.feature 'mobile hybrid flow entry', :js do include IdvStepHelper let(:link_sent_via_sms) do diff --git a/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb b/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb index 99637e6dbd2..e1b04ea52ed 100644 --- a/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb +++ b/spec/features/idv/hybrid_mobile/hybrid_mobile_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.describe 'Hybrid Flow', :allow_net_connect_on_start, allowed_extra_analytics: [:*] do +RSpec.describe 'Hybrid Flow', :allow_net_connect_on_start do include IdvHelper include IdvStepHelper include DocAuthHelper diff --git a/spec/features/idv/phone_errors_spec.rb b/spec/features/idv/phone_errors_spec.rb index 4b33e565e77..6313e740354 100644 --- a/spec/features/idv/phone_errors_spec.rb +++ b/spec/features/idv/phone_errors_spec.rb @@ -1,5 +1,6 @@ require 'rails_helper' -RSpec.feature 'phone errors', :js, allowed_extra_analytics: [:*] do + +RSpec.feature 'phone errors', :js do include IdvStepHelper include IdvHelper diff --git a/spec/features/idv/proof_address_rate_limit_spec.rb b/spec/features/idv/proof_address_rate_limit_spec.rb index 52ed6b61ed6..f1605506b0e 100644 --- a/spec/features/idv/proof_address_rate_limit_spec.rb +++ b/spec/features/idv/proof_address_rate_limit_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'address proofing rate limit', allowed_extra_analytics: [:*] do +RSpec.feature 'address proofing rate limit' do include IdvStepHelper include IdvHelper diff --git a/spec/features/idv/puerto_rican_address_spec.rb b/spec/features/idv/puerto_rican_address_spec.rb index fa57bae9717..8bb24149a5b 100644 --- a/spec/features/idv/puerto_rican_address_spec.rb +++ b/spec/features/idv/puerto_rican_address_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.describe 'proofing flow with a Puerto Rican document', :js, allowed_extra_analytics: [:*] do +RSpec.describe 'proofing flow with a Puerto Rican document', :js do include DocAuthHelper include IdvStepHelper diff --git a/spec/features/idv/steps/in_person/ssn_spec.rb b/spec/features/idv/steps/in_person/ssn_spec.rb index 364ba60dd80..88156e68063 100644 --- a/spec/features/idv/steps/in_person/ssn_spec.rb +++ b/spec/features/idv/steps/in_person/ssn_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.describe 'doc auth IPP ssn step', js: true, allowed_extra_analytics: [:*] do +RSpec.describe 'doc auth IPP ssn step', :js do include IdvStepHelper include InPersonHelper diff --git a/spec/features/idv/steps/in_person/state_id_50_50_spec.rb b/spec/features/idv/steps/in_person/state_id_50_50_spec.rb index 19cfff04a87..d08dd7c2a71 100644 --- a/spec/features/idv/steps/in_person/state_id_50_50_spec.rb +++ b/spec/features/idv/steps/in_person/state_id_50_50_spec.rb @@ -1,7 +1,6 @@ require 'rails_helper' -RSpec.describe 'state id 50/50 state', js: true, allowed_extra_analytics: [:*], - allow_browser_log: true do +RSpec.describe 'state id 50/50 state', :js, allow_browser_log: true do include IdvStepHelper include InPersonHelper diff --git a/spec/features/idv/steps/in_person/state_id_controller_spec.rb b/spec/features/idv/steps/in_person/state_id_controller_spec.rb index d61427536d6..52275c4d36a 100644 --- a/spec/features/idv/steps/in_person/state_id_controller_spec.rb +++ b/spec/features/idv/steps/in_person/state_id_controller_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.describe 'state id controller enabled', js: true, allowed_extra_analytics: [:*] do +RSpec.describe 'state id controller enabled', :js do include IdvStepHelper include InPersonHelper diff --git a/spec/features/idv/steps/in_person_opt_in_ipp_spec.rb b/spec/features/idv/steps/in_person_opt_in_ipp_spec.rb index 14be95d501a..46cad186ab2 100644 --- a/spec/features/idv/steps/in_person_opt_in_ipp_spec.rb +++ b/spec/features/idv/steps/in_person_opt_in_ipp_spec.rb @@ -1,7 +1,7 @@ require 'rails_helper' require 'axe-rspec' -RSpec.describe 'In Person Proofing - Opt-in IPP ', js: true, allowed_extra_analytics: [:*] do +RSpec.describe 'In Person Proofing - Opt-in IPP ', js: true do include IdvStepHelper include SpAuthHelper include InPersonHelper diff --git a/spec/features/idv/steps/phone_step_spec.rb b/spec/features/idv/steps/phone_step_spec.rb index a9e25ad98c7..0cc49fbda6f 100644 --- a/spec/features/idv/steps/phone_step_spec.rb +++ b/spec/features/idv/steps/phone_step_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'idv phone step', :js, allowed_extra_analytics: [:*] do +RSpec.feature 'idv phone step', :js do include IdvStepHelper include IdvHelper diff --git a/spec/features/idv/steps/request_letter_step_spec.rb b/spec/features/idv/steps/request_letter_step_spec.rb index 1c144b7473e..20aed20e87d 100644 --- a/spec/features/idv/steps/request_letter_step_spec.rb +++ b/spec/features/idv/steps/request_letter_step_spec.rb @@ -1,6 +1,6 @@ require 'rails_helper' -RSpec.feature 'idv request letter step', allowed_extra_analytics: [:*] do +RSpec.feature 'idv request letter step' do include IdvStepHelper include OidcAuthHelper