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

Unified Diff: chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc

Issue 33173005: Correctly deal with tokens for supervised users in ProfileOAuth2TokenService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 7 years, 1 month 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/policy/cloud/user_policy_signin_service_unittest.cc
diff --git a/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc b/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc
index 22a9e59236eaf901586a32d5643f00d60ac8fdd9..2e345ce912c6684d436ac156848cb5ff13e6be08 100644
--- a/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc
+++ b/chrome/browser/policy/cloud/user_policy_signin_service_unittest.cc
@@ -109,7 +109,8 @@ class UserPolicySigninServiceTest : public testing::Test {
// a valid login token, while on other platforms, the login refresh token
// is specified directly.
#if defined(OS_ANDROID)
- GetTokenService()->IssueRefreshToken("oauth2_login_refresh_token");
+ GetTokenService()->IssueRefreshTokenForUser(kTestUser,
+ "oauth2_login_refresh_token");
#endif
service->RegisterPolicyClient(
kTestUser,
@@ -359,7 +360,8 @@ TEST_F(UserPolicySigninServiceTest, InitWhileSignedIn) {
ASSERT_FALSE(IsRequestActive());
// Make oauth token available.
- GetTokenService()->IssueRefreshToken("oauth_login_refresh_token");
+ GetTokenService()->IssueRefreshTokenForUser(kTestUser,
+ "oauth_login_refresh_token");
// Client registration should be in progress since we now have an oauth token.
EXPECT_EQ(mock_store_->signin_username_, kTestUser);
@@ -387,7 +389,8 @@ TEST_F(UserPolicySigninServiceTest, InitWhileSignedInOAuthError) {
ASSERT_FALSE(IsRequestActive());
// Make oauth token available.
- GetTokenService()->IssueRefreshToken("oauth_login_refresh_token");
+ GetTokenService()->IssueRefreshTokenForUser(kTestUser,
+ "oauth_login_refresh_token");
// Client registration should be in progress since we now have an oauth token.
ASSERT_TRUE(IsRequestActive());
@@ -419,7 +422,8 @@ TEST_F(UserPolicySigninServiceTest, SignInAfterInit) {
mock_store_->NotifyStoreLoaded();
// Make oauth token available.
- GetTokenService()->IssueRefreshToken("oauth_login_refresh_token");
+ GetTokenService()->IssueRefreshTokenForUser(kTestUser,
+ "oauth_login_refresh_token");
// UserCloudPolicyManager should be initialized.
EXPECT_EQ(mock_store_->signin_username_, kTestUser);
@@ -449,7 +453,8 @@ TEST_F(UserPolicySigninServiceTest, SignInWithNonEnterpriseUser) {
mock_store_->NotifyStoreLoaded();
// Make oauth token available.
- GetTokenService()->IssueRefreshToken("oauth_login_refresh_token");
+ GetTokenService()->IssueRefreshTokenForUser(kTestUser,
+ "oauth_login_refresh_token");
// UserCloudPolicyManager should not be initialized and there should be no
// DMToken request active.
@@ -474,7 +479,8 @@ TEST_F(UserPolicySigninServiceTest, UnregisteredClient) {
kTestUser);
// Make oauth token available.
- GetTokenService()->IssueRefreshToken("oauth_login_refresh_token");
+ GetTokenService()->IssueRefreshTokenForUser(kTestUser,
+ "oauth_login_refresh_token");
// UserCloudPolicyManager should be initialized.
EXPECT_EQ(mock_store_->signin_username_, kTestUser);
@@ -508,7 +514,8 @@ TEST_F(UserPolicySigninServiceTest, RegisteredClient) {
kTestUser);
// Make oauth token available.
- GetTokenService()->IssueRefreshToken("oauth_login_refresh_token");
+ GetTokenService()->IssueRefreshTokenForUser(kTestUser,
+ "oauth_login_refresh_token");
// UserCloudPolicyManager should be initialized.
EXPECT_EQ(mock_store_->signin_username_, kTestUser);
« no previous file with comments | « chrome/browser/policy/cloud/user_policy_signin_service.cc ('k') | chrome/browser/signin/profile_oauth2_token_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698