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

Commit 2c3594e6 authored by Bryce Lee's avatar Bryce Lee Committed by Beverly Tai
Browse files

Revert "Reset dismissAction and notifyCancel dismissCallbacks when altBouncer removed" 

Bug: 351712715

This reverts commit 48bb658f.

Reason for revert: Causing dismiss action to be cleared when going from fingerprint to pin/pattern auth.

Change-Id: I87ba6a8814a65ffd3ac2eabc59b3bbdc58c0e4e1
parent 48bb658f

packages/SystemUI/src/com/android/systemui/keyguard/ui/viewmodel/AlternateBouncerViewModel.kt

+0 −5
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ package com.android.systemui.keyguard.ui.viewmodel 


import android.graphics.Color

import com.android.systemui.bouncer.domain.interactor.AlternateBouncerInteractor

import com.android.systemui.bouncer.domain.interactor.PrimaryBouncerInteractor

import com.android.systemui.keyguard.DismissCallbackRegistry

import com.android.systemui.keyguard.domain.interactor.KeyguardTransitionInteractor

import com.android.systemui.keyguard.shared.model.KeyguardState.ALTERNATE_BOUNCER
@@ -42,7 +41,6 @@ constructor( 
    keyguardTransitionInteractor: KeyguardTransitionInteractor,

    private val dismissCallbackRegistry: DismissCallbackRegistry,

    alternateBouncerInteractor: Lazy<AlternateBouncerInteractor>,

    private val primaryBouncerInteractor: PrimaryBouncerInteractor,

) {

    // When we're fully transitioned to the AlternateBouncer, the alpha of the scrim should be:

    private val alternateBouncerScrimAlpha = .66f
@@ -70,13 +68,10 @@ constructor( 


    fun onRemovedFromWindow() {

        statusBarKeyguardViewManager.hideAlternateBouncer(false)

        primaryBouncerInteractor.setDismissAction(null, null)

        dismissCallbackRegistry.notifyDismissCancelled()

    }



    fun onBackRequested() {

        statusBarKeyguardViewManager.hideAlternateBouncer(false)

        primaryBouncerInteractor.setDismissAction(null, null)

        dismissCallbackRegistry.notifyDismissCancelled()

    }

}

packages/SystemUI/tests/src/com/android/systemui/keyguard/ui/viewmodel/AlternateBouncerViewModelTest.kt

+0 −21
Original line number Diff line number Diff line
@@ -21,7 +21,6 @@ import androidx.test.ext.junit.runners.AndroidJUnit4 
import androidx.test.filters.SmallTest

import com.android.internal.policy.IKeyguardDismissCallback

import com.android.systemui.SysuiTestCase

import com.android.systemui.bouncer.domain.interactor.primaryBouncerInteractor

import com.android.systemui.concurrency.fakeExecutor

import com.android.systemui.coroutines.collectLastValue

import com.android.systemui.coroutines.collectValues
@@ -31,7 +30,6 @@ import com.android.systemui.keyguard.shared.model.KeyguardState 
import com.android.systemui.keyguard.shared.model.TransitionState

import com.android.systemui.keyguard.shared.model.TransitionStep

import com.android.systemui.kosmos.testScope

import com.android.systemui.plugins.ActivityStarter

import com.android.systemui.statusbar.phone.statusBarKeyguardViewManager

import com.android.systemui.testKosmos

import com.android.systemui.util.mockito.any
@@ -64,31 +62,13 @@ class AlternateBouncerViewModelTest : SysuiTestCase() { 
    @Test

    fun onRemovedFromWindow() =

        testScope.runTest {

            kosmos.primaryBouncerInteractor.setDismissAction(

                mock(ActivityStarter.OnDismissAction::class.java),

                {},

            )

            assertThat(kosmos.primaryBouncerInteractor.bouncerDismissAction).isNotNull()



            val dismissCallback = mock(IKeyguardDismissCallback::class.java)

            kosmos.dismissCallbackRegistry.addCallback(dismissCallback)

            underTest.onRemovedFromWindow()



            kosmos.fakeExecutor.runAllReady()

            verify(statusBarKeyguardViewManager).hideAlternateBouncer(any())

            verify(dismissCallback).onDismissCancelled()

            assertThat(kosmos.primaryBouncerInteractor.bouncerDismissAction).isNull()

        }



    @Test

    fun onBackRequested() =

        testScope.runTest {

            kosmos.primaryBouncerInteractor.setDismissAction(

                mock(ActivityStarter.OnDismissAction::class.java),

                {},

            )

            assertThat(kosmos.primaryBouncerInteractor.bouncerDismissAction).isNotNull()



            val dismissCallback = mock(IKeyguardDismissCallback::class.java)

            kosmos.dismissCallbackRegistry.addCallback(dismissCallback)
@@ -96,7 +76,6 @@ class AlternateBouncerViewModelTest : SysuiTestCase() { 
            kosmos.fakeExecutor.runAllReady()

            verify(statusBarKeyguardViewManager).hideAlternateBouncer(any())

            verify(dismissCallback).onDismissCancelled()

            assertThat(kosmos.primaryBouncerInteractor.bouncerDismissAction).isNull()

        }



    @Test

packages/SystemUI/tests/utils/src/com/android/systemui/keyguard/ui/viewmodel/AlternateBouncerViewModelKosmos.kt

+0 −2
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ 
package com.android.systemui.keyguard.ui.viewmodel



import com.android.systemui.bouncer.domain.interactor.alternateBouncerInteractor

import com.android.systemui.bouncer.domain.interactor.primaryBouncerInteractor

import com.android.systemui.keyguard.dismissCallbackRegistry

import com.android.systemui.keyguard.domain.interactor.keyguardTransitionInteractor

import com.android.systemui.kosmos.Kosmos
@@ -33,6 +32,5 @@ val Kosmos.alternateBouncerViewModel by Fixture { 
        keyguardTransitionInteractor = keyguardTransitionInteractor,

        dismissCallbackRegistry = dismissCallbackRegistry,

        alternateBouncerInteractor = { alternateBouncerInteractor },

        primaryBouncerInteractor = primaryBouncerInteractor,

    )

}