Fix lint errors

Kaaveh · Kaaveh · commit 59b4f25a4581 · 2026-02-15T09:01:14.000+03:30
diff --git a/app-watch/app/src/androidTest/java/ir/composenews/ui/MainActivityTest.kt b/app-watch/app/src/androidTest/java/ir/composenews/ui/MainActivityTest.kt
@@ -10,22 +10,22 @@ import org.junit.Rule
 import org.junit.Test
 
 class MainActivityTest {
-
     @get:Rule
     val composeTestRule = createAndroidComposeRule<MainActivity>()
 
     @Test
     fun marketListScreen_DisplayedCorrectly() {
-
         composeTestRule.waitUntil(timeoutMillis = 5_000) {
             composeTestRule.onAllNodesWithText("BNB").fetchSemanticsNodes().isNotEmpty() ||
-                    composeTestRule.onAllNodesWithText("BTC").fetchSemanticsNodes().isNotEmpty()
+                composeTestRule.onAllNodesWithText("BTC").fetchSemanticsNodes().isNotEmpty()
         }
 
         composeTestRule.onNodeWithText("BNB").assertExists()
-        composeTestRule.onNodeWithText("Bitcoin").assertExists().onParent().performClick()
+        composeTestRule
+            .onNodeWithText("Bitcoin")
+            .assertExists()
+            .onParent()
+            .performClick()
         composeTestRule.onNodeWithContentDescription("Loading. PLease wait").assertExists()
-
-
     }
 }
diff --git a/app-watch/app/src/main/java/ir/composenews/MainViewModel.kt b/app-watch/app/src/main/java/ir/composenews/MainViewModel.kt
@@ -13,23 +13,28 @@ import kotlinx.coroutines.launch
 import javax.inject.Inject
 
 @HiltViewModel
