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

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

Issue 14843009: Wire the device requisition parameter for enterprise enrollment. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix mock Created 7 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: chrome/browser/policy/cloud/cloud_policy_client_unittest.cc
diff --git a/chrome/browser/policy/cloud/cloud_policy_client_unittest.cc b/chrome/browser/policy/cloud/cloud_policy_client_unittest.cc
index c3ffdba0acbd105916859bdbf2c93acd7f155e54..527a8c6cfa05b2c66b35a5e8e0ecf48a4106554b 100644
--- a/chrome/browser/policy/cloud/cloud_policy_client_unittest.cc
+++ b/chrome/browser/policy/cloud/cloud_policy_client_unittest.cc
@@ -34,6 +34,7 @@ const char kMachineModel[] = "fake-machine-model";
const char kOAuthToken[] = "fake-oauth-token";
const char kDMToken[] = "fake-dm-token";
const char kDeviceCertificate[] = "fake-device-certificate";
+const char kRequisition[] = "fake-requisition";
class MockStatusProvider : public CloudPolicyClient::StatusProvider {
public:
@@ -220,7 +221,7 @@ TEST_F(CloudPolicyClientTest, RegistrationAndPolicyFetch) {
ExpectRegistration(kOAuthToken);
EXPECT_CALL(observer_, OnRegistrationStateChanged(_));
client_->Register(em::DeviceRegisterRequest::USER,
- kOAuthToken, std::string(), false);
+ kOAuthToken, std::string(), false, std::string());
EXPECT_TRUE(client_->is_registered());
EXPECT_FALSE(client_->GetPolicyFor(policy_ns_key_));
EXPECT_EQ(DM_STATUS_SUCCESS, client_->status());
@@ -236,10 +237,12 @@ TEST_F(CloudPolicyClientTest, RegistrationAndPolicyFetch) {
TEST_F(CloudPolicyClientTest, RegistrationParameters) {
registration_request_.mutable_register_request()->set_reregister(true);
registration_request_.mutable_register_request()->set_auto_enrolled(true);
+ registration_request_.mutable_register_request()->set_requisition(
+ kRequisition);
ExpectRegistration(kOAuthToken);
EXPECT_CALL(observer_, OnRegistrationStateChanged(_));
client_->Register(em::DeviceRegisterRequest::USER,
- kOAuthToken, kClientID, true);
+ kOAuthToken, kClientID, true, kRequisition);
EXPECT_EQ(kClientID, client_id_);
}
@@ -249,7 +252,7 @@ TEST_F(CloudPolicyClientTest, RegistrationNoToken) {
ExpectRegistration(kOAuthToken);
EXPECT_CALL(observer_, OnClientError(_));
client_->Register(em::DeviceRegisterRequest::USER,
- kOAuthToken, std::string(), false);
+ kOAuthToken, std::string(), false, std::string());
EXPECT_FALSE(client_->is_registered());
EXPECT_FALSE(client_->GetPolicyFor(policy_ns_key_));
EXPECT_EQ(DM_STATUS_RESPONSE_DECODING_ERROR, client_->status());
@@ -262,7 +265,7 @@ TEST_F(CloudPolicyClientTest, RegistrationFailure) {
EXPECT_CALL(service_, StartJob(_, _, _, _, _, _, _));
EXPECT_CALL(observer_, OnClientError(_));
client_->Register(em::DeviceRegisterRequest::USER,
- kOAuthToken, std::string(), false);
+ kOAuthToken, std::string(), false, std::string());
EXPECT_FALSE(client_->is_registered());
EXPECT_FALSE(client_->GetPolicyFor(policy_ns_key_));
EXPECT_EQ(DM_STATUS_REQUEST_FAILED, client_->status());
@@ -280,7 +283,7 @@ TEST_F(CloudPolicyClientTest, RetryRegistration) {
"", kOAuthToken, "", "", _,
MatchProto(registration_request_)));
client_->Register(em::DeviceRegisterRequest::USER,
- kOAuthToken, std::string(), false);
+ kOAuthToken, std::string(), false, std::string());
EXPECT_FALSE(client_->is_registered());
Mock::VerifyAndClearExpectations(&service_);
« no previous file with comments | « chrome/browser/policy/cloud/cloud_policy_client.cc ('k') | chrome/browser/policy/cloud/cloud_policy_manager_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698