From f3bce239422befb4600a564a5631366d3a9180b8 Mon Sep 17 00:00:00 2001 From: Andrew Haisting <142518658+ahaisting-livefront@users.noreply.github.com> Date: Tue, 8 Oct 2024 15:08:35 -0500 Subject: [PATCH] Use correct feature for syncing with Bitwarden (#230) --- .../authenticator/repository/AuthenticatorRepositoryImpl.kt | 2 +- .../authenticator/repository/AuthenticatorRepositoryTest.kt | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/src/main/kotlin/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryImpl.kt b/app/src/main/kotlin/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryImpl.kt index 7351661a38..13d4004adb 100644 --- a/app/src/main/kotlin/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryImpl.kt +++ b/app/src/main/kotlin/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryImpl.kt @@ -146,7 +146,7 @@ class AuthenticatorRepositoryImpl @Inject constructor( @OptIn(ExperimentalCoroutinesApi::class) override val sharedCodesStateFlow: StateFlow by lazy { - if (!featureFlagManager.getFeatureFlag(LocalFeatureFlag.BitwardenAuthenticationEnabled)) { + if (!featureFlagManager.getFeatureFlag(LocalFeatureFlag.PasswordManagerSync)) { MutableStateFlow(SharedVerificationCodesState.FeatureNotEnabled) } else { authenticatorBridgeManager diff --git a/app/src/test/java/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryTest.kt b/app/src/test/java/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryTest.kt index 31590dd13f..8cb19e14dd 100644 --- a/app/src/test/java/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryTest.kt +++ b/app/src/test/java/com/bitwarden/authenticator/data/authenticator/repository/AuthenticatorRepositoryTest.kt @@ -43,7 +43,7 @@ class AuthenticatorRepositoryTest { private val mockImportManager = mockk() private val mockDispatcherManager = FakeDispatcherManager() private val mockFeatureFlagManager = mockk { - every { getFeatureFlag(LocalFeatureFlag.BitwardenAuthenticationEnabled) } returns true + every { getFeatureFlag(LocalFeatureFlag.PasswordManagerSync) } returns true } private val authenticatorRepository = AuthenticatorRepositoryImpl( @@ -79,7 +79,7 @@ class AuthenticatorRepositoryTest { @Test fun `sharedCodesStateFlow value should be FeatureNotEnabled when feature flag is off`() { every { - mockFeatureFlagManager.getFeatureFlag(LocalFeatureFlag.BitwardenAuthenticationEnabled) + mockFeatureFlagManager.getFeatureFlag(LocalFeatureFlag.PasswordManagerSync) } returns false val repository = AuthenticatorRepositoryImpl( authenticatorDiskSource = fakeAuthenticatorDiskSource, @@ -111,7 +111,7 @@ class AuthenticatorRepositoryTest { @Test fun `sharedCodesStateFlow should emit FeatureNotEnabled when feature flag is off`() = runTest { every { - mockFeatureFlagManager.getFeatureFlag(LocalFeatureFlag.BitwardenAuthenticationEnabled) + mockFeatureFlagManager.getFeatureFlag(LocalFeatureFlag.PasswordManagerSync) } returns false val repository = AuthenticatorRepositoryImpl( authenticatorDiskSource = fakeAuthenticatorDiskSource,