fix: disable bubbles feature flag (#4307)

(cherry picked from commit 34c4f5e)

Author: Garzas

app/src/main/kotlin/com/wire/android/ui/home/conversations/info/ConversationInfoViewModel.kt

@@ -39,7 +39,6 @@ import com.wire.kalium.logic.data.id.QualifiedID
 import com.wire.kalium.logic.data.id.QualifiedIdMapper
 import com.wire.kalium.logic.data.user.ConnectionState
 import com.wire.kalium.logic.data.user.UserId
-import com.wire.kalium.logic.feature.client.IsChatBubblesEnabledUseCase
 import com.wire.kalium.logic.feature.client.IsWireCellsEnabledUseCase
 import com.wire.kalium.logic.feature.conversation.ObserveConversationDetailsUseCase
 import com.wire.kalium.logic.feature.e2ei.usecase.FetchConversationMLSVerificationStatusUseCase
@@ -57,7 +56,7 @@ class ConversationInfoViewModel @Inject constructor(
     private val globalDataStore: GlobalDataStore,
     private val fetchConversationMLSVerificationStatus: FetchConversationMLSVerificationStatusUseCase,
     private val isWireCellFeatureEnabled: IsWireCellsEnabledUseCase,
-    private val isChatBubblesEnabledUseCase: IsChatBubblesEnabledUseCase,
+//    private val isChatBubblesEnabledUseCase: IsChatBubblesEnabledUseCase,
     @CurrentAccount private val selfUserId: UserId,
 ) : ViewModel() {
 
@@ -130,7 +129,8 @@ class ConversationInfoViewModel @Inject constructor(
         )
     }
 
-    private suspend fun isBubbleUiEnabled() = (globalDataStore.observeIsBubbleUI().firstOrNull() ?: false) || isChatBubblesEnabledUseCase()
+    private suspend fun isBubbleUiEnabled() = (globalDataStore.observeIsBubbleUI().firstOrNull() ?: false)
+//            || isChatBubblesEnabledUseCase() // TODO uncomment when production ready
 
     private fun getAccentId(conversationDetails: ConversationDetails): Int {
         return if (conversationDetails is ConversationDetails.OneOne) {

app/src/test/kotlin/com/wire/android/ui/home/conversations/info/ConversationInfoViewModelArrangement.kt

@@ -30,7 +30,6 @@ import com.wire.kalium.logic.data.id.ConversationId
 import com.wire.kalium.logic.data.id.QualifiedID
 import com.wire.kalium.logic.data.id.QualifiedIdMapper
 import com.wire.kalium.logic.data.user.UserId
-import com.wire.kalium.logic.feature.client.IsChatBubblesEnabledUseCase
 import com.wire.kalium.logic.feature.client.IsWireCellsEnabledUseCase
 import com.wire.kalium.logic.feature.conversation.ObserveConversationDetailsUseCase
 import com.wire.kalium.logic.feature.e2ei.usecase.FetchConversationMLSVerificationStatusUseCase
@@ -67,8 +66,8 @@ class ConversationInfoViewModelArrangement {
     @MockK
     lateinit var globalDataStore: GlobalDataStore
 
-    @MockK
-    lateinit var isChatBubblesEnabled: IsChatBubblesEnabledUseCase
+//    @MockK
+//    lateinit var isChatBubblesEnabled: IsChatBubblesEnabledUseCase
 
     private val viewModel: ConversationInfoViewModel by lazy {
         ConversationInfoViewModel(
@@ -79,7 +78,7 @@ class ConversationInfoViewModelArrangement {
             selfUserId = TestUser.SELF_USER_ID,
             isWireCellFeatureEnabled = isCellsEnabled,
             globalDataStore = globalDataStore,
-            isChatBubblesEnabledUseCase = isChatBubblesEnabled
+//            isChatBubblesEnabledUseCase = isChatBubblesEnabled
         )
     }
 
@@ -97,7 +96,7 @@ class ConversationInfoViewModelArrangement {
         coEvery { fetchConversationMLSVerificationStatus.invoke(any()) } returns Unit
         coEvery { isCellsEnabled() } returns false
         coEvery { globalDataStore.observeIsBubbleUI() } returns flowOf(false)
-        coEvery { isChatBubblesEnabled() } returns false
+//        coEvery { isChatBubblesEnabled() } returns false
     }
 
     suspend fun withConversationDetailUpdate(conversationDetails: ConversationDetails) = apply {