[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 1 | // Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 5 | #include <utility> |
| 6 | |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 7 | #include "base/json/json_reader.h" |
Nick Peterson | 6bdf582 | 2017-06-01 20:42:45 | [diff] [blame] | 8 | #include "base/strings/stringprintf.h" |
Nick Peterson | d952cb77 | 2018-03-07 15:46:03 | [diff] [blame] | 9 | #include "build/build_config.h" |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 10 | #include "chrome/browser/extensions/external_policy_loader.h" |
| 11 | #include "chrome/browser/extensions/policy_handlers.h" |
[email protected] | f6c403b | 2013-12-05 19:01:25 | [diff] [blame] | 12 | #include "components/policy/core/browser/policy_error_map.h" |
[email protected] | c4a138a | 2013-11-21 19:54:57 | [diff] [blame] | 13 | #include "components/policy/core/common/policy_map.h" |
fhorschig | 64834b71 | 2015-09-21 14:20:23 | [diff] [blame] | 14 | #include "components/policy/core/common/policy_types.h" |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 15 | #include "components/policy/core/common/schema.h" |
brettw | 39d6ba4 | 2016-08-24 16:56:38 | [diff] [blame] | 16 | #include "components/policy/policy_constants.h" |
brettw | b1fc1b8 | 2016-02-02 00:19:08 | [diff] [blame] | 17 | #include "components/prefs/pref_value_map.h" |
[email protected] | 234fc5ff | 2014-01-16 23:32:28 | [diff] [blame] | 18 | #include "extensions/browser/pref_names.h" |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 19 | #include "testing/gtest/include/gtest/gtest.h" |
| 20 | |
Nick Peterson | d952cb77 | 2018-03-07 15:46:03 | [diff] [blame] | 21 | #if defined(OS_WIN) |
| 22 | #include "base/win/win_util.h" |
| 23 | #endif |
| 24 | |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 25 | namespace extensions { |
| 26 | |
| 27 | const char kTestPref[] = "unit_test.test_pref"; |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 28 | const char kTestManagementPolicy1[] = |
| 29 | "{" |
| 30 | " \"abcdefghijklmnopabcdefghijklmnop\": {" |
| 31 | " \"installation_mode\": \"force_installed\"," |
| 32 | " }," |
| 33 | "}"; |
| 34 | const char kTestManagementPolicy2[] = |
| 35 | "{" |
| 36 | " \"abcdefghijklmnopabcdefghijklmnop\": {" |
| 37 | " \"installation_mode\": \"force_installed\"," |
| 38 | " \"update_url\": \"http://example.com/app\"," |
| 39 | " }," |
| 40 | " \"*\": {" |
| 41 | " \"installation_mode\": \"blocked\"," |
| 42 | " }," |
| 43 | "}"; |
Nick Peterson | 6bdf582 | 2017-06-01 20:42:45 | [diff] [blame] | 44 | const char kTestManagementPolicy3[] = |
| 45 | "{" |
| 46 | " \"*\": {" |
| 47 | " \"runtime_blocked_hosts\": [\"%s\"]" |
| 48 | " }" |
| 49 | "}"; |
| 50 | const char kTestManagementPolicy4[] = |
| 51 | "{" |
| 52 | " \"*\": {" |
| 53 | " \"runtime_allowed_hosts\": [\"%s\"]" |
| 54 | " }" |
| 55 | "}"; |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 56 | |
| 57 | TEST(ExtensionListPolicyHandlerTest, CheckPolicySettings) { |
| 58 | base::ListValue list; |
| 59 | policy::PolicyMap policy_map; |
| 60 | policy::PolicyErrorMap errors; |
| 61 | ExtensionListPolicyHandler handler( |
| 62 | policy::key::kExtensionInstallBlacklist, kTestPref, true); |
| 63 | |
| 64 | policy_map.Set(policy::key::kExtensionInstallBlacklist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 65 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 66 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 67 | errors.Clear(); |
| 68 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 69 | EXPECT_TRUE(errors.empty()); |
| 70 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 71 | list.AppendString("abcdefghijklmnopabcdefghijklmnop"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 72 | policy_map.Set(policy::key::kExtensionInstallBlacklist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 73 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 74 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 75 | errors.Clear(); |
| 76 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 77 | EXPECT_TRUE(errors.empty()); |
| 78 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 79 | list.AppendString("*"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 80 | policy_map.Set(policy::key::kExtensionInstallBlacklist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 81 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 82 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 83 | errors.Clear(); |
| 84 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 85 | EXPECT_TRUE(errors.empty()); |
| 86 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 87 | list.AppendString("invalid"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 88 | policy_map.Set(policy::key::kExtensionInstallBlacklist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 89 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 90 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 91 | errors.Clear(); |
| 92 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 93 | EXPECT_FALSE(errors.empty()); |
| 94 | EXPECT_FALSE( |
| 95 | errors.GetErrors(policy::key::kExtensionInstallBlacklist).empty()); |
| 96 | } |
| 97 | |
Nick Peterson | 6bdf582 | 2017-06-01 20:42:45 | [diff] [blame] | 98 | TEST(ExtensionSettingsPolicyHandlerTest, CheckPolicySettingsURL) { |
Nick Peterson | 87ecb10 | 2018-10-16 04:55:01 | [diff] [blame] | 99 | std::vector<std::string> good_urls = {"*://*.example.com", "*://example.com", |
| 100 | "http://cat.example.com", "<all_urls>"}; |
Nick Peterson | 6bdf582 | 2017-06-01 20:42:45 | [diff] [blame] | 101 | |
| 102 | // Invalid URLPattern or with a non-standard path |
| 103 | std::vector<std::string> bad_urls = { |
| 104 | "://*.example.com", "*://example.com/cat*", "*://example.com/", |
| 105 | "*://*.example.com/*cat", "*://example.com/cat/*", "bad", |
Nick Peterson | 87ecb10 | 2018-10-16 04:55:01 | [diff] [blame] | 106 | "*://example.com/*", "https://example.*", "*://*.example.*"}; |
Nick Peterson | 6bdf582 | 2017-06-01 20:42:45 | [diff] [blame] | 107 | |
| 108 | // Crafts and parses a ExtensionSettings policy to test URL parsing. |
| 109 | auto url_parses_successfully = [](const char* policy_template, |
| 110 | const std::string& url) { |
| 111 | std::string policy = base::StringPrintf(policy_template, url.c_str()); |
| 112 | std::string error; |
| 113 | std::unique_ptr<base::Value> policy_value = |
Lei Zhang | 582ecd1 | 2019-02-13 20:28:54 | [diff] [blame^] | 114 | base::JSONReader::ReadAndReturnErrorDeprecated( |
Nick Peterson | 6bdf582 | 2017-06-01 20:42:45 | [diff] [blame] | 115 | policy, base::JSONParserOptions::JSON_ALLOW_TRAILING_COMMAS, |
| 116 | nullptr, &error); |
| 117 | if (!policy_value) |
| 118 | return false; |
| 119 | |
| 120 | policy::Schema chrome_schema = |
| 121 | policy::Schema::Wrap(policy::GetChromeSchemaData()); |
| 122 | policy::PolicyMap policy_map; |
| 123 | ExtensionSettingsPolicyHandler handler(chrome_schema); |
| 124 | |
| 125 | policy_map.Set(policy::key::kExtensionSettings, |
| 126 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 127 | policy::POLICY_SOURCE_CLOUD, std::move(policy_value), |
| 128 | nullptr); |
| 129 | |
| 130 | policy::PolicyErrorMap errors; |
| 131 | return handler.CheckPolicySettings(policy_map, &errors) && errors.empty(); |
| 132 | }; |
| 133 | |
| 134 | for (const std::string& url : good_urls) { |
| 135 | EXPECT_TRUE(url_parses_successfully(kTestManagementPolicy3, url)) << url; |
| 136 | EXPECT_TRUE(url_parses_successfully(kTestManagementPolicy4, url)) << url; |
| 137 | } |
| 138 | |
| 139 | for (const std::string& url : bad_urls) { |
| 140 | EXPECT_FALSE(url_parses_successfully(kTestManagementPolicy3, url)) << url; |
| 141 | EXPECT_FALSE(url_parses_successfully(kTestManagementPolicy4, url)) << url; |
| 142 | } |
| 143 | } |
| 144 | |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 145 | TEST(ExtensionListPolicyHandlerTest, ApplyPolicySettings) { |
| 146 | base::ListValue policy; |
| 147 | base::ListValue expected; |
| 148 | policy::PolicyMap policy_map; |
| 149 | PrefValueMap prefs; |
| 150 | base::Value* value = NULL; |
| 151 | ExtensionListPolicyHandler handler( |
| 152 | policy::key::kExtensionInstallBlacklist, kTestPref, false); |
| 153 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 154 | policy.AppendString("abcdefghijklmnopabcdefghijklmnop"); |
| 155 | expected.AppendString("abcdefghijklmnopabcdefghijklmnop"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 156 | |
| 157 | policy_map.Set(policy::key::kExtensionInstallBlacklist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 158 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 159 | policy::POLICY_SOURCE_CLOUD, policy.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 160 | handler.ApplyPolicySettings(policy_map, &prefs); |
| 161 | EXPECT_TRUE(prefs.GetValue(kTestPref, &value)); |
jdoerrie | 8551f92 | 2017-07-25 10:55:13 | [diff] [blame] | 162 | EXPECT_EQ(expected, *value); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 163 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 164 | policy.AppendString("invalid"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 165 | policy_map.Set(policy::key::kExtensionInstallBlacklist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 166 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 167 | policy::POLICY_SOURCE_CLOUD, policy.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 168 | handler.ApplyPolicySettings(policy_map, &prefs); |
| 169 | EXPECT_TRUE(prefs.GetValue(kTestPref, &value)); |
jdoerrie | 8551f92 | 2017-07-25 10:55:13 | [diff] [blame] | 170 | EXPECT_EQ(expected, *value); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 171 | } |
| 172 | |
| 173 | TEST(ExtensionInstallForcelistPolicyHandlerTest, CheckPolicySettings) { |
| 174 | base::ListValue list; |
| 175 | policy::PolicyMap policy_map; |
| 176 | policy::PolicyErrorMap errors; |
| 177 | ExtensionInstallForcelistPolicyHandler handler; |
| 178 | |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 179 | // Start with an empty policy. |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 180 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 181 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 182 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 183 | errors.Clear(); |
| 184 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 185 | EXPECT_TRUE(errors.empty()); |
| 186 | |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 187 | // Add a correct entry. No errors should be generated. |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 188 | list.AppendString("abcdefghijklmnopabcdefghijklmnop;http://example.com"); |
| 189 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 190 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 191 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 192 | errors.Clear(); |
| 193 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 194 | EXPECT_TRUE(errors.empty()); |
| 195 | |
| 196 | // Add an erroneous entry. This should generate an error, but the good |
| 197 | // entry should still be translated successfully. |
| 198 | list.AppendString("adfasdf;http://example.com"); |
| 199 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 200 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 201 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 202 | errors.Clear(); |
| 203 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 204 | EXPECT_EQ(1U, errors.size()); |
| 205 | |
| 206 | // Add an entry with bad URL, which should generate another error. |
| 207 | list.AppendString("abcdefghijklmnopabcdefghijklmnop;nourl"); |
| 208 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 209 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 210 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 211 | errors.Clear(); |
| 212 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 213 | EXPECT_EQ(2U, errors.size()); |
| 214 | |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 215 | // Just an extension ID should be accepted. |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 216 | list.AppendString("abcdefghijklmnopabcdefghijklmnop"); |
| 217 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 218 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 219 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 220 | errors.Clear(); |
| 221 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 222 | EXPECT_EQ(2U, errors.size()); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 223 | } |
| 224 | |
| 225 | TEST(ExtensionInstallForcelistPolicyHandlerTest, ApplyPolicySettings) { |
| 226 | base::ListValue policy; |
| 227 | base::DictionaryValue expected; |
| 228 | policy::PolicyMap policy_map; |
| 229 | PrefValueMap prefs; |
| 230 | base::Value* value = NULL; |
| 231 | ExtensionInstallForcelistPolicyHandler handler; |
| 232 | |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 233 | // Start with the policy being missing. This shouldn't affect the pref. |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 234 | handler.ApplyPolicySettings(policy_map, &prefs); |
[email protected] | 234fc5ff | 2014-01-16 23:32:28 | [diff] [blame] | 235 | EXPECT_FALSE(prefs.GetValue(pref_names::kInstallForceList, &value)); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 236 | EXPECT_FALSE(value); |
| 237 | |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 238 | // Set the policy to an empty value. This shouldn't affect the pref. |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 239 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 240 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 241 | policy::POLICY_SOURCE_CLOUD, policy.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 242 | handler.ApplyPolicySettings(policy_map, &prefs); |
[email protected] | 234fc5ff | 2014-01-16 23:32:28 | [diff] [blame] | 243 | EXPECT_TRUE(prefs.GetValue(pref_names::kInstallForceList, &value)); |
jdoerrie | 8551f92 | 2017-07-25 10:55:13 | [diff] [blame] | 244 | EXPECT_EQ(expected, *value); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 245 | |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 246 | // Add a correct entry to the policy. The pref should contain a corresponding |
| 247 | // entry. |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 248 | policy.AppendString("abcdefghijklmnopabcdefghijklmnop;http://example.com"); |
| 249 | extensions::ExternalPolicyLoader::AddExtension( |
| 250 | &expected, "abcdefghijklmnopabcdefghijklmnop", "http://example.com"); |
| 251 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 252 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 253 | policy::POLICY_SOURCE_CLOUD, policy.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 254 | handler.ApplyPolicySettings(policy_map, &prefs); |
[email protected] | 234fc5ff | 2014-01-16 23:32:28 | [diff] [blame] | 255 | EXPECT_TRUE(prefs.GetValue(pref_names::kInstallForceList, &value)); |
jdoerrie | 8551f92 | 2017-07-25 10:55:13 | [diff] [blame] | 256 | EXPECT_EQ(expected, *value); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 257 | |
Maksim Ivanov | eaac2ff | 2018-04-16 16:23:24 | [diff] [blame] | 258 | // Add a correct entry with an omitted update URL. The pref should contain now |
| 259 | // two entries, with the default update URL substituted for the new entry. |
| 260 | // Note: the URL hardcoded below is part of the public policy contract (as |
| 261 | // documented in the policy_templates.json file), and therefore any changes to |
| 262 | // it must be carefully thought out. |
| 263 | policy.AppendString("bcdefghijklmnopabcdefghijklmnopa"); |
| 264 | extensions::ExternalPolicyLoader::AddExtension( |
| 265 | &expected, "bcdefghijklmnopabcdefghijklmnopa", |
| 266 | "https://clients2.google.com/service/update2/crx"); |
| 267 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
| 268 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 269 | policy::POLICY_SOURCE_CLOUD, policy.CreateDeepCopy(), nullptr); |
| 270 | handler.ApplyPolicySettings(policy_map, &prefs); |
| 271 | EXPECT_TRUE(prefs.GetValue(pref_names::kInstallForceList, &value)); |
| 272 | EXPECT_EQ(expected, *value); |
| 273 | |
| 274 | // Add an invalid entry. The pref should still contain two previous entries. |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 275 | policy.AppendString("invalid"); |
| 276 | policy_map.Set(policy::key::kExtensionInstallForcelist, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 277 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 278 | policy::POLICY_SOURCE_CLOUD, policy.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 279 | handler.ApplyPolicySettings(policy_map, &prefs); |
[email protected] | 234fc5ff | 2014-01-16 23:32:28 | [diff] [blame] | 280 | EXPECT_TRUE(prefs.GetValue(pref_names::kInstallForceList, &value)); |
jdoerrie | 8551f92 | 2017-07-25 10:55:13 | [diff] [blame] | 281 | EXPECT_EQ(expected, *value); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 282 | } |
| 283 | |
| 284 | TEST(ExtensionURLPatternListPolicyHandlerTest, CheckPolicySettings) { |
| 285 | base::ListValue list; |
| 286 | policy::PolicyMap policy_map; |
| 287 | policy::PolicyErrorMap errors; |
| 288 | ExtensionURLPatternListPolicyHandler handler( |
| 289 | policy::key::kExtensionInstallSources, kTestPref); |
| 290 | |
| 291 | policy_map.Set(policy::key::kExtensionInstallSources, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 292 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 293 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 294 | errors.Clear(); |
| 295 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 296 | EXPECT_TRUE(errors.empty()); |
| 297 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 298 | list.AppendString("http://*.google.com/*"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 299 | policy_map.Set(policy::key::kExtensionInstallSources, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 300 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 301 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 302 | errors.Clear(); |
| 303 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 304 | EXPECT_TRUE(errors.empty()); |
| 305 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 306 | list.AppendString("<all_urls>"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 307 | policy_map.Set(policy::key::kExtensionInstallSources, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 308 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 309 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 310 | errors.Clear(); |
| 311 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 312 | EXPECT_TRUE(errors.empty()); |
| 313 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 314 | list.AppendString("invalid"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 315 | policy_map.Set(policy::key::kExtensionInstallSources, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 316 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 317 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 318 | errors.Clear(); |
| 319 | EXPECT_FALSE(handler.CheckPolicySettings(policy_map, &errors)); |
| 320 | EXPECT_FALSE(errors.empty()); |
| 321 | EXPECT_FALSE(errors.GetErrors(policy::key::kExtensionInstallSources).empty()); |
| 322 | |
| 323 | // URLPattern syntax has a different way to express 'all urls'. Though '*' |
| 324 | // would be compatible today, it would be brittle, so we disallow. |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 325 | list.AppendString("*"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 326 | policy_map.Set(policy::key::kExtensionInstallSources, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 327 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 328 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 329 | errors.Clear(); |
| 330 | EXPECT_FALSE(handler.CheckPolicySettings(policy_map, &errors)); |
| 331 | EXPECT_FALSE(errors.empty()); |
| 332 | EXPECT_FALSE(errors.GetErrors(policy::key::kExtensionInstallSources).empty()); |
| 333 | } |
| 334 | |
| 335 | TEST(ExtensionURLPatternListPolicyHandlerTest, ApplyPolicySettings) { |
| 336 | base::ListValue list; |
| 337 | policy::PolicyMap policy_map; |
| 338 | PrefValueMap prefs; |
| 339 | base::Value* value = NULL; |
| 340 | ExtensionURLPatternListPolicyHandler handler( |
| 341 | policy::key::kExtensionInstallSources, kTestPref); |
| 342 | |
dcheng | d9ea6386 | 2016-06-03 02:27:18 | [diff] [blame] | 343 | list.AppendString("https://corp.monkey.net/*"); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 344 | policy_map.Set(policy::key::kExtensionInstallSources, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 345 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 346 | policy::POLICY_SOURCE_CLOUD, list.CreateDeepCopy(), nullptr); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 347 | handler.ApplyPolicySettings(policy_map, &prefs); |
| 348 | ASSERT_TRUE(prefs.GetValue(kTestPref, &value)); |
jdoerrie | 8551f92 | 2017-07-25 10:55:13 | [diff] [blame] | 349 | EXPECT_EQ(list, *value); |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 350 | } |
| 351 | |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 352 | TEST(ExtensionSettingsPolicyHandlerTest, CheckPolicySettings) { |
| 353 | std::string error; |
dcheng | c963c714 | 2016-04-08 03:55:22 | [diff] [blame] | 354 | std::unique_ptr<base::Value> policy_value = |
Lei Zhang | 582ecd1 | 2019-02-13 20:28:54 | [diff] [blame^] | 355 | base::JSONReader::ReadAndReturnErrorDeprecated( |
dcheng | c963c714 | 2016-04-08 03:55:22 | [diff] [blame] | 356 | kTestManagementPolicy1, |
| 357 | base::JSONParserOptions::JSON_ALLOW_TRAILING_COMMAS, NULL, &error); |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 358 | ASSERT_TRUE(policy_value.get()) << error; |
| 359 | |
| 360 | policy::Schema chrome_schema = |
| 361 | policy::Schema::Wrap(policy::GetChromeSchemaData()); |
| 362 | policy::PolicyMap policy_map; |
| 363 | policy::PolicyErrorMap errors; |
| 364 | ExtensionSettingsPolicyHandler handler(chrome_schema); |
| 365 | |
| 366 | policy_map.Set(policy::key::kExtensionSettings, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 367 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 368 | policy::POLICY_SOURCE_CLOUD, std::move(policy_value), nullptr); |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 369 | // CheckPolicySettings() fails due to missing update URL. |
| 370 | EXPECT_FALSE(handler.CheckPolicySettings(policy_map, &errors)); |
| 371 | EXPECT_FALSE(errors.empty()); |
| 372 | } |
| 373 | |
| 374 | TEST(ExtensionSettingsPolicyHandlerTest, ApplyPolicySettings) { |
Nick Peterson | d952cb77 | 2018-03-07 15:46:03 | [diff] [blame] | 375 | // Mark as enterprise managed. |
| 376 | #if defined(OS_WIN) |
Roger Tawa | c1b544a8 | 2018-08-24 16:23:25 | [diff] [blame] | 377 | base::win::ScopedDomainStateForTesting scoped_domain(true); |
Nick Peterson | d952cb77 | 2018-03-07 15:46:03 | [diff] [blame] | 378 | #endif |
| 379 | |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 380 | std::string error; |
dcheng | c963c714 | 2016-04-08 03:55:22 | [diff] [blame] | 381 | std::unique_ptr<base::Value> policy_value = |
Lei Zhang | 582ecd1 | 2019-02-13 20:28:54 | [diff] [blame^] | 382 | base::JSONReader::ReadAndReturnErrorDeprecated( |
dcheng | c963c714 | 2016-04-08 03:55:22 | [diff] [blame] | 383 | kTestManagementPolicy2, |
| 384 | base::JSONParserOptions::JSON_ALLOW_TRAILING_COMMAS, NULL, &error); |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 385 | ASSERT_TRUE(policy_value.get()) << error; |
| 386 | |
| 387 | policy::Schema chrome_schema = |
| 388 | policy::Schema::Wrap(policy::GetChromeSchemaData()); |
| 389 | policy::PolicyMap policy_map; |
| 390 | policy::PolicyErrorMap errors; |
| 391 | PrefValueMap prefs; |
| 392 | ExtensionSettingsPolicyHandler handler(chrome_schema); |
| 393 | |
| 394 | policy_map.Set(policy::key::kExtensionSettings, |
dcheng | 3b344bc2 | 2016-05-10 02:26:09 | [diff] [blame] | 395 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 396 | policy::POLICY_SOURCE_CLOUD, policy_value->CreateDeepCopy(), |
| 397 | nullptr); |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 398 | EXPECT_TRUE(handler.CheckPolicySettings(policy_map, &errors)); |
| 399 | handler.ApplyPolicySettings(policy_map, &prefs); |
| 400 | base::Value* value = NULL; |
| 401 | ASSERT_TRUE(prefs.GetValue(pref_names::kExtensionManagement, &value)); |
jdoerrie | 8551f92 | 2017-07-25 10:55:13 | [diff] [blame] | 402 | EXPECT_EQ(*policy_value, *value); |
binjin | 1e1cc33a | 2014-10-09 18:08:16 | [diff] [blame] | 403 | } |
| 404 | |
Nick Peterson | d952cb77 | 2018-03-07 15:46:03 | [diff] [blame] | 405 | // Only enterprise managed machines can auto install extensions from a location |
| 406 | // other than the webstore https://crbug.com/809004. |
| 407 | #if defined(OS_WIN) |
| 408 | TEST(ExtensionSettingsPolicyHandlerTest, NonManagedOffWebstoreExtension) { |
| 409 | // Mark as not enterprise managed. |
Roger Tawa | c1b544a8 | 2018-08-24 16:23:25 | [diff] [blame] | 410 | base::win::ScopedDomainStateForTesting scoped_domain(false); |
Nick Peterson | d952cb77 | 2018-03-07 15:46:03 | [diff] [blame] | 411 | |
| 412 | std::string error; |
| 413 | std::unique_ptr<base::Value> policy_value = |
Lei Zhang | 582ecd1 | 2019-02-13 20:28:54 | [diff] [blame^] | 414 | base::JSONReader::ReadAndReturnErrorDeprecated( |
Nick Peterson | d952cb77 | 2018-03-07 15:46:03 | [diff] [blame] | 415 | kTestManagementPolicy2, |
| 416 | base::JSONParserOptions::JSON_ALLOW_TRAILING_COMMAS, nullptr, &error); |
| 417 | ASSERT_TRUE(policy_value.get()) << error; |
| 418 | |
| 419 | policy::Schema chrome_schema = |
| 420 | policy::Schema::Wrap(policy::GetChromeSchemaData()); |
| 421 | policy::PolicyMap policy_map; |
| 422 | policy::PolicyErrorMap errors; |
| 423 | PrefValueMap prefs; |
| 424 | ExtensionSettingsPolicyHandler handler(chrome_schema); |
| 425 | |
| 426 | policy_map.Set(policy::key::kExtensionSettings, |
| 427 | policy::POLICY_LEVEL_MANDATORY, policy::POLICY_SCOPE_USER, |
| 428 | policy::POLICY_SOURCE_CLOUD, policy_value->CreateDeepCopy(), |
| 429 | nullptr); |
| 430 | EXPECT_FALSE(handler.CheckPolicySettings(policy_map, &errors)); |
| 431 | EXPECT_FALSE(errors.empty()); |
| 432 | } |
| 433 | #endif |
| 434 | |
[email protected] | 01253d27 | 2013-10-21 17:07:50 | [diff] [blame] | 435 | } // namespace extensions |