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

Unified Diff: chrome/browser/chromeos/ownership/owner_settings_service_chromeos_unittest.cc

Issue 2230533002: Delete dead consumer enrollment code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 4 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/chromeos/ownership/owner_settings_service_chromeos_unittest.cc
diff --git a/chrome/browser/chromeos/ownership/owner_settings_service_chromeos_unittest.cc b/chrome/browser/chromeos/ownership/owner_settings_service_chromeos_unittest.cc
index 4384762a1acd592f6b94bd1055dc48841953265f..a2f5b2384eec9862ac610acefc36bb44e92e27be 100644
--- a/chrome/browser/chromeos/ownership/owner_settings_service_chromeos_unittest.cc
+++ b/chrome/browser/chromeos/ownership/owner_settings_service_chromeos_unittest.cc
@@ -182,128 +182,6 @@ TEST_F(OwnerSettingsServiceChromeOSTest, FailedSetRequest) {
.release_channel());
}
-TEST_F(OwnerSettingsServiceChromeOSTest, SetManagementSettingsModeTransition) {
- ReloadDeviceSettings();
- EXPECT_EQ(DeviceSettingsService::STORE_SUCCESS,
- device_settings_service_.status());
-
- // The initial management mode should be LOCAL_OWNER.
- EXPECT_TRUE(device_settings_service_.policy_data()->has_management_mode());
- EXPECT_EQ(em::PolicyData::LOCAL_OWNER,
- device_settings_service_.policy_data()->management_mode());
-
- OwnerSettingsServiceChromeOS::ManagementSettings management_settings;
- management_settings.management_mode =
- policy::MANAGEMENT_MODE_CONSUMER_MANAGED;
- management_settings.request_token = "fake_request_token";
- management_settings.device_id = "fake_device_id";
- OwnerSettingsServiceChromeOS::OnManagementSettingsSetCallback
- on_management_settings_set_callback =
- base::Bind(&OwnerSettingsServiceChromeOSTest::OnManagementSettingsSet,
- base::Unretained(this));
-
- // LOCAL_OWNER -> CONSUMER_MANAGED: Okay.
- service_->SetManagementSettings(management_settings,
- on_management_settings_set_callback);
- FlushDeviceSettings();
-
- EXPECT_TRUE(management_settings_set_);
- EXPECT_EQ(em::PolicyData::CONSUMER_MANAGED,
- device_settings_service_.policy_data()->management_mode());
-
- // CONSUMER_MANAGED -> ENTERPRISE_MANAGED: Invalid.
- management_settings.management_mode =
- policy::MANAGEMENT_MODE_ENTERPRISE_MANAGED;
- service_->SetManagementSettings(management_settings,
- on_management_settings_set_callback);
- FlushDeviceSettings();
-
- EXPECT_FALSE(management_settings_set_);
- EXPECT_EQ(em::PolicyData::CONSUMER_MANAGED,
- device_settings_service_.policy_data()->management_mode());
-
- // CONSUMER_MANAGED -> LOCAL_OWNER: Okay.
- management_settings.management_mode = policy::MANAGEMENT_MODE_LOCAL_OWNER;
- service_->SetManagementSettings(management_settings,
- on_management_settings_set_callback);
- FlushDeviceSettings();
-
- EXPECT_TRUE(management_settings_set_);
- EXPECT_EQ(em::PolicyData::LOCAL_OWNER,
- device_settings_service_.policy_data()->management_mode());
-
- // LOCAL_OWNER -> ENTERPRISE_MANAGED: Invalid.
- management_settings.management_mode =
- policy::MANAGEMENT_MODE_ENTERPRISE_MANAGED;
- service_->SetManagementSettings(management_settings,
- on_management_settings_set_callback);
- FlushDeviceSettings();
-
- EXPECT_FALSE(management_settings_set_);
- EXPECT_EQ(em::PolicyData::LOCAL_OWNER,
- device_settings_service_.policy_data()->management_mode());
-
- // Inject a policy data with management mode set to ENTERPRISE_MANAGED.
- device_policy_.policy_data().set_management_mode(
- em::PolicyData::ENTERPRISE_MANAGED);
- device_policy_.Build();
- device_settings_test_helper_.set_policy_blob(device_policy_.GetBlob());
- ReloadDeviceSettings();
- EXPECT_EQ(em::PolicyData::ENTERPRISE_MANAGED,
- device_settings_service_.policy_data()->management_mode());
-
- // ENTERPRISE_MANAGED -> LOCAL_OWNER: Invalid.
- management_settings.management_mode = policy::MANAGEMENT_MODE_LOCAL_OWNER;
- service_->SetManagementSettings(management_settings,
- on_management_settings_set_callback);
- FlushDeviceSettings();
-
- EXPECT_FALSE(management_settings_set_);
- EXPECT_EQ(em::PolicyData::ENTERPRISE_MANAGED,
- device_settings_service_.policy_data()->management_mode());
-
- // ENTERPRISE_MANAGED -> CONSUMER_MANAGED: Invalid.
- management_settings.management_mode =
- policy::MANAGEMENT_MODE_CONSUMER_MANAGED;
- service_->SetManagementSettings(management_settings,
- on_management_settings_set_callback);
- FlushDeviceSettings();
-
- EXPECT_FALSE(management_settings_set_);
- EXPECT_EQ(em::PolicyData::ENTERPRISE_MANAGED,
- device_settings_service_.policy_data()->management_mode());
-}
-
-TEST_F(OwnerSettingsServiceChromeOSTest, SetManagementSettingsSuccess) {
- ReloadDeviceSettings();
- EXPECT_EQ(DeviceSettingsService::STORE_SUCCESS,
- device_settings_service_.status());
-
- OwnerSettingsServiceChromeOS::ManagementSettings management_settings;
- management_settings.management_mode =
- policy::MANAGEMENT_MODE_CONSUMER_MANAGED;
- management_settings.request_token = "fake_request_token";
- management_settings.device_id = "fake_device_id";
- service_->SetManagementSettings(
- management_settings,
- base::Bind(&OwnerSettingsServiceChromeOSTest::OnManagementSettingsSet,
- base::Unretained(this)));
- FlushDeviceSettings();
-
- EXPECT_EQ(DeviceSettingsService::STORE_SUCCESS,
- device_settings_service_.status());
- ASSERT_TRUE(device_settings_service_.device_settings());
-
- // Check that the loaded policy_data contains the expected values.
- const em::PolicyData* policy_data = device_settings_service_.policy_data();
- EXPECT_EQ(policy::dm_protocol::kChromeDevicePolicyType,
- policy_data->policy_type());
- EXPECT_EQ(device_settings_service_.GetUsername(), policy_data->username());
- EXPECT_EQ(em::PolicyData::CONSUMER_MANAGED, policy_data->management_mode());
- EXPECT_EQ("fake_request_token", policy_data->request_token());
- EXPECT_EQ("fake_device_id", policy_data->device_id());
-}
-
TEST_F(OwnerSettingsServiceChromeOSTest, ForceWhitelist) {
EXPECT_FALSE(FindInListValue(device_policy_.policy_data().username(),
provider_->Get(kAccountsPrefUsers)));

Powered by Google App Engine
This is Rietveld 408576698