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

Unified Diff: chrome/browser/sync/profile_sync_service_unittest.cc

Issue 23382008: Making OAuth2TokenService multi-login aware, updating callers, minor fixes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebasing to include the update to ProfileSyncService: r224220 Created 7 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/browser/sync/profile_sync_service_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc
index 69f5d266e6ffd36b6c8b69d23831fc2e85c83812..1899863c4778b9dead2f74e48ce56955a38a60d8 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -101,7 +101,7 @@ class ProfileSyncServiceTest : public testing::Test {
SigninManagerBase* signin =
SigninManagerFactory::GetForProfile(profile_.get());
signin->SetAuthenticatedUsername("test");
- OAuth2TokenService* oauth2_token_service =
+ ProfileOAuth2TokenService* oauth2_token_service =
ProfileOAuth2TokenServiceFactory::GetForProfile(profile_.get());
ProfileSyncComponentsFactoryMock* factory =
new ProfileSyncComponentsFactoryMock();
@@ -147,12 +147,10 @@ class ProfileSyncServiceTest : public testing::Test {
}
void IssueTestTokens() {
- TokenService* token_service =
- TokenServiceFactory::GetForProfile(profile_.get());
- token_service->IssueAuthTokenForTest(
- GaiaConstants::kGaiaOAuth2LoginRefreshToken, "oauth2_login_token");
- token_service->IssueAuthTokenForTest(
- GaiaConstants::kSyncService, "token");
+ ProfileOAuth2TokenServiceFactory::GetForProfile(profile_.get())
+ ->UpdateCredentials("test", "oauth2_login_token");
+ TokenServiceFactory::GetForProfile(profile_.get())
+ ->IssueAuthTokenForTest(GaiaConstants::kSyncService, "token");
}
scoped_ptr<TestProfileSyncService> service_;
@@ -178,7 +176,7 @@ class TestProfileSyncServiceObserver : public ProfileSyncServiceObserver {
TEST_F(ProfileSyncServiceTest, InitialState) {
SigninManagerBase* signin =
SigninManagerFactory::GetForProfile(profile_.get());
- OAuth2TokenService* oauth2_token_service =
+ ProfileOAuth2TokenService* oauth2_token_service =
ProfileOAuth2TokenServiceFactory::GetForProfile(profile_.get());
service_.reset(new TestProfileSyncService(
new ProfileSyncComponentsFactoryMock(),
@@ -215,7 +213,7 @@ TEST_F(ProfileSyncServiceTest, DisabledByPolicy) {
Value::CreateBooleanValue(true));
SigninManagerBase* signin =
SigninManagerFactory::GetForProfile(profile_.get());
- OAuth2TokenService* oauth2_token_service =
+ ProfileOAuth2TokenService* oauth2_token_service =
ProfileOAuth2TokenServiceFactory::GetForProfile(profile_.get());
service_.reset(new TestProfileSyncService(
new ProfileSyncComponentsFactoryMock(),
@@ -232,7 +230,7 @@ TEST_F(ProfileSyncServiceTest, AbortedByShutdown) {
SigninManagerBase* signin =
SigninManagerFactory::GetForProfile(profile_.get());
signin->SetAuthenticatedUsername("test");
- OAuth2TokenService* oauth2_token_service =
+ ProfileOAuth2TokenService* oauth2_token_service =
ProfileOAuth2TokenServiceFactory::GetForProfile(profile_.get());
ProfileSyncComponentsFactoryMock* factory =
new ProfileSyncComponentsFactoryMock();
@@ -260,7 +258,7 @@ TEST_F(ProfileSyncServiceTest, DisableAndEnableSyncTemporarily) {
SigninManagerBase* signin =
SigninManagerFactory::GetForProfile(profile_.get());
signin->SetAuthenticatedUsername("test");
- OAuth2TokenService* oauth2_token_service =
+ ProfileOAuth2TokenService* oauth2_token_service =
ProfileOAuth2TokenServiceFactory::GetForProfile(profile_.get());
ProfileSyncComponentsFactoryMock* factory =
new ProfileSyncComponentsFactoryMock();
@@ -301,8 +299,8 @@ TEST_F(ProfileSyncServiceTest, DisableAndEnableSyncTemporarily) {
TEST_F(ProfileSyncServiceTest, EnableSyncAndSignOut) {
SigninManager* signin =
SigninManagerFactory::GetForProfile(profile_.get());
- signin->SetAuthenticatedUsername("test@test.com");
- OAuth2TokenService* oauth2_token_service =
+ signin->SetAuthenticatedUsername("test");
+ ProfileOAuth2TokenService* oauth2_token_service =
ProfileOAuth2TokenServiceFactory::GetForProfile(profile_.get());
ProfileSyncComponentsFactoryMock* factory =
new ProfileSyncComponentsFactoryMock();
« no previous file with comments | « chrome/browser/sync/profile_sync_service_typed_url_unittest.cc ('k') | chrome/browser/sync/test_profile_sync_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698