Merge pull request #13406 from woocommerce/13405-order-creation-track…

…-orders-with-customers-that-are-registered-or-not [Order Creation] Track orders with customers that are registered or not
woocommerce · Jan 30, 2025 · 57b11e5 · 57b11e5
2 parents b45f1e6 + 18bbc62
commit 57b11e5
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 6 deletions.
diff --git a/...com/woocommerce/android/ui/orders/creation/customerlist/CustomerListSelectionViewModel.kt b/...com/woocommerce/android/ui/orders/creation/customerlist/CustomerListSelectionViewModel.kt
@@ -35,7 +35,10 @@ class CustomerListSelectionViewModel @Inject constructor(
     private var loadingMoreInfoAboutCustomerJob: Job? = null
 
     override fun onCustomerSelected(customerModel: WCCustomerModel) {
-        analyticsTracker.track(AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED)
+        analyticsTracker.track(
+            AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED,
+            mapOf("is_customer_registered" to (customerModel.remoteCustomerId > 0L).toString())
+        )
         when {
             customerModel.remoteCustomerId > 0L -> {
                 // this customer is registered, so we may have more info on them

diff --git a/...woocommerce/android/ui/orders/creation/customerlist/CustomerListSelectionViewModelTest.kt b/...woocommerce/android/ui/orders/creation/customerlist/CustomerListSelectionViewModelTest.kt
@@ -863,7 +863,10 @@ class CustomerListSelectionViewModelTest : BaseUnitTest() {
                     )
                 )
             )
-            verify(analyticsTrackerWrapper).track(AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED)
+            verify(analyticsTrackerWrapper).track(
+                AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED,
+                mapOf("is_customer_registered" to "true")
+            )
         }
 
     @Test
@@ -913,7 +916,10 @@ class CustomerListSelectionViewModelTest : BaseUnitTest() {
             // THEN
             assertThat(states[0].partialLoading).isFalse
             assertThat(states[1].partialLoading).isTrue
-            verify(analyticsTrackerWrapper).track(AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED)
+            verify(analyticsTrackerWrapper).track(
+                AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED,
+                mapOf("is_customer_registered" to "true")
+            )
         }
 
     @Test
@@ -966,7 +972,10 @@ class CustomerListSelectionViewModelTest : BaseUnitTest() {
                     )
                 )
             )
-            verify(analyticsTrackerWrapper).track(AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED)
+            verify(analyticsTrackerWrapper).track(
+                AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED,
+                mapOf("is_customer_registered" to "true")
+            )
         }
 
     @Test
@@ -1026,7 +1035,10 @@ class CustomerListSelectionViewModelTest : BaseUnitTest() {
                     )
                 )
             )
-            verify(analyticsTrackerWrapper).track(AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED)
+            verify(analyticsTrackerWrapper).track(
+                AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED,
+                mapOf("is_customer_registered" to "true")
+            )
         }
 
     @Test
@@ -1076,7 +1088,10 @@ class CustomerListSelectionViewModelTest : BaseUnitTest() {
                     )
                 )
             )
-            verify(analyticsTrackerWrapper).track(AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED)
+            verify(analyticsTrackerWrapper).track(
+                AnalyticsEvent.ORDER_CREATION_CUSTOMER_ADDED,
+                mapOf("is_customer_registered" to "false")
+            )
         }
 
     @Test