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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/appmenu/DataSaverAppMenuTest.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/appmenu/DataSaverAppMenuTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/appmenu/DataSaverAppMenuTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/appmenu/DataSaverAppMenuTest.java
index 902b468cf7e4358dee60d4aaa7e9df87f66a54f4..69f05afef871e6b2dd117fd0201e2b86435a742d 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/appmenu/DataSaverAppMenuTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/appmenu/DataSaverAppMenuTest.java
@@ -6,22 +6,37 @@ package org.chromium.chrome.browser.appmenu;
import android.app.Activity;
import android.support.test.filters.SmallTest;
-import android.test.UiThreadTest;
+
+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.ContextUtils;
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.ChromeActivity;
+import org.chromium.chrome.browser.ChromeSwitches;
import org.chromium.chrome.browser.ChromeTabbedActivity;
import org.chromium.chrome.browser.net.spdyproxy.DataReductionProxySettings;
-import org.chromium.chrome.test.ChromeActivityTestCaseBase;
+import org.chromium.chrome.test.ChromeActivityTestRule;
+import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
/**
* Tests the Data Saver AppMenu footer
*/
+@RunWith(ChromeJUnit4ClassRunner.class)
@RetryOnFailure
-public class DataSaverAppMenuTest extends ChromeActivityTestCaseBase<ChromeTabbedActivity> {
+@CommandLineFlags.Add({ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE,
+ ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG})
+public class DataSaverAppMenuTest {
+ @Rule
+ public ChromeActivityTestRule<ChromeActivity> mActivityTestRule =
+ new ChromeActivityTestRule<>(ChromeActivity.class);
+
private AppMenuHandlerForTest mAppMenuHandler;
/**
@@ -44,17 +59,8 @@ public class DataSaverAppMenuTest extends ChromeActivityTestCaseBase<ChromeTabbe
}
}
- public DataSaverAppMenuTest() {
- super(ChromeTabbedActivity.class);
- }
-
- @Override
- public void startMainActivity() throws InterruptedException {
- startMainActivityOnBlankPage();
- }
-
- @Override
- protected void setUp() throws Exception {
+ @Before
+ public void setUp() throws Exception {
ChromeTabbedActivity.setAppMenuHandlerFactoryForTesting(
new ChromeTabbedActivity.AppMenuHandlerFactory() {
@Override
@@ -66,76 +72,91 @@ public class DataSaverAppMenuTest extends ChromeActivityTestCaseBase<ChromeTabbe
}
});
- super.setUp();
+ mActivityTestRule.startMainActivityOnBlankPage();
}
/**
* Verify the Data Saver item does not show when the feature isn't on, and the proxy is enabled.
*/
+ @Test
@SmallTest
- @UiThreadTest
@CommandLineFlags.Add("disable-field-trial-config")
@Feature({"Browser", "Main"})
- public void testMenuDataSaverNoFeature() {
- ContextUtils.getAppSharedPreferences().edit().clear().apply();
- assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
- DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
- getActivity().getApplicationContext(), true);
- assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
+ public void testMenuDataSaverNoFeature() throws Throwable {
+ mActivityTestRule.runOnUiThread(new Runnable() {
+ @Override
+ public void run() {
+ ContextUtils.getAppSharedPreferences().edit().clear().apply();
+ Assert.assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
+ DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
+ mActivityTestRule.getActivity().getApplicationContext(), true);
+ Assert.assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
+ }
+ });
}
/**
* Verify the Data Saver footer shows with the flag when the proxy is on.
*/
+ @Test
@SmallTest
- @UiThreadTest
@CommandLineFlags.Add({"enable-features=DataReductionProxyMainMenu",
"disable-field-trial-config"})
@Feature({"Browser", "Main"})
- public void testMenuDataSaver() {
- ContextUtils.getAppSharedPreferences().edit().clear().apply();
- // Data Saver hasn't been turned on, the footer shouldn't show.
- assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
-
- // Turn Data Saver on, the footer should show.
- DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
- getActivity().getApplicationContext(), true);
- assertEquals(R.layout.data_reduction_main_menu_footer,
- mAppMenuHandler.getDelegate().getFooterResourceId());
-
- // Ensure the footer is removed if the proxy is turned off.
- DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
- getActivity().getApplicationContext(), false);
- assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
+ public void testMenuDataSaver() throws Throwable {
+ mActivityTestRule.runOnUiThread(new Runnable() {
+ @Override
+ public void run() {
+ ContextUtils.getAppSharedPreferences().edit().clear().apply();
+ // Data Saver hasn't been turned on, the footer shouldn't show.
+ Assert.assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
+
+ // Turn Data Saver on, the footer should show.
+ DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
+ mActivityTestRule.getActivity().getApplicationContext(), true);
+ Assert.assertEquals(R.layout.data_reduction_main_menu_footer,
+ mAppMenuHandler.getDelegate().getFooterResourceId());
+
+ // Ensure the footer is removed if the proxy is turned off.
+ DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
+ mActivityTestRule.getActivity().getApplicationContext(), false);
+ Assert.assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
+ }
+ });
}
/**
* Verify the Data Saver footer shows with the flag when the proxy turns on and remains in the
* main menu.
*/
+ @Test
@SmallTest
- @UiThreadTest
@CommandLineFlags.Add({"enable-features=DataReductionProxyMainMenu<DataReductionProxyMainMenu",
"force-fieldtrials=DataReductionProxyMainMenu/Enabled",
"force-fieldtrial-params=DataReductionProxyMainMenu.Enabled:"
+ "persistent_menu_item_enabled/true",
"disable-field-trial-config"})
@Feature({"Browser", "Main"})
- public void testMenuDataSaverPersistent() {
- ContextUtils.getAppSharedPreferences().edit().clear().apply();
- // Data Saver hasn't been turned on, the footer shouldn't show.
- assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
-
- // Turn Data Saver on, the footer should show.
- DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
- getActivity().getApplicationContext(), true);
- assertEquals(R.layout.data_reduction_main_menu_footer,
- mAppMenuHandler.getDelegate().getFooterResourceId());
-
- // Ensure the footer remains if the proxy is turned off.
- DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
- getActivity().getApplicationContext(), false);
- assertEquals(R.layout.data_reduction_main_menu_footer,
- mAppMenuHandler.getDelegate().getFooterResourceId());
+ public void testMenuDataSaverPersistent() throws Throwable {
+ mActivityTestRule.runOnUiThread(new Runnable() {
+ @Override
+ public void run() {
+ ContextUtils.getAppSharedPreferences().edit().clear().apply();
+ // Data Saver hasn't been turned on, the footer shouldn't show.
+ Assert.assertEquals(0, mAppMenuHandler.getDelegate().getFooterResourceId());
+
+ // Turn Data Saver on, the footer should show.
+ DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
+ mActivityTestRule.getActivity().getApplicationContext(), true);
+ Assert.assertEquals(R.layout.data_reduction_main_menu_footer,
+ mAppMenuHandler.getDelegate().getFooterResourceId());
+
+ // Ensure the footer remains if the proxy is turned off.
+ DataReductionProxySettings.getInstance().setDataReductionProxyEnabled(
+ mActivityTestRule.getActivity().getApplicationContext(), false);
+ Assert.assertEquals(R.layout.data_reduction_main_menu_footer,
+ mAppMenuHandler.getDelegate().getFooterResourceId());
+ }
+ });
}
}

Powered by Google App Engine
This is Rietveld 408576698