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

Unified Diff: components/signin/core/browser/account_tracker_service_unittest.cc

Issue 1130853003: Add service flags to the AccountTrackerService. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review feedback. 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
Index: components/signin/core/browser/account_tracker_service_unittest.cc
diff --git a/components/signin/core/browser/account_tracker_service_unittest.cc b/components/signin/core/browser/account_tracker_service_unittest.cc
index 3ea1ab4119ed87dffbe38cefbd34f07c1370ea6c..fe4a929915c54d1cd705397c2235a113809c90b4 100644
--- a/components/signin/core/browser/account_tracker_service_unittest.cc
+++ b/components/signin/core/browser/account_tracker_service_unittest.cc
@@ -39,6 +39,10 @@ const std::string kTokenInfoIncompleteResponseFormat =
\"hd\": \"\", \
}";
+const std::string kLSIDResponse = "{ lsid: \"Foo\" }";
+
+const std::string kServiceFlags = "allServices=Service1,Service2";
+
enum TrackingEventType {
UPDATED,
REMOVED,
@@ -74,6 +78,9 @@ void CheckAccountDetails(const std::string account_id,
EXPECT_EQ(AccountIdToFullName(account_id), info.full_name);
EXPECT_EQ(AccountIdToGivenName(account_id), info.given_name);
EXPECT_EQ(AccountIdToLocale(account_id), info.locale);
+ EXPECT_EQ(2U, info.service_flags.size());
+ EXPECT_EQ("Service1", info.service_flags[0]);
+ EXPECT_EQ("Service2", info.service_flags[1]);
}
class TrackingEvent {
@@ -340,6 +347,8 @@ void AccountTrackerServiceTest::ReturnOAuthUrlFetchSuccess(
ReturnOAuthUrlFetchResults(gaia::GaiaOAuthClient::kUrlFetcherId,
net::HTTP_OK,
GenerateValidTokenInfoResponse(account_id));
+ ReturnOAuthUrlFetchResults(0, net::HTTP_OK, kLSIDResponse);
+ ReturnOAuthUrlFetchResults(0, net::HTTP_OK, kServiceFlags);
}
void AccountTrackerServiceTest::ReturnOAuthUrlFetchSuccessIncomplete(

Powered by Google App Engine
This is Rietveld 408576698