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

Side by Side Diff: chrome/browser/chromeos/policy/power_policy_browsertest.cc

Issue 14092002: Move MockUpdateEngineClient::GetLastStatus() expectation to MockDBusThreadManager's ctor (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: remove .Times(AnyNumber()) Created 7 years, 8 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 (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 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 <string> 5 #include <string>
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "chrome/browser/policy/browser_policy_connector.h" 10 #include "chrome/browser/policy/browser_policy_connector.h"
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
68 private: 68 private:
69 MockConfigurationPolicyProvider provider_; 69 MockConfigurationPolicyProvider provider_;
70 }; 70 };
71 71
72 void PowerPolicyBrowserTest::SetUpInProcessBrowserTestFixture() { 72 void PowerPolicyBrowserTest::SetUpInProcessBrowserTestFixture() {
73 chromeos::MockDBusThreadManager* dbus_thread_manager = 73 chromeos::MockDBusThreadManager* dbus_thread_manager =
74 new chromeos::MockDBusThreadManager; 74 new chromeos::MockDBusThreadManager;
75 chromeos::DBusThreadManager::InitializeForTesting(dbus_thread_manager); 75 chromeos::DBusThreadManager::InitializeForTesting(dbus_thread_manager);
76 EXPECT_CALL(*dbus_thread_manager->mock_session_manager_client(), 76 EXPECT_CALL(*dbus_thread_manager->mock_session_manager_client(),
77 RetrieveUserPolicy(_)); 77 RetrieveUserPolicy(_));
78 EXPECT_CALL(*dbus_thread_manager->mock_update_engine_client(),
79 GetLastStatus())
80 .Times(1)
81 .WillOnce(Return(chromeos::MockUpdateEngineClient::Status()));
82 power_manager_client_ = dbus_thread_manager->mock_power_manager_client(); 78 power_manager_client_ = dbus_thread_manager->mock_power_manager_client();
83 EXPECT_CALL(provider_, IsInitializationComplete(_)) 79 EXPECT_CALL(provider_, IsInitializationComplete(_))
84 .WillRepeatedly(Return(true)); 80 .WillRepeatedly(Return(true));
85 EXPECT_CALL(provider_, RegisterPolicyDomain(_, _)).Times(AnyNumber()); 81 EXPECT_CALL(provider_, RegisterPolicyDomain(_, _)).Times(AnyNumber());
86 BrowserPolicyConnector::SetPolicyProviderForTesting(&provider_); 82 BrowserPolicyConnector::SetPolicyProviderForTesting(&provider_);
87 } 83 }
88 84
89 // Verify that setting |user_policy_name| to |user_policy_value| causes the 85 // Verify that setting |user_policy_name| to |user_policy_value| causes the
90 // updated |power_management_policy| to be sent to the power management backend. 86 // updated |power_management_policy| to be sent to the power management backend.
91 void PowerPolicyBrowserTest::SetUserPolicyAndVerifyPowerManagementPolicy( 87 void PowerPolicyBrowserTest::SetUserPolicyAndVerifyPowerManagementPolicy(
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
218 214
219 // During teardown, an empty power management policy should be sent to the 215 // During teardown, an empty power management policy should be sent to the
220 // backend. 216 // backend.
221 power_management_policy.Clear(); 217 power_management_policy.Clear();
222 EXPECT_CALL(*power_manager_client_, SetPolicy( 218 EXPECT_CALL(*power_manager_client_, SetPolicy(
223 PowerManagementPolicyMatches(power_management_policy))); 219 PowerManagementPolicyMatches(power_management_policy)));
224 EXPECT_CALL(*power_manager_client_, RemoveObserver(_)).Times(AnyNumber()); 220 EXPECT_CALL(*power_manager_client_, RemoveObserver(_)).Times(AnyNumber());
225 } 221 }
226 222
227 } // namespace policy 223 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698