Chromium Code Reviews| OLD | NEW |
|---|---|
| (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
| |
| OLD | NEW |