PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Tweak refresh of notification channel group preferences" into udc-dev am: c49cb1014c

Browse Source 
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/22761580

Change-Id: I1f89f6afcb175adfa0f79de6084481c3404caa58
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Matías Hernández
2023-04-25 14:42:21 +00:00
committed by Automerger Merge Worker
parent c070721d96 c49cb1014c
commit b168628c13
2 changed files with 78 additions and 58 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
src/com/android/settings/notification/app/ChannelListPreferenceController.java
View File

@@ -236,13 +236,16 @@ public class ChannelListPreferenceController extends NotificationPreferenceContr
@NonNull PreferenceGroup groupPrefGroup) { @NonNull PreferenceGroup groupPrefGroup) {
int initialPrefCount = groupPrefGroup.getPreferenceCount(); int initialPrefCount = groupPrefGroup.getPreferenceCount();
List<Preference> finalOrderedPrefs = new ArrayList<>(); List<Preference> finalOrderedPrefs = new ArrayList<>();
Preference appDefinedGroupToggle;
if (group.getId() == null) { if (group.getId() == null) {
// For the 'null' group, set the "Other" title. // For the 'null' group, set the "Other" title.
groupPrefGroup.setTitle(R.string.notification_channels_other); groupPrefGroup.setTitle(R.string.notification_channels_other);
appDefinedGroupToggle = null;
} else { } else {
// For an app-defined group, set their name and create a row to toggle 'isBlocked'. // For an app-defined group, set their name and create a row to toggle 'isBlocked'.
groupPrefGroup.setTitle(group.getName()); groupPrefGroup.setTitle(group.getName());
finalOrderedPrefs.add(addOrUpdateGroupToggle(groupPrefGroup, group)); appDefinedGroupToggle = addOrUpdateGroupToggle(groupPrefGroup, group);
finalOrderedPrefs.add(appDefinedGroupToggle);
} }
// Here "empty" means having no channel rows; the group toggle is ignored for this purpose. // Here "empty" means having no channel rows; the group toggle is ignored for this purpose.
boolean initiallyEmpty = groupPrefGroup.getPreferenceCount() == finalOrderedPrefs.size(); boolean initiallyEmpty = groupPrefGroup.getPreferenceCount() == finalOrderedPrefs.size();
@@ -268,13 +271,25 @@ public class ChannelListPreferenceController extends NotificationPreferenceContr
// remove all preferences and re-add them all. // remove all preferences and re-add them all.
// This is required to ensure proper ordering of inserted channels, and it simplifies logic // This is required to ensure proper ordering of inserted channels, and it simplifies logic
// at the cost of computation in the rare case that the list is changing. // at the cost of computation in the rare case that the list is changing.
// As an optimization, keep the app-defined-group toggle. That way it doesn't "flicker"
// (due to remove+add) when toggling the group.
int numFinalGroups = finalOrderedPrefs.size(); int numFinalGroups = finalOrderedPrefs.size();
boolean hasInsertions = !initiallyEmpty && initialPrefCount != numFinalGroups; boolean hasInsertions = !initiallyEmpty && initialPrefCount != numFinalGroups;
boolean requiresRemoval = postAddPrefCount != numFinalGroups; boolean requiresRemoval = postAddPrefCount != numFinalGroups;
boolean keepGroupToggle =
appDefinedGroupToggle != null && groupPrefGroup.getPreferenceCount() > 0
&& groupPrefGroup.getPreference(0) == appDefinedGroupToggle
&& finalOrderedPrefs.get(0) == appDefinedGroupToggle;
if (hasInsertions || requiresRemoval) { if (hasInsertions || requiresRemoval) {
if (keepGroupToggle) {
while (groupPrefGroup.getPreferenceCount() > 1) {
groupPrefGroup.removePreference(groupPrefGroup.getPreference(1));
}
} else {
groupPrefGroup.removeAll(); groupPrefGroup.removeAll();
for (Preference preference : finalOrderedPrefs) { }
groupPrefGroup.addPreference(preference); for (int i = (keepGroupToggle ? 1 : 0); i < finalOrderedPrefs.size(); i++) {
groupPrefGroup.addPreference(finalOrderedPrefs.get(i));
} }
} }
} }
@@ -322,7 +337,7 @@ public class ChannelListPreferenceController extends NotificationPreferenceContr
if (channel.getImportance() > IMPORTANCE_LOW) { if (channel.getImportance() > IMPORTANCE_LOW) {
channelPref.setIcon(getAlertingIcon()); channelPref.setIcon(getAlertingIcon());
} else { } else {
channelPref.setIcon(R.drawable.empty_icon); channelPref.setIcon(mContext.getDrawable(R.drawable.empty_icon));
} }
channelPref.setIconSize(PrimarySwitchPreference.ICON_SIZE_SMALL); channelPref.setIconSize(PrimarySwitchPreference.ICON_SIZE_SMALL);
channelPref.setTitle(channel.getName()); channelPref.setTitle(channel.getName());

