Commit f39ae5fc authored by David Roger's avatar David Roger Committed by Commit Bot

[signin] Remove TODOs in MutableProfileOAuth2TokenServiceDelegateTest

Change-Id: I4355145d640484cf056bb3fabb038fc99d682b71
Reviewed-on: https://chromium-review.googlesource.com/c/1335568Reviewed-by: default avatarMihai Sardarescu <msarda@chromium.org>
Commit-Queue: David Roger <droger@chromium.org>
Cr-Commit-Position: refs/heads/master@{#607960}
parent 21369b35
...@@ -360,16 +360,14 @@ TEST_F(MutableProfileOAuth2TokenServiceDelegateTest, ...@@ -360,16 +360,14 @@ TEST_F(MutableProfileOAuth2TokenServiceDelegateTest,
std::string account_id_2 = "account_id_2"; std::string account_id_2 = "account_id_2";
std::string refresh_token_2 = "refresh_token_2"; std::string refresh_token_2 = "refresh_token_2";
// TODO(fgorski): Enable below when implemented: EXPECT_FALSE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_1));
// EXPECT_FALSE(oauth2_servive_->RefreshTokenIsAvailable(account_id_1)); EXPECT_FALSE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_2));
// EXPECT_FALSE(oauth2_servive_->RefreshTokenIsAvailable(account_id_2));
oauth2_service_delegate_->UpdateCredentials(account_id_1, refresh_token_1); oauth2_service_delegate_->UpdateCredentials(account_id_1, refresh_token_1);
oauth2_service_delegate_->UpdateCredentials(account_id_2, refresh_token_2); oauth2_service_delegate_->UpdateCredentials(account_id_2, refresh_token_2);
EXPECT_EQ(2, start_batch_changes_); EXPECT_EQ(2, start_batch_changes_);
EXPECT_EQ(2, end_batch_changes_); EXPECT_EQ(2, end_batch_changes_);
// TODO(fgorski): Enable below when implemented: EXPECT_TRUE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_1));
// EXPECT_TRUE(oauth2_servive_->RefreshTokenIsAvailable(account_id_1));
EXPECT_TRUE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_2)); EXPECT_TRUE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_2));
ResetObserverCounts(); ResetObserverCounts();
...@@ -378,8 +376,7 @@ TEST_F(MutableProfileOAuth2TokenServiceDelegateTest, ...@@ -378,8 +376,7 @@ TEST_F(MutableProfileOAuth2TokenServiceDelegateTest,
EXPECT_EQ(1, end_batch_changes_); EXPECT_EQ(1, end_batch_changes_);
ExpectOneTokenRevokedNotification(); ExpectOneTokenRevokedNotification();
// TODO(fgorski): Enable below when implemented: EXPECT_FALSE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_1));
// EXPECT_FALSE(oauth2_servive_->RefreshTokenIsAvailable(account_id_1));
EXPECT_TRUE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_2)); EXPECT_TRUE(oauth2_service_delegate_->RefreshTokenIsAvailable(account_id_2));
oauth2_service_delegate_->RevokeAllCredentials(); oauth2_service_delegate_->RevokeAllCredentials();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment