[Paywall Experiment] - Tweaks for Explat (#3140)

Author: mebarbosa

automotive/src/main/java/au/com/shiftyjelly/pocketcasts/AutomotiveApplication.kt

@@ -8,6 +8,7 @@ import androidx.core.content.getSystemService
 import androidx.hilt.work.HiltWorkerFactory
 import androidx.work.Configuration
 import au.com.shiftyjelly.pocketcasts.analytics.AnalyticsTracker
+import au.com.shiftyjelly.pocketcasts.analytics.experiments.ExperimentProvider
 import au.com.shiftyjelly.pocketcasts.crashlogging.InitializeRemoteLogging
 import au.com.shiftyjelly.pocketcasts.preferences.Settings
 import au.com.shiftyjelly.pocketcasts.repositories.di.ApplicationScope
@@ -60,6 +61,8 @@ class AutomotiveApplication : Application(), Configuration.Provider {
 
     @Inject lateinit var analyticsTracker: AnalyticsTracker
 
+    @Inject lateinit var experimentProvider: ExperimentProvider
+
     @Inject @ApplicationScope
     lateinit var applicationScope: CoroutineScope
 
@@ -120,5 +123,6 @@ class AutomotiveApplication : Application(), Configuration.Provider {
 
     private fun setupAnalytics() {
         analyticsTracker.clearAllData()
+        experimentProvider.initialize()
     }
 }

modules/features/account/src/main/java/au/com/shiftyjelly/pocketcasts/account/viewmodel/OnboardingCreateAccountViewModel.kt

@@ -6,6 +6,7 @@ import androidx.lifecycle.AndroidViewModel
 import androidx.lifecycle.viewModelScope
 import au.com.shiftyjelly.pocketcasts.analytics.AnalyticsEvent
 import au.com.shiftyjelly.pocketcasts.analytics.AnalyticsTracker
+import au.com.shiftyjelly.pocketcasts.analytics.experiments.ExperimentProvider
 import au.com.shiftyjelly.pocketcasts.repositories.podcast.PodcastManager
 import au.com.shiftyjelly.pocketcasts.repositories.subscription.SubscriptionManager
 import au.com.shiftyjelly.pocketcasts.repositories.sync.LoginResult
@@ -29,6 +30,7 @@ class OnboardingCreateAccountViewModel @Inject constructor(
     private val analyticsTracker: AnalyticsTracker,
     private val subscriptionManager: SubscriptionManager,
     private val podcastManager: PodcastManager,
+    private val experimentProvider: ExperimentProvider,
     @ApplicationContext context: Context,
 ) : AndroidViewModel(context as Application), CoroutineScope {
     override val coroutineContext: CoroutineContext
@@ -82,6 +84,7 @@ class OnboardingCreateAccountViewModel @Inject constructor(
             when (result) {
                 is LoginResult.Success -> {
                     podcastManager.refreshPodcastsAfterSignIn()
+                    experimentProvider.refreshExperiments()
                     analyticsTracker.refreshMetadata()
                     onAccountCreated()
                 }

modules/services/analytics/src/main/java/au/com/shiftyjelly/pocketcasts/analytics/experiments/ExperimentProvider.kt

@@ -26,7 +26,7 @@ class ExperimentProvider @Inject constructor(
 
     fun initialize() {
         if (FeatureFlag.isEnabled(Feature.EXPLAT_EXPERIMENT)) {
-            val uuid = accountStatusInfo.getUuid() ?: UUID.randomUUID().toString().replace("-", "")
+            val uuid = accountStatusInfo.getUuid().takeIf { !it.isNullOrBlank() } ?: UUID.randomUUID().toString().replace("-", "")
 
             LogBuffer.i(TAG, "Initializing experiments with uuid: $uuid")
 

modules/services/analytics/src/test/kotlin/au/com/shiftyjelly/pocketcasts/analytics/experiments/ExperimentProviderTest.kt

@@ -70,6 +70,18 @@ class ExperimentProviderTest {
         verify(accountStatusInfo).getUuid()
     }
 
+    @Test
+    fun `should not initialize with empty uuid`() {
+        FeatureFlag.setEnabled(Feature.EXPLAT_EXPERIMENT, true)
+
+        `when`(accountStatusInfo.getUuid()).thenReturn("")
+
+        experimentProvider.initialize()
+
+        verify(repository, never()).initialize(eq(""), eq(null))
+        verify(accountStatusInfo).getUuid()
+    }
+
     @Test
     fun `getVariation should return Control when repository returns Control`() {
         FeatureFlag.setEnabled(Feature.EXPLAT_EXPERIMENT, true)

wear/src/main/kotlin/au/com/shiftyjelly/pocketcasts/wear/PocketCastsWearApplication.kt

@@ -5,6 +5,7 @@ import androidx.hilt.work.HiltWorkerFactory
 import androidx.work.Configuration
 import au.com.shiftyjelly.pocketcasts.BuildConfig
 import au.com.shiftyjelly.pocketcasts.analytics.AnalyticsTracker
+import au.com.shiftyjelly.pocketcasts.analytics.experiments.ExperimentProvider
 import au.com.shiftyjelly.pocketcasts.crashlogging.InitializeRemoteLogging
 import au.com.shiftyjelly.pocketcasts.preferences.Settings
 import au.com.shiftyjelly.pocketcasts.repositories.download.DownloadManager
@@ -59,6 +60,8 @@ class PocketCastsWearApplication : Application(), Configuration.Provider {
 
     @Inject lateinit var analyticsTracker: AnalyticsTracker
 
+    @Inject lateinit var experimentProvider: ExperimentProvider
+
     @Inject lateinit var downloadStatisticsReporter: DownloadStatisticsReporter
 
     @Inject lateinit var initializeRemoteLogging: InitializeRemoteLogging
@@ -122,6 +125,7 @@ class PocketCastsWearApplication : Application(), Configuration.Provider {
     private fun setupAnalytics() {
         analyticsTracker.clearAllData()
         analyticsTracker.refreshMetadata()
+        experimentProvider.initialize()
         downloadStatisticsReporter.setup()
     }