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

Unified Diff: trunk/src/google_apis/gcm/engine/checkin_request_unittest.cc

Issue 227413010: Revert 262242 "Adding basic G-services handling" (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « trunk/src/google_apis/gcm/engine/checkin_request.cc ('k') | trunk/src/google_apis/gcm/engine/gcm_store.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/src/google_apis/gcm/engine/checkin_request_unittest.cc
===================================================================
--- trunk/src/google_apis/gcm/engine/checkin_request_unittest.cc (revision 262265)
+++ trunk/src/google_apis/gcm/engine/checkin_request_unittest.cc (working copy)
@@ -51,7 +51,6 @@
const uint64 kBlankSecurityToken = 999999UL;
const char kChromeVersion[] = "Version String";
const uint64 kSecurityToken = 77;
-const char kSettingsDigest[] = "settings_digest";
class CheckinRequestTest : public testing::Test {
public:
@@ -66,8 +65,7 @@
CheckinRequestTest();
virtual ~CheckinRequestTest();
- void FetcherCallback(
- const checkin_proto::AndroidCheckinResponse& response);
+ void FetcherCallback(uint64 android_id, uint64 security_token);
void CreateRequest(uint64 android_id, uint64 security_token);
@@ -104,13 +102,11 @@
CheckinRequestTest::~CheckinRequestTest() {}
-void CheckinRequestTest::FetcherCallback(
- const checkin_proto::AndroidCheckinResponse& checkin_response) {
+void CheckinRequestTest::FetcherCallback(uint64 android_id,
+ uint64 security_token) {
callback_called_ = true;
- if (checkin_response.has_android_id())
- android_id_ = checkin_response.android_id();
- if (checkin_response.has_security_token())
- security_token_ = checkin_response.security_token();
+ android_id_ = android_id;
+ security_token_ = security_token;
}
void CheckinRequestTest::CreateRequest(uint64 android_id,
@@ -121,19 +117,15 @@
chrome_build_proto_.set_channel(
checkin_proto::ChromeBuildProto::CHANNEL_CANARY);
chrome_build_proto_.set_chrome_version(kChromeVersion);
-
- CheckinRequest::RequestInfo request_info(
- android_id,
- security_token,
- kSettingsDigest,
- account_ids_,
- chrome_build_proto_);
// Then create a request with that protobuf and specified android_id,
// security_token.
request_.reset(new CheckinRequest(
- request_info,
- kDefaultBackoffPolicy,
base::Bind(&CheckinRequestTest::FetcherCallback, base::Unretained(this)),
+ kDefaultBackoffPolicy,
+ chrome_build_proto_,
+ android_id,
+ security_token,
+ account_ids_,
url_request_context_getter_.get()));
// Setting android_id_ and security_token_ to blank value, not used elsewhere
@@ -206,7 +198,6 @@
request_proto.checkin().type());
#endif
- EXPECT_EQ(kSettingsDigest, request_proto.digest());
EXPECT_EQ(1, request_proto.account_cookie_size());
EXPECT_EQ("[account_id]", request_proto.account_cookie(0));
}
@@ -253,8 +244,8 @@
CompleteFetch();
EXPECT_TRUE(callback_called_);
- EXPECT_EQ(kBlankAndroidId, android_id_);
- EXPECT_EQ(kBlankSecurityToken, security_token_);
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
}
TEST_F(CheckinRequestTest, ResponseHttpStatusBadRequest) {
@@ -265,8 +256,8 @@
CompleteFetch();
EXPECT_TRUE(callback_called_);
- EXPECT_EQ(kBlankAndroidId, android_id_);
- EXPECT_EQ(kBlankSecurityToken, security_token_);
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
}
TEST_F(CheckinRequestTest, ResponseHttpStatusNotOK) {
« no previous file with comments | « trunk/src/google_apis/gcm/engine/checkin_request.cc ('k') | trunk/src/google_apis/gcm/engine/gcm_store.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698