111
tests/legacy_unit/src/com/android/settings/notification/app/ChannelListPreferenceControllerTest.java → tests/unit/src/com/android/settings/notification/app/ChannelListPreferenceControllerTest.java
View File

@@ -1,5 +1,5 @@
/* /*
* Copyright (C) 2020 The Android Open Source Project * Copyright (C) 2023 The Android Open Source Project
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@@ -22,7 +22,6 @@ import static android.app.NotificationManager.IMPORTANCE_NONE;
import static junit.framework.TestCase.assertEquals; import static junit.framework.TestCase.assertEquals;
import static junit.framework.TestCase.assertFalse; import static junit.framework.TestCase.assertFalse;
import static junit.framework.TestCase.assertNotNull;
import static junit.framework.TestCase.assertNull; import static junit.framework.TestCase.assertNull;
import static junit.framework.TestCase.assertTrue; import static junit.framework.TestCase.assertTrue;
@@ -37,6 +36,7 @@ import androidx.preference.PreferenceManager;
import androidx.preference.PreferenceScreen; import androidx.preference.PreferenceScreen;
import androidx.preference.SwitchPreference; import androidx.preference.SwitchPreference;
import androidx.test.annotation.UiThreadTest; import androidx.test.annotation.UiThreadTest;
import androidx.test.core.app.ApplicationProvider;
import androidx.test.ext.junit.runners.AndroidJUnit4; import androidx.test.ext.junit.runners.AndroidJUnit4;
import androidx.test.filters.SmallTest; import androidx.test.filters.SmallTest;
import androidx.test.platform.app.InstrumentationRegistry; import androidx.test.platform.app.InstrumentationRegistry;
@@ -65,15 +65,14 @@ public class ChannelListPreferenceControllerTest {
@Before @Before
public void setUp() throws Exception { public void setUp() throws Exception {
mContext = ApplicationProvider.getApplicationContext();
Instrumentation instrumentation = InstrumentationRegistry.getInstrumentation(); Instrumentation instrumentation = InstrumentationRegistry.getInstrumentation();
mContext = instrumentation.getTargetContext();
instrumentation.runOnMainSync(() -> { instrumentation.runOnMainSync(() -> {
mBackend = new NotificationBackend(); mBackend = new NotificationBackend();
mAppRow = mBackend.loadAppRow(mContext, mAppRow = mBackend.loadAppRow(mContext,
mContext.getPackageManager(), mContext.getApplicationInfo()); mContext.getPackageManager(), mContext.getApplicationInfo());
mController = new ChannelListPreferenceController(mContext, mBackend); mController = new ChannelListPreferenceController(mContext, mBackend);
mController.onResume(mAppRow, null, null, null, null, null); mController.onResume(mAppRow, null, null, null, null, null, null);
mPreferenceManager = new PreferenceManager(mContext); mPreferenceManager = new PreferenceManager(mContext);
mPreferenceScreen = mPreferenceManager.createPreferenceScreen(mContext); mPreferenceScreen = mPreferenceManager.createPreferenceScreen(mContext);
mGroupList = new PreferenceCategory(mContext); mGroupList = new PreferenceCategory(mContext);
@@ -103,9 +102,10 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group1", group1.getKey()); assertEquals("group1", group1.getKey());
assertEquals(2, group1.getPreferenceCount()); assertEquals(2, group1.getPreferenceCount());
assertNull(group1.getPreference(0).getKey()); assertNull(group1.getPreference(0).getKey());
assertEquals("All \"Group 1\" notifications", group1.getPreference(0).getTitle()); assertEquals("All \"Group 1\" notifications",
group1.getPreference(0).getTitle().toString());
assertEquals("ch1a", group1.getPreference(1).getKey()); assertEquals("ch1a", group1.getPreference(1).getKey());
assertEquals("Channel 1A", group1.getPreference(1).getTitle()); assertEquals("Channel 1A", group1.getPreference(1).getTitle().toString());
} }
// Test that adding a channel works -- no dupes or omissions // Test that adding a channel works -- no dupes or omissions
@@ -117,11 +117,12 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group1", group1.getKey()); assertEquals("group1", group1.getKey());
assertEquals(3, group1.getPreferenceCount()); assertEquals(3, group1.getPreferenceCount());
assertNull(group1.getPreference(0).getKey()); assertNull(group1.getPreference(0).getKey());
assertEquals("All \"Group 1\" notifications", group1.getPreference(0).getTitle()); assertEquals("All \"Group 1\" notifications",
group1.getPreference(0).getTitle().toString());
assertEquals("ch1a", group1.getPreference(1).getKey()); assertEquals("ch1a", group1.getPreference(1).getKey());
assertEquals("Channel 1A", group1.getPreference(1).getTitle()); assertEquals("Channel 1A", group1.getPreference(1).getTitle().toString());
assertEquals("ch1b", group1.getPreference(2).getKey()); assertEquals("ch1b", group1.getPreference(2).getKey());
assertEquals("Channel 1B", group1.getPreference(2).getTitle()); assertEquals("Channel 1B", group1.getPreference(2).getTitle().toString());
} }
// Test that renaming a channel does in fact rename the preferences // Test that renaming a channel does in fact rename the preferences
@@ -133,11 +134,12 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group1", group1.getKey()); assertEquals("group1", group1.getKey());
assertEquals(3, group1.getPreferenceCount()); assertEquals(3, group1.getPreferenceCount());
assertNull(group1.getPreference(0).getKey()); assertNull(group1.getPreference(0).getKey());
assertEquals("All \"Group 1\" notifications", group1.getPreference(0).getTitle()); assertEquals("All \"Group 1\" notifications",
group1.getPreference(0).getTitle().toString());
assertEquals("ch1a", group1.getPreference(1).getKey()); assertEquals("ch1a", group1.getPreference(1).getKey());
assertEquals("Channel 1A", group1.getPreference(1).getTitle()); assertEquals("Channel 1A", group1.getPreference(1).getTitle().toString());
assertEquals("ch1b", group1.getPreference(2).getKey()); assertEquals("ch1b", group1.getPreference(2).getKey());
assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle()); assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle().toString());
} }
// Test that adding a group works and results in the correct sorting. // Test that adding a group works and results in the correct sorting.
@@ -152,18 +154,20 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group0", group0.getKey()); assertEquals("group0", group0.getKey());
assertEquals(2, group0.getPreferenceCount()); assertEquals(2, group0.getPreferenceCount());
assertNull(group0.getPreference(0).getKey()); assertNull(group0.getPreference(0).getKey());
assertEquals("All \"Group 0\" notifications", group0.getPreference(0).getTitle()); assertEquals("All \"Group 0\" notifications",
group0.getPreference(0).getTitle().toString());
assertEquals("ch0b", group0.getPreference(1).getKey()); assertEquals("ch0b", group0.getPreference(1).getKey());
assertEquals("Channel 0B", group0.getPreference(1).getTitle()); assertEquals("Channel 0B", group0.getPreference(1).getTitle().toString());
PreferenceGroup group1 = (PreferenceGroup) mGroupList.getPreference(1); PreferenceGroup group1 = (PreferenceGroup) mGroupList.getPreference(1);
assertEquals("group1", group1.getKey()); assertEquals("group1", group1.getKey());
assertEquals(3, group1.getPreferenceCount()); assertEquals(3, group1.getPreferenceCount());
assertNull(group1.getPreference(0).getKey()); assertNull(group1.getPreference(0).getKey());
assertEquals("All \"Group 1\" notifications", group1.getPreference(0).getTitle()); assertEquals("All \"Group 1\" notifications",
group1.getPreference(0).getTitle().toString());
assertEquals("ch1a", group1.getPreference(1).getKey()); assertEquals("ch1a", group1.getPreference(1).getKey());
assertEquals("Channel 1A", group1.getPreference(1).getTitle()); assertEquals("Channel 1A", group1.getPreference(1).getTitle().toString());
assertEquals("ch1b", group1.getPreference(2).getKey()); assertEquals("ch1b", group1.getPreference(2).getKey());
assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle()); assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle().toString());
} }
// Test that adding a channel that comes before another works and has correct ordering. // Test that adding a channel that comes before another works and has correct ordering.
@@ -176,20 +180,22 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group0", group0.getKey()); assertEquals("group0", group0.getKey());
assertEquals(3, group0.getPreferenceCount()); assertEquals(3, group0.getPreferenceCount());
assertNull(group0.getPreference(0).getKey()); assertNull(group0.getPreference(0).getKey());
assertEquals("All \"Group 0\" notifications", group0.getPreference(0).getTitle()); assertEquals("All \"Group 0\" notifications",
group0.getPreference(0).getTitle().toString());
assertEquals("ch0a", group0.getPreference(1).getKey()); assertEquals("ch0a", group0.getPreference(1).getKey());
assertEquals("Channel 0A", group0.getPreference(1).getTitle()); assertEquals("Channel 0A", group0.getPreference(1).getTitle().toString());
assertEquals("ch0b", group0.getPreference(2).getKey()); assertEquals("ch0b", group0.getPreference(2).getKey());
assertEquals("Channel 0B", group0.getPreference(2).getTitle()); assertEquals("Channel 0B", group0.getPreference(2).getTitle().toString());
PreferenceGroup group1 = (PreferenceGroup) mGroupList.getPreference(1); PreferenceGroup group1 = (PreferenceGroup) mGroupList.getPreference(1);
assertEquals("group1", group1.getKey()); assertEquals("group1", group1.getKey());
assertEquals(3, group1.getPreferenceCount()); assertEquals(3, group1.getPreferenceCount());
assertNull(group1.getPreference(0).getKey()); assertNull(group1.getPreference(0).getKey());
assertEquals("All \"Group 1\" notifications", group1.getPreference(0).getTitle()); assertEquals("All \"Group 1\" notifications",
group1.getPreference(0).getTitle().toString());
assertEquals("ch1a", group1.getPreference(1).getKey()); assertEquals("ch1a", group1.getPreference(1).getKey());
assertEquals("Channel 1A", group1.getPreference(1).getTitle()); assertEquals("Channel 1A", group1.getPreference(1).getTitle().toString());
assertEquals("ch1b", group1.getPreference(2).getKey()); assertEquals("ch1b", group1.getPreference(2).getKey());
assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle()); assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle().toString());
} }
// Test that the "Other" group works. // Test that the "Other" group works.
@@ -206,18 +212,19 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group1", group1.getKey()); assertEquals("group1", group1.getKey());
assertEquals(3, group1.getPreferenceCount()); assertEquals(3, group1.getPreferenceCount());
assertNull(group1.getPreference(0).getKey()); assertNull(group1.getPreference(0).getKey());
assertEquals("All \"Group 1\" notifications", group1.getPreference(0).getTitle()); assertEquals("All \"Group 1\" notifications",
group1.getPreference(0).getTitle().toString());
assertEquals("ch1a", group1.getPreference(1).getKey()); assertEquals("ch1a", group1.getPreference(1).getKey());
assertEquals("Channel 1A", group1.getPreference(1).getTitle()); assertEquals("Channel 1A", group1.getPreference(1).getTitle().toString());
assertEquals("ch1b", group1.getPreference(2).getKey()); assertEquals("ch1b", group1.getPreference(2).getKey());
assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle()); assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle().toString());
PreferenceGroup groupOther = (PreferenceGroup) mGroupList.getPreference(1); PreferenceGroup groupOther = (PreferenceGroup) mGroupList.getPreference(1);
assertEquals("categories", groupOther.getKey()); assertEquals("categories", groupOther.getKey());
assertEquals(2, groupOther.getPreferenceCount()); assertEquals(2, groupOther.getPreferenceCount());
assertEquals("chXa", groupOther.getPreference(0).getKey()); assertEquals("chXa", groupOther.getPreference(0).getKey());
assertEquals("Other A", groupOther.getPreference(0).getTitle()); assertEquals("Other A", groupOther.getPreference(0).getTitle().toString());
assertEquals("chXb", groupOther.getPreference(1).getKey()); assertEquals("chXb", groupOther.getPreference(1).getKey());
assertEquals("Other B", groupOther.getPreference(1).getTitle()); assertEquals("Other B", groupOther.getPreference(1).getTitle().toString());
} }
// Test that the removal of a channel works. // Test that the removal of a channel works.
@@ -229,16 +236,17 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group1", group1.getKey()); assertEquals("group1", group1.getKey());
assertEquals(3, group1.getPreferenceCount()); assertEquals(3, group1.getPreferenceCount());
assertNull(group1.getPreference(0).getKey()); assertNull(group1.getPreference(0).getKey());
assertEquals("All \"Group 1\" notifications", group1.getPreference(0).getTitle()); assertEquals("All \"Group 1\" notifications",
group1.getPreference(0).getTitle().toString());
assertEquals("ch1a", group1.getPreference(1).getKey()); assertEquals("ch1a", group1.getPreference(1).getKey());
assertEquals("Channel 1A", group1.getPreference(1).getTitle()); assertEquals("Channel 1A", group1.getPreference(1).getTitle().toString());
assertEquals("ch1b", group1.getPreference(2).getKey()); assertEquals("ch1b", group1.getPreference(2).getKey());
assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle()); assertEquals("Channel 1B - Renamed", group1.getPreference(2).getTitle().toString());
PreferenceGroup groupOther = (PreferenceGroup) mGroupList.getPreference(1); PreferenceGroup groupOther = (PreferenceGroup) mGroupList.getPreference(1);
assertEquals("categories", groupOther.getKey()); assertEquals("categories", groupOther.getKey());
assertEquals(1, groupOther.getPreferenceCount()); assertEquals(1, groupOther.getPreferenceCount());
assertEquals("chXb", groupOther.getPreference(0).getKey()); assertEquals("chXb", groupOther.getPreference(0).getKey());
assertEquals("Other B", groupOther.getPreference(0).getTitle()); assertEquals("Other B", groupOther.getPreference(0).getTitle().toString());
} }
// Test that we go back to the empty state when clearing all groups and channels. // Test that we go back to the empty state when clearing all groups and channels.
@@ -269,15 +277,15 @@ public class ChannelListPreferenceControllerTest {
assertEquals(3, group.getPreferenceCount()); assertEquals(3, group.getPreferenceCount());
SwitchPreference groupBlockPref = (SwitchPreference) group.getPreference(0); SwitchPreference groupBlockPref = (SwitchPreference) group.getPreference(0);
assertNull(groupBlockPref.getKey()); assertNull(groupBlockPref.getKey());
assertEquals("All \"My Group\" notifications", groupBlockPref.getTitle()); assertEquals("All \"My Group\" notifications", groupBlockPref.getTitle().toString());
assertTrue(groupBlockPref.isChecked()); assertTrue(groupBlockPref.isChecked());
PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1); PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1);
assertEquals("channelA", channelAPref.getKey()); assertEquals("channelA", channelAPref.getKey());
assertEquals("Channel A", channelAPref.getTitle()); assertEquals("Channel A", channelAPref.getTitle().toString());
assertEquals(Boolean.TRUE, channelAPref.getCheckedState()); assertEquals(Boolean.TRUE, channelAPref.getCheckedState());
PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2); PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2);
assertEquals("channelB", channelBPref.getKey()); assertEquals("channelB", channelBPref.getKey());
assertEquals("Channel B", channelBPref.getTitle()); assertEquals("Channel B", channelBPref.getTitle().toString());
assertEquals(Boolean.FALSE, channelBPref.getCheckedState()); assertEquals(Boolean.FALSE, channelBPref.getCheckedState());
} }
@@ -291,7 +299,7 @@ public class ChannelListPreferenceControllerTest {
assertEquals(1, group.getPreferenceCount()); assertEquals(1, group.getPreferenceCount());
SwitchPreference groupBlockPref = (SwitchPreference) group.getPreference(0); SwitchPreference groupBlockPref = (SwitchPreference) group.getPreference(0);
assertNull(groupBlockPref.getKey()); assertNull(groupBlockPref.getKey());
assertEquals("All \"My Group\" notifications", groupBlockPref.getTitle()); assertEquals("All \"My Group\" notifications", groupBlockPref.getTitle().toString());
assertFalse(groupBlockPref.isChecked()); assertFalse(groupBlockPref.isChecked());
} }
@@ -305,15 +313,15 @@ public class ChannelListPreferenceControllerTest {
assertEquals(3, group.getPreferenceCount()); assertEquals(3, group.getPreferenceCount());
SwitchPreference groupBlockPref = (SwitchPreference) group.getPreference(0); SwitchPreference groupBlockPref = (SwitchPreference) group.getPreference(0);
assertNull(groupBlockPref.getKey()); assertNull(groupBlockPref.getKey());
assertEquals("All \"My Group\" notifications", groupBlockPref.getTitle()); assertEquals("All \"My Group\" notifications", groupBlockPref.getTitle().toString());
assertTrue(groupBlockPref.isChecked()); assertTrue(groupBlockPref.isChecked());
PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1); PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1);
assertEquals("channelA", channelAPref.getKey()); assertEquals("channelA", channelAPref.getKey());
assertEquals("Channel A", channelAPref.getTitle()); assertEquals("Channel A", channelAPref.getTitle().toString());
assertEquals(Boolean.TRUE, channelAPref.getCheckedState()); assertEquals(Boolean.TRUE, channelAPref.getCheckedState());
PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2); PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2);
assertEquals("channelB", channelBPref.getKey()); assertEquals("channelB", channelBPref.getKey());
assertEquals("Channel B", channelBPref.getTitle()); assertEquals("Channel B", channelBPref.getTitle().toString());
assertEquals(Boolean.FALSE, channelBPref.getCheckedState()); assertEquals(Boolean.FALSE, channelBPref.getCheckedState());
} }
} }
@@ -347,19 +355,18 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group", group.getKey()); assertEquals("group", group.getKey());
assertEquals(3, group.getPreferenceCount()); assertEquals(3, group.getPreferenceCount());
assertNull(group.getPreference(0).getKey()); assertNull(group.getPreference(0).getKey());
assertEquals("All \"Group\" notifications", group.getPreference(0).getTitle()); assertEquals("All \"Group\" notifications",
group.getPreference(0).getTitle().toString());
PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1); PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1);
assertEquals("channelA", channelAPref.getKey()); assertEquals("channelA", channelAPref.getKey());
assertEquals("Channel A", channelAPref.getTitle()); assertEquals("Channel A", channelAPref.getTitle().toString());
assertEquals(Boolean.TRUE, channelAPref.getCheckedState()); assertEquals(Boolean.TRUE, channelAPref.getCheckedState());
assertEquals("~2 notifications per day", channelAPref.getSummary()); assertEquals("About 2 notifications per day", channelAPref.getSummary().toString());
assertNotNull(channelAPref.getIcon());
PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2); PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2);
assertEquals("channelB", channelBPref.getKey()); assertEquals("channelB", channelBPref.getKey());
assertEquals("Channel B", channelBPref.getTitle()); assertEquals("Channel B", channelBPref.getTitle().toString());
assertEquals(Boolean.FALSE, channelBPref.getCheckedState()); assertEquals(Boolean.FALSE, channelBPref.getCheckedState());
assertNull(channelBPref.getSummary()); assertNull(channelBPref.getSummary());
assertNull(channelBPref.getIcon());
} }
channelA.setImportance(IMPORTANCE_NONE); channelA.setImportance(IMPORTANCE_NONE);
@@ -376,20 +383,18 @@ public class ChannelListPreferenceControllerTest {
assertEquals("group", group.getKey()); assertEquals("group", group.getKey());
assertEquals(3, group.getPreferenceCount()); assertEquals(3, group.getPreferenceCount());
assertNull(group.getPreference(0).getKey()); assertNull(group.getPreference(0).getKey());
assertEquals("All \"Group\" notifications", group.getPreference(0).getTitle()); assertEquals("All \"Group\" notifications",
group.getPreference(0).getTitle().toString());
PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1); PrimarySwitchPreference channelAPref = (PrimarySwitchPreference) group.getPreference(1);
assertEquals("channelA", channelAPref.getKey()); assertEquals("channelA", channelAPref.getKey());
assertEquals("Channel A", channelAPref.getTitle()); assertEquals("Channel A", channelAPref.getTitle().toString());
assertEquals(Boolean.FALSE, channelAPref.getCheckedState()); assertEquals(Boolean.FALSE, channelAPref.getCheckedState());
assertNull(channelAPref.getSummary()); assertNull(channelAPref.getSummary());
assertNull(channelAPref.getIcon());
PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2); PrimarySwitchPreference channelBPref = (PrimarySwitchPreference) group.getPreference(2);
assertEquals("channelB", channelBPref.getKey()); assertEquals("channelB", channelBPref.getKey());
assertEquals("Channel B", channelBPref.getTitle()); assertEquals("Channel B", channelBPref.getTitle().toString());
assertEquals(Boolean.TRUE, channelBPref.getCheckedState()); assertEquals(Boolean.TRUE, channelBPref.getCheckedState());
assertEquals("~2 notifications per week", channelBPref.getSummary()); assertEquals("About 2 notifications per week", channelBPref.getSummary().toString());
assertNotNull(channelBPref.getIcon());
} }
} }
} }