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

Unified Diff: chrome/browser/policy/configuration_policy_provider_test.h

Issue 8467011: Include only policy definitions that apply to the platfrom in the policy definition list. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed dead declarations. Created 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/configuration_policy_provider_test.h
diff --git a/chrome/browser/policy/configuration_policy_provider_test.h b/chrome/browser/policy/configuration_policy_provider_test.h
new file mode 100644
index 0000000000000000000000000000000000000000..90b36f034a14b4115513d753e8be0c452e7a4a51
--- /dev/null
+++ b/chrome/browser/policy/configuration_policy_provider_test.h
@@ -0,0 +1,112 @@
+// 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.
+
+#ifndef CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_PROVIDER_TEST_H_
+#define CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_PROVIDER_TEST_H_
+#pragma once
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/message_loop.h"
+#include "chrome/browser/policy/asynchronous_policy_test_base.h"
+#include "content/test/test_browser_thread.h"
+#include "policy/configuration_policy_type.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace base {
+class ListValue;
+}
+
+namespace policy {
+
+class AsynchronousPolicyProvider;
+struct PolicyDefinitionList;
+
+// A stripped-down policy definition list that contains entries for the
+// different policy setting types supported.
+namespace test_policy_definitions {
+
+// String policy keys.
+extern const char kKeyString[];
+extern const char kKeyBoolean[];
+extern const char kKeyInteger[];
+extern const char kKeyStringList[];
+
+// Corresponding type constants.
+extern const ConfigurationPolicyType kPolicyString;
+extern const ConfigurationPolicyType kPolicyBoolean;
+extern const ConfigurationPolicyType kPolicyInteger;
+extern const ConfigurationPolicyType kPolicyStringList;
+
+// Policy definition list that contains entries for the keys above.
+extern const PolicyDefinitionList kList;
+
+} // namespace test_policy_definitions
+
+// An interface for creating a test policy provider and creating a policy
+// provider instance for testing. Used as the parameter to the abstract
+// ConfigurationPolicyProviderTest below.
+class PolicyProviderTestHarness {
+ public:
+ PolicyProviderTestHarness();
+ virtual ~PolicyProviderTestHarness();
+
+ // Actions to run at gtest SetUp() time.
+ virtual void SetUp() = 0;
+
+ // Create a new policy provider.
+ virtual AsynchronousPolicyProvider* CreateProvider(
+ const PolicyDefinitionList* policy_definition_list) = 0;
+
+ // Helpers to configure the environment the policy provider reads from.
+ virtual void InstallEmptyPolicy() = 0;
+ virtual void InstallStringPolicy(const std::string& policy_name,
+ const std::string& policy_value) = 0;
+ virtual void InstallIntegerPolicy(const std::string& policy_name,
+ int policy_value) = 0;
+ virtual void InstallBooleanPolicy(const std::string& policy_name,
+ bool policy_value) = 0;
+ virtual void InstallStringListPolicy(const std::string& policy_name,
+ const ListValue* policy_value) = 0;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(PolicyProviderTestHarness);
+};
+
+// A factory method for creating a test harness.
+typedef PolicyProviderTestHarness* (*CreatePolicyProviderTestHarness)();
+
+// Abstract policy provider test. This is meant to be instantiated for each
+// policy provider implementation, passing in a suitable harness factory
+// function as the test parameter.
+class ConfigurationPolicyProviderTest
+ : public AsynchronousPolicyTestBase,
+ public testing::WithParamInterface<CreatePolicyProviderTestHarness> {
+ protected:
+ ConfigurationPolicyProviderTest();
+ virtual ~ConfigurationPolicyProviderTest();
+
+ virtual void SetUp();
+ virtual void TearDown();
+
+ // Installs a valid policy and checks whether the provider returns the
+ // |expected_value|.
+ void CheckValue(const char* policy_name,
+ ConfigurationPolicyType policy_type,
+ const base::Value& expected_value,
+ base::Closure install_value);
+
+ scoped_ptr<PolicyProviderTestHarness> test_harness_;
+ scoped_ptr<AsynchronousPolicyProvider> provider_;
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(ConfigurationPolicyProviderTest);
+};
+
+} // namespace policy
+
+#endif // CHROME_BROWSER_POLICY_CONFIGURATION_POLICY_PROVIDER_TEST_H_

Powered by Google App Engine
This is Rietveld 408576698