From 8de3d07f4f9353af819f40ae16d08412e6a19d1d Mon Sep 17 00:00:00 2001
From: Sarah Koop <skoop@paypal.com>
Date: Mon, 11 Dec 2023 09:34:21 -0600
Subject: [PATCH] Update app switch failure analytics

---
 .../main/java/com/braintreepayments/api/VenmoClient.java  | 8 ++------
 .../com/braintreepayments/api/VenmoClientUnitTest.java    | 3 ++-
 2 files changed, 4 insertions(+), 7 deletions(-)

diff --git a/Venmo/src/main/java/com/braintreepayments/api/VenmoClient.java b/Venmo/src/main/java/com/braintreepayments/api/VenmoClient.java
index 8dbd8c532a..93c0df4557 100644
--- a/Venmo/src/main/java/com/braintreepayments/api/VenmoClient.java
+++ b/Venmo/src/main/java/com/braintreepayments/api/VenmoClient.java
@@ -93,7 +93,8 @@ public void createPaymentAuthRequest(@NonNull final FragmentActivity activity,
                 return;
             }
             if (!deviceInspector.isVenmoAppSwitchAvailable(activity)) {
-                callbackPaymentAuthAppSwitchFailure(callback,
+                braintreeClient.sendAnalyticsEvent(VenmoAnalytics.APP_SWITCH_FAILED);
+                callbackPaymentAuthFailure(callback,
                         new VenmoPaymentAuthRequest.Failure(new AppSwitchNotAvailableException("Venmo is not installed")));
                 return;
             }
@@ -254,11 +255,6 @@ public void isReadyToPay(final Context context, final VenmoIsReadyToPayCallback
         });
     }
 
-    private void callbackPaymentAuthAppSwitchFailure(VenmoPaymentAuthRequestCallback callback, VenmoPaymentAuthRequest request) {
-        braintreeClient.sendAnalyticsEvent(VenmoAnalytics.APP_SWITCH_FAILED);
-        callback.onVenmoPaymentAuthRequest(request);
-    }
-
     private void callbackPaymentAuthFailure(VenmoPaymentAuthRequestCallback callback, VenmoPaymentAuthRequest request) {
         braintreeClient.sendAnalyticsEvent(VenmoAnalytics.TOKENIZE_FAILED);
         callback.onVenmoPaymentAuthRequest(request);
diff --git a/Venmo/src/test/java/com/braintreepayments/api/VenmoClientUnitTest.java b/Venmo/src/test/java/com/braintreepayments/api/VenmoClientUnitTest.java
index 8a8d252c04..38e5f2e3ae 100644
--- a/Venmo/src/test/java/com/braintreepayments/api/VenmoClientUnitTest.java
+++ b/Venmo/src/test/java/com/braintreepayments/api/VenmoClientUnitTest.java
@@ -216,13 +216,14 @@ public void createPaymentAuthRequest_whenVenmoNotInstalled_forwardsExceptionToLi
 
         verify(deviceInspector).isVenmoAppSwitchAvailable(same(activity));
 
+        verify(braintreeClient).sendAnalyticsEvent(VenmoAnalytics.APP_SWITCH_FAILED);
         ArgumentCaptor<VenmoPaymentAuthRequest> captor =
                 ArgumentCaptor.forClass(VenmoPaymentAuthRequest.class);
         verify(venmoPaymentAuthRequestCallback).onVenmoPaymentAuthRequest(captor.capture());
         VenmoPaymentAuthRequest paymentAuthRequest = captor.getValue();
         assertTrue(paymentAuthRequest instanceof VenmoPaymentAuthRequest.Failure);
         assertEquals("Venmo is not installed", ((VenmoPaymentAuthRequest.Failure) paymentAuthRequest).getError().getMessage());
-        verify(braintreeClient).sendAnalyticsEvent(VenmoAnalytics.APP_SWITCH_FAILED);
+        verify(braintreeClient).sendAnalyticsEvent(VenmoAnalytics.TOKENIZE_FAILED);
     }
 
     @Test