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

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: Remove some now-dead code. 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 273 matching lines...) Expand 10 before | Expand all | Expand 10 after
441 DevicePolicyResponseDelegateMock mock; 441 DevicePolicyResponseDelegateMock mock;
442 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0); 442 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0);
443 em::DevicePolicyRequest request; 443 em::DevicePolicyRequest request;
444 request.set_policy_scope(kChromePolicyScope); 444 request.set_policy_scope(kChromePolicyScope);
445 em::DevicePolicySettingRequest* setting_request = 445 em::DevicePolicySettingRequest* setting_request =
446 request.add_setting_request(); 446 request.add_setting_request();
447 setting_request->set_key(kChromeDevicePolicySettingKey); 447 setting_request->set_key(kChromeDevicePolicySettingKey);
448 setting_request->set_watermark("stale"); 448 setting_request->set_watermark("stale");
449 backend_->ProcessPolicyRequest(kDMToken, kDeviceId, 449 backend_->ProcessPolicyRequest(kDMToken, kDeviceId,
450 CloudPolicyDataStore::USER_AFFILIATION_NONE, 450 CloudPolicyDataStore::USER_AFFILIATION_NONE,
451 request, &mock); 451 request, NULL, &mock);
452 TestURLFetcher* fetcher = factory_.GetFetcherByID(0); 452 TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
453 ASSERT_TRUE(fetcher); 453 ASSERT_TRUE(fetcher);
454 454
455 // There shouldn't be any callbacks. 455 // There shouldn't be any callbacks.
456 backend_.reset(); 456 backend_.reset();
457 } 457 }
458 458
459 TEST_F(DeviceManagementServiceTest, JobQueueing) { 459 TEST_F(DeviceManagementServiceTest, JobQueueing) {
460 // Start with a non-initialized service. 460 // Start with a non-initialized service.
461 ResetService(); 461 ResetService();
(...skipping 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
494 DevicePolicyResponseDelegateMock mock; 494 DevicePolicyResponseDelegateMock mock;
495 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0); 495 EXPECT_CALL(mock, HandlePolicyResponse(_)).Times(0);
496 em::DevicePolicyRequest request; 496 em::DevicePolicyRequest request;
497 request.set_policy_scope(kChromePolicyScope); 497 request.set_policy_scope(kChromePolicyScope);
498 em::DevicePolicySettingRequest* setting_request = 498 em::DevicePolicySettingRequest* setting_request =
499 request.add_setting_request(); 499 request.add_setting_request();
500 setting_request->set_key(kChromeDevicePolicySettingKey); 500 setting_request->set_key(kChromeDevicePolicySettingKey);
501 setting_request->set_watermark("stale"); 501 setting_request->set_watermark("stale");
502 backend_->ProcessPolicyRequest(kDMToken, kDeviceId, 502 backend_->ProcessPolicyRequest(kDMToken, kDeviceId,
503 CloudPolicyDataStore::USER_AFFILIATION_NONE, 503 CloudPolicyDataStore::USER_AFFILIATION_NONE,
504 request, &mock); 504 request, NULL, &mock);
505 TestURLFetcher* fetcher = factory_.GetFetcherByID(0); 505 TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
506 ASSERT_TRUE(fetcher); 506 ASSERT_TRUE(fetcher);
507 507
508 // Shutdown the service and cancel the job afterwards. 508 // Shutdown the service and cancel the job afterwards.
509 service_->Shutdown(); 509 service_->Shutdown();
510 backend_.reset(); 510 backend_.reset();
511 } 511 }
512 512
513 TEST_F(DeviceManagementServiceTest, CancelDuringCallback) { 513 TEST_F(DeviceManagementServiceTest, CancelDuringCallback) {
514 // Make a request. 514 // Make a request.
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
595 595
596 // Verify that a new URLFetcher was started that bypasses the proxy. 596 // Verify that a new URLFetcher was started that bypasses the proxy.
597 fetcher = factory_.GetFetcherByID(0); 597 fetcher = factory_.GetFetcherByID(0);
598 ASSERT_TRUE(fetcher); 598 ASSERT_TRUE(fetcher);
599 EXPECT_TRUE((fetcher->GetLoadFlags() & net::LOAD_BYPASS_PROXY) != 0); 599 EXPECT_TRUE((fetcher->GetLoadFlags() & net::LOAD_BYPASS_PROXY) != 0);
600 EXPECT_EQ(original_url, fetcher->GetOriginalURL()); 600 EXPECT_EQ(original_url, fetcher->GetOriginalURL());
601 EXPECT_EQ(upload_data, fetcher->upload_data()); 601 EXPECT_EQ(upload_data, fetcher->upload_data());
602 } 602 }
603 603
604 } // namespace policy 604 } // namespace policy
OLDNEW
« no previous file with comments | « chrome/browser/policy/device_management_service_browsertest.cc ('k') | chrome/browser/policy/device_status_collector.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698