Do not show Winback offer if there is no active purchase

Automattic · Feb 7, 2025 · 378c098 · 378c098
1 parent 344d205
commit 378c098
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/.../profile/src/main/java/au/com/shiftyjelly/pocketcasts/profile/winback/WinbackViewModel.kt b/.../profile/src/main/java/au/com/shiftyjelly/pocketcasts/profile/winback/WinbackViewModel.kt
@@ -337,8 +337,10 @@ class WinbackViewModel @Inject constructor(
     }
 
     private fun UiState.applyWinbackResponse(response: WinbackResponse?): UiState {
+        val activePurchase = (subscriptionPlansState as? SubscriptionPlansState.Loaded)?.activePurchase
         return copy(
             winbackOfferState = response
+                ?.takeIf { activePurchase != null }
                 ?.toWinbackOffer(productsDetails)
                 ?.let(::WinbackOfferState),
         )

diff --git a/...le/src/test/kotlin/au/com/shiftyjelly/pocketcasts/profile/winback/WinbackViewModelTest.kt b/...le/src/test/kotlin/au/com/shiftyjelly/pocketcasts/profile/winback/WinbackViewModelTest.kt
@@ -348,10 +348,7 @@ class WinbackViewModelTest {
             assertNull(changedPlanState.winbackOfferState)
 
             winbackManager.addWinbackResponse(winbackResponse)
-            assertEquals(
-                "offer-token-${Subscription.PLUS_MONTHLY_PRODUCT_ID}",
-                awaitOfferState().offer.offerToken,
-            )
+            expectNoEvents()
         }
     }