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

Side by Side Diff: chrome/browser/policy/device_management_service_unittest.cc

Issue 8702009: Add device status reports to policy requests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix one issue I missed. Created 9 years 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <ostream> 5 #include <ostream>
6 #include <vector> 6 #include <vector>
7 7
8 #include "base/message_loop.h" 8 #include "base/message_loop.h"
9 #include "base/string_split.h" 9 #include "base/string_split.h"
10 #include "chrome/browser/policy/device_management_backend_impl.h" 10 #include "chrome/browser/policy/device_management_backend_impl.h"
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
147 TEST_P(DeviceManagementServiceFailedRequestTest, PolicyRequest) { 147 TEST_P(DeviceManagementServiceFailedRequestTest, PolicyRequest) {
148 DevicePolicyResponseDelegateMock mock; 148 DevicePolicyResponseDelegateMock mock;
149 EXPECT_CALL(mock, OnError(GetParam().expected_error_)); 149 EXPECT_CALL(mock, OnError(GetParam().expected_error_));
150 em::DevicePolicyRequest request; 150 em::DevicePolicyRequest request;
151 request.set_policy_scope(kChromePolicyScope); 151 request.set_policy_scope(kChromePolicyScope);
152 em::DevicePolicySettingRequest* setting_request = 152 em::DevicePolicySettingRequest* setting_request =
153 request.add_setting_request(); 153 request.add_setting_request();
154 setting_request->set_key(kChromeDevicePolicySettingKey); 154 setting_request->set_key(kChromeDevicePolicySettingKey);
155 backend_->ProcessPolicyRequest(kDMToken, kDeviceId, 155 backend_->ProcessPolicyRequest(kDMToken, kDeviceId,
156 CloudPolicyDataStore::USER_AFFILIATION_NONE, 156 CloudPolicyDataStore::USER_AFFILIATION_NONE,
157 request, &mock); 157 request, NULL, &mock);
158 TestURLFetcher* fetcher = factory_.GetFetcherByID(0); 158 TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
159 ASSERT_TRUE(fetcher); 159 ASSERT_TRUE(fetcher);
160 160
161 fetcher->set_url(GURL(kServiceUrl)); 161 fetcher->set_url(GURL(kServiceUrl));
162 fetcher->set_status(GetParam().request_status_); 162 fetcher->set_status(GetParam().request_status_);
163 fetcher->set_response_code(GetParam().http_status_); 163 fetcher->set_response_code(GetParam().http_status_);
164 fetcher->SetResponseString(GetParam().response_); 164 fetcher->SetResponseString(GetParam().response_);
165 fetcher->delegate()->OnURLFetchComplete(fetcher); 165 fetcher->delegate()->OnURLFetchComplete(fetcher);
166 } 166 }
167 167
(...skipping 256 matching lines...) Expand 10 before | Expand all | Expand 10 after
424 DevicePolicyResponseDelegateMock mock; 424 DevicePolicyResponseDelegateMock mock;
425 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0); 425 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0);
426 em::DevicePolicyRequest request; 426 em::DevicePolicyRequest request;
427 request.set_policy_scope(kChromePolicyScope); 427 request.set_policy_scope(kChromePolicyScope);
428 em::DevicePolicySettingRequest* setting_request = 428 em::DevicePolicySettingRequest* setting_request =
429 request.add_setting_request(); 429 request.add_setting_request();
430 setting_request->set_key(kChromeDevicePolicySettingKey); 430 setting_request->set_key(kChromeDevicePolicySettingKey);
431 setting_request->set_watermark("stale"); 431 setting_request->set_watermark("stale");
432 backend_->ProcessPolicyRequest(kDMToken, kDeviceId, 432 backend_->ProcessPolicyRequest(kDMToken, kDeviceId,
433 CloudPolicyDataStore::USER_AFFILIATION_NONE, 433 CloudPolicyDataStore::USER_AFFILIATION_NONE,
434 request, &mock); 434 request, NULL, &mock);
435 TestURLFetcher* fetcher = factory_.GetFetcherByID(0); 435 TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
436 ASSERT_TRUE(fetcher); 436 ASSERT_TRUE(fetcher);
437 437
438 // There shouldn't be any callbacks. 438 // There shouldn't be any callbacks.
439 backend_.reset(); 439 backend_.reset();
440 } 440 }
441 441
442 TEST_F(DeviceManagementServiceTest, JobQueueing) { 442 TEST_F(DeviceManagementServiceTest, JobQueueing) {
443 // Start with a non-initialized service. 443 // Start with a non-initialized service.
444 ResetService(); 444 ResetService();
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
477 DevicePolicyResponseDelegateMock mock; 477 DevicePolicyResponseDelegateMock mock;
478 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0); 478 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0);
479 em::DevicePolicyRequest request; 479 em::DevicePolicyRequest request;
480 request.set_policy_scope(kChromePolicyScope); 480 request.set_policy_scope(kChromePolicyScope);
481 em::DevicePolicySettingRequest* setting_request = 481 em::DevicePolicySettingRequest* setting_request =
482 request.add_setting_request(); 482 request.add_setting_request();
483 setting_request->set_key(kChromeDevicePolicySettingKey); 483 setting_request->set_key(kChromeDevicePolicySettingKey);
484 setting_request->set_watermark("stale"); 484 setting_request->set_watermark("stale");
485 backend_->ProcessPolicyRequest(kDMToken, kDeviceId, 485 backend_->ProcessPolicyRequest(kDMToken, kDeviceId,
486 CloudPolicyDataStore::USER_AFFILIATION_NONE, 486 CloudPolicyDataStore::USER_AFFILIATION_NONE,
487 request, &mock); 487 request, NULL, &mock);
488 TestURLFetcher* fetcher = factory_.GetFetcherByID(0); 488 TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
489 ASSERT_TRUE(fetcher); 489 ASSERT_TRUE(fetcher);
490 490
491 // Shutdown the service and cancel the job afterwards. 491 // Shutdown the service and cancel the job afterwards.
492 service_->Shutdown(); 492 service_->Shutdown();
493 backend_.reset(); 493 backend_.reset();
494 } 494 }
495 495
496 TEST_F(DeviceManagementServiceTest, CancelDuringCallback) { 496 TEST_F(DeviceManagementServiceTest, CancelDuringCallback) {
497 // Make a request. 497 // Make a request.
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
578 578
579 // Verify that a new URLFetcher was started that bypasses the proxy. 579 // Verify that a new URLFetcher was started that bypasses the proxy.
580 fetcher = factory_.GetFetcherByID(0); 580 fetcher = factory_.GetFetcherByID(0);
581 ASSERT_TRUE(fetcher); 581 ASSERT_TRUE(fetcher);
582 EXPECT_TRUE((fetcher->GetLoadFlags() & net::LOAD_BYPASS_PROXY) != 0); 582 EXPECT_TRUE((fetcher->GetLoadFlags() & net::LOAD_BYPASS_PROXY) != 0);
583 EXPECT_EQ(original_url, fetcher->GetOriginalURL()); 583 EXPECT_EQ(original_url, fetcher->GetOriginalURL());
584 EXPECT_EQ(upload_data, fetcher->upload_data()); 584 EXPECT_EQ(upload_data, fetcher->upload_data());
585 } 585 }
586 586
587 } // namespace policy 587 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698