Added JLLeitschuh ktlint-gradle plugin

app/build.gradle.kts

@@ -7,6 +7,7 @@ plugins {
     kotlin("android")
     id("prieto.fernando.android.plugin")
     id("dagger.hilt.android.plugin")
+    id("org.jlleitschuh.gradle.ktlint") version "10.2.1"
 }
 
 androidPlugin {
@@ -54,7 +55,7 @@ dependencies {
     implementation(Dependencies.AndroidX.fragmentKtx)
     implementation(Dependencies.AndroidX.lifecycleLivedataKtx)
     implementation(Dependencies.AndroidX.Compose.viewModel)
-    annotationProcessor(Dependencies.AndroidX.lifecycleCompiler)
+    kapt(Dependencies.AndroidX.lifecycleCompiler)
     implementation(Dependencies.AndroidX.archComponents)
     implementation(Dependencies.AndroidX.browser)
 
@@ -99,4 +100,4 @@ dependencies {
     androidTestImplementation(TestDependencies.Hilt.androidTesting)
     kaptAndroidTest(TestDependencies.Hilt.androidCompiler)
     androidTestAnnotationProcessor(TestDependencies.Hilt.androidCompiler)
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/di/FakeNetworkModule.kt

@@ -13,4 +13,4 @@ import prieto.fernando.data_api.di.NetworkModule
 )
 class FakeNetworkModule : NetworkModule() {
     override fun getBaseUrl() = "http://127.0.0.1:${BuildConfig.PORT}"
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/presentation/screens/MainScreenKtTest.kt

@@ -12,7 +12,6 @@ import org.junit.runner.RunWith
 import prieto.fernando.spacex.presentation.screens.base.BaseScreenTest
 import prieto.fernando.spacex.webmock.SuccessDispatcher
 
-
 @ExperimentalMaterialApi
 @RunWith(AndroidJUnit4::class)
 @HiltAndroidTest
@@ -47,4 +46,4 @@ class MainScreenKtTest : BaseScreenTest() {
             .performClick()
         composeTestRule.onNodeWithText("LAUNCHES", useUnmergedTree = true).assertIsDisplayed()
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/presentation/screens/base/BaseScreenTest.kt

@@ -42,4 +42,4 @@ open class BaseScreenTest {
             }
         }
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/presentation/screens/dashboard/DashboardScreenTest.kt

@@ -42,4 +42,4 @@ class DashboardScreenTest : BaseScreenTest() {
             onNodeWithContentDescription("404 Animation").assertIsDisplayed()
         }
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/presentation/screens/launches/LaunchesScreenKtTest.kt

@@ -28,7 +28,7 @@ class LaunchesScreenKtTest : BaseScreenTest() {
         mockWebServer.dispatcher = SuccessDispatcher()
         setMainContent()
 
-       launchesScreenRobot(composeTestRule) {
+        launchesScreenRobot(composeTestRule) {
             clickOnLaunchesTab()
             initialElementsShowed()
         }
@@ -94,7 +94,8 @@ class LaunchesScreenKtTest : BaseScreenTest() {
                         onEventSent = {},
                         effectFlow = flow { emit(LaunchesContract.Effect.ClickableLink.None) },
                         onLinkClicked = { },
-                        onClickableLinkRetrieved = { })
+                        onClickableLinkRetrieved = { }
+                    )
                 }
             }
 
@@ -119,23 +120,29 @@ class LaunchesScreenKtTest : BaseScreenTest() {
                         state = LaunchesContract.State(
                             listOf(
                                 LaunchUiModel(
-                                    "Mission1", "08-12-2021", true, "0", RocketUiModel(
+                                    "Mission1", "08-12-2021", true, "0",
+                                    RocketUiModel(
                                         "Rocket1", "Rocket Type1"
-                                    ), LinksUiModel("", "", "Youtube Link"), true
+                                    ),
+                                    LinksUiModel("", "", "Youtube Link"), true
                                 ),
                                 LaunchUiModel(
-                                    "Mission2", "09-12-2021", false, "0", RocketUiModel(
+                                    "Mission2", "09-12-2021", false, "0",
+                                    RocketUiModel(
                                         "Rocket2", "Rocket Type2"
-                                    ), LinksUiModel("", "WikiPedia Link", "Youtube Link"), false
+                                    ),
+                                    LinksUiModel("", "WikiPedia Link", "Youtube Link"), false
                                 )
-                            ), isLoading = false, isError = false
+                            ),
+                            isLoading = false, isError = false
                         ),
                         bottomSheetScaffoldState = bottomSheetScaffoldState,
                         coroutineScope = coroutineScope,
                         onEventSent = {},
                         effectFlow = flow { emit(LaunchesContract.Effect.ClickableLink.None) },
                         onLinkClicked = { },
-                        onClickableLinkRetrieved = { })
+                        onClickableLinkRetrieved = { }
+                    )
                 }
             }
 
