blob: cb6c5bdb77508c08bd26b61ab77ae3ad072a7720 [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 Petkov6a5b3222010-07-13 14:55:28 -070016
Alex Deymo8427b4a2014-11-05 14:00:32 -080017#include "update_engine/omaha_request_action.h"
18
Ben Chan9abb7632014-08-07 00:10:53 -070019#include <stdint.h>
20
Ben Chanab5a0af2017-10-12 14:57:50 -070021#include <memory>
Darin Petkov6a5b3222010-07-13 14:55:28 -070022#include <string>
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -080023#include <utility>
Darin Petkov6a5b3222010-07-13 14:55:28 -070024#include <vector>
Darin Petkov0dc8e9a2010-07-14 14:51:57 -070025
Alex Deymo60ca1a72015-06-18 18:19:15 -070026#include <base/bind.h>
Alex Deymo110e0302015-10-19 20:35:21 -070027#include <base/files/file_util.h>
Sen Jiang297e5832016-03-17 14:45:51 -070028#include <base/files/scoped_temp_dir.h>
Ben Chan5c02c132017-06-27 07:10:36 -070029#include <base/memory/ptr_util.h>
Alex Deymo8e18f932015-03-27 16:16:59 -070030#include <base/strings/string_number_conversions.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070031#include <base/strings/string_util.h>
32#include <base/strings/stringprintf.h>
33#include <base/time/time.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070034#include <brillo/bind_lambda.h>
35#include <brillo/message_loops/fake_message_loop.h>
36#include <brillo/message_loops/message_loop.h>
37#include <brillo/message_loops/message_loop_utils.h>
Alex Deymoe1e3afe2014-10-30 13:02:49 -070038#include <gtest/gtest.h>
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -080039#include <policy/libpolicy.h>
40#include <policy/mock_libpolicy.h>
Jay Srinivasand29695d2013-04-08 15:08:05 -070041
Alex Deymo39910dc2015-11-09 17:04:30 -080042#include "update_engine/common/action_pipe.h"
43#include "update_engine/common/constants.h"
44#include "update_engine/common/fake_prefs.h"
45#include "update_engine/common/hash_calculator.h"
46#include "update_engine/common/mock_http_fetcher.h"
47#include "update_engine/common/platform_constants.h"
48#include "update_engine/common/prefs.h"
49#include "update_engine/common/test_utils.h"
Alex Deymoc1c17b42015-11-23 03:53:15 -030050#include "update_engine/fake_system_state.h"
Tianjie Xu282aa1f2017-09-05 13:42:45 -070051#include "update_engine/metrics_reporter_interface.h"
Chris Sosa77f79e82014-06-02 18:16:24 -070052#include "update_engine/mock_connection_manager.h"
Gilad Arnold74b5f552014-10-07 08:17:16 -070053#include "update_engine/mock_payload_state.h"
Darin Petkova4a8a8c2010-07-15 22:21:12 -070054#include "update_engine/omaha_request_params.h"
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -080055#include "update_engine/update_manager/rollback_prefs.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070056
Darin Petkov1cbd78f2010-07-29 12:38:34 -070057using base::Time;
58using base::TimeDelta;
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -080059using chromeos_update_manager::kRollforwardInfinity;
Darin Petkov6a5b3222010-07-13 14:55:28 -070060using std::string;
61using std::vector;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070062using testing::AllOf;
Alex Deymof329b932014-10-30 01:37:48 -070063using testing::AnyNumber;
Jay Srinivasan34b5d862012-07-23 11:43:22 -070064using testing::DoAll;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070065using testing::Ge;
66using testing::Le;
Darin Petkov9c096d62010-11-17 14:49:04 -080067using testing::NiceMock;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070068using testing::Return;
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -080069using testing::ReturnRef;
Gilad Arnold74b5f552014-10-07 08:17:16 -070070using testing::ReturnPointee;
71using testing::SaveArg;
Ben Chan672c1f52017-10-23 15:41:39 -070072using testing::SetArgPointee;
Alex Deymof329b932014-10-30 01:37:48 -070073using testing::_;
Darin Petkov6a5b3222010-07-13 14:55:28 -070074
Alex Deymo8e18f932015-03-27 16:16:59 -070075namespace {
76
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -080077static_assert(kRollforwardInfinity == 0xfffffffe,
78 "Don't change the value of kRollforward infinity unless its "
79 "size has been changed in firmware.");
80
Alex Deymo85616652015-10-15 18:48:31 -070081const char kTestAppId[] = "test-app-id";
Aaron Wood7dcdedf2017-09-06 17:17:41 -070082const char kTestAppId2[] = "test-app2-id";
Alex Deymo85616652015-10-15 18:48:31 -070083
Alex Deymo8e18f932015-03-27 16:16:59 -070084// This is a helper struct to allow unit tests build an update response with the
85// values they care about.
86struct FakeUpdateResponse {
87 string GetNoUpdateResponse() const {
88 string entity_str;
89 if (include_entity)
90 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Sen Jiang81259682017-03-30 15:11:30 -070091 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" + entity_str +
92 "<response protocol=\"3.0\">"
93 "<daystart elapsed_seconds=\"100\"/>"
94 "<app appid=\"" +
95 app_id + "\" " +
96 (include_cohorts
97 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
98 "\" cohortname=\"" + cohortname + "\" "
99 : "") +
100 " status=\"ok\">"
101 "<ping status=\"ok\"/>"
102 "<updatecheck status=\"noupdate\"/></app>" +
103 (multi_app_no_update
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700104 ? "<app appid=\"" + app_id2 +
105 "\"><updatecheck status=\"noupdate\"/></app>"
Sen Jiang81259682017-03-30 15:11:30 -0700106 : "") +
107 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700108 }
109
110 string GetUpdateResponse() const {
Sen Jiang2703ef42017-03-16 13:36:21 -0700111 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
112 "protocol=\"3.0\">"
113 "<daystart elapsed_seconds=\"100\"" +
114 (elapsed_days.empty() ? ""
115 : (" elapsed_days=\"" + elapsed_days + "\"")) +
116 "/>"
117 "<app appid=\"" +
118 app_id + "\" " +
119 (include_cohorts
120 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
121 "\" cohortname=\"" + cohortname + "\" "
122 : "") +
123 " status=\"ok\">"
124 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
125 "<urls><url codebase=\"" +
126 codebase +
127 "\"/></urls>"
128 "<manifest version=\"" +
129 version +
130 "\">"
131 "<packages><package hash=\"not-used\" name=\"" +
132 filename + "\" size=\"" + base::Int64ToString(size) +
Sen Jiang0affc2c2017-02-10 15:55:05 -0800133 "\" hash_sha256=\"" + hash + "\"/>" +
134 (multi_package ? "<package name=\"package2\" size=\"222\" "
135 "hash_sha256=\"hash2\"/>"
136 : "") +
137 "</packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700138 "<actions><action event=\"postinstall\" MetadataSize=\"11" +
139 (multi_package ? ":22" : "") + "\" ChromeOSVersion=\"" + version +
140 "\" MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt +
Sen Jiang2703ef42017-03-16 13:36:21 -0700141 "\" "
142 "IsDelta=\"true\" "
Sen Jiangcdd52062017-05-18 15:33:10 -0700143 "IsDeltaPayload=\"true" +
144 (multi_package ? ":false" : "") +
145 "\" "
Sen Jiang2703ef42017-03-16 13:36:21 -0700146 "MaxDaysToScatter=\"" +
147 max_days_to_scatter +
148 "\" "
149 "sha256=\"not-used\" "
150 "needsadmin=\"" +
151 needsadmin + "\" " +
152 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
153 (disable_p2p_for_downloading ? "DisableP2PForDownloading=\"true\" "
154 : "") +
155 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
Sen Jiang81259682017-03-30 15:11:30 -0700156 "/></actions></manifest></updatecheck></app>" +
157 (multi_app
Sen Jiangb1e063a2017-09-15 17:44:31 -0700158 ? "<app appid=\"" + app_id2 + "\"" +
159 (include_cohorts ? " cohort=\"cohort2\"" : "") +
160 "><updatecheck status=\"ok\"><urls><url codebase=\"" +
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700161 codebase2 + "\"/></urls><manifest version=\"" + version2 +
162 "\"><packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700163 "<package name=\"package3\" size=\"333\" "
164 "hash_sha256=\"hash3\"/></packages>"
Sen Jiang00adf7b2017-06-26 15:57:29 -0700165 "<actions><action event=\"postinstall\" " +
166 (multi_app_self_update
167 ? "noupdate=\"true\" IsDeltaPayload=\"true\" "
168 : "IsDeltaPayload=\"false\" ") +
169 "MetadataSize=\"33\"/></actions>"
Sen Jiang81259682017-03-30 15:11:30 -0700170 "</manifest></updatecheck></app>"
171 : "") +
172 (multi_app_no_update
173 ? "<app><updatecheck status=\"noupdate\"/></app>"
174 : "") +
175 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700176 }
177
178 // Return the payload URL, which is split in two fields in the XML response.
179 string GetPayloadUrl() {
180 return codebase + filename;
181 }
182
Alex Deymo85616652015-10-15 18:48:31 -0700183 string app_id = kTestAppId;
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700184 string app_id2 = kTestAppId2;
Alex Deymo8e18f932015-03-27 16:16:59 -0700185 string version = "1.2.3.4";
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700186 string version2 = "2.3.4.5";
Alex Deymo8e18f932015-03-27 16:16:59 -0700187 string more_info_url = "http://more/info";
188 string prompt = "true";
189 string codebase = "http://code/base/";
Sen Jiang81259682017-03-30 15:11:30 -0700190 string codebase2 = "http://code/base/2/";
Alex Deymo8e18f932015-03-27 16:16:59 -0700191 string filename = "file.signed";
Sen Jiang2703ef42017-03-16 13:36:21 -0700192 string hash = "4841534831323334";
Alex Deymo8e18f932015-03-27 16:16:59 -0700193 string needsadmin = "false";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800194 uint64_t size = 123;
Alex Deymo8e18f932015-03-27 16:16:59 -0700195 string deadline = "";
196 string max_days_to_scatter = "7";
197 string elapsed_days = "42";
198
199 // P2P setting defaults to allowed.
200 bool disable_p2p_for_downloading = false;
201 bool disable_p2p_for_sharing = false;
202
203 // Omaha cohorts settings.
204 bool include_cohorts = false;
205 string cohort = "";
206 string cohorthint = "";
207 string cohortname = "";
208
209 // Whether to include the CrOS <!ENTITY> in the XML response.
210 bool include_entity = false;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800211
Sen Jiang81259682017-03-30 15:11:30 -0700212 // Whether to include more than one app.
213 bool multi_app = false;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700214 // Whether to include an app with noupdate="true".
215 bool multi_app_self_update = false;
216 // Whether to include an additional app with status="noupdate".
Sen Jiang81259682017-03-30 15:11:30 -0700217 bool multi_app_no_update = false;
218 // Whether to include more than one package in an app.
Sen Jiang0affc2c2017-02-10 15:55:05 -0800219 bool multi_package = false;
Alex Deymo8e18f932015-03-27 16:16:59 -0700220};
221
222} // namespace
223
Darin Petkov6a5b3222010-07-13 14:55:28 -0700224namespace chromeos_update_engine {
225
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700226class OmahaRequestActionTest : public ::testing::Test {
227 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800228 void SetUp() override {
Marton Hunyady2abda312018-04-24 18:21:49 +0200229 request_params_.set_os_sp("service_pack");
230 request_params_.set_os_board("x86-generic");
231 request_params_.set_app_id(kTestAppId);
232 request_params_.set_app_version("0.1.0.0");
233 request_params_.set_app_lang("en-US");
234 request_params_.set_current_channel("unittest");
235 request_params_.set_target_channel("unittest");
236 request_params_.set_hwid("OEM MODEL 09235 7471");
237 request_params_.set_fw_version("ChromeOSFirmware.1.0");
238 request_params_.set_ec_version("0X0A1");
239 request_params_.set_delta_okay(true);
240 request_params_.set_interactive(false);
241 request_params_.set_update_url("http://url");
242 request_params_.set_target_version_prefix("");
243 request_params_.set_is_powerwash_allowed(false);
244
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700245 fake_system_state_.set_request_params(&request_params_);
246 fake_system_state_.set_prefs(&fake_prefs_);
247 }
248
249 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800250 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700251 // is used. |payload_state| may be null, in which case a local mock is used.
252 // |p2p_manager| may be null, in which case a local mock is used.
253 // |connection_manager| may be null, in which case a local mock is used.
254 // out_response may be null. If |fail_http_response_code| is non-negative,
255 // the transfer will fail with that code. |ping_only| is passed through to the
256 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
257 // post-data received by the mock HttpFetcher is returned.
258 //
259 // The |expected_check_result|, |expected_check_reaction| and
260 // |expected_error_code| parameters are for checking expectations
261 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
262 // UMA statistics. Use the appropriate ::kUnset value to specify that
263 // the given metric should not be reported.
Marton Hunyady2abda312018-04-24 18:21:49 +0200264 bool TestUpdateCheck(const string& http_response,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700265 int fail_http_response_code,
266 bool ping_only,
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800267 bool is_consumer_device,
268 int rollback_allowed_milestones,
269 bool is_policy_loaded,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700270 ErrorCode expected_code,
271 metrics::CheckResult expected_check_result,
272 metrics::CheckReaction expected_check_reaction,
273 metrics::DownloadErrorCode expected_download_error_code,
274 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700275 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700276
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800277 // Overload of TestUpdateCheck that does not supply |is_consumer_device| or
278 // |rollback_allowed_milestones| which are only required for rollback tests.
279 bool TestUpdateCheck(const string& http_response,
280 int fail_http_response_code,
281 bool ping_only,
282 ErrorCode expected_code,
283 metrics::CheckResult expected_check_result,
284 metrics::CheckReaction expected_check_reaction,
285 metrics::DownloadErrorCode expected_download_error_code,
286 OmahaResponse* out_response,
287 brillo::Blob* out_post_data);
288
289 void TestRollbackCheck(bool is_consumer_device,
290 int rollback_allowed_milestones,
291 bool is_policy_loaded);
292
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800293 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700294 // only a ping or also an updatecheck.
295 void PingTest(bool ping_only);
296
297 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700298 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700299 OmahaResponse *response);
300
301 // P2P test helper function.
302 void P2PTest(
303 bool initial_allow_p2p_for_downloading,
304 bool initial_allow_p2p_for_sharing,
305 bool omaha_disable_p2p_for_downloading,
306 bool omaha_disable_p2p_for_sharing,
307 bool payload_state_allow_p2p_attempt,
308 bool expect_p2p_client_lookup,
309 const string& p2p_client_result_url,
310 bool expected_allow_p2p_for_downloading,
311 bool expected_allow_p2p_for_sharing,
312 const string& expected_p2p_url);
313
314 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700315 FakeUpdateResponse fake_update_response_;
Marton Hunyady2abda312018-04-24 18:21:49 +0200316 // Used by all tests.
317 OmahaRequestParams request_params_{&fake_system_state_};
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700318
319 FakePrefs fake_prefs_;
320};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700321
322namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700323class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
324 public:
325 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700326 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800327 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700328 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800329 void ProcessingDone(const ActionProcessor* processor,
330 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700331 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700332 }
333
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800334 void ActionCompleted(ActionProcessor* processor,
335 AbstractAction* action,
336 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700337 // make sure actions always succeed
338 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700339 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700340 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700341 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700342 }
David Zeuthena99981f2013-04-29 13:42:47 -0700343 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700344};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700345} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700346
347class OutputObjectCollectorAction;
348
349template<>
350class ActionTraits<OutputObjectCollectorAction> {
351 public:
352 // Does not take an object for input
353 typedef OmahaResponse InputObjectType;
354 // On success, puts the output path on output
355 typedef NoneType OutputObjectType;
356};
357
358class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
359 public:
360 OutputObjectCollectorAction() : has_input_object_(false) {}
361 void PerformAction() {
362 // copy input object
363 has_input_object_ = HasInputObject();
364 if (has_input_object_)
365 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700366 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700367 }
368 // Should never be called
369 void TerminateProcessing() {
370 CHECK(false);
371 }
372 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700373 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700374 return "OutputObjectCollectorAction";
375 }
Alex Deymof329b932014-10-30 01:37:48 -0700376 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800377 using InputObjectType =
378 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
379 using OutputObjectType =
380 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700381 bool has_input_object_;
382 OmahaResponse omaha_response_;
383};
384
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700385bool OmahaRequestActionTest::TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700386 const string& http_response,
387 int fail_http_response_code,
388 bool ping_only,
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800389 bool is_consumer_device,
390 int rollback_allowed_milestones,
391 bool is_policy_loaded,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700392 ErrorCode expected_code,
393 metrics::CheckResult expected_check_result,
394 metrics::CheckReaction expected_check_reaction,
395 metrics::DownloadErrorCode expected_download_error_code,
396 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700397 brillo::Blob* out_post_data) {
398 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700399 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700400 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800401 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700402 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700403 if (fail_http_response_code >= 0) {
404 fetcher->FailTransfer(fail_http_response_code);
405 }
Marton Hunyady2abda312018-04-24 18:21:49 +0200406 // This ensures the tests didn't forget to update fake_system_state_ if they
407 // are not using the default request_params_.
408 EXPECT_EQ(&request_params_, fake_system_state_.request_params());
409
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700410 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700411 nullptr,
Ben Chan5c02c132017-06-27 07:10:36 -0700412 base::WrapUnique(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700413 ping_only);
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800414 auto mock_policy_provider =
415 std::make_unique<NiceMock<policy::MockPolicyProvider>>();
416 EXPECT_CALL(*mock_policy_provider, IsConsumerDevice())
417 .WillRepeatedly(Return(is_consumer_device));
418
419 EXPECT_CALL(*mock_policy_provider, device_policy_is_loaded())
420 .WillRepeatedly(Return(is_policy_loaded));
421
422 const policy::MockDevicePolicy device_policy;
423 const bool get_allowed_milestone_succeeds = rollback_allowed_milestones >= 0;
424 EXPECT_CALL(device_policy, GetRollbackAllowedMilestones(_))
425 .WillRepeatedly(DoAll(SetArgPointee<0>(rollback_allowed_milestones),
426 Return(get_allowed_milestone_succeeds)));
427
428 EXPECT_CALL(*mock_policy_provider, GetDevicePolicy())
429 .WillRepeatedly(ReturnRef(device_policy));
430
431 action.policy_provider_ = std::move(mock_policy_provider);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700432 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700433 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700434
Darin Petkov6a5b3222010-07-13 14:55:28 -0700435 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700436 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700437 processor.EnqueueAction(&action);
438
439 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700440 BondActions(&action, &collector_action);
441 processor.EnqueueAction(&collector_action);
442
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700443 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
444 ReportUpdateCheckMetrics(_, _, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800445 .Times(AnyNumber());
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700446
447 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
448 ReportUpdateCheckMetrics(_,
449 expected_check_result,
450 expected_check_reaction,
451 expected_download_error_code))
452 .Times(ping_only ? 0 : 1);
David Zeuthen33bae492014-02-25 16:16:18 -0800453
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700454 loop.PostTask(base::Bind(
455 [](ActionProcessor* processor) { processor->StartProcessing(); },
456 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700457 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700458 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700459 if (collector_action.has_input_object_ && out_response)
460 *out_response = collector_action.omaha_response_;
461 if (out_post_data)
462 *out_post_data = fetcher->post_data();
463 return collector_action.has_input_object_;
464}
465
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800466bool OmahaRequestActionTest::TestUpdateCheck(
467 const string& http_response,
468 int fail_http_response_code,
469 bool ping_only,
470 ErrorCode expected_code,
471 metrics::CheckResult expected_check_result,
472 metrics::CheckReaction expected_check_reaction,
473 metrics::DownloadErrorCode expected_download_error_code,
474 OmahaResponse* out_response,
475 brillo::Blob* out_post_data) {
476 return TestUpdateCheck(http_response,
477 fail_http_response_code,
478 ping_only,
479 true, // is_consumer_device
480 0, // rollback_allowed_milestones
481 false, // is_policy_loaded
482 expected_code,
483 expected_check_result,
484 expected_check_reaction,
485 expected_download_error_code,
486 out_response,
487 out_post_data);
488}
489
490void OmahaRequestActionTest::TestRollbackCheck(bool is_consumer_device,
491 int rollback_allowed_milestones,
492 bool is_policy_loaded) {
493 OmahaResponse response;
494 fake_update_response_.deadline = "20101020";
495 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
496 -1,
497 false, // ping_only
498 is_consumer_device,
499 rollback_allowed_milestones,
500 is_policy_loaded,
501 ErrorCode::kSuccess,
502 metrics::CheckResult::kUpdateAvailable,
503 metrics::CheckReaction::kUpdating,
504 metrics::DownloadErrorCode::kUnset,
505 &response,
506 nullptr));
507 ASSERT_TRUE(response.update_exists);
508}
509
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700510// Tests Event requests -- they should always succeed. |out_post_data|
511// may be null; if non-null, the post-data received by the mock
512// HttpFetcher is returned.
Marton Hunyady2abda312018-04-24 18:21:49 +0200513void TestEvent(OmahaEvent* event,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700514 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700515 brillo::Blob* out_post_data) {
516 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700517 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700518 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800519 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700520 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700521 FakeSystemState fake_system_state;
Alex Deymoc1c17b42015-11-23 03:53:15 -0300522 OmahaRequestAction action(&fake_system_state,
523 event,
Ben Chan5c02c132017-06-27 07:10:36 -0700524 base::WrapUnique(fetcher),
Alex Deymoc1c17b42015-11-23 03:53:15 -0300525 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700526 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700527 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700528 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700529 processor.EnqueueAction(&action);
530
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700531 loop.PostTask(base::Bind(
532 [](ActionProcessor* processor) { processor->StartProcessing(); },
533 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700534 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700535 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700536
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700537 if (out_post_data)
538 *out_post_data = fetcher->post_data();
539}
540
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700541TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400542 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700543 fake_update_response_.include_entity = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200544 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
545 -1,
546 false, // ping_only
547 ErrorCode::kOmahaRequestXMLHasEntityDecl,
548 metrics::CheckResult::kParsingError,
549 metrics::CheckReaction::kUnset,
550 metrics::DownloadErrorCode::kUnset,
551 &response,
552 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400553 EXPECT_FALSE(response.update_exists);
554}
555
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700556TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700557 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +0200558 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
559 -1,
560 false, // ping_only
561 ErrorCode::kSuccess,
562 metrics::CheckResult::kNoUpdateAvailable,
563 metrics::CheckReaction::kUnset,
564 metrics::DownloadErrorCode::kUnset,
565 &response,
566 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700567 EXPECT_FALSE(response.update_exists);
568}
569
Sen Jiang81259682017-03-30 15:11:30 -0700570TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
571 OmahaResponse response;
572 fake_update_response_.multi_app_no_update = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200573 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700574 -1,
575 false, // ping_only
576 ErrorCode::kSuccess,
577 metrics::CheckResult::kNoUpdateAvailable,
578 metrics::CheckReaction::kUnset,
579 metrics::DownloadErrorCode::kUnset,
580 &response,
581 nullptr));
582 EXPECT_FALSE(response.update_exists);
583}
584
Sen Jiang00adf7b2017-06-26 15:57:29 -0700585TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
586 OmahaResponse response;
587 fake_update_response_.multi_app_no_update = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200588 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang00adf7b2017-06-26 15:57:29 -0700589 -1,
590 false, // ping_only
591 ErrorCode::kSuccess,
592 metrics::CheckResult::kNoUpdateAvailable,
593 metrics::CheckReaction::kUnset,
594 metrics::DownloadErrorCode::kUnset,
595 &response,
596 nullptr));
597 EXPECT_FALSE(response.update_exists);
598}
599
600TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
601 OmahaResponse response;
602 ASSERT_TRUE(TestUpdateCheck(
Sen Jiang00adf7b2017-06-26 15:57:29 -0700603 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
604 "event=\"postinstall\" noupdate=\"true\"/></actions>"
605 "</manifest></updatecheck></app></response>",
606 -1,
607 false, // ping_only
608 ErrorCode::kSuccess,
609 metrics::CheckResult::kNoUpdateAvailable,
610 metrics::CheckReaction::kUnset,
611 metrics::DownloadErrorCode::kUnset,
612 &response,
613 nullptr));
614 EXPECT_FALSE(response.update_exists);
615}
616
Alex Deymo8e18f932015-03-27 16:16:59 -0700617// Test that all the values in the response are parsed in a normal update
618// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700619TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700620 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700621 fake_update_response_.deadline = "20101020";
Marton Hunyady2abda312018-04-24 18:21:49 +0200622 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
623 -1,
624 false, // ping_only
625 ErrorCode::kSuccess,
626 metrics::CheckResult::kUpdateAvailable,
627 metrics::CheckReaction::kUpdating,
628 metrics::DownloadErrorCode::kUnset,
629 &response,
630 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700631 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700632 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700633 EXPECT_EQ("", response.system_version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800634 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
635 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700636 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800637 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
638 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700639 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700640 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
641 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700642 // Omaha cohort attributes are not set in the response, so they should not be
Alex Deymo8e18f932015-03-27 16:16:59 -0700643 // persisted.
644 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
645 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
646 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700647}
648
Sen Jiang0affc2c2017-02-10 15:55:05 -0800649TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
650 OmahaResponse response;
651 fake_update_response_.multi_package = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200652 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang0affc2c2017-02-10 15:55:05 -0800653 -1,
654 false, // ping_only
655 ErrorCode::kSuccess,
656 metrics::CheckResult::kUpdateAvailable,
657 metrics::CheckReaction::kUpdating,
658 metrics::DownloadErrorCode::kUnset,
659 &response,
660 nullptr));
661 EXPECT_TRUE(response.update_exists);
662 EXPECT_EQ(fake_update_response_.version, response.version);
663 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
664 response.packages[0].payload_urls[0]);
665 EXPECT_EQ(fake_update_response_.codebase + "package2",
666 response.packages[1].payload_urls[0]);
667 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
668 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700669 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700670 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800671 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700672 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800673 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700674 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700675 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700676}
677
678TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
679 OmahaResponse response;
680 fake_update_response_.multi_app = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200681 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700682 -1,
683 false, // ping_only
684 ErrorCode::kSuccess,
685 metrics::CheckResult::kUpdateAvailable,
686 metrics::CheckReaction::kUpdating,
687 metrics::DownloadErrorCode::kUnset,
688 &response,
689 nullptr));
690 EXPECT_TRUE(response.update_exists);
691 EXPECT_EQ(fake_update_response_.version, response.version);
692 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
693 response.packages[0].payload_urls[0]);
694 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
695 response.packages[1].payload_urls[0]);
696 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
697 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
698 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700699 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700700 ASSERT_EQ(2u, response.packages.size());
701 EXPECT_EQ(string("hash3"), response.packages[1].hash);
702 EXPECT_EQ(333u, response.packages[1].size);
703 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700704 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700705}
706
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700707TEST_F(OmahaRequestActionTest, MultiAppAndSystemUpdateTest) {
708 OmahaResponse response;
709 fake_update_response_.multi_app = true;
710 // trigger the lining up of the app and system versions
711 request_params_.set_system_app_id(fake_update_response_.app_id2);
712
Marton Hunyady2abda312018-04-24 18:21:49 +0200713 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700714 -1,
715 false, // ping_only
716 ErrorCode::kSuccess,
717 metrics::CheckResult::kUpdateAvailable,
718 metrics::CheckReaction::kUpdating,
719 metrics::DownloadErrorCode::kUnset,
720 &response,
721 nullptr));
722 EXPECT_TRUE(response.update_exists);
723 EXPECT_EQ(fake_update_response_.version, response.version);
724 EXPECT_EQ(fake_update_response_.version2, response.system_version);
725 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
726 response.packages[0].payload_urls[0]);
727 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
728 response.packages[1].payload_urls[0]);
729 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
730 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
731 EXPECT_EQ(11u, response.packages[0].metadata_size);
732 EXPECT_EQ(true, response.packages[0].is_delta);
733 ASSERT_EQ(2u, response.packages.size());
734 EXPECT_EQ(string("hash3"), response.packages[1].hash);
735 EXPECT_EQ(333u, response.packages[1].size);
736 EXPECT_EQ(33u, response.packages[1].metadata_size);
737 EXPECT_EQ(false, response.packages[1].is_delta);
738}
739
Sen Jiang81259682017-03-30 15:11:30 -0700740TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
741 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700742 fake_update_response_.multi_app = true;
743 fake_update_response_.multi_app_self_update = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200744 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700745 -1,
746 false, // ping_only
747 ErrorCode::kSuccess,
748 metrics::CheckResult::kUpdateAvailable,
749 metrics::CheckReaction::kUpdating,
750 metrics::DownloadErrorCode::kUnset,
751 &response,
752 nullptr));
753 EXPECT_TRUE(response.update_exists);
754 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700755 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700756 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
757 response.packages[0].payload_urls[0]);
758 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
759 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
760 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700761 ASSERT_EQ(2u, response.packages.size());
762 EXPECT_EQ(string("hash3"), response.packages[1].hash);
763 EXPECT_EQ(333u, response.packages[1].size);
764 EXPECT_EQ(33u, response.packages[1].metadata_size);
765 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700766}
767
768TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
769 OmahaResponse response;
770 fake_update_response_.multi_app = true;
771 fake_update_response_.multi_package = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200772 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700773 -1,
774 false, // ping_only
775 ErrorCode::kSuccess,
776 metrics::CheckResult::kUpdateAvailable,
777 metrics::CheckReaction::kUpdating,
778 metrics::DownloadErrorCode::kUnset,
779 &response,
780 nullptr));
781 EXPECT_TRUE(response.update_exists);
782 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700783 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700784 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
785 response.packages[0].payload_urls[0]);
786 EXPECT_EQ(fake_update_response_.codebase + "package2",
787 response.packages[1].payload_urls[0]);
788 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
789 response.packages[2].payload_urls[0]);
790 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
791 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
792 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700793 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700794 ASSERT_EQ(3u, response.packages.size());
795 EXPECT_EQ(string("hash2"), response.packages[1].hash);
796 EXPECT_EQ(222u, response.packages[1].size);
797 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700798 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700799 EXPECT_EQ(string("hash3"), response.packages[2].hash);
800 EXPECT_EQ(333u, response.packages[2].size);
801 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700802 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800803}
804
Alex Deymo14ad88e2016-06-29 12:30:14 -0700805TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
806 const string http_response = "<?xml invalid response";
807 request_params_.set_interactive(true);
808
809 brillo::FakeMessageLoop loop(nullptr);
810 loop.SetAsCurrent();
811
812 MockHttpFetcher* fetcher =
813 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
Ben Chan5c02c132017-06-27 07:10:36 -0700814 OmahaRequestAction action(&fake_system_state_, nullptr,
815 base::WrapUnique(fetcher), false);
Alex Deymo14ad88e2016-06-29 12:30:14 -0700816 ActionProcessor processor;
817 processor.EnqueueAction(&action);
818
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700819 loop.PostTask(base::Bind(
820 [](ActionProcessor* processor) { processor->StartProcessing(); },
821 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700822 loop.Run();
823 EXPECT_FALSE(loop.PendingTasks());
824
825 // Check that the headers were set in the fetcher during the action. Note that
826 // we set this request as "interactive".
Sen Jiang42fa45e2018-03-12 11:02:14 -0700827 EXPECT_EQ("fg", fetcher->GetHeader("X-Goog-Update-Interactivity"));
828 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-Goog-Update-AppId"));
829 EXPECT_NE("", fetcher->GetHeader("X-Goog-Update-Updater"));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700830}
831
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700832TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700833 OmahaResponse response;
834 // Set up a connection manager that doesn't allow a valid update over
835 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700836 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700837 fake_system_state_.set_connection_manager(&mock_cm);
838
Alex Deymo30534502015-07-20 15:06:33 -0700839 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
840 .WillRepeatedly(
Ben Chan672c1f52017-10-23 15:41:39 -0700841 DoAll(SetArgPointee<0>(ConnectionType::kEthernet),
842 SetArgPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700843 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700844 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
845 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700846
Marton Hunyady2abda312018-04-24 18:21:49 +0200847 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
848 -1,
849 false, // ping_only
850 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
851 metrics::CheckResult::kUpdateAvailable,
852 metrics::CheckReaction::kIgnored,
853 metrics::DownloadErrorCode::kUnset,
854 &response,
855 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700856 EXPECT_FALSE(response.update_exists);
857}
858
Weidong Guo421ff332017-04-17 10:08:38 -0700859TEST_F(OmahaRequestActionTest, ValidUpdateOverCellularAllowedByDevicePolicy) {
860 // This test tests that update over cellular is allowed as device policy
861 // says yes.
862 OmahaResponse response;
863 MockConnectionManager mock_cm;
864
865 fake_system_state_.set_connection_manager(&mock_cm);
866
867 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
868 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
869 SetArgPointee<1>(ConnectionTethering::kUnknown),
870 Return(true)));
871 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
872 .WillRepeatedly(Return(true));
873 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
874 .WillRepeatedly(Return(true));
875
Marton Hunyady2abda312018-04-24 18:21:49 +0200876 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700877 -1,
878 false, // ping_only
879 ErrorCode::kSuccess,
880 metrics::CheckResult::kUpdateAvailable,
881 metrics::CheckReaction::kUpdating,
882 metrics::DownloadErrorCode::kUnset,
883 &response,
884 nullptr));
885 EXPECT_TRUE(response.update_exists);
886}
887
888TEST_F(OmahaRequestActionTest, ValidUpdateOverCellularBlockedByDevicePolicy) {
889 // This test tests that update over cellular is blocked as device policy
890 // says no.
891 OmahaResponse response;
892 MockConnectionManager mock_cm;
893
894 fake_system_state_.set_connection_manager(&mock_cm);
895
896 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
897 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
898 SetArgPointee<1>(ConnectionTethering::kUnknown),
899 Return(true)));
900 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
901 .WillRepeatedly(Return(true));
902 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
903 .WillRepeatedly(Return(false));
904
Marton Hunyady2abda312018-04-24 18:21:49 +0200905 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700906 -1,
907 false, // ping_only
908 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
909 metrics::CheckResult::kUpdateAvailable,
910 metrics::CheckReaction::kIgnored,
911 metrics::DownloadErrorCode::kUnset,
912 &response,
913 nullptr));
914 EXPECT_FALSE(response.update_exists);
915}
916
917TEST_F(OmahaRequestActionTest,
918 ValidUpdateOverCellularAllowedByUserPermissionTrue) {
919 // This test tests that, when device policy is not set, update over cellular
920 // is allowed as permission for update over cellular is set to true.
921 OmahaResponse response;
922 MockConnectionManager mock_cm;
923
924 fake_prefs_.SetBoolean(kPrefsUpdateOverCellularPermission, true);
925 fake_system_state_.set_connection_manager(&mock_cm);
926
927 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
928 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
929 SetArgPointee<1>(ConnectionTethering::kUnknown),
930 Return(true)));
931 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
932 .WillRepeatedly(Return(false));
933 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
934 .WillRepeatedly(Return(true));
935
Marton Hunyady2abda312018-04-24 18:21:49 +0200936 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700937 -1,
938 false, // ping_only
939 ErrorCode::kSuccess,
940 metrics::CheckResult::kUpdateAvailable,
941 metrics::CheckReaction::kUpdating,
942 metrics::DownloadErrorCode::kUnset,
943 &response,
944 nullptr));
945 EXPECT_TRUE(response.update_exists);
946}
947
948TEST_F(OmahaRequestActionTest,
949 ValidUpdateOverCellularBlockedByUpdateTargetNotMatch) {
950 // This test tests that, when device policy is not set and permission for
951 // update over cellular is set to false or does not exist, update over
952 // cellular is blocked as update target does not match the omaha response.
953 OmahaResponse response;
954 MockConnectionManager mock_cm;
955 // A version different from the version in omaha response.
956 string diff_version = "99.99.99";
957 // A size different from the size in omaha response.
958 int64_t diff_size = 999;
959
960 fake_prefs_.SetString(kPrefsUpdateOverCellularTargetVersion, diff_version);
961 fake_prefs_.SetInt64(kPrefsUpdateOverCellularTargetSize, diff_size);
962 // This test tests cellular (3G) being the only connection type being allowed.
963 fake_system_state_.set_connection_manager(&mock_cm);
964
965 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
966 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
967 SetArgPointee<1>(ConnectionTethering::kUnknown),
968 Return(true)));
969 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
970 .WillRepeatedly(Return(false));
971 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
972 .WillRepeatedly(Return(true));
973
Marton Hunyady2abda312018-04-24 18:21:49 +0200974 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700975 -1,
976 false, // ping_only
977 ErrorCode::kOmahaUpdateIgnoredOverCellular,
978 metrics::CheckResult::kUpdateAvailable,
979 metrics::CheckReaction::kIgnored,
980 metrics::DownloadErrorCode::kUnset,
981 &response,
982 nullptr));
983 EXPECT_FALSE(response.update_exists);
984}
985
986TEST_F(OmahaRequestActionTest,
987 ValidUpdateOverCellularAllowedByUpdateTargetMatch) {
988 // This test tests that, when device policy is not set and permission for
989 // update over cellular is set to false or does not exist, update over
990 // cellular is allowed as update target matches the omaha response.
991 OmahaResponse response;
992 MockConnectionManager mock_cm;
993 // A version same as the version in omaha response.
994 string new_version = fake_update_response_.version;
995 // A size same as the size in omaha response.
996 int64_t new_size = fake_update_response_.size;
997
998 fake_prefs_.SetString(kPrefsUpdateOverCellularTargetVersion, new_version);
999 fake_prefs_.SetInt64(kPrefsUpdateOverCellularTargetSize, new_size);
1000 fake_system_state_.set_connection_manager(&mock_cm);
1001
1002 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
1003 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
1004 SetArgPointee<1>(ConnectionTethering::kUnknown),
1005 Return(true)));
1006 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
1007 .WillRepeatedly(Return(false));
1008 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
1009 .WillRepeatedly(Return(true));
1010
Marton Hunyady2abda312018-04-24 18:21:49 +02001011 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -07001012 -1,
1013 false, // ping_only
1014 ErrorCode::kSuccess,
1015 metrics::CheckResult::kUpdateAvailable,
1016 metrics::CheckReaction::kUpdating,
1017 metrics::DownloadErrorCode::kUnset,
1018 &response,
1019 nullptr));
1020 EXPECT_TRUE(response.update_exists);
1021}
1022
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001023TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -07001024 string rollback_version = "1234.0.0";
1025 OmahaResponse response;
1026
1027 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001028 fake_system_state_.set_payload_state(&mock_payload_state);
1029
Chris Sosa77f79e82014-06-02 18:16:24 -07001030 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
1031 .WillRepeatedly(Return(rollback_version));
1032
Alex Deymo8e18f932015-03-27 16:16:59 -07001033 fake_update_response_.version = rollback_version;
Marton Hunyady2abda312018-04-24 18:21:49 +02001034 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1035 -1,
1036 false, // ping_only
1037 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
1038 metrics::CheckResult::kUpdateAvailable,
1039 metrics::CheckReaction::kIgnored,
1040 metrics::DownloadErrorCode::kUnset,
1041 &response,
1042 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -07001043 EXPECT_FALSE(response.update_exists);
1044}
1045
Kevin Cernekee2494e282016-03-29 18:03:53 -07001046// Verify that update checks called during OOBE will only try to download
1047// an update if the response includes a non-empty deadline field.
1048TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
1049 OmahaResponse response;
1050
Sen Jiang8cd42342018-01-31 12:06:59 -08001051 // TODO(senj): set better default value for metrics::checkresult in
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001052 // OmahaRequestAction::ActionCompleted.
Kevin Cernekee2494e282016-03-29 18:03:53 -07001053 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Marton Hunyady2abda312018-04-24 18:21:49 +02001054 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001055 -1,
1056 false, // ping_only
1057 ErrorCode::kNonCriticalUpdateInOOBE,
1058 metrics::CheckResult::kParsingError,
1059 metrics::CheckReaction::kUnset,
1060 metrics::DownloadErrorCode::kUnset,
1061 &response,
1062 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -07001063 EXPECT_FALSE(response.update_exists);
1064
Alex Deymo46a9aae2016-05-04 20:20:11 -07001065 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
1066 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
Marton Hunyady2abda312018-04-24 18:21:49 +02001067 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo46a9aae2016-05-04 20:20:11 -07001068 -1,
1069 false, // ping_only
1070 ErrorCode::kSuccess,
1071 metrics::CheckResult::kUpdateAvailable,
1072 metrics::CheckReaction::kUpdating,
1073 metrics::DownloadErrorCode::kUnset,
1074 &response,
1075 nullptr));
1076 EXPECT_TRUE(response.update_exists);
1077 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
1078
1079 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -07001080 fake_update_response_.deadline = "20101020";
Marton Hunyady2abda312018-04-24 18:21:49 +02001081 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1082 -1,
1083 false, // ping_only
1084 ErrorCode::kSuccess,
1085 metrics::CheckResult::kUpdateAvailable,
1086 metrics::CheckReaction::kUpdating,
1087 metrics::DownloadErrorCode::kUnset,
1088 &response,
1089 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -07001090 EXPECT_TRUE(response.update_exists);
1091}
1092
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001093TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001094 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001095 request_params_.set_wall_clock_based_wait_enabled(true);
1096 request_params_.set_update_check_count_wait_enabled(false);
1097 request_params_.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001098
Marton Hunyady2abda312018-04-24 18:21:49 +02001099 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1100 -1,
1101 false, // ping_only
1102 ErrorCode::kOmahaUpdateDeferredPerPolicy,
1103 metrics::CheckResult::kUpdateAvailable,
1104 metrics::CheckReaction::kDeferring,
1105 metrics::DownloadErrorCode::kUnset,
1106 &response,
1107 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001108 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001109
1110 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02001111 request_params_.set_interactive(true);
1112 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1113 -1,
1114 false, // ping_only
1115 ErrorCode::kSuccess,
1116 metrics::CheckResult::kUpdateAvailable,
1117 metrics::CheckReaction::kUpdating,
1118 metrics::DownloadErrorCode::kUnset,
1119 &response,
1120 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001121 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001122}
1123
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001124TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001125 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001126 request_params_.set_wall_clock_based_wait_enabled(false);
1127 request_params_.set_waiting_period(TimeDelta::FromDays(2));
1128 request_params_.set_update_check_count_wait_enabled(true);
1129 request_params_.set_min_update_checks_needed(1);
1130 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001131
Marton Hunyady2abda312018-04-24 18:21:49 +02001132 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1133 -1,
1134 false, // ping_only
1135 ErrorCode::kSuccess,
1136 metrics::CheckResult::kUpdateAvailable,
1137 metrics::CheckReaction::kUpdating,
1138 metrics::DownloadErrorCode::kUnset,
1139 &response,
1140 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001141 EXPECT_TRUE(response.update_exists);
1142}
1143
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001144TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001145 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001146 request_params_.set_wall_clock_based_wait_enabled(true);
1147 request_params_.set_waiting_period(TimeDelta::FromDays(2));
1148 request_params_.set_update_check_count_wait_enabled(true);
1149 request_params_.set_min_update_checks_needed(1);
1150 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001151
Alex Deymo8e18f932015-03-27 16:16:59 -07001152 fake_update_response_.max_days_to_scatter = "0";
Marton Hunyady2abda312018-04-24 18:21:49 +02001153 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1154 -1,
1155 false, // ping_only
1156 ErrorCode::kSuccess,
1157 metrics::CheckResult::kUpdateAvailable,
1158 metrics::CheckReaction::kUpdating,
1159 metrics::DownloadErrorCode::kUnset,
1160 &response,
1161 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001162 EXPECT_TRUE(response.update_exists);
1163}
1164
1165
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001166TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001167 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001168 request_params_.set_wall_clock_based_wait_enabled(true);
1169 request_params_.set_waiting_period(TimeDelta());
1170 request_params_.set_update_check_count_wait_enabled(true);
1171 request_params_.set_min_update_checks_needed(0);
1172 request_params_.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001173
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001174 ASSERT_TRUE(TestUpdateCheck(
Alex Deymo8e18f932015-03-27 16:16:59 -07001175 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001176 -1,
1177 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001178 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001179 metrics::CheckResult::kUpdateAvailable,
1180 metrics::CheckReaction::kUpdating,
1181 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001182 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001183 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001184
Ben Chan9abb7632014-08-07 00:10:53 -07001185 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001186 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001187 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001188 EXPECT_TRUE(response.update_exists);
1189}
1190
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001191TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001192 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001193 request_params_.set_wall_clock_based_wait_enabled(true);
1194 request_params_.set_waiting_period(TimeDelta());
1195 request_params_.set_update_check_count_wait_enabled(true);
1196 request_params_.set_min_update_checks_needed(1);
1197 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001198
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001199 ASSERT_FALSE(TestUpdateCheck(
Alex Deymo8e18f932015-03-27 16:16:59 -07001200 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001201 -1,
1202 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001203 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001204 metrics::CheckResult::kUpdateAvailable,
1205 metrics::CheckReaction::kDeferring,
1206 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001207 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001208 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001209
Ben Chan9abb7632014-08-07 00:10:53 -07001210 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001211 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001212 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001213 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001214
1215 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02001216 request_params_.set_interactive(true);
1217 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1218 -1,
1219 false, // ping_only
1220 ErrorCode::kSuccess,
1221 metrics::CheckResult::kUpdateAvailable,
1222 metrics::CheckReaction::kUpdating,
1223 metrics::DownloadErrorCode::kUnset,
1224 &response,
1225 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001226 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001227}
1228
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001229TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001230 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001231 request_params_.set_wall_clock_based_wait_enabled(true);
1232 request_params_.set_waiting_period(TimeDelta());
1233 request_params_.set_update_check_count_wait_enabled(true);
1234 request_params_.set_min_update_checks_needed(1);
1235 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001236
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001237 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001238
1239 ASSERT_FALSE(TestUpdateCheck(
Alex Deymo8e18f932015-03-27 16:16:59 -07001240 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001241 -1,
1242 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001243 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001244 metrics::CheckResult::kUpdateAvailable,
1245 metrics::CheckReaction::kDeferring,
1246 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001247 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001248 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001249
Ben Chan9abb7632014-08-07 00:10:53 -07001250 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001251 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001252 // count remains the same, as the decrementing happens in update_attempter
1253 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001254 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001255 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001256
1257 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02001258 request_params_.set_interactive(true);
1259 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1260 -1,
1261 false, // ping_only
1262 ErrorCode::kSuccess,
1263 metrics::CheckResult::kUpdateAvailable,
1264 metrics::CheckReaction::kUpdating,
1265 metrics::DownloadErrorCode::kUnset,
1266 &response,
1267 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001268 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001269}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001270
Alex Deymo8e18f932015-03-27 16:16:59 -07001271TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1272 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001273 fake_update_response_.include_cohorts = true;
1274 fake_update_response_.cohort = "s/154454/8479665";
1275 fake_update_response_.cohorthint = "please-put-me-on-beta";
1276 fake_update_response_.cohortname = "stable";
1277
Marton Hunyady2abda312018-04-24 18:21:49 +02001278 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo8e18f932015-03-27 16:16:59 -07001279 -1,
1280 false, // ping_only
1281 ErrorCode::kSuccess,
1282 metrics::CheckResult::kUpdateAvailable,
1283 metrics::CheckReaction::kUpdating,
1284 metrics::DownloadErrorCode::kUnset,
1285 &response,
1286 nullptr));
1287
1288 string value;
1289 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1290 EXPECT_EQ(fake_update_response_.cohort, value);
1291
1292 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1293 EXPECT_EQ(fake_update_response_.cohorthint, value);
1294
1295 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1296 EXPECT_EQ(fake_update_response_.cohortname, value);
1297}
1298
1299TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1300 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001301 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1302 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1303 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1304 fake_update_response_.include_cohorts = true;
1305 fake_update_response_.cohort = "s/154454/8479665";
1306 fake_update_response_.cohorthint = "please-put-me-on-beta";
1307 fake_update_response_.cohortname = "";
1308
Marton Hunyady2abda312018-04-24 18:21:49 +02001309 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo8e18f932015-03-27 16:16:59 -07001310 -1,
1311 false, // ping_only
1312 ErrorCode::kSuccess,
1313 metrics::CheckResult::kUpdateAvailable,
1314 metrics::CheckReaction::kUpdating,
1315 metrics::DownloadErrorCode::kUnset,
1316 &response,
1317 nullptr));
1318
1319 string value;
1320 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1321 EXPECT_EQ(fake_update_response_.cohort, value);
1322
1323 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1324 EXPECT_EQ(fake_update_response_.cohorthint, value);
1325
1326 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1327}
1328
1329TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1330 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001331 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1332
Marton Hunyady2abda312018-04-24 18:21:49 +02001333 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo8e18f932015-03-27 16:16:59 -07001334 -1,
1335 false, // ping_only
1336 ErrorCode::kSuccess,
1337 metrics::CheckResult::kUpdateAvailable,
1338 metrics::CheckReaction::kUpdating,
1339 metrics::DownloadErrorCode::kUnset,
1340 &response,
1341 nullptr));
1342
1343 string value;
1344 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1345 EXPECT_EQ("old_value", value);
1346
1347 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1348 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1349}
1350
Alex Deymo00d79ac2015-06-29 15:41:49 -07001351TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1352 OmahaResponse response;
Alex Deymo00d79ac2015-06-29 15:41:49 -07001353 fake_update_response_.include_cohorts = true;
1354 fake_update_response_.cohort = "s/154454/8479665";
1355 fake_update_response_.cohorthint = "please-put-me-on-beta";
1356 fake_update_response_.cohortname = "stable";
1357
Marton Hunyady2abda312018-04-24 18:21:49 +02001358 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
Alex Deymo00d79ac2015-06-29 15:41:49 -07001359 -1,
1360 false, // ping_only
1361 ErrorCode::kSuccess,
1362 metrics::CheckResult::kNoUpdateAvailable,
1363 metrics::CheckReaction::kUnset,
1364 metrics::DownloadErrorCode::kUnset,
1365 &response,
1366 nullptr));
1367
1368 string value;
1369 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1370 EXPECT_EQ(fake_update_response_.cohort, value);
1371
1372 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1373 EXPECT_EQ(fake_update_response_.cohorthint, value);
1374
1375 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1376 EXPECT_EQ(fake_update_response_.cohortname, value);
1377}
1378
Sen Jiangb1e063a2017-09-15 17:44:31 -07001379TEST_F(OmahaRequestActionTest, MultiAppCohortTest) {
1380 OmahaResponse response;
Sen Jiangb1e063a2017-09-15 17:44:31 -07001381 fake_update_response_.multi_app = true;
1382 fake_update_response_.include_cohorts = true;
1383 fake_update_response_.cohort = "s/154454/8479665";
1384 fake_update_response_.cohorthint = "please-put-me-on-beta";
1385 fake_update_response_.cohortname = "stable";
1386
Marton Hunyady2abda312018-04-24 18:21:49 +02001387 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiangb1e063a2017-09-15 17:44:31 -07001388 -1,
1389 false, // ping_only
1390 ErrorCode::kSuccess,
1391 metrics::CheckResult::kUpdateAvailable,
1392 metrics::CheckReaction::kUpdating,
1393 metrics::DownloadErrorCode::kUnset,
1394 &response,
1395 nullptr));
1396
1397 string value;
1398 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1399 EXPECT_EQ(fake_update_response_.cohort, value);
1400
1401 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1402 EXPECT_EQ(fake_update_response_.cohorthint, value);
1403
1404 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1405 EXPECT_EQ(fake_update_response_.cohortname, value);
1406}
1407
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001408TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001409 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001410
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001411 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001412 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001413
Ben Chan5c02c132017-06-27 07:10:36 -07001414 OmahaRequestAction action(
1415 &fake_system_state_,
1416 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001417 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001418 http_response.size(),
1419 nullptr),
1420 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001421 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001422 ActionProcessor processor;
1423 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001424 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001425
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001426 loop.PostTask(base::Bind(
1427 [](ActionProcessor* processor) { processor->StartProcessing(); },
1428 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001429 loop.Run();
1430 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001431 EXPECT_FALSE(processor.IsRunning());
1432}
1433
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001434TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001435 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001436 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
1437 -1,
1438 false, // ping_only
1439 ErrorCode::kOmahaRequestXMLParseError,
1440 metrics::CheckResult::kParsingError,
1441 metrics::CheckReaction::kUnset,
1442 metrics::DownloadErrorCode::kUnset,
1443 &response,
1444 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001445 EXPECT_FALSE(response.update_exists);
1446}
1447
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001448TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001449 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001450 ASSERT_FALSE(TestUpdateCheck("",
1451 -1,
1452 false, // ping_only
1453 ErrorCode::kOmahaRequestEmptyResponseError,
1454 metrics::CheckResult::kParsingError,
1455 metrics::CheckReaction::kUnset,
1456 metrics::DownloadErrorCode::kUnset,
1457 &response,
1458 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001459 EXPECT_FALSE(response.update_exists);
1460}
1461
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001462TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001463 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001464 ASSERT_FALSE(TestUpdateCheck(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001465 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1466 "<daystart elapsed_seconds=\"100\"/>"
1467 "<app appid=\"foo\" status=\"ok\">"
1468 "<ping status=\"ok\"/>"
1469 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001470 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001471 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001472 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001473 metrics::CheckResult::kParsingError,
1474 metrics::CheckReaction::kUnset,
1475 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001476 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001477 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001478 EXPECT_FALSE(response.update_exists);
1479}
1480
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001481TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001482 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001483 ASSERT_FALSE(TestUpdateCheck(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001484 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1485 "<daystart elapsed_seconds=\"100\"/>"
1486 "<app appid=\"foo\" status=\"ok\">"
1487 "<ping status=\"ok\"/>"
1488 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001489 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001490 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001491 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001492 metrics::CheckResult::kParsingError,
1493 metrics::CheckReaction::kUnset,
1494 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001495 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001496 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001497 EXPECT_FALSE(response.update_exists);
1498}
1499
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001500TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001501 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001502 ASSERT_FALSE(TestUpdateCheck(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001503 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1504 "<daystart elapsed_seconds=\"100\"/>"
1505 "<app appid=\"foo\" status=\"ok\">"
1506 "<ping status=\"ok\"/>"
1507 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001508 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001509 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001510 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001511 metrics::CheckResult::kParsingError,
1512 metrics::CheckReaction::kUnset,
1513 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001514 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001515 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001516 EXPECT_FALSE(response.update_exists);
1517}
1518
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001519TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001520 string input_response =
1521 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1522 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001523 // the appid needs to match that in the request params
1524 "<app appid=\"" +
1525 fake_update_response_.app_id +
1526 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001527 "<updatecheck status=\"ok\">"
1528 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001529 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001530 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001531 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001532 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001533 "ChromeOSVersion=\"10.2.3.4\" "
1534 "Prompt=\"false\" "
1535 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001536 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001537 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001538 "needsadmin=\"true\" "
1539 "/></actions></manifest></updatecheck></app></response>";
1540 LOG(INFO) << "Input Response = " << input_response;
1541
Darin Petkov6a5b3222010-07-13 14:55:28 -07001542 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001543 ASSERT_TRUE(TestUpdateCheck(input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001544 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001545 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001546 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001547 metrics::CheckResult::kUpdateAvailable,
1548 metrics::CheckReaction::kUpdating,
1549 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001550 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001551 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001552 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001553 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001554 EXPECT_EQ("http://missing/field/test/f",
1555 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001556 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001557 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1558 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001559 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001560 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001561}
1562
1563namespace {
1564class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1565 public:
1566 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001567 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001568 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001569};
1570
Alex Deymo60ca1a72015-06-18 18:19:15 -07001571void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001572 processor->StartProcessing();
1573 CHECK(processor->IsRunning());
1574 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001575}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001576} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001577
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001578TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001579 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001580 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001581
Alex Deymo60ca1a72015-06-18 18:19:15 -07001582 string http_response("doesn't matter");
Ben Chan5c02c132017-06-27 07:10:36 -07001583 OmahaRequestAction action(
1584 &fake_system_state_,
1585 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001586 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001587 http_response.size(),
1588 nullptr),
1589 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001590 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001591 ActionProcessor processor;
1592 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001593 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001594
Alex Deymo60ca1a72015-06-18 18:19:15 -07001595 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1596 loop.Run();
1597 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001598}
1599
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001600TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001601 string output;
1602 EXPECT_TRUE(XmlEncode("ab", &output));
1603 EXPECT_EQ("ab", output);
1604 EXPECT_TRUE(XmlEncode("a<b", &output));
1605 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001606 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1607 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001608 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1609 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001610 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001611 EXPECT_FALSE(XmlEncode("\xc2", &output));
1612 // Fail with invalid ASCII-7 chars.
1613 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1614}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001615
Alex Deymob0d74eb2015-03-30 17:59:17 -07001616TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1617 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1618 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1619}
1620
1621TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001622 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001623
1624 // Make sure XML Encode is being called on the params
Marton Hunyady2abda312018-04-24 18:21:49 +02001625 request_params_.set_os_sp("testtheservice_pack>");
1626 request_params_.set_os_board("x86 generic<id");
1627 request_params_.set_current_channel("unittest_track&lt;");
1628 request_params_.set_target_channel("unittest_track&lt;");
1629 request_params_.set_hwid("<OEM MODEL>");
Alex Deymo8e18f932015-03-27 16:16:59 -07001630 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1631 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1632 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001633 base::JoinString(
1634 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001635 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001636 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
1637 -1,
1638 false, // ping_only
1639 ErrorCode::kOmahaRequestXMLParseError,
1640 metrics::CheckResult::kParsingError,
1641 metrics::CheckReaction::kUnset,
1642 metrics::DownloadErrorCode::kUnset,
1643 &response,
1644 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001645 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001646 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001647 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1648 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1649 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1650 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1651 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1652 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1653 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1654 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1655 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1656 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1657 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1658 // Values from Prefs that are too big are removed from the XML instead of
1659 // encoded.
1660 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001661}
1662
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001663TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001664 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001665 fake_update_response_.deadline = "&lt;20110101";
1666 fake_update_response_.more_info_url = "testthe&lt;url";
1667 fake_update_response_.codebase = "testthe&amp;codebase/";
Marton Hunyady2abda312018-04-24 18:21:49 +02001668 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1669 -1,
1670 false, // ping_only
1671 ErrorCode::kSuccess,
1672 metrics::CheckResult::kUpdateAvailable,
1673 metrics::CheckReaction::kUpdating,
1674 metrics::DownloadErrorCode::kUnset,
1675 &response,
1676 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001677
Sen Jiang0affc2c2017-02-10 15:55:05 -08001678 EXPECT_EQ("testthe<url", response.more_info_url);
1679 EXPECT_EQ("testthe&codebase/file.signed",
1680 response.packages[0].payload_urls[0]);
1681 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001682}
1683
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001684TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001685 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001686 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001687 fake_update_response_.size = 123123123123123ull;
Marton Hunyady2abda312018-04-24 18:21:49 +02001688 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1689 -1,
1690 false, // ping_only
1691 ErrorCode::kSuccess,
1692 metrics::CheckResult::kUpdateAvailable,
1693 metrics::CheckReaction::kUpdating,
1694 metrics::DownloadErrorCode::kUnset,
1695 &response,
1696 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001697
Sen Jiang0affc2c2017-02-10 15:55:05 -08001698 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001699}
1700
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001701TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001702 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001703 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001704 fake_system_state_.set_prefs(&prefs);
1705
Darin Petkov95508da2011-01-05 12:42:29 -08001706 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001707 .WillOnce(DoAll(SetArgPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001708 // An existing but empty previous version means that we didn't reboot to a new
1709 // update, therefore, no need to update the previous version.
1710 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Marton Hunyady2abda312018-04-24 18:21:49 +02001711 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001712 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001713 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001714 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001715 metrics::CheckResult::kParsingError,
1716 metrics::CheckReaction::kUnset,
1717 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001718 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001719 &post_data));
1720 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001721 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001722 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001723 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001724 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001725 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001726 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1727 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001728 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1729 string::npos);
1730 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1731 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001732 // No <event> tag should be sent if we didn't reboot to an update.
1733 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001734}
1735
Jay Srinivasan0a708742012-03-20 11:26:12 -07001736
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001737TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001738 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001739 TestEvent(new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
Darin Petkove17f86b2010-07-20 09:12:01 -07001740 "invalid xml>",
1741 &post_data);
1742 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001743 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001744 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001745 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001746 OmahaEvent::kTypeUpdateDownloadStarted,
1747 OmahaEvent::kResultSuccess);
1748 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001749 EXPECT_EQ(post_str.find("ping"), string::npos);
1750 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001751}
1752
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001753TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001754 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001755 TestEvent(new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
Darin Petkove17f86b2010-07-20 09:12:01 -07001756 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001757 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001758 "invalid xml>",
1759 &post_data);
1760 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001761 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001762 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001763 " <event eventtype=\"%d\" eventresult=\"%d\" "
1764 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001765 OmahaEvent::kTypeDownloadComplete,
1766 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001767 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001768 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001769 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001770}
1771
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001772TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001773 string http_response("doesn't matter");
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001774 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001775 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001776 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001777 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001778 http_response.size(),
1779 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001780 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001781 EXPECT_FALSE(update_check_action.IsEvent());
1782
1783 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001784 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001785 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Ben Chanab5a0af2017-10-12 14:57:50 -07001786 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001787 http_response.size(),
1788 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001789 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001790 EXPECT_TRUE(event_action.IsEvent());
1791}
1792
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001793TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001794 for (int i = 0; i < 2; i++) {
1795 bool delta_okay = i == 1;
1796 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001797 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001798
1799 request_params_.set_delta_okay(delta_okay);
1800
1801 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001802 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001803 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001804 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001805 metrics::CheckResult::kParsingError,
1806 metrics::CheckReaction::kUnset,
1807 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001808 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001809 &post_data));
1810 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001811 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001812 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1813 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001814 string::npos)
1815 << "i = " << i;
1816 }
1817}
1818
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001819TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001820 for (int i = 0; i < 2; i++) {
1821 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001822 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001823 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001824 FakeSystemState fake_system_state;
Marton Hunyady2abda312018-04-24 18:21:49 +02001825
1826 request_params_.set_interactive(interactive);
1827
1828 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001829 -1,
1830 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001831 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001832 metrics::CheckResult::kParsingError,
1833 metrics::CheckReaction::kUnset,
1834 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001835 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001836 &post_data));
1837 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001838 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001839 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1840 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001841 string::npos)
1842 << "i = " << i;
1843 }
1844}
1845
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001846TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001847 OmahaEvent default_event;
1848 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1849 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001850 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001851
1852 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1853 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1854 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001855 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001856
1857 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1858 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001859 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001860 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1861 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001862 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001863}
1864
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001865void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001866 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001867 fake_system_state_.set_prefs(&prefs);
1868 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1869 .Times(AnyNumber());
1870 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1871 // Add a few hours to the day difference to test no rounding, etc.
1872 int64_t five_days_ago =
1873 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1874 int64_t six_days_ago =
1875 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1876 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001877 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001878 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001879 .WillOnce(DoAll(SetArgPointee<1>(six_days_ago), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001880 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001881 .WillOnce(DoAll(SetArgPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001882 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001883 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001884 -1,
1885 ping_only,
1886 ErrorCode::kSuccess,
1887 metrics::CheckResult::kNoUpdateAvailable,
1888 metrics::CheckReaction::kUnset,
1889 metrics::DownloadErrorCode::kUnset,
1890 nullptr,
1891 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001892 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001893 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1894 string::npos);
1895 if (ping_only) {
1896 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1897 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1898 } else {
1899 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1900 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001901 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001902}
1903
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001904TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1905 PingTest(true /* ping_only */);
1906}
1907
1908TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1909 PingTest(false /* ping_only */);
1910}
1911
1912TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001913 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001914 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001915 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1916 .Times(AnyNumber());
1917 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001918 int64_t three_days_ago =
1919 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1920 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001921 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001922 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001923 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001924 .WillOnce(DoAll(SetArgPointee<1>(three_days_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001925 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001926 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001927 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001928 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
1929 -1,
1930 false, // ping_only
1931 ErrorCode::kSuccess,
1932 metrics::CheckResult::kNoUpdateAvailable,
1933 metrics::CheckReaction::kUnset,
1934 metrics::DownloadErrorCode::kUnset,
1935 nullptr,
1936 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001937 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001938 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001939 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001940}
1941
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001942TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001943 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001944 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001945 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1946 .Times(AnyNumber());
1947 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001948 int64_t four_days_ago =
1949 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1950 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001951 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001952 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001953 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001954 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001955 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001956 .WillOnce(DoAll(SetArgPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001957 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001958 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
1959 -1,
1960 false, // ping_only
1961 ErrorCode::kSuccess,
1962 metrics::CheckResult::kNoUpdateAvailable,
1963 metrics::CheckReaction::kUnset,
1964 metrics::DownloadErrorCode::kUnset,
1965 nullptr,
1966 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001967 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001968 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001969 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001970}
1971
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001972TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001973 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001974 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001975 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1976 .Times(AnyNumber());
1977 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001978 int64_t one_hour_ago =
1979 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001980 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001981 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001982 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001983 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001984 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001985 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001986 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1987 // send a ping.
1988 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1989 .WillOnce(Return(true));
1990 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1991 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001992 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001993 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
1994 -1,
1995 false, // ping_only
1996 ErrorCode::kSuccess,
1997 metrics::CheckResult::kNoUpdateAvailable,
1998 metrics::CheckReaction::kUnset,
1999 metrics::DownloadErrorCode::kUnset,
2000 nullptr,
2001 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002002 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002003 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002004}
2005
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002006TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07002007 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08002008 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002009 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07002010 int64_t now = Time::Now().ToInternalValue();
2011 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002012 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07002013 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002014 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07002015 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2016 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002017 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002018 EXPECT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2019 -1,
2020 true, // ping_only
2021 ErrorCode::kSuccess,
2022 metrics::CheckResult::kUnset,
2023 metrics::CheckReaction::kUnset,
2024 metrics::DownloadErrorCode::kUnset,
2025 nullptr,
2026 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08002027 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07002028}
2029
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002030TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002031 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002032 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002033 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
2034 .Times(AnyNumber());
2035 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002036 int64_t future =
2037 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08002038 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002039 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002040 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002041 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002042 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002043 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002044 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
2045 .WillOnce(Return(true));
2046 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
2047 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002048 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002049 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002050 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002051 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002052 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002053 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002054 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002055 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002056 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002057 metrics::CheckResult::kNoUpdateAvailable,
2058 metrics::CheckReaction::kUnset,
2059 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002060 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002061 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002062 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002063 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002064}
2065
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002066TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002067 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07002068 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002069 // may fail if it runs for longer than 5 seconds. It shouldn't run
2070 // that long though.
2071 int64_t midnight =
2072 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
2073 int64_t midnight_slack =
2074 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08002075 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002076 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002077 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2078 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002079 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
2080 AllOf(Ge(midnight), Le(midnight_slack))))
2081 .WillOnce(Return(true));
2082 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
2083 AllOf(Ge(midnight), Le(midnight_slack))))
2084 .WillOnce(Return(true));
2085 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002086 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002087 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002088 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002089 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002090 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002091 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002092 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002093 metrics::CheckResult::kNoUpdateAvailable,
2094 metrics::CheckReaction::kUnset,
2095 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002096 nullptr,
2097 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002098}
2099
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002100TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002101 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002102 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002103 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2104 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002105 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2106 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2107 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002108 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002109 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002110 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002111 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002112 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002113 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002114 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002115 metrics::CheckResult::kNoUpdateAvailable,
2116 metrics::CheckReaction::kUnset,
2117 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002118 nullptr,
2119 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002120}
2121
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002122TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002123 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002124 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002125 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2126 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002127 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2128 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2129 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002130 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002131 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002132 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002133 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002134 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002135 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002136 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002137 metrics::CheckResult::kNoUpdateAvailable,
2138 metrics::CheckReaction::kUnset,
2139 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002140 nullptr,
2141 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002142}
2143
Alex Deymob3fa53b2016-04-18 19:57:58 -07002144TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
2145 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2146 // the "eol" attribute.
2147 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002148 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Alex Deymob3fa53b2016-04-18 19:57:58 -07002149 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2150 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2151 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2152 "</app></response>",
2153 -1,
2154 false, // ping_only
2155 ErrorCode::kSuccess,
2156 metrics::CheckResult::kNoUpdateAvailable,
2157 metrics::CheckReaction::kUnset,
2158 metrics::DownloadErrorCode::kUnset,
2159 nullptr,
2160 nullptr));
2161 string eol_pref;
2162 EXPECT_TRUE(
2163 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2164 // Note that the eol="eol" attribute should be ignored and the _eol should be
2165 // used instead.
2166 EXPECT_EQ("security-only", eol_pref);
2167}
2168
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002169TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002170 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002171 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002172 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002173 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002174 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002175 metrics::CheckResult::kParsingError,
2176 metrics::CheckReaction::kUnset,
2177 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002178 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002179 &post_data));
2180 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002181 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002182 EXPECT_EQ(post_str.find("machineid="), string::npos);
2183 EXPECT_EQ(post_str.find("userid="), string::npos);
2184}
2185
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002186TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002187 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002188 const int http_error_code =
2189 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Marton Hunyady2abda312018-04-24 18:21:49 +02002190 ASSERT_FALSE(TestUpdateCheck("",
2191 501,
2192 false, // ping_only
2193 static_cast<ErrorCode>(http_error_code),
2194 metrics::CheckResult::kDownloadError,
2195 metrics::CheckReaction::kUnset,
2196 static_cast<metrics::DownloadErrorCode>(501),
2197 &response,
2198 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002199 EXPECT_FALSE(response.update_exists);
2200}
2201
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002202TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002203 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002204 const int http_error_code =
2205 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Marton Hunyady2abda312018-04-24 18:21:49 +02002206 ASSERT_FALSE(TestUpdateCheck("",
2207 1500,
2208 false, // ping_only
2209 static_cast<ErrorCode>(http_error_code),
2210 metrics::CheckResult::kDownloadError,
2211 metrics::CheckReaction::kUnset,
2212 metrics::DownloadErrorCode::kHttpStatusOther,
2213 &response,
2214 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002215 EXPECT_FALSE(response.update_exists);
2216}
2217
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002218TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002219 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02002220 request_params_.set_wall_clock_based_wait_enabled(true);
2221 request_params_.set_waiting_period(TimeDelta().FromDays(1));
2222 request_params_.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002223
Sen Jiang7c1171e2016-06-23 11:35:40 -07002224 Time arbitrary_date;
Amin Hassani71818c82018-03-06 13:25:40 -08002225 Time::FromString("6/4/1989", &arbitrary_date);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002226 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
Marton Hunyady2abda312018-04-24 18:21:49 +02002227 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang7c1171e2016-06-23 11:35:40 -07002228 -1,
2229 false, // ping_only
2230 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2231 metrics::CheckResult::kUpdateAvailable,
2232 metrics::CheckReaction::kDeferring,
2233 metrics::DownloadErrorCode::kUnset,
2234 &response,
2235 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002236
Ben Chan9abb7632014-08-07 00:10:53 -07002237 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002238 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002239 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002240 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002241
2242 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02002243 request_params_.set_interactive(true);
2244 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang7c1171e2016-06-23 11:35:40 -07002245 -1,
2246 false, // ping_only
2247 ErrorCode::kSuccess,
2248 metrics::CheckResult::kUpdateAvailable,
2249 metrics::CheckReaction::kUpdating,
2250 metrics::DownloadErrorCode::kUnset,
2251 &response,
2252 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002253 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002254}
2255
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002256TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002257 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02002258 request_params_.set_wall_clock_based_wait_enabled(true);
2259 request_params_.set_waiting_period(TimeDelta().FromDays(1));
2260 request_params_.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002261
Sen Jiang7c1171e2016-06-23 11:35:40 -07002262 Time t1, t2;
Amin Hassani71818c82018-03-06 13:25:40 -08002263 Time::FromString("1/1/2012", &t1);
2264 Time::FromString("1/3/2012", &t2);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002265 ASSERT_TRUE(
2266 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2267 fake_system_state_.fake_clock()->SetWallclockTime(t2);
Marton Hunyady2abda312018-04-24 18:21:49 +02002268 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang7c1171e2016-06-23 11:35:40 -07002269 -1,
2270 false, // ping_only
2271 ErrorCode::kSuccess,
2272 metrics::CheckResult::kUpdateAvailable,
2273 metrics::CheckReaction::kUpdating,
2274 metrics::DownloadErrorCode::kUnset,
2275 &response,
2276 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002277
2278 EXPECT_TRUE(response.update_exists);
2279
2280 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002281 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002282 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002283 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2284}
2285
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002286TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002287 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002288 base::ScopedTempDir tempdir;
2289 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002290
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002291 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002292 request_params_.set_root(tempdir.GetPath().value());
2293 request_params_.set_app_id("{22222222-2222-2222-2222-222222222222}");
2294 request_params_.set_app_version("1.2.3.4");
2295 request_params_.set_product_components("o.bundle=1");
2296 request_params_.set_current_channel("canary-channel");
2297 EXPECT_TRUE(
2298 request_params_.SetTargetChannel("stable-channel", true, nullptr));
2299 request_params_.UpdateDownloadChannel();
2300 EXPECT_TRUE(request_params_.ShouldPowerwash());
2301 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002302 -1,
2303 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002304 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002305 metrics::CheckResult::kParsingError,
2306 metrics::CheckReaction::kUnset,
2307 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002308 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002309 &post_data));
2310 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002311 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002312 EXPECT_NE(string::npos, post_str.find(
2313 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2314 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2315 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Sen Jiang8cd42342018-01-31 12:06:59 -08002316 EXPECT_EQ(string::npos, post_str.find("o.bundle"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002317}
2318
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002319TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002320 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002321 base::ScopedTempDir tempdir;
2322 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002323
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002324 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002325 request_params_.set_root(tempdir.GetPath().value());
2326 request_params_.set_app_id("{11111111-1111-1111-1111-111111111111}");
2327 request_params_.set_app_version("5.6.7.8");
2328 request_params_.set_product_components("o.bundle=1");
2329 request_params_.set_current_channel("stable-channel");
2330 EXPECT_TRUE(
2331 request_params_.SetTargetChannel("canary-channel", false, nullptr));
2332 request_params_.UpdateDownloadChannel();
2333 EXPECT_FALSE(request_params_.ShouldPowerwash());
2334 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002335 -1,
2336 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002337 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002338 metrics::CheckResult::kParsingError,
2339 metrics::CheckReaction::kUnset,
2340 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002341 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002342 &post_data));
2343 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002344 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002345 EXPECT_NE(string::npos, post_str.find(
2346 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2347 "version=\"5.6.7.8\" "
2348 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002349 EXPECT_EQ(string::npos, post_str.find("from_version"));
Sen Jiang8cd42342018-01-31 12:06:59 -08002350 EXPECT_NE(string::npos, post_str.find("o.bundle.version=\"1\""));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002351}
2352
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002353// Checks that the initial ping with a=-1 r=-1 is not send when the device
2354// was powerwashed.
2355TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2356 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2357
2358 // Flag that the device was powerwashed in the past.
2359 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2360
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002361 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002362 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2363 -1,
2364 false, // ping_only
2365 ErrorCode::kSuccess,
2366 metrics::CheckResult::kNoUpdateAvailable,
2367 metrics::CheckReaction::kUnset,
2368 metrics::DownloadErrorCode::kUnset,
2369 nullptr,
2370 &post_data));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002371 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002372 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002373 EXPECT_EQ(string::npos, post_str.find("<ping"));
2374}
2375
Amin Hassani1677e812017-06-21 13:36:36 -07002376// Checks that the initial ping with a=-1 r=-1 is not send when the device
2377// first_active_omaha_ping_sent is set.
2378TEST_F(OmahaRequestActionTest, PingWhenFirstActiveOmahaPingIsSent) {
2379 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2380
2381 // Flag that the device was not powerwashed in the past.
2382 fake_system_state_.fake_hardware()->SetPowerwashCount(0);
2383
2384 // Flag that the device has sent first active ping in the past.
2385 fake_system_state_.fake_hardware()->SetFirstActiveOmahaPingSent();
2386
2387 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002388 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2389 -1,
2390 false, // ping_only
2391 ErrorCode::kSuccess,
2392 metrics::CheckResult::kNoUpdateAvailable,
2393 metrics::CheckReaction::kUnset,
2394 metrics::DownloadErrorCode::kUnset,
2395 nullptr,
2396 &post_data));
Amin Hassani1677e812017-06-21 13:36:36 -07002397 // We shouldn't send a ping in this case since
2398 // first_active_omaha_ping_sent=true
2399 string post_str(post_data.begin(), post_data.end());
2400 EXPECT_EQ(string::npos, post_str.find("<ping"));
2401}
2402
Alex Deymo9fded1e2015-11-05 12:31:19 -08002403// Checks that the event 54 is sent on a reboot to a new update.
2404TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2405 // Flag that the device was updated in a previous boot.
2406 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2407
2408 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002409 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2410 -1,
2411 false, // ping_only
2412 ErrorCode::kSuccess,
2413 metrics::CheckResult::kNoUpdateAvailable,
2414 metrics::CheckReaction::kUnset,
2415 metrics::DownloadErrorCode::kUnset,
2416 nullptr,
2417 &post_data));
Alex Deymo9fded1e2015-11-05 12:31:19 -08002418 string post_str(post_data.begin(), post_data.end());
2419
2420 // An event 54 is included and has the right version.
2421 EXPECT_NE(string::npos,
2422 post_str.find(base::StringPrintf(
2423 "<event eventtype=\"%d\"",
2424 OmahaEvent::kTypeRebootedAfterUpdate)));
2425 EXPECT_NE(string::npos,
2426 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2427
2428 // The previous version flag should have been removed.
2429 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2430 string prev_version;
2431 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2432 EXPECT_TRUE(prev_version.empty());
2433}
2434
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002435void OmahaRequestActionTest::P2PTest(
2436 bool initial_allow_p2p_for_downloading,
2437 bool initial_allow_p2p_for_sharing,
2438 bool omaha_disable_p2p_for_downloading,
2439 bool omaha_disable_p2p_for_sharing,
2440 bool payload_state_allow_p2p_attempt,
2441 bool expect_p2p_client_lookup,
2442 const string& p2p_client_result_url,
2443 bool expected_allow_p2p_for_downloading,
2444 bool expected_allow_p2p_for_sharing,
2445 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002446 OmahaResponse response;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002447 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2448 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2449 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002450
2451 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002452 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002453 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2454 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002455 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2456 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2457 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2458 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2459 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2460 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2461 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2462 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2463 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2464 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2465
David Zeuthen8f191b22013-08-06 12:27:50 -07002466 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002467 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002468 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2469
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002470 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2471 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002472 .Times(expect_p2p_client_lookup ? 1 : 0);
2473
Alex Deymo8e18f932015-03-27 16:16:59 -07002474 fake_update_response_.disable_p2p_for_downloading =
2475 omaha_disable_p2p_for_downloading;
2476 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
Marton Hunyady2abda312018-04-24 18:21:49 +02002477 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
2478 -1,
2479 false, // ping_only
2480 ErrorCode::kSuccess,
2481 metrics::CheckResult::kUpdateAvailable,
2482 metrics::CheckReaction::kUpdating,
2483 metrics::DownloadErrorCode::kUnset,
2484 &response,
2485 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002486 EXPECT_TRUE(response.update_exists);
2487
Gilad Arnold74b5f552014-10-07 08:17:16 -07002488 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2489 response.disable_p2p_for_downloading);
2490 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2491 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002492
Gilad Arnold74b5f552014-10-07 08:17:16 -07002493 EXPECT_EQ(expected_allow_p2p_for_downloading,
2494 actual_allow_p2p_for_downloading);
2495 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2496 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002497}
2498
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002499TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002500 P2PTest(true, // initial_allow_p2p_for_downloading
2501 true, // initial_allow_p2p_for_sharing
2502 false, // omaha_disable_p2p_for_downloading
2503 false, // omaha_disable_p2p_for_sharing
2504 true, // payload_state_allow_p2p_attempt
2505 true, // expect_p2p_client_lookup
2506 "http://1.3.5.7/p2p", // p2p_client_result_url
2507 true, // expected_allow_p2p_for_downloading
2508 true, // expected_allow_p2p_for_sharing
2509 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002510}
2511
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002512TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002513 P2PTest(true, // initial_allow_p2p_for_downloading
2514 true, // initial_allow_p2p_for_sharing
2515 false, // omaha_disable_p2p_for_downloading
2516 false, // omaha_disable_p2p_for_sharing
2517 true, // payload_state_allow_p2p_attempt
2518 true, // expect_p2p_client_lookup
2519 "", // p2p_client_result_url
2520 false, // expected_allow_p2p_for_downloading
2521 true, // expected_allow_p2p_for_sharing
2522 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002523}
2524
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002525TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002526 P2PTest(false, // initial_allow_p2p_for_downloading
2527 true, // initial_allow_p2p_for_sharing
2528 false, // omaha_disable_p2p_for_downloading
2529 false, // omaha_disable_p2p_for_sharing
2530 true, // payload_state_allow_p2p_attempt
2531 false, // expect_p2p_client_lookup
2532 "unset", // p2p_client_result_url
2533 false, // expected_allow_p2p_for_downloading
2534 true, // expected_allow_p2p_for_sharing
2535 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002536}
2537
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002538TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002539 P2PTest(true, // initial_allow_p2p_for_downloading
2540 true, // initial_allow_p2p_for_sharing
2541 true, // omaha_disable_p2p_for_downloading
2542 false, // omaha_disable_p2p_for_sharing
2543 true, // payload_state_allow_p2p_attempt
2544 false, // expect_p2p_client_lookup
2545 "unset", // p2p_client_result_url
2546 false, // expected_allow_p2p_for_downloading
2547 true, // expected_allow_p2p_for_sharing
2548 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002549}
2550
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002551TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002552 P2PTest(true, // initial_allow_p2p_for_downloading
2553 true, // initial_allow_p2p_for_sharing
2554 false, // omaha_disable_p2p_for_downloading
2555 true, // omaha_disable_p2p_for_sharing
2556 true, // payload_state_allow_p2p_attempt
2557 true, // expect_p2p_client_lookup
2558 "http://1.3.5.7/p2p", // p2p_client_result_url
2559 true, // expected_allow_p2p_for_downloading
2560 false, // expected_allow_p2p_for_sharing
2561 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002562}
2563
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002564TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002565 P2PTest(true, // initial_allow_p2p_for_downloading
2566 true, // initial_allow_p2p_for_sharing
2567 true, // omaha_disable_p2p_for_downloading
2568 true, // omaha_disable_p2p_for_sharing
2569 true, // payload_state_allow_p2p_attempt
2570 false, // expect_p2p_client_lookup
2571 "unset", // p2p_client_result_url
2572 false, // expected_allow_p2p_for_downloading
2573 false, // expected_allow_p2p_for_sharing
2574 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002575}
2576
Alex Deymof329b932014-10-30 01:37:48 -07002577bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2578 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002579 fake_update_response_.elapsed_days = elapsed_days;
Marton Hunyady2abda312018-04-24 18:21:49 +02002580 return TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
2581 -1,
2582 false, // ping_only
2583 ErrorCode::kSuccess,
2584 metrics::CheckResult::kUpdateAvailable,
2585 metrics::CheckReaction::kUpdating,
2586 metrics::DownloadErrorCode::kUnset,
2587 response,
2588 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002589}
2590
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002591TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002592 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002593
Kevin Cernekee2494e282016-03-29 18:03:53 -07002594 // Simulate a successful update check that happens during OOBE. The
2595 // deadline in the response is needed to force the update attempt to
2596 // occur; responses without a deadline seen during OOBE will normally
2597 // return ErrorCode::kNonCriticalUpdateInOOBE.
2598 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2599 fake_update_response_.deadline = "20101020";
2600
David Zeuthen639aa362014-02-03 16:23:44 -08002601 // Check that we parse elapsed_days in the Omaha Response correctly.
2602 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002603 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2604 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002605 EXPECT_TRUE(response.update_exists);
2606 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002607 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002608 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002609 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002610 EXPECT_EQ(prefs_days, 42);
2611
2612 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002613 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002614 EXPECT_TRUE(response.update_exists);
2615 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002616 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002617 EXPECT_EQ(prefs_days, 42);
2618
2619 // Note that elapsed_days is not necessarily divisible by 7 so check
2620 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002621 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2622 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002623 EXPECT_TRUE(response.update_exists);
2624 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002625 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002626 EXPECT_EQ(prefs_days, 21);
2627
2628 // Check that we correctly handle elapsed_days not being included in
2629 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002630 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002631 EXPECT_TRUE(response.update_exists);
2632 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002633}
2634
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002635// If there is no prefs and OOBE is not complete, we should not
2636// report anything to Omaha.
2637TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002638 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002639 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2640 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2641}
David Zeuthen639aa362014-02-03 16:23:44 -08002642
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002643// If OOBE is complete and happened on a valid date (e.g. after Jan
2644// 1 2007 0:00 PST), that date should be used and written to
2645// prefs. However, first try with an invalid date and check we do
2646// nothing.
2647TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2648 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2649 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2650 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2651 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2652}
David Zeuthen639aa362014-02-03 16:23:44 -08002653
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002654// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2655// should yield an InstallDate of 14.
2656TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2657 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2658 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2659 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2660 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002661
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002662 int64_t prefs_days;
2663 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2664 EXPECT_EQ(prefs_days, 14);
2665}
David Zeuthen639aa362014-02-03 16:23:44 -08002666
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002667// Now that we have a valid date in prefs, check that we keep using
2668// that even if OOBE date reports something else. The date Jan 30,
2669// 2007 0:00 PST should yield an InstallDate of 28... but since
2670// there's a prefs file, we should still get 14.
2671TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2672 // Set a valid date in the prefs first.
2673 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002674
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002675 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2676 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2677 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002678
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002679 int64_t prefs_days;
2680 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2681 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002682
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002683 // If we delete the prefs file, we should get 28 days.
2684 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2685 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2686 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2687 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002688}
2689
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -08002690// Verifies that a device with no device policy, and is not a consumer
2691// device sets the max kernel key version to the current version.
2692// ie. the same behavior as if rollback is enabled.
2693TEST_F(OmahaRequestActionTest, NoPolicyEnterpriseDevicesSetMaxRollback) {
2694 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2695
2696 // Setup and verify some initial default values for the kernel TPM
2697 // values that control verified boot and rollback.
2698 const int min_kernel_version = 4;
2699 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2700 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2701 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2702 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2703
2704 TestRollbackCheck(false /* is_consumer_device */,
2705 3 /* rollback_allowed_milestones */,
2706 false /* is_policy_loaded */);
2707
2708 // Verify max_kernel_rollforward was set to the current minimum
2709 // kernel key version. This has the effect of freezing roll
2710 // forwards indefinitely. This will hold the rollback window
2711 // open until a future change will be able to move this forward
2712 // relative the configured window.
2713 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2714 EXPECT_EQ(min_kernel_version, fake_hw->GetMaxKernelKeyRollforward());
2715}
2716
2717// Verifies that a conmsumer device with no device policy sets the
2718// max kernel key version to the current version. ie. the same
2719// behavior as if rollback is enabled.
2720TEST_F(OmahaRequestActionTest, NoPolicyConsumerDevicesSetMaxRollback) {
2721 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2722
2723 // Setup and verify some initial default values for the kernel TPM
2724 // values that control verified boot and rollback.
2725 const int min_kernel_version = 3;
2726 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2727 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2728 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2729 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2730
2731 TestRollbackCheck(true /* is_consumer_device */,
2732 3 /* rollback_allowed_milestones */,
2733 false /* is_policy_loaded */);
2734
2735 // Verify that with rollback disabled that max_kernel_rollforward
2736 // was set to logical infinity. This is the expected behavior for
2737 // consumer devices and matches the existing behavior prior to the
2738 // rollback features.
2739 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2740 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2741}
2742
2743// Verifies that a device with rollback enabled sets max_kernel_rollforward
2744// in the TPM to prevent roll forward.
2745TEST_F(OmahaRequestActionTest, RollbackEnabledDevicesSetMaxRollback) {
2746 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2747
2748 // Setup and verify some initial default values for the kernel TPM
2749 // values that control verified boot and rollback.
2750 const int allowed_milestones = 4;
2751 const int min_kernel_version = 3;
2752 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2753 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2754 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2755 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2756
2757 TestRollbackCheck(false /* is_consumer_device */,
2758 allowed_milestones,
2759 true /* is_policy_loaded */);
2760
2761 // Verify that with rollback enabled that max_kernel_rollforward
2762 // was set to the current minimum kernel key version. This has
2763 // the effect of freezing roll forwards indefinitely. This will
2764 // hold the rollback window open until a future change will
2765 // be able to move this forward relative the configured window.
2766 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2767 EXPECT_EQ(min_kernel_version, fake_hw->GetMaxKernelKeyRollforward());
2768}
2769
2770// Verifies that a device with rollback disabled sets max_kernel_rollforward
2771// in the TPM to logical infinity, to allow roll forward.
2772TEST_F(OmahaRequestActionTest, RollbackDisabledDevicesSetMaxRollback) {
2773 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2774
2775 // Setup and verify some initial default values for the kernel TPM
2776 // values that control verified boot and rollback.
2777 const int allowed_milestones = 0;
2778 const int min_kernel_version = 3;
2779 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2780 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2781 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2782 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2783
2784 TestRollbackCheck(false /* is_consumer_device */,
2785 allowed_milestones,
2786 true /* is_policy_loaded */);
2787
2788 // Verify that with rollback disabled that max_kernel_rollforward
2789 // was set to logical infinity.
2790 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2791 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2792}
2793
Darin Petkov6a5b3222010-07-13 14:55:28 -07002794} // namespace chromeos_update_engine