feat: Hide personal to team migration feature if not supported by backend (WPB-12022)

app/src/main/kotlin/com/wire/android/di/accountScoped/UserModule.kt

@@ -33,6 +33,7 @@ import com.wire.kalium.logic.feature.e2ei.usecase.GetUserE2eiCertificatesUseCase
 import com.wire.kalium.logic.feature.e2ei.usecase.IsOtherUserE2EIVerifiedUseCase
 import com.wire.kalium.logic.feature.e2ei.usecase.ObserveCertificateRevocationForSelfClientUseCase
 import com.wire.kalium.logic.feature.featureConfig.FeatureFlagsSyncWorker
+import com.wire.kalium.logic.feature.personaltoteamaccount.CanMigrateFromPersonalToTeamUseCase
 import com.wire.kalium.logic.feature.publicuser.GetAllContactsUseCase
 import com.wire.kalium.logic.feature.publicuser.GetKnownUserUseCase
 import com.wire.kalium.logic.feature.publicuser.RefreshUsersWithoutMetadataUseCase
@@ -240,6 +241,11 @@ class UserModule {
     fun provideFeatureFlagsSyncWorker(userScope: UserScope): FeatureFlagsSyncWorker =
         userScope.featureFlagsSyncWorker
 
+    @ViewModelScoped
+    @Provides
+    fun provideIsPersonalToTeamAccountSupportedByBackendUseCase(userScope: UserScope): CanMigrateFromPersonalToTeamUseCase =
+        userScope.isPersonalToTeamAccountSupportedByBackend
+
     @ViewModelScoped
     @Provides
     fun provideObserveCertificateRevocationForSelfClientUseCase(userScope: UserScope): ObserveCertificateRevocationForSelfClientUseCase =

app/src/main/kotlin/com/wire/android/ui/home/HomeViewModel.kt

@@ -37,6 +37,7 @@ import com.wire.android.util.ui.WireSessionImageLoader
 import com.wire.kalium.logic.feature.client.NeedsToRegisterClientUseCase
 import com.wire.kalium.logic.feature.legalhold.LegalHoldStateForSelfUser
 import com.wire.kalium.logic.feature.legalhold.ObserveLegalHoldStateForSelfUserUseCase
+import com.wire.kalium.logic.feature.personaltoteamaccount.CanMigrateFromPersonalToTeamUseCase
 import com.wire.kalium.logic.feature.user.GetSelfUserUseCase
 import dagger.hilt.android.lifecycle.HiltViewModel
 import kotlinx.coroutines.flow.collectLatest
@@ -52,6 +53,7 @@ class HomeViewModel @Inject constructor(
     private val dataStore: UserDataStore,
     private val getSelf: GetSelfUserUseCase,
     private val needsToRegisterClient: NeedsToRegisterClientUseCase,
+    private val canMigrateFromPersonalToTeam: CanMigrateFromPersonalToTeamUseCase,
     private val observeLegalHoldStatusForSelfUser: ObserveLegalHoldStateForSelfUserUseCase,
     private val wireSessionImageLoader: WireSessionImageLoader,
     private val shouldTriggerMigrationForUser: ShouldTriggerMigrationForUserUserCase,
@@ -80,15 +82,17 @@ class HomeViewModel @Inject constructor(
 
     private fun observeCreateTeamIndicator() {
         viewModelScope.launch {
-            getSelf().first().let { selfUser ->
-                val isPersonalUser = selfUser.teamId == null
-                if (isPersonalUser) {
-                    dataStore.isCreateTeamNoticeRead().collect { isRead ->
-                        homeState = homeState.copy(
-                            shouldShowCreateTeamUnreadIndicator = !isRead
-                        )
-                    }
-                }
+            if (!canMigrateFromPersonalToTeam()) {
+                homeState = homeState.copy(
+                    shouldShowCreateTeamUnreadIndicator = false
+                )
+                return@launch
+            }
+
+            dataStore.isCreateTeamNoticeRead().collect { isRead ->
+                homeState = homeState.copy(
+                    shouldShowCreateTeamUnreadIndicator = !isRead
+                )
             }
         }
     }

app/src/main/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileScreen.kt

@@ -234,7 +234,7 @@ private fun SelfUserProfileContent(
                         .fillMaxHeight()
                         .scrollable(state = scrollState, orientation = Orientation.Vertical)
                 ) {
-                    if (state.teamName == null) {
+                    if (state.isAbleToMigrateToTeamAccount) {
                         stickyHeader {
                             Column(
                                 modifier = Modifier

app/src/main/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileState.kt

@@ -39,6 +39,7 @@ data class SelfUserProfileState(
     val isAvatarLoading: Boolean = false,
     val maxAccountsReached: Boolean = false, // todo. cleanup unused code
     val isReadOnlyAccount: Boolean = true,
+    val isAbleToMigrateToTeamAccount: Boolean = false,
     val isLoggingOut: Boolean = false,
     val legalHoldStatus: LegalHoldUIState = LegalHoldUIState.None,
     val accentId: Int = -1

app/src/main/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileViewModel.kt

@@ -54,6 +54,7 @@ import com.wire.kalium.logic.feature.call.usecase.EndCallUseCase
 import com.wire.kalium.logic.feature.call.usecase.ObserveEstablishedCallsUseCase
 import com.wire.kalium.logic.feature.legalhold.LegalHoldStateForSelfUser
 import com.wire.kalium.logic.feature.legalhold.ObserveLegalHoldStateForSelfUserUseCase
+import com.wire.kalium.logic.feature.personaltoteamaccount.CanMigrateFromPersonalToTeamUseCase
 import com.wire.kalium.logic.feature.team.GetUpdatedSelfTeamUseCase
 import com.wire.kalium.logic.feature.user.GetSelfUserUseCase
 import com.wire.kalium.logic.feature.user.IsReadOnlyAccountUseCase
@@ -84,6 +85,7 @@ class SelfUserProfileViewModel @Inject constructor(
     private val dataStore: UserDataStore,
     private val getSelf: GetSelfUserUseCase,
     private val getSelfTeam: GetUpdatedSelfTeamUseCase,
+    private val isPersonalToTeamAccountSupportedByBackend: CanMigrateFromPersonalToTeamUseCase,
     private val observeValidAccounts: ObserveValidAccountsUseCase,
     private val updateStatus: UpdateSelfAvailabilityStatusUseCase,
     private val logout: LogoutUseCase,
@@ -109,13 +111,19 @@ class SelfUserProfileViewModel @Inject constructor(
     init {
         viewModelScope.launch {
             fetchSelfUser()
+            checkIfUserAbleToMigrateToTeamAccount()
             observeEstablishedCall()
             fetchIsReadOnlyAccount()
             observeLegalHoldStatus()
             markCreateTeamNoticeAsRead()
         }
     }
 
+    private suspend fun checkIfUserAbleToMigrateToTeamAccount() {
+        val isAbleToMigrateToTeamAccount = isPersonalToTeamAccountSupportedByBackend() && userProfileState.teamName.isNullOrBlank()
+        userProfileState = userProfileState.copy(isAbleToMigrateToTeamAccount = isAbleToMigrateToTeamAccount)
+    }
+
     private suspend fun fetchIsReadOnlyAccount() {
         val isReadOnlyAccount = isReadOnlyAccount()
         userProfileState = userProfileState.copy(isReadOnlyAccount = isReadOnlyAccount)

app/src/test/kotlin/com/wire/android/ui/home/HomeViewModelTest.kt

@@ -31,6 +31,7 @@ import com.wire.kalium.logic.data.user.UserAvailabilityStatus
 import com.wire.kalium.logic.feature.client.NeedsToRegisterClientUseCase
 import com.wire.kalium.logic.feature.legalhold.LegalHoldStateForSelfUser
 import com.wire.kalium.logic.feature.legalhold.ObserveLegalHoldStateForSelfUserUseCase
+import com.wire.kalium.logic.feature.personaltoteamaccount.CanMigrateFromPersonalToTeamUseCase
 import com.wire.kalium.logic.feature.user.GetSelfUserUseCase
 import io.mockk.MockKAnnotations
 import io.mockk.coEvery
@@ -143,6 +144,9 @@ class HomeViewModelTest {
         @MockK
         lateinit var analyticsManager: AnonymousAnalyticsManager
 
+        @MockK
+        lateinit var canMigrateFromPersonalToTeam: CanMigrateFromPersonalToTeamUseCase
+
         private val viewModel by lazy {
             HomeViewModel(
                 savedStateHandle = savedStateHandle,
@@ -153,19 +157,25 @@ class HomeViewModelTest {
                 observeLegalHoldStatusForSelfUser = observeLegalHoldStatusForSelfUser,
                 wireSessionImageLoader = wireSessionImageLoader,
                 shouldTriggerMigrationForUser = shouldTriggerMigrationForUser,
-                analyticsManager = analyticsManager
+                analyticsManager = analyticsManager,
+                isPersonalToTeamAccountSupportedByBackend = canMigrateFromPersonalToTeam
             )
         }
 
         init {
             MockKAnnotations.init(this, relaxUnitFun = true)
             withGetSelf(flowOf(TestUser.SELF_USER))
+            withCanMigrateFromPersonalToTeamReturning(true)
         }
 
         fun withGetSelf(result: Flow<SelfUser>) = apply {
             coEvery { getSelf.invoke() } returns result
         }
 
+        private fun withCanMigrateFromPersonalToTeamReturning(result: Boolean) = apply {
+            coEvery { canMigrateFromPersonalToTeam.invoke() } returns result
+        }
+
         fun withLegalHoldStatus(result: Flow<LegalHoldStateForSelfUser>) = apply {
             coEvery { observeLegalHoldStatusForSelfUser.invoke() } returns result
         }

app/src/test/kotlin/com/wire/android/ui/userprofile/self/SelfUserProfileViewModelArrangement.kt

@@ -35,6 +35,7 @@ import com.wire.kalium.logic.feature.call.usecase.EndCallUseCase
 import com.wire.kalium.logic.feature.call.usecase.ObserveEstablishedCallsUseCase
 import com.wire.kalium.logic.feature.legalhold.LegalHoldStateForSelfUser
 import com.wire.kalium.logic.feature.legalhold.ObserveLegalHoldStateForSelfUserUseCase
+import com.wire.kalium.logic.feature.personaltoteamaccount.CanMigrateFromPersonalToTeamUseCase
 import com.wire.kalium.logic.feature.team.GetUpdatedSelfTeamUseCase
 import com.wire.kalium.logic.feature.user.GetSelfUserUseCase
 import com.wire.kalium.logic.feature.user.IsReadOnlyAccountUseCase
@@ -101,6 +102,9 @@ class SelfUserProfileViewModelArrangement {
     @MockK
     lateinit var anonymousAnalyticsManager: AnonymousAnalyticsManager
 
+    @MockK
+    lateinit var isPersonalToTeamAccountSupportedByBackend: CanMigrateFromPersonalToTeamUseCase
+
     private val viewModel by lazy {
         SelfUserProfileViewModel(
             selfUserId = TestUser.SELF_USER.id,
@@ -121,7 +125,8 @@ class SelfUserProfileViewModelArrangement {
             notificationManager = notificationManager,
             globalDataStore = globalDataStore,
             qualifiedIdMapper = qualifiedIdMapper,
-            anonymousAnalyticsManager = anonymousAnalyticsManager
+            anonymousAnalyticsManager = anonymousAnalyticsManager,
+            isPersonalToTeamAccountSupportedByBackend = isPersonalToTeamAccountSupportedByBackend
         )
     }