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

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

Issue 2442843002: Override SigninManager::SignOut if force-signin is enabled. (Closed)
Patch Set: tommycli's comments Created 4 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
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "chrome/browser/signin/force_signin_manager.h"
6
7 #include <memory>
8
9 #include "chrome/browser/prefs/browser_prefs.h"
10 #include "chrome/common/pref_names.h"
11 #include "chrome/test/base/browser_with_test_window_test.h"
12 #include "chrome/test/base/testing_browser_process.h"
13 #include "components/prefs/testing_pref_service.h"
14 #include "components/signin/core/browser/test_signin_client.h"
15 #include "testing/gmock/include/gmock/gmock.h"
16
17 class MockForceSigninManager : public ForceSigninManager {
18 public:
19 explicit MockForceSigninManager(Profile* profile)
20 : ForceSigninManager(profile,
21 &test_client_,
22 nullptr,
23 &fake_service_,
24 nullptr) {}
25
26 MOCK_METHOD3(DoSignOut,
27 void(signin_metrics::ProfileSignout,
28 signin_metrics::SignoutDelete,
29 const base::FilePath&));
30 MOCK_METHOD1(ShowUserManager, void(const base::FilePath&));
31
32 TestSigninClient test_client_{nullptr};
33 AccountTrackerService fake_service_;
34 };
35
36 class ForceSigninManagerTest : public BrowserWithTestWindowTest {
37 public:
38 void SetUp() override {
39 BrowserWithTestWindowTest::SetUp();
40 manager_.reset(new MockForceSigninManager(browser()->profile()));
41
42 prefs_.reset(new TestingPrefServiceSimple());
43 chrome::RegisterLocalState(prefs_->registry());
44 TestingBrowserProcess::GetGlobal()->SetLocalState(prefs_.get());
45 prefs_->SetBoolean(prefs::kForceBrowserSignin, true);
46 }
47
48 void TearDown() override {
49 BrowserWithTestWindowTest::TearDown();
50 TestingBrowserProcess::GetGlobal()->SetLocalState(nullptr);
51 }
52
53 std::unique_ptr<MockForceSigninManager> manager_;
54 std::unique_ptr<TestingPrefServiceSimple> prefs_;
55 };
56
57 TEST_F(ForceSigninManagerTest, Signout) {
58 signin_metrics::ProfileSignout source_metric =
59 signin_metrics::ProfileSignout::SIGNOUT_TEST;
60 signin_metrics::SignoutDelete delete_metric =
61 signin_metrics::SignoutDelete::IGNORE_METRIC;
62 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric,
63 browser()->profile()->GetPath()))
64 .Times(1);
65 EXPECT_CALL(*manager_, ShowUserManager(browser()->profile()->GetPath()))
66 .Times(1);
67 manager_->SignOut(source_metric, delete_metric);
68 }
69
70 TEST_F(ForceSigninManagerTest, SignoutWithoutManager) {
71 signin_metrics::ProfileSignout source_metric =
72 signin_metrics::ProfileSignout::SIGNOUT_TEST;
73 signin_metrics::SignoutDelete delete_metric =
74 signin_metrics::SignoutDelete::IGNORE_METRIC;
75
76 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric,
77 browser()->profile()->GetPath()))
78 .Times(1);
79 ForceSigninManager::DisableUserManagerDisplayForNextSignOut(manager_.get());
80 manager_->SignOut(source_metric, delete_metric);
81
82 EXPECT_CALL(*manager_, DoSignOut(source_metric, delete_metric,
83 browser()->profile()->GetPath()))
84 .Times(1);
85 EXPECT_CALL(*manager_, ShowUserManager(browser()->profile()->GetPath()))
86 .Times(1);
87 manager_->SignOut(source_metric, delete_metric);
88 }
Roger Tawa OOO till Jul 10th 2016/10/26 18:25:31 Should have tests for system profile and guest pro
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698