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 am: 1528d932ed

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

Change-Id: Iafdae12f789f21d26c59d74382b5a147feeb0454
This commit is contained in:
TreeHugger Robot
2020-06-10 06:15:23 +00:00
committed by Automerger Merge Worker
parent dbe7263055 1528d932ed
commit 367aeee6b8
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