Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "CHOOSER_UNBUNDLED -> droidfood"

Browse Source 
This reverts commit d2afe4bf2d.

Reason for revert: b/267869885

Change-Id: Ia68d8a71a7278a4efb59ac9094dd4b9cfd5ba722
This commit is contained in:
Sam Dubey
2023-02-06 00:39:35 +00:00
committed by Android (Google) Code Review
parent d2afe4bf2d
commit 70c8493962
2 changed files with 9 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
packages/SystemUI/src/com/android/systemui/flags/Flags.kt
View File

@@ -529,7 +529,7 @@ object Flags {
// 1500 - chooser aka sharesheet
// TODO(b/254512507): Tracking Bug
val CHOOSER_UNBUNDLED = releasedFlag(1500, "chooser_unbundled")
val CHOOSER_UNBUNDLED = unreleasedFlag(1500, "chooser_unbundled", teamfood = true)
// TODO(b/266983432) Tracking Bug
val SHARESHEET_CUSTOM_ACTIONS = unreleasedFlag(1501, "sharesheet_custom_actions")

21
packages/SystemUI/tests/src/com/android/systemui/ChooserSelectorTest.kt
View File

@@ -11,7 +11,6 @@ import com.android.systemui.flags.FeatureFlags
import com.android.systemui.flags.Flag
import com.android.systemui.flags.FlagListenable
import com.android.systemui.flags.Flags
import com.android.systemui.flags.ReleasedFlag
import com.android.systemui.flags.UnreleasedFlag
import com.android.systemui.settings.UserTracker
import com.android.systemui.util.mockito.any
@@ -103,7 +102,7 @@ class ChooserSelectorTest : SysuiTestCase() {
@Test
fun initialize_enablesUnbundledChooser_whenFlagEnabled() {
// Arrange
setFlagMock(true)
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(true)
// Act
chooserSelector.start()
@@ -119,7 +118,7 @@ class ChooserSelectorTest : SysuiTestCase() {
@Test
fun initialize_disablesUnbundledChooser_whenFlagDisabled() {
// Arrange
setFlagMock(false)
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(false)
// Act
chooserSelector.start()
@@ -135,7 +134,7 @@ class ChooserSelectorTest : SysuiTestCase() {
@Test
fun enablesUnbundledChooser_whenFlagBecomesEnabled() {
// Arrange
setFlagMock(false)
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(false)
chooserSelector.start()
verify(mockFeatureFlags).addListener(
eq<Flag<*>>(Flags.CHOOSER_UNBUNDLED),
@@ -148,7 +147,7 @@ class ChooserSelectorTest : SysuiTestCase() {
)
// Act
setFlagMock(true)
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(true)
flagListener.value.onFlagChanged(TestFlagEvent(Flags.CHOOSER_UNBUNDLED.name))
// Assert
@@ -162,7 +161,7 @@ class ChooserSelectorTest : SysuiTestCase() {
@Test
fun disablesUnbundledChooser_whenFlagBecomesDisabled() {
// Arrange
setFlagMock(true)
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(true)
chooserSelector.start()
verify(mockFeatureFlags).addListener(
eq<Flag<*>>(Flags.CHOOSER_UNBUNDLED),
@@ -175,7 +174,7 @@ class ChooserSelectorTest : SysuiTestCase() {
)
// Act
setFlagMock(false)
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(false)
flagListener.value.onFlagChanged(TestFlagEvent(Flags.CHOOSER_UNBUNDLED.name))
// Assert
@@ -189,7 +188,7 @@ class ChooserSelectorTest : SysuiTestCase() {
@Test
fun doesNothing_whenAnotherFlagChanges() {
// Arrange
setFlagMock(false)
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(false)
chooserSelector.start()
verify(mockFeatureFlags).addListener(
eq<Flag<*>>(Flags.CHOOSER_UNBUNDLED),
@@ -198,17 +197,13 @@ class ChooserSelectorTest : SysuiTestCase() {
clearInvocations(mockPackageManager)
// Act
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(false)
flagListener.value.onFlagChanged(TestFlagEvent("other flag"))
// Assert
verifyZeroInteractions(mockPackageManager)
}
private fun setFlagMock(enabled: Boolean) {
whenever(mockFeatureFlags.isEnabled(any<UnreleasedFlag>())).thenReturn(enabled)
whenever(mockFeatureFlags.isEnabled(any<ReleasedFlag>())).thenReturn(enabled)
}
private class TestFlagEvent(override val flagName: String) : FlagListenable.FlagEvent {
override fun requestNoRestart() {}
}