[#101] Minor grammar concerns

Author: kaungkhantsoe

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 getConnectionStatsUseCase: GetConnectionStatsUseCase,
+    private val getConnectionStatusUseCase: GetConnectionStatusUseCase,
 ) : BaseViewModel(dispatchers), Input, Output {
 
     override val input = this
@@ -92,7 +92,7 @@ class HomeViewModel @Inject constructor(
         loadData()
         // TODO remove in integration ticket
         execute {
-            getConnectionStatsUseCase()
+            getConnectionStatusUseCase()
                 .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 mockGetConnectionStatsUseCase = mockk<GetConnectionStatsUseCase>()
+    private val mockGetConnectionStatusUseCase = mockk<GetConnectionStatusUseCase>()
 
     private lateinit var viewModel: HomeViewModel
 
     private var appDestination: AppDestination? = null
 
     @Before
     fun setUp() {
-        every { mockGetConnectionStatsUseCase() } returns flowOf(null)
+        every { mockGetConnectionStatusUseCase() } returns flowOf(null)
         composeAndroidTestRule.activity.setContent {
             HomeScreen(
                 viewModel = viewModel,
@@ -220,7 +220,7 @@ class HomeScreenTest : BaseViewModelTest() {
             dispatchers = testDispatcherProvider,
             getMyCoinsUseCase = mockGetMyCoinsUseCase,
             getTrendingCoinsUseCase = mockGetTrendingCoinsUseCase,
-            getConnectionStatsUseCase = mockGetConnectionStatsUseCase
+            getConnectionStatusUseCase = mockGetConnectionStatusUseCase
         )
     }
 }

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 mockGetConnectionStatsUseCase = mockk<GetConnectionStatsUseCase>()
+    private val mockGetConnectionStatusUseCase = mockk<GetConnectionStatusUseCase>()
 
     @Before
     fun setUp() {
@@ -146,7 +146,7 @@ class HomeViewModelTest : BaseViewModelTest() {
             testDispatcherProvider,
             mockGetMyCoinsUseCase,
             mockGetTrendingCoinsUseCase,
-            mockGetConnectionStatsUseCase
+            mockGetConnectionStatusUseCase
         )
     }
 }

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

@@ -10,7 +10,7 @@ class GlobalRepositoryImpl(
     context: Context
 ) : GlobalRepository {
 
-    private var connectionStatus: MutableStateFlow<Boolean?> = MutableStateFlow(null)
+    private var isNetworkConnected: MutableStateFlow<Boolean?> = MutableStateFlow(null)
 
     private val networkRequest = NetworkRequest.Builder()
         .addCapability(NetworkCapabilities.NET_CAPABILITY_INTERNET)
@@ -22,7 +22,7 @@ class GlobalRepositoryImpl(
         // network is available for use
         override fun onAvailable(network: Network) {
             super.onAvailable(network)
-            connectionStatus.value = true
+            isNetworkConnected.value = true
         }
 
         // Network capabilities have changed for the network
@@ -32,15 +32,15 @@ class GlobalRepositoryImpl(
         ) {
             super.onCapabilitiesChanged(network, networkCapabilities)
             val unmetered = networkCapabilities.hasCapability(NetworkCapabilities.NET_CAPABILITY_NOT_METERED)
-            if(unmetered) {
-                connectionStatus.value = true
+            if (unmetered) {
+                isNetworkConnected.value = true
             }
         }
 
         // lost network connection
         override fun onLost(network: Network) {
             super.onLost(network)
-            connectionStatus.value = false
+            isNetworkConnected.value = false
         }
     }
 
@@ -49,5 +49,5 @@ class GlobalRepositoryImpl(
         connectivityManager.requestNetwork(networkRequest, networkCallback)
     }
 
-    override fun getConnectionStatus(): Flow<Boolean?> = connectionStatus
+    override fun getConnectionStatus(): Flow<Boolean?> = isNetworkConnected
 }

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

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