[#102] Update isNetworkConnected to be StateFlow and add test

app/src/main/java/co/nimblehq/compose/crypto/CryptoAppState.kt

@@ -1,6 +1,7 @@
 package co.nimblehq.compose.crypto
 
 import androidx.compose.runtime.Composable
+import androidx.compose.runtime.Stable
 import androidx.compose.runtime.remember
 import co.nimblehq.compose.crypto.domain.usecase.IsNetworkConnectedUseCase
 import co.nimblehq.compose.crypto.util.DispatchersProvider
@@ -15,17 +16,16 @@ fun rememberCryptoAppState(
     CryptoAppState(isNetworkConnectedUseCase, dispatchersProvider)
 }
 
+@Stable
 class CryptoAppState(
     isNetworkConnectedUseCase: IsNetworkConnectedUseCase,
     dispatchersProvider: DispatchersProvider,
 ) {
-    private val _isNetworkConnected = MutableSharedFlow<Boolean?>()
-    val isNetworkConnected: SharedFlow<Boolean?>
-        get() = _isNetworkConnected
+    private val _isNetworkConnected = MutableStateFlow<Boolean?>(null)
+    val isNetworkConnected = _isNetworkConnected.asStateFlow()
 
     private val _networkError = MutableSharedFlow<Throwable?>()
-    val networkError: SharedFlow<Throwable?>
-        get() = _networkError
+    val networkError = _networkError.asSharedFlow()
 
     init {
         isNetworkConnectedUseCase()

app/src/main/java/co/nimblehq/compose/crypto/ui/navigation/ComposeCryptoApp.kt

@@ -2,6 +2,9 @@ package co.nimblehq.compose.crypto.ui.navigation
 
 import android.widget.Toast
 import androidx.compose.runtime.Composable
+import androidx.compose.runtime.LaunchedEffect
+import androidx.compose.runtime.collectAsState
+import androidx.compose.runtime.getValue
 import androidx.compose.ui.platform.LocalContext
 import androidx.navigation.NavHostController
 import androidx.navigation.compose.rememberNavController
@@ -15,7 +18,9 @@ fun ComposeCryptoApp(
 ) {
     val context = LocalContext.current
 
-    cryptoAppState.isNetworkConnected.collectAsEffect { isNetworkConnected ->
+    val isNetworkConnected by cryptoAppState.isNetworkConnected.collectAsState()
+
+    LaunchedEffect(isNetworkConnected) {
         if (isNetworkConnected == false) {
             val destination = AppDestination.NoNetwork
 
@@ -30,7 +35,5 @@ fun ComposeCryptoApp(
         Toast.makeText(context, error?.message, Toast.LENGTH_SHORT).show()
     }
 
-    AppNavigation(
-        navController = navController
-    )
+    AppNavigation(navController = navController)
 }

app/src/test/java/co/nimblehq/compose/crypto/ui/screens/ComposeCryptoAppTest.kt

@@ -0,0 +1,62 @@
+package co.nimblehq.compose.crypto.ui.screens
+
+import androidx.activity.compose.setContent
+import androidx.compose.ui.test.assertIsDisplayed
+import androidx.compose.ui.test.junit4.createAndroidComposeRule
+import androidx.compose.ui.test.onNodeWithText
+import co.nimblehq.compose.crypto.R
+import co.nimblehq.compose.crypto.domain.usecase.IsNetworkConnectedUseCase
+import co.nimblehq.compose.crypto.rememberCryptoAppState
+import co.nimblehq.compose.crypto.ui.navigation.ComposeCryptoApp
+import io.mockk.every
+import io.mockk.mockk
+import kotlinx.coroutines.ExperimentalCoroutinesApi
+import kotlinx.coroutines.flow.flowOf
+import org.junit.Before
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+import org.robolectric.RobolectricTestRunner
+
+@OptIn(ExperimentalCoroutinesApi::class)
+@RunWith(RobolectricTestRunner::class)
+class ComposeCryptoAppTest : BaseViewModelTest() {
+
+    @get:Rule
+    val composeAndroidTestRule = createAndroidComposeRule<MainActivity>()
+
+    private val mockIsNetworkConnectedUseCase: IsNetworkConnectedUseCase = mockk()
+
+    private val noInternetMessage: String
+        get() = composeAndroidTestRule.activity.getString(R.string.no_internet_message)
+
+    @Before
+    fun setUp() {
+        composeAndroidTestRule.activity.setContent {
+            ComposeCryptoApp(
+                cryptoAppState = rememberCryptoAppState(
+                    isNetworkConnectedUseCase = mockIsNetworkConnectedUseCase,
+                    dispatchersProvider = testDispatcherProvider
+                )
+            )
+        }
+    }
+
+    @Test
+    fun `When there is no internet, it shows dialog`() {
+        every { mockIsNetworkConnectedUseCase.invoke() } returns flowOf(false)
+
+        with(composeAndroidTestRule) {
+            onNodeWithText(noInternetMessage).assertIsDisplayed()
+        }
+    }
+
+    @Test
+    fun `When there is internet, it does not show dialog`() {
+        every { mockIsNetworkConnectedUseCase.invoke() } returns flowOf(true)
+
+        with(composeAndroidTestRule) {
+            onNodeWithText(noInternetMessage).assertDoesNotExist()
+        }
+    }
+}

app/src/test/java/co/nimblehq/compose/crypto/ui/screens/detail/DetailScreenTest.kt

@@ -69,7 +69,7 @@ class DetailScreenTest : BaseViewModelTest() {
                 viewModel = viewModel,
                 navigator = { destination -> appDestination = destination },
                 coinId = "Bitcoin",
-                onNetworkReconnected = {}
+                onShowGlobalDialog = {}
             )
         }
     }

app/src/test/java/co/nimblehq/compose/crypto/ui/screens/home/HomeScreenTest.kt

@@ -63,7 +63,8 @@ class HomeScreenTest : BaseViewModelTest() {
         composeAndroidTestRule.activity.setContent {
             HomeScreen(
                 viewModel = viewModel,
-                navigator = { destination -> appDestination = destination }
+                navigator = { destination -> appDestination = destination },
+                onShowGlobalDialog = {}
             )
         }
     }