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

Side by Side Diff: chrome/browser/signin/chrome_signin_client_unittest.cc

Issue 2944713003: After signin token check failed, show force reauth dialog and start window closing countdown. (Closed)
Patch Set: nit Created 3 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
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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/signin/chrome_signin_client.h" 5 #include "chrome/browser/signin/chrome_signin_client.h"
6 6
7 #include <memory> 7 #include <memory>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/message_loop/message_loop.h" 10 #include "base/message_loop/message_loop.h"
(...skipping 157 matching lines...) Expand 10 before | Expand all | Expand 10 after
168 fake_controller_.reset(controller); 168 fake_controller_.reset(controller);
169 } 169 }
170 170
171 std::unique_ptr<SigninErrorController> fake_controller_; 171 std::unique_ptr<SigninErrorController> fake_controller_;
172 std::unique_ptr<MockChromeSigninClient> client_; 172 std::unique_ptr<MockChromeSigninClient> client_;
173 std::unique_ptr<MockSigninManager> manager_; 173 std::unique_ptr<MockSigninManager> manager_;
174 }; 174 };
175 175
176 TEST_F(ChromeSigninClientSignoutTest, SignOut) { 176 TEST_F(ChromeSigninClientSignoutTest, SignOut) {
177 signin_metrics::ProfileSignout source_metric = 177 signin_metrics::ProfileSignout source_metric =
178 signin_metrics::ProfileSignout::SIGNOUT_TEST; 178 signin_metrics::ProfileSignout::ABORT_SIGNIN;
179 signin_metrics::SignoutDelete delete_metric = 179 signin_metrics::SignoutDelete delete_metric =
180 signin_metrics::SignoutDelete::IGNORE_METRIC; 180 signin_metrics::SignoutDelete::IGNORE_METRIC;
181 181
182 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath())) 182 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath()))
183 .Times(1); 183 .Times(1);
184 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath())) 184 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath()))
185 .Times(1); 185 .Times(1);
186 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1); 186 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1);
187 187
188 manager_->SignOut(source_metric, delete_metric); 188 manager_->SignOut(source_metric, delete_metric);
189 } 189 }
190 190
191 TEST_F(ChromeSigninClientSignoutTest, SignOutWithoutManager) { 191 TEST_F(ChromeSigninClientSignoutTest, SignOutWithoutManager) {
192 signin_metrics::ProfileSignout source_metric = 192 signin_metrics::ProfileSignout source_metric =
193 signin_metrics::ProfileSignout::SIGNOUT_TEST; 193 signin_metrics::ProfileSignout::ABORT_SIGNIN;
194 signin_metrics::SignoutDelete delete_metric = 194 signin_metrics::SignoutDelete delete_metric =
195 signin_metrics::SignoutDelete::IGNORE_METRIC; 195 signin_metrics::SignoutDelete::IGNORE_METRIC;
196 196
197 MockSigninManager other_manager(client_.get()); 197 MockSigninManager other_manager(client_.get());
198 other_manager.CopyCredentialsFrom(*manager_.get()); 198 other_manager.CopyCredentialsFrom(*manager_.get());
199 199
200 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath())) 200 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath()))
201 .Times(0); 201 .Times(0);
202 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath())) 202 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath()))
203 .Times(1); 203 .Times(1);
204 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1); 204 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1);
205 manager_->SignOut(source_metric, delete_metric); 205 manager_->SignOut(source_metric, delete_metric);
206 206
207 ::testing::Mock::VerifyAndClearExpectations(manager_.get()); 207 ::testing::Mock::VerifyAndClearExpectations(manager_.get());
208 208
209 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath())) 209 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath()))
210 .Times(1); 210 .Times(1);
211 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath())) 211 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath()))
212 .Times(1); 212 .Times(1);
213 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1); 213 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1);
214 manager_->SignOut(source_metric, delete_metric); 214 manager_->SignOut(source_metric, delete_metric);
215 } 215 }
216 216
217 TEST_F(ChromeSigninClientSignoutTest, SignOutWithoutForceSignin) { 217 TEST_F(ChromeSigninClientSignoutTest, SignOutWithoutForceSignin) {
218 signin_util::SetForceSigninForTesting(false); 218 signin_util::SetForceSigninForTesting(false);
219 CreateClient(browser()->profile()); 219 CreateClient(browser()->profile());
220 manager_.reset(new MockSigninManager(client_.get())); 220 manager_.reset(new MockSigninManager(client_.get()));
221 221
222 signin_metrics::ProfileSignout source_metric = 222 signin_metrics::ProfileSignout source_metric =
223 signin_metrics::ProfileSignout::SIGNOUT_TEST; 223 signin_metrics::ProfileSignout::ABORT_SIGNIN;
224 signin_metrics::SignoutDelete delete_metric = 224 signin_metrics::SignoutDelete delete_metric =
225 signin_metrics::SignoutDelete::IGNORE_METRIC; 225 signin_metrics::SignoutDelete::IGNORE_METRIC;
226 226
227 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath())) 227 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath()))
228 .Times(0); 228 .Times(0);
229 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath())) 229 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath()))
230 .Times(0); 230 .Times(0);
231 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1); 231 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1);
232 manager_->SignOut(source_metric, delete_metric); 232 manager_->SignOut(source_metric, delete_metric);
233 } 233 }
234 234
235 TEST_F(ChromeSigninClientSignoutTest, SignOutGuestSession) { 235 TEST_F(ChromeSigninClientSignoutTest, SignOutGuestSession) {
236 TestingProfile::Builder builder; 236 TestingProfile::Builder builder;
237 builder.SetGuestSession(); 237 builder.SetGuestSession();
238 std::unique_ptr<TestingProfile> profile = builder.Build(); 238 std::unique_ptr<TestingProfile> profile = builder.Build();
239 239
240 CreateClient(profile.get()); 240 CreateClient(profile.get());
241 manager_.reset(new MockSigninManager(client_.get())); 241 manager_.reset(new MockSigninManager(client_.get()));
242 242
243 signin_metrics::ProfileSignout source_metric = 243 signin_metrics::ProfileSignout source_metric =
244 signin_metrics::ProfileSignout::SIGNOUT_TEST; 244 signin_metrics::ProfileSignout::ABORT_SIGNIN;
245 signin_metrics::SignoutDelete delete_metric = 245 signin_metrics::SignoutDelete delete_metric =
246 signin_metrics::SignoutDelete::IGNORE_METRIC; 246 signin_metrics::SignoutDelete::IGNORE_METRIC;
247 247
248 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath())) 248 EXPECT_CALL(*client_, ShowUserManager(browser()->profile()->GetPath()))
249 .Times(0); 249 .Times(0);
250 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath())) 250 EXPECT_CALL(*client_, LockForceSigninProfile(browser()->profile()->GetPath()))
251 .Times(0); 251 .Times(0);
252 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1); 252 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric)).Times(1);
253 manager_->SignOut(source_metric, delete_metric); 253 manager_->SignOut(source_metric, delete_metric);
254 } 254 }
255 255
256 #endif // !defined(OS_ANDROID) 256 #endif // !defined(OS_ANDROID)
257 #endif // !defined(OS_CHROMEOS) 257 #endif // !defined(OS_CHROMEOS)
OLDNEW
« no previous file with comments | « chrome/browser/signin/chrome_signin_client.cc ('k') | chrome/browser/signin/force_signin_verifier.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698