Merge pull request #1039 from SimonMarquis/kotlinify

Kotlinify codebase

Author: dturner

app/src/androidTest/kotlin/com/google/samples/apps/nowinandroid/ui/NavigationTest.kt

@@ -90,7 +90,7 @@ class NavigationTest {
     lateinit var topicsRepository: TopicsRepository
 
     private fun AndroidComposeTestRule<*, *>.stringResource(@StringRes resId: Int) =
-        ReadOnlyProperty<Any?, String> { _, _ -> activity.getString(resId) }
+        ReadOnlyProperty<Any, String> { _, _ -> activity.getString(resId) }
 
     // The strings used for matching in these tests
     private val navigateUp by composeTestRule.stringResource(FeatureForyouR.string.feature_foryou_navigate_up)

app/src/main/kotlin/com/google/samples/apps/nowinandroid/MainActivity.kt

@@ -90,9 +90,7 @@ class MainActivity : ComponentActivity() {
         lifecycleScope.launch {
             lifecycle.repeatOnLifecycle(Lifecycle.State.STARTED) {
                 viewModel.uiState
-                    .onEach {
-                        uiState = it
-                    }
+                    .onEach { uiState = it }
                     .collect()
             }
         }

app/src/main/kotlin/com/google/samples/apps/nowinandroid/di/JankStatsModule.kt

@@ -20,6 +20,7 @@ import android.app.Activity
 import android.util.Log
 import android.view.Window
 import androidx.metrics.performance.JankStats
+import androidx.metrics.performance.JankStats.OnFrameListener
 import dagger.Module
 import dagger.Provides
 import dagger.hilt.InstallIn
@@ -29,26 +30,20 @@ import dagger.hilt.android.components.ActivityComponent
 @InstallIn(ActivityComponent::class)
 object JankStatsModule {
     @Provides
-    fun providesOnFrameListener(): JankStats.OnFrameListener {
-        return JankStats.OnFrameListener { frameData ->
-            // Make sure to only log janky frames.
-            if (frameData.isJank) {
-                // We're currently logging this but would better report it to a backend.
-                Log.v("NiA Jank", frameData.toString())
-            }
+    fun providesOnFrameListener(): OnFrameListener = OnFrameListener { frameData ->
+        // Make sure to only log janky frames.
+        if (frameData.isJank) {
+            // We're currently logging this but would better report it to a backend.
+            Log.v("NiA Jank", frameData.toString())
         }
     }
 
     @Provides
-    fun providesWindow(activity: Activity): Window {
-        return activity.window
-    }
+    fun providesWindow(activity: Activity): Window = activity.window
 
     @Provides
     fun providesJankStats(
         window: Window,
-        frameListener: JankStats.OnFrameListener,
-    ): JankStats {
-        return JankStats.createAndTrack(window, frameListener)
-    }
+        frameListener: OnFrameListener,
+    ): JankStats = JankStats.createAndTrack(window, frameListener)
 }

app/src/main/kotlin/com/google/samples/apps/nowinandroid/ui/NiaApp.kt

@@ -97,9 +97,7 @@ fun NiaApp(
 ) {
     val shouldShowGradientBackground =
         appState.currentTopLevelDestination == TopLevelDestination.FOR_YOU
-    var showSettingsDialog by rememberSaveable {
-        mutableStateOf(false)
-    }
+    var showSettingsDialog by rememberSaveable { mutableStateOf(false) }
 
     NiaBackground {
         NiaGradientBackground(

app/src/main/kotlin/com/google/samples/apps/nowinandroid/ui/NiaAppState.kt

@@ -164,9 +164,7 @@ class NiaAppState(
         }
     }
 
-    fun navigateToSearch() {
-        navController.navigateToSearch()
-    }
+    fun navigateToSearch() = navController.navigateToSearch()
 }
 
 /**

benchmarks/src/main/kotlin/androidx/test/uiautomator/UiAutomatorHelpers.kt

@@ -29,15 +29,11 @@ import androidx.test.uiautomator.HasChildrenOp.EXACTLY
 fun untilHasChildren(
     childCount: Int = 1,
     op: HasChildrenOp = AT_LEAST,
-): UiObject2Condition<Boolean> {
-    return object : UiObject2Condition<Boolean>() {
-        override fun apply(element: UiObject2): Boolean {
-            return when (op) {
-                AT_LEAST -> element.childCount >= childCount
-                EXACTLY -> element.childCount == childCount
-                AT_MOST -> element.childCount <= childCount
-            }
-        }
+): UiObject2Condition<Boolean> = object : UiObject2Condition<Boolean>() {
+    override fun apply(element: UiObject2): Boolean = when (op) {
+        AT_LEAST -> element.childCount >= childCount
+        EXACTLY -> element.childCount == childCount
+        AT_MOST -> element.childCount <= childCount
     }
 }
 

benchmarks/src/main/kotlin/com/google/samples/apps/nowinandroid/baselineprofile/StartupBaselineProfile.kt

@@ -16,6 +16,7 @@
 
 package com.google.samples.apps.nowinandroid.baselineprofile
 
+import androidx.benchmark.macro.MacrobenchmarkScope
 import androidx.benchmark.macro.junit4.BaselineProfileRule
 import com.google.samples.apps.nowinandroid.PACKAGE_NAME
 import com.google.samples.apps.nowinandroid.startActivityAndAllowNotifications
@@ -30,11 +31,9 @@ class StartupBaselineProfile {
     @get:Rule val baselineProfileRule = BaselineProfileRule()
 
     @Test
-    fun generate() =
-        baselineProfileRule.collect(
-            PACKAGE_NAME,
-            includeInStartupProfile = true,
-        ) {
-            startActivityAndAllowNotifications()
-        }
+    fun generate() = baselineProfileRule.collect(
+        PACKAGE_NAME,
+        includeInStartupProfile = true,
+        profileBlock = MacrobenchmarkScope::startActivityAndAllowNotifications,
+    )
 }

build-logic/convention/src/main/kotlin/com/google/samples/apps/nowinandroid/PrintTestApks.kt

@@ -86,14 +86,12 @@ internal abstract class PrintApkLocationTask : DefaultTask() {
     fun taskAction() {
         val hasFiles = sources.orNull?.any { directory ->
             directory.asFileTree.files.any {
-                it.isFile && it.parentFile.path.contains("build${File.separator}generated").not()
+                it.isFile && "build${File.separator}generated" !in it.parentFile.path
             }
         } ?: throw RuntimeException("Cannot check androidTest sources")
 
         // Don't print APK location if there are no androidTest source files
-        if (!hasFiles) {
-            return
-        }
+        if (!hasFiles) return
 
         val builtArtifacts = builtArtifactsLoader.get().load(apkFolder.get())
             ?: throw RuntimeException("Cannot load APKs")

core/common/src/main/kotlin/com/google/samples/apps/nowinandroid/core/result/Result.kt

@@ -23,15 +23,10 @@ import kotlinx.coroutines.flow.onStart
 
 sealed interface Result<out T> {
     data class Success<T>(val data: T) : Result<T>
-    data class Error(val exception: Throwable? = null) : Result<Nothing>
+    data class Error(val exception: Throwable) : Result<Nothing>
     data object Loading : Result<Nothing>
 }
 
-fun <T> Flow<T>.asResult(): Flow<Result<T>> {
-    return this
-        .map<T, Result<T>> {
-            Result.Success(it)
-        }
-        .onStart { emit(Result.Loading) }
-        .catch { emit(Result.Error(it)) }
-}
+fun <T> Flow<T>.asResult(): Flow<Result<T>> = map<T, Result<T>> { Result.Success(it) }
+    .onStart { emit(Result.Loading) }
+    .catch { emit(Result.Error(it)) }

core/data/src/main/kotlin/com/google/samples/apps/nowinandroid/core/data/repository/DefaultRecentSearchRepository.kt

@@ -39,9 +39,7 @@ internal class DefaultRecentSearchRepository @Inject constructor(
 
     override fun getRecentSearchQueries(limit: Int): Flow<List<RecentSearchQuery>> =
         recentSearchQueryDao.getRecentSearchQueryEntities(limit).map { searchQueries ->
-            searchQueries.map {
-                it.asExternalModel()
-            }
+            searchQueries.map { it.asExternalModel() }
         }
 
     override suspend fun clearRecentSearches() = recentSearchQueryDao.clearRecentSearchQueries()