blob: b612811aa84e23d7657ae63a1f82f975fd1d1a92 [file] [log] [blame]
Alex Deymoaea4c1c2015-08-19 20:24:43 -07001//
2// Copyright (C) 2012 The Android Open Source Project
3//
4// Licensed under the Apache License, Version 2.0 (the "License");
5// you may not use this file except in compliance with the License.
6// You may obtain a copy of the License at
7//
8// http://www.apache.org/licenses/LICENSE-2.0
9//
10// Unless required by applicable law or agreed to in writing, software
11// distributed under the License is distributed on an "AS IS" BASIS,
12// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13// See the License for the specific language governing permissions and
14// limitations under the License.
15//
Darin Petkovf42cc1c2010-09-01 09:03:02 -070016
Alex Deymo2c0db7b2014-11-04 12:23:39 -080017#include "update_engine/update_attempter.h"
18
Ben Chan9abb7632014-08-07 00:10:53 -070019#include <stdint.h>
20
Ben Chan02f7c1d2014-10-18 15:18:02 -070021#include <memory>
22
Ben Chan06c76a42014-09-05 08:21:06 -070023#include <base/files/file_util.h>
Alex Deymo0b3db6b2015-08-10 15:19:37 -070024#include <base/message_loop/message_loop.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070025#include <brillo/make_unique_ptr.h>
26#include <brillo/message_loops/base_message_loop.h>
27#include <brillo/message_loops/message_loop.h>
28#include <brillo/message_loops/message_loop_utils.h>
Darin Petkovf42cc1c2010-09-01 09:03:02 -070029#include <gtest/gtest.h>
Patrick Dubroy7fbbe8a2011-08-01 17:28:22 +020030#include <policy/libpolicy.h>
31#include <policy/mock_device_policy.h>
Darin Petkovf42cc1c2010-09-01 09:03:02 -070032
Amin Hassanie94ece52017-08-01 15:03:08 -070033#if USE_CHROME_NETWORK_PROXY
Daniel Erate5f6f252017-04-20 12:09:58 -060034#include "network_proxy/dbus-proxies.h"
35#include "network_proxy/dbus-proxy-mocks.h"
Amin Hassanie94ece52017-08-01 15:03:08 -070036#endif // USE_CHROME_NETWORK_PROXY
Alex Deymo39910dc2015-11-09 17:04:30 -080037#include "update_engine/common/fake_clock.h"
38#include "update_engine/common/fake_prefs.h"
Alex Deymo14fd1ec2016-02-24 22:03:57 -080039#include "update_engine/common/mock_action.h"
40#include "update_engine/common/mock_action_processor.h"
Alex Deymo39910dc2015-11-09 17:04:30 -080041#include "update_engine/common/mock_http_fetcher.h"
42#include "update_engine/common/mock_prefs.h"
43#include "update_engine/common/platform_constants.h"
44#include "update_engine/common/prefs.h"
45#include "update_engine/common/test_utils.h"
46#include "update_engine/common/utils.h"
Gilad Arnold5bb4c902014-04-10 12:32:13 -070047#include "update_engine/fake_system_state.h"
David Zeuthen8f191b22013-08-06 12:27:50 -070048#include "update_engine/mock_p2p_manager.h"
Jay Srinivasan6f6ea002012-12-14 11:26:28 -080049#include "update_engine/mock_payload_state.h"
Aaron Wood9321f502017-09-07 11:18:54 -070050#include "update_engine/mock_service_observer.h"
Alex Deymo39910dc2015-11-09 17:04:30 -080051#include "update_engine/payload_consumer/filesystem_verifier_action.h"
52#include "update_engine/payload_consumer/install_plan.h"
53#include "update_engine/payload_consumer/payload_constants.h"
54#include "update_engine/payload_consumer/postinstall_runner_action.h"
Darin Petkovf42cc1c2010-09-01 09:03:02 -070055
Daniel Erate5f6f252017-04-20 12:09:58 -060056namespace org {
57namespace chromium {
58class NetworkProxyServiceInterfaceProxyMock;
59} // namespace chromium
60} // namespace org
61
David Zeuthen985b1122013-10-09 12:13:15 -070062using base::Time;
63using base::TimeDelta;
Aaron Wood224dfc22017-10-04 10:58:36 -070064using chromeos_update_manager::EvalStatus;
65using chromeos_update_manager::UpdateCheckParams;
Daniel Erate5f6f252017-04-20 12:09:58 -060066using org::chromium::NetworkProxyServiceInterfaceProxyInterface;
67using org::chromium::NetworkProxyServiceInterfaceProxyMock;
Aaron Wood224dfc22017-10-04 10:58:36 -070068#if USE_LIBCROS
69using org::chromium::LibCrosServiceInterfaceProxyMock;
70using org::chromium::UpdateEngineLibcrosProxyResolvedInterfaceProxyMock;
71#endif // USE_LIBCROS
Darin Petkovf42cc1c2010-09-01 09:03:02 -070072using std::string;
Ben Chan02f7c1d2014-10-18 15:18:02 -070073using std::unique_ptr;
Aaron Wood224dfc22017-10-04 10:58:36 -070074using testing::_;
Darin Petkov36275772010-10-01 11:40:57 -070075using testing::DoAll;
Aaron Wood7f92e2b2017-08-28 14:51:21 -070076using testing::Field;
Darin Petkovf42cc1c2010-09-01 09:03:02 -070077using testing::InSequence;
Darin Petkov2dd01092010-10-08 15:43:05 -070078using testing::Ne;
Darin Petkov9c096d62010-11-17 14:49:04 -080079using testing::NiceMock;
Darin Petkovf42cc1c2010-09-01 09:03:02 -070080using testing::Property;
81using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070082using testing::ReturnPointee;
Alex Deymo2c0db7b2014-11-04 12:23:39 -080083using testing::SaveArg;
Darin Petkov36275772010-10-01 11:40:57 -070084using testing::SetArgumentPointee;
Aaron Wood224dfc22017-10-04 10:58:36 -070085using update_engine::UpdateAttemptFlags;
Aaron Wood7f92e2b2017-08-28 14:51:21 -070086using update_engine::UpdateEngineStatus;
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -070087using update_engine::UpdateStatus;
Darin Petkovf42cc1c2010-09-01 09:03:02 -070088
89namespace chromeos_update_engine {
90
91// Test a subclass rather than the main class directly so that we can mock out
Darin Petkovcd1666f2010-09-23 09:53:44 -070092// methods within the class. There're explicit unit tests for the mocked out
Darin Petkovf42cc1c2010-09-01 09:03:02 -070093// methods.
94class UpdateAttempterUnderTest : public UpdateAttempter {
95 public:
Daniel Erate5f6f252017-04-20 12:09:58 -060096 UpdateAttempterUnderTest(
97 SystemState* system_state,
98 NetworkProxyServiceInterfaceProxyInterface* network_proxy_service_proxy)
99 : UpdateAttempter(system_state, nullptr, network_proxy_service_proxy) {}
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700100
101 // Wrap the update scheduling method, allowing us to opt out of scheduled
102 // updates for testing purposes.
103 void ScheduleUpdates() override {
104 schedule_updates_called_ = true;
105 if (do_schedule_updates_) {
106 UpdateAttempter::ScheduleUpdates();
107 } else {
108 LOG(INFO) << "[TEST] Update scheduling disabled.";
109 }
110 }
111 void EnableScheduleUpdates() { do_schedule_updates_ = true; }
112 void DisableScheduleUpdates() { do_schedule_updates_ = false; }
113
114 // Indicates whether ScheduleUpdates() was called.
115 bool schedule_updates_called() const { return schedule_updates_called_; }
116
David Pursell02c18642014-11-06 11:26:11 -0800117 // Need to expose forced_omaha_url_ so we can test it.
Alex Deymo60ca1a72015-06-18 18:19:15 -0700118 const string& forced_omaha_url() const { return forced_omaha_url_; }
David Pursell02c18642014-11-06 11:26:11 -0800119
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700120 private:
121 bool schedule_updates_called_ = false;
122 bool do_schedule_updates_ = true;
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700123};
124
125class UpdateAttempterTest : public ::testing::Test {
126 protected:
Jay Srinivasan43488792012-06-19 00:25:31 -0700127 UpdateAttempterTest()
Daniel Erate5f6f252017-04-20 12:09:58 -0600128 : certificate_checker_(fake_system_state_.mock_prefs(),
Alex Deymo33e91e72015-12-01 18:26:08 -0300129 &openssl_wrapper_) {
Gilad Arnold1f847232014-04-07 12:07:49 -0700130 // Override system state members.
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700131 fake_system_state_.set_connection_manager(&mock_connection_manager);
132 fake_system_state_.set_update_attempter(&attempter_);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700133 loop_.SetAsCurrent();
Gilad Arnold1f847232014-04-07 12:07:49 -0700134
Alex Deymo33e91e72015-12-01 18:26:08 -0300135 certificate_checker_.Init();
136
Sen Jiange67bb5b2016-06-20 15:53:56 -0700137 // Finish initializing the attempter.
Gilad Arnold1f847232014-04-07 12:07:49 -0700138 attempter_.Init();
Jay Srinivasan43488792012-06-19 00:25:31 -0700139 }
Gilad Arnoldeff87cc2013-07-22 18:32:09 -0700140
Alex Deymo610277e2014-11-11 21:18:11 -0800141 void SetUp() override {
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700142 EXPECT_NE(nullptr, attempter_.system_state_);
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700143 EXPECT_EQ(0, attempter_.http_response_code_);
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700144 EXPECT_EQ(UpdateStatus::IDLE, attempter_.status_);
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700145 EXPECT_EQ(0.0, attempter_.download_progress_);
146 EXPECT_EQ(0, attempter_.last_checked_time_);
147 EXPECT_EQ("0.0.0.0", attempter_.new_version_);
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700148 EXPECT_EQ(0ULL, attempter_.new_payload_size_);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800149 processor_ = new NiceMock<MockActionProcessor>();
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700150 attempter_.processor_.reset(processor_); // Transfers ownership.
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700151 prefs_ = fake_system_state_.mock_prefs();
Gilad Arnold74b5f552014-10-07 08:17:16 -0700152
153 // Set up store/load semantics of P2P properties via the mock PayloadState.
154 actual_using_p2p_for_downloading_ = false;
155 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
156 SetUsingP2PForDownloading(_))
157 .WillRepeatedly(SaveArg<0>(&actual_using_p2p_for_downloading_));
158 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
159 GetUsingP2PForDownloading())
160 .WillRepeatedly(ReturnPointee(&actual_using_p2p_for_downloading_));
161 actual_using_p2p_for_sharing_ = false;
162 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
163 SetUsingP2PForSharing(_))
164 .WillRepeatedly(SaveArg<0>(&actual_using_p2p_for_sharing_));
165 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
166 GetUsingP2PForDownloading())
167 .WillRepeatedly(ReturnPointee(&actual_using_p2p_for_sharing_));
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700168 }
169
Alex Deymo60ca1a72015-06-18 18:19:15 -0700170 public:
171 void ScheduleQuitMainLoop();
Patrick Dubroy7fbbe8a2011-08-01 17:28:22 +0200172
Alex Deymo60ca1a72015-06-18 18:19:15 -0700173 // Callbacks to run the different tests from the main loop.
Darin Petkove6ef2f82011-03-07 17:31:11 -0800174 void UpdateTestStart();
175 void UpdateTestVerify();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700176 void RollbackTestStart(bool enterprise_rollback, bool valid_slot);
Chris Sosa76a29ae2013-07-11 17:59:24 -0700177 void RollbackTestVerify();
Thieu Le116fda32011-04-19 11:01:54 -0700178 void PingOmahaTestStart();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700179 void ReadScatterFactorFromPolicyTestStart();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700180 void DecrementUpdateCheckCountTestStart();
Jay Srinivasan08fce042012-06-07 16:31:01 -0700181 void NoScatteringDoneDuringManualUpdateTestStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700182 void P2PNotEnabledStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700183 void P2PEnabledStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700184 void P2PEnabledInteractiveStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700185 void P2PEnabledStartingFailsStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700186 void P2PEnabledHousekeepingFailsStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700187
Gilad Arnold74b5f552014-10-07 08:17:16 -0700188 bool actual_using_p2p_for_downloading() {
189 return actual_using_p2p_for_downloading_;
190 }
191 bool actual_using_p2p_for_sharing() {
192 return actual_using_p2p_for_sharing_;
193 }
194
Alex Deymo0b3db6b2015-08-10 15:19:37 -0700195 base::MessageLoopForIO base_loop_;
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700196 brillo::BaseMessageLoop loop_{&base_loop_};
Alex Deymo60ca1a72015-06-18 18:19:15 -0700197
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700198 FakeSystemState fake_system_state_;
Amin Hassanie94ece52017-08-01 15:03:08 -0700199#if USE_CHROME_NETWORK_PROXY
Daniel Erate5f6f252017-04-20 12:09:58 -0600200 NetworkProxyServiceInterfaceProxyMock network_proxy_service_proxy_mock_;
201 UpdateAttempterUnderTest attempter_{&fake_system_state_,
202 &network_proxy_service_proxy_mock_};
Sen Jiangc92195c2016-06-13 15:48:44 -0700203#else
204 UpdateAttempterUnderTest attempter_{&fake_system_state_, nullptr};
Amin Hassanie94ece52017-08-01 15:03:08 -0700205#endif // USE_CHROME_NETWORK_PROXY
Alex Deymo33e91e72015-12-01 18:26:08 -0300206 OpenSSLWrapper openssl_wrapper_;
207 CertificateChecker certificate_checker_;
Alex Deymo30534502015-07-20 15:06:33 -0700208
Alex Deymo8427b4a2014-11-05 14:00:32 -0800209 NiceMock<MockActionProcessor>* processor_;
210 NiceMock<MockPrefs>* prefs_; // Shortcut to fake_system_state_->mock_prefs().
Jay Srinivasan55f50c22013-01-10 19:24:35 -0800211 NiceMock<MockConnectionManager> mock_connection_manager;
Gilad Arnoldeff87cc2013-07-22 18:32:09 -0700212
Gilad Arnold74b5f552014-10-07 08:17:16 -0700213 bool actual_using_p2p_for_downloading_;
214 bool actual_using_p2p_for_sharing_;
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700215};
216
Alex Deymo60ca1a72015-06-18 18:19:15 -0700217void UpdateAttempterTest::ScheduleQuitMainLoop() {
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700218 loop_.PostTask(
219 FROM_HERE,
220 base::Bind([](brillo::BaseMessageLoop* loop) { loop->BreakLoop(); },
221 base::Unretained(&loop_)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700222}
223
Darin Petkov1b003102010-11-30 10:18:36 -0800224TEST_F(UpdateAttempterTest, ActionCompletedDownloadTest) {
Ben Chan02f7c1d2014-10-18 15:18:02 -0700225 unique_ptr<MockHttpFetcher> fetcher(new MockHttpFetcher("", 0, nullptr));
Darin Petkov1b003102010-11-30 10:18:36 -0800226 fetcher->FailTransfer(503); // Sets the HTTP response code.
Alex Deymo1b3556c2016-02-03 09:54:02 -0800227 DownloadAction action(prefs_, nullptr, nullptr, nullptr, fetcher.release());
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800228 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _)).Times(0);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700229 attempter_.ActionCompleted(nullptr, &action, ErrorCode::kSuccess);
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700230 EXPECT_EQ(UpdateStatus::FINALIZING, attempter_.status());
Aaron Wood9321f502017-09-07 11:18:54 -0700231 EXPECT_EQ(0.0, attempter_.download_progress_);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700232 ASSERT_EQ(nullptr, attempter_.error_event_.get());
Darin Petkov1b003102010-11-30 10:18:36 -0800233}
234
235TEST_F(UpdateAttempterTest, ActionCompletedErrorTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -0800236 MockAction action;
237 EXPECT_CALL(action, Type()).WillRepeatedly(Return("MockAction"));
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700238 attempter_.status_ = UpdateStatus::DOWNLOADING;
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800239 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov1b003102010-11-30 10:18:36 -0800240 .WillOnce(Return(false));
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700241 attempter_.ActionCompleted(nullptr, &action, ErrorCode::kError);
242 ASSERT_NE(nullptr, attempter_.error_event_.get());
Darin Petkov1b003102010-11-30 10:18:36 -0800243}
244
Aaron Wood9321f502017-09-07 11:18:54 -0700245TEST_F(UpdateAttempterTest, DownloadProgressAccumulationTest) {
246 // Simple test case, where all the values match (nothing was skipped)
247 uint64_t bytes_progressed_1 = 1024 * 1024; // 1MB
248 uint64_t bytes_progressed_2 = 1024 * 1024; // 1MB
249 uint64_t bytes_received_1 = bytes_progressed_1;
250 uint64_t bytes_received_2 = bytes_received_1 + bytes_progressed_2;
251 uint64_t bytes_total = 20 * 1024 * 1024; // 20MB
252
253 double progress_1 =
254 static_cast<double>(bytes_received_1) / static_cast<double>(bytes_total);
255 double progress_2 =
256 static_cast<double>(bytes_received_2) / static_cast<double>(bytes_total);
257
258 EXPECT_EQ(0.0, attempter_.download_progress_);
259 // This is set via inspecting the InstallPlan payloads when the
260 // OmahaResponseAction is completed
261 attempter_.new_payload_size_ = bytes_total;
262 NiceMock<MockServiceObserver> observer;
263 EXPECT_CALL(observer,
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700264 SendStatusUpdate(AllOf(
265 Field(&UpdateEngineStatus::progress, progress_1),
266 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
267 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700268 EXPECT_CALL(observer,
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700269 SendStatusUpdate(AllOf(
270 Field(&UpdateEngineStatus::progress, progress_2),
271 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
272 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700273 attempter_.AddObserver(&observer);
274 attempter_.BytesReceived(bytes_progressed_1, bytes_received_1, bytes_total);
275 EXPECT_EQ(progress_1, attempter_.download_progress_);
276 // This iteration validates that a later set of updates to the variables are
277 // properly handled (so that |getStatus()| will return the same progress info
278 // as the callback is receiving.
279 attempter_.BytesReceived(bytes_progressed_2, bytes_received_2, bytes_total);
280 EXPECT_EQ(progress_2, attempter_.download_progress_);
281}
282
283TEST_F(UpdateAttempterTest, ChangeToDownloadingOnReceivedBytesTest) {
284 // The transition into UpdateStatus::DOWNLOADING happens when the
285 // first bytes are received.
286 uint64_t bytes_progressed = 1024 * 1024; // 1MB
287 uint64_t bytes_received = 2 * 1024 * 1024; // 2MB
288 uint64_t bytes_total = 20 * 1024 * 1024; // 300MB
289 attempter_.status_ = UpdateStatus::CHECKING_FOR_UPDATE;
290 // This is set via inspecting the InstallPlan payloads when the
291 // OmahaResponseAction is completed
292 attempter_.new_payload_size_ = bytes_total;
293 EXPECT_EQ(0.0, attempter_.download_progress_);
294 NiceMock<MockServiceObserver> observer;
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700295 EXPECT_CALL(observer,
296 SendStatusUpdate(AllOf(
297 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
298 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700299 attempter_.AddObserver(&observer);
300 attempter_.BytesReceived(bytes_progressed, bytes_received, bytes_total);
301 EXPECT_EQ(UpdateStatus::DOWNLOADING, attempter_.status_);
302}
303
304TEST_F(UpdateAttempterTest, BroadcastCompleteDownloadTest) {
305 // There is a special case to ensure that at 100% downloaded,
306 // download_progress_ is updated and that value broadcast. This test confirms
307 // that.
308 uint64_t bytes_progressed = 0; // ignored
309 uint64_t bytes_received = 5 * 1024 * 1024; // ignored
310 uint64_t bytes_total = 5 * 1024 * 1024; // 300MB
311 attempter_.status_ = UpdateStatus::DOWNLOADING;
312 attempter_.new_payload_size_ = bytes_total;
313 EXPECT_EQ(0.0, attempter_.download_progress_);
314 NiceMock<MockServiceObserver> observer;
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700315 EXPECT_CALL(observer,
316 SendStatusUpdate(AllOf(
317 Field(&UpdateEngineStatus::progress, 1.0),
318 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
319 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700320 attempter_.AddObserver(&observer);
321 attempter_.BytesReceived(bytes_progressed, bytes_received, bytes_total);
322 EXPECT_EQ(1.0, attempter_.download_progress_);
323}
324
Darin Petkov1b003102010-11-30 10:18:36 -0800325TEST_F(UpdateAttempterTest, ActionCompletedOmahaRequestTest) {
Ben Chan02f7c1d2014-10-18 15:18:02 -0700326 unique_ptr<MockHttpFetcher> fetcher(new MockHttpFetcher("", 0, nullptr));
Darin Petkov1b003102010-11-30 10:18:36 -0800327 fetcher->FailTransfer(500); // Sets the HTTP response code.
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700328 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300329 std::move(fetcher), false);
Darin Petkov1b003102010-11-30 10:18:36 -0800330 ObjectCollectorAction<OmahaResponse> collector_action;
331 BondActions(&action, &collector_action);
332 OmahaResponse response;
333 response.poll_interval = 234;
334 action.SetOutputObject(response);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800335 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _)).Times(0);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700336 attempter_.ActionCompleted(nullptr, &action, ErrorCode::kSuccess);
Darin Petkov1b003102010-11-30 10:18:36 -0800337 EXPECT_EQ(500, attempter_.http_response_code());
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700338 EXPECT_EQ(UpdateStatus::IDLE, attempter_.status());
Alex Deymo80f70ff2016-02-10 16:08:11 -0800339 EXPECT_EQ(234U, attempter_.server_dictated_poll_interval_);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700340 ASSERT_TRUE(attempter_.error_event_.get() == nullptr);
Darin Petkov1b003102010-11-30 10:18:36 -0800341}
342
Alex Deymo30534502015-07-20 15:06:33 -0700343TEST_F(UpdateAttempterTest, ConstructWithUpdatedMarkerTest) {
Alex Deymo906191f2015-10-12 12:22:44 -0700344 FakePrefs fake_prefs;
345 string boot_id;
346 EXPECT_TRUE(utils::GetBootId(&boot_id));
347 fake_prefs.SetString(kPrefsUpdateCompletedOnBootId, boot_id);
348 fake_system_state_.set_prefs(&fake_prefs);
Sen Jiangaeeb2e02016-06-09 15:00:16 -0700349 attempter_.Init();
350 EXPECT_EQ(UpdateStatus::UPDATED_NEED_REBOOT, attempter_.status());
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700351}
352
353TEST_F(UpdateAttempterTest, GetErrorCodeForActionTest) {
David Zeuthena99981f2013-04-29 13:42:47 -0700354 extern ErrorCode GetErrorCodeForAction(AbstractAction* action,
355 ErrorCode code);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700356 EXPECT_EQ(ErrorCode::kSuccess,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700357 GetErrorCodeForAction(nullptr, ErrorCode::kSuccess));
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700358
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700359 FakeSystemState fake_system_state;
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700360 OmahaRequestAction omaha_request_action(&fake_system_state, nullptr,
361 nullptr, false);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700362 EXPECT_EQ(ErrorCode::kOmahaRequestError,
363 GetErrorCodeForAction(&omaha_request_action, ErrorCode::kError));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700364 OmahaResponseHandlerAction omaha_response_handler_action(&fake_system_state_);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700365 EXPECT_EQ(ErrorCode::kOmahaResponseHandlerError,
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700366 GetErrorCodeForAction(&omaha_response_handler_action,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700367 ErrorCode::kError));
Sen Jiange6e4bb92016-04-05 14:59:12 -0700368 FilesystemVerifierAction filesystem_verifier_action;
Allie Woodeb9e6d82015-04-17 13:55:30 -0700369 EXPECT_EQ(ErrorCode::kFilesystemVerifierError,
370 GetErrorCodeForAction(&filesystem_verifier_action,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700371 ErrorCode::kError));
Alex Deymob15a0b82015-11-25 20:30:40 -0300372 PostinstallRunnerAction postinstall_runner_action(
Alex Deymofb905d92016-06-03 19:26:58 -0700373 fake_system_state.fake_boot_control(), fake_system_state.fake_hardware());
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700374 EXPECT_EQ(ErrorCode::kPostinstallRunnerError,
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700375 GetErrorCodeForAction(&postinstall_runner_action,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700376 ErrorCode::kError));
Alex Deymo8427b4a2014-11-05 14:00:32 -0800377 MockAction action_mock;
378 EXPECT_CALL(action_mock, Type()).WillOnce(Return("MockAction"));
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700379 EXPECT_EQ(ErrorCode::kError,
380 GetErrorCodeForAction(&action_mock, ErrorCode::kError));
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700381}
382
Darin Petkov36275772010-10-01 11:40:57 -0700383TEST_F(UpdateAttempterTest, DisableDeltaUpdateIfNeededTest) {
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700384 attempter_.omaha_request_params_->set_delta_okay(true);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800385 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700386 .WillOnce(Return(false));
387 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700388 EXPECT_TRUE(attempter_.omaha_request_params_->delta_okay());
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800389 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700390 .WillOnce(DoAll(
391 SetArgumentPointee<1>(UpdateAttempter::kMaxDeltaUpdateFailures - 1),
392 Return(true)));
393 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700394 EXPECT_TRUE(attempter_.omaha_request_params_->delta_okay());
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800395 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700396 .WillOnce(DoAll(
397 SetArgumentPointee<1>(UpdateAttempter::kMaxDeltaUpdateFailures),
398 Return(true)));
399 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700400 EXPECT_FALSE(attempter_.omaha_request_params_->delta_okay());
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800401 EXPECT_CALL(*prefs_, GetInt64(_, _)).Times(0);
Darin Petkov36275772010-10-01 11:40:57 -0700402 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700403 EXPECT_FALSE(attempter_.omaha_request_params_->delta_okay());
Darin Petkov36275772010-10-01 11:40:57 -0700404}
405
406TEST_F(UpdateAttempterTest, MarkDeltaUpdateFailureTest) {
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800407 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700408 .WillOnce(Return(false))
409 .WillOnce(DoAll(SetArgumentPointee<1>(-1), Return(true)))
410 .WillOnce(DoAll(SetArgumentPointee<1>(1), Return(true)))
411 .WillOnce(DoAll(
412 SetArgumentPointee<1>(UpdateAttempter::kMaxDeltaUpdateFailures),
413 Return(true)));
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800414 EXPECT_CALL(*prefs_, SetInt64(Ne(kPrefsDeltaUpdateFailures), _))
Darin Petkov2dd01092010-10-08 15:43:05 -0700415 .WillRepeatedly(Return(true));
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800416 EXPECT_CALL(*prefs_, SetInt64(kPrefsDeltaUpdateFailures, 1)).Times(2);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700417 EXPECT_CALL(*prefs_, SetInt64(kPrefsDeltaUpdateFailures, 2));
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800418 EXPECT_CALL(*prefs_, SetInt64(kPrefsDeltaUpdateFailures,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700419 UpdateAttempter::kMaxDeltaUpdateFailures + 1));
Darin Petkov36275772010-10-01 11:40:57 -0700420 for (int i = 0; i < 4; i ++)
421 attempter_.MarkDeltaUpdateFailure();
422}
423
Darin Petkov1b003102010-11-30 10:18:36 -0800424TEST_F(UpdateAttempterTest, ScheduleErrorEventActionNoEventTest) {
425 EXPECT_CALL(*processor_, EnqueueAction(_)).Times(0);
426 EXPECT_CALL(*processor_, StartProcessing()).Times(0);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700427 EXPECT_CALL(*fake_system_state_.mock_payload_state(), UpdateFailed(_))
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800428 .Times(0);
429 OmahaResponse response;
Jay Srinivasan53173b92013-05-17 17:13:01 -0700430 string url1 = "http://url1";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800431 response.packages.push_back({.payload_urls = {url1, "https://url"}});
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700432 EXPECT_CALL(*(fake_system_state_.mock_payload_state()), GetCurrentUrl())
Jay Srinivasan53173b92013-05-17 17:13:01 -0700433 .WillRepeatedly(Return(url1));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700434 fake_system_state_.mock_payload_state()->SetResponse(response);
Darin Petkov1b003102010-11-30 10:18:36 -0800435 attempter_.ScheduleErrorEventAction();
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700436 EXPECT_EQ(url1, fake_system_state_.mock_payload_state()->GetCurrentUrl());
Darin Petkov1b003102010-11-30 10:18:36 -0800437}
438
439TEST_F(UpdateAttempterTest, ScheduleErrorEventActionTest) {
440 EXPECT_CALL(*processor_,
441 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700442 OmahaRequestAction::StaticType())));
443 EXPECT_CALL(*processor_, StartProcessing());
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700444 ErrorCode err = ErrorCode::kError;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700445 EXPECT_CALL(*fake_system_state_.mock_payload_state(), UpdateFailed(err));
Darin Petkov1b003102010-11-30 10:18:36 -0800446 attempter_.error_event_.reset(new OmahaEvent(OmahaEvent::kTypeUpdateComplete,
447 OmahaEvent::kResultError,
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800448 err));
Darin Petkov1b003102010-11-30 10:18:36 -0800449 attempter_.ScheduleErrorEventAction();
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700450 EXPECT_EQ(UpdateStatus::REPORTING_ERROR_EVENT, attempter_.status());
Darin Petkov1b003102010-11-30 10:18:36 -0800451}
452
Darin Petkove6ef2f82011-03-07 17:31:11 -0800453namespace {
Chris Sosa76a29ae2013-07-11 17:59:24 -0700454// Actions that will be built as part of an update check.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700455const string kUpdateActionTypes[] = { // NOLINT(runtime/string)
Darin Petkove6ef2f82011-03-07 17:31:11 -0800456 OmahaRequestAction::StaticType(),
457 OmahaResponseHandlerAction::StaticType(),
Darin Petkove6ef2f82011-03-07 17:31:11 -0800458 OmahaRequestAction::StaticType(),
459 DownloadAction::StaticType(),
460 OmahaRequestAction::StaticType(),
Allie Woodeb9e6d82015-04-17 13:55:30 -0700461 FilesystemVerifierAction::StaticType(),
Darin Petkove6ef2f82011-03-07 17:31:11 -0800462 PostinstallRunnerAction::StaticType(),
463 OmahaRequestAction::StaticType()
464};
Chris Sosa76a29ae2013-07-11 17:59:24 -0700465
466// Actions that will be built as part of a user-initiated rollback.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700467const string kRollbackActionTypes[] = { // NOLINT(runtime/string)
Chris Sosa76a29ae2013-07-11 17:59:24 -0700468 InstallPlanAction::StaticType(),
469 PostinstallRunnerAction::StaticType(),
470};
471
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700472} // namespace
Darin Petkove6ef2f82011-03-07 17:31:11 -0800473
474void UpdateAttempterTest::UpdateTestStart() {
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700475 attempter_.set_http_response_code(200);
Alex Deymo749ecf12014-10-21 20:06:57 -0700476
477 // Expect that the device policy is loaded by the UpdateAttempter at some
478 // point by calling RefreshDevicePolicy.
479 policy::MockDevicePolicy* device_policy = new policy::MockDevicePolicy();
480 attempter_.policy_provider_.reset(new policy::PolicyProvider(device_policy));
481 EXPECT_CALL(*device_policy, LoadPolicy())
482 .Times(testing::AtLeast(1)).WillRepeatedly(Return(true));
483
484 {
485 InSequence s;
486 for (size_t i = 0; i < arraysize(kUpdateActionTypes); ++i) {
487 EXPECT_CALL(*processor_,
488 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700489 kUpdateActionTypes[i])));
Alex Deymo749ecf12014-10-21 20:06:57 -0700490 }
Gilad Arnold74b5f552014-10-07 08:17:16 -0700491 EXPECT_CALL(*processor_, StartProcessing());
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700492 }
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700493
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700494 attempter_.Update("", "", "", "", false, false);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700495 loop_.PostTask(FROM_HERE,
496 base::Bind(&UpdateAttempterTest::UpdateTestVerify,
497 base::Unretained(this)));
Darin Petkove6ef2f82011-03-07 17:31:11 -0800498}
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700499
Darin Petkove6ef2f82011-03-07 17:31:11 -0800500void UpdateAttempterTest::UpdateTestVerify() {
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700501 EXPECT_EQ(0, attempter_.http_response_code());
502 EXPECT_EQ(&attempter_, processor_->delegate());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700503 EXPECT_EQ(arraysize(kUpdateActionTypes), attempter_.actions_.size());
504 for (size_t i = 0; i < arraysize(kUpdateActionTypes); ++i) {
505 EXPECT_EQ(kUpdateActionTypes[i], attempter_.actions_[i]->Type());
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700506 }
507 EXPECT_EQ(attempter_.response_handler_action_.get(),
508 attempter_.actions_[1].get());
Sen Jiangfef85fd2016-03-25 15:32:49 -0700509 AbstractAction* action_3 = attempter_.actions_[3].get();
510 ASSERT_NE(nullptr, action_3);
511 ASSERT_EQ(DownloadAction::StaticType(), action_3->Type());
512 DownloadAction* download_action = static_cast<DownloadAction*>(action_3);
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700513 EXPECT_EQ(&attempter_, download_action->delegate());
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700514 EXPECT_EQ(UpdateStatus::CHECKING_FOR_UPDATE, attempter_.status());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700515 loop_.BreakLoop();
Darin Petkove6ef2f82011-03-07 17:31:11 -0800516}
517
Chris Sosa28e479c2013-07-12 11:39:53 -0700518void UpdateAttempterTest::RollbackTestStart(
Chris Sosa44b9b7e2014-04-02 13:53:46 -0700519 bool enterprise_rollback, bool valid_slot) {
Chris Sosa76a29ae2013-07-11 17:59:24 -0700520 // Create a device policy so that we can change settings.
521 policy::MockDevicePolicy* device_policy = new policy::MockDevicePolicy();
522 attempter_.policy_provider_.reset(new policy::PolicyProvider(device_policy));
523
524 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700525 fake_system_state_.set_device_policy(device_policy);
Chris Sosa76a29ae2013-07-11 17:59:24 -0700526
Alex Deymo763e7db2015-08-27 21:08:08 -0700527 if (valid_slot) {
528 BootControlInterface::Slot rollback_slot = 1;
529 LOG(INFO) << "Test Mark Bootable: "
530 << BootControlInterface::SlotName(rollback_slot);
Alex Deymoe5e5fe92015-10-05 09:28:19 -0700531 fake_system_state_.fake_boot_control()->SetSlotBootable(rollback_slot,
532 true);
Don Garrett6646b442013-11-13 15:29:11 -0800533 }
534
Chris Sosa28e479c2013-07-12 11:39:53 -0700535 bool is_rollback_allowed = false;
Chris Sosa76a29ae2013-07-11 17:59:24 -0700536
Chris Sosad38b1132014-03-25 10:43:59 -0700537 // We only allow rollback on devices that are not enterprise enrolled and
538 // which have a valid slot to rollback to.
539 if (!enterprise_rollback && valid_slot) {
Chris Sosa28e479c2013-07-12 11:39:53 -0700540 is_rollback_allowed = true;
541 }
542
Don Garrett6646b442013-11-13 15:29:11 -0800543 if (enterprise_rollback) {
Chris Sosa28e479c2013-07-12 11:39:53 -0700544 // We return an empty owner as this is an enterprise.
Don Garrett6646b442013-11-13 15:29:11 -0800545 EXPECT_CALL(*device_policy, GetOwner(_)).WillRepeatedly(
Alex Deymof329b932014-10-30 01:37:48 -0700546 DoAll(SetArgumentPointee<0>(string("")),
Chris Sosa28e479c2013-07-12 11:39:53 -0700547 Return(true)));
548 } else {
549 // We return a fake owner as this is an owned consumer device.
Don Garrett6646b442013-11-13 15:29:11 -0800550 EXPECT_CALL(*device_policy, GetOwner(_)).WillRepeatedly(
Alex Deymof329b932014-10-30 01:37:48 -0700551 DoAll(SetArgumentPointee<0>(string("fake.mail@fake.com")),
Chris Sosa76a29ae2013-07-11 17:59:24 -0700552 Return(true)));
Chris Sosa28e479c2013-07-12 11:39:53 -0700553 }
Chris Sosa76a29ae2013-07-11 17:59:24 -0700554
Chris Sosa28e479c2013-07-12 11:39:53 -0700555 if (is_rollback_allowed) {
Chris Sosa76a29ae2013-07-11 17:59:24 -0700556 InSequence s;
557 for (size_t i = 0; i < arraysize(kRollbackActionTypes); ++i) {
558 EXPECT_CALL(*processor_,
559 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700560 kRollbackActionTypes[i])));
Chris Sosa76a29ae2013-07-11 17:59:24 -0700561 }
Gilad Arnold74b5f552014-10-07 08:17:16 -0700562 EXPECT_CALL(*processor_, StartProcessing());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700563
Chris Sosa44b9b7e2014-04-02 13:53:46 -0700564 EXPECT_TRUE(attempter_.Rollback(true));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700565 loop_.PostTask(FROM_HERE,
566 base::Bind(&UpdateAttempterTest::RollbackTestVerify,
567 base::Unretained(this)));
Chris Sosa76a29ae2013-07-11 17:59:24 -0700568 } else {
Chris Sosa44b9b7e2014-04-02 13:53:46 -0700569 EXPECT_FALSE(attempter_.Rollback(true));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700570 loop_.BreakLoop();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700571 }
572}
573
574void UpdateAttempterTest::RollbackTestVerify() {
575 // Verifies the actions that were enqueued.
576 EXPECT_EQ(&attempter_, processor_->delegate());
577 EXPECT_EQ(arraysize(kRollbackActionTypes), attempter_.actions_.size());
578 for (size_t i = 0; i < arraysize(kRollbackActionTypes); ++i) {
579 EXPECT_EQ(kRollbackActionTypes[i], attempter_.actions_[i]->Type());
580 }
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700581 EXPECT_EQ(UpdateStatus::ATTEMPTING_ROLLBACK, attempter_.status());
Alex Deymo80f70ff2016-02-10 16:08:11 -0800582 AbstractAction* action_0 = attempter_.actions_[0].get();
583 ASSERT_NE(nullptr, action_0);
584 ASSERT_EQ(InstallPlanAction::StaticType(), action_0->Type());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700585 InstallPlanAction* install_plan_action =
Alex Deymo80f70ff2016-02-10 16:08:11 -0800586 static_cast<InstallPlanAction*>(action_0);
Chris Sosa76a29ae2013-07-11 17:59:24 -0700587 InstallPlan* install_plan = install_plan_action->install_plan();
Alex Deymo80f70ff2016-02-10 16:08:11 -0800588 EXPECT_EQ(0U, install_plan->partitions.size());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700589 EXPECT_EQ(install_plan->powerwash_required, true);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700590 loop_.BreakLoop();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700591}
592
Darin Petkove6ef2f82011-03-07 17:31:11 -0800593TEST_F(UpdateAttempterTest, UpdateTest) {
Alex Deymo461b2592015-07-24 20:10:52 -0700594 UpdateTestStart();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700595 loop_.Run();
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700596}
597
Chris Sosa76a29ae2013-07-11 17:59:24 -0700598TEST_F(UpdateAttempterTest, RollbackTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700599 loop_.PostTask(FROM_HERE,
600 base::Bind(&UpdateAttempterTest::RollbackTestStart,
601 base::Unretained(this),
602 false, true));
603 loop_.Run();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700604}
605
Don Garrett6646b442013-11-13 15:29:11 -0800606TEST_F(UpdateAttempterTest, InvalidSlotRollbackTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700607 loop_.PostTask(FROM_HERE,
608 base::Bind(&UpdateAttempterTest::RollbackTestStart,
609 base::Unretained(this),
610 false, false));
611 loop_.Run();
Don Garrett6646b442013-11-13 15:29:11 -0800612}
613
Chris Sosa76a29ae2013-07-11 17:59:24 -0700614TEST_F(UpdateAttempterTest, EnterpriseRollbackTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700615 loop_.PostTask(FROM_HERE,
616 base::Bind(&UpdateAttempterTest::RollbackTestStart,
617 base::Unretained(this),
618 true, true));
619 loop_.Run();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700620}
621
Thieu Le116fda32011-04-19 11:01:54 -0700622void UpdateAttempterTest::PingOmahaTestStart() {
623 EXPECT_CALL(*processor_,
624 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700625 OmahaRequestAction::StaticType())));
626 EXPECT_CALL(*processor_, StartProcessing());
Thieu Le116fda32011-04-19 11:01:54 -0700627 attempter_.PingOmaha();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700628 ScheduleQuitMainLoop();
Thieu Le116fda32011-04-19 11:01:54 -0700629}
630
631TEST_F(UpdateAttempterTest, PingOmahaTest) {
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700632 EXPECT_FALSE(attempter_.waiting_for_scheduled_check_);
633 EXPECT_FALSE(attempter_.schedule_updates_called());
634 // Disable scheduling of subsequnet checks; we're using the DefaultPolicy in
635 // testing, which is more permissive than we want to handle here.
636 attempter_.DisableScheduleUpdates();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700637 loop_.PostTask(FROM_HERE,
638 base::Bind(&UpdateAttempterTest::PingOmahaTestStart,
639 base::Unretained(this)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700640 brillo::MessageLoopRunMaxIterations(&loop_, 100);
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700641 EXPECT_EQ(UpdateStatus::UPDATED_NEED_REBOOT, attempter_.status());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700642 EXPECT_TRUE(attempter_.schedule_updates_called());
Thieu Le116fda32011-04-19 11:01:54 -0700643}
644
Darin Petkov18c7bce2011-06-16 14:07:00 -0700645TEST_F(UpdateAttempterTest, CreatePendingErrorEventTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -0800646 MockAction action;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700647 const ErrorCode kCode = ErrorCode::kDownloadTransferError;
Darin Petkov18c7bce2011-06-16 14:07:00 -0700648 attempter_.CreatePendingErrorEvent(&action, kCode);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700649 ASSERT_NE(nullptr, attempter_.error_event_.get());
Darin Petkov18c7bce2011-06-16 14:07:00 -0700650 EXPECT_EQ(OmahaEvent::kTypeUpdateComplete, attempter_.error_event_->type);
651 EXPECT_EQ(OmahaEvent::kResultError, attempter_.error_event_->result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700652 EXPECT_EQ(
653 static_cast<ErrorCode>(static_cast<int>(kCode) |
654 static_cast<int>(ErrorCode::kTestOmahaUrlFlag)),
655 attempter_.error_event_->error_code);
Darin Petkov18c7bce2011-06-16 14:07:00 -0700656}
657
658TEST_F(UpdateAttempterTest, CreatePendingErrorEventResumedTest) {
659 OmahaResponseHandlerAction *response_action =
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700660 new OmahaResponseHandlerAction(&fake_system_state_);
Darin Petkov18c7bce2011-06-16 14:07:00 -0700661 response_action->install_plan_.is_resume = true;
662 attempter_.response_handler_action_.reset(response_action);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800663 MockAction action;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700664 const ErrorCode kCode = ErrorCode::kInstallDeviceOpenError;
Darin Petkov18c7bce2011-06-16 14:07:00 -0700665 attempter_.CreatePendingErrorEvent(&action, kCode);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700666 ASSERT_NE(nullptr, attempter_.error_event_.get());
Darin Petkov18c7bce2011-06-16 14:07:00 -0700667 EXPECT_EQ(OmahaEvent::kTypeUpdateComplete, attempter_.error_event_->type);
668 EXPECT_EQ(OmahaEvent::kResultError, attempter_.error_event_->result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700669 EXPECT_EQ(
670 static_cast<ErrorCode>(
671 static_cast<int>(kCode) |
672 static_cast<int>(ErrorCode::kResumedFlag) |
673 static_cast<int>(ErrorCode::kTestOmahaUrlFlag)),
674 attempter_.error_event_->error_code);
Darin Petkov18c7bce2011-06-16 14:07:00 -0700675}
676
David Zeuthen8f191b22013-08-06 12:27:50 -0700677TEST_F(UpdateAttempterTest, P2PNotStartedAtStartupWhenNotEnabled) {
678 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700679 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700680 mock_p2p_manager.fake().SetP2PEnabled(false);
681 EXPECT_CALL(mock_p2p_manager, EnsureP2PRunning()).Times(0);
682 attempter_.UpdateEngineStarted();
683}
684
685TEST_F(UpdateAttempterTest, P2PNotStartedAtStartupWhenEnabledButNotSharing) {
686 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700687 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700688 mock_p2p_manager.fake().SetP2PEnabled(true);
689 EXPECT_CALL(mock_p2p_manager, EnsureP2PRunning()).Times(0);
690 attempter_.UpdateEngineStarted();
691}
692
693TEST_F(UpdateAttempterTest, P2PStartedAtStartupWhenEnabledAndSharing) {
694 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700695 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700696 mock_p2p_manager.fake().SetP2PEnabled(true);
697 mock_p2p_manager.fake().SetCountSharedFilesResult(1);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700698 EXPECT_CALL(mock_p2p_manager, EnsureP2PRunning());
David Zeuthen8f191b22013-08-06 12:27:50 -0700699 attempter_.UpdateEngineStarted();
700}
701
702TEST_F(UpdateAttempterTest, P2PNotEnabled) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700703 loop_.PostTask(FROM_HERE,
704 base::Bind(&UpdateAttempterTest::P2PNotEnabledStart,
705 base::Unretained(this)));
706 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700707}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700708
David Zeuthen8f191b22013-08-06 12:27:50 -0700709void UpdateAttempterTest::P2PNotEnabledStart() {
710 // If P2P is not enabled, check that we do not attempt housekeeping
711 // and do not convey that p2p is to be used.
712 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700713 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700714 mock_p2p_manager.fake().SetP2PEnabled(false);
715 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping()).Times(0);
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700716 attempter_.Update("", "", "", "", false, false);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700717 EXPECT_FALSE(actual_using_p2p_for_downloading_);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700718 EXPECT_FALSE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700719 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700720}
721
722TEST_F(UpdateAttempterTest, P2PEnabledStartingFails) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700723 loop_.PostTask(FROM_HERE,
724 base::Bind(&UpdateAttempterTest::P2PEnabledStartingFailsStart,
725 base::Unretained(this)));
726 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700727}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700728
David Zeuthen8f191b22013-08-06 12:27:50 -0700729void UpdateAttempterTest::P2PEnabledStartingFailsStart() {
730 // If p2p is enabled, but starting it fails ensure we don't do
731 // any housekeeping and do not convey that p2p should be used.
732 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700733 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700734 mock_p2p_manager.fake().SetP2PEnabled(true);
735 mock_p2p_manager.fake().SetEnsureP2PRunningResult(false);
736 mock_p2p_manager.fake().SetPerformHousekeepingResult(false);
737 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping()).Times(0);
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700738 attempter_.Update("", "", "", "", false, false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700739 EXPECT_FALSE(actual_using_p2p_for_downloading());
740 EXPECT_FALSE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700741 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700742}
743
744TEST_F(UpdateAttempterTest, P2PEnabledHousekeepingFails) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700745 loop_.PostTask(
746 FROM_HERE,
747 base::Bind(&UpdateAttempterTest::P2PEnabledHousekeepingFailsStart,
748 base::Unretained(this)));
749 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700750}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700751
David Zeuthen8f191b22013-08-06 12:27:50 -0700752void UpdateAttempterTest::P2PEnabledHousekeepingFailsStart() {
753 // If p2p is enabled, starting it works but housekeeping fails, ensure
754 // we do not convey p2p is to be used.
755 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700756 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700757 mock_p2p_manager.fake().SetP2PEnabled(true);
758 mock_p2p_manager.fake().SetEnsureP2PRunningResult(true);
759 mock_p2p_manager.fake().SetPerformHousekeepingResult(false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700760 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700761 attempter_.Update("", "", "", "", false, false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700762 EXPECT_FALSE(actual_using_p2p_for_downloading());
763 EXPECT_FALSE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700764 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700765}
766
767TEST_F(UpdateAttempterTest, P2PEnabled) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700768 loop_.PostTask(FROM_HERE,
769 base::Bind(&UpdateAttempterTest::P2PEnabledStart,
770 base::Unretained(this)));
771 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700772}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700773
David Zeuthen8f191b22013-08-06 12:27:50 -0700774void UpdateAttempterTest::P2PEnabledStart() {
775 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700776 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700777 // If P2P is enabled and starting it works, check that we performed
778 // housekeeping and that we convey p2p should be used.
779 mock_p2p_manager.fake().SetP2PEnabled(true);
780 mock_p2p_manager.fake().SetEnsureP2PRunningResult(true);
781 mock_p2p_manager.fake().SetPerformHousekeepingResult(true);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700782 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700783 attempter_.Update("", "", "", "", false, false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700784 EXPECT_TRUE(actual_using_p2p_for_downloading());
785 EXPECT_TRUE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700786 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700787}
788
789TEST_F(UpdateAttempterTest, P2PEnabledInteractive) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700790 loop_.PostTask(FROM_HERE,
791 base::Bind(&UpdateAttempterTest::P2PEnabledInteractiveStart,
792 base::Unretained(this)));
793 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700794}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700795
David Zeuthen8f191b22013-08-06 12:27:50 -0700796void UpdateAttempterTest::P2PEnabledInteractiveStart() {
797 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700798 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700799 // For an interactive check, if P2P is enabled and starting it
800 // works, check that we performed housekeeping and that we convey
801 // p2p should be used for sharing but NOT for downloading.
802 mock_p2p_manager.fake().SetP2PEnabled(true);
803 mock_p2p_manager.fake().SetEnsureP2PRunningResult(true);
804 mock_p2p_manager.fake().SetPerformHousekeepingResult(true);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700805 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700806 attempter_.Update("", "", "", "", false, true /* interactive */);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700807 EXPECT_FALSE(actual_using_p2p_for_downloading());
808 EXPECT_TRUE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700809 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700810}
811
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700812TEST_F(UpdateAttempterTest, ReadScatterFactorFromPolicy) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700813 loop_.PostTask(
814 FROM_HERE,
815 base::Bind(&UpdateAttempterTest::ReadScatterFactorFromPolicyTestStart,
816 base::Unretained(this)));
817 loop_.Run();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700818}
819
820// Tests that the scatter_factor_in_seconds value is properly fetched
821// from the device policy.
822void UpdateAttempterTest::ReadScatterFactorFromPolicyTestStart() {
Ben Chan9abb7632014-08-07 00:10:53 -0700823 int64_t scatter_factor_in_seconds = 36000;
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700824
825 policy::MockDevicePolicy* device_policy = new policy::MockDevicePolicy();
826 attempter_.policy_provider_.reset(new policy::PolicyProvider(device_policy));
827
828 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700829 fake_system_state_.set_device_policy(device_policy);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700830
831 EXPECT_CALL(*device_policy, GetScatterFactorInSeconds(_))
832 .WillRepeatedly(DoAll(
833 SetArgumentPointee<0>(scatter_factor_in_seconds),
834 Return(true)));
835
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700836 attempter_.Update("", "", "", "", false, false);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700837 EXPECT_EQ(scatter_factor_in_seconds, attempter_.scatter_factor_.InSeconds());
838
Alex Deymo60ca1a72015-06-18 18:19:15 -0700839 ScheduleQuitMainLoop();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700840}
841
842TEST_F(UpdateAttempterTest, DecrementUpdateCheckCountTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700843 loop_.PostTask(
844 FROM_HERE,
845 base::Bind(&UpdateAttempterTest::DecrementUpdateCheckCountTestStart,
846 base::Unretained(this)));
847 loop_.Run();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700848}
849
850void UpdateAttempterTest::DecrementUpdateCheckCountTestStart() {
851 // Tests that the scatter_factor_in_seconds value is properly fetched
852 // from the device policy and is decremented if value > 0.
Ben Chan9abb7632014-08-07 00:10:53 -0700853 int64_t initial_value = 5;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800854 FakePrefs fake_prefs;
855 attempter_.prefs_ = &fake_prefs;
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700856
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700857 fake_system_state_.fake_hardware()->SetIsOOBEComplete(Time::UnixEpoch());
Jay Srinivasan08fce042012-06-07 16:31:01 -0700858
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800859 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsUpdateCheckCount, initial_value));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700860
Ben Chan9abb7632014-08-07 00:10:53 -0700861 int64_t scatter_factor_in_seconds = 10;
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700862
863 policy::MockDevicePolicy* device_policy = new policy::MockDevicePolicy();
864 attempter_.policy_provider_.reset(new policy::PolicyProvider(device_policy));
865
866 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700867 fake_system_state_.set_device_policy(device_policy);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700868
869 EXPECT_CALL(*device_policy, GetScatterFactorInSeconds(_))
870 .WillRepeatedly(DoAll(
871 SetArgumentPointee<0>(scatter_factor_in_seconds),
872 Return(true)));
873
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700874 attempter_.Update("", "", "", "", false, false);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700875 EXPECT_EQ(scatter_factor_in_seconds, attempter_.scatter_factor_.InSeconds());
876
877 // Make sure the file still exists.
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800878 EXPECT_TRUE(fake_prefs.Exists(kPrefsUpdateCheckCount));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700879
Ben Chan9abb7632014-08-07 00:10:53 -0700880 int64_t new_value;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800881 EXPECT_TRUE(fake_prefs.GetInt64(kPrefsUpdateCheckCount, &new_value));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700882 EXPECT_EQ(initial_value - 1, new_value);
883
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700884 EXPECT_TRUE(
885 attempter_.omaha_request_params_->update_check_count_wait_enabled());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700886
887 // However, if the count is already 0, it's not decremented. Test that.
888 initial_value = 0;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800889 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsUpdateCheckCount, initial_value));
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700890 attempter_.Update("", "", "", "", false, false);
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800891 EXPECT_TRUE(fake_prefs.Exists(kPrefsUpdateCheckCount));
892 EXPECT_TRUE(fake_prefs.GetInt64(kPrefsUpdateCheckCount, &new_value));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700893 EXPECT_EQ(initial_value, new_value);
894
Alex Deymo60ca1a72015-06-18 18:19:15 -0700895 ScheduleQuitMainLoop();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700896}
897
Jay Srinivasan08fce042012-06-07 16:31:01 -0700898TEST_F(UpdateAttempterTest, NoScatteringDoneDuringManualUpdateTestStart) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700899 loop_.PostTask(FROM_HERE, base::Bind(
900 &UpdateAttempterTest::NoScatteringDoneDuringManualUpdateTestStart,
901 base::Unretained(this)));
902 loop_.Run();
Jay Srinivasan08fce042012-06-07 16:31:01 -0700903}
904
905void UpdateAttempterTest::NoScatteringDoneDuringManualUpdateTestStart() {
906 // Tests that no scattering logic is enabled if the update check
907 // is manually done (as opposed to a scheduled update check)
Ben Chan9abb7632014-08-07 00:10:53 -0700908 int64_t initial_value = 8;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800909 FakePrefs fake_prefs;
910 attempter_.prefs_ = &fake_prefs;
Jay Srinivasan08fce042012-06-07 16:31:01 -0700911
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700912 fake_system_state_.fake_hardware()->SetIsOOBEComplete(Time::UnixEpoch());
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800913 fake_system_state_.set_prefs(&fake_prefs);
Jay Srinivasan08fce042012-06-07 16:31:01 -0700914
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800915 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsWallClockWaitPeriod, initial_value));
916 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsUpdateCheckCount, initial_value));
Jay Srinivasan08fce042012-06-07 16:31:01 -0700917
918 // make sure scatter_factor is non-zero as scattering is disabled
919 // otherwise.
Ben Chan9abb7632014-08-07 00:10:53 -0700920 int64_t scatter_factor_in_seconds = 50;
Jay Srinivasan08fce042012-06-07 16:31:01 -0700921
922 policy::MockDevicePolicy* device_policy = new policy::MockDevicePolicy();
923 attempter_.policy_provider_.reset(new policy::PolicyProvider(device_policy));
924
925 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700926 fake_system_state_.set_device_policy(device_policy);
Jay Srinivasan08fce042012-06-07 16:31:01 -0700927
928 EXPECT_CALL(*device_policy, GetScatterFactorInSeconds(_))
929 .WillRepeatedly(DoAll(
930 SetArgumentPointee<0>(scatter_factor_in_seconds),
931 Return(true)));
932
Gilad Arnoldb92f0df2013-01-10 16:32:45 -0800933 // Trigger an interactive check so we can test that scattering is disabled.
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700934 attempter_.Update("", "", "", "", false, true);
Jay Srinivasan08fce042012-06-07 16:31:01 -0700935 EXPECT_EQ(scatter_factor_in_seconds, attempter_.scatter_factor_.InSeconds());
936
937 // Make sure scattering is disabled for manual (i.e. user initiated) update
Jay Srinivasan21be0752012-07-25 15:44:56 -0700938 // checks and all artifacts are removed.
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700939 EXPECT_FALSE(
940 attempter_.omaha_request_params_->wall_clock_based_wait_enabled());
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800941 EXPECT_FALSE(fake_prefs.Exists(kPrefsWallClockWaitPeriod));
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700942 EXPECT_EQ(0, attempter_.omaha_request_params_->waiting_period().InSeconds());
943 EXPECT_FALSE(
944 attempter_.omaha_request_params_->update_check_count_wait_enabled());
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800945 EXPECT_FALSE(fake_prefs.Exists(kPrefsUpdateCheckCount));
Jay Srinivasan08fce042012-06-07 16:31:01 -0700946
Alex Deymo60ca1a72015-06-18 18:19:15 -0700947 ScheduleQuitMainLoop();
Jay Srinivasan08fce042012-06-07 16:31:01 -0700948}
949
David Zeuthen985b1122013-10-09 12:13:15 -0700950// Checks that we only report daily metrics at most every 24 hours.
951TEST_F(UpdateAttempterTest, ReportDailyMetrics) {
952 FakeClock fake_clock;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800953 FakePrefs fake_prefs;
David Zeuthen985b1122013-10-09 12:13:15 -0700954
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700955 fake_system_state_.set_clock(&fake_clock);
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800956 fake_system_state_.set_prefs(&fake_prefs);
David Zeuthen985b1122013-10-09 12:13:15 -0700957
958 Time epoch = Time::FromInternalValue(0);
959 fake_clock.SetWallclockTime(epoch);
960
961 // If there is no kPrefsDailyMetricsLastReportedAt state variable,
962 // we should report.
963 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
964 // We should not report again if no time has passed.
965 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
966
967 // We should not report if only 10 hours has passed.
968 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(10));
969 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
970
971 // We should not report if only 24 hours - 1 sec has passed.
972 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(24) -
973 TimeDelta::FromSeconds(1));
974 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
975
976 // We should report if 24 hours has passed.
977 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(24));
978 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
979
980 // But then we should not report again..
981 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
982
983 // .. until another 24 hours has passed
984 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(47));
985 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
986 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(48));
987 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
988 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
989
990 // .. and another 24 hours
991 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(71));
992 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
993 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(72));
994 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
995 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
996
997 // If the span between time of reporting and present time is
998 // negative, we report. This is in order to reset the timestamp and
999 // avoid an edge condition whereby a distant point in the future is
1000 // in the state variable resulting in us never ever reporting again.
1001 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(71));
1002 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
1003 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
1004
1005 // In this case we should not update until the clock reads 71 + 24 = 95.
1006 // Check that.
1007 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(94));
1008 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
1009 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(95));
1010 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
1011 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
David Zeuthen985b1122013-10-09 12:13:15 -07001012}
1013
David Zeuthen3c55abd2013-10-14 12:48:03 -07001014TEST_F(UpdateAttempterTest, BootTimeInUpdateMarkerFile) {
David Zeuthen3c55abd2013-10-14 12:48:03 -07001015 FakeClock fake_clock;
1016 fake_clock.SetBootTime(Time::FromTimeT(42));
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001017 fake_system_state_.set_clock(&fake_clock);
Alex Deymo906191f2015-10-12 12:22:44 -07001018 FakePrefs fake_prefs;
1019 fake_system_state_.set_prefs(&fake_prefs);
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001020 attempter_.Init();
David Zeuthen3c55abd2013-10-14 12:48:03 -07001021
1022 Time boot_time;
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001023 EXPECT_FALSE(attempter_.GetBootTimeAtUpdate(&boot_time));
David Zeuthen3c55abd2013-10-14 12:48:03 -07001024
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001025 attempter_.WriteUpdateCompletedMarker();
David Zeuthen3c55abd2013-10-14 12:48:03 -07001026
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001027 EXPECT_TRUE(attempter_.GetBootTimeAtUpdate(&boot_time));
David Zeuthen3c55abd2013-10-14 12:48:03 -07001028 EXPECT_EQ(boot_time.ToTimeT(), 42);
1029}
1030
David Pursell02c18642014-11-06 11:26:11 -08001031TEST_F(UpdateAttempterTest, AnyUpdateSourceAllowedUnofficial) {
1032 fake_system_state_.fake_hardware()->SetIsOfficialBuild(false);
1033 EXPECT_TRUE(attempter_.IsAnyUpdateSourceAllowed());
1034}
1035
1036TEST_F(UpdateAttempterTest, AnyUpdateSourceAllowedOfficialDevmode) {
1037 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001038 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(true);
David Pursell02c18642014-11-06 11:26:11 -08001039 EXPECT_TRUE(attempter_.IsAnyUpdateSourceAllowed());
1040}
1041
1042TEST_F(UpdateAttempterTest, AnyUpdateSourceDisallowedOfficialNormal) {
1043 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001044 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(false);
David Pursell02c18642014-11-06 11:26:11 -08001045 EXPECT_FALSE(attempter_.IsAnyUpdateSourceAllowed());
1046}
1047
1048TEST_F(UpdateAttempterTest, CheckForUpdateAUTest) {
1049 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001050 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(false);
Aaron Wood3a9c9622017-10-19 17:14:58 -07001051 attempter_.CheckForUpdate("", "autest", UpdateAttemptFlags::kNone);
Alex Deymoac41a822015-09-15 20:52:53 -07001052 EXPECT_EQ(constants::kOmahaDefaultAUTestURL, attempter_.forced_omaha_url());
David Pursell02c18642014-11-06 11:26:11 -08001053}
1054
1055TEST_F(UpdateAttempterTest, CheckForUpdateScheduledAUTest) {
1056 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001057 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(false);
Aaron Wood3a9c9622017-10-19 17:14:58 -07001058 attempter_.CheckForUpdate("", "autest-scheduled", UpdateAttemptFlags::kNone);
Alex Deymoac41a822015-09-15 20:52:53 -07001059 EXPECT_EQ(constants::kOmahaDefaultAUTestURL, attempter_.forced_omaha_url());
David Pursell02c18642014-11-06 11:26:11 -08001060}
1061
Xiyuan Xiac0e8f9a2017-02-22 13:19:35 -08001062TEST_F(UpdateAttempterTest, TargetVersionPrefixSetAndReset) {
1063 attempter_.CalculateUpdateParams("", "", "", "1234", false, false);
1064 EXPECT_EQ("1234",
1065 fake_system_state_.request_params()->target_version_prefix());
1066
1067 attempter_.CalculateUpdateParams("", "", "", "", false, false);
1068 EXPECT_TRUE(
1069 fake_system_state_.request_params()->target_version_prefix().empty());
1070}
1071
Aaron Wood987ffc12017-10-06 14:48:25 -07001072TEST_F(UpdateAttempterTest, UpdateDeferredByPolicyTest) {
1073 // Construct an OmahaResponseHandlerAction that has processed an InstallPlan,
1074 // but the update is being deferred by the Policy.
1075 OmahaResponseHandlerAction* response_action =
1076 new OmahaResponseHandlerAction(&fake_system_state_);
1077 response_action->install_plan_.version = "a.b.c.d";
1078 response_action->install_plan_.system_version = "b.c.d.e";
1079 response_action->install_plan_.payloads.push_back(
1080 {.size = 1234ULL, .type = InstallPayloadType::kFull});
1081 attempter_.response_handler_action_.reset(response_action);
1082 // Inform the UpdateAttempter that the OmahaResponseHandlerAction has
1083 // completed, with the deferred-update error code.
1084 attempter_.ActionCompleted(
1085 nullptr, response_action, ErrorCode::kOmahaUpdateDeferredPerPolicy);
1086 {
1087 UpdateEngineStatus status;
1088 attempter_.GetStatus(&status);
1089 EXPECT_EQ(UpdateStatus::UPDATE_AVAILABLE, status.status);
1090 EXPECT_EQ(response_action->install_plan_.version, status.new_version);
1091 EXPECT_EQ(response_action->install_plan_.system_version,
1092 status.new_system_version);
1093 EXPECT_EQ(response_action->install_plan_.payloads[0].size,
1094 status.new_size_bytes);
1095 }
1096 // An "error" event should have been created to tell Omaha that the update is
1097 // being deferred.
1098 EXPECT_TRUE(nullptr != attempter_.error_event_);
1099 EXPECT_EQ(OmahaEvent::kTypeUpdateComplete, attempter_.error_event_->type);
1100 EXPECT_EQ(OmahaEvent::kResultUpdateDeferred, attempter_.error_event_->result);
1101 ErrorCode expected_code = static_cast<ErrorCode>(
1102 static_cast<int>(ErrorCode::kOmahaUpdateDeferredPerPolicy) |
1103 static_cast<int>(ErrorCode::kTestOmahaUrlFlag));
1104 EXPECT_EQ(expected_code, attempter_.error_event_->error_code);
1105 // End the processing
1106 attempter_.ProcessingDone(nullptr, ErrorCode::kOmahaUpdateDeferredPerPolicy);
1107 // Validate the state of the attempter.
1108 {
1109 UpdateEngineStatus status;
1110 attempter_.GetStatus(&status);
1111 EXPECT_EQ(UpdateStatus::REPORTING_ERROR_EVENT, status.status);
1112 EXPECT_EQ(response_action->install_plan_.version, status.new_version);
1113 EXPECT_EQ(response_action->install_plan_.system_version,
1114 status.new_system_version);
1115 EXPECT_EQ(response_action->install_plan_.payloads[0].size,
1116 status.new_size_bytes);
1117 }
1118}
1119
1120TEST_F(UpdateAttempterTest, UpdateIsNotRunningWhenUpdateAvailable) {
1121 EXPECT_FALSE(attempter_.IsUpdateRunningOrScheduled());
1122 // Verify in-progress update with UPDATE_AVAILABLE is running
1123 attempter_.status_ = UpdateStatus::UPDATE_AVAILABLE;
1124 EXPECT_TRUE(attempter_.IsUpdateRunningOrScheduled());
1125}
1126
Aaron Wood224dfc22017-10-04 10:58:36 -07001127TEST_F(UpdateAttempterTest, UpdateAttemptFlagsCachedAtUpdateStart) {
1128 attempter_.SetUpdateAttemptFlags(UpdateAttemptFlags::kFlagRestrictDownload);
1129
1130 UpdateCheckParams params = {.updates_enabled = true};
1131 attempter_.OnUpdateScheduled(EvalStatus::kSucceeded, params);
1132
1133 EXPECT_EQ(UpdateAttemptFlags::kFlagRestrictDownload,
1134 attempter_.GetCurrentUpdateAttemptFlags());
1135}
1136
Aaron Wood3a9c9622017-10-19 17:14:58 -07001137TEST_F(UpdateAttempterTest, InteractiveUpdateUsesPassedRestrictions) {
1138 attempter_.SetUpdateAttemptFlags(UpdateAttemptFlags::kFlagRestrictDownload);
1139
1140 attempter_.CheckForUpdate("", "", UpdateAttemptFlags::kNone);
1141 EXPECT_EQ(UpdateAttemptFlags::kNone,
1142 attempter_.GetCurrentUpdateAttemptFlags());
1143}
1144
1145TEST_F(UpdateAttempterTest, NonInteractiveUpdateUsesSetRestrictions) {
1146 attempter_.SetUpdateAttemptFlags(UpdateAttemptFlags::kNone);
1147
1148 // This tests that when CheckForUpdate() is called with the non-interactive
1149 // flag set, that it doesn't change the current UpdateAttemptFlags.
1150 attempter_.CheckForUpdate("",
1151 "",
1152 UpdateAttemptFlags::kFlagNonInteractive |
1153 UpdateAttemptFlags::kFlagRestrictDownload);
1154 EXPECT_EQ(UpdateAttemptFlags::kNone,
1155 attempter_.GetCurrentUpdateAttemptFlags());
1156}
1157
Darin Petkovf42cc1c2010-09-01 09:03:02 -07001158} // namespace chromeos_update_engine