@@ -145,4 +152,4 @@ class LaunchesScreenKtTest : BaseScreenTest() {
             }
         }
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/presentation/screens/launches/LaunchesScreenRobot.kt

@@ -1,12 +1,13 @@
 package prieto.fernando.spacex.presentation.screens.launches
 
-import androidx.compose.ui.test.*
+import androidx.compose.ui.test.assertCountEquals
+import androidx.compose.ui.test.assertIsDisplayed
 import androidx.compose.ui.test.junit4.AndroidComposeTestRule
-import androidx.test.espresso.Espresso.onView
-import androidx.test.espresso.assertion.ViewAssertions.matches
-import androidx.test.espresso.matcher.ViewMatchers.*
+import androidx.compose.ui.test.onAllNodesWithContentDescription
+import androidx.compose.ui.test.onNodeWithContentDescription
+import androidx.compose.ui.test.onNodeWithText
+import androidx.compose.ui.test.performClick
 import androidx.test.ext.junit.rules.ActivityScenarioRule
-import kotlinx.coroutines.InternalCoroutinesApi
 import prieto.fernando.spacex.presentation.EntryPointActivity
 
 internal fun launchesScreenRobot(
@@ -93,8 +94,8 @@ internal open class LaunchesScreenRobot constructor(
         listItemsShowed(0)
     }
 
-    fun missionOneAndTwoShowed(){
+    fun missionOneAndTwoShowed() {
         missionOneText.assertIsDisplayed()
         missionTwoText.assertIsDisplayed()
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/webmock/AssetReaderUtil.kt

@@ -23,4 +23,4 @@ object AssetReaderUtil {
         }
         return builder.toString()
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/webmock/ErrorDispatcher.kt

@@ -8,4 +8,4 @@ class ErrorDispatcher : Dispatcher() {
     override fun dispatch(request: RecordedRequest): MockResponse {
         return MockResponse().setResponseCode(404)
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/webmock/MockTestRunner.kt

@@ -20,4 +20,4 @@ class MockTestRunner : AndroidJUnitRunner() {
     ): Application {
         return super.newApplication(cl, HiltTestApplication::class.java.name, context)
     }
-}
+}

app/src/androidTest/java/prieto/fernando/spacex/webmock/SuccessDispatcher.kt

@@ -35,4 +35,4 @@ class SuccessDispatcher(
             errorResponse
         }
     }
-}
+}

app/src/main/java/prieto/fernando/spacex/BaseApplication.kt

@@ -2,4 +2,4 @@ package prieto.fernando.spacex
 
 import android.app.Application
 
-open class BaseApplication : Application()
+open class BaseApplication : Application()

app/src/main/java/prieto/fernando/spacex/SpaceXApp.kt

@@ -1,7 +1,6 @@
 package prieto.fernando.spacex
 
 import dagger.hilt.android.HiltAndroidApp
-import android.app.Application
 
 @HiltAndroidApp
 class SpaceXApp : BaseApplication()

app/src/main/java/prieto/fernando/spacex/di/AppModule.kt

@@ -1,15 +1,9 @@
 package prieto.fernando.spacex.di
 
-import android.app.Application
-import android.content.Context
-import android.content.res.Resources
 import dagger.Module
-import dagger.Provides
 import dagger.hilt.InstallIn
-import dagger.hilt.android.qualifiers.ApplicationContext
 import dagger.hilt.components.SingletonComponent
 
