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

Side by Side Diff: chrome/browser/ui/webui/chromeos/login/signin_userlist_unittest.cc

Issue 1412813003: This CL replaces user_manager::UserID with AccountId. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@468875--Chrome-OS-handles-deletion-of-Gmail-account-poorly--Create-AccountID-structure-part2--user_names
Patch Set: Fix Win GN build. Created 5 years, 1 month 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 "base/compiler_specific.h" 5 #include "base/compiler_specific.h"
6 #include "chrome/browser/chromeos/login/screens/user_selection_screen.h" 6 #include "chrome/browser/chromeos/login/screens/user_selection_screen.h"
7 #include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h" 7 #include "chrome/browser/chromeos/login/users/fake_chrome_user_manager.h"
8 #include "chrome/browser/chromeos/login/users/multi_profile_user_controller.h" 8 #include "chrome/browser/chromeos/login/users/multi_profile_user_controller.h"
9 #include "chrome/browser/chromeos/login/users/multi_profile_user_controller_dele gate.h" 9 #include "chrome/browser/chromeos/login/users/multi_profile_user_controller_dele gate.h"
10 #include "chrome/browser/chromeos/login/users/scoped_user_manager_enabler.h" 10 #include "chrome/browser/chromeos/login/users/scoped_user_manager_enabler.h"
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 42
43 void SetUp() override { 43 void SetUp() override {
44 profile_manager_.reset( 44 profile_manager_.reset(
45 new TestingProfileManager(TestingBrowserProcess::GetGlobal())); 45 new TestingProfileManager(TestingBrowserProcess::GetGlobal()));
46 ASSERT_TRUE(profile_manager_->SetUp()); 46 ASSERT_TRUE(profile_manager_->SetUp());
47 controller_.reset(new MultiProfileUserController( 47 controller_.reset(new MultiProfileUserController(
48 this, TestingBrowserProcess::GetGlobal()->local_state())); 48 this, TestingBrowserProcess::GetGlobal()->local_state()));
49 fake_user_manager_->set_multi_profile_user_controller(controller_.get()); 49 fake_user_manager_->set_multi_profile_user_controller(controller_.get());
50 50
51 for (size_t i = 0; i < arraysize(kUsersPublic); ++i) 51 for (size_t i = 0; i < arraysize(kUsersPublic); ++i)
52 fake_user_manager_->AddPublicAccountUser(kUsersPublic[i]); 52 fake_user_manager_->AddPublicAccountUser(
53 AccountId::FromUserEmail(kUsersPublic[i]));
53 54
54 for (size_t i = 0; i < arraysize(kUsers); ++i) 55 for (size_t i = 0; i < arraysize(kUsers); ++i)
55 fake_user_manager_->AddUser(kUsers[i]); 56 fake_user_manager_->AddUser(AccountId::FromUserEmail(kUsers[i]));
56 57
57 fake_user_manager_->set_owner_email(kOwner); 58 fake_user_manager_->set_owner_id(AccountId::FromUserEmail(kOwner));
58 } 59 }
59 60
60 void TearDown() override { 61 void TearDown() override {
61 controller_.reset(); 62 controller_.reset();
62 profile_manager_.reset(); 63 profile_manager_.reset();
63 } 64 }
64 65
65 // MultiProfileUserControllerDelegate overrides: 66 // MultiProfileUserControllerDelegate overrides:
66 void OnUserNotAllowed(const std::string& user_email) override {} 67 void OnUserNotAllowed(const std::string& user_email) override {}
67 68
68 FakeChromeUserManager* fake_user_manager_; 69 FakeChromeUserManager* fake_user_manager_;
69 ScopedUserManagerEnabler user_manager_enabler_; 70 ScopedUserManagerEnabler user_manager_enabler_;
70 scoped_ptr<TestingProfileManager> profile_manager_; 71 scoped_ptr<TestingProfileManager> profile_manager_;
71 std::map<std::string, proximity_auth::ScreenlockBridge::LockHandler::AuthType> 72 std::map<std::string, proximity_auth::ScreenlockBridge::LockHandler::AuthType>
72 user_auth_type_map; 73 user_auth_type_map;
73 scoped_ptr<MultiProfileUserController> controller_; 74 scoped_ptr<MultiProfileUserController> controller_;
74 75
75 DISALLOW_COPY_AND_ASSIGN(SigninPrepareUserListTest); 76 DISALLOW_COPY_AND_ASSIGN(SigninPrepareUserListTest);
76 }; 77 };
77 78
78 TEST_F(SigninPrepareUserListTest, AlwaysKeepOwnerInList) { 79 TEST_F(SigninPrepareUserListTest, AlwaysKeepOwnerInList) {
79 EXPECT_LT(kMaxUsers, fake_user_manager_->GetUsers().size()); 80 EXPECT_LT(kMaxUsers, fake_user_manager_->GetUsers().size());
80 user_manager::UserList users_to_send = 81 user_manager::UserList users_to_send =
81 UserSelectionScreen::PrepareUserListForSending( 82 UserSelectionScreen::PrepareUserListForSending(
82 fake_user_manager_->GetUsers(), kOwner, true /* is signin to add */); 83 fake_user_manager_->GetUsers(), kOwner, true /* is signin to add */);
83 84
84 EXPECT_EQ(kMaxUsers, users_to_send.size()); 85 EXPECT_EQ(kMaxUsers, users_to_send.size());
85 EXPECT_EQ(kOwner, users_to_send.back()->email()); 86 EXPECT_EQ(kOwner, users_to_send.back()->email());
86 87
87 fake_user_manager_->RemoveUserFromList("a16@gmail.com"); 88 fake_user_manager_->RemoveUserFromList(
88 fake_user_manager_->RemoveUserFromList("a17@gmail.com"); 89 AccountId::FromUserEmail("a16@gmail.com"));
90 fake_user_manager_->RemoveUserFromList(
91 AccountId::FromUserEmail("a17@gmail.com"));
89 users_to_send = UserSelectionScreen::PrepareUserListForSending( 92 users_to_send = UserSelectionScreen::PrepareUserListForSending(
90 fake_user_manager_->GetUsers(), 93 fake_user_manager_->GetUsers(),
91 kOwner, 94 kOwner,
92 true /* is signin to add */); 95 true /* is signin to add */);
93 96
94 EXPECT_EQ(kMaxUsers, users_to_send.size()); 97 EXPECT_EQ(kMaxUsers, users_to_send.size());
95 EXPECT_EQ("a18@gmail.com", users_to_send.back()->email()); 98 EXPECT_EQ("a18@gmail.com", users_to_send.back()->email());
96 EXPECT_EQ(kOwner, users_to_send[kMaxUsers-2]->email()); 99 EXPECT_EQ(kOwner, users_to_send[kMaxUsers-2]->email());
97 } 100 }
98 101
99 TEST_F(SigninPrepareUserListTest, PublicAccounts) { 102 TEST_F(SigninPrepareUserListTest, PublicAccounts) {
100 user_manager::UserList users_to_send = 103 user_manager::UserList users_to_send =
101 UserSelectionScreen::PrepareUserListForSending( 104 UserSelectionScreen::PrepareUserListForSending(
102 fake_user_manager_->GetUsers(), kOwner, true /* is signin to add */); 105 fake_user_manager_->GetUsers(), kOwner, true /* is signin to add */);
103 106
104 EXPECT_EQ(kMaxUsers, users_to_send.size()); 107 EXPECT_EQ(kMaxUsers, users_to_send.size());
105 EXPECT_EQ("a0@gmail.com", users_to_send.front()->email()); 108 EXPECT_EQ("a0@gmail.com", users_to_send.front()->email());
106 109
107 users_to_send = UserSelectionScreen::PrepareUserListForSending( 110 users_to_send = UserSelectionScreen::PrepareUserListForSending(
108 fake_user_manager_->GetUsers(), 111 fake_user_manager_->GetUsers(),
109 kOwner, 112 kOwner,
110 false /* is signin to add */); 113 false /* is signin to add */);
111 114
112 EXPECT_EQ(kMaxUsers, users_to_send.size()); 115 EXPECT_EQ(kMaxUsers, users_to_send.size());
113 EXPECT_EQ("public0@gmail.com", users_to_send.front()->email()); 116 EXPECT_EQ("public0@gmail.com", users_to_send.front()->email());
114 } 117 }
115 118
116 } // namespace chromeos 119 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/ui/webui/chromeos/login/signin_screen_handler.cc ('k') | chrome/browser/ui/webui/chromeos/ui_account_tweaks.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698