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

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

Issue 2336803005: Mark flaky chrome_public tests with @RetryOnFailure Batch 5 (Closed)
Patch Set: Created 4 years, 3 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/signin/SigninHelperTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/signin/SigninHelperTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/signin/SigninHelperTest.java
index 3aa783669e4088642db6583b692014d6b092b716..f51391f894ad2b71afab067e2d35852c4db5459d 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/signin/SigninHelperTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/signin/SigninHelperTest.java
@@ -10,6 +10,7 @@ import android.test.suitebuilder.annotation.SmallTest;
import org.chromium.base.test.util.AdvancedMockContext;
import org.chromium.base.test.util.DisabledTest;
+import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.chrome.test.util.browser.signin.MockChangeEventChecker;
import org.chromium.components.sync.signin.AccountManagerHelper;
import org.chromium.components.sync.signin.ChromeSigninController;
@@ -35,6 +36,7 @@ public class SigninHelperTest extends InstrumentationTestCase {
}
@SmallTest
+ @RetryOnFailure
public void testAccountsChangedPref() {
assertEquals("Should never return true before the pref has ever been set.",
false, SigninHelper.checkAndClearAccountsChangedPref(mContext));
@@ -91,6 +93,7 @@ public class SigninHelperTest extends InstrumentationTestCase {
}
@SmallTest
+ @RetryOnFailure
public void testNotSignedInAccountRename2() {
setSignedInAccountName("A");
mEventChecker.insertRenameEvent("B", "C");
@@ -100,6 +103,7 @@ public class SigninHelperTest extends InstrumentationTestCase {
}
@SmallTest
+ @RetryOnFailure
public void testChainedAccountRename2() {
setSignedInAccountName("A");
mEventChecker.insertRenameEvent("Z", "Y"); // Unrelated.

Powered by Google App Engine
This is Rietveld 408576698