Test: Reorder and add experimental coroutines api on top of test suites

This is done so that the '@ExperimentalCoroutinesApi' annotation is only
presented itself once per test suite and avoid having to think and add
this boilerplate configuration multiple times, per test configuration or
function, whenever and if it is needed.

Author: ParaskP7

WordPress/src/test/java/org/wordpress/android/ui/jetpack/scan/ScanViewModelTest.kt

@@ -64,9 +64,9 @@ private const val ON_ENTER_SERVER_CREDS_MESSAGE_CLICKED_PARAM_POSITION = 7
 private const val TEST_SITE_ID = 1L
 private const val SERVER_CREDS_LINK = "${Constants.URL_JETPACK_SETTINGS}/$TEST_SITE_ID}"
 
-@ExperimentalCoroutinesApi
-@InternalCoroutinesApi
 @Suppress("LargeClass")
+@InternalCoroutinesApi
+@ExperimentalCoroutinesApi
 class ScanViewModelTest : BaseUnitTest() {
     @Rule
     @JvmField val coroutineScope = MainCoroutineScopeRule()

WordPress/src/test/java/org/wordpress/android/ui/jetpack/scan/usecases/FetchFixThreatsStatusUseCaseTest.kt

@@ -30,8 +30,8 @@ import org.wordpress.android.ui.jetpack.scan.usecases.FetchFixThreatsStatusUseCa
 import org.wordpress.android.ui.jetpack.scan.usecases.FetchFixThreatsStatusUseCase.FetchFixThreatsState.NotStarted
 import org.wordpress.android.util.NetworkUtilsWrapper
 
-@ExperimentalCoroutinesApi
 @InternalCoroutinesApi
+@ExperimentalCoroutinesApi
 class FetchFixThreatsStatusUseCaseTest : BaseUnitTest() {
     @Rule
     @JvmField val coroutineScope = MainCoroutineScopeRule()

WordPress/src/test/java/org/wordpress/android/ui/jetpack/scan/usecases/FetchScanStateUseCaseTest.kt

@@ -27,8 +27,8 @@ import org.wordpress.android.ui.jetpack.scan.usecases.FetchScanStateUseCase.Fetc
 import org.wordpress.android.ui.jetpack.scan.usecases.FetchScanStateUseCase.FetchScanState.Success
 import org.wordpress.android.util.NetworkUtilsWrapper
 
-@ExperimentalCoroutinesApi
 @InternalCoroutinesApi
+@ExperimentalCoroutinesApi
 class FetchScanStateUseCaseTest : BaseUnitTest() {
     private lateinit var useCase: FetchScanStateUseCase
     @Mock private lateinit var site: SiteModel

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

@@ -150,8 +150,8 @@ import java.util.Date
 
 private const val DYNAMIC_CARDS_BUILDER_MORE_CLICK_PARAM_POSITION = 3
 
-@ExperimentalCoroutinesApi
 @Suppress("LargeClass")
+@ExperimentalCoroutinesApi
 @RunWith(MockitoJUnitRunner::class)
 class MySiteViewModelTest : BaseUnitTest() {
     @Mock lateinit var siteItemsBuilder: SiteItemsBuilder

WordPress/src/test/java/org/wordpress/android/ui/qrcodeauth/QRCodeAuthViewModelTest.kt

@@ -62,9 +62,9 @@ const val SCANNED_VALUE =
 const val VALID_EXPIRED_MESSAGE = "qr code data expired"
 const val INVALID_EXPIRED_MESSAGE = "invalid qr code data expired"
 
+@Suppress("LargeClass")
 @InternalCoroutinesApi
 @ExperimentalCoroutinesApi
-@Suppress("LargeClass")
 class QRCodeAuthViewModelTest : BaseUnitTest() {
     private lateinit var viewModel: QRCodeAuthViewModel
     @Mock lateinit var store: QRCodeAuthStore

WordPress/src/test/java/org/wordpress/android/ui/reader/usecases/ReaderFetchRelatedPostsUseCaseTest.kt

@@ -23,12 +23,12 @@ import org.wordpress.android.ui.reader.usecases.ReaderFetchRelatedPostsUseCase.F
 import org.wordpress.android.util.NetworkUtilsWrapper
 
 @InternalCoroutinesApi
+@ExperimentalCoroutinesApi
 @RunWith(MockitoJUnitRunner::class)
 class ReaderFetchRelatedPostsUseCaseTest {
     @Rule
     @JvmField val rule = InstantTaskExecutorRule()
 
-    @ExperimentalCoroutinesApi
     @Rule
     @JvmField val coroutineScope = MainCoroutineScopeRule()
 

WordPress/src/test/java/org/wordpress/android/ui/stories/StoriesIntroViewModelTest.kt

@@ -20,6 +20,7 @@ import org.wordpress.android.util.NoDelayCoroutineDispatcher
 import org.wordpress.android.util.analytics.AnalyticsTrackerWrapper
 
 @InternalCoroutinesApi
+@ExperimentalCoroutinesApi
 class StoriesIntroViewModelTest : BaseUnitTest() {
     private lateinit var viewModel: StoriesIntroViewModel
     @Mock lateinit var onDialogClosedObserver: Observer<Unit>
@@ -28,7 +29,6 @@ class StoriesIntroViewModelTest : BaseUnitTest() {
     @Mock lateinit var analyticsTrackerWrapper: AnalyticsTrackerWrapper
     @Mock private lateinit var appPrefsWrapper: AppPrefsWrapper
 
-    @ExperimentalCoroutinesApi
     @Before
     fun setUp() = runBlockingTest {
         viewModel = StoriesIntroViewModel(

WordPress/src/test/java/org/wordpress/android/util/SnackbarSequencerConcurrentTest.kt

@@ -28,8 +28,8 @@ import org.wordpress.android.widgets.WPSnackbarWrapper
 private const val TEST_MESSAGE_TEMPLATE = "This is test message number "
 private const val SNACKBAR_DURATION_MS = 500L
 
-@ExperimentalCoroutinesApi
 @InternalCoroutinesApi
+@ExperimentalCoroutinesApi
 @RunWith(MockitoJUnitRunner::class)
 class SnackbarSequencerConcurrentTest {
     @get:Rule val rule = InstantTaskExecutorRule()

WordPress/src/test/java/org/wordpress/android/viewmodel/mlp/ModalLayoutPickerViewModelTest.kt

@@ -42,13 +42,13 @@ import org.wordpress.android.util.SiteUtils.GB_EDITOR_NAME
 import org.wordpress.android.viewmodel.mlp.ModalLayoutPickerViewModel.PageRequest.Blank
 import org.wordpress.android.viewmodel.mlp.ModalLayoutPickerViewModel.PageRequest.Create
 
-@RunWith(MockitoJUnitRunner::class)
 @InternalCoroutinesApi
+@ExperimentalCoroutinesApi
+@RunWith(MockitoJUnitRunner::class)
 class ModalLayoutPickerViewModelTest {
     @Rule
     @JvmField val rule = InstantTaskExecutorRule()
 
-    @ExperimentalCoroutinesApi
     @Rule
     @JvmField val coroutineScope = MainCoroutineScopeRule()
 
@@ -104,7 +104,6 @@ class ModalLayoutPickerViewModelTest {
         )
     }
 
-    @ExperimentalCoroutinesApi
     private fun <T> mockFetchingSelectedSite(
         isError: Boolean = false,
         isSiteUnavailable: Boolean = false,
@@ -144,67 +143,58 @@ class ModalLayoutPickerViewModelTest {
         }
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the user scroll beyond a threshold the title becomes visible`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         viewModel.onAppBarOffsetChanged(9, 10)
         assertThat(requireNotNull(viewModel.uiState.value as Content).isHeaderVisible).isEqualTo(true)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the user scroll bellow a threshold the title remains hidden`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         viewModel.onAppBarOffsetChanged(11, 10)
         assertThat(requireNotNull(viewModel.uiState.value as Content).isHeaderVisible).isEqualTo(false)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when modal layout picker starts the categories are loaded`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(requireNotNull(viewModel.uiState.value as Content).categories.size).isGreaterThan(0)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when modal layout picker starts the layouts are loaded`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(requireNotNull(viewModel.uiState.value as Content).layoutCategories.size).isGreaterThan(0)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when modal layout picker starts fetch errors are handled`() = mockFetchingSelectedSite(true) {
         viewModel.createPageFlowTriggered()
         assertThat(viewModel.uiState.value is Error).isEqualTo(true)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when modal layout picker starts and the site is unavailable errors are handled`() =
             mockFetchingSelectedSite(isSiteUnavailable = true) {
                 viewModel.createPageFlowTriggered()
                 assertThat(viewModel.uiState.value is Error).isEqualTo(true)
             }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `modal layout picker is shown when triggered`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(viewModel.isModalLayoutPickerShowing.value!!.peekContent()).isEqualTo(true)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `modal layout picker is dismissed when the user hits the back button`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         viewModel.dismiss()
         assertThat(viewModel.isModalLayoutPickerShowing.value!!.peekContent()).isEqualTo(false)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when no layout is selected and the create page is triggered the blank page creation flow starts`() =
             mockFetchingSelectedSite {
@@ -215,7 +205,6 @@ class ModalLayoutPickerViewModelTest {
                 assertThat(captor.value).isEqualTo(Blank)
             }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when a layout is selected and the create page is triggered the page creation flow starts with a template`() =
             mockFetchingSelectedSite {
@@ -228,14 +217,12 @@ class ModalLayoutPickerViewModelTest {
                 assertThat(captor.value.template).isEqualTo("about")
             }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when modal layout picker starts no layout is selected`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(requireNotNull(viewModel.uiState.value as Content).selectedLayoutSlug).isNull()
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the user taps on a layout the layout is selected if the thumbnail has loaded`() =
             mockFetchingSelectedSite {
@@ -246,7 +233,6 @@ class ModalLayoutPickerViewModelTest {
                         .isEqualTo("about-1")
             }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the user taps on a layout the layout is selected if the thumbnail has not loaded`() =
             mockFetchingSelectedSite {
@@ -256,7 +242,6 @@ class ModalLayoutPickerViewModelTest {
                         .isNotEqualTo("about-1")
             }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the user taps on a selected layout the layout is deselected`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
@@ -265,7 +250,6 @@ class ModalLayoutPickerViewModelTest {
         assertThat(requireNotNull(viewModel.uiState.value as Content).selectedLayoutSlug).isNull()
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the modal layout picker is dismissed the layout is deselected`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
@@ -274,14 +258,12 @@ class ModalLayoutPickerViewModelTest {
         assertThat(requireNotNull(viewModel.uiState.value as Content).selectedLayoutSlug).isNull()
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when modal layout picker starts no category is selected`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(requireNotNull(viewModel.uiState.value as Content).selectedCategoriesSlugs).isEmpty()
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the user taps on a category the category is selected`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
@@ -290,7 +272,6 @@ class ModalLayoutPickerViewModelTest {
                 .contains("about")
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the user taps on a selected category the category is deselected`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
@@ -300,7 +281,6 @@ class ModalLayoutPickerViewModelTest {
                 .doesNotContain("about")
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when the modal layout picker is dismissed the category is deselected`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
@@ -309,15 +289,13 @@ class ModalLayoutPickerViewModelTest {
         assertThat(requireNotNull(viewModel.uiState.value as Content).selectedCategoriesSlugs).isEmpty()
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when no layout is selected the create blank page button is visible`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(requireNotNull(viewModel.uiState.value as Content).buttonsUiState.createBlankPageVisible)
                 .isEqualTo(true)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when a layout is selected the create blank page button is not visible`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
@@ -327,15 +305,13 @@ class ModalLayoutPickerViewModelTest {
                 .isEqualTo(false)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when no layout is selected the create page button is not visible`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(requireNotNull(viewModel.uiState.value as Content).buttonsUiState.createPageVisible)
                 .isEqualTo(false)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when a layout is selected the create page button is visible`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
@@ -345,15 +321,13 @@ class ModalLayoutPickerViewModelTest {
                 .isEqualTo(true)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when no layout is selected the preview button is not visible`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()
         assertThat(requireNotNull(viewModel.uiState.value as Content).buttonsUiState.previewVisible)
                 .isEqualTo(false)
     }
 
-    @ExperimentalCoroutinesApi
     @Test
     fun `when a layout is selected the preview button is visible`() = mockFetchingSelectedSite {
         viewModel.createPageFlowTriggered()