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

Side by Side Diff: chrome/browser/chromeos/login/screens/device_disabled_screen_unittest.cc

Issue 679133002: ScreenObserver is renamed to BaseScreenDelegate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixes. Created 6 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 "chrome/browser/chromeos/login/screens/device_disabled_screen.h" 5 #include "chrome/browser/chromeos/login/screens/device_disabled_screen.h"
6 6
7 #include "base/command_line.h" 7 #include "base/command_line.h"
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/prefs/scoped_user_pref_update.h" 10 #include "base/prefs/scoped_user_pref_update.h"
11 #include "base/prefs/testing_pref_service.h" 11 #include "base/prefs/testing_pref_service.h"
12 #include "chrome/browser/browser_process_platform_part.h" 12 #include "chrome/browser/browser_process_platform_part.h"
13 #include "chrome/browser/chromeos/login/screens/base_screen_delegate.h"
13 #include "chrome/browser/chromeos/login/screens/mock_device_disabled_screen_acto r.h" 14 #include "chrome/browser/chromeos/login/screens/mock_device_disabled_screen_acto r.h"
14 #include "chrome/browser/chromeos/login/screens/screen_observer.h"
15 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h" 15 #include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
16 #include "chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.h" 16 #include "chrome/browser/chromeos/policy/device_cloud_policy_manager_chromeos.h"
17 #include "chrome/browser/chromeos/policy/server_backed_device_state.h" 17 #include "chrome/browser/chromeos/policy/server_backed_device_state.h"
18 #include "chrome/browser/chromeos/policy/stub_enterprise_install_attributes.h" 18 #include "chrome/browser/chromeos/policy/stub_enterprise_install_attributes.h"
19 #include "chrome/common/pref_names.h" 19 #include "chrome/common/pref_names.h"
20 #include "chrome/test/base/testing_browser_process.h" 20 #include "chrome/test/base/testing_browser_process.h"
21 #include "chromeos/chromeos_switches.h" 21 #include "chromeos/chromeos_switches.h"
22 #include "components/policy/core/common/cloud/cloud_policy_constants.h" 22 #include "components/policy/core/common/cloud/cloud_policy_constants.h"
23 #include "testing/gmock/include/gmock/gmock.h" 23 #include "testing/gmock/include/gmock/gmock.h"
24 #include "testing/gtest/include/gtest/gtest.h" 24 #include "testing/gtest/include/gtest/gtest.h"
25 25
26 using ::testing::_; 26 using ::testing::_;
27 27
28 namespace chromeos { 28 namespace chromeos {
29 29
30 namespace { 30 namespace {
31 31
32 const char kDisabledMessage[] = "Device disabled."; 32 const char kDisabledMessage[] = "Device disabled.";
33 33
34 } 34 }
35 35
36 class DeviceDisabledScreenTest : public testing::Test, public ScreenObserver { 36 class DeviceDisabledScreenTest : public testing::Test,
37 public BaseScreenDelegate {
37 public: 38 public:
38 DeviceDisabledScreenTest(); 39 DeviceDisabledScreenTest();
39 ~DeviceDisabledScreenTest() override; 40 ~DeviceDisabledScreenTest() override;
40 41
41 // testing::Test: 42 // testing::Test:
42 void SetUp() override; 43 void SetUp() override;
43 void TearDown() override; 44 void TearDown() override;
44 45
45 // ScreenObserver: 46 // BaseScreenDelegate:
46 MOCK_METHOD1(OnExit, void(ExitCodes)); 47 MOCK_METHOD1(OnExit, void(ExitCodes));
47 void ShowCurrentScreen() override; 48 void ShowCurrentScreen() override;
48 ErrorScreen* GetErrorScreen() override; 49 ErrorScreen* GetErrorScreen() override;
49 void ShowErrorScreen() override; 50 void ShowErrorScreen() override;
50 void HideErrorScreen(BaseScreen* parent_screen) override; 51 void HideErrorScreen(BaseScreen* parent_screen) override;
51 52
52 void SetDeviceDisabled(bool disabled); 53 void SetDeviceDisabled(bool disabled);
53 void SetDeviceMode(policy::DeviceMode device_mode); 54 void SetDeviceMode(policy::DeviceMode device_mode);
54 55
55 void ExpectScreenToNotShow(); 56 void ExpectScreenToNotShow();
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 109
109 void DeviceDisabledScreenTest::SetDeviceMode(policy::DeviceMode device_mode) { 110 void DeviceDisabledScreenTest::SetDeviceMode(policy::DeviceMode device_mode) {
110 reinterpret_cast<policy::StubEnterpriseInstallAttributes*>( 111 reinterpret_cast<policy::StubEnterpriseInstallAttributes*>(
111 TestingBrowserProcess::GetGlobal()->platform_part()-> 112 TestingBrowserProcess::GetGlobal()->platform_part()->
112 browser_policy_connector_chromeos()->GetInstallAttributes())-> 113 browser_policy_connector_chromeos()->GetInstallAttributes())->
113 SetMode(device_mode); 114 SetMode(device_mode);
114 } 115 }
115 116
116 void DeviceDisabledScreenTest::ExpectScreenToNotShow() { 117 void DeviceDisabledScreenTest::ExpectScreenToNotShow() {
117 EXPECT_CALL(*actor_, Show(_)).Times(0); 118 EXPECT_CALL(*actor_, Show(_)).Times(0);
118 EXPECT_CALL(*this, OnExit(ScreenObserver::DEVICE_NOT_DISABLED)).Times(1); 119 EXPECT_CALL(*this, OnExit(BaseScreenDelegate::DEVICE_NOT_DISABLED)).Times(1);
119 } 120 }
120 121
121 void DeviceDisabledScreenTest::ExpectScreenToShow() { 122 void DeviceDisabledScreenTest::ExpectScreenToShow() {
122 EXPECT_CALL(*actor_, Show(kDisabledMessage)).Times(1); 123 EXPECT_CALL(*actor_, Show(kDisabledMessage)).Times(1);
123 EXPECT_CALL(*this, OnExit(ScreenObserver::DEVICE_NOT_DISABLED)).Times(0); 124 EXPECT_CALL(*this, OnExit(BaseScreenDelegate::DEVICE_NOT_DISABLED)).Times(0);
124 } 125 }
125 126
126 void DeviceDisabledScreenTest::TryToShowScreen() { 127 void DeviceDisabledScreenTest::TryToShowScreen() {
127 screen_->Show(); 128 screen_->Show();
128 } 129 }
129 130
130 // Verifies that the device disabled screen is not shown by default. 131 // Verifies that the device disabled screen is not shown by default.
131 TEST_F(DeviceDisabledScreenTest, DoNotShowByDefault) { 132 TEST_F(DeviceDisabledScreenTest, DoNotShowByDefault) {
132 ExpectScreenToNotShow(); 133 ExpectScreenToNotShow();
133 TryToShowScreen(); 134 TryToShowScreen();
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
172 // Verifies that the device disabled screen is shown when the device is marked 173 // Verifies that the device disabled screen is shown when the device is marked
173 // as disabled, device disabling is not turned off by flag and the device is not 174 // as disabled, device disabling is not turned off by flag and the device is not
174 // owned yet. 175 // owned yet.
175 TEST_F(DeviceDisabledScreenTest, ShowWhenDisabledAndNotOwned) { 176 TEST_F(DeviceDisabledScreenTest, ShowWhenDisabledAndNotOwned) {
176 SetDeviceDisabled(true); 177 SetDeviceDisabled(true);
177 ExpectScreenToShow(); 178 ExpectScreenToShow();
178 TryToShowScreen(); 179 TryToShowScreen();
179 } 180 }
180 181
181 } // namespace chromeos 182 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/login/screens/device_disabled_screen.cc ('k') | chrome/browser/chromeos/login/screens/error_screen.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698