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

Unified Diff: chrome/browser/chromeos/policy/variations_service_policy_browsertest.cc

Issue 14268009: Support VariationsRestrictParameter in VariationsService for Chrome OS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fixed a test Created 7 years, 8 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/policy/variations_service_policy_browsertest.cc
diff --git a/chrome/browser/chromeos/policy/variations_service_policy_browsertest.cc b/chrome/browser/chromeos/policy/variations_service_policy_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..21478396795e706ad28042215071cce56e4d861b
--- /dev/null
+++ b/chrome/browser/chromeos/policy/variations_service_policy_browsertest.cc
@@ -0,0 +1,48 @@
+// Copyright (c) 2013 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/chromeos/policy/device_policy_builder.h"
+#include "chrome/browser/chromeos/policy/device_policy_cros_browser_test.h"
+#include "chrome/browser/chromeos/policy/proto/chrome_device_policy.pb.h"
+#include "chrome/browser/metrics/variations/variations_service.h"
+#include "chrome/test/base/testing_browser_process.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace policy {
+
+class VariationsServiceDevicePolicyTest : public DevicePolicyCrosBrowserTest {
+ protected:
+ VariationsServiceDevicePolicyTest() {}
+
+ void SetUpInProcessBrowserTestFixture() OVERRIDE {
+ DevicePolicyCrosBrowserTest::SetUpInProcessBrowserTestFixture();
+
+ InstallOwnerKey();
+ SetSpecificDevicePolicies();
+ RefreshDevicePolicy();
+ }
+
+ void SetSpecificDevicePolicies() {
+ // Setup the device policy DeviceVariationsRestrictParameter.
+ enterprise_management::ChromeDeviceSettingsProto& proto(
+ device_policy()->payload());
+ proto.mutable_variations_parameter()->set_parameter("restricted");
+ }
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(VariationsServiceDevicePolicyTest);
+};
+
+IN_PROC_BROWSER_TEST_F(VariationsServiceDevicePolicyTest, VariationsURLValid) {
+ const std::string default_variations_url =
+ chrome_variations::VariationsService::
+ GetDefaultVariationsServerURLForTesting();
+
+ // Device policy has updated the cros settings.
+ EXPECT_EQ(default_variations_url + "?restrict=restricted",
+ chrome_variations::VariationsService::GetVariationsServerURL(
+ TestingBrowserProcess::GetGlobal()->local_state()).spec());
+}
+
+} // namespace policy

Powered by Google App Engine
This is Rietveld 408576698