Try fix "The firmware update button is constantly active until the ap…

…plication is restarted" (#716)

**Background**

Right now we have problem with infinite update button active

**Changes**

- Reinvalidate update button when connection state changed

**Test plan**

Try update flipper

CHANGELOG.md

@@ -28,6 +28,7 @@
 - [FIX] Openning animation
 - [FIX] Swipe refresh modifier use in root level
 - [FIX] Fix overflow in key small
+- [FIX] Try fix "The firmware update button is constantly active until the application is restarted"
 
 # 1.6.4 
 

components/updater/card/src/main/java/com/flipperdevices/updater/card/helpers/UpdateOfferProvider.kt

@@ -6,6 +6,7 @@ import com.flipperdevices.updater.card.helpers.delegates.UpdateOfferDelegate
 import com.squareup.anvil.annotations.ContributesBinding
 import kotlinx.coroutines.flow.Flow
 import kotlinx.coroutines.flow.combine
+import kotlinx.coroutines.flow.flatMapLatest
 import javax.inject.Inject
 
 interface UpdateOfferProviderApi {
@@ -18,10 +19,12 @@ class UpdateOfferProvider @Inject constructor(
 ) : UpdateOfferProviderApi {
 
     override fun isUpdateRequire(serviceApi: FlipperServiceApi): Flow<Boolean> {
-        return combine(
-            delegates.map { it.isRequire(serviceApi) }
-        ) { delegate ->
-            return@combine delegate.any { it }
+        return serviceApi.connectionInformationApi.getConnectionStateFlow().flatMapLatest {
+            combine(
+                delegates.map { it.isRequire(serviceApi) }
+            ) { delegate ->
+                return@combine delegate.any { it }
+            }
         }
     }
 }

components/updater/card/src/test/kotlin/com/flipperdevices/updater/card/helpers/UpdateOfferProviderTest.kt

@@ -1,5 +1,8 @@
 package com.flipperdevices.updater.card.helpers
 
+import com.flipperdevices.bridge.api.manager.delegates.FlipperConnectionInformationApi
+import com.flipperdevices.bridge.api.manager.ktx.state.ConnectionState
+import com.flipperdevices.bridge.api.manager.ktx.state.FlipperSupportedState
 import com.flipperdevices.bridge.service.api.FlipperServiceApi
 import com.flipperdevices.updater.card.helpers.delegates.UpdateOfferDebugFlagAlways
 import com.flipperdevices.updater.card.helpers.delegates.UpdateOfferFlipperManifest
@@ -34,6 +37,11 @@ class UpdateOfferProviderTest {
         every { delegateFlagAlways.isRequire(serviceApi) } returns flowOf(false)
         every { delegateManifest.isRequire(serviceApi) } returns flowOf(false)
         every { delegateRegionFile.isRequire(serviceApi) } returns flowOf(false)
+        val connectionInformationApi: FlipperConnectionInformationApi = mockk {
+            every { getConnectionStateFlow() } returns
+                flowOf(ConnectionState.Ready(supportedState = FlipperSupportedState.READY))
+        }
+        every { serviceApi.connectionInformationApi } returns connectionInformationApi
     }
 
     @Test

components/updater/impl/src/main/java/com/flipperdevices/updater/impl/api/UpdaterApiImpl.kt

@@ -44,7 +44,7 @@ class UpdaterApiImpl @Inject constructor(
 ) : UpdaterApi, LogTagProvider {
     override val TAG = "UpdaterApi"
 
-    private val updatingState = MutableStateFlow<UpdatingStateWithRequest>(
+    private val updatingState = MutableStateFlow(
         UpdatingStateWithRequest(UpdatingState.NotStarted, request = null)
     )