blob: dd61fa57ec5523a761e22217badee4ad821807c4 [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>
Amin Hassani71818c82018-03-06 13:25:40 -080025#include <brillo/bind_lambda.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070026#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
Alex Deymo39910dc2015-11-09 17:04:30 -080033#include "update_engine/common/fake_clock.h"
34#include "update_engine/common/fake_prefs.h"
Alex Deymo14fd1ec2016-02-24 22:03:57 -080035#include "update_engine/common/mock_action.h"
36#include "update_engine/common/mock_action_processor.h"
Alex Deymo39910dc2015-11-09 17:04:30 -080037#include "update_engine/common/mock_http_fetcher.h"
38#include "update_engine/common/mock_prefs.h"
39#include "update_engine/common/platform_constants.h"
40#include "update_engine/common/prefs.h"
41#include "update_engine/common/test_utils.h"
42#include "update_engine/common/utils.h"
Gilad Arnold5bb4c902014-04-10 12:32:13 -070043#include "update_engine/fake_system_state.h"
David Zeuthen8f191b22013-08-06 12:27:50 -070044#include "update_engine/mock_p2p_manager.h"
Jay Srinivasan6f6ea002012-12-14 11:26:28 -080045#include "update_engine/mock_payload_state.h"
Aaron Wood9321f502017-09-07 11:18:54 -070046#include "update_engine/mock_service_observer.h"
Alex Deymo39910dc2015-11-09 17:04:30 -080047#include "update_engine/payload_consumer/filesystem_verifier_action.h"
48#include "update_engine/payload_consumer/install_plan.h"
49#include "update_engine/payload_consumer/payload_constants.h"
50#include "update_engine/payload_consumer/postinstall_runner_action.h"
Darin Petkovf42cc1c2010-09-01 09:03:02 -070051
David Zeuthen985b1122013-10-09 12:13:15 -070052using base::Time;
53using base::TimeDelta;
Aaron Woodbf5a2522017-10-04 10:58:36 -070054using chromeos_update_manager::EvalStatus;
55using chromeos_update_manager::UpdateCheckParams;
Darin Petkovf42cc1c2010-09-01 09:03:02 -070056using std::string;
Ben Chan02f7c1d2014-10-18 15:18:02 -070057using std::unique_ptr;
Aaron Woodbf5a2522017-10-04 10:58:36 -070058using testing::_;
Darin Petkov36275772010-10-01 11:40:57 -070059using testing::DoAll;
Aaron Wood7f92e2b2017-08-28 14:51:21 -070060using testing::Field;
Darin Petkovf42cc1c2010-09-01 09:03:02 -070061using testing::InSequence;
Darin Petkov2dd01092010-10-08 15:43:05 -070062using testing::Ne;
Darin Petkov9c096d62010-11-17 14:49:04 -080063using testing::NiceMock;
Darin Petkovf42cc1c2010-09-01 09:03:02 -070064using testing::Property;
65using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070066using testing::ReturnPointee;
Alex Deymo2c0db7b2014-11-04 12:23:39 -080067using testing::SaveArg;
Ben Chan672c1f52017-10-23 15:41:39 -070068using testing::SetArgPointee;
Aaron Woodbf5a2522017-10-04 10:58:36 -070069using update_engine::UpdateAttemptFlags;
Aaron Wood7f92e2b2017-08-28 14:51:21 -070070using update_engine::UpdateEngineStatus;
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -070071using update_engine::UpdateStatus;
Darin Petkovf42cc1c2010-09-01 09:03:02 -070072
73namespace chromeos_update_engine {
74
75// Test a subclass rather than the main class directly so that we can mock out
Darin Petkovcd1666f2010-09-23 09:53:44 -070076// methods within the class. There're explicit unit tests for the mocked out
Darin Petkovf42cc1c2010-09-01 09:03:02 -070077// methods.
78class UpdateAttempterUnderTest : public UpdateAttempter {
79 public:
Jeffrey Kardatzkecf5f1f12017-10-02 16:08:44 -070080 explicit UpdateAttempterUnderTest(SystemState* system_state)
81 : UpdateAttempter(system_state, nullptr) {}
Gilad Arnoldec7f9162014-07-15 13:24:46 -070082
83 // Wrap the update scheduling method, allowing us to opt out of scheduled
84 // updates for testing purposes.
85 void ScheduleUpdates() override {
86 schedule_updates_called_ = true;
87 if (do_schedule_updates_) {
88 UpdateAttempter::ScheduleUpdates();
89 } else {
90 LOG(INFO) << "[TEST] Update scheduling disabled.";
91 }
92 }
93 void EnableScheduleUpdates() { do_schedule_updates_ = true; }
94 void DisableScheduleUpdates() { do_schedule_updates_ = false; }
95
96 // Indicates whether ScheduleUpdates() was called.
97 bool schedule_updates_called() const { return schedule_updates_called_; }
98
David Pursell02c18642014-11-06 11:26:11 -080099 // Need to expose forced_omaha_url_ so we can test it.
Alex Deymo60ca1a72015-06-18 18:19:15 -0700100 const string& forced_omaha_url() const { return forced_omaha_url_; }
David Pursell02c18642014-11-06 11:26:11 -0800101
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700102 private:
103 bool schedule_updates_called_ = false;
104 bool do_schedule_updates_ = true;
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700105};
106
107class UpdateAttempterTest : public ::testing::Test {
108 protected:
Jay Srinivasan43488792012-06-19 00:25:31 -0700109 UpdateAttempterTest()
Daniel Erate5f6f252017-04-20 12:09:58 -0600110 : certificate_checker_(fake_system_state_.mock_prefs(),
Alex Deymo33e91e72015-12-01 18:26:08 -0300111 &openssl_wrapper_) {
Gilad Arnold1f847232014-04-07 12:07:49 -0700112 // Override system state members.
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700113 fake_system_state_.set_connection_manager(&mock_connection_manager);
114 fake_system_state_.set_update_attempter(&attempter_);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700115 loop_.SetAsCurrent();
Gilad Arnold1f847232014-04-07 12:07:49 -0700116
Alex Deymo33e91e72015-12-01 18:26:08 -0300117 certificate_checker_.Init();
118
Sen Jiange67bb5b2016-06-20 15:53:56 -0700119 // Finish initializing the attempter.
Gilad Arnold1f847232014-04-07 12:07:49 -0700120 attempter_.Init();
Jay Srinivasan43488792012-06-19 00:25:31 -0700121 }
Gilad Arnoldeff87cc2013-07-22 18:32:09 -0700122
Alex Deymo610277e2014-11-11 21:18:11 -0800123 void SetUp() override {
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700124 EXPECT_NE(nullptr, attempter_.system_state_);
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700125 EXPECT_EQ(0, attempter_.http_response_code_);
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700126 EXPECT_EQ(UpdateStatus::IDLE, attempter_.status_);
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700127 EXPECT_EQ(0.0, attempter_.download_progress_);
128 EXPECT_EQ(0, attempter_.last_checked_time_);
129 EXPECT_EQ("0.0.0.0", attempter_.new_version_);
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700130 EXPECT_EQ(0ULL, attempter_.new_payload_size_);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800131 processor_ = new NiceMock<MockActionProcessor>();
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700132 attempter_.processor_.reset(processor_); // Transfers ownership.
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700133 prefs_ = fake_system_state_.mock_prefs();
Gilad Arnold74b5f552014-10-07 08:17:16 -0700134
135 // Set up store/load semantics of P2P properties via the mock PayloadState.
136 actual_using_p2p_for_downloading_ = false;
137 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
138 SetUsingP2PForDownloading(_))
139 .WillRepeatedly(SaveArg<0>(&actual_using_p2p_for_downloading_));
140 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
141 GetUsingP2PForDownloading())
142 .WillRepeatedly(ReturnPointee(&actual_using_p2p_for_downloading_));
143 actual_using_p2p_for_sharing_ = false;
144 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
145 SetUsingP2PForSharing(_))
146 .WillRepeatedly(SaveArg<0>(&actual_using_p2p_for_sharing_));
147 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
148 GetUsingP2PForDownloading())
149 .WillRepeatedly(ReturnPointee(&actual_using_p2p_for_sharing_));
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700150 }
151
Alex Deymo60ca1a72015-06-18 18:19:15 -0700152 public:
153 void ScheduleQuitMainLoop();
Patrick Dubroy7fbbe8a2011-08-01 17:28:22 +0200154
Alex Deymo60ca1a72015-06-18 18:19:15 -0700155 // Callbacks to run the different tests from the main loop.
Darin Petkove6ef2f82011-03-07 17:31:11 -0800156 void UpdateTestStart();
157 void UpdateTestVerify();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700158 void RollbackTestStart(bool enterprise_rollback, bool valid_slot);
Chris Sosa76a29ae2013-07-11 17:59:24 -0700159 void RollbackTestVerify();
Thieu Le116fda32011-04-19 11:01:54 -0700160 void PingOmahaTestStart();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700161 void ReadScatterFactorFromPolicyTestStart();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700162 void DecrementUpdateCheckCountTestStart();
Jay Srinivasan08fce042012-06-07 16:31:01 -0700163 void NoScatteringDoneDuringManualUpdateTestStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700164 void P2PNotEnabledStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700165 void P2PEnabledStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700166 void P2PEnabledInteractiveStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700167 void P2PEnabledStartingFailsStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700168 void P2PEnabledHousekeepingFailsStart();
David Zeuthen8f191b22013-08-06 12:27:50 -0700169
Gilad Arnold74b5f552014-10-07 08:17:16 -0700170 bool actual_using_p2p_for_downloading() {
171 return actual_using_p2p_for_downloading_;
172 }
173 bool actual_using_p2p_for_sharing() {
174 return actual_using_p2p_for_sharing_;
175 }
176
Alex Deymo0b3db6b2015-08-10 15:19:37 -0700177 base::MessageLoopForIO base_loop_;
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700178 brillo::BaseMessageLoop loop_{&base_loop_};
Alex Deymo60ca1a72015-06-18 18:19:15 -0700179
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700180 FakeSystemState fake_system_state_;
Jeffrey Kardatzkecf5f1f12017-10-02 16:08:44 -0700181 UpdateAttempterUnderTest attempter_{&fake_system_state_};
Alex Deymo33e91e72015-12-01 18:26:08 -0300182 OpenSSLWrapper openssl_wrapper_;
183 CertificateChecker certificate_checker_;
Alex Deymo30534502015-07-20 15:06:33 -0700184
Alex Deymo8427b4a2014-11-05 14:00:32 -0800185 NiceMock<MockActionProcessor>* processor_;
186 NiceMock<MockPrefs>* prefs_; // Shortcut to fake_system_state_->mock_prefs().
Jay Srinivasan55f50c22013-01-10 19:24:35 -0800187 NiceMock<MockConnectionManager> mock_connection_manager;
Gilad Arnoldeff87cc2013-07-22 18:32:09 -0700188
Gilad Arnold74b5f552014-10-07 08:17:16 -0700189 bool actual_using_p2p_for_downloading_;
190 bool actual_using_p2p_for_sharing_;
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700191};
192
Alex Deymo60ca1a72015-06-18 18:19:15 -0700193void UpdateAttempterTest::ScheduleQuitMainLoop() {
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700194 loop_.PostTask(
195 FROM_HERE,
196 base::Bind([](brillo::BaseMessageLoop* loop) { loop->BreakLoop(); },
197 base::Unretained(&loop_)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700198}
199
Darin Petkov1b003102010-11-30 10:18:36 -0800200TEST_F(UpdateAttempterTest, ActionCompletedDownloadTest) {
Ben Chan02f7c1d2014-10-18 15:18:02 -0700201 unique_ptr<MockHttpFetcher> fetcher(new MockHttpFetcher("", 0, nullptr));
Darin Petkov1b003102010-11-30 10:18:36 -0800202 fetcher->FailTransfer(503); // Sets the HTTP response code.
Amin Hassani7ecda262017-07-11 17:10:50 -0700203 DownloadAction action(prefs_,
204 nullptr,
205 nullptr,
206 nullptr,
207 fetcher.release(),
208 false /* is_interactive */);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800209 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _)).Times(0);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700210 attempter_.ActionCompleted(nullptr, &action, ErrorCode::kSuccess);
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700211 EXPECT_EQ(UpdateStatus::FINALIZING, attempter_.status());
Aaron Wood9321f502017-09-07 11:18:54 -0700212 EXPECT_EQ(0.0, attempter_.download_progress_);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700213 ASSERT_EQ(nullptr, attempter_.error_event_.get());
Darin Petkov1b003102010-11-30 10:18:36 -0800214}
215
216TEST_F(UpdateAttempterTest, ActionCompletedErrorTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -0800217 MockAction action;
218 EXPECT_CALL(action, Type()).WillRepeatedly(Return("MockAction"));
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700219 attempter_.status_ = UpdateStatus::DOWNLOADING;
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800220 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov1b003102010-11-30 10:18:36 -0800221 .WillOnce(Return(false));
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700222 attempter_.ActionCompleted(nullptr, &action, ErrorCode::kError);
223 ASSERT_NE(nullptr, attempter_.error_event_.get());
Darin Petkov1b003102010-11-30 10:18:36 -0800224}
225
Aaron Wood9321f502017-09-07 11:18:54 -0700226TEST_F(UpdateAttempterTest, DownloadProgressAccumulationTest) {
227 // Simple test case, where all the values match (nothing was skipped)
228 uint64_t bytes_progressed_1 = 1024 * 1024; // 1MB
229 uint64_t bytes_progressed_2 = 1024 * 1024; // 1MB
230 uint64_t bytes_received_1 = bytes_progressed_1;
231 uint64_t bytes_received_2 = bytes_received_1 + bytes_progressed_2;
232 uint64_t bytes_total = 20 * 1024 * 1024; // 20MB
233
234 double progress_1 =
235 static_cast<double>(bytes_received_1) / static_cast<double>(bytes_total);
236 double progress_2 =
237 static_cast<double>(bytes_received_2) / static_cast<double>(bytes_total);
238
239 EXPECT_EQ(0.0, attempter_.download_progress_);
240 // This is set via inspecting the InstallPlan payloads when the
241 // OmahaResponseAction is completed
242 attempter_.new_payload_size_ = bytes_total;
243 NiceMock<MockServiceObserver> observer;
244 EXPECT_CALL(observer,
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700245 SendStatusUpdate(AllOf(
246 Field(&UpdateEngineStatus::progress, progress_1),
247 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
248 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700249 EXPECT_CALL(observer,
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700250 SendStatusUpdate(AllOf(
251 Field(&UpdateEngineStatus::progress, progress_2),
252 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
253 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700254 attempter_.AddObserver(&observer);
255 attempter_.BytesReceived(bytes_progressed_1, bytes_received_1, bytes_total);
256 EXPECT_EQ(progress_1, attempter_.download_progress_);
257 // This iteration validates that a later set of updates to the variables are
258 // properly handled (so that |getStatus()| will return the same progress info
259 // as the callback is receiving.
260 attempter_.BytesReceived(bytes_progressed_2, bytes_received_2, bytes_total);
261 EXPECT_EQ(progress_2, attempter_.download_progress_);
262}
263
264TEST_F(UpdateAttempterTest, ChangeToDownloadingOnReceivedBytesTest) {
265 // The transition into UpdateStatus::DOWNLOADING happens when the
266 // first bytes are received.
267 uint64_t bytes_progressed = 1024 * 1024; // 1MB
268 uint64_t bytes_received = 2 * 1024 * 1024; // 2MB
269 uint64_t bytes_total = 20 * 1024 * 1024; // 300MB
270 attempter_.status_ = UpdateStatus::CHECKING_FOR_UPDATE;
271 // This is set via inspecting the InstallPlan payloads when the
272 // OmahaResponseAction is completed
273 attempter_.new_payload_size_ = bytes_total;
274 EXPECT_EQ(0.0, attempter_.download_progress_);
275 NiceMock<MockServiceObserver> observer;
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700276 EXPECT_CALL(observer,
277 SendStatusUpdate(AllOf(
278 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
279 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700280 attempter_.AddObserver(&observer);
281 attempter_.BytesReceived(bytes_progressed, bytes_received, bytes_total);
282 EXPECT_EQ(UpdateStatus::DOWNLOADING, attempter_.status_);
283}
284
285TEST_F(UpdateAttempterTest, BroadcastCompleteDownloadTest) {
286 // There is a special case to ensure that at 100% downloaded,
287 // download_progress_ is updated and that value broadcast. This test confirms
288 // that.
289 uint64_t bytes_progressed = 0; // ignored
290 uint64_t bytes_received = 5 * 1024 * 1024; // ignored
291 uint64_t bytes_total = 5 * 1024 * 1024; // 300MB
292 attempter_.status_ = UpdateStatus::DOWNLOADING;
293 attempter_.new_payload_size_ = bytes_total;
294 EXPECT_EQ(0.0, attempter_.download_progress_);
295 NiceMock<MockServiceObserver> observer;
Aaron Wood7f92e2b2017-08-28 14:51:21 -0700296 EXPECT_CALL(observer,
297 SendStatusUpdate(AllOf(
298 Field(&UpdateEngineStatus::progress, 1.0),
299 Field(&UpdateEngineStatus::status, UpdateStatus::DOWNLOADING),
300 Field(&UpdateEngineStatus::new_size_bytes, bytes_total))));
Aaron Wood9321f502017-09-07 11:18:54 -0700301 attempter_.AddObserver(&observer);
302 attempter_.BytesReceived(bytes_progressed, bytes_received, bytes_total);
303 EXPECT_EQ(1.0, attempter_.download_progress_);
304}
305
Darin Petkov1b003102010-11-30 10:18:36 -0800306TEST_F(UpdateAttempterTest, ActionCompletedOmahaRequestTest) {
Ben Chan02f7c1d2014-10-18 15:18:02 -0700307 unique_ptr<MockHttpFetcher> fetcher(new MockHttpFetcher("", 0, nullptr));
Darin Petkov1b003102010-11-30 10:18:36 -0800308 fetcher->FailTransfer(500); // Sets the HTTP response code.
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700309 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300310 std::move(fetcher), false);
Darin Petkov1b003102010-11-30 10:18:36 -0800311 ObjectCollectorAction<OmahaResponse> collector_action;
312 BondActions(&action, &collector_action);
313 OmahaResponse response;
314 response.poll_interval = 234;
315 action.SetOutputObject(response);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800316 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _)).Times(0);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700317 attempter_.ActionCompleted(nullptr, &action, ErrorCode::kSuccess);
Darin Petkov1b003102010-11-30 10:18:36 -0800318 EXPECT_EQ(500, attempter_.http_response_code());
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700319 EXPECT_EQ(UpdateStatus::IDLE, attempter_.status());
Alex Deymo80f70ff2016-02-10 16:08:11 -0800320 EXPECT_EQ(234U, attempter_.server_dictated_poll_interval_);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700321 ASSERT_TRUE(attempter_.error_event_.get() == nullptr);
Darin Petkov1b003102010-11-30 10:18:36 -0800322}
323
Alex Deymo30534502015-07-20 15:06:33 -0700324TEST_F(UpdateAttempterTest, ConstructWithUpdatedMarkerTest) {
Alex Deymo906191f2015-10-12 12:22:44 -0700325 FakePrefs fake_prefs;
326 string boot_id;
327 EXPECT_TRUE(utils::GetBootId(&boot_id));
328 fake_prefs.SetString(kPrefsUpdateCompletedOnBootId, boot_id);
329 fake_system_state_.set_prefs(&fake_prefs);
Sen Jiangaeeb2e02016-06-09 15:00:16 -0700330 attempter_.Init();
331 EXPECT_EQ(UpdateStatus::UPDATED_NEED_REBOOT, attempter_.status());
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700332}
333
334TEST_F(UpdateAttempterTest, GetErrorCodeForActionTest) {
David Zeuthena99981f2013-04-29 13:42:47 -0700335 extern ErrorCode GetErrorCodeForAction(AbstractAction* action,
336 ErrorCode code);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700337 EXPECT_EQ(ErrorCode::kSuccess,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700338 GetErrorCodeForAction(nullptr, ErrorCode::kSuccess));
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700339
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700340 FakeSystemState fake_system_state;
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700341 OmahaRequestAction omaha_request_action(&fake_system_state, nullptr,
342 nullptr, false);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700343 EXPECT_EQ(ErrorCode::kOmahaRequestError,
344 GetErrorCodeForAction(&omaha_request_action, ErrorCode::kError));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700345 OmahaResponseHandlerAction omaha_response_handler_action(&fake_system_state_);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700346 EXPECT_EQ(ErrorCode::kOmahaResponseHandlerError,
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700347 GetErrorCodeForAction(&omaha_response_handler_action,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700348 ErrorCode::kError));
Sen Jiange6e4bb92016-04-05 14:59:12 -0700349 FilesystemVerifierAction filesystem_verifier_action;
Allie Woodeb9e6d82015-04-17 13:55:30 -0700350 EXPECT_EQ(ErrorCode::kFilesystemVerifierError,
351 GetErrorCodeForAction(&filesystem_verifier_action,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700352 ErrorCode::kError));
Alex Deymob15a0b82015-11-25 20:30:40 -0300353 PostinstallRunnerAction postinstall_runner_action(
Alex Deymofb905d92016-06-03 19:26:58 -0700354 fake_system_state.fake_boot_control(), fake_system_state.fake_hardware());
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700355 EXPECT_EQ(ErrorCode::kPostinstallRunnerError,
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700356 GetErrorCodeForAction(&postinstall_runner_action,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700357 ErrorCode::kError));
Alex Deymo8427b4a2014-11-05 14:00:32 -0800358 MockAction action_mock;
359 EXPECT_CALL(action_mock, Type()).WillOnce(Return("MockAction"));
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700360 EXPECT_EQ(ErrorCode::kError,
361 GetErrorCodeForAction(&action_mock, ErrorCode::kError));
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700362}
363
Darin Petkov36275772010-10-01 11:40:57 -0700364TEST_F(UpdateAttempterTest, DisableDeltaUpdateIfNeededTest) {
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700365 attempter_.omaha_request_params_->set_delta_okay(true);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800366 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700367 .WillOnce(Return(false));
368 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700369 EXPECT_TRUE(attempter_.omaha_request_params_->delta_okay());
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800370 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700371 .WillOnce(DoAll(
Ben Chan672c1f52017-10-23 15:41:39 -0700372 SetArgPointee<1>(UpdateAttempter::kMaxDeltaUpdateFailures - 1),
Darin Petkov36275772010-10-01 11:40:57 -0700373 Return(true)));
374 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700375 EXPECT_TRUE(attempter_.omaha_request_params_->delta_okay());
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800376 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700377 .WillOnce(DoAll(
Ben Chan672c1f52017-10-23 15:41:39 -0700378 SetArgPointee<1>(UpdateAttempter::kMaxDeltaUpdateFailures),
Darin Petkov36275772010-10-01 11:40:57 -0700379 Return(true)));
380 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700381 EXPECT_FALSE(attempter_.omaha_request_params_->delta_okay());
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800382 EXPECT_CALL(*prefs_, GetInt64(_, _)).Times(0);
Darin Petkov36275772010-10-01 11:40:57 -0700383 attempter_.DisableDeltaUpdateIfNeeded();
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700384 EXPECT_FALSE(attempter_.omaha_request_params_->delta_okay());
Darin Petkov36275772010-10-01 11:40:57 -0700385}
386
387TEST_F(UpdateAttempterTest, MarkDeltaUpdateFailureTest) {
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800388 EXPECT_CALL(*prefs_, GetInt64(kPrefsDeltaUpdateFailures, _))
Darin Petkov36275772010-10-01 11:40:57 -0700389 .WillOnce(Return(false))
Ben Chan672c1f52017-10-23 15:41:39 -0700390 .WillOnce(DoAll(SetArgPointee<1>(-1), Return(true)))
391 .WillOnce(DoAll(SetArgPointee<1>(1), Return(true)))
Darin Petkov36275772010-10-01 11:40:57 -0700392 .WillOnce(DoAll(
Ben Chan672c1f52017-10-23 15:41:39 -0700393 SetArgPointee<1>(UpdateAttempter::kMaxDeltaUpdateFailures),
Darin Petkov36275772010-10-01 11:40:57 -0700394 Return(true)));
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800395 EXPECT_CALL(*prefs_, SetInt64(Ne(kPrefsDeltaUpdateFailures), _))
Darin Petkov2dd01092010-10-08 15:43:05 -0700396 .WillRepeatedly(Return(true));
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800397 EXPECT_CALL(*prefs_, SetInt64(kPrefsDeltaUpdateFailures, 1)).Times(2);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700398 EXPECT_CALL(*prefs_, SetInt64(kPrefsDeltaUpdateFailures, 2));
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800399 EXPECT_CALL(*prefs_, SetInt64(kPrefsDeltaUpdateFailures,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700400 UpdateAttempter::kMaxDeltaUpdateFailures + 1));
Darin Petkov36275772010-10-01 11:40:57 -0700401 for (int i = 0; i < 4; i ++)
402 attempter_.MarkDeltaUpdateFailure();
403}
404
Darin Petkov1b003102010-11-30 10:18:36 -0800405TEST_F(UpdateAttempterTest, ScheduleErrorEventActionNoEventTest) {
406 EXPECT_CALL(*processor_, EnqueueAction(_)).Times(0);
407 EXPECT_CALL(*processor_, StartProcessing()).Times(0);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700408 EXPECT_CALL(*fake_system_state_.mock_payload_state(), UpdateFailed(_))
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800409 .Times(0);
410 OmahaResponse response;
Jay Srinivasan53173b92013-05-17 17:13:01 -0700411 string url1 = "http://url1";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800412 response.packages.push_back({.payload_urls = {url1, "https://url"}});
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700413 EXPECT_CALL(*(fake_system_state_.mock_payload_state()), GetCurrentUrl())
Jay Srinivasan53173b92013-05-17 17:13:01 -0700414 .WillRepeatedly(Return(url1));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700415 fake_system_state_.mock_payload_state()->SetResponse(response);
Darin Petkov1b003102010-11-30 10:18:36 -0800416 attempter_.ScheduleErrorEventAction();
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700417 EXPECT_EQ(url1, fake_system_state_.mock_payload_state()->GetCurrentUrl());
Darin Petkov1b003102010-11-30 10:18:36 -0800418}
419
420TEST_F(UpdateAttempterTest, ScheduleErrorEventActionTest) {
421 EXPECT_CALL(*processor_,
422 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700423 OmahaRequestAction::StaticType())));
424 EXPECT_CALL(*processor_, StartProcessing());
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700425 ErrorCode err = ErrorCode::kError;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700426 EXPECT_CALL(*fake_system_state_.mock_payload_state(), UpdateFailed(err));
Darin Petkov1b003102010-11-30 10:18:36 -0800427 attempter_.error_event_.reset(new OmahaEvent(OmahaEvent::kTypeUpdateComplete,
428 OmahaEvent::kResultError,
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800429 err));
Darin Petkov1b003102010-11-30 10:18:36 -0800430 attempter_.ScheduleErrorEventAction();
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700431 EXPECT_EQ(UpdateStatus::REPORTING_ERROR_EVENT, attempter_.status());
Darin Petkov1b003102010-11-30 10:18:36 -0800432}
433
Darin Petkove6ef2f82011-03-07 17:31:11 -0800434namespace {
Chris Sosa76a29ae2013-07-11 17:59:24 -0700435// Actions that will be built as part of an update check.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700436const string kUpdateActionTypes[] = { // NOLINT(runtime/string)
Darin Petkove6ef2f82011-03-07 17:31:11 -0800437 OmahaRequestAction::StaticType(),
438 OmahaResponseHandlerAction::StaticType(),
Darin Petkove6ef2f82011-03-07 17:31:11 -0800439 OmahaRequestAction::StaticType(),
440 DownloadAction::StaticType(),
441 OmahaRequestAction::StaticType(),
Allie Woodeb9e6d82015-04-17 13:55:30 -0700442 FilesystemVerifierAction::StaticType(),
Darin Petkove6ef2f82011-03-07 17:31:11 -0800443 PostinstallRunnerAction::StaticType(),
444 OmahaRequestAction::StaticType()
445};
Chris Sosa76a29ae2013-07-11 17:59:24 -0700446
447// Actions that will be built as part of a user-initiated rollback.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700448const string kRollbackActionTypes[] = { // NOLINT(runtime/string)
Chris Sosa76a29ae2013-07-11 17:59:24 -0700449 InstallPlanAction::StaticType(),
450 PostinstallRunnerAction::StaticType(),
451};
452
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700453} // namespace
Darin Petkove6ef2f82011-03-07 17:31:11 -0800454
455void UpdateAttempterTest::UpdateTestStart() {
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700456 attempter_.set_http_response_code(200);
Alex Deymo749ecf12014-10-21 20:06:57 -0700457
458 // Expect that the device policy is loaded by the UpdateAttempter at some
459 // point by calling RefreshDevicePolicy.
Igor9fd76b62017-12-11 15:24:18 +0100460 auto device_policy = std::make_unique<policy::MockDevicePolicy>();
Alex Deymo749ecf12014-10-21 20:06:57 -0700461 EXPECT_CALL(*device_policy, LoadPolicy())
462 .Times(testing::AtLeast(1)).WillRepeatedly(Return(true));
Igor9fd76b62017-12-11 15:24:18 +0100463 attempter_.policy_provider_.reset(
464 new policy::PolicyProvider(std::move(device_policy)));
Alex Deymo749ecf12014-10-21 20:06:57 -0700465
466 {
467 InSequence s;
468 for (size_t i = 0; i < arraysize(kUpdateActionTypes); ++i) {
469 EXPECT_CALL(*processor_,
470 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700471 kUpdateActionTypes[i])));
Alex Deymo749ecf12014-10-21 20:06:57 -0700472 }
Gilad Arnold74b5f552014-10-07 08:17:16 -0700473 EXPECT_CALL(*processor_, StartProcessing());
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700474 }
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700475
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700476 attempter_.Update("", "", "", "", false, false);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700477 loop_.PostTask(FROM_HERE,
478 base::Bind(&UpdateAttempterTest::UpdateTestVerify,
479 base::Unretained(this)));
Darin Petkove6ef2f82011-03-07 17:31:11 -0800480}
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700481
Darin Petkove6ef2f82011-03-07 17:31:11 -0800482void UpdateAttempterTest::UpdateTestVerify() {
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700483 EXPECT_EQ(0, attempter_.http_response_code());
484 EXPECT_EQ(&attempter_, processor_->delegate());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700485 EXPECT_EQ(arraysize(kUpdateActionTypes), attempter_.actions_.size());
486 for (size_t i = 0; i < arraysize(kUpdateActionTypes); ++i) {
487 EXPECT_EQ(kUpdateActionTypes[i], attempter_.actions_[i]->Type());
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700488 }
489 EXPECT_EQ(attempter_.response_handler_action_.get(),
490 attempter_.actions_[1].get());
Sen Jiangfef85fd2016-03-25 15:32:49 -0700491 AbstractAction* action_3 = attempter_.actions_[3].get();
492 ASSERT_NE(nullptr, action_3);
493 ASSERT_EQ(DownloadAction::StaticType(), action_3->Type());
494 DownloadAction* download_action = static_cast<DownloadAction*>(action_3);
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700495 EXPECT_EQ(&attempter_, download_action->delegate());
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700496 EXPECT_EQ(UpdateStatus::CHECKING_FOR_UPDATE, attempter_.status());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700497 loop_.BreakLoop();
Darin Petkove6ef2f82011-03-07 17:31:11 -0800498}
499
Chris Sosa28e479c2013-07-12 11:39:53 -0700500void UpdateAttempterTest::RollbackTestStart(
Chris Sosa44b9b7e2014-04-02 13:53:46 -0700501 bool enterprise_rollback, bool valid_slot) {
Chris Sosa76a29ae2013-07-11 17:59:24 -0700502 // Create a device policy so that we can change settings.
Igor9fd76b62017-12-11 15:24:18 +0100503 auto device_policy = std::make_unique<policy::MockDevicePolicy>();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700504 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Igor9fd76b62017-12-11 15:24:18 +0100505 fake_system_state_.set_device_policy(device_policy.get());
506 if (enterprise_rollback) {
507 // We return an empty owner as this is an enterprise.
508 EXPECT_CALL(*device_policy, GetOwner(_)).WillRepeatedly(
509 DoAll(SetArgPointee<0>(string("")),
510 Return(true)));
511 } else {
512 // We return a fake owner as this is an owned consumer device.
513 EXPECT_CALL(*device_policy, GetOwner(_)).WillRepeatedly(
514 DoAll(SetArgPointee<0>(string("fake.mail@fake.com")),
515 Return(true)));
516 }
517
518 attempter_.policy_provider_.reset(
519 new policy::PolicyProvider(std::move(device_policy)));
Chris Sosa76a29ae2013-07-11 17:59:24 -0700520
Alex Deymo763e7db2015-08-27 21:08:08 -0700521 if (valid_slot) {
522 BootControlInterface::Slot rollback_slot = 1;
523 LOG(INFO) << "Test Mark Bootable: "
524 << BootControlInterface::SlotName(rollback_slot);
Alex Deymoe5e5fe92015-10-05 09:28:19 -0700525 fake_system_state_.fake_boot_control()->SetSlotBootable(rollback_slot,
526 true);
Don Garrett6646b442013-11-13 15:29:11 -0800527 }
528
Chris Sosa28e479c2013-07-12 11:39:53 -0700529 bool is_rollback_allowed = false;
Chris Sosa76a29ae2013-07-11 17:59:24 -0700530
Chris Sosad38b1132014-03-25 10:43:59 -0700531 // We only allow rollback on devices that are not enterprise enrolled and
532 // which have a valid slot to rollback to.
533 if (!enterprise_rollback && valid_slot) {
Chris Sosa28e479c2013-07-12 11:39:53 -0700534 is_rollback_allowed = true;
535 }
536
Chris Sosa28e479c2013-07-12 11:39:53 -0700537 if (is_rollback_allowed) {
Chris Sosa76a29ae2013-07-11 17:59:24 -0700538 InSequence s;
539 for (size_t i = 0; i < arraysize(kRollbackActionTypes); ++i) {
540 EXPECT_CALL(*processor_,
541 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700542 kRollbackActionTypes[i])));
Chris Sosa76a29ae2013-07-11 17:59:24 -0700543 }
Gilad Arnold74b5f552014-10-07 08:17:16 -0700544 EXPECT_CALL(*processor_, StartProcessing());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700545
Chris Sosa44b9b7e2014-04-02 13:53:46 -0700546 EXPECT_TRUE(attempter_.Rollback(true));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700547 loop_.PostTask(FROM_HERE,
548 base::Bind(&UpdateAttempterTest::RollbackTestVerify,
549 base::Unretained(this)));
Chris Sosa76a29ae2013-07-11 17:59:24 -0700550 } else {
Chris Sosa44b9b7e2014-04-02 13:53:46 -0700551 EXPECT_FALSE(attempter_.Rollback(true));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700552 loop_.BreakLoop();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700553 }
554}
555
556void UpdateAttempterTest::RollbackTestVerify() {
557 // Verifies the actions that were enqueued.
558 EXPECT_EQ(&attempter_, processor_->delegate());
559 EXPECT_EQ(arraysize(kRollbackActionTypes), attempter_.actions_.size());
560 for (size_t i = 0; i < arraysize(kRollbackActionTypes); ++i) {
561 EXPECT_EQ(kRollbackActionTypes[i], attempter_.actions_[i]->Type());
562 }
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700563 EXPECT_EQ(UpdateStatus::ATTEMPTING_ROLLBACK, attempter_.status());
Alex Deymo80f70ff2016-02-10 16:08:11 -0800564 AbstractAction* action_0 = attempter_.actions_[0].get();
565 ASSERT_NE(nullptr, action_0);
566 ASSERT_EQ(InstallPlanAction::StaticType(), action_0->Type());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700567 InstallPlanAction* install_plan_action =
Alex Deymo80f70ff2016-02-10 16:08:11 -0800568 static_cast<InstallPlanAction*>(action_0);
Chris Sosa76a29ae2013-07-11 17:59:24 -0700569 InstallPlan* install_plan = install_plan_action->install_plan();
Alex Deymo80f70ff2016-02-10 16:08:11 -0800570 EXPECT_EQ(0U, install_plan->partitions.size());
Chris Sosa76a29ae2013-07-11 17:59:24 -0700571 EXPECT_EQ(install_plan->powerwash_required, true);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700572 loop_.BreakLoop();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700573}
574
Darin Petkove6ef2f82011-03-07 17:31:11 -0800575TEST_F(UpdateAttempterTest, UpdateTest) {
Alex Deymo461b2592015-07-24 20:10:52 -0700576 UpdateTestStart();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700577 loop_.Run();
Darin Petkovf42cc1c2010-09-01 09:03:02 -0700578}
579
Chris Sosa76a29ae2013-07-11 17:59:24 -0700580TEST_F(UpdateAttempterTest, RollbackTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700581 loop_.PostTask(FROM_HERE,
582 base::Bind(&UpdateAttempterTest::RollbackTestStart,
583 base::Unretained(this),
584 false, true));
585 loop_.Run();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700586}
587
Don Garrett6646b442013-11-13 15:29:11 -0800588TEST_F(UpdateAttempterTest, InvalidSlotRollbackTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700589 loop_.PostTask(FROM_HERE,
590 base::Bind(&UpdateAttempterTest::RollbackTestStart,
591 base::Unretained(this),
592 false, false));
593 loop_.Run();
Don Garrett6646b442013-11-13 15:29:11 -0800594}
595
Chris Sosa76a29ae2013-07-11 17:59:24 -0700596TEST_F(UpdateAttempterTest, EnterpriseRollbackTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700597 loop_.PostTask(FROM_HERE,
598 base::Bind(&UpdateAttempterTest::RollbackTestStart,
599 base::Unretained(this),
600 true, true));
601 loop_.Run();
Chris Sosa76a29ae2013-07-11 17:59:24 -0700602}
603
Thieu Le116fda32011-04-19 11:01:54 -0700604void UpdateAttempterTest::PingOmahaTestStart() {
605 EXPECT_CALL(*processor_,
606 EnqueueAction(Property(&AbstractAction::Type,
Gilad Arnold74b5f552014-10-07 08:17:16 -0700607 OmahaRequestAction::StaticType())));
608 EXPECT_CALL(*processor_, StartProcessing());
Thieu Le116fda32011-04-19 11:01:54 -0700609 attempter_.PingOmaha();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700610 ScheduleQuitMainLoop();
Thieu Le116fda32011-04-19 11:01:54 -0700611}
612
613TEST_F(UpdateAttempterTest, PingOmahaTest) {
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700614 EXPECT_FALSE(attempter_.waiting_for_scheduled_check_);
615 EXPECT_FALSE(attempter_.schedule_updates_called());
616 // Disable scheduling of subsequnet checks; we're using the DefaultPolicy in
617 // testing, which is more permissive than we want to handle here.
618 attempter_.DisableScheduleUpdates();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700619 loop_.PostTask(FROM_HERE,
620 base::Bind(&UpdateAttempterTest::PingOmahaTestStart,
621 base::Unretained(this)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700622 brillo::MessageLoopRunMaxIterations(&loop_, 100);
Christopher Wiley6b6cc1b2015-10-05 17:50:07 -0700623 EXPECT_EQ(UpdateStatus::UPDATED_NEED_REBOOT, attempter_.status());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700624 EXPECT_TRUE(attempter_.schedule_updates_called());
Thieu Le116fda32011-04-19 11:01:54 -0700625}
626
Darin Petkov18c7bce2011-06-16 14:07:00 -0700627TEST_F(UpdateAttempterTest, CreatePendingErrorEventTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -0800628 MockAction action;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700629 const ErrorCode kCode = ErrorCode::kDownloadTransferError;
Darin Petkov18c7bce2011-06-16 14:07:00 -0700630 attempter_.CreatePendingErrorEvent(&action, kCode);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700631 ASSERT_NE(nullptr, attempter_.error_event_.get());
Darin Petkov18c7bce2011-06-16 14:07:00 -0700632 EXPECT_EQ(OmahaEvent::kTypeUpdateComplete, attempter_.error_event_->type);
633 EXPECT_EQ(OmahaEvent::kResultError, attempter_.error_event_->result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700634 EXPECT_EQ(
635 static_cast<ErrorCode>(static_cast<int>(kCode) |
636 static_cast<int>(ErrorCode::kTestOmahaUrlFlag)),
637 attempter_.error_event_->error_code);
Darin Petkov18c7bce2011-06-16 14:07:00 -0700638}
639
640TEST_F(UpdateAttempterTest, CreatePendingErrorEventResumedTest) {
641 OmahaResponseHandlerAction *response_action =
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700642 new OmahaResponseHandlerAction(&fake_system_state_);
Darin Petkov18c7bce2011-06-16 14:07:00 -0700643 response_action->install_plan_.is_resume = true;
644 attempter_.response_handler_action_.reset(response_action);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800645 MockAction action;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700646 const ErrorCode kCode = ErrorCode::kInstallDeviceOpenError;
Darin Petkov18c7bce2011-06-16 14:07:00 -0700647 attempter_.CreatePendingErrorEvent(&action, kCode);
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700648 ASSERT_NE(nullptr, attempter_.error_event_.get());
Darin Petkov18c7bce2011-06-16 14:07:00 -0700649 EXPECT_EQ(OmahaEvent::kTypeUpdateComplete, attempter_.error_event_->type);
650 EXPECT_EQ(OmahaEvent::kResultError, attempter_.error_event_->result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700651 EXPECT_EQ(
652 static_cast<ErrorCode>(
653 static_cast<int>(kCode) |
654 static_cast<int>(ErrorCode::kResumedFlag) |
655 static_cast<int>(ErrorCode::kTestOmahaUrlFlag)),
656 attempter_.error_event_->error_code);
Darin Petkov18c7bce2011-06-16 14:07:00 -0700657}
658
David Zeuthen8f191b22013-08-06 12:27:50 -0700659TEST_F(UpdateAttempterTest, P2PNotStartedAtStartupWhenNotEnabled) {
660 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700661 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700662 mock_p2p_manager.fake().SetP2PEnabled(false);
663 EXPECT_CALL(mock_p2p_manager, EnsureP2PRunning()).Times(0);
664 attempter_.UpdateEngineStarted();
665}
666
667TEST_F(UpdateAttempterTest, P2PNotStartedAtStartupWhenEnabledButNotSharing) {
668 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700669 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700670 mock_p2p_manager.fake().SetP2PEnabled(true);
671 EXPECT_CALL(mock_p2p_manager, EnsureP2PRunning()).Times(0);
672 attempter_.UpdateEngineStarted();
673}
674
675TEST_F(UpdateAttempterTest, P2PStartedAtStartupWhenEnabledAndSharing) {
676 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700677 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700678 mock_p2p_manager.fake().SetP2PEnabled(true);
679 mock_p2p_manager.fake().SetCountSharedFilesResult(1);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700680 EXPECT_CALL(mock_p2p_manager, EnsureP2PRunning());
David Zeuthen8f191b22013-08-06 12:27:50 -0700681 attempter_.UpdateEngineStarted();
682}
683
684TEST_F(UpdateAttempterTest, P2PNotEnabled) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700685 loop_.PostTask(FROM_HERE,
686 base::Bind(&UpdateAttempterTest::P2PNotEnabledStart,
687 base::Unretained(this)));
688 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700689}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700690
David Zeuthen8f191b22013-08-06 12:27:50 -0700691void UpdateAttempterTest::P2PNotEnabledStart() {
692 // If P2P is not enabled, check that we do not attempt housekeeping
693 // and do not convey that p2p is to be used.
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(false);
697 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping()).Times(0);
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700698 attempter_.Update("", "", "", "", false, false);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700699 EXPECT_FALSE(actual_using_p2p_for_downloading_);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700700 EXPECT_FALSE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700701 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700702}
703
704TEST_F(UpdateAttempterTest, P2PEnabledStartingFails) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700705 loop_.PostTask(FROM_HERE,
706 base::Bind(&UpdateAttempterTest::P2PEnabledStartingFailsStart,
707 base::Unretained(this)));
708 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700709}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700710
David Zeuthen8f191b22013-08-06 12:27:50 -0700711void UpdateAttempterTest::P2PEnabledStartingFailsStart() {
712 // If p2p is enabled, but starting it fails ensure we don't do
713 // any housekeeping and do not convey that p2p should be used.
714 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700715 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700716 mock_p2p_manager.fake().SetP2PEnabled(true);
717 mock_p2p_manager.fake().SetEnsureP2PRunningResult(false);
718 mock_p2p_manager.fake().SetPerformHousekeepingResult(false);
719 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping()).Times(0);
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700720 attempter_.Update("", "", "", "", false, false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700721 EXPECT_FALSE(actual_using_p2p_for_downloading());
722 EXPECT_FALSE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700723 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700724}
725
726TEST_F(UpdateAttempterTest, P2PEnabledHousekeepingFails) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700727 loop_.PostTask(
728 FROM_HERE,
729 base::Bind(&UpdateAttempterTest::P2PEnabledHousekeepingFailsStart,
730 base::Unretained(this)));
731 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700732}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700733
David Zeuthen8f191b22013-08-06 12:27:50 -0700734void UpdateAttempterTest::P2PEnabledHousekeepingFailsStart() {
735 // If p2p is enabled, starting it works but housekeeping fails, ensure
736 // we do not convey p2p is to be used.
737 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700738 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700739 mock_p2p_manager.fake().SetP2PEnabled(true);
740 mock_p2p_manager.fake().SetEnsureP2PRunningResult(true);
741 mock_p2p_manager.fake().SetPerformHousekeepingResult(false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700742 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700743 attempter_.Update("", "", "", "", false, false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700744 EXPECT_FALSE(actual_using_p2p_for_downloading());
745 EXPECT_FALSE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700746 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700747}
748
749TEST_F(UpdateAttempterTest, P2PEnabled) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700750 loop_.PostTask(FROM_HERE,
751 base::Bind(&UpdateAttempterTest::P2PEnabledStart,
752 base::Unretained(this)));
753 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700754}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700755
David Zeuthen8f191b22013-08-06 12:27:50 -0700756void UpdateAttempterTest::P2PEnabledStart() {
757 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700758 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700759 // If P2P is enabled and starting it works, check that we performed
760 // housekeeping and that we convey p2p should be used.
761 mock_p2p_manager.fake().SetP2PEnabled(true);
762 mock_p2p_manager.fake().SetEnsureP2PRunningResult(true);
763 mock_p2p_manager.fake().SetPerformHousekeepingResult(true);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700764 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700765 attempter_.Update("", "", "", "", false, false);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700766 EXPECT_TRUE(actual_using_p2p_for_downloading());
767 EXPECT_TRUE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700768 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700769}
770
771TEST_F(UpdateAttempterTest, P2PEnabledInteractive) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700772 loop_.PostTask(FROM_HERE,
773 base::Bind(&UpdateAttempterTest::P2PEnabledInteractiveStart,
774 base::Unretained(this)));
775 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700776}
Alex Deymo60ca1a72015-06-18 18:19:15 -0700777
David Zeuthen8f191b22013-08-06 12:27:50 -0700778void UpdateAttempterTest::P2PEnabledInteractiveStart() {
779 MockP2PManager mock_p2p_manager;
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700780 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -0700781 // For an interactive check, if P2P is enabled and starting it
782 // works, check that we performed housekeeping and that we convey
783 // p2p should be used for sharing but NOT for downloading.
784 mock_p2p_manager.fake().SetP2PEnabled(true);
785 mock_p2p_manager.fake().SetEnsureP2PRunningResult(true);
786 mock_p2p_manager.fake().SetPerformHousekeepingResult(true);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700787 EXPECT_CALL(mock_p2p_manager, PerformHousekeeping());
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700788 attempter_.Update("", "", "", "", false, true /* interactive */);
Gilad Arnold74b5f552014-10-07 08:17:16 -0700789 EXPECT_FALSE(actual_using_p2p_for_downloading());
790 EXPECT_TRUE(actual_using_p2p_for_sharing());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700791 ScheduleQuitMainLoop();
David Zeuthen8f191b22013-08-06 12:27:50 -0700792}
793
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700794TEST_F(UpdateAttempterTest, ReadScatterFactorFromPolicy) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700795 loop_.PostTask(
796 FROM_HERE,
797 base::Bind(&UpdateAttempterTest::ReadScatterFactorFromPolicyTestStart,
798 base::Unretained(this)));
799 loop_.Run();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700800}
801
802// Tests that the scatter_factor_in_seconds value is properly fetched
803// from the device policy.
804void UpdateAttempterTest::ReadScatterFactorFromPolicyTestStart() {
Ben Chan9abb7632014-08-07 00:10:53 -0700805 int64_t scatter_factor_in_seconds = 36000;
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700806
Igor9fd76b62017-12-11 15:24:18 +0100807 auto device_policy = std::make_unique<policy::MockDevicePolicy>();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700808 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Igor9fd76b62017-12-11 15:24:18 +0100809 fake_system_state_.set_device_policy(device_policy.get());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700810
811 EXPECT_CALL(*device_policy, GetScatterFactorInSeconds(_))
812 .WillRepeatedly(DoAll(
Ben Chan672c1f52017-10-23 15:41:39 -0700813 SetArgPointee<0>(scatter_factor_in_seconds),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700814 Return(true)));
815
Igor9fd76b62017-12-11 15:24:18 +0100816 attempter_.policy_provider_.reset(
817 new policy::PolicyProvider(std::move(device_policy)));
818
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700819 attempter_.Update("", "", "", "", false, false);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700820 EXPECT_EQ(scatter_factor_in_seconds, attempter_.scatter_factor_.InSeconds());
821
Alex Deymo60ca1a72015-06-18 18:19:15 -0700822 ScheduleQuitMainLoop();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700823}
824
825TEST_F(UpdateAttempterTest, DecrementUpdateCheckCountTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700826 loop_.PostTask(
827 FROM_HERE,
828 base::Bind(&UpdateAttempterTest::DecrementUpdateCheckCountTestStart,
829 base::Unretained(this)));
830 loop_.Run();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700831}
832
833void UpdateAttempterTest::DecrementUpdateCheckCountTestStart() {
834 // Tests that the scatter_factor_in_seconds value is properly fetched
835 // from the device policy and is decremented if value > 0.
Ben Chan9abb7632014-08-07 00:10:53 -0700836 int64_t initial_value = 5;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800837 FakePrefs fake_prefs;
838 attempter_.prefs_ = &fake_prefs;
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700839
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700840 fake_system_state_.fake_hardware()->SetIsOOBEComplete(Time::UnixEpoch());
Jay Srinivasan08fce042012-06-07 16:31:01 -0700841
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800842 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsUpdateCheckCount, initial_value));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700843
Ben Chan9abb7632014-08-07 00:10:53 -0700844 int64_t scatter_factor_in_seconds = 10;
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700845
Igor9fd76b62017-12-11 15:24:18 +0100846 auto device_policy = std::make_unique<policy::MockDevicePolicy>();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700847 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Igor9fd76b62017-12-11 15:24:18 +0100848 fake_system_state_.set_device_policy(device_policy.get());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700849
850 EXPECT_CALL(*device_policy, GetScatterFactorInSeconds(_))
851 .WillRepeatedly(DoAll(
Ben Chan672c1f52017-10-23 15:41:39 -0700852 SetArgPointee<0>(scatter_factor_in_seconds),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700853 Return(true)));
854
Igor9fd76b62017-12-11 15:24:18 +0100855 attempter_.policy_provider_.reset(
856 new policy::PolicyProvider(std::move(device_policy)));
857
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700858 attempter_.Update("", "", "", "", false, false);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700859 EXPECT_EQ(scatter_factor_in_seconds, attempter_.scatter_factor_.InSeconds());
860
861 // Make sure the file still exists.
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800862 EXPECT_TRUE(fake_prefs.Exists(kPrefsUpdateCheckCount));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700863
Ben Chan9abb7632014-08-07 00:10:53 -0700864 int64_t new_value;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800865 EXPECT_TRUE(fake_prefs.GetInt64(kPrefsUpdateCheckCount, &new_value));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700866 EXPECT_EQ(initial_value - 1, new_value);
867
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700868 EXPECT_TRUE(
869 attempter_.omaha_request_params_->update_check_count_wait_enabled());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700870
871 // However, if the count is already 0, it's not decremented. Test that.
872 initial_value = 0;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800873 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsUpdateCheckCount, initial_value));
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700874 attempter_.Update("", "", "", "", false, false);
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800875 EXPECT_TRUE(fake_prefs.Exists(kPrefsUpdateCheckCount));
876 EXPECT_TRUE(fake_prefs.GetInt64(kPrefsUpdateCheckCount, &new_value));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700877 EXPECT_EQ(initial_value, new_value);
878
Alex Deymo60ca1a72015-06-18 18:19:15 -0700879 ScheduleQuitMainLoop();
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700880}
881
Jay Srinivasan08fce042012-06-07 16:31:01 -0700882TEST_F(UpdateAttempterTest, NoScatteringDoneDuringManualUpdateTestStart) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700883 loop_.PostTask(FROM_HERE, base::Bind(
884 &UpdateAttempterTest::NoScatteringDoneDuringManualUpdateTestStart,
885 base::Unretained(this)));
886 loop_.Run();
Jay Srinivasan08fce042012-06-07 16:31:01 -0700887}
888
889void UpdateAttempterTest::NoScatteringDoneDuringManualUpdateTestStart() {
890 // Tests that no scattering logic is enabled if the update check
891 // is manually done (as opposed to a scheduled update check)
Ben Chan9abb7632014-08-07 00:10:53 -0700892 int64_t initial_value = 8;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800893 FakePrefs fake_prefs;
894 attempter_.prefs_ = &fake_prefs;
Jay Srinivasan08fce042012-06-07 16:31:01 -0700895
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700896 fake_system_state_.fake_hardware()->SetIsOOBEComplete(Time::UnixEpoch());
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800897 fake_system_state_.set_prefs(&fake_prefs);
Jay Srinivasan08fce042012-06-07 16:31:01 -0700898
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800899 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsWallClockWaitPeriod, initial_value));
900 EXPECT_TRUE(fake_prefs.SetInt64(kPrefsUpdateCheckCount, initial_value));
Jay Srinivasan08fce042012-06-07 16:31:01 -0700901
902 // make sure scatter_factor is non-zero as scattering is disabled
903 // otherwise.
Ben Chan9abb7632014-08-07 00:10:53 -0700904 int64_t scatter_factor_in_seconds = 50;
Jay Srinivasan08fce042012-06-07 16:31:01 -0700905
Igor9fd76b62017-12-11 15:24:18 +0100906 auto device_policy = std::make_unique<policy::MockDevicePolicy>();
Jay Srinivasan08fce042012-06-07 16:31:01 -0700907 EXPECT_CALL(*device_policy, LoadPolicy()).WillRepeatedly(Return(true));
Igor9fd76b62017-12-11 15:24:18 +0100908 fake_system_state_.set_device_policy(device_policy.get());
Jay Srinivasan08fce042012-06-07 16:31:01 -0700909
910 EXPECT_CALL(*device_policy, GetScatterFactorInSeconds(_))
911 .WillRepeatedly(DoAll(
Ben Chan672c1f52017-10-23 15:41:39 -0700912 SetArgPointee<0>(scatter_factor_in_seconds),
Jay Srinivasan08fce042012-06-07 16:31:01 -0700913 Return(true)));
914
Igor9fd76b62017-12-11 15:24:18 +0100915 attempter_.policy_provider_.reset(
916 new policy::PolicyProvider(std::move(device_policy)));
917
Gilad Arnoldb92f0df2013-01-10 16:32:45 -0800918 // Trigger an interactive check so we can test that scattering is disabled.
Gilad Arnoldec7f9162014-07-15 13:24:46 -0700919 attempter_.Update("", "", "", "", false, true);
Jay Srinivasan08fce042012-06-07 16:31:01 -0700920 EXPECT_EQ(scatter_factor_in_seconds, attempter_.scatter_factor_.InSeconds());
921
922 // Make sure scattering is disabled for manual (i.e. user initiated) update
Jay Srinivasan21be0752012-07-25 15:44:56 -0700923 // checks and all artifacts are removed.
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700924 EXPECT_FALSE(
925 attempter_.omaha_request_params_->wall_clock_based_wait_enabled());
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800926 EXPECT_FALSE(fake_prefs.Exists(kPrefsWallClockWaitPeriod));
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700927 EXPECT_EQ(0, attempter_.omaha_request_params_->waiting_period().InSeconds());
928 EXPECT_FALSE(
929 attempter_.omaha_request_params_->update_check_count_wait_enabled());
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800930 EXPECT_FALSE(fake_prefs.Exists(kPrefsUpdateCheckCount));
Jay Srinivasan08fce042012-06-07 16:31:01 -0700931
Alex Deymo60ca1a72015-06-18 18:19:15 -0700932 ScheduleQuitMainLoop();
Jay Srinivasan08fce042012-06-07 16:31:01 -0700933}
934
David Zeuthen985b1122013-10-09 12:13:15 -0700935// Checks that we only report daily metrics at most every 24 hours.
936TEST_F(UpdateAttempterTest, ReportDailyMetrics) {
937 FakeClock fake_clock;
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800938 FakePrefs fake_prefs;
David Zeuthen985b1122013-10-09 12:13:15 -0700939
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700940 fake_system_state_.set_clock(&fake_clock);
Alex Deymo2c0db7b2014-11-04 12:23:39 -0800941 fake_system_state_.set_prefs(&fake_prefs);
David Zeuthen985b1122013-10-09 12:13:15 -0700942
943 Time epoch = Time::FromInternalValue(0);
944 fake_clock.SetWallclockTime(epoch);
945
946 // If there is no kPrefsDailyMetricsLastReportedAt state variable,
947 // we should report.
948 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
949 // We should not report again if no time has passed.
950 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
951
952 // We should not report if only 10 hours has passed.
953 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(10));
954 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
955
956 // We should not report if only 24 hours - 1 sec has passed.
957 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(24) -
958 TimeDelta::FromSeconds(1));
959 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
960
961 // We should report if 24 hours has passed.
962 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(24));
963 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
964
965 // But then we should not report again..
966 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
967
968 // .. until another 24 hours has passed
969 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(47));
970 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
971 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(48));
972 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
973 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
974
975 // .. and another 24 hours
976 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(71));
977 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
978 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(72));
979 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
980 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
981
982 // If the span between time of reporting and present time is
983 // negative, we report. This is in order to reset the timestamp and
984 // avoid an edge condition whereby a distant point in the future is
985 // in the state variable resulting in us never ever reporting again.
986 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(71));
987 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
988 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
989
990 // In this case we should not update until the clock reads 71 + 24 = 95.
991 // Check that.
992 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(94));
993 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
994 fake_clock.SetWallclockTime(epoch + TimeDelta::FromHours(95));
995 EXPECT_TRUE(attempter_.CheckAndReportDailyMetrics());
996 EXPECT_FALSE(attempter_.CheckAndReportDailyMetrics());
David Zeuthen985b1122013-10-09 12:13:15 -0700997}
998
David Zeuthen3c55abd2013-10-14 12:48:03 -0700999TEST_F(UpdateAttempterTest, BootTimeInUpdateMarkerFile) {
David Zeuthen3c55abd2013-10-14 12:48:03 -07001000 FakeClock fake_clock;
1001 fake_clock.SetBootTime(Time::FromTimeT(42));
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001002 fake_system_state_.set_clock(&fake_clock);
Alex Deymo906191f2015-10-12 12:22:44 -07001003 FakePrefs fake_prefs;
1004 fake_system_state_.set_prefs(&fake_prefs);
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001005 attempter_.Init();
David Zeuthen3c55abd2013-10-14 12:48:03 -07001006
1007 Time boot_time;
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001008 EXPECT_FALSE(attempter_.GetBootTimeAtUpdate(&boot_time));
David Zeuthen3c55abd2013-10-14 12:48:03 -07001009
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001010 attempter_.WriteUpdateCompletedMarker();
David Zeuthen3c55abd2013-10-14 12:48:03 -07001011
Sen Jiangaeeb2e02016-06-09 15:00:16 -07001012 EXPECT_TRUE(attempter_.GetBootTimeAtUpdate(&boot_time));
David Zeuthen3c55abd2013-10-14 12:48:03 -07001013 EXPECT_EQ(boot_time.ToTimeT(), 42);
1014}
1015
David Pursell02c18642014-11-06 11:26:11 -08001016TEST_F(UpdateAttempterTest, AnyUpdateSourceAllowedUnofficial) {
1017 fake_system_state_.fake_hardware()->SetIsOfficialBuild(false);
1018 EXPECT_TRUE(attempter_.IsAnyUpdateSourceAllowed());
1019}
1020
1021TEST_F(UpdateAttempterTest, AnyUpdateSourceAllowedOfficialDevmode) {
1022 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001023 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(true);
David Pursell02c18642014-11-06 11:26:11 -08001024 EXPECT_TRUE(attempter_.IsAnyUpdateSourceAllowed());
1025}
1026
1027TEST_F(UpdateAttempterTest, AnyUpdateSourceDisallowedOfficialNormal) {
1028 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001029 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(false);
David Pursell02c18642014-11-06 11:26:11 -08001030 EXPECT_FALSE(attempter_.IsAnyUpdateSourceAllowed());
1031}
1032
1033TEST_F(UpdateAttempterTest, CheckForUpdateAUTest) {
1034 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001035 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(false);
Aaron Wood081c0232017-10-19 17:14:58 -07001036 attempter_.CheckForUpdate("", "autest", UpdateAttemptFlags::kNone);
Alex Deymoac41a822015-09-15 20:52:53 -07001037 EXPECT_EQ(constants::kOmahaDefaultAUTestURL, attempter_.forced_omaha_url());
David Pursell02c18642014-11-06 11:26:11 -08001038}
1039
1040TEST_F(UpdateAttempterTest, CheckForUpdateScheduledAUTest) {
1041 fake_system_state_.fake_hardware()->SetIsOfficialBuild(true);
Sen Jiange67bb5b2016-06-20 15:53:56 -07001042 fake_system_state_.fake_hardware()->SetAreDevFeaturesEnabled(false);
Aaron Wood081c0232017-10-19 17:14:58 -07001043 attempter_.CheckForUpdate("", "autest-scheduled", UpdateAttemptFlags::kNone);
Alex Deymoac41a822015-09-15 20:52:53 -07001044 EXPECT_EQ(constants::kOmahaDefaultAUTestURL, attempter_.forced_omaha_url());
David Pursell02c18642014-11-06 11:26:11 -08001045}
1046
Xiyuan Xiac0e8f9a2017-02-22 13:19:35 -08001047TEST_F(UpdateAttempterTest, TargetVersionPrefixSetAndReset) {
1048 attempter_.CalculateUpdateParams("", "", "", "1234", false, false);
1049 EXPECT_EQ("1234",
1050 fake_system_state_.request_params()->target_version_prefix());
1051
1052 attempter_.CalculateUpdateParams("", "", "", "", false, false);
1053 EXPECT_TRUE(
1054 fake_system_state_.request_params()->target_version_prefix().empty());
1055}
1056
Aaron Wood23bd3392017-10-06 14:48:25 -07001057TEST_F(UpdateAttempterTest, UpdateDeferredByPolicyTest) {
1058 // Construct an OmahaResponseHandlerAction that has processed an InstallPlan,
1059 // but the update is being deferred by the Policy.
1060 OmahaResponseHandlerAction* response_action =
1061 new OmahaResponseHandlerAction(&fake_system_state_);
1062 response_action->install_plan_.version = "a.b.c.d";
1063 response_action->install_plan_.system_version = "b.c.d.e";
1064 response_action->install_plan_.payloads.push_back(
1065 {.size = 1234ULL, .type = InstallPayloadType::kFull});
1066 attempter_.response_handler_action_.reset(response_action);
1067 // Inform the UpdateAttempter that the OmahaResponseHandlerAction has
1068 // completed, with the deferred-update error code.
1069 attempter_.ActionCompleted(
1070 nullptr, response_action, ErrorCode::kOmahaUpdateDeferredPerPolicy);
1071 {
1072 UpdateEngineStatus status;
1073 attempter_.GetStatus(&status);
1074 EXPECT_EQ(UpdateStatus::UPDATE_AVAILABLE, status.status);
1075 EXPECT_EQ(response_action->install_plan_.version, status.new_version);
1076 EXPECT_EQ(response_action->install_plan_.system_version,
1077 status.new_system_version);
1078 EXPECT_EQ(response_action->install_plan_.payloads[0].size,
1079 status.new_size_bytes);
1080 }
1081 // An "error" event should have been created to tell Omaha that the update is
1082 // being deferred.
1083 EXPECT_TRUE(nullptr != attempter_.error_event_);
1084 EXPECT_EQ(OmahaEvent::kTypeUpdateComplete, attempter_.error_event_->type);
1085 EXPECT_EQ(OmahaEvent::kResultUpdateDeferred, attempter_.error_event_->result);
1086 ErrorCode expected_code = static_cast<ErrorCode>(
1087 static_cast<int>(ErrorCode::kOmahaUpdateDeferredPerPolicy) |
1088 static_cast<int>(ErrorCode::kTestOmahaUrlFlag));
1089 EXPECT_EQ(expected_code, attempter_.error_event_->error_code);
1090 // End the processing
1091 attempter_.ProcessingDone(nullptr, ErrorCode::kOmahaUpdateDeferredPerPolicy);
1092 // Validate the state of the attempter.
1093 {
1094 UpdateEngineStatus status;
1095 attempter_.GetStatus(&status);
1096 EXPECT_EQ(UpdateStatus::REPORTING_ERROR_EVENT, status.status);
1097 EXPECT_EQ(response_action->install_plan_.version, status.new_version);
1098 EXPECT_EQ(response_action->install_plan_.system_version,
1099 status.new_system_version);
1100 EXPECT_EQ(response_action->install_plan_.payloads[0].size,
1101 status.new_size_bytes);
1102 }
1103}
1104
1105TEST_F(UpdateAttempterTest, UpdateIsNotRunningWhenUpdateAvailable) {
1106 EXPECT_FALSE(attempter_.IsUpdateRunningOrScheduled());
1107 // Verify in-progress update with UPDATE_AVAILABLE is running
1108 attempter_.status_ = UpdateStatus::UPDATE_AVAILABLE;
1109 EXPECT_TRUE(attempter_.IsUpdateRunningOrScheduled());
1110}
1111
Aaron Woodbf5a2522017-10-04 10:58:36 -07001112TEST_F(UpdateAttempterTest, UpdateAttemptFlagsCachedAtUpdateStart) {
1113 attempter_.SetUpdateAttemptFlags(UpdateAttemptFlags::kFlagRestrictDownload);
1114
1115 UpdateCheckParams params = {.updates_enabled = true};
1116 attempter_.OnUpdateScheduled(EvalStatus::kSucceeded, params);
1117
1118 EXPECT_EQ(UpdateAttemptFlags::kFlagRestrictDownload,
1119 attempter_.GetCurrentUpdateAttemptFlags());
1120}
1121
Aaron Wood081c0232017-10-19 17:14:58 -07001122TEST_F(UpdateAttempterTest, InteractiveUpdateUsesPassedRestrictions) {
1123 attempter_.SetUpdateAttemptFlags(UpdateAttemptFlags::kFlagRestrictDownload);
1124
1125 attempter_.CheckForUpdate("", "", UpdateAttemptFlags::kNone);
1126 EXPECT_EQ(UpdateAttemptFlags::kNone,
1127 attempter_.GetCurrentUpdateAttemptFlags());
1128}
1129
1130TEST_F(UpdateAttempterTest, NonInteractiveUpdateUsesSetRestrictions) {
1131 attempter_.SetUpdateAttemptFlags(UpdateAttemptFlags::kNone);
1132
1133 // This tests that when CheckForUpdate() is called with the non-interactive
1134 // flag set, that it doesn't change the current UpdateAttemptFlags.
1135 attempter_.CheckForUpdate("",
1136 "",
1137 UpdateAttemptFlags::kFlagNonInteractive |
1138 UpdateAttemptFlags::kFlagRestrictDownload);
1139 EXPECT_EQ(UpdateAttemptFlags::kNone,
1140 attempter_.GetCurrentUpdateAttemptFlags());
1141}
1142
Darin Petkovf42cc1c2010-09-01 09:03:02 -07001143} // namespace chromeos_update_engine