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

Unified Diff: components/sync/android/javatests/src/org/chromium/components/sync/AndroidSyncSettingsTest.java

Issue 2857103004: [Sync] Disable 2 flaky tests on Android (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/sync/android/javatests/src/org/chromium/components/sync/AndroidSyncSettingsTest.java
diff --git a/components/sync/android/javatests/src/org/chromium/components/sync/AndroidSyncSettingsTest.java b/components/sync/android/javatests/src/org/chromium/components/sync/AndroidSyncSettingsTest.java
index 83c115df3ac0ee51a1d59349fd1418a01b70b138..4187be76b70e66025c898db5de8ab6bfd1dead8e 100644
--- a/components/sync/android/javatests/src/org/chromium/components/sync/AndroidSyncSettingsTest.java
+++ b/components/sync/android/javatests/src/org/chromium/components/sync/AndroidSyncSettingsTest.java
@@ -13,6 +13,7 @@ import android.test.InstrumentationTestCase;
import org.chromium.base.Callback;
import org.chromium.base.ThreadUtils;
import org.chromium.base.test.util.CallbackHelper;
+import org.chromium.base.test.util.DisabledTest;
import org.chromium.base.test.util.Feature;
import org.chromium.components.signin.AccountManagerHelper;
import org.chromium.components.signin.test.util.AccountHolder;
@@ -152,8 +153,9 @@ public class AndroidSyncSettingsTest extends InstrumentationTestCase {
});
}
- @SmallTest
- @Feature({"Sync"})
+ // @SmallTest
+ // @Feature({"Sync"})
+ @DisabledTest(message = "crbug.com/717960")
public void testAccountInitialization() throws InterruptedException, TimeoutException {
// mAccount was set to be syncable and not have periodic syncs.
assertEquals(1, mSyncContentResolverDelegate.mSetIsSyncableCalls);
@@ -365,8 +367,9 @@ public class AndroidSyncSettingsTest extends InstrumentationTestCase {
mSyncSettingsObserver.receivedNotification());
}
- @SmallTest
- @Feature({"Sync"})
+ // @SmallTest
+ // @Feature({"Sync"})
+ @DisabledTest(message = "crbug.com/717960")
public void testIsSyncableOnSigninAndNotOnSignout()
throws InterruptedException, TimeoutException {
assertTrue(mSyncContentResolverDelegate.getIsSyncable(mAccount, mAuthority) == 1);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698