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

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

Issue 98173009: GCM Checkin implementation with unit tests and protobufs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@gcm-store
Patch Set: Updating the copyright notice Created 6 years, 11 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: google_apis/gcm/engine/checkin_request_unittest.cc
diff --git a/google_apis/gcm/engine/checkin_request_unittest.cc b/google_apis/gcm/engine/checkin_request_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..bad412cb454ee06c8eaa4846c7cfa099a0784862
--- /dev/null
+++ b/google_apis/gcm/engine/checkin_request_unittest.cc
@@ -0,0 +1,260 @@
+// Copyright (c) 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include <string>
+
+#include "google_apis/gcm/engine/checkin_request.h"
+#include "google_apis/gcm/protocol/checkin.pb.h"
+#include "net/url_request/test_url_fetcher_factory.h"
+#include "net/url_request/url_request_test_util.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace gcm {
+
+const uint64 kAndroidId = 42UL;
+const uint64 kBlankAndroidId = 999999UL;
+const uint64 kBlankSecurityToken = 999999UL;
+const char kChromeVersion[] = "Version String";
+const uint64 kSecurityToken = 77;
+const int64 kUserSerialNumber = 7;
+
+class CheckinRequestTest : public testing::Test {
+ public:
+ enum ResponseScenario {
+ VALID_RESPONSE, // Both android_id and security_token set in response.
+ MISSING_ANDROID_ID, // android_id is missing.
+ MISSING_SECURITY_TOKEN, // security_token is missing.
+ ANDROID_ID_IS_ZER0, // android_id is 0.
+ SECURITY_TOKEN_IS_ZERO // security_token is 0.
+ };
+
+ CheckinRequestTest();
+ virtual ~CheckinRequestTest();
+
+ void FetcherCallback(uint64 android_id, uint64 security_token);
+
+ void CreateRequest(uint64 android_id, uint64 security_token);
+
+ void SetResponseStatusAndString(
+ net::HttpStatusCode status_code,
+ const std::string& response_data);
+
+ void CompleteFetch();
+
+ void SetResponse(ResponseScenario response_scenario);
+
+ protected:
+ uint64 android_id_;
+ uint64 security_token_;
+ int checkin_device_type_;
+ base::MessageLoop message_loop_;
+ net::TestURLFetcherFactory url_fetcher_factory_;
+ scoped_refptr<net::TestURLRequestContextGetter> url_request_context_getter_;
+ checkin_proto::ChromeBuildProto chrome_build_proto_;
+ scoped_ptr<CheckinRequest> request_;
+};
+
+CheckinRequestTest::CheckinRequestTest()
+ : android_id_(kBlankAndroidId),
+ security_token_(kBlankSecurityToken),
+ checkin_device_type_(0),
+ url_request_context_getter_(new net::TestURLRequestContextGetter(
+ message_loop_.message_loop_proxy())) {}
+
+CheckinRequestTest::~CheckinRequestTest() {}
+
+void CheckinRequestTest::FetcherCallback(uint64 android_id,
+ uint64 security_token) {
+ android_id_ = android_id;
+ security_token_ = security_token;
+}
+
+void CheckinRequestTest::CreateRequest(uint64 android_id,
+ uint64 security_token) {
+ // First setup a chrome_build protobuf.
+ chrome_build_proto_.set_platform(
+ checkin_proto::ChromeBuildProto::PLATFORM_LINUX);
+ chrome_build_proto_.set_channel(
+ checkin_proto::ChromeBuildProto::CHANNEL_CANARY);
+ chrome_build_proto_.set_chrome_version(kChromeVersion);
+ // Then create a request with that protobuf and specified android_id,
+ // security_token.
+ request_.reset(new CheckinRequest(
+ base::Bind(&CheckinRequestTest::FetcherCallback,
+ base::Unretained(this)),
+ chrome_build_proto_,
+ kUserSerialNumber,
+ android_id,
+ security_token,
+ url_request_context_getter_.get()));
+
+ // Setting android_id_ and security_token_ to blank value, not used elsewhere
+ // in the tests.
+ android_id_ = kBlankAndroidId;
+ security_token_ = kBlankSecurityToken;
+}
+
+void CheckinRequestTest::SetResponseStatusAndString(
+ net::HttpStatusCode status_code,
+ const std::string& response_data) {
+ net::TestURLFetcher* fetcher = url_fetcher_factory_.GetFetcherByID(0);
+ ASSERT_TRUE(fetcher);
+ fetcher->set_response_code(status_code);
+ fetcher->SetResponseString(response_data);
+}
+
+void CheckinRequestTest::CompleteFetch() {
+ net::TestURLFetcher* fetcher = url_fetcher_factory_.GetFetcherByID(0);
+ ASSERT_TRUE(fetcher);
+ fetcher->delegate()->OnURLFetchComplete(fetcher);
+}
+
+void CheckinRequestTest::SetResponse(ResponseScenario response_scenario) {
+ checkin_proto::AndroidCheckinResponse response;
+ response.set_stats_ok(true);
+
+ uint64 android_id = response_scenario == ANDROID_ID_IS_ZER0 ? 0 : kAndroidId;
+ uint64 security_token =
+ response_scenario == SECURITY_TOKEN_IS_ZERO ? 0 : kSecurityToken;
+
+ if (response_scenario != MISSING_ANDROID_ID)
+ response.set_android_id(android_id);
+
+ if (response_scenario != MISSING_SECURITY_TOKEN)
+ response.set_security_token(security_token);
+
+ std::string response_string;
+ response.SerializeToString(&response_string);
+ SetResponseStatusAndString(net::HTTP_OK, response_string);
+}
+
+TEST_F(CheckinRequestTest, FetcherData) {
+ CreateRequest(kAndroidId, kSecurityToken);
+ request_->Start();
+
+ // Get data sent by request.
+ net::TestURLFetcher* fetcher = url_fetcher_factory_.GetFetcherByID(0);
+ ASSERT_TRUE(fetcher);
+ fetcher->set_response_code(net::HTTP_OK);
+ checkin_proto::AndroidCheckinRequest request_proto;
+ request_proto.ParseFromString(fetcher->upload_data());
+
+ EXPECT_EQ(kAndroidId, static_cast<uint64>(request_proto.id()));
+ EXPECT_EQ(kSecurityToken, request_proto.security_token());
+ EXPECT_EQ(kUserSerialNumber, request_proto.user_serial_number());
+ EXPECT_EQ(chrome_build_proto_.platform(),
+ request_proto.checkin().chrome_build().platform());
+ EXPECT_EQ(chrome_build_proto_.chrome_version(),
+ request_proto.checkin().chrome_build().chrome_version());
+ EXPECT_EQ(chrome_build_proto_.channel(),
+ request_proto.checkin().chrome_build().channel());
+
+#if defined(CHROME_OS)
+ EXPECT_EQ(checkin_proto::DEVICE_CHROME_OS, request_proto.checkin().type());
+#else
+ EXPECT_EQ(checkin_proto::DEVICE_CHROME_BROWSER,
+ request_proto.checkin().type());
+#endif
+}
+
+TEST_F(CheckinRequestTest, ResponseBodyEmpty) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponseStatusAndString(net::HTTP_OK, std::string());
+ CompleteFetch();
+
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
+}
+
+TEST_F(CheckinRequestTest, ResponseBodyCorrupted) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponseStatusAndString(net::HTTP_OK, "Corrupted response body");
+ CompleteFetch();
+
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
+}
+
+TEST_F(CheckinRequestTest, ResponseHttpStatusNotOK) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponseStatusAndString(net::HTTP_UNAUTHORIZED, std::string());
+ CompleteFetch();
+
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
+}
+
+TEST_F(CheckinRequestTest, ResponseMissingAndroidId) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponse(MISSING_ANDROID_ID);
+ CompleteFetch();
+
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
+}
+
+TEST_F(CheckinRequestTest, ResponseMissingSecurityToken) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponse(MISSING_SECURITY_TOKEN);
+ CompleteFetch();
+
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
+}
+
+TEST_F(CheckinRequestTest, AndroidIdEqualsZeroInResponse) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponse(ANDROID_ID_IS_ZER0);
+ CompleteFetch();
+
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
+}
+
+TEST_F(CheckinRequestTest, SecurityTokenEqualsZeroInResponse) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponse(SECURITY_TOKEN_IS_ZERO);
+ CompleteFetch();
+
+ EXPECT_EQ(0u, android_id_);
+ EXPECT_EQ(0u, security_token_);
+}
+
+TEST_F(CheckinRequestTest, SuccessfulFirstTimeCheckin) {
+ CreateRequest(0u, 0u);
+ request_->Start();
+
+ SetResponse(VALID_RESPONSE);
+ CompleteFetch();
+
+ EXPECT_EQ(kAndroidId, android_id_);
+ EXPECT_EQ(kSecurityToken, security_token_);
+}
+
+TEST_F(CheckinRequestTest, SuccessfulSubsequentCheckin) {
+ CreateRequest(kAndroidId, kSecurityToken);
+ request_->Start();
+
+ SetResponse(VALID_RESPONSE);
+ CompleteFetch();
+
+ EXPECT_EQ(kAndroidId, android_id_);
+ EXPECT_EQ(kSecurityToken, security_token_);
+}
+
+} // namespace gcm

Powered by Google App Engine
This is Rietveld 408576698