Commit 01e7b50e authored by boliu's avatar boliu Committed by Commit bot

Mark flaky chrome_public tests with @RetryOnFailure Batch 6

BUG=619055
TBR=tedchoc@chromium.org

Review-Url: https://codereview.chromium.org/2337973005
Cr-Commit-Position: refs/heads/master@{#418642}
parent 1e6e9e83
...@@ -11,6 +11,7 @@ import android.test.suitebuilder.annotation.MediumTest; ...@@ -11,6 +11,7 @@ import android.test.suitebuilder.annotation.MediumTest;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.base.test.util.TestFileUtil; import org.chromium.base.test.util.TestFileUtil;
import org.chromium.base.test.util.UrlUtils; import org.chromium.base.test.util.UrlUtils;
import org.chromium.chrome.test.ChromeActivityTestCaseBase; import org.chromium.chrome.test.ChromeActivityTestCaseBase;
...@@ -23,6 +24,7 @@ import java.util.concurrent.Callable; ...@@ -23,6 +24,7 @@ import java.util.concurrent.Callable;
/** Test suite for different Android URL schemes. */ /** Test suite for different Android URL schemes. */
@RetryOnFailure
public class UrlSchemeTest extends ChromeActivityTestCaseBase<ChromeActivity> { public class UrlSchemeTest extends ChromeActivityTestCaseBase<ChromeActivity> {
public UrlSchemeTest() { public UrlSchemeTest() {
......
...@@ -11,6 +11,7 @@ import android.widget.Button; ...@@ -11,6 +11,7 @@ import android.widget.Button;
import android.widget.ListView; import android.widget.ListView;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.test.ChromeActivityTestCaseBase; import org.chromium.chrome.test.ChromeActivityTestCaseBase;
import org.chromium.components.security_state.ConnectionSecurityLevel; import org.chromium.components.security_state.ConnectionSecurityLevel;
...@@ -24,6 +25,7 @@ import java.util.concurrent.Callable; ...@@ -24,6 +25,7 @@ import java.util.concurrent.Callable;
/** /**
* Tests for the UsbChooserDialog class. * Tests for the UsbChooserDialog class.
*/ */
@RetryOnFailure
public class UsbChooserDialogTest extends ChromeActivityTestCaseBase<ChromeActivity> { public class UsbChooserDialogTest extends ChromeActivityTestCaseBase<ChromeActivity> {
/** /**
* Works like the UsbChooserDialog class, but records calls to native methods instead of * Works like the UsbChooserDialog class, but records calls to native methods instead of
......
...@@ -9,6 +9,7 @@ import android.test.suitebuilder.annotation.LargeTest; ...@@ -9,6 +9,7 @@ import android.test.suitebuilder.annotation.LargeTest;
import org.chromium.base.test.util.DisableIf; import org.chromium.base.test.util.DisableIf;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.ChromeActivity; import org.chromium.chrome.browser.ChromeActivity;
import org.chromium.chrome.browser.tab.Tab; import org.chromium.chrome.browser.tab.Tab;
import org.chromium.chrome.test.ChromeActivityTestCaseBase; import org.chromium.chrome.test.ChromeActivityTestCaseBase;
...@@ -30,6 +31,7 @@ public class VideoTest extends ChromeActivityTestCaseBase<ChromeActivity> { ...@@ -30,6 +31,7 @@ public class VideoTest extends ChromeActivityTestCaseBase<ChromeActivity> {
@DisableIf.Build(sdk_is_less_than = 19, message = "crbug.com/582067") @DisableIf.Build(sdk_is_less_than = 19, message = "crbug.com/582067")
@Feature({"Media", "Media-Video", "Main"}) @Feature({"Media", "Media-Video", "Main"})
@LargeTest @LargeTest
@RetryOnFailure
public void testLoadMediaUrl() throws InterruptedException, TimeoutException { public void testLoadMediaUrl() throws InterruptedException, TimeoutException {
EmbeddedTestServer testServer = EmbeddedTestServer.createAndStartFileServer( EmbeddedTestServer testServer = EmbeddedTestServer.createAndStartFileServer(
getInstrumentation().getContext(), Environment.getExternalStorageDirectory()); getInstrumentation().getContext(), Environment.getExternalStorageDirectory());
......
...@@ -12,6 +12,7 @@ import org.chromium.base.ContextUtils; ...@@ -12,6 +12,7 @@ import org.chromium.base.ContextUtils;
import org.chromium.base.metrics.RecordHistogram; import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.test.util.AdvancedMockContext; import org.chromium.base.test.util.AdvancedMockContext;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import java.util.HashMap; import java.util.HashMap;
import java.util.HashSet; import java.util.HashSet;
...@@ -78,6 +79,7 @@ public class ActivityAssignerTest extends InstrumentationTestCase { ...@@ -78,6 +79,7 @@ public class ActivityAssignerTest extends InstrumentationTestCase {
@UiThreadTest @UiThreadTest
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testCorruptedPreferences() { public void testCorruptedPreferences() {
String wrongVariableType = "omgwtfbbq"; String wrongVariableType = "omgwtfbbq";
int index = ActivityAssigner.WEBAPP_NAMESPACE; int index = ActivityAssigner.WEBAPP_NAMESPACE;
......
...@@ -11,6 +11,7 @@ import android.test.suitebuilder.annotation.SmallTest; ...@@ -11,6 +11,7 @@ import android.test.suitebuilder.annotation.SmallTest;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeActivity; import org.chromium.chrome.browser.ChromeActivity;
import org.chromium.chrome.browser.tab.Tab; import org.chromium.chrome.browser.tab.Tab;
...@@ -46,6 +47,7 @@ public class AddToHomescreenDialogTest extends ChromeActivityTestCaseBase<Chrome ...@@ -46,6 +47,7 @@ public class AddToHomescreenDialogTest extends ChromeActivityTestCaseBase<Chrome
@SmallTest @SmallTest
@Feature("{Webapp}") @Feature("{Webapp}")
@RetryOnFailure
public void testSmoke() throws InterruptedException { public void testSmoke() throws InterruptedException {
ThreadUtils.runOnUiThreadBlocking(new Runnable() { ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override @Override
......
...@@ -15,6 +15,7 @@ import org.chromium.base.ThreadUtils; ...@@ -15,6 +15,7 @@ import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.CommandLineFlags; import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.Restriction; import org.chromium.base.test.util.Restriction;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.base.test.util.UrlUtils; import org.chromium.base.test.util.UrlUtils;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeActivity; import org.chromium.chrome.browser.ChromeActivity;
...@@ -33,6 +34,7 @@ import java.util.concurrent.Callable; ...@@ -33,6 +34,7 @@ import java.util.concurrent.Callable;
/** /**
* Tests org.chromium.chrome.browser.webapps.AddToHomescreenManager and its C++ counterpart. * Tests org.chromium.chrome.browser.webapps.AddToHomescreenManager and its C++ counterpart.
*/ */
@RetryOnFailure
public class AddToHomescreenManagerTest extends ChromeActivityTestCaseBase<ChromeActivity> { public class AddToHomescreenManagerTest extends ChromeActivityTestCaseBase<ChromeActivity> {
private static final String WEBAPP_ACTION_NAME = "WEBAPP_ACTION"; private static final String WEBAPP_ACTION_NAME = "WEBAPP_ACTION";
......
...@@ -11,6 +11,7 @@ import android.test.suitebuilder.annotation.MediumTest; ...@@ -11,6 +11,7 @@ import android.test.suitebuilder.annotation.MediumTest;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.tab.Tab; import org.chromium.chrome.browser.tab.Tab;
import org.chromium.chrome.test.ChromeTabbedActivityTestBase; import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
import org.chromium.chrome.test.util.browser.TabLoadObserver; import org.chromium.chrome.test.util.browser.TabLoadObserver;
...@@ -121,6 +122,7 @@ public class ManifestUpgradeDetectorFetcherTest extends ChromeTabbedActivityTest ...@@ -121,6 +122,7 @@ public class ManifestUpgradeDetectorFetcherTest extends ChromeTabbedActivityTest
*/ */
@MediumTest @MediumTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testLaunchWithDifferentManifestUrl() throws Exception { public void testLaunchWithDifferentManifestUrl() throws Exception {
CallbackWaiter waiter = new CallbackWaiter(); CallbackWaiter waiter = new CallbackWaiter();
startManifestUpgradeDetectorFetcher(mTestServer.getURL(WEB_MANIFEST_SCOPE), startManifestUpgradeDetectorFetcher(mTestServer.getURL(WEB_MANIFEST_SCOPE),
......
...@@ -14,6 +14,7 @@ import android.view.View; ...@@ -14,6 +14,7 @@ import android.view.View;
import org.chromium.base.ApplicationStatus; import org.chromium.base.ApplicationStatus;
import org.chromium.base.ContextUtils; import org.chromium.base.ContextUtils;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.base.test.util.UrlUtils; import org.chromium.base.test.util.UrlUtils;
import org.chromium.blink_public.platform.WebDisplayMode; import org.chromium.blink_public.platform.WebDisplayMode;
import org.chromium.chrome.browser.ChromeActivity; import org.chromium.chrome.browser.ChromeActivity;
...@@ -41,6 +42,7 @@ import org.chromium.content_public.common.ScreenOrientationValues; ...@@ -41,6 +42,7 @@ import org.chromium.content_public.common.ScreenOrientationValues;
* FLAG_ACTIVITY_NEW_DOCUMENT mechanism. Moreover, we don't have access to the task list pre-L so * FLAG_ACTIVITY_NEW_DOCUMENT mechanism. Moreover, we don't have access to the task list pre-L so
* we have to assume that any non-running WebappActivities are not listed in Android's Overview. * we have to assume that any non-running WebappActivities are not listed in Android's Overview.
*/ */
@RetryOnFailure
public class WebappModeTest extends MultiActivityTestBase { public class WebappModeTest extends MultiActivityTestBase {
private static final String WEBAPP_1_ID = "webapp_id_1"; private static final String WEBAPP_1_ID = "webapp_id_1";
private static final String WEBAPP_1_URL = UrlUtils.encodeHtmlDataUri( private static final String WEBAPP_1_URL = UrlUtils.encodeHtmlDataUri(
......
...@@ -13,6 +13,7 @@ import android.widget.ImageView; ...@@ -13,6 +13,7 @@ import android.widget.ImageView;
import org.chromium.base.metrics.RecordHistogram; import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.ShortcutHelper; import org.chromium.chrome.browser.ShortcutHelper;
import org.chromium.chrome.browser.metrics.WebappUma; import org.chromium.chrome.browser.metrics.WebappUma;
...@@ -48,6 +49,7 @@ public class WebappSplashScreenHomescreenIconTest extends WebappActivityTestBase ...@@ -48,6 +49,7 @@ public class WebappSplashScreenHomescreenIconTest extends WebappActivityTestBase
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testUmaFallbackIcon() { public void testUmaFallbackIcon() {
assertEquals(1, RecordHistogram.getHistogramValueCountForTesting( assertEquals(1, RecordHistogram.getHistogramValueCountForTesting(
WebappUma.HISTOGRAM_SPLASHSCREEN_ICON_TYPE, WebappUma.HISTOGRAM_SPLASHSCREEN_ICON_TYPE,
......
...@@ -22,6 +22,7 @@ import org.chromium.base.ApiCompatibilityUtils; ...@@ -22,6 +22,7 @@ import org.chromium.base.ApiCompatibilityUtils;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.metrics.RecordHistogram; import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.ShortcutHelper; import org.chromium.chrome.browser.ShortcutHelper;
import org.chromium.chrome.browser.metrics.WebappUma; import org.chromium.chrome.browser.metrics.WebappUma;
...@@ -53,6 +54,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -53,6 +54,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testDefaultBackgroundColor() throws Exception { public void testDefaultBackgroundColor() throws Exception {
startWebappActivity(); startWebappActivity();
ViewGroup splashScreen = waitUntilSplashScreenAppears(); ViewGroup splashScreen = waitUntilSplashScreenAppears();
...@@ -66,6 +68,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -66,6 +68,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@TargetApi(Build.VERSION_CODES.LOLLIPOP) @TargetApi(Build.VERSION_CODES.LOLLIPOP)
@RetryOnFailure
public void testThemeColorWhenNotSpecified() throws Exception { public void testThemeColorWhenNotSpecified() throws Exception {
startWebappActivity(); startWebappActivity();
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.LOLLIPOP) return; if (Build.VERSION.SDK_INT < Build.VERSION_CODES.LOLLIPOP) return;
...@@ -75,6 +78,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -75,6 +78,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testHidesAfterFirstPaint() throws Exception { public void testHidesAfterFirstPaint() throws Exception {
startWebappActivity(); startWebappActivity();
assertTrue(getActivity().isSplashScreenVisibleForTests()); assertTrue(getActivity().isSplashScreenVisibleForTests());
...@@ -91,6 +95,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -91,6 +95,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testHidesAfterCrash() throws Exception { public void testHidesAfterCrash() throws Exception {
startWebappActivity(); startWebappActivity();
assertTrue(getActivity().isSplashScreenVisibleForTests()); assertTrue(getActivity().isSplashScreenVisibleForTests());
...@@ -107,6 +112,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -107,6 +112,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testHidesAfterLoadCompletes() throws Exception { public void testHidesAfterLoadCompletes() throws Exception {
startWebappActivity(); startWebappActivity();
assertTrue(getActivity().isSplashScreenVisibleForTests()); assertTrue(getActivity().isSplashScreenVisibleForTests());
...@@ -139,6 +145,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -139,6 +145,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testHidesAfterMultipleEvents() throws Exception { public void testHidesAfterMultipleEvents() throws Exception {
startWebappActivity(); startWebappActivity();
assertTrue(getActivity().isSplashScreenVisibleForTests()); assertTrue(getActivity().isSplashScreenVisibleForTests());
...@@ -159,6 +166,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -159,6 +166,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testUmaOnNativeLoad() throws Exception { public void testUmaOnNativeLoad() throws Exception {
startWebappActivity(); startWebappActivity();
...@@ -192,6 +200,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -192,6 +200,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testUmaWhenSplashHides() throws Exception { public void testUmaWhenSplashHides() throws Exception {
startWebappActivity(); startWebappActivity();
ThreadUtils.runOnUiThread(new Runnable() { ThreadUtils.runOnUiThread(new Runnable() {
...@@ -220,6 +229,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -220,6 +229,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testRegularSplashScreenAppears() throws Exception { public void testRegularSplashScreenAppears() throws Exception {
// Register a properly-sized icon for the splash screen. // Register a properly-sized icon for the splash screen.
Context context = getInstrumentation().getTargetContext(); Context context = getInstrumentation().getTargetContext();
...@@ -246,6 +256,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase { ...@@ -246,6 +256,7 @@ public class WebappSplashScreenTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testSmallSplashScreenAppears() throws Exception { public void testSmallSplashScreenAppears() throws Exception {
// Register a smaller icon for the splash screen. // Register a smaller icon for the splash screen.
Context context = getInstrumentation().getTargetContext(); Context context = getInstrumentation().getTargetContext();
......
...@@ -13,6 +13,7 @@ import android.test.suitebuilder.annotation.SmallTest; ...@@ -13,6 +13,7 @@ import android.test.suitebuilder.annotation.SmallTest;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.metrics.RecordHistogram; import org.chromium.base.metrics.RecordHistogram;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.ShortcutHelper; import org.chromium.chrome.browser.ShortcutHelper;
import org.chromium.chrome.browser.metrics.WebappUma; import org.chromium.chrome.browser.metrics.WebappUma;
import org.chromium.chrome.browser.tab.TabTestUtils; import org.chromium.chrome.browser.tab.TabTestUtils;
...@@ -54,6 +55,7 @@ public class WebappSplashScreenThemeColorTest extends WebappActivityTestBase { ...@@ -54,6 +55,7 @@ public class WebappSplashScreenThemeColorTest extends WebappActivityTestBase {
@SmallTest @SmallTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@TargetApi(Build.VERSION_CODES.LOLLIPOP) @TargetApi(Build.VERSION_CODES.LOLLIPOP)
@RetryOnFailure
public void testThemeColorNotUsedIfPagesHasOne() throws InterruptedException { public void testThemeColorNotUsedIfPagesHasOne() throws InterruptedException {
if (Build.VERSION.SDK_INT < Build.VERSION_CODES.LOLLIPOP) return; if (Build.VERSION.SDK_INT < Build.VERSION_CODES.LOLLIPOP) return;
......
...@@ -9,6 +9,7 @@ import android.test.UiThreadTest; ...@@ -9,6 +9,7 @@ import android.test.UiThreadTest;
import android.test.suitebuilder.annotation.MediumTest; import android.test.suitebuilder.annotation.MediumTest;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.browser.ShortcutHelper; import org.chromium.chrome.browser.ShortcutHelper;
import org.chromium.components.security_state.ConnectionSecurityLevel; import org.chromium.components.security_state.ConnectionSecurityLevel;
...@@ -36,6 +37,7 @@ public class WebappUrlBarTest extends WebappActivityTestBase { ...@@ -36,6 +37,7 @@ public class WebappUrlBarTest extends WebappActivityTestBase {
@UiThreadTest @UiThreadTest
@MediumTest @MediumTest
@Feature({"Webapps"}) @Feature({"Webapps"})
@RetryOnFailure
public void testUrlDisplay() { public void testUrlDisplay() {
final String scheme = "https://"; final String scheme = "https://";
final String host = "lorem.com"; final String host = "lorem.com";
......
...@@ -15,6 +15,7 @@ import org.chromium.base.CommandLine; ...@@ -15,6 +15,7 @@ import org.chromium.base.CommandLine;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.Restriction; import org.chromium.base.test.util.Restriction;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeSwitches; import org.chromium.chrome.browser.ChromeSwitches;
import org.chromium.chrome.browser.compositor.overlays.strip.StripLayoutTab; import org.chromium.chrome.browser.compositor.overlays.strip.StripLayoutTab;
...@@ -176,6 +177,7 @@ public class OverviewListLayoutTest extends ChromeTabbedActivityTestBase { ...@@ -176,6 +177,7 @@ public class OverviewListLayoutTest extends ChromeTabbedActivityTestBase {
@Restriction(ChromeRestriction.RESTRICTION_TYPE_PHONE) @Restriction(ChromeRestriction.RESTRICTION_TYPE_PHONE)
@MediumTest @MediumTest
@Feature({"Accessibility"}) @Feature({"Accessibility"})
@RetryOnFailure
public void testCanCloseWithCloseButton() throws InterruptedException, TimeoutException { public void testCanCloseWithCloseButton() throws InterruptedException, TimeoutException {
setupTabs(); setupTabs();
...@@ -311,6 +313,7 @@ public class OverviewListLayoutTest extends ChromeTabbedActivityTestBase { ...@@ -311,6 +313,7 @@ public class OverviewListLayoutTest extends ChromeTabbedActivityTestBase {
@Restriction(ChromeRestriction.RESTRICTION_TYPE_PHONE) @Restriction(ChromeRestriction.RESTRICTION_TYPE_PHONE)
@MediumTest @MediumTest
@Feature({"Accessibility"}) @Feature({"Accessibility"})
@RetryOnFailure
public void testCloseAllIncognito() throws InterruptedException { public void testCloseAllIncognito() throws InterruptedException {
setupTabs(); setupTabs();
newIncognitoTabsFromMenu(2); newIncognitoTabsFromMenu(2);
...@@ -391,6 +394,7 @@ public class OverviewListLayoutTest extends ChromeTabbedActivityTestBase { ...@@ -391,6 +394,7 @@ public class OverviewListLayoutTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"Accessibility"}) @Feature({"Accessibility"})
@RetryOnFailure
public void testObservesTitleChanges() throws Exception { public void testObservesTitleChanges() throws Exception {
loadUrl(PAGE_1_HTML); loadUrl(PAGE_1_HTML);
......
...@@ -13,6 +13,7 @@ import android.view.View; ...@@ -13,6 +13,7 @@ import android.view.View;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.Restriction; import org.chromium.base.test.util.Restriction;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeTabbedActivity; import org.chromium.chrome.browser.ChromeTabbedActivity;
import org.chromium.chrome.test.ChromeActivityTestCaseBase; import org.chromium.chrome.test.ChromeActivityTestCaseBase;
...@@ -33,6 +34,7 @@ import java.util.concurrent.atomic.AtomicReference; ...@@ -33,6 +34,7 @@ import java.util.concurrent.atomic.AtomicReference;
/** /**
* Tests related to the ToolbarProgressBar. * Tests related to the ToolbarProgressBar.
*/ */
@RetryOnFailure
public class ToolbarProgressBarTest extends ChromeActivityTestCaseBase<ChromeTabbedActivity> { public class ToolbarProgressBarTest extends ChromeActivityTestCaseBase<ChromeTabbedActivity> {
static final int TEST_WAIT_TIME_MS = 60000; static final int TEST_WAIT_TIME_MS = 60000;
......
...@@ -26,6 +26,7 @@ import android.widget.TextView; ...@@ -26,6 +26,7 @@ import android.widget.TextView;
import org.chromium.base.ThreadUtils; import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.R; import org.chromium.chrome.R;
import org.chromium.chrome.test.ChromeTabbedActivityTestBase; import org.chromium.chrome.test.ChromeTabbedActivityTestBase;
import org.chromium.chrome.test.util.MenuUtils; import org.chromium.chrome.test.util.MenuUtils;
...@@ -150,6 +151,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -150,6 +151,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"FindInPage", "Main"}) @Feature({"FindInPage", "Main"})
@RetryOnFailure
public void testFind() throws InterruptedException { public void testFind() throws InterruptedException {
loadTestAndVerifyFindInPage("pitts", "1/7"); loadTestAndVerifyFindInPage("pitts", "1/7");
} }
...@@ -159,6 +161,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -159,6 +161,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testFind101() throws InterruptedException { public void testFind101() throws InterruptedException {
loadTestAndVerifyFindInPage("it", "1/101"); loadTestAndVerifyFindInPage("it", "1/101");
} }
...@@ -168,6 +171,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -168,6 +171,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testFindMultiLine() throws InterruptedException { public void testFindMultiLine() throws InterruptedException {
String multiLineSearchTerm = "This is the text of this document.\n" String multiLineSearchTerm = "This is the text of this document.\n"
+ " I am going to write the word \'Pitts\' 7 times. (That was one.)"; + " I am going to write the word \'Pitts\' 7 times. (That was one.)";
...@@ -180,6 +184,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -180,6 +184,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testFindMultiLineFalse() throws InterruptedException { public void testFindMultiLineFalse() throws InterruptedException {
String multiLineSearchTerm = "aThis is the text of this document.\n" String multiLineSearchTerm = "aThis is the text of this document.\n"
+ " I am going to write the word \'Pitts\' 7 times. (That was one.)"; + " I am going to write the word \'Pitts\' 7 times. (That was one.)";
...@@ -191,6 +196,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -191,6 +196,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testFindNext() throws InterruptedException { public void testFindNext() throws InterruptedException {
String query = "pitts"; String query = "pitts";
loadTestAndVerifyFindInPage(query, "1/7"); loadTestAndVerifyFindInPage(query, "1/7");
...@@ -220,6 +226,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -220,6 +226,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testResultsBarInitiallyVisible() throws InterruptedException { public void testResultsBarInitiallyVisible() throws InterruptedException {
loadUrl(mTestServer.getURL(FILEPATH)); loadUrl(mTestServer.getURL(FILEPATH));
findInPageFromMenu(); findInPageFromMenu();
...@@ -231,6 +238,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -231,6 +238,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testResultsBarVisibleAfterTypingText() throws InterruptedException { public void testResultsBarVisibleAfterTypingText() throws InterruptedException {
loadUrl(mTestServer.getURL(FILEPATH)); loadUrl(mTestServer.getURL(FILEPATH));
findInPageFromMenu(); findInPageFromMenu();
...@@ -289,6 +297,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -289,6 +297,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testFipTextNotRestoredIncognitoTab() throws InterruptedException { public void testFipTextNotRestoredIncognitoTab() throws InterruptedException {
newIncognitoTabFromMenu(); newIncognitoTabFromMenu();
loadTestAndVerifyFindInPage("pitts", "1/7"); loadTestAndVerifyFindInPage("pitts", "1/7");
...@@ -311,6 +320,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -311,6 +320,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@SmallTest @SmallTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testPastedTextStylingRemoved() throws InterruptedException { public void testPastedTextStylingRemoved() throws InterruptedException {
loadUrl(mTestServer.getURL(FILEPATH)); loadUrl(mTestServer.getURL(FILEPATH));
findInPageFromMenu(); findInPageFromMenu();
...@@ -365,6 +375,7 @@ public class FindTest extends ChromeTabbedActivityTestBase { ...@@ -365,6 +375,7 @@ public class FindTest extends ChromeTabbedActivityTestBase {
*/ */
@MediumTest @MediumTest
@Feature({"FindInPage"}) @Feature({"FindInPage"})
@RetryOnFailure
public void testBackKeyDismissesFind() throws InterruptedException { public void testBackKeyDismissesFind() throws InterruptedException {
loadUrl(mTestServer.getURL(FILEPATH)); loadUrl(mTestServer.getURL(FILEPATH));
findInPageFromMenu(); findInPageFromMenu();
......
...@@ -10,6 +10,7 @@ import android.test.suitebuilder.annotation.SmallTest; ...@@ -10,6 +10,7 @@ import android.test.suitebuilder.annotation.SmallTest;
import org.chromium.base.test.util.EnormousTest; import org.chromium.base.test.util.EnormousTest;
import org.chromium.base.test.util.FlakyTest; import org.chromium.base.test.util.FlakyTest;
import org.chromium.base.test.util.Restriction; import org.chromium.base.test.util.Restriction;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.base.test.util.parameter.Parameter; import org.chromium.base.test.util.parameter.Parameter;
import org.chromium.base.test.util.parameter.ParameterizedTest; import org.chromium.base.test.util.parameter.ParameterizedTest;
import org.chromium.base.test.util.parameter.parameters.MethodParameter; import org.chromium.base.test.util.parameter.parameters.MethodParameter;
...@@ -44,6 +45,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -44,6 +45,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
} }
@SmallTest @SmallTest
@RetryOnFailure
public void testNoParameterizedTestAnnotation() { public void testNoParameterizedTestAnnotation() {
assertFalse("This is a parameterized test when it should not be.", getParameterReader() assertFalse("This is a parameterized test when it should not be.", getParameterReader()
.isParameterizedTest()); .isParameterizedTest());
...@@ -55,6 +57,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -55,6 +57,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@SmallTest @SmallTest
@ParameterizedTest() @ParameterizedTest()
@RetryOnFailure
public void testEmptyParameterizedTestAnnotation() { public void testEmptyParameterizedTestAnnotation() {
assertTrue("This is not a parameterized test.", getParameterReader() assertTrue("This is not a parameterized test.", getParameterReader()
.isParameterizedTest()); .isParameterizedTest());
...@@ -66,6 +69,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -66,6 +69,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@SmallTest @SmallTest
@ParameterizedTest(parameters = {}) @ParameterizedTest(parameters = {})
@RetryOnFailure
public void testParameterizedTestWithEmptyParameters() { public void testParameterizedTestWithEmptyParameters() {
assertTrue("This is not a parameterized test.", getParameterReader() assertTrue("This is not a parameterized test.", getParameterReader()
.isParameterizedTest()); .isParameterizedTest());
...@@ -77,6 +81,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -77,6 +81,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@SmallTest @SmallTest
@ParameterizedTest(parameters = {}) @ParameterizedTest(parameters = {})
@RetryOnFailure
public void testParameterDoesNotExist() { public void testParameterDoesNotExist() {
Parameter parameter = getParameterReader().getParameter(MethodParameter.PARAMETER_TAG); Parameter parameter = getParameterReader().getParameter(MethodParameter.PARAMETER_TAG);
assertNull("method-parameter should not exist.", parameter); assertNull("method-parameter should not exist.", parameter);
...@@ -84,6 +89,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -84,6 +89,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@SmallTest @SmallTest
@ParameterizedTest(parameters = {@Parameter(tag = MethodParameter.PARAMETER_TAG)}) @ParameterizedTest(parameters = {@Parameter(tag = MethodParameter.PARAMETER_TAG)})
@RetryOnFailure
public void testGetParameter() { public void testGetParameter() {
String expected = "method-parameter"; String expected = "method-parameter";
String actual = getParameterReader().getParameter(MethodParameter.PARAMETER_TAG).tag(); String actual = getParameterReader().getParameter(MethodParameter.PARAMETER_TAG).tag();
...@@ -92,6 +98,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -92,6 +98,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@SmallTest @SmallTest
@ParameterizedTest(parameters = {@Parameter(tag = MethodParameter.PARAMETER_TAG)}) @ParameterizedTest(parameters = {@Parameter(tag = MethodParameter.PARAMETER_TAG)})
@RetryOnFailure
public void testParameterArgumentDoesNotExist() { public void testParameterArgumentDoesNotExist() {
Parameter.Argument actual = getArgument("arg"); Parameter.Argument actual = getArgument("arg");
assertNull("arg should not exist.", actual); assertNull("arg should not exist.", actual);
...@@ -101,6 +108,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -101,6 +108,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@ParameterizedTest(parameters = { @ParameterizedTest(parameters = {
@Parameter(tag = MethodParameter.PARAMETER_TAG, @Parameter(tag = MethodParameter.PARAMETER_TAG,
arguments = {@Parameter.Argument(name = "string", stringVar = "value")})}) arguments = {@Parameter.Argument(name = "string", stringVar = "value")})})
@RetryOnFailure
public void testMethodParametersWithOneStringValue() { public void testMethodParametersWithOneStringValue() {
String expected = "value"; String expected = "value";
String actual = getArgument("string").stringVar(); String actual = getArgument("string").stringVar();
...@@ -111,6 +119,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -111,6 +119,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@ParameterizedTest(parameters = { @ParameterizedTest(parameters = {
@Parameter(tag = MethodParameter.PARAMETER_TAG, @Parameter(tag = MethodParameter.PARAMETER_TAG,
arguments = {@Parameter.Argument(name = "int", intVar = 0)})}) arguments = {@Parameter.Argument(name = "int", intVar = 0)})})
@RetryOnFailure
public void testMethodParametersWithOneIntValue() { public void testMethodParametersWithOneIntValue() {
int expected = 0; int expected = 0;
int actual = getArgument("int").intVar(); int actual = getArgument("int").intVar();
...@@ -123,6 +132,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -123,6 +132,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
arguments = { arguments = {
@Parameter.Argument(name = "intArray", intArray = {5, 10, -6, 0, -1})}) @Parameter.Argument(name = "intArray", intArray = {5, 10, -6, 0, -1})})
}) })
@RetryOnFailure
public void testMethodParametersWithOneIntArrayValue() { public void testMethodParametersWithOneIntArrayValue() {
int[] expected = new int[] {5, 10, -6, 0, -1}; int[] expected = new int[] {5, 10, -6, 0, -1};
int[] actual = getArgument("intArray").intArray(); int[] actual = getArgument("intArray").intArray();
...@@ -136,6 +146,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -136,6 +146,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@Parameter(tag = MethodParameter.PARAMETER_TAG, @Parameter(tag = MethodParameter.PARAMETER_TAG,
arguments = {@Parameter.Argument(name = "stringArray", stringArray = { arguments = {@Parameter.Argument(name = "stringArray", stringArray = {
"apple", "banana", "orange", "melon", "lemon"})})}) "apple", "banana", "orange", "melon", "lemon"})})})
@RetryOnFailure
public void testMethodParametersWithOneStringArrayValue() { public void testMethodParametersWithOneStringArrayValue() {
String[] expected = new String[] {"apple", "banana", "orange", "melon", "lemon"}; String[] expected = new String[] {"apple", "banana", "orange", "melon", "lemon"};
String[] actual = getArgument("stringArray").stringArray(); String[] actual = getArgument("stringArray").stringArray();
...@@ -159,6 +170,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -159,6 +170,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@Parameter.Argument(name = "intArray1", intArray = { @Parameter.Argument(name = "intArray1", intArray = {
2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37}), 2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37}),
@Parameter.Argument(name = "intArray2", intArray = {4})})}) @Parameter.Argument(name = "intArray2", intArray = {4})})})
@RetryOnFailure
public void testMethodParametersWithMultipleArguments1() { public void testMethodParametersWithMultipleArguments1() {
String stringVar = getArgument("string1").stringVar(); String stringVar = getArgument("string1").stringVar();
assertEquals(mismatchMessage("string1"), "has vowel", stringVar); assertEquals(mismatchMessage("string1"), "has vowel", stringVar);
...@@ -198,6 +210,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -198,6 +210,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@Parameter.Argument(name = "string2", stringVar = "blahblah"), @Parameter.Argument(name = "string2", stringVar = "blahblah"),
@Parameter.Argument(name = "int1", intVar = 4), @Parameter.Argument(name = "int1", intVar = 4),
@Parameter.Argument(name = "int2", intVar = 0)})}) @Parameter.Argument(name = "int2", intVar = 0)})})
@RetryOnFailure
public void testMethodParametersWithMultipleArguments2() { public void testMethodParametersWithMultipleArguments2() {
assertEquals("bar variable should equals \"testvalue\"", "testvalue", assertEquals("bar variable should equals \"testvalue\"", "testvalue",
getArgument("string1").stringVar()); getArgument("string1").stringVar());
...@@ -219,6 +232,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -219,6 +232,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@Parameter.Argument(name = "string2", stringVar = "blahblah"), @Parameter.Argument(name = "string2", stringVar = "blahblah"),
@Parameter.Argument(name = "int1", intVar = 4), @Parameter.Argument(name = "int1", intVar = 4),
@Parameter.Argument(name = "int2", intVar = 0)})})}) @Parameter.Argument(name = "int2", intVar = 0)})})})
@RetryOnFailure
public void testParameterArgumentsWithParameterSetOfOneTest() { public void testParameterArgumentsWithParameterSetOfOneTest() {
assertEquals("bar variable should equals \"testvalue\"", "testvalue", assertEquals("bar variable should equals \"testvalue\"", "testvalue",
getArgument("string1").stringVar()); getArgument("string1").stringVar());
...@@ -300,6 +314,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -300,6 +314,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
arguments = { arguments = {
@Parameter.Argument(name = "input", intVar = 10), @Parameter.Argument(name = "input", intVar = 10),
@Parameter.Argument(name = "output", intVar = 34)})})}) @Parameter.Argument(name = "output", intVar = 34)})})})
@RetryOnFailure
public void testParameterArgumentsWithParameterSetOfMoreThanOneTest() { public void testParameterArgumentsWithParameterSetOfMoreThanOneTest() {
int input = getArgument("input").intVar(); int input = getArgument("input").intVar();
int expected = getArgument("output").intVar(); int expected = getArgument("output").intVar();
...@@ -326,6 +341,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -326,6 +341,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
intArray = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10}), intArray = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10}),
@Parameter.Argument(name = "expected", @Parameter.Argument(name = "expected",
intArray = {0, 1, 1, 2, 3, 5, 8, 13, 21, 34})})})}) intArray = {0, 1, 1, 2, 3, 5, 8, 13, 21, 34})})})})
@RetryOnFailure
public void testSingleTestParameterArgumentsWithParameterSet() { public void testSingleTestParameterArgumentsWithParameterSet() {
int[] input = getArgument("input").intArray(); int[] input = getArgument("input").intArray();
int[] expected = getArgument("expected").intArray(); int[] expected = getArgument("expected").intArray();
...@@ -352,6 +368,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -352,6 +368,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
intArray = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10}), intArray = {1, 2, 3, 4, 5, 6, 7, 8, 9, 10}),
@Parameter.Argument(name = "expected", @Parameter.Argument(name = "expected",
intArray = {0, 1, 1, 2, 3, 5, 8, 13, 21, 34})})}) intArray = {0, 1, 1, 2, 3, 5, 8, 13, 21, 34})})})
@RetryOnFailure
public void testSingleTestParameterArgumentsWithoutParameterSet() { public void testSingleTestParameterArgumentsWithoutParameterSet() {
int[] input = getArgument("input").intArray(); int[] input = getArgument("input").intArray();
int[] expected = getArgument("expected").intArray(); int[] expected = getArgument("expected").intArray();
...@@ -392,6 +409,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -392,6 +409,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
* TODO (crbug.com/522503): Merge tests together when this is fixed. * TODO (crbug.com/522503): Merge tests together when this is fixed.
*/ */
@SmallTest @SmallTest
@RetryOnFailure
public void testActivityIsNotSignedInOnAppOrFakeOSorGoogleOS() { public void testActivityIsNotSignedInOnAppOrFakeOSorGoogleOS() {
assertFalse("Should not be signed into app.", assertFalse("Should not be signed into app.",
mAddFakeAccountToAppParameter.isSignedIn()); mAddFakeAccountToAppParameter.isSignedIn());
...@@ -404,6 +422,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -404,6 +422,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@SmallTest @SmallTest
@ParameterizedTest(parameters = { @ParameterizedTest(parameters = {
@Parameter(tag = AddFakeAccountToAppParameter.PARAMETER_TAG)}) @Parameter(tag = AddFakeAccountToAppParameter.PARAMETER_TAG)})
@RetryOnFailure
public void testIsSignedInOnApp() { public void testIsSignedInOnApp() {
assertTrue("Should not be signed into app.", assertTrue("Should not be signed into app.",
mAddFakeAccountToAppParameter.isSignedIn()); mAddFakeAccountToAppParameter.isSignedIn());
...@@ -416,6 +435,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -416,6 +435,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@SmallTest @SmallTest
@ParameterizedTest(parameters = { @ParameterizedTest(parameters = {
@Parameter(tag = AddFakeAccountToOsParameter.PARAMETER_TAG)}) @Parameter(tag = AddFakeAccountToOsParameter.PARAMETER_TAG)})
@RetryOnFailure
public void testIsSignedInOnFakeOS() { public void testIsSignedInOnFakeOS() {
assertFalse("Should not be signed in on app.", assertFalse("Should not be signed in on app.",
mAddFakeAccountToAppParameter.isSignedIn()); mAddFakeAccountToAppParameter.isSignedIn());
...@@ -451,6 +471,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase { ...@@ -451,6 +471,7 @@ public class ParametersOnMultiTest extends MultiActivityTestBase {
@ParameterizedTest(parameters = { @ParameterizedTest(parameters = {
@Parameter(tag = AddFakeAccountToAppParameter.PARAMETER_TAG), @Parameter(tag = AddFakeAccountToAppParameter.PARAMETER_TAG),
@Parameter(tag = AddFakeAccountToOsParameter.PARAMETER_TAG)}) @Parameter(tag = AddFakeAccountToOsParameter.PARAMETER_TAG)})
@RetryOnFailure
public void testIsSignedInOnFakeOSandApp() { public void testIsSignedInOnFakeOSandApp() {
assertTrue("Should be signed in on app.", assertTrue("Should be signed in on app.",
mAddFakeAccountToAppParameter.isSignedIn()); mAddFakeAccountToAppParameter.isSignedIn());
......
...@@ -10,6 +10,7 @@ import org.chromium.base.test.util.DisabledTest; ...@@ -10,6 +10,7 @@ import org.chromium.base.test.util.DisabledTest;
import org.chromium.base.test.util.EnormousTest; import org.chromium.base.test.util.EnormousTest;
import org.chromium.base.test.util.FlakyTest; import org.chromium.base.test.util.FlakyTest;
import org.chromium.base.test.util.Restriction; import org.chromium.base.test.util.Restriction;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.base.test.util.parameter.Parameter; import org.chromium.base.test.util.parameter.Parameter;
import org.chromium.base.test.util.parameter.ParameterizedTest; import org.chromium.base.test.util.parameter.ParameterizedTest;
import org.chromium.chrome.browser.ChromeActivity; import org.chromium.chrome.browser.ChromeActivity;
...@@ -18,6 +19,7 @@ import org.chromium.chrome.test.ChromeActivityTestCaseBase; ...@@ -18,6 +19,7 @@ import org.chromium.chrome.test.ChromeActivityTestCaseBase;
/** /**
* Tester class for implementation of Signin testing and ParameterizedTest. * Tester class for implementation of Signin testing and ParameterizedTest.
*/ */
@RetryOnFailure
public class SigninParametersTest extends ChromeActivityTestCaseBase<ChromeActivity> { public class SigninParametersTest extends ChromeActivityTestCaseBase<ChromeActivity> {
private static final String GOOGLE_ACCOUNT_USERNAME = "chromiumforandroid01@gmail.com"; private static final String GOOGLE_ACCOUNT_USERNAME = "chromiumforandroid01@gmail.com";
private static final String GOOGLE_ACCOUNT_PASSWORD = "chromeforandroid"; private static final String GOOGLE_ACCOUNT_PASSWORD = "chromeforandroid";
......
...@@ -12,6 +12,7 @@ import com.google.ipc.invalidation.external.client.types.ObjectId; ...@@ -12,6 +12,7 @@ import com.google.ipc.invalidation.external.client.types.ObjectId;
import org.chromium.base.CollectionUtil; import org.chromium.base.CollectionUtil;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import java.util.Arrays; import java.util.Arrays;
import java.util.Set; import java.util.Set;
...@@ -21,6 +22,7 @@ import java.util.Set; ...@@ -21,6 +22,7 @@ import java.util.Set;
* *
* @author dsmyers@google.com (Daniel Myers) * @author dsmyers@google.com (Daniel Myers)
*/ */
@RetryOnFailure
public class InvalidationPreferencesTest extends InstrumentationTestCase { public class InvalidationPreferencesTest extends InstrumentationTestCase {
@SmallTest @SmallTest
@Feature({"Sync"}) @Feature({"Sync"})
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment