Show modes that need setup in dialog (ad971777) · Commits · e / os / android_frameworks_base

packages/SettingsLib/src/com/android/settingslib/notification/modes/TestModeBuilder.java

+10 −0

Original line number	Diff line number	Diff line
		@@ -16,6 +16,9 @@

		package com.android.settingslib.notification.modes;

		import static android.service.notification.ZenModeConfig.UPDATE_ORIGIN_UNKNOWN;
		import static android.service.notification.ZenModeConfig.UPDATE_ORIGIN_USER;

		import android.app.AutomaticZenRule;
		import android.app.NotificationManager;
		import android.content.ComponentName;
		@@ -144,8 +147,15 @@ public class TestModeBuilder {
		}

		public TestModeBuilder setEnabled(boolean enabled) {
		return setEnabled(enabled, /* byUser= */ false);
		}

		public TestModeBuilder setEnabled(boolean enabled, boolean byUser) {
		mRule.setEnabled(enabled);
		mConfigZenRule.enabled = enabled;
		if (!enabled) {
		mConfigZenRule.disabledOrigin = byUser ? UPDATE_ORIGIN_USER : UPDATE_ORIGIN_UNKNOWN;
		}
		return this;
		}

packages/SystemUI/multivalentTests/src/com/android/systemui/statusbar/policy/ui/dialog/viewmodel/ModesDialogViewModelTest.kt

+103 −6

