Add error types to reportPurchaseFailureStore (#4415)

Task/Issue URL: https://app.asana.com/0/488551667048375/1207045790899643/f ### Description Add error types to `reportPurchaseFailureStore` pixel ### Steps to test this PR Code review
duckduckgo · Apr 10, 2024 · 53ff7f2 · 53ff7f2
1 parent ca825b7
commit 53ff7f2
Show file tree

Hide file tree

Showing 4 changed files with 31 additions and 11 deletions.
diff --git a/...ions-impl/src/main/java/com/duckduckgo/subscriptions/impl/billing/BillingClientAdapter.kt b/...ions-impl/src/main/java/com/duckduckgo/subscriptions/impl/billing/BillingClientAdapter.kt
@@ -72,7 +72,7 @@ sealed class PurchasesUpdateResult {
 
     data object PurchaseAbsent : PurchasesUpdateResult()
     data object UserCancelled : PurchasesUpdateResult()
-    data object Failure : PurchasesUpdateResult()
+    data class Failure(val errorType: String) : PurchasesUpdateResult()
 }
 
 enum class BillingError {

diff --git a/...ptions-impl/src/main/java/com/duckduckgo/subscriptions/impl/billing/PlayBillingManager.kt b/...ptions-impl/src/main/java/com/duckduckgo/subscriptions/impl/billing/PlayBillingManager.kt
@@ -219,8 +219,8 @@ class RealPlayBillingManager @Inject constructor(
                     // Handle an error caused by a user cancelling the purchase flow.
                 }
 
-                PurchasesUpdateResult.Failure -> {
-                    pixelSender.reportPurchaseFailureStore()
+                is PurchasesUpdateResult.Failure -> {
+                    pixelSender.reportPurchaseFailureStore(result.errorType)
                     _purchaseState.emit(Canceled)
                 }
             }

diff --git a/...-impl/src/main/java/com/duckduckgo/subscriptions/impl/billing/RealBillingClientAdapter.kt b/...-impl/src/main/java/com/duckduckgo/subscriptions/impl/billing/RealBillingClientAdapter.kt
@@ -175,8 +175,27 @@ class RealBillingClientAdapter @Inject constructor(
     private fun mapToPurchasesUpdateResult(
         billingResult: BillingResult,
         purchases: List<Purchase>?,
-    ): PurchasesUpdateResult =
-        when (billingResult.responseCode) {
+    ): PurchasesUpdateResult {
+        fun Int.asString(): String {
+            return when (this) {
+                BillingResponseCode.SERVICE_TIMEOUT -> "SERVICE_TIMEOUT"
+                BillingResponseCode.FEATURE_NOT_SUPPORTED -> "FEATURE_NOT_SUPPORTED"
+                BillingResponseCode.SERVICE_DISCONNECTED -> "SERVICE_DISCONNECTED"
+                BillingResponseCode.OK -> "OK"
+                BillingResponseCode.USER_CANCELED -> "USER_CANCELED"
+                BillingResponseCode.SERVICE_UNAVAILABLE -> "SERVICE_UNAVAILABLE"
+                BillingResponseCode.BILLING_UNAVAILABLE -> "BILLING_UNAVAILABLE"
+                BillingResponseCode.ITEM_UNAVAILABLE -> "ITEM_UNAVAILABLE"
+                BillingResponseCode.DEVELOPER_ERROR -> "DEVELOPER_ERROR"
+                BillingResponseCode.ERROR -> "ERROR"
+                BillingResponseCode.ITEM_ALREADY_OWNED -> "ITEM_ALREADY_OWNED"
+                BillingResponseCode.ITEM_NOT_OWNED -> "ITEM_NOT_OWNED"
+                BillingResponseCode.NETWORK_ERROR -> "NETWORK_ERROR"
+                else -> "UNKNOWN"
+            }
+        }
+
+        return when (billingResult.responseCode) {
             BillingResponseCode.OK -> {
                 val purchase = purchases?.lastOrNull { it.purchaseState == PurchaseState.PURCHASED }
                 if (purchase != null) {
@@ -190,8 +209,9 @@ class RealBillingClientAdapter @Inject constructor(
             }
 
             BillingResponseCode.USER_CANCELED -> PurchasesUpdateResult.UserCancelled
-            else -> PurchasesUpdateResult.Failure
+            else -> PurchasesUpdateResult.Failure(billingResult.responseCode.asString())
         }
+    }
 }
 
 private fun Int.toBillingError(): BillingError = when (this) {

diff --git a/...ns-impl/src/main/java/com/duckduckgo/subscriptions/impl/pixels/SubscriptionPixelSender.kt b/...ns-impl/src/main/java/com/duckduckgo/subscriptions/impl/pixels/SubscriptionPixelSender.kt
@@ -60,7 +60,7 @@ interface SubscriptionPixelSender {
     fun reportOfferScreenShown()
     fun reportOfferSubscribeClick()
     fun reportPurchaseFailureOther()
-    fun reportPurchaseFailureStore()
+    fun reportPurchaseFailureStore(errorType: String)
     fun reportPurchaseFailureBackend()
     fun reportPurchaseFailureAccountCreation()
     fun reportPurchaseSuccess()
@@ -109,8 +109,8 @@ class SubscriptionPixelSenderImpl @Inject constructor(
     override fun reportPurchaseFailureOther() =
         fire(PURCHASE_FAILURE_OTHER)
 
-    override fun reportPurchaseFailureStore() =
-        fire(PURCHASE_FAILURE_STORE)
+    override fun reportPurchaseFailureStore(errorType: String) =
+        fire(PURCHASE_FAILURE_STORE, mapOf("errorType" to errorType))
 
     override fun reportPurchaseFailureBackend() =
         fire(PURCHASE_FAILURE_BACKEND)
@@ -199,9 +199,9 @@ class SubscriptionPixelSenderImpl @Inject constructor(
     override fun reportPrivacyProRedirect() =
         fire(SUBSCRIPTION_PRIVACY_PRO_REDIRECT)
 
-    private fun fire(pixel: SubscriptionPixel) {
+    private fun fire(pixel: SubscriptionPixel, params: Map<String, String> = emptyMap()) {
         pixel.getPixelNames().forEach { (pixelType, pixelName) ->
-            pixelSender.fire(pixelName = pixelName, type = pixelType)
+            pixelSender.fire(pixelName = pixelName, type = pixelType, parameters = params)
         }
     }
 }