-
 @Module
 @InstallIn(SingletonComponent::class)
 open class AppModule {

app/src/main/java/prieto/fernando/spacex/presentation/EntryPointActivity.kt

@@ -12,7 +12,7 @@ import prieto.fernando.spacex.theme.SpaceXTheme
 @AndroidEntryPoint
 class EntryPointActivity : ComponentActivity() {
 
-    @OptIn(InternalCoroutinesApi::class, ExperimentalMaterialApi::class )
+    @OptIn(InternalCoroutinesApi::class, ExperimentalMaterialApi::class)
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         setContent {

app/src/main/java/prieto/fernando/spacex/presentation/navigation/BottomNavigationScreens.kt

@@ -20,4 +20,4 @@ sealed class BottomNavigationScreens(
         R.string.tab_title_launches,
         R.drawable.ic_launches
     )
-}
+}

app/src/main/java/prieto/fernando/spacex/presentation/screens/MainScreen.kt

@@ -21,7 +21,6 @@ import androidx.compose.ui.platform.LocalContext
 import androidx.compose.ui.res.painterResource
 import androidx.compose.ui.res.stringResource
 import androidx.compose.ui.tooling.preview.Preview
-import androidx.compose.ui.tooling.preview.PreviewParameter
 import androidx.compose.ui.unit.Dp
 import androidx.compose.ui.unit.dp
 import androidx.hilt.navigation.compose.hiltViewModel
@@ -34,15 +33,15 @@ import com.google.accompanist.systemuicontroller.rememberSystemUiController
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.InternalCoroutinesApi
 import prieto.fernando.spacex.R
+import prieto.fernando.spacex.presentation.navigation.BottomNavigationScreens
 import prieto.fernando.spacex.presentation.screens.dashboard.DashboardScreen
 import prieto.fernando.spacex.presentation.screens.launches.LaunchesContract
 import prieto.fernando.spacex.presentation.screens.launches.LaunchesScreen
-import prieto.fernando.spacex.presentation.navigation.BottomNavigationScreens
+import prieto.fernando.spacex.presentation.vm.DashboardViewModel
+import prieto.fernando.spacex.presentation.vm.LaunchesViewModel
 import prieto.fernando.spacex.theme.Dark
 import prieto.fernando.spacex.theme.Light
 import prieto.fernando.spacex.theme.SpaceXTypography
-import prieto.fernando.spacex.presentation.vm.DashboardViewModel
-import prieto.fernando.spacex.presentation.vm.LaunchesViewModel
 
 @InternalCoroutinesApi
 @ExperimentalMaterialApi
@@ -167,16 +166,19 @@ private fun BottomSheet(
                         youTubeLinkState.value = effect.youTubeLink
                         wikipediaLinkState.value = effect.wikipedia
                     }
-                    is LaunchesContract.Effect.ClickableLink.Youtube -> youTubeLinkState.value =
-                        effect.youTubeLink
-                    is LaunchesContract.Effect.ClickableLink.Wikipedia -> wikipediaLinkState.value =
-                        effect.wikipedia
+                    is LaunchesContract.Effect.ClickableLink.Youtube ->
+                        youTubeLinkState.value =
+                            effect.youTubeLink
+                    is LaunchesContract.Effect.ClickableLink.Wikipedia ->
+                        wikipediaLinkState.value =
+                            effect.wikipedia
                     else -> {
                         youTubeLinkState.value = ""
                         wikipediaLinkState.value = ""
                     }
                 }
-            })
+            }
+        )
     }
 }
 
@@ -323,7 +325,6 @@ private fun getTabColour(selected: Boolean) =
         else Dark.UnselectedTab
     }
 
-
 @Composable
 private fun AnimatableIcon(
     modifier: Modifier,
@@ -382,4 +383,4 @@ private fun AnimatableText(
 private fun currentRoute(navController: NavHostController): String? {
     val navBackStackEntry by navController.currentBackStackEntryAsState()
     return navBackStackEntry?.destination?.route
-}
+}

app/src/main/java/prieto/fernando/spacex/presentation/screens/common/ErrorAnimation.kt

@@ -41,4 +41,4 @@ fun ErrorAnimation(
             modifier = modifier.semantics { contentDescription = "404 Animation" }
         )
     }
-}
+}

