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

Unified Diff: chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc

Issue 835683003: [NOT FOR REVIEW] only for running build bots only Testing Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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
« no previous file with comments | « chrome/browser/supervised_user/supervised_user_settings_service.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc
diff --git a/chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc b/chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc
index c3c8836a304c3bf0c567c9f55e95f1e216050dc8..f3a43d5c4d1460381b286f9f161ede01b4ddac3c 100644
--- a/chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc
+++ b/chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc
@@ -70,13 +70,15 @@ class SupervisedUserSettingsServiceTest : public ::testing::Test {
new syncer::SyncChangeProcessorWrapperForTest(sync_processor_.get()));
}
- void StartSyncing(const syncer::SyncDataList& initial_sync_data) {
+ syncer::SyncMergeResult StartSyncing(
+ const syncer::SyncDataList& initial_sync_data) {
scoped_ptr<syncer::SyncErrorFactory> error_handler(
new MockSyncErrorFactory(syncer::SUPERVISED_USER_SETTINGS));
syncer::SyncMergeResult result = settings_service_.MergeDataAndStartSyncing(
syncer::SUPERVISED_USER_SETTINGS, initial_sync_data,
CreateSyncProcessor(), std::move(error_handler));
EXPECT_FALSE(result.error().IsSet());
+ return result;
}
void UploadSplitItem(const std::string& key, const std::string& value) {
@@ -97,7 +99,7 @@ class SupervisedUserSettingsServiceTest : public ::testing::Test {
void VerifySyncDataItem(syncer::SyncData sync_data) {
const sync_pb::ManagedUserSettingSpecifics& supervised_user_setting =
sync_data.GetSpecifics().managed_user_setting();
- base::Value* expected_value = NULL;
+ base::Value* expected_value = nullptr;
if (supervised_user_setting.name() == kAtomicItemName) {
expected_value = atomic_setting_value_.get();
} else {
@@ -150,7 +152,7 @@ class SupervisedUserSettingsServiceTest : public ::testing::Test {
TEST_F(SupervisedUserSettingsServiceTest, ProcessAtomicSetting) {
StartSyncing(syncer::SyncDataList());
ASSERT_TRUE(settings_);
- const base::Value* value = NULL;
+ const base::Value* value = nullptr;
EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
settings_.reset();
@@ -173,7 +175,7 @@ TEST_F(SupervisedUserSettingsServiceTest, ProcessAtomicSetting) {
TEST_F(SupervisedUserSettingsServiceTest, ProcessSplitSetting) {
StartSyncing(syncer::SyncDataList());
ASSERT_TRUE(settings_);
- const base::Value* value = NULL;
+ const base::Value* value = nullptr;
EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
base::DictionaryValue dict;
@@ -197,13 +199,75 @@ TEST_F(SupervisedUserSettingsServiceTest, ProcessSplitSetting) {
EXPECT_FALSE(error.IsSet()) << error.ToString();
ASSERT_TRUE(settings_);
ASSERT_TRUE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
- const base::DictionaryValue* dict_value = NULL;
+ const base::DictionaryValue* dict_value = nullptr;
ASSERT_TRUE(value->GetAsDictionary(&dict_value));
EXPECT_TRUE(dict_value->Equals(&dict));
}
+TEST_F(SupervisedUserSettingsServiceTest, Merge) {
+ syncer::SyncMergeResult result = StartSyncing(syncer::SyncDataList());
+ EXPECT_EQ(0, result.num_items_added());
+ EXPECT_EQ(0, result.num_items_deleted());
+ EXPECT_EQ(0, result.num_items_modified());
+ EXPECT_EQ(0, result.num_items_before_association());
+ EXPECT_EQ(0, result.num_items_after_association());
+
+ ASSERT_TRUE(settings_);
+ const base::Value* value = nullptr;
+ EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
+
+ base::DictionaryValue dict;
+ dict.SetString("foo", "bar");
+ dict.SetBoolean("awesomesauce", true);
+ dict.SetInteger("eaudecologne", 4711);
+
+ settings_.reset();
+ syncer::SyncDataList sync_data;
+ sync_data.push_back(SupervisedUserSettingsService::CreateSyncDataForSetting(
+ kSettingsName, base::StringValue(kSettingsValue)));
+ for (base::DictionaryValue::Iterator it(dict); !it.IsAtEnd(); it.Advance()) {
+ sync_data.push_back(SupervisedUserSettingsService::CreateSyncDataForSetting(
+ SupervisedUserSettingsService::MakeSplitSettingKey(kSettingsName,
+ it.key()),
+ it.value()));
+ }
+
+ result = StartSyncing(sync_data);
+ EXPECT_EQ(4, result.num_items_added());
+ EXPECT_EQ(0, result.num_items_deleted());
+ EXPECT_EQ(0, result.num_items_modified());
+ EXPECT_EQ(0, result.num_items_before_association());
+ EXPECT_EQ(4, result.num_items_after_association());
+
+ settings_service_.StopSyncing(syncer::SUPERVISED_USER_SETTINGS);
+ sync_data.clear();
+ // Adding split Item in the queue.
+ UploadSplitItem("blurp", "baz");
+ // Adding atomic Item in the queue.
+ UploadAtomicItem("hurdle");
+
+ dict.Clear();
+ dict.SetString("foo", "burp");
+ dict.SetString("item", "first");
+
+ for (base::DictionaryValue::Iterator it(dict); !it.IsAtEnd(); it.Advance()) {
+ sync_data.push_back(SupervisedUserSettingsService::CreateSyncDataForSetting(
+ SupervisedUserSettingsService::MakeSplitSettingKey(kSettingsName,
+ it.key()),
+ it.value()));
+ }
+
+ LOG(ERROR) << "Deepak calling StartSyncing:";
+ result = StartSyncing(sync_data);
+ EXPECT_EQ(3, result.num_items_added());
+ EXPECT_EQ(3, result.num_items_deleted());
+ EXPECT_EQ(1, result.num_items_modified());
+ EXPECT_EQ(4, result.num_items_before_association());
+ EXPECT_EQ(4, result.num_items_after_association());
+}
+
TEST_F(SupervisedUserSettingsServiceTest, SetLocalSetting) {
- const base::Value* value = NULL;
+ const base::Value* value = nullptr;
EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSettingsName, &value));
settings_.reset();
@@ -284,7 +348,7 @@ TEST_F(SupervisedUserSettingsServiceTest, UploadItem) {
VerifySyncDataItem(sync_data_item);
// The uploaded items should not show up as settings.
- const base::Value* value = NULL;
+ const base::Value* value = nullptr;
EXPECT_FALSE(settings_->GetWithoutPathExpansion(kAtomicItemName, &value));
EXPECT_FALSE(settings_->GetWithoutPathExpansion(kSplitItemName, &value));
« no previous file with comments | « chrome/browser/supervised_user/supervised_user_settings_service.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698