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

Side by Side Diff: chrome/android/sync_shell/javatests/src/chromium/chrome/browser/sync/SyncTest.java

Issue 964563002: Replace SetAuthenticatedUsername with SetAuthenticatedAccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@priv
Patch Set: Address review comments Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.chrome.browser.sync; 5 package org.chromium.chrome.browser.sync;
6 6
7 import android.accounts.Account; 7 import android.accounts.Account;
8 import android.app.Activity; 8 import android.app.Activity;
9 import android.content.Context;
9 import android.test.suitebuilder.annotation.LargeTest; 10 import android.test.suitebuilder.annotation.LargeTest;
10 import android.util.Log; 11 import android.util.Log;
11 12
12 import org.chromium.base.ActivityState; 13 import org.chromium.base.ActivityState;
13 import org.chromium.base.ApplicationStatus; 14 import org.chromium.base.ApplicationStatus;
14 import org.chromium.base.ThreadUtils; 15 import org.chromium.base.ThreadUtils;
15 import org.chromium.base.test.util.Feature; 16 import org.chromium.base.test.util.Feature;
16 import org.chromium.chrome.browser.identity.UniqueIdentificationGenerator; 17 import org.chromium.chrome.browser.identity.UniqueIdentificationGenerator;
17 import org.chromium.chrome.browser.identity.UniqueIdentificationGeneratorFactory ; 18 import org.chromium.chrome.browser.identity.UniqueIdentificationGeneratorFactory ;
18 import org.chromium.chrome.browser.identity.UuidBasedUniqueIdentificationGenerat or; 19 import org.chromium.chrome.browser.identity.UuidBasedUniqueIdentificationGenerat or;
20 import org.chromium.chrome.browser.signin.AccountIdProvider;
19 import org.chromium.chrome.shell.ChromeShellActivity; 21 import org.chromium.chrome.shell.ChromeShellActivity;
20 import org.chromium.chrome.shell.ChromeShellTestBase; 22 import org.chromium.chrome.shell.ChromeShellTestBase;
21 import org.chromium.chrome.test.util.browser.sync.SyncTestUtil; 23 import org.chromium.chrome.test.util.browser.sync.SyncTestUtil;
22 import org.chromium.content.browser.ContentViewCore; 24 import org.chromium.content.browser.ContentViewCore;
23 import org.chromium.content.browser.test.util.Criteria; 25 import org.chromium.content.browser.test.util.Criteria;
24 import org.chromium.content.browser.test.util.CriteriaHelper; 26 import org.chromium.content.browser.test.util.CriteriaHelper;
25 import org.chromium.content.browser.test.util.JavaScriptUtils; 27 import org.chromium.content.browser.test.util.JavaScriptUtils;
26 import org.chromium.content_public.browser.LoadUrlParams; 28 import org.chromium.content_public.browser.LoadUrlParams;
27 import org.chromium.sync.AndroidSyncSettings; 29 import org.chromium.sync.AndroidSyncSettings;
28 import org.chromium.sync.internal_api.pub.base.ModelType; 30 import org.chromium.sync.internal_api.pub.base.ModelType;
(...skipping 19 matching lines...) Expand all
48 private MockAccountManager mAccountManager; 50 private MockAccountManager mAccountManager;
49 private SyncController mSyncController; 51 private SyncController mSyncController;
50 private FakeServerHelper mFakeServerHelper; 52 private FakeServerHelper mFakeServerHelper;
51 53
52 @Override 54 @Override
53 protected void setUp() throws Exception { 55 protected void setUp() throws Exception {
54 super.setUp(); 56 super.setUp();
55 57
56 clearAppData(); 58 clearAppData();
57 59
60 // Setup fake mapper from accountNames to Ids.
61 ThreadUtils.runOnUiThreadBlocking(new Runnable() {
62 @Override
63 public void run() {
64 if (AccountIdProvider.getInstance() != null) {
65 return;
66 }
67
68 AccountIdProvider.setInstance(new AccountIdProvider() {
69 public String getAccountId(Context ctx, String accountName) {
70 return "gaia-id-" + accountName;
71 }
72 });
73 }
74 });
75
58 // Mock out the account manager on the device. 76 // Mock out the account manager on the device.
59 mContext = new SyncTestUtil.SyncTestContext(getInstrumentation().getTarg etContext()); 77 mContext = new SyncTestUtil.SyncTestContext(getInstrumentation().getTarg etContext());
60 mAccountManager = new MockAccountManager(mContext, getInstrumentation(). getContext()); 78 mAccountManager = new MockAccountManager(mContext, getInstrumentation(). getContext());
61 AccountManagerHelper.overrideAccountManagerHelperForTests(mContext, mAcc ountManager); 79 AccountManagerHelper.overrideAccountManagerHelperForTests(mContext, mAcc ountManager);
62 MockSyncContentResolverDelegate syncContentResolverDelegate = 80 MockSyncContentResolverDelegate syncContentResolverDelegate =
63 new MockSyncContentResolverDelegate(); 81 new MockSyncContentResolverDelegate();
64 syncContentResolverDelegate.setMasterSyncAutomatically(true); 82 syncContentResolverDelegate.setMasterSyncAutomatically(true);
65 AndroidSyncSettings.overrideForTests(mContext, syncContentResolverDelega te); 83 AndroidSyncSettings.overrideForTests(mContext, syncContentResolverDelega te);
66 // This call initializes the ChromeSigninController to use our test cont ext. 84 // This call initializes the ChromeSigninController to use our test cont ext.
67 ChromeSigninController.get(mContext); 85 ChromeSigninController.get(mContext);
(...skipping 229 matching lines...) Expand 10 before | Expand all | Expand 10 after
297 315
298 SyncTestUtil.verifySyncIsSignedIn(mContext, defaultTestAccount); 316 SyncTestUtil.verifySyncIsSignedIn(mContext, defaultTestAccount);
299 assertTrue("Sync everything should be enabled", 317 assertTrue("Sync everything should be enabled",
300 SyncTestUtil.isSyncEverythingEnabled(mContext)); 318 SyncTestUtil.isSyncEverythingEnabled(mContext));
301 } 319 }
302 320
303 private static ContentViewCore getContentViewCore(ChromeShellActivity activi ty) { 321 private static ContentViewCore getContentViewCore(ChromeShellActivity activi ty) {
304 return activity.getActiveContentViewCore(); 322 return activity.getActiveContentViewCore();
305 } 323 }
306 } 324 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698