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

Merge "Fix blocked notifications screen" into rvc-dev

Browse Source
This commit is contained in:
TreeHugger Robot
2020-06-10 05:59:31 +00:00
committed by Android (Google) Code Review
parent 73bf03748e 425504c2de
commit 1528d932ed
2 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
src/com/android/settings/applications/AppStateNotificationBridge.java
View File

@@ -84,6 +84,9 @@ public class AppStateNotificationBridge extends AppStateBaseBridge {
for (AppEntry entry : apps) {
NotificationsSentState stats =
map.get(getKey(UserHandle.getUserId(entry.info.uid), entry.info.packageName));
if (stats == null) {
stats = new NotificationsSentState();
}
calculateAvgSentCounts(stats);
addBlockStatus(entry, stats);
entry.extraInfo = stats;

7
tests/robotests/src/com/android/settings/applications/AppStateNotificationBridgeTest.java
View File

@@ -211,7 +211,12 @@ public class AppStateNotificationBridgeTest {
when(mSession.getAllApps()).thenReturn(apps);
mBridge.loadAllExtraInfo();
assertThat(apps.get(0).extraInfo).isNull();
// extra info should exist and blocked status should be populated
assertThat(apps.get(0).extraInfo).isNotNull();
verify(mBackend).getNotificationsBanned(PKG1, 0);
// but the recent/frequent counts should be 0 so they don't appear on those screens
assertThat(((NotificationsSentState) apps.get(0).extraInfo).avgSentDaily).isEqualTo(0);
assertThat(((NotificationsSentState) apps.get(0).extraInfo).lastSent).isEqualTo(0);
}
@Test