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

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

Issue 56623005: Policy providers all get a SchemaRegistry to work with. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@chrome-policy-schema-9-purge-with-callback
Patch Set: rebase Created 7 years, 1 month 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_local_account_policy_service.h" 5 #include "chrome/browser/chromeos/policy/device_local_account_policy_service.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/bind_helpers.h" 8 #include "base/bind_helpers.h"
9 #include "base/callback.h" 9 #include "base/callback.h"
10 #include "base/file_util.h" 10 #include "base/file_util.h"
(...skipping 13 matching lines...) Expand all
24 #include "chrome/browser/chromeos/settings/device_settings_service.h" 24 #include "chrome/browser/chromeos/settings/device_settings_service.h"
25 #include "chrome/browser/chromeos/settings/device_settings_test_helper.h" 25 #include "chrome/browser/chromeos/settings/device_settings_test_helper.h"
26 #include "chrome/browser/policy/cloud/cloud_policy_client.h" 26 #include "chrome/browser/policy/cloud/cloud_policy_client.h"
27 #include "chrome/browser/policy/cloud/cloud_policy_constants.h" 27 #include "chrome/browser/policy/cloud/cloud_policy_constants.h"
28 #include "chrome/browser/policy/cloud/cloud_policy_service.h" 28 #include "chrome/browser/policy/cloud/cloud_policy_service.h"
29 #include "chrome/browser/policy/cloud/mock_device_management_service.h" 29 #include "chrome/browser/policy/cloud/mock_device_management_service.h"
30 #include "chrome/browser/policy/cloud/policy_builder.h" 30 #include "chrome/browser/policy/cloud/policy_builder.h"
31 #include "chrome/browser/policy/external_data_fetcher.h" 31 #include "chrome/browser/policy/external_data_fetcher.h"
32 #include "chrome/browser/policy/mock_configuration_policy_provider.h" 32 #include "chrome/browser/policy/mock_configuration_policy_provider.h"
33 #include "chrome/browser/policy/proto/chromeos/chrome_device_policy.pb.h" 33 #include "chrome/browser/policy/proto/chromeos/chrome_device_policy.pb.h"
34 #include "chrome/browser/policy/schema_registry.h"
34 #include "chrome/common/chrome_paths.h" 35 #include "chrome/common/chrome_paths.h"
35 #include "chromeos/chromeos_paths.h" 36 #include "chromeos/chromeos_paths.h"
36 #include "chromeos/dbus/power_policy_controller.h" 37 #include "chromeos/dbus/power_policy_controller.h"
37 #include "net/url_request/url_request_context_getter.h" 38 #include "net/url_request/url_request_context_getter.h"
38 #include "policy/policy_constants.h" 39 #include "policy/policy_constants.h"
39 #include "policy/proto/cloud_policy.pb.h" 40 #include "policy/proto/cloud_policy.pb.h"
40 #include "testing/gtest/include/gtest/gtest.h" 41 #include "testing/gtest/include/gtest/gtest.h"
41 42
42 using testing::AnyNumber; 43 using testing::AnyNumber;
43 using testing::AtLeast; 44 using testing::AtLeast;
(...skipping 723 matching lines...) Expand 10 before | Expand all | Expand 10 after
767 } 768 }
768 769
769 class DeviceLocalAccountPolicyProviderTest 770 class DeviceLocalAccountPolicyProviderTest
770 : public DeviceLocalAccountPolicyServiceTestBase { 771 : public DeviceLocalAccountPolicyServiceTestBase {
771 protected: 772 protected:
772 DeviceLocalAccountPolicyProviderTest(); 773 DeviceLocalAccountPolicyProviderTest();
773 774
774 virtual void SetUp() OVERRIDE; 775 virtual void SetUp() OVERRIDE;
775 virtual void TearDown() OVERRIDE; 776 virtual void TearDown() OVERRIDE;
776 777
778 SchemaRegistry schema_registry_;
777 scoped_ptr<DeviceLocalAccountPolicyProvider> provider_; 779 scoped_ptr<DeviceLocalAccountPolicyProvider> provider_;
778 MockConfigurationPolicyObserver provider_observer_; 780 MockConfigurationPolicyObserver provider_observer_;
779 781
780 private: 782 private:
781 DISALLOW_COPY_AND_ASSIGN(DeviceLocalAccountPolicyProviderTest); 783 DISALLOW_COPY_AND_ASSIGN(DeviceLocalAccountPolicyProviderTest);
782 }; 784 };
783 785
784 DeviceLocalAccountPolicyProviderTest::DeviceLocalAccountPolicyProviderTest() { 786 DeviceLocalAccountPolicyProviderTest::DeviceLocalAccountPolicyProviderTest() {
785 CreatePolicyService(); 787 CreatePolicyService();
786 provider_.reset(new DeviceLocalAccountPolicyProvider( 788 provider_.reset(new DeviceLocalAccountPolicyProvider(
787 GenerateDeviceLocalAccountUserId(kAccount1, 789 GenerateDeviceLocalAccountUserId(kAccount1,
788 DeviceLocalAccount::TYPE_PUBLIC_SESSION), 790 DeviceLocalAccount::TYPE_PUBLIC_SESSION),
789 service_.get())); 791 service_.get()));
790 } 792 }
791 793
792 void DeviceLocalAccountPolicyProviderTest::SetUp() { 794 void DeviceLocalAccountPolicyProviderTest::SetUp() {
793 DeviceLocalAccountPolicyServiceTestBase::SetUp(); 795 DeviceLocalAccountPolicyServiceTestBase::SetUp();
794 provider_->Init(); 796 provider_->Init(&schema_registry_);
795 provider_->AddObserver(&provider_observer_); 797 provider_->AddObserver(&provider_observer_);
796 } 798 }
797 799
798 void DeviceLocalAccountPolicyProviderTest::TearDown() { 800 void DeviceLocalAccountPolicyProviderTest::TearDown() {
799 provider_->RemoveObserver(&provider_observer_); 801 provider_->RemoveObserver(&provider_observer_);
800 provider_->Shutdown(); 802 provider_->Shutdown();
801 provider_.reset(); 803 provider_.reset();
802 DeviceLocalAccountPolicyServiceTestBase::TearDown(); 804 DeviceLocalAccountPolicyServiceTestBase::TearDown();
803 } 805 }
804 806
(...skipping 135 matching lines...) Expand 10 before | Expand all | Expand 10 after
940 em::DeviceManagementResponse response; 942 em::DeviceManagementResponse response;
941 device_local_account_policy_.Build(); 943 device_local_account_policy_.Build();
942 response.mutable_policy_response()->add_response()->CopyFrom( 944 response.mutable_policy_response()->add_response()->CopyFrom(
943 device_local_account_policy_.policy()); 945 device_local_account_policy_.policy());
944 request_job->SendResponse(DM_STATUS_SUCCESS, response); 946 request_job->SendResponse(DM_STATUS_SUCCESS, response);
945 FlushDeviceSettings(); 947 FlushDeviceSettings();
946 Mock::VerifyAndClearExpectations(&provider_observer_); 948 Mock::VerifyAndClearExpectations(&provider_observer_);
947 } 949 }
948 950
949 } // namespace policy 951 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698