From 44e192bf0285b6d36f8e7dfe91ab1fb5ed3a6d20 Mon Sep 17 00:00:00 2001 From: Flavio Fiszman Date: Thu, 1 Jul 2021 15:20:17 +0100 Subject: [PATCH] Increase People Tile restore timeout to 48h Change-Id: Ia218b8b99e0fc5bd2fbbd185d781ff1673570ed4 Test: PeopleBackupFollowUpJobTest Bug: 183084949 --- .../com/android/systemui/people/PeopleBackupFollowUpJob.java | 2 +- .../android/systemui/people/PeopleBackupFollowUpJobTest.java | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/SystemUI/src/com/android/systemui/people/PeopleBackupFollowUpJob.java b/packages/SystemUI/src/com/android/systemui/people/PeopleBackupFollowUpJob.java index 452484f599a86..3bc1f30ea3212 100644 --- a/packages/SystemUI/src/com/android/systemui/people/PeopleBackupFollowUpJob.java +++ b/packages/SystemUI/src/com/android/systemui/people/PeopleBackupFollowUpJob.java @@ -58,7 +58,7 @@ public class PeopleBackupFollowUpJob extends JobService { public static final int JOB_ID = 74823873; private static final long JOB_PERIODIC_DURATION = Duration.ofHours(6).toMillis(); - private static final long CLEAN_UP_STORAGE_AFTER_DURATION = Duration.ofHours(24).toMillis(); + private static final long CLEAN_UP_STORAGE_AFTER_DURATION = Duration.ofHours(48).toMillis(); /** SharedPreferences file name for follow-up specific storage.*/ public static final String SHARED_FOLLOW_UP = "shared_follow_up"; diff --git a/packages/SystemUI/tests/src/com/android/systemui/people/PeopleBackupFollowUpJobTest.java b/packages/SystemUI/tests/src/com/android/systemui/people/PeopleBackupFollowUpJobTest.java index 00e012e5d30c7..0d1749c51994f 100644 --- a/packages/SystemUI/tests/src/com/android/systemui/people/PeopleBackupFollowUpJobTest.java +++ b/packages/SystemUI/tests/src/com/android/systemui/people/PeopleBackupFollowUpJobTest.java @@ -156,7 +156,7 @@ public class PeopleBackupFollowUpJobTest extends SysuiTestCase { @Test public void testShouldCancelJob_noRemainingWidgets_longTimeElapsed_shouldCancel() { assertThat(mPeopleBackupFollowUpJob.shouldCancelJob( - new HashMap<>(), 10, Duration.ofHours(25).toMillis())).isTrue(); + new HashMap<>(), 10, Duration.ofHours(50).toMillis())).isTrue(); } @Test @@ -171,7 +171,7 @@ public class PeopleBackupFollowUpJobTest extends SysuiTestCase { Map> remainingWidgets = new HashMap<>(); remainingWidgets.put(PEOPLE_TILE_KEY.toString(), WIDGET_IDS); assertThat(mPeopleBackupFollowUpJob.shouldCancelJob( - remainingWidgets, 10, 1000 * 60 * 60 * 25)).isTrue(); + remainingWidgets, 10, Duration.ofHours(50).toMillis())).isTrue(); } @Test