Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e7f2e5bc authored by Caitlin Shkuratov's avatar Caitlin Shkuratov Committed by Android (Google) Code Review
Browse files

Merge "[SB Refactor] Use the calculated mobileIsDefault for failed connection." into udc-dev

parents d5b4b9b9 970a8f60

packages/SystemUI/src/com/android/systemui/statusbar/pipeline/mobile/domain/interactor/MobileIconsInteractor.kt

+1 −1
Original line number Diff line number Diff line
@@ -259,7 +259,7 @@ constructor( 
     */

    override val isDefaultConnectionFailed: StateFlow<Boolean> =

        combine(

                mobileConnectionsRepo.mobileIsDefault,

                mobileIsDefault,

                mobileConnectionsRepo.defaultConnectionIsValidated,

                forcingCellularValidation,

            ) { mobileIsDefault, defaultConnectionIsValidated, forcingCellularValidation ->

packages/SystemUI/tests/src/com/android/systemui/statusbar/pipeline/mobile/domain/interactor/MobileIconsInteractorTest.kt

+15 −0
Original line number Diff line number Diff line
@@ -361,6 +361,21 @@ class MobileIconsInteractorTest : SysuiTestCase() { 
            job.cancel()

        }



    @Test

    fun failedConnection_carrierMergedDefault_notValidated_failed() =

        testScope.runTest {

            var latest: Boolean? = null

            val job = underTest.isDefaultConnectionFailed.onEach { latest = it }.launchIn(this)



            connectionsRepository.hasCarrierMergedConnection.value = true

            connectionsRepository.defaultConnectionIsValidated.value = false

            yield()



            assertThat(latest).isTrue()



            job.cancel()

        }



    /** Regression test for b/275076959. */

    @Test

    fun failedConnection_dataSwitchInSameGroup_notFailed() =