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

Side by Side Diff: chrome/browser/extensions/api/enterprise_device_attributes/enterprise_device_attributes_apitest.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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "base/memory/ptr_util.h" 5 #include "base/memory/ptr_util.h"
6 #include "base/strings/stringprintf.h" 6 #include "base/strings/stringprintf.h"
7 #include "chrome/browser/browser_process.h" 7 #include "chrome/browser/browser_process.h"
8 #include "chrome/browser/chromeos/policy/affiliation_test_helper.h" 8 #include "chrome/browser/chromeos/policy/affiliation_test_helper.h"
9 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h" 9 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
10 #include "chrome/browser/chromeos/policy/device_policy_cros_browser_test.h" 10 #include "chrome/browser/chromeos/policy/device_policy_cros_browser_test.h"
(...skipping 82 matching lines...) Expand 10 before | Expand all | Expand 10 after
93 } 93 }
94 policy::UserPolicyBuilder user_policy; 94 policy::UserPolicyBuilder user_policy;
95 policy::affiliation_test_helper::SetUserAffiliationIDs( 95 policy::affiliation_test_helper::SetUserAffiliationIDs(
96 &user_policy, fake_session_manager_client, 96 &user_policy, fake_session_manager_client,
97 affiliated_account_id_.GetUserEmail(), user_affiliation_ids); 97 affiliated_account_id_.GetUserEmail(), user_affiliation_ids);
98 98
99 // Set up fake install attributes. 99 // Set up fake install attributes.
100 std::unique_ptr<chromeos::StubInstallAttributes> attributes = 100 std::unique_ptr<chromeos::StubInstallAttributes> attributes =
101 base::MakeUnique<chromeos::StubInstallAttributes>(); 101 base::MakeUnique<chromeos::StubInstallAttributes>();
102 102
103 attributes->SetRegistrationUser(affiliated_account_id_.GetUserEmail()); 103 attributes->SetEnterprise("fake-domain", "fake-id");
104 policy::BrowserPolicyConnectorChromeOS::SetInstallAttributesForTesting( 104 policy::BrowserPolicyConnectorChromeOS::SetInstallAttributesForTesting(
105 attributes.release()); 105 attributes.release());
106 106
107 test_helper_.InstallOwnerKey(); 107 test_helper_.InstallOwnerKey();
108 // Init the device policy. 108 // Init the device policy.
109 policy::DevicePolicyBuilder* device_policy = test_helper_.device_policy(); 109 policy::DevicePolicyBuilder* device_policy = test_helper_.device_policy();
110 device_policy->SetDefaultSigningKey(); 110 device_policy->SetDefaultSigningKey();
111 device_policy->policy_data().set_directory_api_id(kDeviceId); 111 device_policy->policy_data().set_directory_api_id(kDeviceId);
112 device_policy->Build(); 112 device_policy->Build();
113 113
(...skipping 125 matching lines...) Expand 10 before | Expand all | Expand 10 after
239 "'enterprise.deviceAttributes' is not allowed for specified install " 239 "'enterprise.deviceAttributes' is not allowed for specified install "
240 "location.", 240 "location.",
241 extension->install_warnings()[0].message); 241 extension->install_warnings()[0].message);
242 } 242 }
243 243
244 // Both cases of affiliated and non-affiliated on the device user are tested. 244 // Both cases of affiliated and non-affiliated on the device user are tested.
245 INSTANTIATE_TEST_CASE_P(AffiliationCheck, 245 INSTANTIATE_TEST_CASE_P(AffiliationCheck,
246 EnterpriseDeviceAttributesTest, 246 EnterpriseDeviceAttributesTest,
247 ::testing::Values(Params(true), Params(false))); 247 ::testing::Values(Params(true), Params(false)));
248 } // namespace extensions 248 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698