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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java

Issue 2853573002: Convert ChromeTabbedActivityTestCaseBase children to JUnit4 (Closed)
Patch Set: rebase Created 3 years, 7 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/javatests/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
index 291d2d9868a29caf36c43f67c32ba33716fe4a09..e10aa0f8cd10261a767cbffe07452e02dd4b5c4c 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/webapps/WebApkUpdateManagerTest.java
@@ -5,8 +5,16 @@
package org.chromium.chrome.browser.webapps;
import android.content.Context;
+import android.support.test.InstrumentationRegistry;
import android.support.test.filters.MediumTest;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
import org.chromium.base.ThreadUtils;
import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.test.util.CallbackHelper;
@@ -15,7 +23,9 @@ import org.chromium.base.test.util.Feature;
import org.chromium.blink_public.platform.WebDisplayMode;
import org.chromium.chrome.browser.ChromeSwitches;
import org.chromium.chrome.browser.tab.Tab;
-import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
+import org.chromium.chrome.test.ChromeActivityTestRule;
+import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
+import org.chromium.chrome.test.ChromeTabbedActivityTestRule;
import org.chromium.chrome.test.util.browser.WebappTestPage;
import org.chromium.content_public.common.ScreenOrientationValues;
import org.chromium.net.test.EmbeddedTestServer;
@@ -27,8 +37,13 @@ import java.util.Map;
/**
* Tests WebApkUpdateManager. This class contains tests which cannot be done as JUnit tests.
*/
-@CommandLineFlags.Add(ChromeSwitches.CHECK_FOR_WEB_MANIFEST_UPDATE_ON_STARTUP)
-public class WebApkUpdateManagerTest extends ChromeTabbedActivityTestBase {
+@RunWith(ChromeJUnit4ClassRunner.class)
+@CommandLineFlags.Add({ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE,
+ ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG,
+ ChromeSwitches.CHECK_FOR_WEB_MANIFEST_UPDATE_ON_STARTUP})
+public class WebApkUpdateManagerTest {
+ @Rule
+ public ChromeTabbedActivityTestRule mActivityTestRule = new ChromeTabbedActivityTestRule();
private static final String WEBAPK_PACKAGE = "test.package";
private static final String WEBAPK_ID = "webapk_id";
@@ -119,29 +134,23 @@ public class WebApkUpdateManagerTest extends ChromeTabbedActivityTestBase {
return creationData;
}
- @Override
- protected void setUp() throws Exception {
- super.setUp();
+ @Before
+ public void setUp() throws Exception {
+ mActivityTestRule.startMainActivityOnBlankPage();
RecordHistogram.setDisabledForTests(true);
- Context context = getInstrumentation().getTargetContext();
+ Context context = InstrumentationRegistry.getInstrumentation().getTargetContext();
mTestServer = EmbeddedTestServer.createAndStartServer(context);
- mTab = getActivity().getActivityTab();
+ mTab = mActivityTestRule.getActivity().getActivityTab();
TestFetchStorageCallback callback = new TestFetchStorageCallback();
WebappRegistry.getInstance().register(WEBAPK_ID, callback);
callback.waitForCallback(0);
}
- @Override
- protected void tearDown() throws Exception {
+ @After
+ public void tearDown() throws Exception {
mTestServer.stopAndDestroyServer();
RecordHistogram.setDisabledForTests(false);
- super.tearDown();
- }
-
- @Override
- public void startMainActivity() throws InterruptedException {
- startMainActivityOnBlankPage();
}
/** Checks whether a WebAPK update is needed. */
@@ -173,6 +182,7 @@ public class WebApkUpdateManagerTest extends ChromeTabbedActivityTestBase {
* the URLs in the Web Manifest have been modified by the WebAPK server prior to being stored in
* the WebAPK Android Manifest. Chrome and the WebAPK server parse URLs differently.
*/
+ @Test
@MediumTest
@Feature({"WebApk"})
public void testCanonicalUrlsIdenticalShouldNotUpgrade() throws Exception {
@@ -183,12 +193,13 @@ public class WebApkUpdateManagerTest extends ChromeTabbedActivityTestBase {
WebappTestPage.navigateToPageWithServiceWorkerAndManifest(
mTestServer, mTab, WEBAPK_MANIFEST_URL);
- assertFalse(checkUpdateNeeded(creationData));
+ Assert.assertFalse(checkUpdateNeeded(creationData));
}
/**
* Test that an upgraded WebAPK is requested if the canonicalized "start URLs" are different.
*/
+ @Test
@MediumTest
@Feature({"WebApk"})
public void testCanonicalUrlsDifferentShouldUpgrade() throws Exception {
@@ -199,6 +210,6 @@ public class WebApkUpdateManagerTest extends ChromeTabbedActivityTestBase {
WebappTestPage.navigateToPageWithServiceWorkerAndManifest(
mTestServer, mTab, WEBAPK_MANIFEST_URL);
- assertTrue(checkUpdateNeeded(creationData));
+ Assert.assertTrue(checkUpdateNeeded(creationData));
}
}

Powered by Google App Engine
This is Rietveld 408576698