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

Side by Side Diff: components/search_engines/default_search_policy_handler_unittest.cc

Issue 397793002: Eliminate CreateBooleanValue from test files (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix bad merge Created 6 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "base/memory/scoped_ptr.h" 5 #include "base/memory/scoped_ptr.h"
6 #include "components/policy/core/browser/configuration_policy_pref_store.h" 6 #include "components/policy/core/browser/configuration_policy_pref_store.h"
7 #include "components/policy/core/browser/configuration_policy_pref_store_test.h" 7 #include "components/policy/core/browser/configuration_policy_pref_store_test.h"
8 #include "components/search_engines/default_search_manager.h" 8 #include "components/search_engines/default_search_manager.h"
9 #include "components/search_engines/default_search_policy_handler.h" 9 #include "components/search_engines/default_search_policy_handler.h"
10 #include "components/search_engines/search_engines_pref_names.h" 10 #include "components/search_engines/search_engines_pref_names.h"
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 "http://test.com/newtab"; 75 "http://test.com/newtab";
76 const char DefaultSearchPolicyHandlerTest::kFileSearchURL[] = 76 const char DefaultSearchPolicyHandlerTest::kFileSearchURL[] =
77 "file:///c:/path/to/search?t={searchTerms}"; 77 "file:///c:/path/to/search?t={searchTerms}";
78 const char DefaultSearchPolicyHandlerTest::kHostName[] = "test.com"; 78 const char DefaultSearchPolicyHandlerTest::kHostName[] = "test.com";
79 79
80 void DefaultSearchPolicyHandlerTest:: 80 void DefaultSearchPolicyHandlerTest::
81 BuildDefaultSearchPolicy(PolicyMap* policy) { 81 BuildDefaultSearchPolicy(PolicyMap* policy) {
82 base::ListValue* encodings = new base::ListValue(); 82 base::ListValue* encodings = new base::ListValue();
83 encodings->AppendString("UTF-16"); 83 encodings->AppendString("UTF-16");
84 encodings->AppendString("UTF-8"); 84 encodings->AppendString("UTF-8");
85 policy->Set(key::kDefaultSearchProviderEnabled, POLICY_LEVEL_MANDATORY, 85 policy->Set(key::kDefaultSearchProviderEnabled,
86 POLICY_SCOPE_USER, base::Value::CreateBooleanValue(true), NULL); 86 POLICY_LEVEL_MANDATORY,
87 POLICY_SCOPE_USER,
88 new base::FundamentalValue(true),
89 NULL);
87 policy->Set(key::kDefaultSearchProviderSearchURL, POLICY_LEVEL_MANDATORY, 90 policy->Set(key::kDefaultSearchProviderSearchURL, POLICY_LEVEL_MANDATORY,
88 POLICY_SCOPE_USER, base::Value::CreateStringValue(kSearchURL), 91 POLICY_SCOPE_USER, base::Value::CreateStringValue(kSearchURL),
89 NULL); 92 NULL);
90 policy->Set(key::kDefaultSearchProviderName, POLICY_LEVEL_MANDATORY, 93 policy->Set(key::kDefaultSearchProviderName, POLICY_LEVEL_MANDATORY,
91 POLICY_SCOPE_USER, base::Value::CreateStringValue(kName), NULL); 94 POLICY_SCOPE_USER, base::Value::CreateStringValue(kName), NULL);
92 policy->Set(key::kDefaultSearchProviderKeyword, POLICY_LEVEL_MANDATORY, 95 policy->Set(key::kDefaultSearchProviderKeyword, POLICY_LEVEL_MANDATORY,
93 POLICY_SCOPE_USER, base::Value::CreateStringValue(kKeyword), 96 POLICY_SCOPE_USER, base::Value::CreateStringValue(kKeyword),
94 NULL); 97 NULL);
95 policy->Set(key::kDefaultSearchProviderSuggestURL, POLICY_LEVEL_MANDATORY, 98 policy->Set(key::kDefaultSearchProviderSuggestURL, POLICY_LEVEL_MANDATORY,
96 POLICY_SCOPE_USER, base::Value::CreateStringValue(kSuggestURL), 99 POLICY_SCOPE_USER, base::Value::CreateStringValue(kSuggestURL),
(...skipping 16 matching lines...) Expand all
113 base::Value::CreateStringValue(kImageParams), NULL); 116 base::Value::CreateStringValue(kImageParams), NULL);
114 policy->Set(key::kDefaultSearchProviderNewTabURL, 117 policy->Set(key::kDefaultSearchProviderNewTabURL,
115 POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER, 118 POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER,
116 base::Value::CreateStringValue(kNewTabURL), NULL); 119 base::Value::CreateStringValue(kNewTabURL), NULL);
117 } 120 }
118 121
119 // Checks that if the policy for default search is valid, i.e. there's a 122 // Checks that if the policy for default search is valid, i.e. there's a
120 // search URL, that all the elements have been given proper defaults. 123 // search URL, that all the elements have been given proper defaults.
121 TEST_F(DefaultSearchPolicyHandlerTest, MinimallyDefined) { 124 TEST_F(DefaultSearchPolicyHandlerTest, MinimallyDefined) {
122 PolicyMap policy; 125 PolicyMap policy;
123 policy.Set(key::kDefaultSearchProviderEnabled, POLICY_LEVEL_MANDATORY, 126 policy.Set(key::kDefaultSearchProviderEnabled,
124 POLICY_SCOPE_USER, base::Value::CreateBooleanValue(true), NULL); 127 POLICY_LEVEL_MANDATORY,
128 POLICY_SCOPE_USER,
129 new base::FundamentalValue(true),
130 NULL);
125 policy.Set(key::kDefaultSearchProviderSearchURL, POLICY_LEVEL_MANDATORY, 131 policy.Set(key::kDefaultSearchProviderSearchURL, POLICY_LEVEL_MANDATORY,
126 POLICY_SCOPE_USER, base::Value::CreateStringValue(kSearchURL), 132 POLICY_SCOPE_USER, base::Value::CreateStringValue(kSearchURL),
127 NULL); 133 NULL);
128 UpdateProviderPolicy(policy); 134 UpdateProviderPolicy(policy);
129 135
130 const base::Value* value = NULL; 136 const base::Value* value = NULL;
131 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderSearchURL, &value)); 137 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderSearchURL, &value));
132 EXPECT_TRUE(base::StringValue(kSearchURL).Equals(value)); 138 EXPECT_TRUE(base::StringValue(kSearchURL).Equals(value));
133 139
134 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderName, &value)); 140 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderName, &value));
(...skipping 155 matching lines...) Expand 10 before | Expand all | Expand 10 after
290 EXPECT_FALSE(store_->GetValue( 296 EXPECT_FALSE(store_->GetValue(
291 prefs::kDefaultSearchProviderImageURLPostParams, NULL)); 297 prefs::kDefaultSearchProviderImageURLPostParams, NULL));
292 EXPECT_FALSE(store_->GetValue(prefs::kDefaultSearchProviderInstantURL, NULL)); 298 EXPECT_FALSE(store_->GetValue(prefs::kDefaultSearchProviderInstantURL, NULL));
293 EXPECT_FALSE(store_->GetValue(prefs::kDefaultSearchProviderNewTabURL, NULL)); 299 EXPECT_FALSE(store_->GetValue(prefs::kDefaultSearchProviderNewTabURL, NULL));
294 } 300 }
295 301
296 // Checks that if the default search policy is invalid, that no elements of the 302 // Checks that if the default search policy is invalid, that no elements of the
297 // default search policy will be present. 303 // default search policy will be present.
298 TEST_F(DefaultSearchPolicyHandlerTest, Disabled) { 304 TEST_F(DefaultSearchPolicyHandlerTest, Disabled) {
299 PolicyMap policy; 305 PolicyMap policy;
300 policy.Set(key::kDefaultSearchProviderEnabled, POLICY_LEVEL_MANDATORY, 306 policy.Set(key::kDefaultSearchProviderEnabled,
301 POLICY_SCOPE_USER, base::Value::CreateBooleanValue(false), NULL); 307 POLICY_LEVEL_MANDATORY,
308 POLICY_SCOPE_USER,
309 new base::FundamentalValue(false),
310 NULL);
302 UpdateProviderPolicy(policy); 311 UpdateProviderPolicy(policy);
303 312
304 const base::Value* value = NULL; 313 const base::Value* value = NULL;
305 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderEnabled, &value)); 314 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderEnabled, &value));
306 base::FundamentalValue expected_enabled(false); 315 base::FundamentalValue expected_enabled(false);
307 EXPECT_TRUE(base::Value::Equals(&expected_enabled, value)); 316 EXPECT_TRUE(base::Value::Equals(&expected_enabled, value));
308 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderSearchURL, &value)); 317 EXPECT_TRUE(store_->GetValue(prefs::kDefaultSearchProviderSearchURL, &value));
309 base::StringValue expected_search_url((std::string())); 318 base::StringValue expected_search_url((std::string()));
310 EXPECT_TRUE(base::Value::Equals(&expected_search_url, value)); 319 EXPECT_TRUE(base::Value::Equals(&expected_search_url, value));
311 } 320 }
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
373 EXPECT_EQ(std::string(), value); 382 EXPECT_EQ(std::string(), value);
374 } 383 }
375 384
376 // Checks that disabling default search is properly reflected the dictionary 385 // Checks that disabling default search is properly reflected the dictionary
377 // pref. 386 // pref.
378 TEST_F(DefaultSearchPolicyHandlerTest, DictionaryPrefDSEDisabled) { 387 TEST_F(DefaultSearchPolicyHandlerTest, DictionaryPrefDSEDisabled) {
379 PolicyMap policy; 388 PolicyMap policy;
380 policy.Set(key::kDefaultSearchProviderEnabled, 389 policy.Set(key::kDefaultSearchProviderEnabled,
381 POLICY_LEVEL_MANDATORY, 390 POLICY_LEVEL_MANDATORY,
382 POLICY_SCOPE_USER, 391 POLICY_SCOPE_USER,
383 base::Value::CreateBooleanValue(false), 392 new base::FundamentalValue(false),
384 NULL); 393 NULL);
385 UpdateProviderPolicy(policy); 394 UpdateProviderPolicy(policy);
386 const base::Value* temp = NULL; 395 const base::Value* temp = NULL;
387 const base::DictionaryValue* dictionary; 396 const base::DictionaryValue* dictionary;
388 EXPECT_TRUE(store_->GetValue(kDefaultSearchProviderData, &temp)); 397 EXPECT_TRUE(store_->GetValue(kDefaultSearchProviderData, &temp));
389 temp->GetAsDictionary(&dictionary); 398 temp->GetAsDictionary(&dictionary);
390 bool disabled = false; 399 bool disabled = false;
391 EXPECT_TRUE(dictionary->GetBoolean(DefaultSearchManager::kDisabledByPolicy, 400 EXPECT_TRUE(dictionary->GetBoolean(DefaultSearchManager::kDisabledByPolicy,
392 &disabled)); 401 &disabled));
393 EXPECT_TRUE(disabled); 402 EXPECT_TRUE(disabled);
394 } 403 }
395 404
396 // Checks that if the policy for default search is valid, i.e. there's a 405 // Checks that if the policy for default search is valid, i.e. there's a
397 // search URL, that all the elements have been given proper defaults. 406 // search URL, that all the elements have been given proper defaults.
398 TEST_F(DefaultSearchPolicyHandlerTest, DictionaryPrefMinimallyDefined) { 407 TEST_F(DefaultSearchPolicyHandlerTest, DictionaryPrefMinimallyDefined) {
399 PolicyMap policy; 408 PolicyMap policy;
400 policy.Set(key::kDefaultSearchProviderEnabled, 409 policy.Set(key::kDefaultSearchProviderEnabled,
401 POLICY_LEVEL_MANDATORY, 410 POLICY_LEVEL_MANDATORY,
402 POLICY_SCOPE_USER, 411 POLICY_SCOPE_USER,
403 base::Value::CreateBooleanValue(true), 412 new base::FundamentalValue(true),
404 NULL); 413 NULL);
405 policy.Set(key::kDefaultSearchProviderSearchURL, 414 policy.Set(key::kDefaultSearchProviderSearchURL,
406 POLICY_LEVEL_MANDATORY, 415 POLICY_LEVEL_MANDATORY,
407 POLICY_SCOPE_USER, 416 POLICY_SCOPE_USER,
408 base::Value::CreateStringValue(kSearchURL), 417 base::Value::CreateStringValue(kSearchURL),
409 NULL); 418 NULL);
410 UpdateProviderPolicy(policy); 419 UpdateProviderPolicy(policy);
411 420
412 const base::Value* temp = NULL; 421 const base::Value* temp = NULL;
413 const base::DictionaryValue* dictionary; 422 const base::DictionaryValue* dictionary;
(...skipping 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
455 EXPECT_EQ(std::string(), value); 464 EXPECT_EQ(std::string(), value);
456 } 465 }
457 466
458 // Checks that setting a file URL as the default search is reflected properly in 467 // Checks that setting a file URL as the default search is reflected properly in
459 // the dictionary pref. 468 // the dictionary pref.
460 TEST_F(DefaultSearchPolicyHandlerTest, DictionaryPrefFileURL) { 469 TEST_F(DefaultSearchPolicyHandlerTest, DictionaryPrefFileURL) {
461 PolicyMap policy; 470 PolicyMap policy;
462 policy.Set(key::kDefaultSearchProviderEnabled, 471 policy.Set(key::kDefaultSearchProviderEnabled,
463 POLICY_LEVEL_MANDATORY, 472 POLICY_LEVEL_MANDATORY,
464 POLICY_SCOPE_USER, 473 POLICY_SCOPE_USER,
465 base::Value::CreateBooleanValue(true), 474 new base::FundamentalValue(true),
466 NULL); 475 NULL);
467 policy.Set(key::kDefaultSearchProviderSearchURL, 476 policy.Set(key::kDefaultSearchProviderSearchURL,
468 POLICY_LEVEL_MANDATORY, 477 POLICY_LEVEL_MANDATORY,
469 POLICY_SCOPE_USER, 478 POLICY_SCOPE_USER,
470 base::Value::CreateStringValue(kFileSearchURL), 479 base::Value::CreateStringValue(kFileSearchURL),
471 NULL); 480 NULL);
472 UpdateProviderPolicy(policy); 481 UpdateProviderPolicy(policy);
473 482
474 const base::Value* temp = NULL; 483 const base::Value* temp = NULL;
475 const base::DictionaryValue* dictionary; 484 const base::DictionaryValue* dictionary;
476 std::string value; 485 std::string value;
477 486
478 EXPECT_TRUE(store_->GetValue(kDefaultSearchProviderData, &temp)); 487 EXPECT_TRUE(store_->GetValue(kDefaultSearchProviderData, &temp));
479 temp->GetAsDictionary(&dictionary); 488 temp->GetAsDictionary(&dictionary);
480 489
481 EXPECT_TRUE(dictionary->GetString(DefaultSearchManager::kURL, &value)); 490 EXPECT_TRUE(dictionary->GetString(DefaultSearchManager::kURL, &value));
482 EXPECT_EQ(kFileSearchURL, value); 491 EXPECT_EQ(kFileSearchURL, value);
483 EXPECT_TRUE(dictionary->GetString(DefaultSearchManager::kShortName, &value)); 492 EXPECT_TRUE(dictionary->GetString(DefaultSearchManager::kShortName, &value));
484 EXPECT_EQ("_", value); 493 EXPECT_EQ("_", value);
485 EXPECT_TRUE(dictionary->GetString(DefaultSearchManager::kKeyword, &value)); 494 EXPECT_TRUE(dictionary->GetString(DefaultSearchManager::kKeyword, &value));
486 EXPECT_EQ("_", value); 495 EXPECT_EQ("_", value);
487 } 496 }
488 } // namespace policy 497 } // namespace policy
OLDNEW
« no previous file with comments | « components/policy/core/common/schema_map_unittest.cc ('k') | tools/json_schema_compiler/test/any_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698