Add connection-disconnection events to POS side. add tests

woocommerce · Feb 5, 2025 · a4152d0 · a4152d0
1 parent f4509cc
commit a4152d0
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 0 deletions.
diff --git a/WooCommerce/Classes/Analytics/WooAnalyticsStat.swift b/WooCommerce/Classes/Analytics/WooAnalyticsStat.swift
@@ -1285,6 +1285,7 @@ enum WooAnalyticsStat: String {
     case pointOfSaleEmailReceiptSendTapped = "email_receipt_send_tapped"
     case pointOfSalePaymentsOnboardingShown = "payments_onboarding_shown"
     case pointOfSalePaymentsOnboardingDismissed = "payments_onboarding_dismissed"
+    case pointOfSaleCardReaderConnectionTapped = "card_reader_connection_tapped"
 
     // MARK: Custom Fields events
     case productDetailCustomFieldsTapped = "product_detail_custom_fields_tapped"

diff --git a/WooCommerce/Classes/POS/Models/PointOfSaleAggregateModel.swift b/WooCommerce/Classes/POS/Models/PointOfSaleAggregateModel.swift
@@ -158,12 +158,14 @@ extension PointOfSaleAggregateModel {
     }
 
     func connectCardReader() {
+        analytics.track(.pointOfSaleCardReaderConnectionTapped)
         Task { @MainActor in
             _ = try await cardPresentPaymentService.connectReader(using: .bluetooth)
         }
     }
 
     func disconnectCardReader() {
+        analytics.track(.cardReaderDisconnectTapped)
         Task { @MainActor in
             await cardPresentPaymentService.disconnectReader()
         }

diff --git a/WooCommerce/WooCommerceTests/POS/Models/PointOfSaleAggregateModelTests.swift b/WooCommerce/WooCommerceTests/POS/Models/PointOfSaleAggregateModelTests.swift
@@ -561,6 +561,22 @@ struct PointOfSaleAggregateModelTests {
             // Then
             #expect(analyticsProvider.receivedEvents.first(where: { $0 == "payments_onboarding_shown" }) != nil)
         }
+
+        @Test func connectCardReader_tracks_event() {
+            // Given/When
+            sut.connectCardReader()
+
+            // Then
+            #expect(analyticsProvider.receivedEvents.first(where: { $0 == "card_reader_connection_tapped" }) != nil)
+        }
+
+        @Test func disconnectCardReader_tracks_event() {
+            // Given/When
+            sut.disconnectCardReader()
+
+            // Then
+            #expect(analyticsProvider.receivedEvents.first(where: { $0 == "card_reader_disconnect_tapped" }) != nil)
+        }
     }
 }