Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1121)

Unified Diff: chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskUnitTest.java

Issue 2914703002: [Offline Prefetch] Backoff support for PrefetchBackgroundTask (Closed)
Patch Set: Fix test Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskUnitTest.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskTest.java b/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskUnitTest.java
similarity index 87%
rename from chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskTest.java
rename to chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskUnitTest.java
index 14523810f4446b8737db18c429128b12e0e96945..3c84028c5abb5364d043c3810a6119ce989ae064 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskTest.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/offlinepages/prefetch/PrefetchBackgroundTaskUnitTest.java
@@ -40,13 +40,14 @@ import org.chromium.testing.local.LocalRobolectricTestRunner;
import java.util.ArrayList;
import java.util.HashMap;
+import java.util.concurrent.TimeUnit;
/** Unit tests for {@link PrefetchBackgroundTask}. */
@RunWith(LocalRobolectricTestRunner.class)
@Config(manifest = Config.NONE,
- shadows = {PrefetchBackgroundTaskTest.ShadowBackgroundTaskScheduler.class,
+ shadows = {PrefetchBackgroundTaskUnitTest.ShadowBackgroundTaskScheduler.class,
ShadowMultiDex.class})
-public class PrefetchBackgroundTaskTest {
+public class PrefetchBackgroundTaskUnitTest {
/**
* Shadow of BackgroundTaskScheduler system service.
*/
@@ -103,10 +104,14 @@ public class PrefetchBackgroundTaskTest {
@Test
public void scheduleTask() {
- PrefetchBackgroundTask.scheduleTask();
+ final int additionalDelaySeconds = 15;
+ PrefetchBackgroundTask.scheduleTask(additionalDelaySeconds);
TaskInfo scheduledTask =
mShadowTaskScheduler.getTaskInfo(TaskIds.OFFLINE_PAGES_PREFETCH_JOB_ID);
assertNotNull(scheduledTask);
+ assertEquals(TimeUnit.SECONDS.toMillis(PrefetchBackgroundTask.DEFAULT_START_DELAY_SECONDS
+ + additionalDelaySeconds),
+ scheduledTask.getOneOffInfo().getWindowStartTimeMs());
assertEquals(true, scheduledTask.isPersisted());
assertEquals(TaskInfo.NETWORK_TYPE_UNMETERED, scheduledTask.getRequiredNetworkType());
}
@@ -117,9 +122,11 @@ public class PrefetchBackgroundTaskTest {
mShadowTaskScheduler.getTaskInfo(TaskIds.OFFLINE_PAGES_PREFETCH_JOB_ID);
assertNull(scheduledTask);
- PrefetchBackgroundTask.scheduleTask();
+ PrefetchBackgroundTask.scheduleTask(0);
scheduledTask = mShadowTaskScheduler.getTaskInfo(TaskIds.OFFLINE_PAGES_PREFETCH_JOB_ID);
assertNotNull(scheduledTask);
+ assertEquals(TimeUnit.SECONDS.toMillis(PrefetchBackgroundTask.DEFAULT_START_DELAY_SECONDS),
+ scheduledTask.getOneOffInfo().getWindowStartTimeMs());
PrefetchBackgroundTask.cancelTask();
scheduledTask = mShadowTaskScheduler.getTaskInfo(TaskIds.OFFLINE_PAGES_PREFETCH_JOB_ID);

Powered by Google App Engine
This is Rietveld 408576698