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

Unified Diff: chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTestBase.java

Issue 1122373004: [Sync] Test signing in and back out on Android (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 5 years, 7 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 | « chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTestBase.java
diff --git a/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTestBase.java b/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTestBase.java
index 6b429df345bbf46893a6bc62bfe3ccd23b60ac54..f28ab17b589073ce468cd6b490c649203277266a 100644
--- a/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTestBase.java
+++ b/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTestBase.java
@@ -5,7 +5,6 @@
package org.chromium.chrome.browser.sync;
import android.accounts.Account;
-import android.app.Activity;
import android.content.Context;
import org.chromium.base.ThreadUtils;
@@ -13,6 +12,7 @@ import org.chromium.chrome.browser.identity.UniqueIdentificationGenerator;
import org.chromium.chrome.browser.identity.UniqueIdentificationGeneratorFactory;
import org.chromium.chrome.browser.identity.UuidBasedUniqueIdentificationGenerator;
import org.chromium.chrome.browser.signin.AccountIdProvider;
+import org.chromium.chrome.browser.signin.SigninManager;
import org.chromium.chrome.shell.ChromeShellTestBase;
import org.chromium.chrome.test.util.browser.sync.SyncTestUtil;
import org.chromium.content.browser.test.util.Criteria;
@@ -131,15 +131,7 @@ public class SyncTestBase extends ChromeShellTestBase {
}, true);
SyncTestUtil.verifySyncIsSignedOut(getActivity());
-
- final Activity activity = launchChromeShellWithBlankPage();
- ThreadUtils.runOnUiThreadBlocking(new Runnable() {
- @Override
- public void run() {
- mSyncController.signIn(activity, SyncTestUtil.DEFAULT_TEST_ACCOUNT);
- }
- });
-
+ signIn(defaultTestAccount);
SyncTestUtil.verifySyncIsSignedIn(mContext, defaultTestAccount);
assertTrue("Sync everything should be enabled",
SyncTestUtil.isSyncEverythingEnabled(mContext));
@@ -165,6 +157,24 @@ public class SyncTestBase extends ChromeShellTestBase {
getInstrumentation().waitForIdleSync();
}
+ protected void signIn(final Account account) {
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ mSyncController.signIn(getActivity(), account.name);
+ }
+ });
+ }
+
+ protected void signOut() throws InterruptedException {
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ @Override
+ public void run() {
+ SigninManager.get(mContext).signOut(getActivity(), null);
+ }
+ });
+ }
+
protected void waitForSyncInitialized() throws InterruptedException {
assertTrue(CriteriaHelper.pollForCriteria(new Criteria() {
@Override
« no previous file with comments | « chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncTest.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698