Disabled parallel mode. It's too unstable and fails builds. (#348)

Author: EranBoudjnah

Diff for: architecture/presentation-test/src/main/java/com/mitteloupe/whoami/architecture/presentation/viewmodel/BaseViewModelTest.kt

@@ -5,11 +5,11 @@ import com.mitteloupe.whoami.architecture.domain.exception.DomainException
 import com.mitteloupe.whoami.architecture.domain.usecase.UseCase
 import com.mitteloupe.whoami.architecture.presentation.notification.PresentationNotification
 import com.mitteloupe.whoami.coroutine.currentValue
-import kotlinx.coroutines.DelicateCoroutinesApi
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.ExperimentalCoroutinesApi
-import kotlinx.coroutines.newSingleThreadContext
 import kotlinx.coroutines.runBlocking
+import kotlinx.coroutines.test.TestCoroutineScheduler
+import kotlinx.coroutines.test.UnconfinedTestDispatcher
 import kotlinx.coroutines.test.resetMain
 import kotlinx.coroutines.test.setMain
 import org.junit.After
@@ -28,8 +28,10 @@ abstract class BaseViewModelTest<
     NOTIFICATION : PresentationNotification,
     VIEW_MODEL : BaseViewModel<VIEW_STATE, NOTIFICATION>
     > {
-    @OptIn(DelicateCoroutinesApi::class, ExperimentalCoroutinesApi::class)
-    private val mainThreadSurrogate = newSingleThreadContext("UI thread")
+    private val testScheduler = TestCoroutineScheduler()
+
+    @OptIn(ExperimentalCoroutinesApi::class)
+    private val testDispatcher = UnconfinedTestDispatcher(testScheduler)
 
     protected lateinit var classUnderTest: VIEW_MODEL
 
@@ -41,7 +43,7 @@ abstract class BaseViewModelTest<
     @OptIn(ExperimentalCoroutinesApi::class)
     @Before
     fun coroutineSetUp() {
-        Dispatchers.setMain(mainThreadSurrogate)
+        Dispatchers.setMain(testDispatcher)
     }
 
     @OptIn(ExperimentalCoroutinesApi::class)

Diff for: architecture/ui/src/main/java/com/mitteloupe/whoami/architecture/ui/view/BaseComposeHolder.kt

@@ -17,7 +17,6 @@ abstract class BaseComposeHolder<VIEW_STATE : Any, NOTIFICATION : PresentationNo
     private val navigationMapper: NavigationEventDestinationMapper<PresentationNavigationEvent>,
     private val notificationMapper: NotificationUiMapper
 ) {
-
     @Composable
     fun ViewModelObserver(navController: NavController) {
         val navigation = viewModel.navigationEvent.collectAsState(

Diff for: gradle.properties

@@ -10,7 +10,7 @@ org.gradle.jvmargs=-Xmx2048m -Dfile.encoding=UTF-8
 # When configured, Gradle will run in incubating parallel mode.
 # This option should only be used with decoupled projects. More details, visit
 # http://www.gradle.org/docs/current/userguide/multi_project_builds.html#sec:decoupled_projects
-org.gradle.parallel=true
+org.gradle.parallel=false
 org.gradle.caching=true
 # AndroidX package structure to make it clearer which packages are bundled with the
 # Android operating system, and which are packaged with your app's APK

Diff for: home/presentation/src/test/java/com/mitteloupe/whoami/home/presentation/viewmodel/HomeViewModelTest.kt

@@ -18,7 +18,6 @@ import com.mitteloupe.whoami.home.presentation.navigation.HomePresentationNaviga
 import kotlinx.coroutines.CoroutineStart.UNDISPATCHED
 import kotlinx.coroutines.async
 import kotlinx.coroutines.flow.first
-import kotlinx.coroutines.runBlocking
 import kotlinx.coroutines.test.runTest
 import org.junit.Assert.assertEquals
 import org.junit.Before
@@ -64,7 +63,7 @@ class HomeViewModelTest :
     }
 
     @Test
-    fun `Given disconnected when onEnter then presents disconnected state`() = runBlocking {
+    fun `Given disconnected when onEnter then presents disconnected state`() = runTest {
         // Given
         val givenConnectionState = Disconnected
         givenSuccessfulUseCaseExecution(