Apply formatter to the entire app (#4698)

Author: david-livefront

app/src/main/java/com/x8bit/bitwarden/data/auth/repository/AuthRepositoryImpl.kt

@@ -1372,6 +1372,7 @@ class AuthRepositoryImpl(
                 // the notice needs to appear again
                 NewDeviceNoticeDisplayStatus.HAS_SEEN ->
                     newDeviceNoticeState.shouldDisplayNoticeIfSeen
+
                 NewDeviceNoticeDisplayStatus.HAS_NOT_SEEN -> true
                 // the user never needs to see the notice again
                 NewDeviceNoticeDisplayStatus.CAN_ACCESS_EMAIL_PERMANENT -> false
@@ -1592,6 +1593,7 @@ class AuthRepositoryImpl(
                     configDiskSource.serverConfig?.isOfficialBitwardenServer == false -> {
                         LoginResult.UnofficialServerError
                     }
+
                     else -> LoginResult.Error(errorMessage = null)
                 }
             },

app/src/main/java/com/x8bit/bitwarden/data/autofill/fido2/manager/Fido2CredentialManagerImpl.kt

@@ -24,7 +24,6 @@ import com.x8bit.bitwarden.data.vault.datasource.sdk.util.toAndroidAttestationRe
 import com.x8bit.bitwarden.data.vault.datasource.sdk.util.toAndroidFido2PublicKeyCredential
 import com.x8bit.bitwarden.ui.platform.base.util.toHostOrPathOrNull
 import kotlinx.serialization.SerializationException
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 
 /**

app/src/main/java/com/x8bit/bitwarden/data/platform/datasource/disk/ConfigDiskSourceImpl.kt

@@ -6,7 +6,6 @@ import com.x8bit.bitwarden.data.platform.repository.util.bufferedMutableSharedFl
 import com.x8bit.bitwarden.data.platform.util.decodeFromStringOrNull
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.onSubscription
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 
 private const val SERVER_CONFIGURATIONS = "serverConfigurations"

app/src/main/java/com/x8bit/bitwarden/data/platform/datasource/disk/EnvironmentDiskSourceImpl.kt

@@ -6,7 +6,6 @@ import com.x8bit.bitwarden.data.platform.repository.util.bufferedMutableSharedFl
 import com.x8bit.bitwarden.data.platform.util.decodeFromStringOrNull
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.onSubscription
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 
 private const val PRE_AUTH_URLS_KEY = "preAuthEnvironmentUrls"

app/src/main/java/com/x8bit/bitwarden/data/platform/datasource/disk/EventDiskSourceImpl.kt

@@ -6,7 +6,6 @@ import com.x8bit.bitwarden.data.platform.datasource.network.model.OrganizationEv
 import com.x8bit.bitwarden.data.platform.manager.dispatcher.DispatcherManager
 import com.x8bit.bitwarden.data.platform.manager.model.OrganizationEventType
 import kotlinx.coroutines.withContext
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 
 /**

app/src/main/java/com/x8bit/bitwarden/data/platform/datasource/disk/SettingsDiskSource.kt

@@ -394,8 +394,8 @@ interface SettingsDiskSource {
     fun getShouldShowGeneratorCoachMarkFlow(): Flow<Boolean?>
 
     /**
-    * Stores the given [screenData] as the screen to resume to identified by [userId].
-    */
+     * Stores the given [screenData] as the screen to resume to identified by [userId].
+     */
     fun storeAppResumeScreen(userId: String, screenData: AppResumeScreenData?)
 
     /**

app/src/main/java/com/x8bit/bitwarden/data/tiles/BitwardenGeneratorTileService.kt

@@ -8,7 +8,6 @@ import com.x8bit.bitwarden.data.platform.annotation.OmitFromCoverage
 import com.x8bit.bitwarden.data.platform.util.isBuildVersionBelow
 import com.x8bit.bitwarden.ui.platform.manager.intent.IntentManager
 import dagger.hilt.android.AndroidEntryPoint
-import kotlinx.coroutines.Runnable
 import javax.inject.Inject
 
 /**

app/src/main/java/com/x8bit/bitwarden/data/tiles/BitwardenVaultTileService.kt

@@ -8,7 +8,6 @@ import com.x8bit.bitwarden.data.platform.annotation.OmitFromCoverage
 import com.x8bit.bitwarden.data.platform.util.isBuildVersionBelow
 import com.x8bit.bitwarden.ui.platform.manager.intent.IntentManager
 import dagger.hilt.android.AndroidEntryPoint
-import kotlinx.coroutines.Runnable
 import javax.inject.Inject
 
 /**

app/src/main/java/com/x8bit/bitwarden/data/tools/generator/datasource/disk/GeneratorDiskSourceImpl.kt

@@ -5,7 +5,6 @@ import com.x8bit.bitwarden.data.platform.datasource.disk.BaseDiskSource
 import com.x8bit.bitwarden.data.platform.util.decodeFromStringOrNull
 import com.x8bit.bitwarden.data.tools.generator.repository.model.PasscodeGenerationOptions
 import com.x8bit.bitwarden.data.tools.generator.repository.model.UsernameGenerationOptions
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 
 private const val PASSWORD_GENERATION_OPTIONS_KEY = "passwordGenerationOptions"

app/src/main/java/com/x8bit/bitwarden/data/vault/datasource/disk/VaultDiskSourceImpl.kt

@@ -23,7 +23,6 @@ import kotlinx.coroutines.flow.map
 import kotlinx.coroutines.flow.merge
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.withContext
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 import timber.log.Timber
 

app/src/main/java/com/x8bit/bitwarden/data/vault/datasource/sdk/VaultSdkSourceImpl.kt

@@ -13,8 +13,8 @@ import com.bitwarden.fido.Fido2CredentialAutofillView
 import com.bitwarden.fido.PublicKeyCredentialAuthenticatorAssertionResponse
 import com.bitwarden.fido.PublicKeyCredentialAuthenticatorAttestationResponse
 import com.bitwarden.sdk.BitwardenException
-import com.bitwarden.sdk.VaultClient
 import com.bitwarden.sdk.Fido2CredentialStore
+import com.bitwarden.sdk.VaultClient
 import com.bitwarden.send.Send
 import com.bitwarden.send.SendView
 import com.bitwarden.vault.Attachment

app/src/main/java/com/x8bit/bitwarden/ui/platform/components/coachmark/CoachMarkScope.kt

@@ -3,7 +3,6 @@ package com.x8bit.bitwarden.ui.platform.components.coachmark
 import androidx.compose.foundation.layout.BoxScope
 import androidx.compose.foundation.layout.RowScope
 import androidx.compose.foundation.lazy.LazyListScope
-import androidx.compose.foundation.lazy.items
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
 import com.x8bit.bitwarden.ui.platform.base.util.Text

app/src/main/java/com/x8bit/bitwarden/ui/platform/components/coachmark/CoachMarkScopeInstance.kt

@@ -24,7 +24,6 @@ import androidx.compose.ui.semantics.SemanticsPropertyReceiver
 import androidx.compose.ui.semantics.semantics
 import androidx.compose.ui.unit.Dp
 import androidx.compose.ui.unit.dp
-import com.x8bit.bitwarden.R
 import com.x8bit.bitwarden.ui.platform.base.util.Text
 import com.x8bit.bitwarden.ui.platform.base.util.toListItemCardStyle
 import com.x8bit.bitwarden.ui.platform.components.coachmark.model.CoachMarkHighlightShape

app/src/main/java/com/x8bit/bitwarden/ui/platform/components/dialog/BitwardenLoadingDialog.kt

@@ -6,8 +6,8 @@ import androidx.compose.runtime.Composable
 import androidx.compose.ui.ExperimentalComposeUiApi
 import androidx.compose.ui.Modifier
 import androidx.compose.ui.semantics.semantics
-import androidx.compose.ui.semantics.testTagsAsResourceId
 import androidx.compose.ui.semantics.testTag
+import androidx.compose.ui.semantics.testTagsAsResourceId
 import androidx.compose.ui.tooling.preview.Preview
 import androidx.compose.ui.window.Popup
 import androidx.compose.ui.window.PopupProperties

app/src/test/java/com/x8bit/bitwarden/data/auth/datasource/sdk/AuthSdkSourceTest.kt

@@ -8,8 +8,8 @@ import com.bitwarden.core.RegisterKeyResponse
 import com.bitwarden.core.RegisterTdeKeyResponse
 import com.bitwarden.crypto.HashPurpose
 import com.bitwarden.crypto.Kdf
-import com.bitwarden.sdk.Client
 import com.bitwarden.sdk.AuthClient
+import com.bitwarden.sdk.Client
 import com.bitwarden.sdk.PlatformClient
 import com.x8bit.bitwarden.data.auth.datasource.sdk.model.PasswordStrength
 import com.x8bit.bitwarden.data.platform.manager.SdkClientManager

app/src/test/java/com/x8bit/bitwarden/data/autofill/fido2/manager/Fido2CredentialManagerTest.kt

@@ -37,7 +37,6 @@ import io.mockk.slot
 import io.mockk.unmockkStatic
 import kotlinx.coroutines.test.runTest
 import kotlinx.serialization.SerializationException
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 import org.junit.jupiter.api.AfterEach
 import org.junit.jupiter.api.Assertions.assertEquals

app/src/test/java/com/x8bit/bitwarden/data/platform/datasource/disk/EnvironmentDiskSourceTest.kt

@@ -6,7 +6,6 @@ import com.x8bit.bitwarden.data.auth.datasource.disk.model.EnvironmentUrlDataJso
 import com.x8bit.bitwarden.data.platform.base.FakeSharedPreferences
 import com.x8bit.bitwarden.data.platform.repository.model.Environment
 import kotlinx.coroutines.test.runTest
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 import kotlinx.serialization.json.encodeToJsonElement
 import org.junit.jupiter.api.Assertions.assertEquals

app/src/test/java/com/x8bit/bitwarden/data/platform/manager/FirstTimeActionManagerTest.kt

@@ -349,13 +349,13 @@ class FirstTimeActionManagerTest {
     fun `shouldShowGeneratorCoachMarkFlow updates when onboarding feature value changes`() =
         runTest {
             firstTimeActionManager.shouldShowGeneratorCoachMarkFlow.test {
-            // Null will be mapped to false
+                // Null will be mapped to false
                 assertFalse(awaitItem())
                 mutableOnboardingFeatureFlow.update { true }
-            // Take the value from disk.
+                // Take the value from disk.
                 assertTrue(awaitItem())
+            }
         }
-    }
 
     @Suppress("MaxLineLength")
     @Test

app/src/test/java/com/x8bit/bitwarden/data/tools/generator/datasource/disk/GeneratorDiskSourceTest.kt

@@ -4,7 +4,6 @@ import androidx.core.content.edit
 import com.x8bit.bitwarden.data.platform.base.FakeSharedPreferences
 import com.x8bit.bitwarden.data.tools.generator.repository.model.PasscodeGenerationOptions
 import com.x8bit.bitwarden.data.tools.generator.repository.model.UsernameGenerationOptions
-import kotlinx.serialization.encodeToString
 import kotlinx.serialization.json.Json
 import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Assertions.assertNotNull

app/src/test/java/com/x8bit/bitwarden/data/vault/datasource/sdk/VaultSdkSourceTest.kt

@@ -15,20 +15,20 @@ import com.bitwarden.fido.Origin
 import com.bitwarden.fido.PublicKeyCredentialAuthenticatorAssertionResponse
 import com.bitwarden.fido.PublicKeyCredentialAuthenticatorAttestationResponse
 import com.bitwarden.fido.UnverifiedAssetLink
+import com.bitwarden.sdk.AuthClient
 import com.bitwarden.sdk.BitwardenException
 import com.bitwarden.sdk.Client
-import com.bitwarden.sdk.AuthClient
 import com.bitwarden.sdk.ClientCiphers
-import com.bitwarden.sdk.CryptoClient
-import com.bitwarden.sdk.ExporterClient
 import com.bitwarden.sdk.ClientFido2
 import com.bitwarden.sdk.ClientFido2Authenticator
 import com.bitwarden.sdk.ClientFido2Client
 import com.bitwarden.sdk.ClientPasswordHistory
+import com.bitwarden.sdk.CryptoClient
+import com.bitwarden.sdk.ExporterClient
+import com.bitwarden.sdk.Fido2CredentialStore
 import com.bitwarden.sdk.PlatformClient
 import com.bitwarden.sdk.SendClient
 import com.bitwarden.sdk.VaultClient
-import com.bitwarden.sdk.Fido2CredentialStore
 import com.bitwarden.send.Send
 import com.bitwarden.send.SendView
 import com.bitwarden.vault.Attachment

app/src/test/java/com/x8bit/bitwarden/ui/auth/feature/newdevicenotice/NewDeviceNoticeEmailAccessViewModelTest.kt

@@ -99,40 +99,43 @@ class NewDeviceNoticeEmailAccessViewModelTest : BaseViewModelTest() {
 
     @Test
     @Suppress("MaxLineLength")
-    fun `ContinueClick should emit NavigateBackToVault if isEmailAccessEnabled and NewDevicePermanentDismiss flag is off`() = runTest {
-        every { featureFlagManager.getFeatureFlag(FlagKey.NewDevicePermanentDismiss) } returns false
+    fun `ContinueClick should emit NavigateBackToVault if isEmailAccessEnabled and NewDevicePermanentDismiss flag is off`() =
+        runTest {
+            every {
+                featureFlagManager.getFeatureFlag(FlagKey.NewDevicePermanentDismiss)
+            } returns false
 
-        val viewModel = createViewModel()
-        viewModel.trySendAction(NewDeviceNoticeEmailAccessAction.EmailAccessToggle(true))
-        viewModel.eventFlow.test {
-            viewModel.trySendAction(NewDeviceNoticeEmailAccessAction.ContinueClick)
-            assertEquals(
-                NewDeviceNoticeEmailAccessEvent.NavigateBackToVault,
-                awaitItem(),
-            )
-            verify(exactly = 1) {
-                authRepository.setNewDeviceNoticeState(
-                    NewDeviceNoticeState(
-                        displayStatus = NewDeviceNoticeDisplayStatus.CAN_ACCESS_EMAIL,
-                        lastSeenDate = null,
-                    ),
+            val viewModel = createViewModel()
+            viewModel.trySendAction(NewDeviceNoticeEmailAccessAction.EmailAccessToggle(true))
+            viewModel.eventFlow.test {
+                viewModel.trySendAction(NewDeviceNoticeEmailAccessAction.ContinueClick)
+                assertEquals(
+                    NewDeviceNoticeEmailAccessEvent.NavigateBackToVault,
+                    awaitItem(),
                 )
+                verify(exactly = 1) {
+                    authRepository.setNewDeviceNoticeState(
+                        NewDeviceNoticeState(
+                            displayStatus = NewDeviceNoticeDisplayStatus.CAN_ACCESS_EMAIL,
+                            lastSeenDate = null,
+                        ),
+                    )
+                }
             }
         }
-    }
 
     @Test
-    @Suppress("MaxLineLength")
-    fun `ContinueClick should emit NavigateToTwoFactorOptions if isEmailAccessEnabled is false`() = runTest {
-        val viewModel = createViewModel()
-        viewModel.eventFlow.test {
-            viewModel.trySendAction(NewDeviceNoticeEmailAccessAction.ContinueClick)
-            assertEquals(
-                NewDeviceNoticeEmailAccessEvent.NavigateToTwoFactorOptions,
-                awaitItem(),
-            )
+    fun `ContinueClick should emit NavigateToTwoFactorOptions if isEmailAccessEnabled is false`() =
+        runTest {
+            val viewModel = createViewModel()
+            viewModel.eventFlow.test {
+                viewModel.trySendAction(NewDeviceNoticeEmailAccessAction.ContinueClick)
+                assertEquals(
+                    NewDeviceNoticeEmailAccessEvent.NavigateToTwoFactorOptions,
+                    awaitItem(),
+                )
+            }
         }
-    }
 
     @Test
     fun `LearnMoreClick should emit NavigateToLearnMore`() =

app/src/test/java/com/x8bit/bitwarden/ui/platform/feature/vaultunlockednavbar/VaultUnlockedNavBarViewModelTest.kt

@@ -248,6 +248,7 @@ class VaultUnlockedNavBarViewModelTest : BaseViewModelTest() {
             viewModel.stateFlow.value,
         )
     }
+
     @Suppress("MaxLineLength")
     @Test
     fun `on init with SendShortcut special circumstance should navigate to the send screen with shortcut event`() =

app/src/test/java/com/x8bit/bitwarden/ui/tools/feature/generator/GeneratorViewModelTest.kt

@@ -2258,15 +2258,15 @@ class GeneratorViewModelTest : BaseViewModelTest() {
     @Test
     fun `StartExploreGeneratorTour action calls first time action manager markCoachMarkTourCompleted called and show coach mark event sent`() =
         runTest {
-        val viewModel = createViewModel()
+            val viewModel = createViewModel()
             viewModel.eventFlow.test {
                 viewModel.trySendAction(GeneratorAction.StartExploreGeneratorTour)
                 assertEquals(GeneratorEvent.StartCoachMarkTour, awaitItem())
             }
-        verify(exactly = 1) {
-            firstTimeActionManager.markCoachMarkTourCompleted(CoachMarkTourType.GENERATOR)
+            verify(exactly = 1) {
+                firstTimeActionManager.markCoachMarkTourCompleted(CoachMarkTourType.GENERATOR)
+            }
         }
-    }
     //region Helper Functions
 
     @Suppress("LongParameterList")

app/src/test/java/com/x8bit/bitwarden/ui/util/ComposeTestHelpers.kt

@@ -65,8 +65,8 @@ fun ComposeContentTestRule.assertNoDialogExists() {
 }
 
 /**
-* Asserts that no popup currently exists.
-*/
+ * Asserts that no popup currently exists.
+ */
 fun ComposeContentTestRule.assertNoPopupExists() {
     this
         .onNode(isPopup())