Remove tests related to My Site Dashboard default tab experiment.

Author: zwarm

WordPress/src/test/java/org/wordpress/android/ui/accounts/LoginEpilogueViewModelTest.kt

@@ -1,7 +1,5 @@
 package org.wordpress.android.ui.accounts
 
-import com.nhaarman.mockitokotlin2.atLeastOnce
-import com.nhaarman.mockitokotlin2.verify
 import com.nhaarman.mockitokotlin2.whenever
 import org.assertj.core.api.Assertions.assertThat
 import org.junit.Before
@@ -10,7 +8,6 @@ import org.junit.Test
 import org.mockito.Mock
 import org.wordpress.android.BaseUnitTest
 import org.wordpress.android.fluxc.store.SiteStore
-import org.wordpress.android.ui.mysite.tabs.MySiteDefaultTabExperiment
 import org.wordpress.android.ui.prefs.AppPrefsWrapper
 import org.wordpress.android.util.BuildConfigWrapper
 
@@ -20,11 +17,10 @@ class LoginEpilogueViewModelTest : BaseUnitTest() {
     @Mock lateinit var appPrefsWrapper: AppPrefsWrapper
     @Mock lateinit var buildConfigWrapper: BuildConfigWrapper
     @Mock lateinit var siteStore: SiteStore
-    @Mock lateinit var mySiteDefaultTabExperiment: MySiteDefaultTabExperiment
 
     @Before
     fun setUp() {
-        viewModel = LoginEpilogueViewModel(appPrefsWrapper, buildConfigWrapper, siteStore, mySiteDefaultTabExperiment)
+        viewModel = LoginEpilogueViewModel(appPrefsWrapper, buildConfigWrapper, siteStore)
     }
 
     @Test
@@ -260,13 +256,6 @@ class LoginEpilogueViewModelTest : BaseUnitTest() {
         assertThat(navigationEvents.last()).isInstanceOf(LoginNavigationEvents.ShowNoJetpackSites::class.java)
     }
 
-    @Test
-    fun `given my site default tab experiment, when requested, then check and set for variant is executed `() {
-        viewModel.checkAndSetVariantForMySiteDefaultTabExperiment()
-
-        verify(mySiteDefaultTabExperiment, atLeastOnce()).checkAndSetVariantIfNeeded()
-    }
-
     private data class Observers(val navigationEvents: List<LoginNavigationEvents>)
 
     private fun initObservers(): Observers {

WordPress/src/test/java/org/wordpress/android/ui/mysite/cards/CardsBuilderTest.kt

@@ -32,7 +32,6 @@ import org.wordpress.android.ui.mysite.cards.quickactions.QuickActionsCardBuilde
 import org.wordpress.android.ui.mysite.cards.quicklinksribbon.QuickLinkRibbonBuilder
 import org.wordpress.android.ui.mysite.cards.quickstart.QuickStartCardBuilder
 import org.wordpress.android.ui.mysite.cards.quickstart.QuickStartRepository.QuickStartCategory
-import org.wordpress.android.ui.mysite.tabs.MySiteDefaultTabExperiment
 import org.wordpress.android.ui.quickstart.QuickStartTaskDetails
 import org.wordpress.android.ui.utils.UiString.UiStringText
 import org.wordpress.android.util.BuildConfigWrapper
@@ -48,7 +47,6 @@ class CardsBuilderTest {
     @Mock lateinit var dashboardCardsBuilder: DashboardCardsBuilder
     @Mock lateinit var quickLinkRibbonBuilder: QuickLinkRibbonBuilder
     @Mock lateinit var site: SiteModel
-    @Mock lateinit var mySiteDefaultTabExperiment: MySiteDefaultTabExperiment
 
     private lateinit var cardsBuilder: CardsBuilder
     private val quickStartCategory: QuickStartCategory
@@ -98,35 +96,6 @@ class CardsBuilderTest {
         assertThat(cards.findQuickActionsCard()).isNull()
     }
 
-    @Test
-    fun `given tabs enabled + experiment not running, when cards built, then quick actions card is built`() {
-        val cards = buildCards(isMySiteTabsEnabled = true, isDefaultTabExperimentRunning = false)
-
-        assertThat(cards.findQuickActionsCard()).isNotNull
-    }
-
-    @Test
-    fun `given tabs enabled + experiment running + variant not assigned, when cards built, then quick actions built`() {
-        val cards = buildCards(
-                isMySiteTabsEnabled = true,
-                isDefaultTabExperimentRunning = true,
-                isDefaultTabVariantAssigned = false
-        )
-
-        assertThat(cards.findQuickActionsCard()).isNotNull
-    }
-
-    @Test
-    fun `given tabs enabled + experiment running + variant assigned, when cards built, then quick actions not built`() {
-        val cards = buildCards(
-                isMySiteTabsEnabled = true,
-                isDefaultTabExperimentRunning = true,
-                isDefaultTabVariantAssigned = true
-        )
-
-        assertThat(cards.findQuickActionsCard()).isNull()
-    }
-
     /* QUICK START CARD */
 
     @Test
@@ -194,14 +163,10 @@ class CardsBuilderTest {
         isDomainCreditAvailable: Boolean = false,
         isQuickStartInProgress: Boolean = false,
         isQuickStartDynamicCardEnabled: Boolean = false,
-        isMySiteTabsEnabled: Boolean = false,
-        isDefaultTabExperimentRunning: Boolean = false,
-        isDefaultTabVariantAssigned: Boolean = false
+        isMySiteTabsEnabled: Boolean = false
     ): List<MySiteCardAndItem> {
         whenever(buildConfigWrapper.isQuickActionEnabled).thenReturn(isQuickActionEnabled)
         whenever(quickStartDynamicCardsFeatureConfig.isEnabled()).thenReturn(isQuickStartDynamicCardEnabled)
-        whenever(mySiteDefaultTabExperiment.isExperimentRunning()).thenReturn(isDefaultTabExperimentRunning)
-        whenever(mySiteDefaultTabExperiment.isVariantAssigned()).thenReturn(isDefaultTabVariantAssigned)
         return cardsBuilder.build(
                 quickActionsCardBuilderParams = QuickActionsCardBuilderParams(
                         siteModel = site,
@@ -270,8 +235,7 @@ class CardsBuilderTest {
                 quickActionsCardBuilder,
                 quickStartCardBuilder,
                 quickLinkRibbonBuilder,
-                dashboardCardsBuilder,
-                mySiteDefaultTabExperiment
+                dashboardCardsBuilder
         )
     }
 

WordPress/src/test/java/org/wordpress/android/viewmodel/main/WPMainActivityViewModelTest.kt

@@ -4,7 +4,6 @@ import androidx.lifecycle.MutableLiveData
 import androidx.lifecycle.Observer
 import com.nhaarman.mockitokotlin2.any
 import com.nhaarman.mockitokotlin2.anyOrNull
-import com.nhaarman.mockitokotlin2.atLeastOnce
 import com.nhaarman.mockitokotlin2.eq
 import com.nhaarman.mockitokotlin2.mock
 import com.nhaarman.mockitokotlin2.never
@@ -48,7 +47,6 @@ import org.wordpress.android.ui.main.MainActionListItem.CreateAction
 import org.wordpress.android.ui.main.MainFabUiState
 import org.wordpress.android.ui.mysite.SelectedSiteRepository
 import org.wordpress.android.ui.mysite.cards.quickstart.QuickStartRepository
-import org.wordpress.android.ui.mysite.tabs.MySiteDefaultTabExperiment
 import org.wordpress.android.ui.prefs.AppPrefsWrapper
 import org.wordpress.android.ui.quickstart.QuickStartType
 import org.wordpress.android.ui.whatsnew.FeatureAnnouncement
@@ -79,7 +77,6 @@ class WPMainActivityViewModelTest : BaseUnitTest() {
     @Mock lateinit var selectedSiteRepository: SelectedSiteRepository
     @Mock lateinit var accountStore: AccountStore
     @Mock lateinit var siteStore: SiteStore
-    @Mock lateinit var mySiteDefaultTabExperiment: MySiteDefaultTabExperiment
     @Mock lateinit var bloggingPromptsFeatureConfig: BloggingPromptsFeatureConfig
     @Mock lateinit var bloggingPromptsStore: BloggingPromptsStore
     @Mock lateinit var quickStartType: QuickStartType
@@ -142,7 +139,6 @@ class WPMainActivityViewModelTest : BaseUnitTest() {
                 selectedSiteRepository,
                 accountStore,
                 siteStore,
-                mySiteDefaultTabExperiment,
                 bloggingPromptsFeatureConfig,
                 bloggingPromptsStore,
                 NoDelayCoroutineDispatcher()
@@ -801,15 +797,6 @@ class WPMainActivityViewModelTest : BaseUnitTest() {
         assertThat(viewModel.firstSite).isEqualTo(null)
     }
 
-    @Test
-    fun `given my site default tab experiment, when requested, then check and set for variant is executed `() {
-        startViewModelWithDefaultParameters()
-
-        viewModel.checkAndSetVariantForMySiteDefaultTabExperiment()
-
-        verify(mySiteDefaultTabExperiment, atLeastOnce()).checkAndSetVariantIfNeeded()
-    }
-
     @Test
     fun `Should track analytics event when onHelpPrompActionClicked is called`() {
         whenever(bloggingPromptsFeatureConfig.isEnabled()).thenReturn(true)