Added ConfigurationPolicyProvider::RefreshPolicies.

chrome/browser/policy/cloud_policy_provider_unittest.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/policy/cloud_policy_provider.h"
 
 #include "base/basictypes.h"
 #include "base/values.h"
+#include "chrome/browser/browser_process.h"
 #include "chrome/browser/policy/cloud_policy_cache_base.h"
 #include "chrome/browser/policy/cloud_policy_provider_impl.h"
 #include "chrome/browser/policy/configuration_policy_pref_store.h"
 #include "policy/policy_constants.h"
 #include "testing/gmock/include/gmock/gmock.h"
 
 using testing::AnyNumber;
 using testing::_;
 
 namespace policy {
@@ skipping 21 matching lines @@
   }
 
   PolicyMap* raw_recommended_policy() {
     return &recommended_policy_;
   }
 
   void SetUnmanaged() {
     is_unmanaged_ = true;
   }
 
-  void SetFetchingDone() {
-    // Implement pure virtual method.
-  }
-
   void set_initialized(bool initialized) {
     initialization_complete_ = initialized;
   }
 
  private:
   DISALLOW_COPY_AND_ASSIGN(MockCloudPolicyCache);
 };
 
 class CloudPolicyProviderTest : public testing::Test {
  protected:
   void CreateCloudPolicyProvider(CloudPolicyCacheBase::PolicyLevel level) {
-    cloud_policy_provider_.reset(new CloudPolicyProviderImpl(
-        GetChromePolicyDefinitionList(), level));
+    cloud_policy_provider_.reset(
+        new CloudPolicyProviderImpl(
+            g_browser_process->browser_policy_connector(),

[Mattias Nissler (ping if slow), 2011/11/18 11:00:25]

shouldn't we rather mock that thing?

[Joao da Silva, 2011/11/18 14:03:30]

It is "mocked" in unit_tests (as in, it has no providers and doesn't initialize
anything; see BrowserPolicyConnector::CreateForTests).

+            GetChromePolicyDefinitionList(),
+            level));
   }
 
   // Appends the caches to a provider and then provides the policies to
   // |policy_map_|.
   void RunCachesThroughProvider(MockCloudPolicyCache caches[], int n,
                                 CloudPolicyCacheBase::PolicyLevel level) {
     CloudPolicyProviderImpl provider(
+        g_browser_process->browser_policy_connector(),
         GetChromePolicyDefinitionList(),
         level);
     for (int i = 0; i < n; i++) {
       provider.AppendCache(&caches[i]);
     }
     policy_map_.reset(new PolicyMap());
     provider.Provide(policy_map_.get());
   }
 
   // Checks a string policy in |policy_map_|.
@@ skipping 150 matching lines @@
   B.Set(kPolicyProxyPacUrl, Value::CreateIntegerValue(b_value));
   B.Set(kPolicyProxyBypassList, Value::CreateIntegerValue(b_value));
 
   CombineTwoPolicyMaps(A, B, &C);
 
   EXPECT_TRUE(A.Equals(C));
   EXPECT_FALSE(B.Equals(C));
 }
 
 }  // namespace policy