app/src/main/java/prieto/fernando/spacex/presentation/screens/dashboard/CompanyInfoUiModel.kt

@@ -7,4 +7,4 @@ data class CompanyInfoUiModel(
     val employees: String,
     val launchSites: Int,
     val valuation: Long
-)
+)

app/src/main/java/prieto/fernando/spacex/presentation/screens/dashboard/DashboardContract.kt

@@ -14,4 +14,4 @@ class DashboardContract {
     ) : ViewState
 
     object Effect : ViewSideEffect
-}
+}

app/src/main/java/prieto/fernando/spacex/presentation/screens/dashboard/DashboardScreen.kt

@@ -53,9 +53,9 @@ fun DashboardScreen(
         when {
             state.isLoading -> {
                 LottieAnimation(
-                   composition =  loadingComposition,
+                    composition = loadingComposition,
                     progress = loadingProgress,
-                    modifier = Modifier.semantics { contentDescription= "Loading Animation" }
+                    modifier = Modifier.semantics { contentDescription = "Loading Animation" }
                 )
             }
             state.isError -> {
@@ -92,4 +92,4 @@ private fun fillCompanyInfo(companyInfoUiModel: CompanyInfoUiModel): String =
         companyInfoUiModel.employees,
         companyInfoUiModel.launchSites,
         companyInfoUiModel.valuation
-    )
+    )

app/src/main/java/prieto/fernando/spacex/presentation/screens/launches/LaunchesContract.kt

@@ -29,4 +29,4 @@ class LaunchesContract {
 
         data class LinkClicked(val link: String) : Effect()
     }
-}
+}

app/src/main/java/prieto/fernando/spacex/presentation/screens/launches/LaunchesScreen.kt

@@ -33,10 +33,10 @@ import kotlinx.coroutines.flow.collect
 import kotlinx.coroutines.flow.onEach
 import kotlinx.coroutines.launch
 import prieto.fernando.spacex.R
+import prieto.fernando.spacex.presentation.screens.common.ErrorAnimation
 import prieto.fernando.spacex.theme.Dark
 import prieto.fernando.spacex.theme.Light
 import prieto.fernando.spacex.theme.SpaceXTypography
-import prieto.fernando.spacex.presentation.screens.common.ErrorAnimation
 
 @InternalCoroutinesApi
 @ExperimentalMaterialApi
@@ -246,7 +246,8 @@ private fun DismissButton(openDialog: MutableState<Boolean>) {
     TextButton(
         onClick = {
             openDialog.value = false
-        }) {
+        }
+    ) {
         Text(
             text = stringResource(id = R.string.dialog_cancel_button),
             style = SpaceXTypography.subtitle2,
@@ -267,7 +268,8 @@ private fun ConfirmButton(
         onClick = {
             openDialog.value = false
             onEventSent(LaunchesContract.Event.Filter(textState, orderChecked.value))
-        }) {
+        }
+    ) {
         Text(
             stringResource(id = R.string.dialog_ok_button),
             style = SpaceXTypography.subtitle2,
@@ -472,4 +474,4 @@ private fun getSuccessDrawable(launchSuccess: Boolean) =
         R.drawable.ic_check
     } else {
         R.drawable.ic_clear
-    }
+    }

app/src/main/java/prieto/fernando/spacex/presentation/vm/DashboardViewModel.kt

@@ -6,10 +6,10 @@ import kotlinx.coroutines.CoroutineExceptionHandler
 import kotlinx.coroutines.flow.catch
 import kotlinx.coroutines.flow.collect
 import kotlinx.coroutines.launch
-import prieto.fernando.spacex.presentation.vm.base.BaseViewModel
 import prieto.fernando.domain.usecase.GetCompanyInfo
 import prieto.fernando.spacex.presentation.screens.dashboard.CompanyInfoUiModel
 import prieto.fernando.spacex.presentation.screens.dashboard.DashboardContract
+import prieto.fernando.spacex.presentation.vm.base.BaseViewModel
 import prieto.fernando.spacex.presentation.vm.mapper.CompanyInfoDomainToUiModelMapper
 import timber.log.Timber
 import javax.inject.Inject
@@ -76,4 +76,4 @@ class DashboardViewModel @Inject constructor(
             )
         }
     }
-}
+}