[#101] Minor name changes

Author: kaungkhantsoe

app/src/main/java/co/nimblehq/compose/crypto/ui/screens/home/HomeScreen.kt

@@ -57,10 +57,10 @@ fun HomeScreen(
     }
 
     // TODO remove in integration ticket
-    val hasConnection by viewModel.hasConnection.collectAsState()
-    LaunchedEffect(key1 = hasConnection) {
-        if (hasConnection != null) {
-            Toast.makeText(context,"Connection: $hasConnection", Toast.LENGTH_SHORT).show()
+    val isNetworkConnected by viewModel.hasConnection.collectAsState()
+    LaunchedEffect(isNetworkConnected) {
+        if (isNetworkConnected != null) {
+            Toast.makeText(context,"Connection: $isNetworkConnected", Toast.LENGTH_SHORT).show()
         }
     }
 

app/src/main/java/co/nimblehq/compose/crypto/ui/screens/home/HomeViewModel.kt

@@ -52,7 +52,7 @@ class HomeViewModel @Inject constructor(
     dispatchers: DispatchersProvider,
     private val getMyCoinsUseCase: GetMyCoinsUseCase,
     private val getTrendingCoinsUseCase: GetTrendingCoinsUseCase,
-    private val getConnectionStatusUseCase: GetConnectionStatusUseCase,
+    private val isNetworkConnectedUseCase: IsNetworkConnectedUseCase,
 ) : BaseViewModel(dispatchers), Input, Output {
 
     override val input = this
@@ -92,7 +92,7 @@ class HomeViewModel @Inject constructor(
         loadData()
         // TODO remove in integration ticket
         execute {
-            getConnectionStatusUseCase()
+            isNetworkConnectedUseCase()
                 .collect {
                     _hasConnection.emit(it)
                 }

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

@@ -55,15 +55,15 @@ class HomeScreenTest : BaseViewModelTest() {
     private val mockGetTrendingCoinsUseCase = mockk<GetTrendingCoinsUseCase>()
 
     // TODO remove in integration ticket
-    private val mockGetConnectionStatusUseCase = mockk<GetConnectionStatusUseCase>()
+    private val mockIsNetworkConnectedUseCase = mockk<IsNetworkConnectedUseCase>()
 
     private lateinit var viewModel: HomeViewModel
 
     private var appDestination: AppDestination? = null
 
     @Before
     fun setUp() {
-        every { mockGetConnectionStatusUseCase() } returns flowOf(null)
+        every { mockIsNetworkConnectedUseCase() } returns flowOf(null)
         composeAndroidTestRule.activity.setContent {
             HomeScreen(
                 viewModel = viewModel,
@@ -220,7 +220,7 @@ class HomeScreenTest : BaseViewModelTest() {
             dispatchers = testDispatcherProvider,
             getMyCoinsUseCase = mockGetMyCoinsUseCase,
             getTrendingCoinsUseCase = mockGetTrendingCoinsUseCase,
-            getConnectionStatusUseCase = mockGetConnectionStatusUseCase
+            isNetworkConnectedUseCase = mockIsNetworkConnectedUseCase
         )
     }
 }

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

@@ -25,7 +25,7 @@ class HomeViewModelTest : BaseViewModelTest() {
     private lateinit var viewModel: HomeViewModel
 
     // TODO remove in integration ticket
-    private val mockGetConnectionStatusUseCase = mockk<GetConnectionStatusUseCase>()
+    private val mockIsNetworkConnectedUseCase = mockk<IsNetworkConnectedUseCase>()
 
     @Before
     fun setUp() {
@@ -146,7 +146,7 @@ class HomeViewModelTest : BaseViewModelTest() {
             testDispatcherProvider,
             mockGetMyCoinsUseCase,
             mockGetTrendingCoinsUseCase,
-            mockGetConnectionStatusUseCase
+            mockIsNetworkConnectedUseCase
         )
     }
 }

data/src/main/java/co/nimblehq/compose/crypto/data/repository/GlobalRepositoryImpl.kt

@@ -10,7 +10,7 @@ class GlobalRepositoryImpl(
     context: Context
 ) : GlobalRepository {
 
-    private var isNetworkConnected: MutableStateFlow<Boolean?> = MutableStateFlow(null)
+    private val isNetworkConnected: MutableStateFlow<Boolean?> = MutableStateFlow(null)
 
     private val networkRequest = NetworkRequest.Builder()
         .addCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET)
@@ -49,5 +49,5 @@ class GlobalRepositoryImpl(
         connectivityManager.requestNetwork(networkRequest, networkCallback)
     }
 
-    override fun getConnectionStatus(): Flow<Boolean?> = isNetworkConnected
+    override fun isNetworkConnected(): Flow<Boolean?> = isNetworkConnected
 }

domain/src/main/java/co/nimblehq/compose/crypto/domain/repository/GlobalRepository.kt

@@ -3,5 +3,5 @@ package co.nimblehq.compose.crypto.domain.repository
 import kotlinx.coroutines.flow.Flow
 
 interface GlobalRepository {
-    fun getConnectionStatus(): Flow<Boolean?>
+    fun isNetworkConnected(): Flow<Boolean?>
 }

domain/src/main/java/co/nimblehq/compose/crypto/domain/usecase/GetConnectionStatusUseCase.kt renamed to domain/src/main/java/co/nimblehq/compose/crypto/domain/usecase/IsNetworkConnectedUseCase.kt

@@ -4,6 +4,6 @@ import co.nimblehq.compose.crypto.domain.repository.GlobalRepository
 import kotlinx.coroutines.flow.Flow
 import javax.inject.Inject
 
-class GetConnectionStatusUseCase @Inject constructor(private val repository: GlobalRepository) {
-    operator fun invoke(): Flow<Boolean?> = repository.getConnectionStatus()
+class IsNetworkConnectedUseCase @Inject constructor(private val repository: GlobalRepository) {
+    operator fun invoke(): Flow<Boolean?> = repository.isNetworkConnected()
 }