Track Cash on Delivery disabled CTA tapped event

Author: joshheald

WooCommerce/Classes/Analytics/WooAnalyticsEvent.swift

@@ -1172,6 +1172,20 @@ extension WooAnalyticsEvent {
                               ])
         }
 
+        /// Tracked when a In-Person Payments onboarding step's CTA is tapped by the user.
+        ///
+        /// - Parameters:
+        ///   - reason: the reason why the onboarding step was shown (effectively the name of the step.)
+        ///   - countryCode: the country code of the store.
+        ///
+        static func cardPresentOnboardingCtaTapped(reason: String, countryCode: String) -> WooAnalyticsEvent {
+            WooAnalyticsEvent(statName: .cardPresentOnboardingCtaTapped,
+                              properties: [
+                                Keys.countryCode: countryCode,
+                                Keys.reason: reason
+                              ])
+        }
+
         /// Tracked when the user taps on the "See Receipt" button to view a receipt.
         /// - Parameter countryCode: the country code of the store.
         ///

WooCommerce/Classes/Analytics/WooAnalyticsStat.swift

@@ -185,6 +185,7 @@ public enum WooAnalyticsStat: String {
     case cardPresentOnboardingLearnMoreTapped = "card_present_onboarding_learn_more_tapped"
     case cardPresentOnboardingNotCompleted = "card_present_onboarding_not_completed"
     case cardPresentOnboardingStepSkipped = "card_present_onboarding_step_skipped"
+    case cardPresentOnboardingCtaTapped = "card_present_onboarding_cta_tapped"
 
     // MARK: Payment Gateways selection
     case cardPresentPaymentGatewaySelected = "card_present_payment_gateway_selected"

WooCommerce/Classes/ViewRelated/Dashboard/Settings/In-Person Payments/Onboarding Errors/InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel.swift

@@ -65,6 +65,8 @@ final class InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel: Obser
     }
 
     func enableTapped() {
+        trackEnableTapped()
+
         guard let siteID = siteID else {
             return completion()
         }
@@ -108,18 +110,24 @@ final class InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel: Obser
 
 // MARK: - Analytics
 private extension InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModel {
-    private typealias Event = WooAnalyticsEvent.InPersonPayments
+    typealias Event = WooAnalyticsEvent.InPersonPayments
 
-    private var reason: String {
+    var reason: String {
         CardPresentPaymentOnboardingState.codPaymentGatewayNotSetUp.reasonForAnalytics ?? ""
     }
 
-    private func trackSkipTapped() {
+    func trackSkipTapped() {
         let event = Event.cardPresentOnboardingStepSkipped(reason: reason,
                                                            remindLater: false,
                                                            countryCode: cardPresentPaymentsConfiguration.countryCode)
         analytics.track(event: event)
     }
+
+    func trackEnableTapped() {
+        let event = Event.cardPresentOnboardingCtaTapped(reason: reason,
+                                                         countryCode: cardPresentPaymentsConfiguration.countryCode)
+        analytics.track(event: event)
+    }
 }
 
 private enum Localization {

WooCommerce/WooCommerceTests/ViewRelated/Dashboard/Settings/In-Person Payments/Onboarding Errors/InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModelTests.swift

@@ -134,10 +134,26 @@ final class InPersonPaymentsCashOnDeliveryPaymentGatewayNotSetUpViewModelTests:
         assertEqual(false, eventProperties[AnalyticProperties.remindLaterKey] as? Bool)
         assertEqual("US", eventProperties[AnalyticProperties.countryCodeKey] as? String)
     }
+
+    func test_enable_tapped_logs_cta_tapped_event() throws {
+        // Given
+        assertEmpty(analyticsProvider.receivedEvents)
+
+        // When
+        sut.enableTapped()
+
+        // Then
+        assertNotEmpty(analyticsProvider.receivedEvents)
+        let indexOfEvent = try XCTUnwrap(analyticsProvider.receivedEvents.firstIndex(where: { $0 == AnalyticEvents.ctaTappedEvent }))
+        let eventProperties = try XCTUnwrap(analyticsProvider.receivedProperties[indexOfEvent])
+        assertEqual(AnalyticProperties.cashOnDeliveryDisabledReason, eventProperties[AnalyticProperties.reasonKey] as? String)
+        assertEqual("US", eventProperties[AnalyticProperties.countryCodeKey] as? String)
+    }
 }
 
 private enum AnalyticEvents {
     static let skippedEvent = "card_present_onboarding_step_skipped"
+    static let ctaTappedEvent = "card_present_onboarding_cta_tapped"
 }
 
 private enum AnalyticProperties {