Merge "Revert "Re-adding logic to hide launcher shortcuts when shade is open"" into main (7c840a78) · Commits · e / os / android_frameworks_base

packages/SystemUI/multivalentTests/src/com/android/systemui/keyguard/domain/interactor/KeyguardQuickAffordanceInteractorTest.kt

+0 −26

Original line number	Diff line number	Diff line
		@@ -27,7 +27,6 @@ import com.android.systemui.animation.DialogTransitionAnimator
		import com.android.systemui.common.shared.model.ContentDescription
		import com.android.systemui.common.shared.model.Icon
		import com.android.systemui.coroutines.collectLastValue
		import com.android.systemui.coroutines.collectValues
		import com.android.systemui.dock.DockManager
		import com.android.systemui.dock.DockManagerFake
		import com.android.systemui.flags.FakeFeatureFlags
		@@ -50,7 +49,6 @@ import com.android.systemui.plugins.ActivityStarter
		import com.android.systemui.res.R
		import com.android.systemui.settings.UserFileManager
		import com.android.systemui.settings.UserTracker
		import com.android.systemui.shade.domain.interactor.ShadeInteractor
		import com.android.systemui.shared.keyguard.shared.model.KeyguardQuickAffordanceSlots
		import com.android.systemui.statusbar.policy.KeyguardStateController
		import com.android.systemui.util.FakeSharedPreferences
		@@ -59,7 +57,6 @@ import com.android.systemui.util.mockito.whenever
		import com.android.systemui.util.settings.FakeSettings
		import com.google.common.truth.Truth.assertThat
		import kotlinx.coroutines.ExperimentalCoroutinesApi
		import kotlinx.coroutines.flow.MutableStateFlow
		import kotlinx.coroutines.test.StandardTestDispatcher
		import kotlinx.coroutines.test.TestScope
		import kotlinx.coroutines.test.runCurrent
		@@ -83,7 +80,6 @@ class KeyguardQuickAffordanceInteractorTest : SysuiTestCase() {
		@Mock private lateinit var activityStarter: ActivityStarter
		@Mock private lateinit var launchAnimator: DialogTransitionAnimator
		@Mock private lateinit var devicePolicyManager: DevicePolicyManager
		@Mock private lateinit var shadeInteractor: ShadeInteractor
		@Mock private lateinit var logger: KeyguardQuickAffordancesMetricsLogger

		private lateinit var underTest: KeyguardQuickAffordanceInteractor
		@@ -183,7 +179,6 @@ class KeyguardQuickAffordanceInteractorTest : SysuiTestCase() {
		underTest =
		KeyguardQuickAffordanceInteractor(
		keyguardInteractor = withDeps.keyguardInteractor,
		shadeInteractor = shadeInteractor,
		lockPatternUtils = lockPatternUtils,
		keyguardStateController = keyguardStateController,
		userTracker = userTracker,
		@@ -198,8 +193,6 @@ class KeyguardQuickAffordanceInteractorTest : SysuiTestCase() {
		backgroundDispatcher = testDispatcher,
		appContext = context,
		)

		whenever(shadeInteractor.anyExpansion).thenReturn(MutableStateFlow(0f))
		}

		@Test
		@@ -345,25 +338,6 @@ class KeyguardQuickAffordanceInteractorTest : SysuiTestCase() {
		assertThat(collectedValue()).isEqualTo(KeyguardQuickAffordanceModel.Hidden)
		}

		@Test
		fun quickAffordance_updateOncePerShadeExpansion() =
		testScope.runTest {
		val shadeExpansion = MutableStateFlow(0f)
		whenever(shadeInteractor.anyExpansion).thenReturn(shadeExpansion)

		val collectedValue by
		collectValues(
		underTest.quickAffordance(KeyguardQuickAffordancePosition.BOTTOM_START)
		)

		val initialSize = collectedValue.size
		for (i in 0..10) {
		shadeExpansion.value = i / 10f
		}

		assertThat(collectedValue.size).isEqualTo(initialSize + 1)
		}

		@Test
		fun quickAffordanceAlwaysVisible_notVisible_restrictedByPolicyManager() =
		testScope.runTest {

packages/SystemUI/src/com/android/systemui/keyguard/domain/interactor/KeyguardQuickAffordanceInteractor.kt

+2 −6

Original line number	Diff line number	Diff line
		@@ -46,7 +46,6 @@ import com.android.systemui.keyguard.shared.quickaffordance.KeyguardQuickAfforda
		import com.android.systemui.plugins.ActivityStarter
		import com.android.systemui.res.R
		import com.android.systemui.settings.UserTracker
		import com.android.systemui.shade.domain.interactor.ShadeInteractor
		import com.android.systemui.shared.customization.data.content.CustomizationProviderContract as Contract
		import com.android.systemui.statusbar.phone.SystemUIDialog
		import com.android.systemui.statusbar.policy.KeyguardStateController
		@@ -56,7 +55,6 @@ import kotlinx.coroutines.CoroutineDispatcher
		import kotlinx.coroutines.ExperimentalCoroutinesApi
		import kotlinx.coroutines.flow.Flow
		import kotlinx.coroutines.flow.combine
		import kotlinx.coroutines.flow.distinctUntilChanged
		import kotlinx.coroutines.flow.flatMapLatest
		import kotlinx.coroutines.flow.flowOf
		import kotlinx.coroutines.flow.map
		@@ -68,7 +66,6 @@ class KeyguardQuickAffordanceInteractor
		@Inject
		constructor(
		private val keyguardInteractor: KeyguardInteractor,
		private val shadeInteractor: ShadeInteractor,
		private val lockPatternUtils: LockPatternUtils,
		private val keyguardStateController: KeyguardStateController,
		private val userTracker: UserTracker,
		@@ -103,10 +100,9 @@ constructor(
		quickAffordanceAlwaysVisible(position),
		keyguardInteractor.isDozing,
		keyguardInteractor.isKeyguardShowing,
		shadeInteractor.anyExpansion.map { it < 1.0f }.distinctUntilChanged(),
		biometricSettingsRepository.isCurrentUserInLockdown,
		) { affordance, isDozing, isKeyguardShowing, isQuickSettingsVisible, isUserInLockdown ->
		if (!isDozing && isKeyguardShowing && isQuickSettingsVisible && !isUserInLockdown) {
		) { affordance, isDozing, isKeyguardShowing, isUserInLockdown ->
		if (!isDozing && isKeyguardShowing && !isUserInLockdown) {
		affordance
		} else {
		KeyguardQuickAffordanceModel.Hidden

packages/SystemUI/tests/src/com/android/systemui/keyguard/CustomizationProviderTest.kt

+0 −5

Original line number	Diff line number	Diff line
		@@ -53,11 +53,9 @@ import com.android.systemui.plugins.ActivityStarter
		import com.android.systemui.res.R
		import com.android.systemui.settings.UserFileManager
		import com.android.systemui.settings.UserTracker
		import com.android.systemui.shade.domain.interactor.shadeInteractor
		import com.android.systemui.shared.customization.data.content.CustomizationProviderContract as Contract
		import com.android.systemui.shared.keyguard.shared.model.KeyguardQuickAffordanceSlots
		import com.android.systemui.statusbar.policy.KeyguardStateController
		import com.android.systemui.testKosmos
		import com.android.systemui.util.FakeSharedPreferences
		import com.android.systemui.util.mockito.any
		import com.android.systemui.util.mockito.mock
		@@ -103,8 +101,6 @@ class CustomizationProviderTest : SysuiTestCase() {
		private lateinit var underTest: CustomizationProvider
		private lateinit var testScope: TestScope

		private val kosmos = testKosmos()

		@Before
		fun setUp() {
		MockitoAnnotations.initMocks(this)
		@@ -189,7 +185,6 @@ class CustomizationProviderTest : SysuiTestCase() {
		},
		)
		.keyguardInteractor,
		shadeInteractor = kosmos.shadeInteractor,
		lockPatternUtils = lockPatternUtils,
		keyguardStateController = keyguardStateController,
		userTracker = userTracker,

packages/SystemUI/tests/src/com/android/systemui/keyguard/domain/interactor/KeyguardQuickAffordanceInteractorParameterizedTest.kt

+0 −5

Original line number	Diff line number	Diff line
		@@ -46,9 +46,7 @@ import com.android.systemui.plugins.ActivityStarter
		import com.android.systemui.settings.FakeUserTracker
		import com.android.systemui.settings.UserFileManager
		import com.android.systemui.settings.UserTracker
		import com.android.systemui.shade.domain.interactor.shadeInteractor
		import com.android.systemui.statusbar.policy.KeyguardStateController
		import com.android.systemui.testKosmos
		import com.android.systemui.util.FakeSharedPreferences
		import com.android.systemui.util.mockito.any
		import com.android.systemui.util.mockito.mock
		@@ -244,8 +242,6 @@ class KeyguardQuickAffordanceInteractorParameterizedTest : SysuiTestCase() {
		private lateinit var biometricSettingsRepository: FakeBiometricSettingsRepository
		private lateinit var userTracker: UserTracker

		private val kosmos = testKosmos()

		@Before
		fun setUp() {
		MockitoAnnotations.initMocks(this)
		@@ -315,7 +311,6 @@ class KeyguardQuickAffordanceInteractorParameterizedTest : SysuiTestCase() {
		featureFlags = featureFlags,
		)
		.keyguardInteractor,
		shadeInteractor = kosmos.shadeInteractor,
		lockPatternUtils = lockPatternUtils,
		keyguardStateController = keyguardStateController,
		userTracker = userTracker,

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

+0 −2

Original line number	Diff line number	Diff line
		@@ -54,7 +54,6 @@ import com.android.systemui.plugins.ActivityStarter
		import com.android.systemui.res.R
		import com.android.systemui.settings.UserFileManager
		import com.android.systemui.settings.UserTracker
		import com.android.systemui.shade.domain.interactor.shadeInteractor
		import com.android.systemui.shared.keyguard.shared.model.KeyguardQuickAffordanceSlots
		import com.android.systemui.statusbar.policy.AccessibilityManagerWrapper
		import com.android.systemui.statusbar.policy.KeyguardStateController
		@@ -219,7 +218,6 @@ class KeyguardBottomAreaViewModelTest : SysuiTestCase() {
		quickAffordanceInteractor =
		KeyguardQuickAffordanceInteractor(
		keyguardInteractor = keyguardInteractor,
		shadeInteractor = kosmos.shadeInteractor,
		lockPatternUtils = lockPatternUtils,
		keyguardStateController = keyguardStateController,
		userTracker = userTracker,