Original line number	Diff line number	Diff line
		@@ -55,13 +55,20 @@ class ModesDialogViewModelTest : SysuiTestCase() {
		ModesDialogViewModel(context, interactor, kosmos.testDispatcher, mockDialogDelegate)

		@Test
		fun tiles_filtersOutDisabledModes() =
		fun tiles_filtersOutUserDisabledModes() =
		testScope.runTest {
		val tiles by collectLastValue(underTest.tiles)

		repository.addModes(
		listOf(
		TestModeBuilder().setName("Disabled").setEnabled(false).build(),
		TestModeBuilder()
		.setName("Disabled by user")
		.setEnabled(false, /* byUser= */ true)
		.build(),
		TestModeBuilder()
		.setName("Disabled by other")
		.setEnabled(false, /* byUser= */ false)
		.build(),
		TestModeBuilder.MANUAL_DND,
		TestModeBuilder()
		.setName("Enabled")
		@@ -70,20 +77,25 @@ class ModesDialogViewModelTest : SysuiTestCase() {
		.build(),
		TestModeBuilder()
		.setName("Disabled with manual")
		.setEnabled(false)
		.setEnabled(false, /* byUser= */ true)
		.setManualInvocationAllowed(true)
		.build(),
		)
		)
		runCurrent()

		assertThat(tiles?.size).isEqualTo(2)
		assertThat(tiles?.size).isEqualTo(3)
		with(tiles?.elementAt(0)!!) {
		assertThat(this.text).isEqualTo("Disabled by other")
		assertThat(this.subtext).isEqualTo("Set up")
		assertThat(this.enabled).isEqualTo(false)
		}
		with(tiles?.elementAt(1)!!) {
		assertThat(this.text).isEqualTo("Manual DND")
		assertThat(this.subtext).isEqualTo("On")
		assertThat(this.enabled).isEqualTo(true)
		}
		with(tiles?.elementAt(1)!!) {
		with(tiles?.elementAt(2)!!) {
		assertThat(this.text).isEqualTo("Enabled")
		assertThat(this.subtext).isEqualTo("Off")
		assertThat(this.enabled).isEqualTo(false)
		@@ -188,7 +200,7 @@ class ModesDialogViewModelTest : SysuiTestCase() {
		assertThat(tiles?.size).isEqualTo(3)
		with(tiles?.elementAt(0)!!) {
		assertThat(this.text).isEqualTo("Active without manual")
		assertThat(this.subtext).isEqualTo("Off")
		assertThat(this.subtext).isEqualTo("Manage in settings")
		assertThat(this.enabled).isEqualTo(false)
		}

		@@ -284,6 +296,91 @@ class ModesDialogViewModelTest : SysuiTestCase() {
		assertThat(tiles?.first()?.enabled).isFalse()
		}

		@Test
		fun onClick_noManualActivation() =
		testScope.runTest {
		val job = Job()
		val tiles by collectLastValue(underTest.tiles, context = job)

		repository.addModes(
		listOf(
		TestModeBuilder()
		.setName("Active without manual")
		.setActive(true)
		.setManualInvocationAllowed(false)
		.build(),
		)
		)
		runCurrent()

		assertThat(tiles?.size).isEqualTo(1)

		// Click tile to toggle it off
		tiles?.elementAt(0)!!.onClick()
		runCurrent()

		assertThat(tiles?.size).isEqualTo(1)
		with(tiles?.elementAt(0)!!) {
		assertThat(this.text).isEqualTo("Active without manual")
		assertThat(this.subtext).isEqualTo("Manage in settings")
		assertThat(this.enabled).isEqualTo(false)

		// Press the tile again
		this.onClick()
		runCurrent()
		}

		// Check that nothing happened
		with(tiles?.elementAt(0)!!) {
		assertThat(this.text).isEqualTo("Active without manual")
		assertThat(this.subtext).isEqualTo("Manage in settings")
		assertThat(this.enabled).isEqualTo(false)
		}
		}

		@Test
		fun onClick_setUp() =
		testScope.runTest {
		val tiles by collectLastValue(underTest.tiles)

		repository.addModes(
		listOf(
		TestModeBuilder()
		.setId("ID")
		.setName("Disabled by other")
		.setEnabled(false, /* byUser= */ false)
		.build(),
		)
		)
		runCurrent()

		assertThat(tiles?.size).isEqualTo(1)
		with(tiles?.elementAt(0)!!) {
		assertThat(this.text).isEqualTo("Disabled by other")
		assertThat(this.subtext).isEqualTo("Set up")
		assertThat(this.enabled).isEqualTo(false)

		// Click the tile
		this.onClick()
		runCurrent()
		}

		// Check that it launched the correct intent
		val intentCaptor = argumentCaptor<Intent>()
		verify(mockDialogDelegate).launchFromDialog(intentCaptor.capture())
		val intent = intentCaptor.lastValue
		assertThat(intent.action).isEqualTo(Settings.ACTION_AUTOMATIC_ZEN_RULE_SETTINGS)
		assertThat(intent.extras?.getString(Settings.EXTRA_AUTOMATIC_ZEN_RULE_ID))
		.isEqualTo("ID")

		// Check that nothing happened to the tile
		with(tiles?.elementAt(0)!!) {
		assertThat(this.text).isEqualTo("Disabled by other")
		assertThat(this.subtext).isEqualTo("Set up")
		assertThat(this.enabled).isEqualTo(false)
		}
		}

		@Test
		fun onLongClick_launchesIntent() =
		testScope.runTest {

packages/SystemUI/res/values/strings.xml

+6 −0

Original line number	Diff line number	Diff line
		@@ -1099,6 +1099,12 @@
		<!-- Priority modes: label for an inactive mode [CHAR LIMIT=35] -->
		<string name="zen_mode_off">Off</string>

		<!-- Priority modes: label for a mode that needs to be set up [CHAR LIMIT=35] -->
		<string name="zen_mode_set_up">Set up</string>

		<!-- Priority modes: label for a mode that cannot be manually turned on [CHAR LIMIT=35] -->
		<string name="zen_mode_no_manual_invocation">Manage in settings</string>

		<!-- Zen mode: Priority only introduction message on first use -->
		<string name="zen_priority_introduction">You won\'t be disturbed by sounds and vibrations, except from alarms, reminders, events, and callers you specify. You\'ll still hear anything you choose to play including music, videos, and games.</string>

packages/SystemUI/src/com/android/systemui/statusbar/policy/ui/dialog/viewmodel/ModesDialogViewModel.kt

+26 −11

Original line number	Diff line number	Diff line
		@@ -66,7 +66,9 @@ constructor(
		// Mode is enabled -> show if active (so user can toggle off), or if it
		// can be manually toggled on
		mode.rule.isEnabled -> mode.isActive \|\| mode.rule.isManualInvocationAllowed
		// TODO(b/346519570): Include modes that have not been set up yet.
		// Mode was created as disabled, or disabled by the app that owns it ->
		// will be shown with a "Set up" text
		!mode.rule.isEnabled -> mode.status == ZenMode.Status.DISABLED_BY_OTHER
		else -> false
		}
		}
		@@ -86,26 +88,39 @@ constructor(
		// "ON: Do Not Disturb, Until Mon 08:09"; see DndTile.
		contentDescription = "",
		onClick = {
		if (mode.isActive) {
		if (!mode.rule.isEnabled) {
		openSettings(mode)
		} else if (mode.isActive) {
		zenModeInteractor.deactivateMode(mode)
		} else {
		if (mode.rule.isManualInvocationAllowed) {
		// TODO(b/346519570): Handle duration for DND mode.
		zenModeInteractor.activateMode(mode)
		}
		}
		},
		onLongClick = {
		onLongClick = { openSettings(mode) }
		)
		}
		}
		.flowOn(bgDispatcher)

		private fun openSettings(mode: ZenMode) {
		val intent: Intent =
		Intent(ACTION_AUTOMATIC_ZEN_RULE_SETTINGS)
		.putExtra(EXTRA_AUTOMATIC_ZEN_RULE_ID, mode.id)

		dialogDelegate.launchFromDialog(intent)
		}
		)

		private fun getTileSubtext(mode: ZenMode): String {
		if (!mode.rule.isEnabled) {
		return context.resources.getString(R.string.zen_mode_set_up)
		}
		if (!mode.rule.isManualInvocationAllowed && !mode.isActive) {
		return context.resources.getString(R.string.zen_mode_no_manual_invocation)
		}
		.flowOn(bgDispatcher)

		private fun getTileSubtext(mode: ZenMode): String {
		// TODO(b/346519570): Use ZenModeConfig.getDescription for manual DND
		val on = context.resources.getString(R.string.zen_mode_on)
		val off = context.resources.getString(R.string.zen_mode_off)
		return mode.rule.triggerDescription ?: if (mode.isActive) on else off