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

Side by Side Diff: chrome/browser/chromeos/policy/device_status_collector_browsertest.cc

Issue 2403053003: Remove unused email from install attributes and add support for AD. (Closed)
Patch Set: Address Maksim's comments. Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 "chrome/browser/chromeos/policy/device_status_collector.h" 5 #include "chrome/browser/chromeos/policy/device_status_collector.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 233 matching lines...) Expand 10 before | Expand all | Expand 10 after
244 244
245 // Though it is a unit test, this test is linked with browser_tests so that it 245 // Though it is a unit test, this test is linked with browser_tests so that it
246 // runs in a separate process. The intention is to avoid overriding the timezone 246 // runs in a separate process. The intention is to avoid overriding the timezone
247 // environment variable for other tests. 247 // environment variable for other tests.
248 class DeviceStatusCollectorTest : public testing::Test { 248 class DeviceStatusCollectorTest : public testing::Test {
249 public: 249 public:
250 DeviceStatusCollectorTest() 250 DeviceStatusCollectorTest()
251 : ui_thread_(content::BrowserThread::UI, &message_loop_), 251 : ui_thread_(content::BrowserThread::UI, &message_loop_),
252 file_thread_(content::BrowserThread::FILE, &message_loop_), 252 file_thread_(content::BrowserThread::FILE, &message_loop_),
253 io_thread_(content::BrowserThread::IO, &message_loop_), 253 io_thread_(content::BrowserThread::IO, &message_loop_),
254 install_attributes_("managed.com", 254 install_attributes_(
255 "user@managed.com", 255 chromeos::ScopedStubInstallAttributes::CreateEnterprise(
256 "device_id", 256 "managed.com",
257 DEVICE_MODE_ENTERPRISE), 257 "device_id")),
258 settings_helper_(false), 258 settings_helper_(false),
259 user_manager_(new chromeos::MockUserManager()), 259 user_manager_(new chromeos::MockUserManager()),
260 user_manager_enabler_(user_manager_), 260 user_manager_enabler_(user_manager_),
261 got_session_status_(false), 261 got_session_status_(false),
262 fake_device_local_account_(policy::DeviceLocalAccount::TYPE_KIOSK_APP, 262 fake_device_local_account_(policy::DeviceLocalAccount::TYPE_KIOSK_APP,
263 kKioskAccountId, 263 kKioskAccountId,
264 kKioskAppId, 264 kKioskAppId,
265 std::string() /* kiosk_app_update_url */), 265 std::string() /* kiosk_app_update_url */),
266 user_data_dir_override_(chrome::DIR_USER_DATA), 266 user_data_dir_override_(chrome::DIR_USER_DATA),
267 update_engine_client_(new chromeos::FakeUpdateEngineClient) { 267 update_engine_client_(new chromeos::FakeUpdateEngineClient) {
(...skipping 1167 matching lines...) Expand 10 before | Expand all | Expand 10 after
1435 AccountId::FromUserEmail(kPublicAccountId)); 1435 AccountId::FromUserEmail(kPublicAccountId));
1436 EXPECT_CALL(*user_manager_, IsLoggedInAsPublicAccount()) 1436 EXPECT_CALL(*user_manager_, IsLoggedInAsPublicAccount())
1437 .WillRepeatedly(Return(true)); 1437 .WillRepeatedly(Return(true));
1438 1438
1439 settings_helper_.SetBoolean(chromeos::kReportDeviceNetworkInterfaces, true); 1439 settings_helper_.SetBoolean(chromeos::kReportDeviceNetworkInterfaces, true);
1440 GetStatus(); 1440 GetStatus();
1441 VerifyNetworkReporting(); 1441 VerifyNetworkReporting();
1442 } 1442 }
1443 1443
1444 } // namespace policy 1444 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698