only include ab test data if enabled

18F · May 14, 2024 · 9b209cc · 9b209cc
1 parent 9c3e020
commit 9b209cc
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/app/services/idv/send_phone_confirmation_otp.rb b/app/services/idv/send_phone_confirmation_otp.rb
@@ -105,19 +105,23 @@ def otp_sent_response
     end
 
     def extra_analytics_attributes
-      {
+      attributes = {
         otp_delivery_preference: delivery_method,
         country_code: parsed_phone.country,
         area_code: parsed_phone.area_code,
         phone_fingerprint: Pii::Fingerprinter.fingerprint(parsed_phone.e164),
         rate_limit_exceeded: rate_limit_exceeded?,
         telephony_response: @telephony_response,
-        ab_tests: {
+      }
+      if IdentityConfig.store.ab_testing_idv_ten_digit_otp_enabled
+        attributes[:ab_tests] = {
           AbTests::IDV_TEN_DIGIT_OTP.experiment_name => {
             bucket: bucket,
           },
-        },
-      }
+        }
+      end
+
+      attributes
     end
 
     def parsed_phone