-class MainViewModel @Inject constructor(
-    dispatcherProvider: DispatcherProvider,
-) : BaseViewModel(dispatcherProvider), MainContract {
+class MainViewModel
+    @Inject
+    constructor(
+        dispatcherProvider: DispatcherProvider,
+    ) : BaseViewModel(dispatcherProvider),
+        MainContract {
+        private val mutableState = MutableStateFlow(MainContract.State())
 
-    private val mutableState = MutableStateFlow(MainContract.State())
-    override val state: StateFlow<MainContract.State> = mutableState.asStateFlow()
-    override fun event(event: MainContract.Event) {
-        when (event) {
-            is MainContract.Event.SetMarket -> setMarket(event.market)
+        override val state: StateFlow<MainContract.State> = mutableState.asStateFlow()
+
+        override fun event(event: MainContract.Event) {
+            when (event) {
+                is MainContract.Event.SetMarket -> setMarket(event.market)
+            }
         }
-    }
 
-    private fun setMarket(market: MarketModel?) = viewModelScope.launch {
-        mutableState.emit(
-            mutableState.value.copy(
-                market
-            )
-        )
+        private fun setMarket(market: MarketModel?) =
+            viewModelScope.launch {
+                mutableState.emit(
+                    mutableState.value.copy(
+                        market,
+                    ),
+                )
+            }
     }
-}
diff --git a/app-watch/app/src/main/java/ir/composenews/navigation/ComposeNewsWearNavHost.kt b/app-watch/app/src/main/java/ir/composenews/navigation/ComposeNewsWearNavHost.kt
@@ -11,11 +11,12 @@ import ir.composenews.appwatch.navigation.graph.wearMarketList
 import ir.composenews.uimarket.model.MarketModel
 
 @Composable
+@Suppress("ktlint:standard:function-naming")
 fun ComposeNewsWearNavHost(
     navController: NavHostController,
     modifier: Modifier,
     onMarketSelected: ((MarketModel) -> Unit)? = null,
-    uiState: MainContract.State
+    uiState: MainContract.State,
 ) {
     SwipeDismissableNavHost(
         navController = navController,
diff --git a/app-watch/app/src/main/java/ir/composenews/ui/MainActivity.kt b/app-watch/app/src/main/java/ir/composenews/ui/MainActivity.kt
@@ -15,14 +15,13 @@ import com.google.android.horologist.compose.layout.AppScaffold
 import com.google.android.horologist.compose.layout.ScreenScaffold
 import dagger.hilt.android.AndroidEntryPoint
 import ir.composenews.MainViewModel
-import ir.composenews.appwatch.navigation.graph.Destinations
 import ir.composenews.appwatch.navigation.MainContract
+import ir.composenews.appwatch.navigation.graph.Destinations
 import ir.composenews.designsystem.theme.ComposeNewsTheme
 import ir.composenews.navigation.ComposeNewsWearNavHost
 
 @AndroidEntryPoint
 class MainActivity : ComponentActivity() {
-
     private val mainViewModel: MainViewModel by viewModels()
 
     override fun onCreate(savedInstanceState: Bundle?) {
@@ -33,6 +32,7 @@ class MainActivity : ComponentActivity() {
     }
 
     @Composable
+    @Suppress("ktlint:standard:function-naming")
     fun WearApp() {
         val navController = rememberSwipeDismissableNavController()
         val state = mainViewModel.state.collectAsStateWithLifecycle()
@@ -46,7 +46,7 @@ class MainActivity : ComponentActivity() {
                             navController.navigate(Destinations.MarketDetailScreen().route)
                         },
                         uiState = state.value,
-                        modifier = Modifier
+                        modifier = Modifier,
                     )
                 }
             }
@@ -55,6 +55,7 @@ class MainActivity : ComponentActivity() {
 
     @Preview(device = WearDevices.LARGE_ROUND, showSystemUi = true)
     @Composable
+    @Suppress("ktlint:standard:function-naming")
     fun DefaultPreview() {
         WearApp()
     }
diff --git a/app/src/main/java/ir/composenews/ComposeNewsApplication.kt b/app/src/main/java/ir/composenews/ComposeNewsApplication.kt
@@ -6,10 +6,8 @@ import ir.composenews.sync.Sync
 
 @HiltAndroidApp
 class ComposeNewsApplication : Application() {
-
     override fun onCreate() {
         super.onCreate()
         Sync.init(this)
     }
-
 }
diff --git a/app/src/main/java/ir/composenews/permission/handler/NotificationPermissionHandler.kt b/app/src/main/java/ir/composenews/permission/handler/NotificationPermissionHandler.kt
@@ -15,9 +15,7 @@ class NotificationPermissionHandler : PermissionHandler {
 
     override val permissionType: PermissionType = PermissionType.POST_NOTIFICATIONS
 
-    override fun hasPermission(
-        context: Context
-    ): Boolean =
+    override fun hasPermission(context: Context): Boolean =
         if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU) {
             NotificationManagerCompat.from(context).areNotificationsEnabled()
         } else {
@@ -30,11 +28,9 @@ class NotificationPermissionHandler : PermissionHandler {
         callback: (isGranted: Boolean) -> Unit,
     ): ActivityResultLauncher<String> = super.permissionLauncher(activity, callback)
 
-
     @RequiresApi(Build.VERSION_CODES.TIRAMISU)
     override fun requestPermission(
         activity: ComponentActivity,
         callback: (isGranted: Boolean) -> Unit,
-    ): Unit? =
-        super.requestPermission(activity, callback)
+    ): Unit? = super.requestPermission(activity, callback)
 }
diff --git a/app/src/main/java/ir/composenews/permission/handler/PermissionHandler.kt b/app/src/main/java/ir/composenews/permission/handler/PermissionHandler.kt
@@ -7,7 +7,6 @@ import androidx.activity.result.contract.ActivityResultContracts
 import ir.composenews.permission.enum.PermissionType
 
 interface PermissionHandler {
-
     val permission: String
     val permissionType: PermissionType
 
@@ -16,26 +15,27 @@ interface PermissionHandler {
     fun permissionLauncher(
         activity: ComponentActivity,
         callback: (isGranted: Boolean) -> Unit,
-    ): ActivityResultLauncher<String> = activity.registerForActivityResult(
-        ActivityResultContracts.RequestPermission()
-    ) { isGranted ->
-        callback(isGranted)
-    }
+    ): ActivityResultLauncher<String> =
+        activity.registerForActivityResult(
+            ActivityResultContracts.RequestPermission(),
+        ) { isGranted ->
+            callback(isGranted)
+        }
 
     fun requestPermission(
         activity: ComponentActivity,
         callback: (isGranted: Boolean) -> Unit,
-    ): Unit? =
-        permissionLauncher(activity, callback).launch(permission)
+    ): Unit? = permissionLauncher(activity, callback).launch(permission)
 
     fun performPermission(
         context: Context,
         activity: ComponentActivity,
         callback: (isGranted: Boolean) -> Unit,
     ) {
-        if (hasPermission(context))
+        if (hasPermission(context)) {
             callback(true)
-        else
+        } else {
             requestPermission(activity, callback)
+        }
     }
 }
diff --git a/app/src/main/java/ir/composenews/permission/handler/ReadStoragePermissionHandler.kt b/app/src/main/java/ir/composenews/permission/handler/ReadStoragePermissionHandler.kt
@@ -11,9 +11,7 @@ class ReadStoragePermissionHandler : PermissionHandler {
 
     override val permissionType: PermissionType = PermissionType.READ_EXTERNAL_STORAGE
 
-    override fun hasPermission(
-        context: Context
-    ): Boolean =
+    override fun hasPermission(context: Context): Boolean =
         ContextCompat.checkSelfPermission(
             context,
             permission,
diff --git a/app/src/main/java/ir/composenews/permission/manager/PermissionManagerImpl.kt b/app/src/main/java/ir/composenews/permission/manager/PermissionManagerImpl.kt
@@ -8,27 +8,22 @@ import ir.composenews.permission.handler.PermissionHandler
 import ir.composenews.permission.handler.ReadStoragePermissionHandler
 
 class PermissionManagerImpl : PermissionManager {
-    private fun getPermissionHandler(
-        permission: PermissionType
-    ): PermissionHandler =
+    private fun getPermissionHandler(permission: PermissionType): PermissionHandler =
         when (permission) {
             PermissionType.POST_NOTIFICATIONS -> NotificationPermissionHandler()
             PermissionType.READ_EXTERNAL_STORAGE -> ReadStoragePermissionHandler()
         }
 
     override fun hasPermission(
         permission: PermissionType,
-        context: Context
-    ): Boolean =
-        getPermissionHandler(permission).hasPermission(context)
+        context: Context,
+    ): Boolean = getPermissionHandler(permission).hasPermission(context)
 
     override fun requestPermission(
         permission: PermissionType,
         activity: ComponentActivity,
         callback: (isGranted: Boolean) -> Unit,
-    ): Unit? =
-        getPermissionHandler(permission).requestPermission(activity, callback)
-
+    ): Unit? = getPermissionHandler(permission).requestPermission(activity, callback)
 
     override fun performPermission(
         permission: PermissionType,
diff --git a/app/src/main/java/ir/composenews/ui/ComposeNewsApp.kt b/app/src/main/java/ir/composenews/ui/ComposeNewsApp.kt
@@ -1,5 +1,8 @@
+@file:Suppress("ktlint:standard:function-naming")
+
 package ir.composenews.ui
 
+import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.padding
 import androidx.compose.material.icons.Icons
 import androidx.compose.material.icons.filled.Favorite
@@ -9,6 +12,7 @@ import androidx.compose.material3.Scaffold
 import androidx.compose.material3.Text
 import androidx.compose.material3.adaptive.ExperimentalMaterial3AdaptiveApi
 import androidx.compose.material3.adaptive.currentWindowAdaptiveInfo
+import androidx.compose.material3.adaptive.navigation.ThreePaneScaffoldNavigator
 import androidx.compose.material3.adaptive.navigation.rememberListDetailPaneScaffoldNavigator
 import androidx.compose.material3.adaptive.navigationsuite.NavigationSuiteScaffold
 import androidx.compose.material3.adaptive.navigationsuite.NavigationSuiteScaffoldDefaults
@@ -21,17 +25,43 @@ import androidx.compose.ui.Modifier
 import ir.composenews.navigation.BottomNavItem
 import ir.composenews.navigation.Destinations
 import ir.composenews.navigation.graph.ListWithDetailScreen
+import kotlinx.collections.immutable.ImmutableList
 import kotlinx.collections.immutable.persistentListOf
 
 @OptIn(ExperimentalMaterial3AdaptiveApi::class)
 @Composable
 fun ComposeNewsApp() {
-    val items = remember {
+    val items = rememberNavigationItems()
+    var currentRoute by remember { mutableStateOf(Destinations.MarketListScreen.route) }
+
+    Scaffold { paddingValues ->
+        NavigationSuiteScaffold(
+            navigationSuiteItems = {
+                items.forEach { item ->
+                    item(
+                        selected = item.route == currentRoute,
+                        onClick = { currentRoute = item.route },
+                        icon = { Icon(imageVector = item.icon, contentDescription = item.name) },
+                        label = { Text(text = item.name) },
+                    )
+                }
+            },
+            layoutType = NavigationSuiteScaffoldDefaults.calculateFromAdaptiveInfo(currentWindowAdaptiveInfo()),
+        ) {
+            val navigator = rememberListDetailPaneScaffoldNavigator<Any>()
+            NavigationContent(currentRoute, paddingValues, navigator)
+        }
+    }
+}
+
+@Composable
+private fun rememberNavigationItems(): ImmutableList<BottomNavItem> =
+    remember {
         persistentListOf(
             BottomNavItem(
                 name = "Markets",
                 route = Destinations.MarketListScreen.route,
-                icon = Icons.Default.Home
+                icon = Icons.Default.Home,
             ),
             BottomNavItem(
                 name = "Favorite",
@@ -41,48 +71,20 @@ fun ComposeNewsApp() {
         )
     }
 
-    var currentRoute by remember { mutableStateOf(Destinations.MarketListScreen.route) }
-
-    Scaffold { paddingValues ->
-        NavigationSuiteScaffold(
-            navigationSuiteItems = {
-                items.forEach { item ->
-                    item(selected = item.route == currentRoute, onClick = {
-                        currentRoute = item.route
-                    }, icon = {
-                        Icon(
-                            imageVector = item.icon,
-                            contentDescription = item.name,
-                        )
-                    }, label = {
-                        Text(
-                            text = item.name,
-                        )
-                    })
-                }
-            },
-            layoutType = NavigationSuiteScaffoldDefaults.calculateFromAdaptiveInfo(
-                currentWindowAdaptiveInfo()
-            )
-        ) {
-            val navigator = rememberListDetailPaneScaffoldNavigator<Any>()
-            when (currentRoute) {
-                Destinations.MarketListScreen.route -> {
-                    ListWithDetailScreen(
-                        Modifier.padding(paddingValues),
-                        navigator,
-                        showFavorite = false,
-                    )
-                }
+@OptIn(ExperimentalMaterial3AdaptiveApi::class)
+@Composable
+private fun NavigationContent(
+    currentRoute: String,
+    paddingValues: PaddingValues,
+    navigator: ThreePaneScaffoldNavigator<Any>,
+) {
+    when (currentRoute) {
+        Destinations.MarketListScreen.route -> {
+            ListWithDetailScreen(Modifier.padding(paddingValues), navigator, showFavorite = false)
+        }
 
-                Destinations.FavoriteMarketScreen.route -> {
-                    ListWithDetailScreen(
-                        Modifier.padding(paddingValues),
-                        navigator,
-                        showFavorite = true,
-                    )
-                }
-            }
+        Destinations.FavoriteMarketScreen.route -> {
+            ListWithDetailScreen(Modifier.padding(paddingValues), navigator, showFavorite = true)
         }
     }
 }
diff --git a/app/src/main/java/ir/composenews/ui/MainActivity.kt b/app/src/main/java/ir/composenews/ui/MainActivity.kt

Original file line number	Diff line number	Diff line change
`@@ -6,10 +6,8 @@ import ir.composenews.sync.Sync`
`6`	`6`
`7`	`7`	`@HiltAndroidApp`
`8`	`8`	`class ComposeNewsApplication : Application() {`
`9`		`-`
`10`	`9`	`override fun onCreate() {`
`11`	`10`	`super.onCreate()`
`12`	`11`	`Sync.init(this)`
`13`	`12`	`}`
`14`		`-`
`15`	`13`	`}`