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

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

Issue 2831823003: Convert ChromeActivityTestCaseBase direct children to JUnit4 (Closed)
Patch Set: rebase and convert newly added test Created 3 years, 8 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/PopupTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/PopupTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/PopupTest.java
index 8e53cfdd44ec5d116e2232e9203ab349b34714b1..20ff6abeee0a6892613ea804e0398d49063f3190 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/PopupTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/PopupTest.java
@@ -4,17 +4,27 @@
package org.chromium.chrome.browser;
+import android.support.test.InstrumentationRegistry;
import android.support.test.filters.MediumTest;
import android.text.TextUtils;
+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.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.infobar.InfoBar;
import org.chromium.chrome.browser.infobar.InfoBarContainer;
import org.chromium.chrome.browser.tabmodel.TabModelSelector;
-import org.chromium.chrome.test.ChromeActivityTestCaseBase;
+import org.chromium.chrome.test.ChromeActivityTestRule;
+import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
import org.chromium.content.browser.test.util.Criteria;
import org.chromium.content.browser.test.util.CriteriaHelper;
import org.chromium.content.browser.test.util.TouchCommon;
@@ -26,51 +36,50 @@ import java.util.concurrent.Callable;
/**
* Tests whether popup windows appear.
*/
+@RunWith(ChromeJUnit4ClassRunner.class)
@RetryOnFailure
-public class PopupTest extends ChromeActivityTestCaseBase<ChromeActivity> {
+@CommandLineFlags.Add({ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE,
+ ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG})
+public class PopupTest {
+ @Rule
+ public ChromeActivityTestRule<ChromeActivity> mActivityTestRule =
+ new ChromeActivityTestRule<>(ChromeActivity.class);
+
private static final String POPUP_HTML_PATH = "/chrome/test/data/android/popup_test.html";
private String mPopupHtmlUrl;
private EmbeddedTestServer mTestServer;
- public PopupTest() {
- super(ChromeActivity.class);
- }
-
private int getNumInfobarsShowing() {
- return getInfoBars().size();
+ return mActivityTestRule.getInfoBars().size();
}
- @Override
- protected void setUp() throws Exception {
- super.setUp();
+ @Before
+ public void setUp() throws Exception {
+ mActivityTestRule.startMainActivityOnBlankPage();
ThreadUtils.runOnUiThread(new Runnable() {
@Override
public void run() {
- assertTrue(getNumInfobarsShowing() == 0);
+ Assert.assertTrue(getNumInfobarsShowing() == 0);
}
});
- mTestServer = EmbeddedTestServer.createAndStartServer(getInstrumentation().getContext());
+ mTestServer = EmbeddedTestServer.createAndStartServer(
+ InstrumentationRegistry.getInstrumentation().getContext());
mPopupHtmlUrl = mTestServer.getURL(POPUP_HTML_PATH);
}
- @Override
- protected void tearDown() throws Exception {
+ @After
+ public void tearDown() throws Exception {
mTestServer.stopAndDestroyServer();
- super.tearDown();
- }
-
- @Override
- public void startMainActivity() throws InterruptedException {
- startMainActivityOnBlankPage();
}
+ @Test
@MediumTest
@Feature({"Popup"})
public void testPopupInfobarAppears() throws Exception {
- loadUrl(mPopupHtmlUrl);
+ mActivityTestRule.loadUrl(mPopupHtmlUrl);
CriteriaHelper.pollUiThread(Criteria.equals(1, new Callable<Integer>() {
@Override
public Integer call() {
@@ -79,22 +88,23 @@ public class PopupTest extends ChromeActivityTestCaseBase<ChromeActivity> {
}));
}
+ @Test
@MediumTest
@Feature({"Popup"})
public void testPopupWindowsAppearWhenAllowed() throws Exception {
- final TabModelSelector selector = getActivity().getTabModelSelector();
+ final TabModelSelector selector = mActivityTestRule.getActivity().getTabModelSelector();
- loadUrl(mPopupHtmlUrl);
+ mActivityTestRule.loadUrl(mPopupHtmlUrl);
CriteriaHelper.pollUiThread(Criteria.equals(1, new Callable<Integer>() {
@Override
public Integer call() {
return getNumInfobarsShowing();
}
}));
- assertEquals(1, selector.getTotalTabCount());
+ Assert.assertEquals(1, selector.getTotalTabCount());
final InfoBarContainer container = selector.getCurrentTab().getInfoBarContainer();
ArrayList<InfoBar> infobars = container.getInfoBarsForTesting();
- assertEquals(1, infobars.size());
+ Assert.assertEquals(1, infobars.size());
// Wait until the animations are done, then click the "open popups" button.
final InfoBar infobar = infobars.get(0);
@@ -116,11 +126,11 @@ public class PopupTest extends ChromeActivityTestCaseBase<ChromeActivity> {
}
}, 7500, CriteriaHelper.DEFAULT_POLLING_INTERVAL);
- assertEquals(4, selector.getTotalTabCount());
+ Assert.assertEquals(4, selector.getTotalTabCount());
int currentTabId = selector.getCurrentTab().getId();
// Test that revisiting the original page makes popup windows immediately.
- loadUrl(mPopupHtmlUrl);
+ mActivityTestRule.loadUrl(mPopupHtmlUrl);
CriteriaHelper.pollUiThread(new Criteria() {
@Override
public boolean isSatisfied() {
@@ -129,6 +139,6 @@ public class PopupTest extends ChromeActivityTestCaseBase<ChromeActivity> {
return TextUtils.equals("Three", selector.getCurrentTab().getTitle());
}
}, 7500, CriteriaHelper.DEFAULT_POLLING_INTERVAL);
- assertNotSame(currentTabId, selector.getCurrentTab().getId());
+ Assert.assertNotSame(currentTabId, selector.getCurrentTab().getId());
}
}

Powered by Google App Engine
This is Rietveld 408576698