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

Side by Side Diff: chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc

Issue 2664753002: Remove base::StringValue (Closed)
Patch Set: Rebase Created 3 years, 9 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
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 "chrome/browser/supervised_user/supervised_user_settings_service.h" 5 #include "chrome/browser/supervised_user/supervised_user_settings_service.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/callback.h" 10 #include "base/callback.h"
(...skipping 67 matching lines...) Expand 10 before | Expand all | Expand 10 after
78 syncer::SUPERVISED_USER_SETTINGS, initial_sync_data, 78 syncer::SUPERVISED_USER_SETTINGS, initial_sync_data,
79 CreateSyncProcessor(), std::move(error_handler)); 79 CreateSyncProcessor(), std::move(error_handler));
80 EXPECT_FALSE(result.error().IsSet()); 80 EXPECT_FALSE(result.error().IsSet());
81 return result; 81 return result;
82 } 82 }
83 83
84 void UploadSplitItem(const std::string& key, const std::string& value) { 84 void UploadSplitItem(const std::string& key, const std::string& value) {
85 split_items_.SetStringWithoutPathExpansion(key, value); 85 split_items_.SetStringWithoutPathExpansion(key, value);
86 settings_service_.UploadItem( 86 settings_service_.UploadItem(
87 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName, key), 87 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName, key),
88 std::unique_ptr<base::Value>(new base::StringValue(value))); 88 std::unique_ptr<base::Value>(new base::Value(value)));
89 } 89 }
90 90
91 void UploadAtomicItem(const std::string& value) { 91 void UploadAtomicItem(const std::string& value) {
92 atomic_setting_value_.reset(new base::StringValue(value)); 92 atomic_setting_value_.reset(new base::Value(value));
93 settings_service_.UploadItem( 93 settings_service_.UploadItem(
94 kAtomicItemName, 94 kAtomicItemName, std::unique_ptr<base::Value>(new base::Value(value)));
95 std::unique_ptr<base::Value>(new base::StringValue(value)));
96 } 95 }
97 96
98 void VerifySyncDataItem(syncer::SyncData sync_data) { 97 void VerifySyncDataItem(syncer::SyncData sync_data) {
99 const sync_pb::ManagedUserSettingSpecifics& supervised_user_setting = 98 const sync_pb::ManagedUserSettingSpecifics& supervised_user_setting =
100 sync_data.GetSpecifics().managed_user_setting(); 99 sync_data.GetSpecifics().managed_user_setting();
101 base::Value* expected_value = nullptr; 100 base::Value* expected_value = nullptr;
102 if (supervised_user_setting.name() == kAtomicItemName) { 101 if (supervised_user_setting.name() == kAtomicItemName) {
103 expected_value = atomic_setting_value_.get(); 102 expected_value = atomic_setting_value_.get();
104 } else { 103 } else {
105 EXPECT_TRUE(base::StartsWith(supervised_user_setting.name(), 104 EXPECT_TRUE(base::StartsWith(supervised_user_setting.name(),
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
151 150
152 TEST_F(SupervisedUserSettingsServiceTest, ProcessAtomicSetting) { 151 TEST_F(SupervisedUserSettingsServiceTest, ProcessAtomicSetting) {
153 StartSyncing(syncer::SyncDataList()); 152 StartSyncing(syncer::SyncDataList());
154 ASSERT_TRUE(settings_); 153 ASSERT_TRUE(settings_);
155 const base::Value* value = nullptr; 154 const base::Value* value = nullptr;
156 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); 155 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
157 156
158 settings_.reset(); 157 settings_.reset();
159 syncer::SyncData data = 158 syncer::SyncData data =
160 SupervisedUserSettingsService::CreateSyncDataForSetting( 159 SupervisedUserSettingsService::CreateSyncDataForSetting(
161 kSettingsName, base::StringValue(kSettingsValue)); 160 kSettingsName, base::Value(kSettingsValue));
162 syncer::SyncChangeList change_list; 161 syncer::SyncChangeList change_list;
163 change_list.push_back( 162 change_list.push_back(
164 syncer::SyncChange(FROM_HERE, syncer::SyncChange::ACTION_ADD, data)); 163 syncer::SyncChange(FROM_HERE, syncer::SyncChange::ACTION_ADD, data));
165 syncer::SyncError error = 164 syncer::SyncError error =
166 settings_service_.ProcessSyncChanges(FROM_HERE, change_list); 165 settings_service_.ProcessSyncChanges(FROM_HERE, change_list);
167 EXPECT_FALSE(error.IsSet()) << error.ToString(); 166 EXPECT_FALSE(error.IsSet()) << error.ToString();
168 ASSERT_TRUE(settings_); 167 ASSERT_TRUE(settings_);
169 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); 168 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
170 std::string string_value; 169 std::string string_value;
171 EXPECT_TRUE(value->GetAsString(&string_value)); 170 EXPECT_TRUE(value->GetAsString(&string_value));
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 ASSERT_TRUE(settings_); 214 ASSERT_TRUE(settings_);
216 const base::Value* value = nullptr; 215 const base::Value* value = nullptr;
217 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); 216 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
218 217
219 settings_.reset(); 218 settings_.reset();
220 219
221 { 220 {
222 syncer::SyncDataList sync_data; 221 syncer::SyncDataList sync_data;
223 // Adding 1 Atomic entry. 222 // Adding 1 Atomic entry.
224 sync_data.push_back(SupervisedUserSettingsService::CreateSyncDataForSetting( 223 sync_data.push_back(SupervisedUserSettingsService::CreateSyncDataForSetting(
225 kSettingsName, base::StringValue(kSettingsValue))); 224 kSettingsName, base::Value(kSettingsValue)));
226 // Adding 2 SplitSettings from dictionary. 225 // Adding 2 SplitSettings from dictionary.
227 base::DictionaryValue dict; 226 base::DictionaryValue dict;
228 dict.SetString("foo", "bar"); 227 dict.SetString("foo", "bar");
229 dict.SetInteger("eaudecologne", 4711); 228 dict.SetInteger("eaudecologne", 4711);
230 for (base::DictionaryValue::Iterator it(dict); !it.IsAtEnd(); 229 for (base::DictionaryValue::Iterator it(dict); !it.IsAtEnd();
231 it.Advance()) { 230 it.Advance()) {
232 sync_data.push_back( 231 sync_data.push_back(
233 SupervisedUserSettingsService::CreateSyncDataForSetting( 232 SupervisedUserSettingsService::CreateSyncDataForSetting(
234 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName, 233 SupervisedUserSettingsService::MakeSplitSettingKey(kSplitItemName,
235 it.key()), 234 it.key()),
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
274 } 273 }
275 } 274 }
276 275
277 TEST_F(SupervisedUserSettingsServiceTest, SetLocalSetting) { 276 TEST_F(SupervisedUserSettingsServiceTest, SetLocalSetting) {
278 const base::Value* value = nullptr; 277 const base::Value* value = nullptr;
279 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); 278 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
280 279
281 settings_.reset(); 280 settings_.reset();
282 settings_service_.SetLocalSetting( 281 settings_service_.SetLocalSetting(
283 kSettingsName, 282 kSettingsName,
284 std::unique_ptr<base::Value>(new base::StringValue(kSettingsValue))); 283 std::unique_ptr<base::Value>(new base::Value(kSettingsValue)));
285 ASSERT_TRUE(settings_); 284 ASSERT_TRUE(settings_);
286 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value)); 285 ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
287 std::string string_value; 286 std::string string_value;
288 EXPECT_TRUE(value->GetAsString(&string_value)); 287 EXPECT_TRUE(value->GetAsString(&string_value));
289 EXPECT_EQ(kSettingsValue, string_value); 288 EXPECT_EQ(kSettingsValue, string_value);
290 } 289 }
291 290
292 TEST_F(SupervisedUserSettingsServiceTest, UploadItem) { 291 TEST_F(SupervisedUserSettingsServiceTest, UploadItem) {
293 UploadSplitItem("foo", "bar"); 292 UploadSplitItem("foo", "bar");
294 UploadSplitItem("blurp", "baz"); 293 UploadSplitItem("blurp", "baz");
(...skipping 63 matching lines...) Expand 10 before | Expand all | Expand 10 after
358 // The uploaded items should not show up as settings. 357 // The uploaded items should not show up as settings.
359 const base::Value* value = nullptr; 358 const base::Value* value = nullptr;
360 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kAtomicItemName, &value)); 359 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kAtomicItemName, &value));
361 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSplitItemName, &value)); 360 EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSplitItemName, &value));
362 361
363 // Restarting sync should not create any new changes. 362 // Restarting sync should not create any new changes.
364 settings_service_.StopSyncing(syncer::SUPERVISED_USER_SETTINGS); 363 settings_service_.StopSyncing(syncer::SUPERVISED_USER_SETTINGS);
365 StartSyncing(sync_data); 364 StartSyncing(sync_data);
366 ASSERT_EQ(0u, sync_processor_->changes().size()); 365 ASSERT_EQ(0u, sync_processor_->changes().size());
367 } 366 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698