Update usages of TestModeBuilder

The API of the builder was changed a bit to clean up some things and
make others more explicit. Update tests in settingslib accordingly.

Bug: 382497909
Flag: TEST_ONLY
Test: this
Change-Id: Ib31991d61e7b232c8a2befda92537d7b388beb5b
This commit is contained in:
Ioana Alexandru
2025-01-03 19:55:24 +01:00
parent b5b9402925
commit 0bb0d0c402
12 changed files with 47 additions and 29 deletions

View File

@@ -71,7 +71,7 @@ public final class InterruptionFilterPreferenceControllerTest {
@Test @Test
public void updateState_dnd_enabled() { public void updateState_dnd_enabled() {
TwoStatePreference preference = mock(TwoStatePreference.class); TwoStatePreference preference = mock(TwoStatePreference.class);
ZenMode dnd = TestModeBuilder.MANUAL_DND_ACTIVE; ZenMode dnd = TestModeBuilder.MANUAL_DND;
mController.updateState(preference, dnd); mController.updateState(preference, dnd);
@@ -81,7 +81,11 @@ public final class InterruptionFilterPreferenceControllerTest {
@Test @Test
public void updateState_specialDnd_disabled() { public void updateState_specialDnd_disabled() {
TwoStatePreference preference = mock(TwoStatePreference.class); TwoStatePreference preference = mock(TwoStatePreference.class);
ZenMode specialDnd = TestModeBuilder.manualDnd(INTERRUPTION_FILTER_NONE, true); ZenMode specialDnd = new TestModeBuilder()
.makeManualDnd()
.setInterruptionFilter(INTERRUPTION_FILTER_NONE)
.setActive(true)
.build();
mController.updateState(preference, specialDnd); mController.updateState(preference, specialDnd);

View File

@@ -70,8 +70,7 @@ public class ManualDurationPreferenceControllerTest {
@Test @Test
public void testIsAvailable_onlyForManualDnd() { public void testIsAvailable_onlyForManualDnd() {
assertThat(mPrefController.isAvailable(TestModeBuilder.EXAMPLE)).isFalse(); assertThat(mPrefController.isAvailable(TestModeBuilder.EXAMPLE)).isFalse();
assertThat(mPrefController.isAvailable(TestModeBuilder.MANUAL_DND_ACTIVE)).isTrue(); assertThat(mPrefController.isAvailable(TestModeBuilder.MANUAL_DND)).isTrue();
assertThat(mPrefController.isAvailable(TestModeBuilder.MANUAL_DND_INACTIVE)).isTrue();
} }
@Test @Test

View File

@@ -63,7 +63,6 @@ import com.android.settingslib.notification.modes.TestModeBuilder;
import com.android.settingslib.notification.modes.ZenMode; import com.android.settingslib.notification.modes.ZenMode;
import com.android.settingslib.notification.modes.ZenModesBackend; import com.android.settingslib.notification.modes.ZenModesBackend;
import com.google.common.collect.ImmutableList;
import com.google.common.util.concurrent.MoreExecutors; import com.google.common.util.concurrent.MoreExecutors;
import org.junit.Before; import org.junit.Before;
@@ -153,14 +152,18 @@ public final class ZenModeAppsLinkPreferenceControllerTest {
@Test @Test
public void updateState_dnd_enabled() { public void updateState_dnd_enabled() {
ZenMode dnd = TestModeBuilder.MANUAL_DND_ACTIVE; ZenMode dnd = TestModeBuilder.MANUAL_DND;
mController.updateState(mPreference, dnd); mController.updateState(mPreference, dnd);
assertThat(mPreference.isEnabled()).isTrue(); assertThat(mPreference.isEnabled()).isTrue();
} }
@Test @Test
public void updateState_specialDnd_disabled() { public void updateState_specialDnd_disabled() {
ZenMode specialDnd = TestModeBuilder.manualDnd(INTERRUPTION_FILTER_NONE, true); ZenMode specialDnd = new TestModeBuilder()
.makeManualDnd()
.setInterruptionFilter(INTERRUPTION_FILTER_NONE)
.setActive(true)
.build();
mController.updateState(mPreference, specialDnd); mController.updateState(mPreference, specialDnd);
assertThat(mPreference.isEnabled()).isFalse(); assertThat(mPreference.isEnabled()).isFalse();
} }

View File

@@ -190,7 +190,7 @@ public final class ZenModeButtonPreferenceControllerTest {
Button button = new Button(mContext); Button button = new Button(mContext);
LayoutPreference pref = mock(LayoutPreference.class); LayoutPreference pref = mock(LayoutPreference.class);
when(pref.findViewById(anyInt())).thenReturn(button); when(pref.findViewById(anyInt())).thenReturn(button);
ZenMode zenMode = TestModeBuilder.MANUAL_DND_INACTIVE; ZenMode zenMode = TestModeBuilder.MANUAL_DND;
mController.updateZenMode(pref, zenMode); mController.updateZenMode(pref, zenMode);
button.callOnClick(); button.callOnClick();
@@ -204,7 +204,7 @@ public final class ZenModeButtonPreferenceControllerTest {
Button button = new Button(mContext); Button button = new Button(mContext);
LayoutPreference pref = mock(LayoutPreference.class); LayoutPreference pref = mock(LayoutPreference.class);
when(pref.findViewById(anyInt())).thenReturn(button); when(pref.findViewById(anyInt())).thenReturn(button);
ZenMode zenMode = TestModeBuilder.MANUAL_DND_INACTIVE; ZenMode zenMode = TestModeBuilder.MANUAL_DND;
mController.updateZenMode(pref, zenMode); mController.updateZenMode(pref, zenMode);
button.callOnClick(); button.callOnClick();

View File

@@ -68,7 +68,7 @@ public final class ZenModeDisplayLinkPreferenceControllerTest {
@Test @Test
public void updateState_dnd_enabled() { public void updateState_dnd_enabled() {
Preference preference = mock(Preference.class); Preference preference = mock(Preference.class);
ZenMode dnd = TestModeBuilder.MANUAL_DND_ACTIVE; ZenMode dnd = TestModeBuilder.MANUAL_DND;
mController.updateState(preference, dnd); mController.updateState(preference, dnd);
@@ -78,7 +78,11 @@ public final class ZenModeDisplayLinkPreferenceControllerTest {
@Test @Test
public void updateState_specialDnd_disabled() { public void updateState_specialDnd_disabled() {
Preference preference = mock(Preference.class); Preference preference = mock(Preference.class);
ZenMode specialDnd = TestModeBuilder.manualDnd(INTERRUPTION_FILTER_NONE, true); ZenMode specialDnd = new TestModeBuilder()
.makeManualDnd()
.setInterruptionFilter(INTERRUPTION_FILTER_NONE)
.setActive(true)
.build();
mController.updateState(preference, specialDnd); mController.updateState(preference, specialDnd);

View File

@@ -66,7 +66,7 @@ public final class ZenModeOtherLinkPreferenceControllerTest {
@Test @Test
public void updateState_dnd_enabled() { public void updateState_dnd_enabled() {
CircularIconsPreference preference = mock(CircularIconsPreference.class); CircularIconsPreference preference = mock(CircularIconsPreference.class);
ZenMode dnd = TestModeBuilder.MANUAL_DND_ACTIVE; ZenMode dnd = TestModeBuilder.MANUAL_DND;
mController.updateState(preference, dnd); mController.updateState(preference, dnd);
@@ -76,7 +76,11 @@ public final class ZenModeOtherLinkPreferenceControllerTest {
@Test @Test
public void updateState_specialDnd_disabled() { public void updateState_specialDnd_disabled() {
CircularIconsPreference preference = mock(CircularIconsPreference.class); CircularIconsPreference preference = mock(CircularIconsPreference.class);
ZenMode specialDnd = TestModeBuilder.manualDnd(INTERRUPTION_FILTER_NONE, true); ZenMode specialDnd = new TestModeBuilder()
.makeManualDnd()
.setInterruptionFilter(INTERRUPTION_FILTER_NONE)
.setActive(true)
.build();
mController.updateState(preference, specialDnd); mController.updateState(preference, specialDnd);

View File

@@ -120,14 +120,18 @@ public final class ZenModePeopleLinkPreferenceControllerTest {
@Test @Test
public void updateState_dnd_enabled() { public void updateState_dnd_enabled() {
ZenMode dnd = TestModeBuilder.MANUAL_DND_ACTIVE; ZenMode dnd = TestModeBuilder.MANUAL_DND;
mController.updateState(mPreference, dnd); mController.updateState(mPreference, dnd);
assertThat(mPreference.isEnabled()).isTrue(); assertThat(mPreference.isEnabled()).isTrue();
} }
@Test @Test
public void updateState_specialDnd_disabled() { public void updateState_specialDnd_disabled() {
ZenMode specialDnd = TestModeBuilder.manualDnd(INTERRUPTION_FILTER_NONE, true); ZenMode specialDnd = new TestModeBuilder()
.makeManualDnd()
.setInterruptionFilter(INTERRUPTION_FILTER_NONE)
.setActive(true)
.build();
mController.updateState(mPreference, specialDnd); mController.updateState(mPreference, specialDnd);
assertThat(mPreference.isEnabled()).isFalse(); assertThat(mPreference.isEnabled()).isFalse();
} }

View File

@@ -529,7 +529,7 @@ public class ZenModeSummaryHelperTest {
@Test @Test
public void getModesSummary_excludesImplicitModes() { public void getModesSummary_excludesImplicitModes() {
ImmutableList<ZenMode> modes = ImmutableList.of( ImmutableList<ZenMode> modes = ImmutableList.of(
TestModeBuilder.MANUAL_DND_INACTIVE, TestModeBuilder.MANUAL_DND,
new TestModeBuilder().implicitForPackage("com.annoying.one").build(), new TestModeBuilder().implicitForPackage("com.annoying.one").build(),
new TestModeBuilder().setName("Chirping").build() new TestModeBuilder().setName("Chirping").build()
); );
@@ -541,7 +541,7 @@ public class ZenModeSummaryHelperTest {
@Test @Test
public void getModesSummary_oneModeActive_listsActiveMode() { public void getModesSummary_oneModeActive_listsActiveMode() {
ImmutableList<ZenMode> modes = ImmutableList.of( ImmutableList<ZenMode> modes = ImmutableList.of(
TestModeBuilder.MANUAL_DND_ACTIVE, new TestModeBuilder().makeManualDnd().setActive(true).build(),
new TestModeBuilder().setName("Inactive").setActive(false).build()); new TestModeBuilder().setName("Inactive").setActive(false).build());
String summary = mSummaryHelper.getModesSummary(modes); String summary = mSummaryHelper.getModesSummary(modes);
@@ -551,7 +551,7 @@ public class ZenModeSummaryHelperTest {
@Test @Test
public void getModesSummary_twoModesActive_listsActiveModes() { public void getModesSummary_twoModesActive_listsActiveModes() {
ImmutableList<ZenMode> modes = ImmutableList.of( ImmutableList<ZenMode> modes = ImmutableList.of(
TestModeBuilder.MANUAL_DND_ACTIVE, new TestModeBuilder().makeManualDnd().setActive(true).build(),
new TestModeBuilder().setName("Inactive").setActive(false).build(), new TestModeBuilder().setName("Inactive").setActive(false).build(),
new TestModeBuilder().setName("Active #1").setActive(true).build()); new TestModeBuilder().setName("Active #1").setActive(true).build());
@@ -562,7 +562,7 @@ public class ZenModeSummaryHelperTest {
@Test @Test
public void getModesSummary_threeModesActive_listsActiveModes() { public void getModesSummary_threeModesActive_listsActiveModes() {
ImmutableList<ZenMode> modes = ImmutableList.of( ImmutableList<ZenMode> modes = ImmutableList.of(
TestModeBuilder.MANUAL_DND_INACTIVE, TestModeBuilder.MANUAL_DND,
new TestModeBuilder().setName("Inactive #1").setActive(false).build(), new TestModeBuilder().setName("Inactive #1").setActive(false).build(),
new TestModeBuilder().setName("Active #1").setActive(true).build(), new TestModeBuilder().setName("Active #1").setActive(true).build(),
new TestModeBuilder().setName("Active #2").setActive(true).build(), new TestModeBuilder().setName("Active #2").setActive(true).build(),
@@ -576,7 +576,7 @@ public class ZenModeSummaryHelperTest {
@Test @Test
public void getModesSummary_manyModesActive_listsSomeActiveModes() { public void getModesSummary_manyModesActive_listsSomeActiveModes() {
ImmutableList<ZenMode> modes = ImmutableList.of( ImmutableList<ZenMode> modes = ImmutableList.of(
TestModeBuilder.MANUAL_DND_ACTIVE, new TestModeBuilder().makeManualDnd().setActive(true).build(),
new TestModeBuilder().setName("Inactive #1").setActive(false).build(), new TestModeBuilder().setName("Inactive #1").setActive(false).build(),
new TestModeBuilder().setName("Active #1").setActive(true).build(), new TestModeBuilder().setName("Active #1").setActive(true).build(),
new TestModeBuilder().setName("Active #2").setActive(true).build(), new TestModeBuilder().setName("Active #2").setActive(true).build(),

View File

@@ -123,7 +123,7 @@ public class ZenModeTriggerAddPreferenceControllerTest {
@Test @Test
public void isAvailable_manualDND_false() { public void isAvailable_manualDND_false() {
mController.setZenMode(TestModeBuilder.MANUAL_DND_INACTIVE); mController.setZenMode(TestModeBuilder.MANUAL_DND);
assertThat(mController.isAvailable()).isFalse(); assertThat(mController.isAvailable()).isFalse();
} }

View File

@@ -113,7 +113,7 @@ public class ZenModeTriggerCategoryPreferenceControllerTest {
@Test @Test
public void isAvailable_manualDND_false() { public void isAvailable_manualDND_false() {
mController.setZenMode(TestModeBuilder.MANUAL_DND_INACTIVE); mController.setZenMode(TestModeBuilder.MANUAL_DND);
assertThat(mController.isAvailable()).isFalse(); assertThat(mController.isAvailable()).isFalse();
} }
} }

View File

@@ -148,7 +148,7 @@ public class ZenModeTriggerUpdatePreferenceControllerTest {
@Test @Test
public void isAvailable_manualDND_false() { public void isAvailable_manualDND_false() {
mController.setZenMode(TestModeBuilder.MANUAL_DND_INACTIVE); mController.setZenMode(TestModeBuilder.MANUAL_DND);
assertThat(mController.isAvailable()).isFalse(); assertThat(mController.isAvailable()).isFalse();
} }

View File

@@ -18,7 +18,7 @@ package com.android.settings.notification.modes;
import static android.app.NotificationManager.INTERRUPTION_FILTER_PRIORITY; import static android.app.NotificationManager.INTERRUPTION_FILTER_PRIORITY;
import static com.android.settingslib.notification.modes.TestModeBuilder.MANUAL_DND_INACTIVE; import static com.android.settingslib.notification.modes.TestModeBuilder.MANUAL_DND;
import static com.google.common.truth.Truth.assertThat; import static com.google.common.truth.Truth.assertThat;
@@ -150,7 +150,7 @@ public class ZenModesListPreferenceControllerTest {
@DisableFlags(Flags.FLAG_MODES_UI) @DisableFlags(Flags.FLAG_MODES_UI)
public void testModesUiOff_notAvailableAndNoSearchData() { public void testModesUiOff_notAvailableAndNoSearchData() {
// There exist modes // There exist modes
when(mBackend.getModes()).thenReturn(List.of(MANUAL_DND_INACTIVE, TEST_MODE)); when(mBackend.getModes()).thenReturn(List.of(MANUAL_DND, TEST_MODE));
assertThat(mPrefController.isAvailable()).isFalse(); assertThat(mPrefController.isAvailable()).isFalse();
List<SearchIndexableRaw> data = new ArrayList<>(); List<SearchIndexableRaw> data = new ArrayList<>();
@@ -185,20 +185,20 @@ public class ZenModesListPreferenceControllerTest {
// Changing mode data so there's a different one mode doesn't keep any previous data // Changing mode data so there's a different one mode doesn't keep any previous data
// (and setting that state up in the caller) // (and setting that state up in the caller)
when(mBackend.getModes()).thenReturn(List.of(MANUAL_DND_INACTIVE)); when(mBackend.getModes()).thenReturn(List.of(MANUAL_DND));
List<SearchIndexableRaw> newData = new ArrayList<>(); List<SearchIndexableRaw> newData = new ArrayList<>();
mPrefController.updateDynamicRawDataToIndex(newData); mPrefController.updateDynamicRawDataToIndex(newData);
assertThat(newData).hasSize(1); assertThat(newData).hasSize(1);
SearchIndexableRaw newItem = newData.get(0); SearchIndexableRaw newItem = newData.get(0);
assertThat(newItem.key).isEqualTo(MANUAL_DND_INACTIVE.getId()); assertThat(newItem.key).isEqualTo(MANUAL_DND.getId());
assertThat(newItem.title).isEqualTo("Do Not Disturb"); // set above assertThat(newItem.title).isEqualTo("Do Not Disturb"); // set above
} }
@Test @Test
@EnableFlags(Flags.FLAG_MODES_UI) @EnableFlags(Flags.FLAG_MODES_UI)
public void testUpdateDynamicRawDataToIndex_multipleModes() { public void testUpdateDynamicRawDataToIndex_multipleModes() {
when(mBackend.getModes()).thenReturn(List.of(MANUAL_DND_INACTIVE, TEST_MODE)); when(mBackend.getModes()).thenReturn(List.of(MANUAL_DND, TEST_MODE));
List<SearchIndexableRaw> data = new ArrayList<>(); List<SearchIndexableRaw> data = new ArrayList<>();
mPrefController.updateDynamicRawDataToIndex(data); mPrefController.updateDynamicRawDataToIndex(data);
@@ -206,7 +206,7 @@ public class ZenModesListPreferenceControllerTest {
// Should keep the order presented by getModes() // Should keep the order presented by getModes()
SearchIndexableRaw item0 = data.get(0); SearchIndexableRaw item0 = data.get(0);
assertThat(item0.key).isEqualTo(MANUAL_DND_INACTIVE.getId()); assertThat(item0.key).isEqualTo(MANUAL_DND.getId());
assertThat(item0.title).isEqualTo("Do Not Disturb"); // set above assertThat(item0.title).isEqualTo("Do Not Disturb"); // set above
SearchIndexableRaw item1 = data.get(1); SearchIndexableRaw item1 = data.get(1);