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

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

Issue 335833003: Rename "managed (mode|user)" to "supervised user" (part 2) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: review comments (+ a few other cleanups) Created 6 years, 6 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 2013 The Chromium Authors. All rights reserved. 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 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 <string> 5 #include <string>
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/prefs/scoped_user_pref_update.h" 9 #include "base/prefs/scoped_user_pref_update.h"
10 #include "base/strings/stringprintf.h" 10 #include "base/strings/stringprintf.h"
11 #include "base/strings/utf_string_conversions.h" 11 #include "base/strings/utf_string_conversions.h"
12 #include "base/threading/sequenced_worker_pool.h" 12 #include "base/threading/sequenced_worker_pool.h"
13 #include "chrome/browser/managed_mode/managed_user_sync_service.h" 13 #include "chrome/browser/supervised_user/supervised_user_sync_service.h"
14 #include "chrome/browser/managed_mode/managed_user_sync_service_factory.h" 14 #include "chrome/browser/supervised_user/supervised_user_sync_service_factory.h"
15 #include "chrome/common/pref_names.h" 15 #include "chrome/common/pref_names.h"
16 #include "chrome/test/base/testing_profile.h" 16 #include "chrome/test/base/testing_profile.h"
17 #include "sync/api/attachments/attachment_id.h" 17 #include "sync/api/attachments/attachment_id.h"
18 #include "sync/api/attachments/attachment_service_proxy_for_test.h" 18 #include "sync/api/attachments/attachment_service_proxy_for_test.h"
19 #include "sync/api/sync_change.h" 19 #include "sync/api/sync_change.h"
20 #include "sync/api/sync_error_factory_mock.h" 20 #include "sync/api/sync_error_factory_mock.h"
21 #include "sync/protocol/sync.pb.h" 21 #include "sync/protocol/sync.pb.h"
22 #include "testing/gtest/include/gtest/gtest.h" 22 #include "testing/gtest/include/gtest/gtest.h"
23 23
24 #if defined(OS_CHROMEOS) 24 #if defined(OS_CHROMEOS)
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 69
70 SyncChange MockChangeProcessor::GetChange(const std::string& id) const { 70 SyncChange MockChangeProcessor::GetChange(const std::string& id) const {
71 for (SyncChangeList::const_iterator it = change_list_.begin(); 71 for (SyncChangeList::const_iterator it = change_list_.begin();
72 it != change_list_.end(); ++it) { 72 it != change_list_.end(); ++it) {
73 if (it->sync_data().GetSpecifics().managed_user().id() == id) 73 if (it->sync_data().GetSpecifics().managed_user().id() == id)
74 return *it; 74 return *it;
75 } 75 }
76 return SyncChange(); 76 return SyncChange();
77 } 77 }
78 78
79 // Callback for ManagedUserSyncService::GetManagedUsersAsync(). 79 // Callback for SupervisedUserSyncService::GetSupervisedUsersAsync().
80 void GetManagedUsersCallback(const base::DictionaryValue** dict, 80 void GetSupervisedUsersCallback(const base::DictionaryValue** dict,
81 const base::DictionaryValue* managed_users) { 81 const base::DictionaryValue* supervised_users) {
82 *dict = managed_users; 82 *dict = supervised_users;
83 } 83 }
84 84
85 } // namespace 85 } // namespace
86 86
87 class ManagedUserSyncServiceTest : public ::testing::Test { 87 class SupervisedUserSyncServiceTest : public ::testing::Test {
88 public: 88 public:
89 ManagedUserSyncServiceTest(); 89 SupervisedUserSyncServiceTest();
90 virtual ~ManagedUserSyncServiceTest(); 90 virtual ~SupervisedUserSyncServiceTest();
91 91
92 protected: 92 protected:
93 scoped_ptr<SyncChangeProcessor> CreateChangeProcessor(); 93 scoped_ptr<SyncChangeProcessor> CreateChangeProcessor();
94 scoped_ptr<SyncErrorFactory> CreateErrorFactory(); 94 scoped_ptr<SyncErrorFactory> CreateErrorFactory();
95 SyncData CreateRemoteData(const std::string& id, 95 SyncData CreateRemoteData(const std::string& id,
96 const std::string& name, 96 const std::string& name,
97 const std::string& avatar); 97 const std::string& avatar);
98 98
99 PrefService* prefs() { return profile_.GetPrefs(); } 99 PrefService* prefs() { return profile_.GetPrefs(); }
100 ManagedUserSyncService* service() { return service_; } 100 SupervisedUserSyncService* service() { return service_; }
101 MockChangeProcessor* change_processor() { return change_processor_; } 101 MockChangeProcessor* change_processor() { return change_processor_; }
102 102
103 private: 103 private:
104 base::MessageLoop message_loop; 104 base::MessageLoop message_loop;
105 TestingProfile profile_; 105 TestingProfile profile_;
106 ManagedUserSyncService* service_; 106 SupervisedUserSyncService* service_;
107 107
108 // Owned by the ManagedUserSyncService. 108 // Owned by the SupervisedUserSyncService.
109 MockChangeProcessor* change_processor_; 109 MockChangeProcessor* change_processor_;
110 110
111 // A unique ID for creating "remote" Sync data. 111 // A unique ID for creating "remote" Sync data.
112 int64 sync_data_id_; 112 int64 sync_data_id_;
113 }; 113 };
114 114
115 ManagedUserSyncServiceTest::ManagedUserSyncServiceTest() 115 SupervisedUserSyncServiceTest::SupervisedUserSyncServiceTest()
116 : change_processor_(NULL), 116 : change_processor_(NULL),
117 sync_data_id_(0) { 117 sync_data_id_(0) {
118 service_ = ManagedUserSyncServiceFactory::GetForProfile(&profile_); 118 service_ = SupervisedUserSyncServiceFactory::GetForProfile(&profile_);
119 } 119 }
120 120
121 ManagedUserSyncServiceTest::~ManagedUserSyncServiceTest() {} 121 SupervisedUserSyncServiceTest::~SupervisedUserSyncServiceTest() {}
122 122
123 scoped_ptr<SyncChangeProcessor> 123 scoped_ptr<SyncChangeProcessor>
124 ManagedUserSyncServiceTest::CreateChangeProcessor() { 124 SupervisedUserSyncServiceTest::CreateChangeProcessor() {
125 EXPECT_FALSE(change_processor_); 125 EXPECT_FALSE(change_processor_);
126 change_processor_ = new MockChangeProcessor(); 126 change_processor_ = new MockChangeProcessor();
127 return scoped_ptr<SyncChangeProcessor>(change_processor_); 127 return scoped_ptr<SyncChangeProcessor>(change_processor_);
128 } 128 }
129 129
130 scoped_ptr<SyncErrorFactory> 130 scoped_ptr<SyncErrorFactory>
131 ManagedUserSyncServiceTest::CreateErrorFactory() { 131 SupervisedUserSyncServiceTest::CreateErrorFactory() {
132 return scoped_ptr<SyncErrorFactory>(new syncer::SyncErrorFactoryMock()); 132 return scoped_ptr<SyncErrorFactory>(new syncer::SyncErrorFactoryMock());
133 } 133 }
134 134
135 SyncData ManagedUserSyncServiceTest::CreateRemoteData( 135 SyncData SupervisedUserSyncServiceTest::CreateRemoteData(
136 const std::string& id, 136 const std::string& id,
137 const std::string& name, 137 const std::string& name,
138 const std::string& chrome_avatar) { 138 const std::string& chrome_avatar) {
139 ::sync_pb::EntitySpecifics specifics; 139 ::sync_pb::EntitySpecifics specifics;
140 specifics.mutable_managed_user()->set_id(id); 140 specifics.mutable_managed_user()->set_id(id);
141 specifics.mutable_managed_user()->set_name(name); 141 specifics.mutable_managed_user()->set_name(name);
142 specifics.mutable_managed_user()->set_acknowledged(true); 142 specifics.mutable_managed_user()->set_acknowledged(true);
143 if (!chrome_avatar.empty()) 143 if (!chrome_avatar.empty())
144 specifics.mutable_managed_user()->set_chrome_avatar(chrome_avatar); 144 specifics.mutable_managed_user()->set_chrome_avatar(chrome_avatar);
145 145
146 return SyncData::CreateRemoteData( 146 return SyncData::CreateRemoteData(
147 ++sync_data_id_, 147 ++sync_data_id_,
148 specifics, 148 specifics,
149 base::Time(), 149 base::Time(),
150 syncer::AttachmentIdList(), 150 syncer::AttachmentIdList(),
151 syncer::AttachmentServiceProxyForTest::Create()); 151 syncer::AttachmentServiceProxyForTest::Create());
152 } 152 }
153 153
154 TEST_F(ManagedUserSyncServiceTest, MergeEmpty) { 154 TEST_F(SupervisedUserSyncServiceTest, MergeEmpty) {
155 SyncMergeResult result = 155 SyncMergeResult result =
156 service()->MergeDataAndStartSyncing(SUPERVISED_USERS, 156 service()->MergeDataAndStartSyncing(SUPERVISED_USERS,
157 SyncDataList(), 157 SyncDataList(),
158 CreateChangeProcessor(), 158 CreateChangeProcessor(),
159 CreateErrorFactory()); 159 CreateErrorFactory());
160 EXPECT_FALSE(result.error().IsSet()); 160 EXPECT_FALSE(result.error().IsSet());
161 EXPECT_EQ(0, result.num_items_added()); 161 EXPECT_EQ(0, result.num_items_added());
162 EXPECT_EQ(0, result.num_items_modified()); 162 EXPECT_EQ(0, result.num_items_modified());
163 EXPECT_EQ(0, result.num_items_deleted()); 163 EXPECT_EQ(0, result.num_items_deleted());
164 EXPECT_EQ(0, result.num_items_before_association()); 164 EXPECT_EQ(0, result.num_items_before_association());
165 EXPECT_EQ(0, result.num_items_after_association()); 165 EXPECT_EQ(0, result.num_items_after_association());
166 EXPECT_EQ(0u, service()->GetManagedUsers()->size()); 166 EXPECT_EQ(0u, service()->GetSupervisedUsers()->size());
167 EXPECT_EQ(0u, change_processor()->changes().size()); 167 EXPECT_EQ(0u, change_processor()->changes().size());
168 168
169 service()->StopSyncing(SUPERVISED_USERS); 169 service()->StopSyncing(SUPERVISED_USERS);
170 service()->Shutdown(); 170 service()->Shutdown();
171 } 171 }
172 172
173 TEST_F(ManagedUserSyncServiceTest, MergeExisting) { 173 TEST_F(SupervisedUserSyncServiceTest, MergeExisting) {
174 const char kNameKey[] = "name"; 174 const char kNameKey[] = "name";
175 const char kAcknowledgedKey[] = "acknowledged"; 175 const char kAcknowledgedKey[] = "acknowledged";
176 const char kChromeAvatarKey[] = "chromeAvatar"; 176 const char kChromeAvatarKey[] = "chromeAvatar";
177 177
178 const char kUserId1[] = "aaaaa"; 178 const char kUserId1[] = "aaaaa";
179 const char kUserId2[] = "bbbbb"; 179 const char kUserId2[] = "bbbbb";
180 const char kUserId3[] = "ccccc"; 180 const char kUserId3[] = "ccccc";
181 const char kUserId4[] = "ddddd"; 181 const char kUserId4[] = "ddddd";
182 const char kName1[] = "Anchor"; 182 const char kName1[] = "Anchor";
183 const char kName2[] = "Buzz"; 183 const char kName2[] = "Buzz";
184 const char kName3[] = "Crush"; 184 const char kName3[] = "Crush";
185 const char kName4[] = "Dory"; 185 const char kName4[] = "Dory";
186 const char kAvatar1[] = ""; 186 const char kAvatar1[] = "";
187 #if defined(OS_CHROMEOS) 187 #if defined(OS_CHROMEOS)
188 const char kAvatar2[] = "chromeos-avatar-index:0"; 188 const char kAvatar2[] = "chromeos-avatar-index:0";
189 const char kAvatar3[] = "chromeos-avatar-index:20"; 189 const char kAvatar3[] = "chromeos-avatar-index:20";
190 #else 190 #else
191 const char kAvatar2[] = "chrome-avatar-index:0"; 191 const char kAvatar2[] = "chrome-avatar-index:0";
192 const char kAvatar3[] = "chrome-avatar-index:20"; 192 const char kAvatar3[] = "chrome-avatar-index:20";
193 #endif 193 #endif
194 const char kAvatar4[] = ""; 194 const char kAvatar4[] = "";
195 { 195 {
196 DictionaryPrefUpdate update(prefs(), prefs::kSupervisedUsers); 196 DictionaryPrefUpdate update(prefs(), prefs::kSupervisedUsers);
197 base::DictionaryValue* managed_users = update.Get(); 197 base::DictionaryValue* supervised_users = update.Get();
198 base::DictionaryValue* dict = new base::DictionaryValue; 198 base::DictionaryValue* dict = new base::DictionaryValue;
199 dict->SetString(kNameKey, kName1); 199 dict->SetString(kNameKey, kName1);
200 managed_users->Set(kUserId1, dict); 200 supervised_users->Set(kUserId1, dict);
201 dict = new base::DictionaryValue; 201 dict = new base::DictionaryValue;
202 dict->SetString(kNameKey, kName2); 202 dict->SetString(kNameKey, kName2);
203 dict->SetBoolean(kAcknowledgedKey, true); 203 dict->SetBoolean(kAcknowledgedKey, true);
204 managed_users->Set(kUserId2, dict); 204 supervised_users->Set(kUserId2, dict);
205 } 205 }
206 206
207 const base::DictionaryValue* async_managed_users = NULL; 207 const base::DictionaryValue* async_supervised_users = NULL;
208 service()->GetManagedUsersAsync( 208 service()->GetSupervisedUsersAsync(
209 base::Bind(&GetManagedUsersCallback, &async_managed_users)); 209 base::Bind(&GetSupervisedUsersCallback, &async_supervised_users));
210 210
211 SyncDataList initial_sync_data; 211 SyncDataList initial_sync_data;
212 initial_sync_data.push_back(CreateRemoteData(kUserId2, kName2, kAvatar2)); 212 initial_sync_data.push_back(CreateRemoteData(kUserId2, kName2, kAvatar2));
213 initial_sync_data.push_back(CreateRemoteData(kUserId3, kName3, kAvatar3)); 213 initial_sync_data.push_back(CreateRemoteData(kUserId3, kName3, kAvatar3));
214 initial_sync_data.push_back(CreateRemoteData(kUserId4, kName4, kAvatar4)); 214 initial_sync_data.push_back(CreateRemoteData(kUserId4, kName4, kAvatar4));
215 215
216 SyncMergeResult result = 216 SyncMergeResult result =
217 service()->MergeDataAndStartSyncing(SUPERVISED_USERS, 217 service()->MergeDataAndStartSyncing(SUPERVISED_USERS,
218 initial_sync_data, 218 initial_sync_data,
219 CreateChangeProcessor(), 219 CreateChangeProcessor(),
220 CreateErrorFactory()); 220 CreateErrorFactory());
221 EXPECT_FALSE(result.error().IsSet()); 221 EXPECT_FALSE(result.error().IsSet());
222 EXPECT_EQ(2, result.num_items_added()); 222 EXPECT_EQ(2, result.num_items_added());
223 EXPECT_EQ(1, result.num_items_modified()); 223 EXPECT_EQ(1, result.num_items_modified());
224 EXPECT_EQ(0, result.num_items_deleted()); 224 EXPECT_EQ(0, result.num_items_deleted());
225 EXPECT_EQ(2, result.num_items_before_association()); 225 EXPECT_EQ(2, result.num_items_before_association());
226 EXPECT_EQ(4, result.num_items_after_association()); 226 EXPECT_EQ(4, result.num_items_after_association());
227 227
228 const base::DictionaryValue* managed_users = service()->GetManagedUsers(); 228 const base::DictionaryValue* supervised_users =
229 EXPECT_EQ(4u, managed_users->size()); 229 service()->GetSupervisedUsers();
230 EXPECT_TRUE(async_managed_users); 230 EXPECT_EQ(4u, supervised_users->size());
231 EXPECT_TRUE(managed_users->Equals(async_managed_users)); 231 EXPECT_TRUE(async_supervised_users);
232 EXPECT_TRUE(supervised_users->Equals(async_supervised_users));
232 233
233 { 234 {
234 const base::DictionaryValue* managed_user = NULL; 235 const base::DictionaryValue* supervised_user = NULL;
235 ASSERT_TRUE(managed_users->GetDictionary(kUserId2, &managed_user)); 236 ASSERT_TRUE(supervised_users->GetDictionary(kUserId2, &supervised_user));
236 ASSERT_TRUE(managed_user); 237 ASSERT_TRUE(supervised_user);
237 std::string name; 238 std::string name;
238 EXPECT_TRUE(managed_user->GetString(kNameKey, &name)); 239 EXPECT_TRUE(supervised_user->GetString(kNameKey, &name));
239 EXPECT_EQ(kName2, name); 240 EXPECT_EQ(kName2, name);
240 bool acknowledged = false; 241 bool acknowledged = false;
241 EXPECT_TRUE(managed_user->GetBoolean(kAcknowledgedKey, &acknowledged)); 242 EXPECT_TRUE(supervised_user->GetBoolean(kAcknowledgedKey, &acknowledged));
242 EXPECT_TRUE(acknowledged); 243 EXPECT_TRUE(acknowledged);
243 std::string avatar; 244 std::string avatar;
244 EXPECT_TRUE(managed_user->GetString(kChromeAvatarKey, &avatar)); 245 EXPECT_TRUE(supervised_user->GetString(kChromeAvatarKey, &avatar));
245 EXPECT_EQ(kAvatar2, avatar); 246 EXPECT_EQ(kAvatar2, avatar);
246 } 247 }
247 { 248 {
248 const base::DictionaryValue* managed_user = NULL; 249 const base::DictionaryValue* supervised_user = NULL;
249 ASSERT_TRUE(managed_users->GetDictionary(kUserId3, &managed_user)); 250 ASSERT_TRUE(supervised_users->GetDictionary(kUserId3, &supervised_user));
250 ASSERT_TRUE(managed_user); 251 ASSERT_TRUE(supervised_user);
251 std::string name; 252 std::string name;
252 EXPECT_TRUE(managed_user->GetString(kNameKey, &name)); 253 EXPECT_TRUE(supervised_user->GetString(kNameKey, &name));
253 EXPECT_EQ(kName3, name); 254 EXPECT_EQ(kName3, name);
254 bool acknowledged = false; 255 bool acknowledged = false;
255 EXPECT_TRUE(managed_user->GetBoolean(kAcknowledgedKey, &acknowledged)); 256 EXPECT_TRUE(supervised_user->GetBoolean(kAcknowledgedKey, &acknowledged));
256 EXPECT_TRUE(acknowledged); 257 EXPECT_TRUE(acknowledged);
257 std::string avatar; 258 std::string avatar;
258 EXPECT_TRUE(managed_user->GetString(kChromeAvatarKey, &avatar)); 259 EXPECT_TRUE(supervised_user->GetString(kChromeAvatarKey, &avatar));
259 EXPECT_EQ(kAvatar3, avatar); 260 EXPECT_EQ(kAvatar3, avatar);
260 } 261 }
261 { 262 {
262 const base::DictionaryValue* managed_user = NULL; 263 const base::DictionaryValue* supervised_user = NULL;
263 ASSERT_TRUE(managed_users->GetDictionary(kUserId4, &managed_user)); 264 ASSERT_TRUE(supervised_users->GetDictionary(kUserId4, &supervised_user));
264 ASSERT_TRUE(managed_user); 265 ASSERT_TRUE(supervised_user);
265 std::string name; 266 std::string name;
266 EXPECT_TRUE(managed_user->GetString(kNameKey, &name)); 267 EXPECT_TRUE(supervised_user->GetString(kNameKey, &name));
267 EXPECT_EQ(kName4, name); 268 EXPECT_EQ(kName4, name);
268 bool acknowledged = false; 269 bool acknowledged = false;
269 EXPECT_TRUE(managed_user->GetBoolean(kAcknowledgedKey, &acknowledged)); 270 EXPECT_TRUE(supervised_user->GetBoolean(kAcknowledgedKey, &acknowledged));
270 EXPECT_TRUE(acknowledged); 271 EXPECT_TRUE(acknowledged);
271 std::string avatar; 272 std::string avatar;
272 EXPECT_TRUE(managed_user->GetString(kChromeAvatarKey, &avatar)); 273 EXPECT_TRUE(supervised_user->GetString(kChromeAvatarKey, &avatar));
273 EXPECT_EQ(kAvatar4, avatar); 274 EXPECT_EQ(kAvatar4, avatar);
274 } 275 }
275 276
276 EXPECT_EQ(1u, change_processor()->changes().size()); 277 EXPECT_EQ(1u, change_processor()->changes().size());
277 { 278 {
278 SyncChange change = change_processor()->GetChange(kUserId1); 279 SyncChange change = change_processor()->GetChange(kUserId1);
279 ASSERT_TRUE(change.IsValid()); 280 ASSERT_TRUE(change.IsValid());
280 EXPECT_EQ(SyncChange::ACTION_ADD, change.change_type()); 281 EXPECT_EQ(SyncChange::ACTION_ADD, change.change_type());
281 const ManagedUserSpecifics& managed_user = 282 const ManagedUserSpecifics& supervised_user =
282 change.sync_data().GetSpecifics().managed_user(); 283 change.sync_data().GetSpecifics().managed_user();
283 EXPECT_EQ(kName1, managed_user.name()); 284 EXPECT_EQ(kName1, supervised_user.name());
284 EXPECT_FALSE(managed_user.acknowledged()); 285 EXPECT_FALSE(supervised_user.acknowledged());
285 EXPECT_EQ(kAvatar1, managed_user.chrome_avatar()); 286 EXPECT_EQ(kAvatar1, supervised_user.chrome_avatar());
286 } 287 }
287 } 288 }
288 289
289 TEST_F(ManagedUserSyncServiceTest, GetAvatarIndex) { 290 TEST_F(SupervisedUserSyncServiceTest, GetAvatarIndex) {
290 int avatar = 100; 291 int avatar = 100;
291 EXPECT_TRUE(ManagedUserSyncService::GetAvatarIndex(std::string(), &avatar)); 292 EXPECT_TRUE(SupervisedUserSyncService::GetAvatarIndex(std::string(),
292 EXPECT_EQ(ManagedUserSyncService::kNoAvatar, avatar); 293 &avatar));
294 EXPECT_EQ(SupervisedUserSyncService::kNoAvatar, avatar);
293 295
294 int avatar_index = 4; 296 int avatar_index = 4;
295 #if defined(OS_CHROMEOS) 297 #if defined(OS_CHROMEOS)
296 avatar_index += chromeos::kFirstDefaultImageIndex; 298 avatar_index += chromeos::kFirstDefaultImageIndex;
297 #endif 299 #endif
298 std::string avatar_str = 300 std::string avatar_str =
299 ManagedUserSyncService::BuildAvatarString(avatar_index); 301 SupervisedUserSyncService::BuildAvatarString(avatar_index);
300 #if defined(OS_CHROMEOS) 302 #if defined(OS_CHROMEOS)
301 EXPECT_EQ(base::StringPrintf("chromeos-avatar-index:%d", avatar_index), 303 EXPECT_EQ(base::StringPrintf("chromeos-avatar-index:%d", avatar_index),
302 avatar_str); 304 avatar_str);
303 #else 305 #else
304 EXPECT_EQ(base::StringPrintf("chrome-avatar-index:%d", avatar_index), 306 EXPECT_EQ(base::StringPrintf("chrome-avatar-index:%d", avatar_index),
305 avatar_str); 307 avatar_str);
306 #endif 308 #endif
307 EXPECT_TRUE(ManagedUserSyncService::GetAvatarIndex(avatar_str, &avatar)); 309 EXPECT_TRUE(SupervisedUserSyncService::GetAvatarIndex(avatar_str, &avatar));
308 EXPECT_EQ(avatar_index, avatar); 310 EXPECT_EQ(avatar_index, avatar);
309 311
310 avatar_index = 0; 312 avatar_index = 0;
311 #if defined(OS_CHROMEOS) 313 #if defined(OS_CHROMEOS)
312 avatar_index += chromeos::kFirstDefaultImageIndex; 314 avatar_index += chromeos::kFirstDefaultImageIndex;
313 #endif 315 #endif
314 avatar_str = ManagedUserSyncService::BuildAvatarString(avatar_index); 316 avatar_str = SupervisedUserSyncService::BuildAvatarString(avatar_index);
315 #if defined(OS_CHROMEOS) 317 #if defined(OS_CHROMEOS)
316 EXPECT_EQ(base::StringPrintf("chromeos-avatar-index:%d", avatar_index), 318 EXPECT_EQ(base::StringPrintf("chromeos-avatar-index:%d", avatar_index),
317 avatar_str); 319 avatar_str);
318 #else 320 #else
319 EXPECT_EQ(base::StringPrintf("chrome-avatar-index:%d", avatar_index), 321 EXPECT_EQ(base::StringPrintf("chrome-avatar-index:%d", avatar_index),
320 avatar_str); 322 avatar_str);
321 #endif 323 #endif
322 EXPECT_TRUE(ManagedUserSyncService::GetAvatarIndex(avatar_str, &avatar)); 324 EXPECT_TRUE(SupervisedUserSyncService::GetAvatarIndex(avatar_str, &avatar));
323 EXPECT_EQ(avatar_index, avatar); 325 EXPECT_EQ(avatar_index, avatar);
324 326
325 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("wrong-prefix:5", 327 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex("wrong-prefix:5",
326 &avatar)); 328 &avatar));
327 #if defined(OS_CHROMEOS) 329 #if defined(OS_CHROMEOS)
328 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chromeos-avatar-indes:2", 330 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
329 &avatar)); 331 "chromeos-avatar-indes:2",
332 &avatar));
330 333
331 EXPECT_FALSE( 334 EXPECT_FALSE(
332 ManagedUserSyncService::GetAvatarIndex("chromeos-avatar-indexxx:2", 335 SupervisedUserSyncService::GetAvatarIndex("chromeos-avatar-indexxx:2",
333 &avatar)); 336 &avatar));
334 337
335 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chromeos-avatar-index:", 338 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
336 &avatar)); 339 "chromeos-avatar-index:",
340 &avatar));
337 341
338 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chromeos-avatar-index:x", 342 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
339 &avatar)); 343 "chromeos-avatar-index:x",
344 &avatar));
340 345
341 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chrome-avatar-index:5", 346 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
342 &avatar)); 347 "chrome-avatar-index:5",
348 &avatar));
343 #else 349 #else
344 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chrome-avatar-indes:2", 350 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
345 &avatar)); 351 "chrome-avatar-indes:2",
352 &avatar));
346 353
347 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chrome-avatar-indexxx:2", 354 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
348 &avatar)); 355 "chrome-avatar-indexxx:2",
356 &avatar));
349 357
350 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chrome-avatar-index:", 358 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
351 &avatar)); 359 "chrome-avatar-index:",
360 &avatar));
352 361
353 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chrome-avatar-index:x", 362 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
354 &avatar)); 363 "chrome-avatar-index:x",
364 &avatar));
355 365
356 EXPECT_FALSE(ManagedUserSyncService::GetAvatarIndex("chromeos-avatar-index:5", 366 EXPECT_FALSE(SupervisedUserSyncService::GetAvatarIndex(
357 &avatar)); 367 "chromeos-avatar-index:5",
368 &avatar));
358 #endif 369 #endif
359 } 370 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698