blob: fa531d254ab1802eda69691cdd2dafbaba454a36 [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("");
Marton Hunyadyba51c3f2018-04-25 15:18:10 +0200243 request_params_.set_rollback_allowed(false);
Marton Hunyady2abda312018-04-24 18:21:49 +0200244 request_params_.set_is_powerwash_allowed(false);
245
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700246 fake_system_state_.set_request_params(&request_params_);
247 fake_system_state_.set_prefs(&fake_prefs_);
248 }
249
250 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800251 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700252 // is used. |payload_state| may be null, in which case a local mock is used.
253 // |p2p_manager| may be null, in which case a local mock is used.
254 // |connection_manager| may be null, in which case a local mock is used.
255 // out_response may be null. If |fail_http_response_code| is non-negative,
256 // the transfer will fail with that code. |ping_only| is passed through to the
257 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
258 // post-data received by the mock HttpFetcher is returned.
259 //
260 // The |expected_check_result|, |expected_check_reaction| and
261 // |expected_error_code| parameters are for checking expectations
262 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
263 // UMA statistics. Use the appropriate ::kUnset value to specify that
264 // the given metric should not be reported.
Marton Hunyady2abda312018-04-24 18:21:49 +0200265 bool TestUpdateCheck(const string& http_response,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700266 int fail_http_response_code,
267 bool ping_only,
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800268 bool is_consumer_device,
269 int rollback_allowed_milestones,
270 bool is_policy_loaded,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700271 ErrorCode expected_code,
272 metrics::CheckResult expected_check_result,
273 metrics::CheckReaction expected_check_reaction,
274 metrics::DownloadErrorCode expected_download_error_code,
275 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700276 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700277
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800278 // Overload of TestUpdateCheck that does not supply |is_consumer_device| or
279 // |rollback_allowed_milestones| which are only required for rollback tests.
280 bool TestUpdateCheck(const string& http_response,
281 int fail_http_response_code,
282 bool ping_only,
283 ErrorCode expected_code,
284 metrics::CheckResult expected_check_result,
285 metrics::CheckReaction expected_check_reaction,
286 metrics::DownloadErrorCode expected_download_error_code,
287 OmahaResponse* out_response,
288 brillo::Blob* out_post_data);
289
290 void TestRollbackCheck(bool is_consumer_device,
291 int rollback_allowed_milestones,
292 bool is_policy_loaded);
293
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800294 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700295 // only a ping or also an updatecheck.
296 void PingTest(bool ping_only);
297
298 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700299 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700300 OmahaResponse *response);
301
302 // P2P test helper function.
303 void P2PTest(
304 bool initial_allow_p2p_for_downloading,
305 bool initial_allow_p2p_for_sharing,
306 bool omaha_disable_p2p_for_downloading,
307 bool omaha_disable_p2p_for_sharing,
308 bool payload_state_allow_p2p_attempt,
309 bool expect_p2p_client_lookup,
310 const string& p2p_client_result_url,
311 bool expected_allow_p2p_for_downloading,
312 bool expected_allow_p2p_for_sharing,
313 const string& expected_p2p_url);
314
315 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700316 FakeUpdateResponse fake_update_response_;
Marton Hunyady2abda312018-04-24 18:21:49 +0200317 // Used by all tests.
318 OmahaRequestParams request_params_{&fake_system_state_};
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700319
320 FakePrefs fake_prefs_;
321};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700322
323namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700324class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
325 public:
326 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700327 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800328 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700329 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800330 void ProcessingDone(const ActionProcessor* processor,
331 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700332 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700333 }
334
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800335 void ActionCompleted(ActionProcessor* processor,
336 AbstractAction* action,
337 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700338 // make sure actions always succeed
339 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700340 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700341 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700342 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700343 }
David Zeuthena99981f2013-04-29 13:42:47 -0700344 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700345};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700346} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700347
348class OutputObjectCollectorAction;
349
350template<>
351class ActionTraits<OutputObjectCollectorAction> {
352 public:
353 // Does not take an object for input
354 typedef OmahaResponse InputObjectType;
355 // On success, puts the output path on output
356 typedef NoneType OutputObjectType;
357};
358
359class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
360 public:
361 OutputObjectCollectorAction() : has_input_object_(false) {}
362 void PerformAction() {
363 // copy input object
364 has_input_object_ = HasInputObject();
365 if (has_input_object_)
366 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700367 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700368 }
369 // Should never be called
370 void TerminateProcessing() {
371 CHECK(false);
372 }
373 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700374 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700375 return "OutputObjectCollectorAction";
376 }
Alex Deymof329b932014-10-30 01:37:48 -0700377 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800378 using InputObjectType =
379 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
380 using OutputObjectType =
381 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700382 bool has_input_object_;
383 OmahaResponse omaha_response_;
384};
385
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700386bool OmahaRequestActionTest::TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700387 const string& http_response,
388 int fail_http_response_code,
389 bool ping_only,
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800390 bool is_consumer_device,
391 int rollback_allowed_milestones,
392 bool is_policy_loaded,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700393 ErrorCode expected_code,
394 metrics::CheckResult expected_check_result,
395 metrics::CheckReaction expected_check_reaction,
396 metrics::DownloadErrorCode expected_download_error_code,
397 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700398 brillo::Blob* out_post_data) {
399 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700400 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700401 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800402 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700403 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700404 if (fail_http_response_code >= 0) {
405 fetcher->FailTransfer(fail_http_response_code);
406 }
Marton Hunyady2abda312018-04-24 18:21:49 +0200407 // This ensures the tests didn't forget to update fake_system_state_ if they
408 // are not using the default request_params_.
409 EXPECT_EQ(&request_params_, fake_system_state_.request_params());
410
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700411 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700412 nullptr,
Ben Chan5c02c132017-06-27 07:10:36 -0700413 base::WrapUnique(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700414 ping_only);
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800415 auto mock_policy_provider =
416 std::make_unique<NiceMock<policy::MockPolicyProvider>>();
417 EXPECT_CALL(*mock_policy_provider, IsConsumerDevice())
418 .WillRepeatedly(Return(is_consumer_device));
419
420 EXPECT_CALL(*mock_policy_provider, device_policy_is_loaded())
421 .WillRepeatedly(Return(is_policy_loaded));
422
423 const policy::MockDevicePolicy device_policy;
424 const bool get_allowed_milestone_succeeds = rollback_allowed_milestones >= 0;
425 EXPECT_CALL(device_policy, GetRollbackAllowedMilestones(_))
426 .WillRepeatedly(DoAll(SetArgPointee<0>(rollback_allowed_milestones),
427 Return(get_allowed_milestone_succeeds)));
428
429 EXPECT_CALL(*mock_policy_provider, GetDevicePolicy())
430 .WillRepeatedly(ReturnRef(device_policy));
431
432 action.policy_provider_ = std::move(mock_policy_provider);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700433 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700434 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700435
Darin Petkov6a5b3222010-07-13 14:55:28 -0700436 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700437 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700438 processor.EnqueueAction(&action);
439
440 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700441 BondActions(&action, &collector_action);
442 processor.EnqueueAction(&collector_action);
443
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700444 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
445 ReportUpdateCheckMetrics(_, _, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800446 .Times(AnyNumber());
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700447
448 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
449 ReportUpdateCheckMetrics(_,
450 expected_check_result,
451 expected_check_reaction,
452 expected_download_error_code))
453 .Times(ping_only ? 0 : 1);
David Zeuthen33bae492014-02-25 16:16:18 -0800454
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700455 loop.PostTask(base::Bind(
456 [](ActionProcessor* processor) { processor->StartProcessing(); },
457 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700458 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700459 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700460 if (collector_action.has_input_object_ && out_response)
461 *out_response = collector_action.omaha_response_;
462 if (out_post_data)
463 *out_post_data = fetcher->post_data();
464 return collector_action.has_input_object_;
465}
466
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -0800467bool OmahaRequestActionTest::TestUpdateCheck(
468 const string& http_response,
469 int fail_http_response_code,
470 bool ping_only,
471 ErrorCode expected_code,
472 metrics::CheckResult expected_check_result,
473 metrics::CheckReaction expected_check_reaction,
474 metrics::DownloadErrorCode expected_download_error_code,
475 OmahaResponse* out_response,
476 brillo::Blob* out_post_data) {
477 return TestUpdateCheck(http_response,
478 fail_http_response_code,
479 ping_only,
480 true, // is_consumer_device
481 0, // rollback_allowed_milestones
482 false, // is_policy_loaded
483 expected_code,
484 expected_check_result,
485 expected_check_reaction,
486 expected_download_error_code,
487 out_response,
488 out_post_data);
489}
490
491void OmahaRequestActionTest::TestRollbackCheck(bool is_consumer_device,
492 int rollback_allowed_milestones,
493 bool is_policy_loaded) {
494 OmahaResponse response;
495 fake_update_response_.deadline = "20101020";
496 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
497 -1,
498 false, // ping_only
499 is_consumer_device,
500 rollback_allowed_milestones,
501 is_policy_loaded,
502 ErrorCode::kSuccess,
503 metrics::CheckResult::kUpdateAvailable,
504 metrics::CheckReaction::kUpdating,
505 metrics::DownloadErrorCode::kUnset,
506 &response,
507 nullptr));
508 ASSERT_TRUE(response.update_exists);
509}
510
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700511// Tests Event requests -- they should always succeed. |out_post_data|
512// may be null; if non-null, the post-data received by the mock
513// HttpFetcher is returned.
Marton Hunyady2abda312018-04-24 18:21:49 +0200514void TestEvent(OmahaEvent* event,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700515 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700516 brillo::Blob* out_post_data) {
517 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700518 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700519 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800520 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700521 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700522 FakeSystemState fake_system_state;
Alex Deymoc1c17b42015-11-23 03:53:15 -0300523 OmahaRequestAction action(&fake_system_state,
524 event,
Ben Chan5c02c132017-06-27 07:10:36 -0700525 base::WrapUnique(fetcher),
Alex Deymoc1c17b42015-11-23 03:53:15 -0300526 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700527 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700528 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700529 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700530 processor.EnqueueAction(&action);
531
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700532 loop.PostTask(base::Bind(
533 [](ActionProcessor* processor) { processor->StartProcessing(); },
534 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700535 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700536 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700537
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700538 if (out_post_data)
539 *out_post_data = fetcher->post_data();
540}
541
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700542TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400543 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700544 fake_update_response_.include_entity = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200545 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
546 -1,
547 false, // ping_only
548 ErrorCode::kOmahaRequestXMLHasEntityDecl,
549 metrics::CheckResult::kParsingError,
550 metrics::CheckReaction::kUnset,
551 metrics::DownloadErrorCode::kUnset,
552 &response,
553 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400554 EXPECT_FALSE(response.update_exists);
555}
556
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700557TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700558 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +0200559 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
560 -1,
561 false, // ping_only
562 ErrorCode::kSuccess,
563 metrics::CheckResult::kNoUpdateAvailable,
564 metrics::CheckReaction::kUnset,
565 metrics::DownloadErrorCode::kUnset,
566 &response,
567 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700568 EXPECT_FALSE(response.update_exists);
569}
570
Sen Jiang81259682017-03-30 15:11:30 -0700571TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
572 OmahaResponse response;
573 fake_update_response_.multi_app_no_update = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200574 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700575 -1,
576 false, // ping_only
577 ErrorCode::kSuccess,
578 metrics::CheckResult::kNoUpdateAvailable,
579 metrics::CheckReaction::kUnset,
580 metrics::DownloadErrorCode::kUnset,
581 &response,
582 nullptr));
583 EXPECT_FALSE(response.update_exists);
584}
585
Sen Jiang00adf7b2017-06-26 15:57:29 -0700586TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
587 OmahaResponse response;
588 fake_update_response_.multi_app_no_update = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200589 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang00adf7b2017-06-26 15:57:29 -0700590 -1,
591 false, // ping_only
592 ErrorCode::kSuccess,
593 metrics::CheckResult::kNoUpdateAvailable,
594 metrics::CheckReaction::kUnset,
595 metrics::DownloadErrorCode::kUnset,
596 &response,
597 nullptr));
598 EXPECT_FALSE(response.update_exists);
599}
600
601TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
602 OmahaResponse response;
603 ASSERT_TRUE(TestUpdateCheck(
Sen Jiang00adf7b2017-06-26 15:57:29 -0700604 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
605 "event=\"postinstall\" noupdate=\"true\"/></actions>"
606 "</manifest></updatecheck></app></response>",
607 -1,
608 false, // ping_only
609 ErrorCode::kSuccess,
610 metrics::CheckResult::kNoUpdateAvailable,
611 metrics::CheckReaction::kUnset,
612 metrics::DownloadErrorCode::kUnset,
613 &response,
614 nullptr));
615 EXPECT_FALSE(response.update_exists);
616}
617
Alex Deymo8e18f932015-03-27 16:16:59 -0700618// Test that all the values in the response are parsed in a normal update
619// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700620TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700621 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700622 fake_update_response_.deadline = "20101020";
Marton Hunyady2abda312018-04-24 18:21:49 +0200623 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
624 -1,
625 false, // ping_only
626 ErrorCode::kSuccess,
627 metrics::CheckResult::kUpdateAvailable,
628 metrics::CheckReaction::kUpdating,
629 metrics::DownloadErrorCode::kUnset,
630 &response,
631 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700632 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700633 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700634 EXPECT_EQ("", response.system_version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800635 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
636 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700637 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800638 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
639 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700640 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700641 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
642 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700643 // Omaha cohort attributes are not set in the response, so they should not be
Alex Deymo8e18f932015-03-27 16:16:59 -0700644 // persisted.
645 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
646 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
647 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700648}
649
Sen Jiang0affc2c2017-02-10 15:55:05 -0800650TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
651 OmahaResponse response;
652 fake_update_response_.multi_package = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200653 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang0affc2c2017-02-10 15:55:05 -0800654 -1,
655 false, // ping_only
656 ErrorCode::kSuccess,
657 metrics::CheckResult::kUpdateAvailable,
658 metrics::CheckReaction::kUpdating,
659 metrics::DownloadErrorCode::kUnset,
660 &response,
661 nullptr));
662 EXPECT_TRUE(response.update_exists);
663 EXPECT_EQ(fake_update_response_.version, response.version);
664 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
665 response.packages[0].payload_urls[0]);
666 EXPECT_EQ(fake_update_response_.codebase + "package2",
667 response.packages[1].payload_urls[0]);
668 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
669 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700670 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700671 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800672 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700673 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800674 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700675 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700676 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700677}
678
679TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
680 OmahaResponse response;
681 fake_update_response_.multi_app = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200682 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700683 -1,
684 false, // ping_only
685 ErrorCode::kSuccess,
686 metrics::CheckResult::kUpdateAvailable,
687 metrics::CheckReaction::kUpdating,
688 metrics::DownloadErrorCode::kUnset,
689 &response,
690 nullptr));
691 EXPECT_TRUE(response.update_exists);
692 EXPECT_EQ(fake_update_response_.version, response.version);
693 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
694 response.packages[0].payload_urls[0]);
695 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
696 response.packages[1].payload_urls[0]);
697 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
698 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
699 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700700 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700701 ASSERT_EQ(2u, response.packages.size());
702 EXPECT_EQ(string("hash3"), response.packages[1].hash);
703 EXPECT_EQ(333u, response.packages[1].size);
704 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700705 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700706}
707
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700708TEST_F(OmahaRequestActionTest, MultiAppAndSystemUpdateTest) {
709 OmahaResponse response;
710 fake_update_response_.multi_app = true;
711 // trigger the lining up of the app and system versions
712 request_params_.set_system_app_id(fake_update_response_.app_id2);
713
Marton Hunyady2abda312018-04-24 18:21:49 +0200714 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700715 -1,
716 false, // ping_only
717 ErrorCode::kSuccess,
718 metrics::CheckResult::kUpdateAvailable,
719 metrics::CheckReaction::kUpdating,
720 metrics::DownloadErrorCode::kUnset,
721 &response,
722 nullptr));
723 EXPECT_TRUE(response.update_exists);
724 EXPECT_EQ(fake_update_response_.version, response.version);
725 EXPECT_EQ(fake_update_response_.version2, response.system_version);
726 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
727 response.packages[0].payload_urls[0]);
728 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
729 response.packages[1].payload_urls[0]);
730 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
731 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
732 EXPECT_EQ(11u, response.packages[0].metadata_size);
733 EXPECT_EQ(true, response.packages[0].is_delta);
734 ASSERT_EQ(2u, response.packages.size());
735 EXPECT_EQ(string("hash3"), response.packages[1].hash);
736 EXPECT_EQ(333u, response.packages[1].size);
737 EXPECT_EQ(33u, response.packages[1].metadata_size);
738 EXPECT_EQ(false, response.packages[1].is_delta);
739}
740
Sen Jiang81259682017-03-30 15:11:30 -0700741TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
742 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700743 fake_update_response_.multi_app = true;
744 fake_update_response_.multi_app_self_update = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200745 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700746 -1,
747 false, // ping_only
748 ErrorCode::kSuccess,
749 metrics::CheckResult::kUpdateAvailable,
750 metrics::CheckReaction::kUpdating,
751 metrics::DownloadErrorCode::kUnset,
752 &response,
753 nullptr));
754 EXPECT_TRUE(response.update_exists);
755 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700756 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700757 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
758 response.packages[0].payload_urls[0]);
759 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
760 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
761 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700762 ASSERT_EQ(2u, response.packages.size());
763 EXPECT_EQ(string("hash3"), response.packages[1].hash);
764 EXPECT_EQ(333u, response.packages[1].size);
765 EXPECT_EQ(33u, response.packages[1].metadata_size);
766 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700767}
768
769TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
770 OmahaResponse response;
771 fake_update_response_.multi_app = true;
772 fake_update_response_.multi_package = true;
Marton Hunyady2abda312018-04-24 18:21:49 +0200773 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang81259682017-03-30 15:11:30 -0700774 -1,
775 false, // ping_only
776 ErrorCode::kSuccess,
777 metrics::CheckResult::kUpdateAvailable,
778 metrics::CheckReaction::kUpdating,
779 metrics::DownloadErrorCode::kUnset,
780 &response,
781 nullptr));
782 EXPECT_TRUE(response.update_exists);
783 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700784 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700785 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
786 response.packages[0].payload_urls[0]);
787 EXPECT_EQ(fake_update_response_.codebase + "package2",
788 response.packages[1].payload_urls[0]);
789 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
790 response.packages[2].payload_urls[0]);
791 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
792 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
793 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700794 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700795 ASSERT_EQ(3u, response.packages.size());
796 EXPECT_EQ(string("hash2"), response.packages[1].hash);
797 EXPECT_EQ(222u, response.packages[1].size);
798 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700799 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700800 EXPECT_EQ(string("hash3"), response.packages[2].hash);
801 EXPECT_EQ(333u, response.packages[2].size);
802 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700803 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800804}
805
Alex Deymo14ad88e2016-06-29 12:30:14 -0700806TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
807 const string http_response = "<?xml invalid response";
808 request_params_.set_interactive(true);
809
810 brillo::FakeMessageLoop loop(nullptr);
811 loop.SetAsCurrent();
812
813 MockHttpFetcher* fetcher =
814 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
Ben Chan5c02c132017-06-27 07:10:36 -0700815 OmahaRequestAction action(&fake_system_state_, nullptr,
816 base::WrapUnique(fetcher), false);
Alex Deymo14ad88e2016-06-29 12:30:14 -0700817 ActionProcessor processor;
818 processor.EnqueueAction(&action);
819
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700820 loop.PostTask(base::Bind(
821 [](ActionProcessor* processor) { processor->StartProcessing(); },
822 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700823 loop.Run();
824 EXPECT_FALSE(loop.PendingTasks());
825
826 // Check that the headers were set in the fetcher during the action. Note that
827 // we set this request as "interactive".
Sen Jiang42fa45e2018-03-12 11:02:14 -0700828 EXPECT_EQ("fg", fetcher->GetHeader("X-Goog-Update-Interactivity"));
829 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-Goog-Update-AppId"));
830 EXPECT_NE("", fetcher->GetHeader("X-Goog-Update-Updater"));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700831}
832
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700833TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700834 OmahaResponse response;
835 // Set up a connection manager that doesn't allow a valid update over
836 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700837 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700838 fake_system_state_.set_connection_manager(&mock_cm);
839
Alex Deymo30534502015-07-20 15:06:33 -0700840 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
841 .WillRepeatedly(
Ben Chan672c1f52017-10-23 15:41:39 -0700842 DoAll(SetArgPointee<0>(ConnectionType::kEthernet),
843 SetArgPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700844 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700845 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
846 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700847
Marton Hunyady2abda312018-04-24 18:21:49 +0200848 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
849 -1,
850 false, // ping_only
851 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
852 metrics::CheckResult::kUpdateAvailable,
853 metrics::CheckReaction::kIgnored,
854 metrics::DownloadErrorCode::kUnset,
855 &response,
856 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700857 EXPECT_FALSE(response.update_exists);
858}
859
Weidong Guo421ff332017-04-17 10:08:38 -0700860TEST_F(OmahaRequestActionTest, ValidUpdateOverCellularAllowedByDevicePolicy) {
861 // This test tests that update over cellular is allowed as device policy
862 // says yes.
863 OmahaResponse response;
864 MockConnectionManager mock_cm;
865
866 fake_system_state_.set_connection_manager(&mock_cm);
867
868 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
869 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
870 SetArgPointee<1>(ConnectionTethering::kUnknown),
871 Return(true)));
872 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
873 .WillRepeatedly(Return(true));
874 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
875 .WillRepeatedly(Return(true));
876
Marton Hunyady2abda312018-04-24 18:21:49 +0200877 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700878 -1,
879 false, // ping_only
880 ErrorCode::kSuccess,
881 metrics::CheckResult::kUpdateAvailable,
882 metrics::CheckReaction::kUpdating,
883 metrics::DownloadErrorCode::kUnset,
884 &response,
885 nullptr));
886 EXPECT_TRUE(response.update_exists);
887}
888
889TEST_F(OmahaRequestActionTest, ValidUpdateOverCellularBlockedByDevicePolicy) {
890 // This test tests that update over cellular is blocked as device policy
891 // says no.
892 OmahaResponse response;
893 MockConnectionManager mock_cm;
894
895 fake_system_state_.set_connection_manager(&mock_cm);
896
897 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
898 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
899 SetArgPointee<1>(ConnectionTethering::kUnknown),
900 Return(true)));
901 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
902 .WillRepeatedly(Return(true));
903 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
904 .WillRepeatedly(Return(false));
905
Marton Hunyady2abda312018-04-24 18:21:49 +0200906 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700907 -1,
908 false, // ping_only
909 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
910 metrics::CheckResult::kUpdateAvailable,
911 metrics::CheckReaction::kIgnored,
912 metrics::DownloadErrorCode::kUnset,
913 &response,
914 nullptr));
915 EXPECT_FALSE(response.update_exists);
916}
917
918TEST_F(OmahaRequestActionTest,
919 ValidUpdateOverCellularAllowedByUserPermissionTrue) {
920 // This test tests that, when device policy is not set, update over cellular
921 // is allowed as permission for update over cellular is set to true.
922 OmahaResponse response;
923 MockConnectionManager mock_cm;
924
925 fake_prefs_.SetBoolean(kPrefsUpdateOverCellularPermission, true);
926 fake_system_state_.set_connection_manager(&mock_cm);
927
928 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
929 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
930 SetArgPointee<1>(ConnectionTethering::kUnknown),
931 Return(true)));
932 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
933 .WillRepeatedly(Return(false));
934 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
935 .WillRepeatedly(Return(true));
936
Marton Hunyady2abda312018-04-24 18:21:49 +0200937 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700938 -1,
939 false, // ping_only
940 ErrorCode::kSuccess,
941 metrics::CheckResult::kUpdateAvailable,
942 metrics::CheckReaction::kUpdating,
943 metrics::DownloadErrorCode::kUnset,
944 &response,
945 nullptr));
946 EXPECT_TRUE(response.update_exists);
947}
948
949TEST_F(OmahaRequestActionTest,
950 ValidUpdateOverCellularBlockedByUpdateTargetNotMatch) {
951 // This test tests that, when device policy is not set and permission for
952 // update over cellular is set to false or does not exist, update over
953 // cellular is blocked as update target does not match the omaha response.
954 OmahaResponse response;
955 MockConnectionManager mock_cm;
956 // A version different from the version in omaha response.
957 string diff_version = "99.99.99";
958 // A size different from the size in omaha response.
959 int64_t diff_size = 999;
960
961 fake_prefs_.SetString(kPrefsUpdateOverCellularTargetVersion, diff_version);
962 fake_prefs_.SetInt64(kPrefsUpdateOverCellularTargetSize, diff_size);
963 // This test tests cellular (3G) being the only connection type being allowed.
964 fake_system_state_.set_connection_manager(&mock_cm);
965
966 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
967 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
968 SetArgPointee<1>(ConnectionTethering::kUnknown),
969 Return(true)));
970 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
971 .WillRepeatedly(Return(false));
972 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
973 .WillRepeatedly(Return(true));
974
Marton Hunyady2abda312018-04-24 18:21:49 +0200975 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -0700976 -1,
977 false, // ping_only
978 ErrorCode::kOmahaUpdateIgnoredOverCellular,
979 metrics::CheckResult::kUpdateAvailable,
980 metrics::CheckReaction::kIgnored,
981 metrics::DownloadErrorCode::kUnset,
982 &response,
983 nullptr));
984 EXPECT_FALSE(response.update_exists);
985}
986
987TEST_F(OmahaRequestActionTest,
988 ValidUpdateOverCellularAllowedByUpdateTargetMatch) {
989 // This test tests that, when device policy is not set and permission for
990 // update over cellular is set to false or does not exist, update over
991 // cellular is allowed as update target matches the omaha response.
992 OmahaResponse response;
993 MockConnectionManager mock_cm;
994 // A version same as the version in omaha response.
995 string new_version = fake_update_response_.version;
996 // A size same as the size in omaha response.
997 int64_t new_size = fake_update_response_.size;
998
999 fake_prefs_.SetString(kPrefsUpdateOverCellularTargetVersion, new_version);
1000 fake_prefs_.SetInt64(kPrefsUpdateOverCellularTargetSize, new_size);
1001 fake_system_state_.set_connection_manager(&mock_cm);
1002
1003 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
1004 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
1005 SetArgPointee<1>(ConnectionTethering::kUnknown),
1006 Return(true)));
1007 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
1008 .WillRepeatedly(Return(false));
1009 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
1010 .WillRepeatedly(Return(true));
1011
Marton Hunyady2abda312018-04-24 18:21:49 +02001012 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Weidong Guo421ff332017-04-17 10:08:38 -07001013 -1,
1014 false, // ping_only
1015 ErrorCode::kSuccess,
1016 metrics::CheckResult::kUpdateAvailable,
1017 metrics::CheckReaction::kUpdating,
1018 metrics::DownloadErrorCode::kUnset,
1019 &response,
1020 nullptr));
1021 EXPECT_TRUE(response.update_exists);
1022}
1023
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001024TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -07001025 string rollback_version = "1234.0.0";
1026 OmahaResponse response;
1027
1028 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001029 fake_system_state_.set_payload_state(&mock_payload_state);
1030
Chris Sosa77f79e82014-06-02 18:16:24 -07001031 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
1032 .WillRepeatedly(Return(rollback_version));
1033
Alex Deymo8e18f932015-03-27 16:16:59 -07001034 fake_update_response_.version = rollback_version;
Marton Hunyady2abda312018-04-24 18:21:49 +02001035 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1036 -1,
1037 false, // ping_only
1038 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
1039 metrics::CheckResult::kUpdateAvailable,
1040 metrics::CheckReaction::kIgnored,
1041 metrics::DownloadErrorCode::kUnset,
1042 &response,
1043 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -07001044 EXPECT_FALSE(response.update_exists);
1045}
1046
Kevin Cernekee2494e282016-03-29 18:03:53 -07001047// Verify that update checks called during OOBE will only try to download
1048// an update if the response includes a non-empty deadline field.
1049TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
1050 OmahaResponse response;
1051
Sen Jiang8cd42342018-01-31 12:06:59 -08001052 // TODO(senj): set better default value for metrics::checkresult in
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001053 // OmahaRequestAction::ActionCompleted.
Kevin Cernekee2494e282016-03-29 18:03:53 -07001054 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Marton Hunyady2abda312018-04-24 18:21:49 +02001055 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001056 -1,
1057 false, // ping_only
1058 ErrorCode::kNonCriticalUpdateInOOBE,
1059 metrics::CheckResult::kParsingError,
1060 metrics::CheckReaction::kUnset,
1061 metrics::DownloadErrorCode::kUnset,
1062 &response,
1063 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -07001064 EXPECT_FALSE(response.update_exists);
1065
Alex Deymo46a9aae2016-05-04 20:20:11 -07001066 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
1067 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
Marton Hunyady2abda312018-04-24 18:21:49 +02001068 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo46a9aae2016-05-04 20:20:11 -07001069 -1,
1070 false, // ping_only
1071 ErrorCode::kSuccess,
1072 metrics::CheckResult::kUpdateAvailable,
1073 metrics::CheckReaction::kUpdating,
1074 metrics::DownloadErrorCode::kUnset,
1075 &response,
1076 nullptr));
1077 EXPECT_TRUE(response.update_exists);
1078 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
1079
1080 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -07001081 fake_update_response_.deadline = "20101020";
Marton Hunyady2abda312018-04-24 18:21:49 +02001082 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1083 -1,
1084 false, // ping_only
1085 ErrorCode::kSuccess,
1086 metrics::CheckResult::kUpdateAvailable,
1087 metrics::CheckReaction::kUpdating,
1088 metrics::DownloadErrorCode::kUnset,
1089 &response,
1090 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -07001091 EXPECT_TRUE(response.update_exists);
1092}
1093
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001094TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001095 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001096 request_params_.set_wall_clock_based_wait_enabled(true);
1097 request_params_.set_update_check_count_wait_enabled(false);
1098 request_params_.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001099
Marton Hunyady2abda312018-04-24 18:21:49 +02001100 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1101 -1,
1102 false, // ping_only
1103 ErrorCode::kOmahaUpdateDeferredPerPolicy,
1104 metrics::CheckResult::kUpdateAvailable,
1105 metrics::CheckReaction::kDeferring,
1106 metrics::DownloadErrorCode::kUnset,
1107 &response,
1108 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001109 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001110
1111 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02001112 request_params_.set_interactive(true);
1113 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1114 -1,
1115 false, // ping_only
1116 ErrorCode::kSuccess,
1117 metrics::CheckResult::kUpdateAvailable,
1118 metrics::CheckReaction::kUpdating,
1119 metrics::DownloadErrorCode::kUnset,
1120 &response,
1121 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001122 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001123}
1124
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001125TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001126 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001127 request_params_.set_wall_clock_based_wait_enabled(false);
1128 request_params_.set_waiting_period(TimeDelta::FromDays(2));
1129 request_params_.set_update_check_count_wait_enabled(true);
1130 request_params_.set_min_update_checks_needed(1);
1131 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001132
Marton Hunyady2abda312018-04-24 18:21:49 +02001133 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1134 -1,
1135 false, // ping_only
1136 ErrorCode::kSuccess,
1137 metrics::CheckResult::kUpdateAvailable,
1138 metrics::CheckReaction::kUpdating,
1139 metrics::DownloadErrorCode::kUnset,
1140 &response,
1141 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001142 EXPECT_TRUE(response.update_exists);
1143}
1144
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001145TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001146 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001147 request_params_.set_wall_clock_based_wait_enabled(true);
1148 request_params_.set_waiting_period(TimeDelta::FromDays(2));
1149 request_params_.set_update_check_count_wait_enabled(true);
1150 request_params_.set_min_update_checks_needed(1);
1151 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001152
Alex Deymo8e18f932015-03-27 16:16:59 -07001153 fake_update_response_.max_days_to_scatter = "0";
Marton Hunyady2abda312018-04-24 18:21:49 +02001154 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1155 -1,
1156 false, // ping_only
1157 ErrorCode::kSuccess,
1158 metrics::CheckResult::kUpdateAvailable,
1159 metrics::CheckReaction::kUpdating,
1160 metrics::DownloadErrorCode::kUnset,
1161 &response,
1162 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001163 EXPECT_TRUE(response.update_exists);
1164}
1165
1166
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001167TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001168 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001169 request_params_.set_wall_clock_based_wait_enabled(true);
1170 request_params_.set_waiting_period(TimeDelta());
1171 request_params_.set_update_check_count_wait_enabled(true);
1172 request_params_.set_min_update_checks_needed(0);
1173 request_params_.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001174
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001175 ASSERT_TRUE(TestUpdateCheck(
Alex Deymo8e18f932015-03-27 16:16:59 -07001176 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001177 -1,
1178 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001179 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001180 metrics::CheckResult::kUpdateAvailable,
1181 metrics::CheckReaction::kUpdating,
1182 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001183 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001184 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001185
Ben Chan9abb7632014-08-07 00:10:53 -07001186 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001187 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001188 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001189 EXPECT_TRUE(response.update_exists);
1190}
1191
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001192TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001193 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001194 request_params_.set_wall_clock_based_wait_enabled(true);
1195 request_params_.set_waiting_period(TimeDelta());
1196 request_params_.set_update_check_count_wait_enabled(true);
1197 request_params_.set_min_update_checks_needed(1);
1198 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001199
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001200 ASSERT_FALSE(TestUpdateCheck(
Alex Deymo8e18f932015-03-27 16:16:59 -07001201 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001202 -1,
1203 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001204 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001205 metrics::CheckResult::kUpdateAvailable,
1206 metrics::CheckReaction::kDeferring,
1207 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001208 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001209 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001210
Ben Chan9abb7632014-08-07 00:10:53 -07001211 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001212 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001213 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001214 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001215
1216 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02001217 request_params_.set_interactive(true);
1218 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1219 -1,
1220 false, // ping_only
1221 ErrorCode::kSuccess,
1222 metrics::CheckResult::kUpdateAvailable,
1223 metrics::CheckReaction::kUpdating,
1224 metrics::DownloadErrorCode::kUnset,
1225 &response,
1226 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001227 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001228}
1229
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001230TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001231 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001232 request_params_.set_wall_clock_based_wait_enabled(true);
1233 request_params_.set_waiting_period(TimeDelta());
1234 request_params_.set_update_check_count_wait_enabled(true);
1235 request_params_.set_min_update_checks_needed(1);
1236 request_params_.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001237
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001238 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001239
1240 ASSERT_FALSE(TestUpdateCheck(
Alex Deymo8e18f932015-03-27 16:16:59 -07001241 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001242 -1,
1243 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001244 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001245 metrics::CheckResult::kUpdateAvailable,
1246 metrics::CheckReaction::kDeferring,
1247 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001248 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001249 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001250
Ben Chan9abb7632014-08-07 00:10:53 -07001251 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001252 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001253 // count remains the same, as the decrementing happens in update_attempter
1254 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001255 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001256 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001257
1258 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02001259 request_params_.set_interactive(true);
1260 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1261 -1,
1262 false, // ping_only
1263 ErrorCode::kSuccess,
1264 metrics::CheckResult::kUpdateAvailable,
1265 metrics::CheckReaction::kUpdating,
1266 metrics::DownloadErrorCode::kUnset,
1267 &response,
1268 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001269 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001270}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001271
Alex Deymo8e18f932015-03-27 16:16:59 -07001272TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1273 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001274 fake_update_response_.include_cohorts = true;
1275 fake_update_response_.cohort = "s/154454/8479665";
1276 fake_update_response_.cohorthint = "please-put-me-on-beta";
1277 fake_update_response_.cohortname = "stable";
1278
Marton Hunyady2abda312018-04-24 18:21:49 +02001279 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo8e18f932015-03-27 16:16:59 -07001280 -1,
1281 false, // ping_only
1282 ErrorCode::kSuccess,
1283 metrics::CheckResult::kUpdateAvailable,
1284 metrics::CheckReaction::kUpdating,
1285 metrics::DownloadErrorCode::kUnset,
1286 &response,
1287 nullptr));
1288
1289 string value;
1290 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1291 EXPECT_EQ(fake_update_response_.cohort, value);
1292
1293 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1294 EXPECT_EQ(fake_update_response_.cohorthint, value);
1295
1296 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1297 EXPECT_EQ(fake_update_response_.cohortname, value);
1298}
1299
1300TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1301 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001302 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1303 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1304 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1305 fake_update_response_.include_cohorts = true;
1306 fake_update_response_.cohort = "s/154454/8479665";
1307 fake_update_response_.cohorthint = "please-put-me-on-beta";
1308 fake_update_response_.cohortname = "";
1309
Marton Hunyady2abda312018-04-24 18:21:49 +02001310 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo8e18f932015-03-27 16:16:59 -07001311 -1,
1312 false, // ping_only
1313 ErrorCode::kSuccess,
1314 metrics::CheckResult::kUpdateAvailable,
1315 metrics::CheckReaction::kUpdating,
1316 metrics::DownloadErrorCode::kUnset,
1317 &response,
1318 nullptr));
1319
1320 string value;
1321 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1322 EXPECT_EQ(fake_update_response_.cohort, value);
1323
1324 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1325 EXPECT_EQ(fake_update_response_.cohorthint, value);
1326
1327 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1328}
1329
1330TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1331 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001332 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1333
Marton Hunyady2abda312018-04-24 18:21:49 +02001334 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Alex Deymo8e18f932015-03-27 16:16:59 -07001335 -1,
1336 false, // ping_only
1337 ErrorCode::kSuccess,
1338 metrics::CheckResult::kUpdateAvailable,
1339 metrics::CheckReaction::kUpdating,
1340 metrics::DownloadErrorCode::kUnset,
1341 &response,
1342 nullptr));
1343
1344 string value;
1345 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1346 EXPECT_EQ("old_value", value);
1347
1348 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1349 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1350}
1351
Alex Deymo00d79ac2015-06-29 15:41:49 -07001352TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1353 OmahaResponse response;
Alex Deymo00d79ac2015-06-29 15:41:49 -07001354 fake_update_response_.include_cohorts = true;
1355 fake_update_response_.cohort = "s/154454/8479665";
1356 fake_update_response_.cohorthint = "please-put-me-on-beta";
1357 fake_update_response_.cohortname = "stable";
1358
Marton Hunyady2abda312018-04-24 18:21:49 +02001359 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
Alex Deymo00d79ac2015-06-29 15:41:49 -07001360 -1,
1361 false, // ping_only
1362 ErrorCode::kSuccess,
1363 metrics::CheckResult::kNoUpdateAvailable,
1364 metrics::CheckReaction::kUnset,
1365 metrics::DownloadErrorCode::kUnset,
1366 &response,
1367 nullptr));
1368
1369 string value;
1370 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1371 EXPECT_EQ(fake_update_response_.cohort, value);
1372
1373 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1374 EXPECT_EQ(fake_update_response_.cohorthint, value);
1375
1376 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1377 EXPECT_EQ(fake_update_response_.cohortname, value);
1378}
1379
Sen Jiangb1e063a2017-09-15 17:44:31 -07001380TEST_F(OmahaRequestActionTest, MultiAppCohortTest) {
1381 OmahaResponse response;
Sen Jiangb1e063a2017-09-15 17:44:31 -07001382 fake_update_response_.multi_app = true;
1383 fake_update_response_.include_cohorts = true;
1384 fake_update_response_.cohort = "s/154454/8479665";
1385 fake_update_response_.cohorthint = "please-put-me-on-beta";
1386 fake_update_response_.cohortname = "stable";
1387
Marton Hunyady2abda312018-04-24 18:21:49 +02001388 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiangb1e063a2017-09-15 17:44:31 -07001389 -1,
1390 false, // ping_only
1391 ErrorCode::kSuccess,
1392 metrics::CheckResult::kUpdateAvailable,
1393 metrics::CheckReaction::kUpdating,
1394 metrics::DownloadErrorCode::kUnset,
1395 &response,
1396 nullptr));
1397
1398 string value;
1399 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1400 EXPECT_EQ(fake_update_response_.cohort, value);
1401
1402 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1403 EXPECT_EQ(fake_update_response_.cohorthint, value);
1404
1405 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1406 EXPECT_EQ(fake_update_response_.cohortname, value);
1407}
1408
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001409TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001410 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001411
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001412 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001413 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001414
Ben Chan5c02c132017-06-27 07:10:36 -07001415 OmahaRequestAction action(
1416 &fake_system_state_,
1417 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001418 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001419 http_response.size(),
1420 nullptr),
1421 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001422 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001423 ActionProcessor processor;
1424 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001425 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001426
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001427 loop.PostTask(base::Bind(
1428 [](ActionProcessor* processor) { processor->StartProcessing(); },
1429 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001430 loop.Run();
1431 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001432 EXPECT_FALSE(processor.IsRunning());
1433}
1434
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001435TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001436 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001437 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
1438 -1,
1439 false, // ping_only
1440 ErrorCode::kOmahaRequestXMLParseError,
1441 metrics::CheckResult::kParsingError,
1442 metrics::CheckReaction::kUnset,
1443 metrics::DownloadErrorCode::kUnset,
1444 &response,
1445 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001446 EXPECT_FALSE(response.update_exists);
1447}
1448
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001449TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001450 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001451 ASSERT_FALSE(TestUpdateCheck("",
1452 -1,
1453 false, // ping_only
1454 ErrorCode::kOmahaRequestEmptyResponseError,
1455 metrics::CheckResult::kParsingError,
1456 metrics::CheckReaction::kUnset,
1457 metrics::DownloadErrorCode::kUnset,
1458 &response,
1459 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001460 EXPECT_FALSE(response.update_exists);
1461}
1462
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001463TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001464 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001465 ASSERT_FALSE(TestUpdateCheck(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001466 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1467 "<daystart elapsed_seconds=\"100\"/>"
1468 "<app appid=\"foo\" status=\"ok\">"
1469 "<ping status=\"ok\"/>"
1470 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001471 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001472 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001473 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001474 metrics::CheckResult::kParsingError,
1475 metrics::CheckReaction::kUnset,
1476 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001477 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001478 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001479 EXPECT_FALSE(response.update_exists);
1480}
1481
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001482TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001483 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001484 ASSERT_FALSE(TestUpdateCheck(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001485 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1486 "<daystart elapsed_seconds=\"100\"/>"
1487 "<app appid=\"foo\" status=\"ok\">"
1488 "<ping status=\"ok\"/>"
1489 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001490 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001491 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001492 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001493 metrics::CheckResult::kParsingError,
1494 metrics::CheckReaction::kUnset,
1495 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001496 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001497 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001498 EXPECT_FALSE(response.update_exists);
1499}
1500
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001501TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001502 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001503 ASSERT_FALSE(TestUpdateCheck(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001504 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1505 "<daystart elapsed_seconds=\"100\"/>"
1506 "<app appid=\"foo\" status=\"ok\">"
1507 "<ping status=\"ok\"/>"
1508 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001509 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001510 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001511 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001512 metrics::CheckResult::kParsingError,
1513 metrics::CheckReaction::kUnset,
1514 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001515 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001516 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001517 EXPECT_FALSE(response.update_exists);
1518}
1519
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001520TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001521 string input_response =
1522 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1523 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001524 // the appid needs to match that in the request params
1525 "<app appid=\"" +
1526 fake_update_response_.app_id +
1527 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001528 "<updatecheck status=\"ok\">"
1529 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001530 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001531 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001532 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001533 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001534 "ChromeOSVersion=\"10.2.3.4\" "
1535 "Prompt=\"false\" "
1536 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001537 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001538 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001539 "needsadmin=\"true\" "
1540 "/></actions></manifest></updatecheck></app></response>";
1541 LOG(INFO) << "Input Response = " << input_response;
1542
Darin Petkov6a5b3222010-07-13 14:55:28 -07001543 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001544 ASSERT_TRUE(TestUpdateCheck(input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001545 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001546 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001547 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001548 metrics::CheckResult::kUpdateAvailable,
1549 metrics::CheckReaction::kUpdating,
1550 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001551 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001552 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001553 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001554 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001555 EXPECT_EQ("http://missing/field/test/f",
1556 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001557 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001558 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1559 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001560 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001561 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001562}
1563
1564namespace {
1565class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1566 public:
1567 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001568 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001569 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001570};
1571
Alex Deymo60ca1a72015-06-18 18:19:15 -07001572void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001573 processor->StartProcessing();
1574 CHECK(processor->IsRunning());
1575 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001576}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001577} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001578
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001579TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001580 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001581 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001582
Alex Deymo60ca1a72015-06-18 18:19:15 -07001583 string http_response("doesn't matter");
Ben Chan5c02c132017-06-27 07:10:36 -07001584 OmahaRequestAction action(
1585 &fake_system_state_,
1586 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001587 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001588 http_response.size(),
1589 nullptr),
1590 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001591 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001592 ActionProcessor processor;
1593 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001594 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001595
Alex Deymo60ca1a72015-06-18 18:19:15 -07001596 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1597 loop.Run();
1598 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001599}
1600
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001601TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001602 string output;
1603 EXPECT_TRUE(XmlEncode("ab", &output));
1604 EXPECT_EQ("ab", output);
1605 EXPECT_TRUE(XmlEncode("a<b", &output));
1606 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001607 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1608 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001609 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1610 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001611 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001612 EXPECT_FALSE(XmlEncode("\xc2", &output));
1613 // Fail with invalid ASCII-7 chars.
1614 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1615}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001616
Alex Deymob0d74eb2015-03-30 17:59:17 -07001617TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1618 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1619 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1620}
1621
1622TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001623 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001624
1625 // Make sure XML Encode is being called on the params
Marton Hunyady2abda312018-04-24 18:21:49 +02001626 request_params_.set_os_sp("testtheservice_pack>");
1627 request_params_.set_os_board("x86 generic<id");
1628 request_params_.set_current_channel("unittest_track&lt;");
1629 request_params_.set_target_channel("unittest_track&lt;");
1630 request_params_.set_hwid("<OEM MODEL>");
Alex Deymo8e18f932015-03-27 16:16:59 -07001631 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1632 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1633 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001634 base::JoinString(
1635 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001636 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02001637 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
1638 -1,
1639 false, // ping_only
1640 ErrorCode::kOmahaRequestXMLParseError,
1641 metrics::CheckResult::kParsingError,
1642 metrics::CheckReaction::kUnset,
1643 metrics::DownloadErrorCode::kUnset,
1644 &response,
1645 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001646 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001647 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001648 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1649 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1650 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1651 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1652 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1653 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1654 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1655 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1656 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1657 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1658 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1659 // Values from Prefs that are too big are removed from the XML instead of
1660 // encoded.
1661 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001662}
1663
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001664TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001665 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001666 fake_update_response_.deadline = "&lt;20110101";
1667 fake_update_response_.more_info_url = "testthe&lt;url";
1668 fake_update_response_.codebase = "testthe&amp;codebase/";
Marton Hunyady2abda312018-04-24 18:21:49 +02001669 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1670 -1,
1671 false, // ping_only
1672 ErrorCode::kSuccess,
1673 metrics::CheckResult::kUpdateAvailable,
1674 metrics::CheckReaction::kUpdating,
1675 metrics::DownloadErrorCode::kUnset,
1676 &response,
1677 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001678
Sen Jiang0affc2c2017-02-10 15:55:05 -08001679 EXPECT_EQ("testthe<url", response.more_info_url);
1680 EXPECT_EQ("testthe&codebase/file.signed",
1681 response.packages[0].payload_urls[0]);
1682 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001683}
1684
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001685TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001686 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001687 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001688 fake_update_response_.size = 123123123123123ull;
Marton Hunyady2abda312018-04-24 18:21:49 +02001689 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
1690 -1,
1691 false, // ping_only
1692 ErrorCode::kSuccess,
1693 metrics::CheckResult::kUpdateAvailable,
1694 metrics::CheckReaction::kUpdating,
1695 metrics::DownloadErrorCode::kUnset,
1696 &response,
1697 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001698
Sen Jiang0affc2c2017-02-10 15:55:05 -08001699 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001700}
1701
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001702TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001703 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001704 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001705 fake_system_state_.set_prefs(&prefs);
1706
Darin Petkov95508da2011-01-05 12:42:29 -08001707 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001708 .WillOnce(DoAll(SetArgPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001709 // An existing but empty previous version means that we didn't reboot to a new
1710 // update, therefore, no need to update the previous version.
1711 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Marton Hunyady2abda312018-04-24 18:21:49 +02001712 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001713 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001714 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001715 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001716 metrics::CheckResult::kParsingError,
1717 metrics::CheckReaction::kUnset,
1718 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001719 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001720 &post_data));
1721 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001722 string post_str(post_data.begin(), post_data.end());
Marton Hunyadyba51c3f2018-04-25 15:18:10 +02001723 EXPECT_NE(
1724 post_str.find(" <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
1725 " <updatecheck></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001726 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001727 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1728 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001729 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1730 string::npos);
1731 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1732 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001733 // No <event> tag should be sent if we didn't reboot to an update.
1734 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001735}
1736
Jay Srinivasan0a708742012-03-20 11:26:12 -07001737
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001738TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001739 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001740 TestEvent(new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
Darin Petkove17f86b2010-07-20 09:12:01 -07001741 "invalid xml>",
1742 &post_data);
1743 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001744 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001745 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001746 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001747 OmahaEvent::kTypeUpdateDownloadStarted,
1748 OmahaEvent::kResultSuccess);
1749 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001750 EXPECT_EQ(post_str.find("ping"), string::npos);
1751 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001752}
1753
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001754TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001755 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001756 TestEvent(new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
Darin Petkove17f86b2010-07-20 09:12:01 -07001757 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001758 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001759 "invalid xml>",
1760 &post_data);
1761 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001762 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001763 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001764 " <event eventtype=\"%d\" eventresult=\"%d\" "
1765 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001766 OmahaEvent::kTypeDownloadComplete,
1767 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001768 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001769 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001770 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001771}
1772
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001773TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001774 string http_response("doesn't matter");
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001775 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001776 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001777 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001778 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001779 http_response.size(),
1780 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001781 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001782 EXPECT_FALSE(update_check_action.IsEvent());
1783
1784 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001785 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001786 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Ben Chanab5a0af2017-10-12 14:57:50 -07001787 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001788 http_response.size(),
1789 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001790 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001791 EXPECT_TRUE(event_action.IsEvent());
1792}
1793
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001794TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001795 for (int i = 0; i < 2; i++) {
1796 bool delta_okay = i == 1;
1797 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001798 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001799
1800 request_params_.set_delta_okay(delta_okay);
1801
1802 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001803 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001804 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001805 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001806 metrics::CheckResult::kParsingError,
1807 metrics::CheckReaction::kUnset,
1808 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001809 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001810 &post_data));
1811 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001812 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001813 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1814 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001815 string::npos)
1816 << "i = " << i;
1817 }
1818}
1819
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001820TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001821 for (int i = 0; i < 2; i++) {
1822 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001823 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001824 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001825 FakeSystemState fake_system_state;
Marton Hunyady2abda312018-04-24 18:21:49 +02001826
1827 request_params_.set_interactive(interactive);
1828
1829 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001830 -1,
1831 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001832 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001833 metrics::CheckResult::kParsingError,
1834 metrics::CheckReaction::kUnset,
1835 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001836 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001837 &post_data));
1838 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001839 string post_str(post_data.begin(), post_data.end());
Marton Hunyadyba51c3f2018-04-25 15:18:10 +02001840 EXPECT_NE(post_str.find(
1841 base::StringPrintf("installsource=\"%s\"", interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001842 string::npos)
1843 << "i = " << i;
1844 }
1845}
1846
Marton Hunyadyba51c3f2018-04-25 15:18:10 +02001847TEST_F(OmahaRequestActionTest, FormatTargetVersionPrefixOutputTest) {
1848 for (int i = 0; i < 2; i++) {
1849 bool target_version_set = i == 1;
1850 const char* target_version_prefix = target_version_set ? "10032." : "";
1851 brillo::Blob post_data;
1852 FakeSystemState fake_system_state;
1853
1854 request_params_.set_target_version_prefix(target_version_prefix);
1855
1856 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
1857 -1,
1858 false, // ping_only
1859 ErrorCode::kOmahaRequestXMLParseError,
1860 metrics::CheckResult::kParsingError,
1861 metrics::CheckReaction::kUnset,
1862 metrics::DownloadErrorCode::kUnset,
1863 nullptr,
1864 &post_data));
1865 // convert post_data to string
1866 string post_str(post_data.begin(), post_data.end());
1867 if (target_version_set) {
1868 EXPECT_NE(post_str.find("<updatecheck targetversionprefix=\"10032.\">"),
1869 string::npos)
1870 << "i = " << i;
1871 } else {
1872 EXPECT_EQ(post_str.find("targetversionprefix"), string::npos)
1873 << "i = " << i;
1874 }
1875 }
1876}
1877
1878TEST_F(OmahaRequestActionTest, FormatRollbackAllowedOutputTest) {
1879 for (int i = 0; i < 4; i++) {
1880 bool rollback_allowed = i / 2 == 0;
1881 bool target_version_set = i % 2 == 0;
1882 brillo::Blob post_data;
1883 FakeSystemState fake_system_state;
1884
1885 request_params_.set_target_version_prefix(target_version_set ? "10032."
1886 : "");
1887 request_params_.set_rollback_allowed(rollback_allowed);
1888
1889 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
1890 -1,
1891 false, // ping_only
1892 ErrorCode::kOmahaRequestXMLParseError,
1893 metrics::CheckResult::kParsingError,
1894 metrics::CheckReaction::kUnset,
1895 metrics::DownloadErrorCode::kUnset,
1896 nullptr,
1897 &post_data));
1898 // convert post_data to string
1899 string post_str(post_data.begin(), post_data.end());
1900 if (rollback_allowed && target_version_set) {
1901 EXPECT_NE(post_str.find("rollback_allowed=\"true\""), string::npos)
1902 << "i = " << i;
1903 } else {
1904 EXPECT_EQ(post_str.find("rollback_allowed"), string::npos) << "i = " << i;
1905 }
1906 }
1907}
1908
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001909TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001910 OmahaEvent default_event;
1911 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1912 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001913 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001914
1915 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1916 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1917 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001918 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001919
1920 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1921 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001922 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001923 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1924 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001925 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001926}
1927
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001928void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001929 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001930 fake_system_state_.set_prefs(&prefs);
1931 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1932 .Times(AnyNumber());
1933 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1934 // Add a few hours to the day difference to test no rounding, etc.
1935 int64_t five_days_ago =
1936 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1937 int64_t six_days_ago =
1938 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1939 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001940 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001941 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001942 .WillOnce(DoAll(SetArgPointee<1>(six_days_ago), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001943 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001944 .WillOnce(DoAll(SetArgPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001945 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001946 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001947 -1,
1948 ping_only,
1949 ErrorCode::kSuccess,
1950 metrics::CheckResult::kNoUpdateAvailable,
1951 metrics::CheckReaction::kUnset,
1952 metrics::DownloadErrorCode::kUnset,
1953 nullptr,
1954 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001955 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001956 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1957 string::npos);
1958 if (ping_only) {
1959 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1960 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1961 } else {
1962 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1963 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001964 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001965}
1966
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001967TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1968 PingTest(true /* ping_only */);
1969}
1970
1971TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1972 PingTest(false /* ping_only */);
1973}
1974
1975TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001976 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001977 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001978 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1979 .Times(AnyNumber());
1980 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001981 int64_t three_days_ago =
1982 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1983 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001984 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001985 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001986 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001987 .WillOnce(DoAll(SetArgPointee<1>(three_days_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001988 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001989 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001990 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02001991 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
1992 -1,
1993 false, // ping_only
1994 ErrorCode::kSuccess,
1995 metrics::CheckResult::kNoUpdateAvailable,
1996 metrics::CheckReaction::kUnset,
1997 metrics::DownloadErrorCode::kUnset,
1998 nullptr,
1999 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002000 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002001 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07002002 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002003}
2004
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002005TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002006 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002007 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002008 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
2009 .Times(AnyNumber());
2010 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002011 int64_t four_days_ago =
2012 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
2013 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08002014 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002015 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002016 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002017 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002018 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002019 .WillOnce(DoAll(SetArgPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002020 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002021 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2022 -1,
2023 false, // ping_only
2024 ErrorCode::kSuccess,
2025 metrics::CheckResult::kNoUpdateAvailable,
2026 metrics::CheckReaction::kUnset,
2027 metrics::DownloadErrorCode::kUnset,
2028 nullptr,
2029 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002030 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002031 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07002032 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002033}
2034
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002035TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002036 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002037 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002038 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
2039 .Times(AnyNumber());
2040 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002041 int64_t one_hour_ago =
2042 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08002043 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002044 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002045 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002046 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002047 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002048 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002049 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
2050 // send a ping.
2051 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
2052 .WillOnce(Return(true));
2053 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
2054 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002055 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002056 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2057 -1,
2058 false, // ping_only
2059 ErrorCode::kSuccess,
2060 metrics::CheckResult::kNoUpdateAvailable,
2061 metrics::CheckReaction::kUnset,
2062 metrics::DownloadErrorCode::kUnset,
2063 nullptr,
2064 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002065 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002066 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002067}
2068
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002069TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07002070 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08002071 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002072 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07002073 int64_t now = Time::Now().ToInternalValue();
2074 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002075 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07002076 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002077 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07002078 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2079 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002080 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002081 EXPECT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2082 -1,
2083 true, // ping_only
2084 ErrorCode::kSuccess,
2085 metrics::CheckResult::kUnset,
2086 metrics::CheckReaction::kUnset,
2087 metrics::DownloadErrorCode::kUnset,
2088 nullptr,
2089 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08002090 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07002091}
2092
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002093TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002094 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002095 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002096 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
2097 .Times(AnyNumber());
2098 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002099 int64_t future =
2100 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08002101 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002102 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002103 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002104 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002105 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002106 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002107 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
2108 .WillOnce(Return(true));
2109 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
2110 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002111 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002112 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002113 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002114 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002115 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002116 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002117 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002118 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002119 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002120 metrics::CheckResult::kNoUpdateAvailable,
2121 metrics::CheckReaction::kUnset,
2122 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002123 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002124 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002125 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002126 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002127}
2128
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002129TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002130 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07002131 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002132 // may fail if it runs for longer than 5 seconds. It shouldn't run
2133 // that long though.
2134 int64_t midnight =
2135 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
2136 int64_t midnight_slack =
2137 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08002138 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002139 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002140 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2141 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002142 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
2143 AllOf(Ge(midnight), Le(midnight_slack))))
2144 .WillOnce(Return(true));
2145 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
2146 AllOf(Ge(midnight), Le(midnight_slack))))
2147 .WillOnce(Return(true));
2148 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002149 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002150 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002151 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002152 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002153 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002154 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002155 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002156 metrics::CheckResult::kNoUpdateAvailable,
2157 metrics::CheckReaction::kUnset,
2158 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002159 nullptr,
2160 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002161}
2162
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002163TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002164 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002165 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002166 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2167 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002168 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2169 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2170 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002171 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002172 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002173 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002174 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002175 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002176 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002177 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002178 metrics::CheckResult::kNoUpdateAvailable,
2179 metrics::CheckReaction::kUnset,
2180 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002181 nullptr,
2182 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002183}
2184
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002185TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002186 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002187 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002188 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2189 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002190 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2191 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2192 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002193 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002194 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002195 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002196 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002197 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002198 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002199 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002200 metrics::CheckResult::kNoUpdateAvailable,
2201 metrics::CheckReaction::kUnset,
2202 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002203 nullptr,
2204 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002205}
2206
Alex Deymob3fa53b2016-04-18 19:57:58 -07002207TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
2208 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2209 // the "eol" attribute.
2210 ASSERT_TRUE(
Marton Hunyady2abda312018-04-24 18:21:49 +02002211 TestUpdateCheck("<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
Alex Deymob3fa53b2016-04-18 19:57:58 -07002212 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2213 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2214 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2215 "</app></response>",
2216 -1,
2217 false, // ping_only
2218 ErrorCode::kSuccess,
2219 metrics::CheckResult::kNoUpdateAvailable,
2220 metrics::CheckReaction::kUnset,
2221 metrics::DownloadErrorCode::kUnset,
2222 nullptr,
2223 nullptr));
2224 string eol_pref;
2225 EXPECT_TRUE(
2226 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2227 // Note that the eol="eol" attribute should be ignored and the _eol should be
2228 // used instead.
2229 EXPECT_EQ("security-only", eol_pref);
2230}
2231
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002232TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002233 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002234 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002235 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002236 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002237 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002238 metrics::CheckResult::kParsingError,
2239 metrics::CheckReaction::kUnset,
2240 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002241 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002242 &post_data));
2243 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002244 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002245 EXPECT_EQ(post_str.find("machineid="), string::npos);
2246 EXPECT_EQ(post_str.find("userid="), string::npos);
2247}
2248
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002249TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002250 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002251 const int http_error_code =
2252 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Marton Hunyady2abda312018-04-24 18:21:49 +02002253 ASSERT_FALSE(TestUpdateCheck("",
2254 501,
2255 false, // ping_only
2256 static_cast<ErrorCode>(http_error_code),
2257 metrics::CheckResult::kDownloadError,
2258 metrics::CheckReaction::kUnset,
2259 static_cast<metrics::DownloadErrorCode>(501),
2260 &response,
2261 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002262 EXPECT_FALSE(response.update_exists);
2263}
2264
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002265TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002266 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002267 const int http_error_code =
2268 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Marton Hunyady2abda312018-04-24 18:21:49 +02002269 ASSERT_FALSE(TestUpdateCheck("",
2270 1500,
2271 false, // ping_only
2272 static_cast<ErrorCode>(http_error_code),
2273 metrics::CheckResult::kDownloadError,
2274 metrics::CheckReaction::kUnset,
2275 metrics::DownloadErrorCode::kHttpStatusOther,
2276 &response,
2277 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002278 EXPECT_FALSE(response.update_exists);
2279}
2280
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002281TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002282 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02002283 request_params_.set_wall_clock_based_wait_enabled(true);
2284 request_params_.set_waiting_period(TimeDelta().FromDays(1));
2285 request_params_.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002286
Sen Jiang7c1171e2016-06-23 11:35:40 -07002287 Time arbitrary_date;
Amin Hassani71818c82018-03-06 13:25:40 -08002288 Time::FromString("6/4/1989", &arbitrary_date);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002289 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
Marton Hunyady2abda312018-04-24 18:21:49 +02002290 ASSERT_FALSE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang7c1171e2016-06-23 11:35:40 -07002291 -1,
2292 false, // ping_only
2293 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2294 metrics::CheckResult::kUpdateAvailable,
2295 metrics::CheckReaction::kDeferring,
2296 metrics::DownloadErrorCode::kUnset,
2297 &response,
2298 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002299
Ben Chan9abb7632014-08-07 00:10:53 -07002300 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002301 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002302 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002303 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002304
2305 // Verify if we are interactive check we don't defer.
Marton Hunyady2abda312018-04-24 18:21:49 +02002306 request_params_.set_interactive(true);
2307 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang7c1171e2016-06-23 11:35:40 -07002308 -1,
2309 false, // ping_only
2310 ErrorCode::kSuccess,
2311 metrics::CheckResult::kUpdateAvailable,
2312 metrics::CheckReaction::kUpdating,
2313 metrics::DownloadErrorCode::kUnset,
2314 &response,
2315 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002316 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002317}
2318
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002319TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002320 OmahaResponse response;
Marton Hunyady2abda312018-04-24 18:21:49 +02002321 request_params_.set_wall_clock_based_wait_enabled(true);
2322 request_params_.set_waiting_period(TimeDelta().FromDays(1));
2323 request_params_.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002324
Sen Jiang7c1171e2016-06-23 11:35:40 -07002325 Time t1, t2;
Amin Hassani71818c82018-03-06 13:25:40 -08002326 Time::FromString("1/1/2012", &t1);
2327 Time::FromString("1/3/2012", &t2);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002328 ASSERT_TRUE(
2329 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2330 fake_system_state_.fake_clock()->SetWallclockTime(t2);
Marton Hunyady2abda312018-04-24 18:21:49 +02002331 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
Sen Jiang7c1171e2016-06-23 11:35:40 -07002332 -1,
2333 false, // ping_only
2334 ErrorCode::kSuccess,
2335 metrics::CheckResult::kUpdateAvailable,
2336 metrics::CheckReaction::kUpdating,
2337 metrics::DownloadErrorCode::kUnset,
2338 &response,
2339 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002340
2341 EXPECT_TRUE(response.update_exists);
2342
2343 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002344 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002345 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002346 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2347}
2348
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002349TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002350 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002351 base::ScopedTempDir tempdir;
2352 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002353
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002354 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002355 request_params_.set_root(tempdir.GetPath().value());
2356 request_params_.set_app_id("{22222222-2222-2222-2222-222222222222}");
2357 request_params_.set_app_version("1.2.3.4");
2358 request_params_.set_product_components("o.bundle=1");
2359 request_params_.set_current_channel("canary-channel");
2360 EXPECT_TRUE(
2361 request_params_.SetTargetChannel("stable-channel", true, nullptr));
2362 request_params_.UpdateDownloadChannel();
2363 EXPECT_TRUE(request_params_.ShouldPowerwash());
2364 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002365 -1,
2366 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002367 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002368 metrics::CheckResult::kParsingError,
2369 metrics::CheckReaction::kUnset,
2370 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002371 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002372 &post_data));
2373 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002374 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002375 EXPECT_NE(string::npos, post_str.find(
2376 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2377 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2378 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Sen Jiang8cd42342018-01-31 12:06:59 -08002379 EXPECT_EQ(string::npos, post_str.find("o.bundle"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002380}
2381
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002382TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002383 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002384 base::ScopedTempDir tempdir;
2385 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002386
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002387 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002388 request_params_.set_root(tempdir.GetPath().value());
2389 request_params_.set_app_id("{11111111-1111-1111-1111-111111111111}");
2390 request_params_.set_app_version("5.6.7.8");
2391 request_params_.set_product_components("o.bundle=1");
2392 request_params_.set_current_channel("stable-channel");
2393 EXPECT_TRUE(
2394 request_params_.SetTargetChannel("canary-channel", false, nullptr));
2395 request_params_.UpdateDownloadChannel();
2396 EXPECT_FALSE(request_params_.ShouldPowerwash());
2397 ASSERT_FALSE(TestUpdateCheck("invalid xml>",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002398 -1,
2399 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002400 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002401 metrics::CheckResult::kParsingError,
2402 metrics::CheckReaction::kUnset,
2403 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002404 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002405 &post_data));
2406 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002407 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002408 EXPECT_NE(string::npos, post_str.find(
2409 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2410 "version=\"5.6.7.8\" "
2411 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002412 EXPECT_EQ(string::npos, post_str.find("from_version"));
Sen Jiang8cd42342018-01-31 12:06:59 -08002413 EXPECT_NE(string::npos, post_str.find("o.bundle.version=\"1\""));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002414}
2415
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002416// Checks that the initial ping with a=-1 r=-1 is not send when the device
2417// was powerwashed.
2418TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2419 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2420
2421 // Flag that the device was powerwashed in the past.
2422 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2423
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002424 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002425 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2426 -1,
2427 false, // ping_only
2428 ErrorCode::kSuccess,
2429 metrics::CheckResult::kNoUpdateAvailable,
2430 metrics::CheckReaction::kUnset,
2431 metrics::DownloadErrorCode::kUnset,
2432 nullptr,
2433 &post_data));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002434 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002435 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002436 EXPECT_EQ(string::npos, post_str.find("<ping"));
2437}
2438
Amin Hassani1677e812017-06-21 13:36:36 -07002439// Checks that the initial ping with a=-1 r=-1 is not send when the device
2440// first_active_omaha_ping_sent is set.
2441TEST_F(OmahaRequestActionTest, PingWhenFirstActiveOmahaPingIsSent) {
2442 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2443
2444 // Flag that the device was not powerwashed in the past.
2445 fake_system_state_.fake_hardware()->SetPowerwashCount(0);
2446
2447 // Flag that the device has sent first active ping in the past.
2448 fake_system_state_.fake_hardware()->SetFirstActiveOmahaPingSent();
2449
2450 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002451 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2452 -1,
2453 false, // ping_only
2454 ErrorCode::kSuccess,
2455 metrics::CheckResult::kNoUpdateAvailable,
2456 metrics::CheckReaction::kUnset,
2457 metrics::DownloadErrorCode::kUnset,
2458 nullptr,
2459 &post_data));
Amin Hassani1677e812017-06-21 13:36:36 -07002460 // We shouldn't send a ping in this case since
2461 // first_active_omaha_ping_sent=true
2462 string post_str(post_data.begin(), post_data.end());
2463 EXPECT_EQ(string::npos, post_str.find("<ping"));
2464}
2465
Alex Deymo9fded1e2015-11-05 12:31:19 -08002466// Checks that the event 54 is sent on a reboot to a new update.
2467TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2468 // Flag that the device was updated in a previous boot.
2469 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2470
2471 brillo::Blob post_data;
Marton Hunyady2abda312018-04-24 18:21:49 +02002472 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetNoUpdateResponse(),
2473 -1,
2474 false, // ping_only
2475 ErrorCode::kSuccess,
2476 metrics::CheckResult::kNoUpdateAvailable,
2477 metrics::CheckReaction::kUnset,
2478 metrics::DownloadErrorCode::kUnset,
2479 nullptr,
2480 &post_data));
Alex Deymo9fded1e2015-11-05 12:31:19 -08002481 string post_str(post_data.begin(), post_data.end());
2482
2483 // An event 54 is included and has the right version.
2484 EXPECT_NE(string::npos,
2485 post_str.find(base::StringPrintf(
2486 "<event eventtype=\"%d\"",
2487 OmahaEvent::kTypeRebootedAfterUpdate)));
2488 EXPECT_NE(string::npos,
2489 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2490
2491 // The previous version flag should have been removed.
2492 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2493 string prev_version;
2494 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2495 EXPECT_TRUE(prev_version.empty());
2496}
2497
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002498void OmahaRequestActionTest::P2PTest(
2499 bool initial_allow_p2p_for_downloading,
2500 bool initial_allow_p2p_for_sharing,
2501 bool omaha_disable_p2p_for_downloading,
2502 bool omaha_disable_p2p_for_sharing,
2503 bool payload_state_allow_p2p_attempt,
2504 bool expect_p2p_client_lookup,
2505 const string& p2p_client_result_url,
2506 bool expected_allow_p2p_for_downloading,
2507 bool expected_allow_p2p_for_sharing,
2508 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002509 OmahaResponse response;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002510 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2511 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2512 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002513
2514 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002515 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002516 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2517 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002518 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2519 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2520 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2521 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2522 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2523 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2524 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2525 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2526 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2527 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2528
David Zeuthen8f191b22013-08-06 12:27:50 -07002529 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002530 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002531 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2532
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002533 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2534 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002535 .Times(expect_p2p_client_lookup ? 1 : 0);
2536
Alex Deymo8e18f932015-03-27 16:16:59 -07002537 fake_update_response_.disable_p2p_for_downloading =
2538 omaha_disable_p2p_for_downloading;
2539 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
Marton Hunyady2abda312018-04-24 18:21:49 +02002540 ASSERT_TRUE(TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
2541 -1,
2542 false, // ping_only
2543 ErrorCode::kSuccess,
2544 metrics::CheckResult::kUpdateAvailable,
2545 metrics::CheckReaction::kUpdating,
2546 metrics::DownloadErrorCode::kUnset,
2547 &response,
2548 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002549 EXPECT_TRUE(response.update_exists);
2550
Gilad Arnold74b5f552014-10-07 08:17:16 -07002551 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2552 response.disable_p2p_for_downloading);
2553 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2554 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002555
Gilad Arnold74b5f552014-10-07 08:17:16 -07002556 EXPECT_EQ(expected_allow_p2p_for_downloading,
2557 actual_allow_p2p_for_downloading);
2558 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2559 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002560}
2561
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002562TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002563 P2PTest(true, // initial_allow_p2p_for_downloading
2564 true, // initial_allow_p2p_for_sharing
2565 false, // omaha_disable_p2p_for_downloading
2566 false, // omaha_disable_p2p_for_sharing
2567 true, // payload_state_allow_p2p_attempt
2568 true, // expect_p2p_client_lookup
2569 "http://1.3.5.7/p2p", // p2p_client_result_url
2570 true, // expected_allow_p2p_for_downloading
2571 true, // expected_allow_p2p_for_sharing
2572 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002573}
2574
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002575TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002576 P2PTest(true, // initial_allow_p2p_for_downloading
2577 true, // initial_allow_p2p_for_sharing
2578 false, // omaha_disable_p2p_for_downloading
2579 false, // omaha_disable_p2p_for_sharing
2580 true, // payload_state_allow_p2p_attempt
2581 true, // expect_p2p_client_lookup
2582 "", // p2p_client_result_url
2583 false, // expected_allow_p2p_for_downloading
2584 true, // expected_allow_p2p_for_sharing
2585 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002586}
2587
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002588TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002589 P2PTest(false, // initial_allow_p2p_for_downloading
2590 true, // initial_allow_p2p_for_sharing
2591 false, // omaha_disable_p2p_for_downloading
2592 false, // omaha_disable_p2p_for_sharing
2593 true, // payload_state_allow_p2p_attempt
2594 false, // expect_p2p_client_lookup
2595 "unset", // p2p_client_result_url
2596 false, // expected_allow_p2p_for_downloading
2597 true, // expected_allow_p2p_for_sharing
2598 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002599}
2600
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002601TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002602 P2PTest(true, // initial_allow_p2p_for_downloading
2603 true, // initial_allow_p2p_for_sharing
2604 true, // omaha_disable_p2p_for_downloading
2605 false, // omaha_disable_p2p_for_sharing
2606 true, // payload_state_allow_p2p_attempt
2607 false, // expect_p2p_client_lookup
2608 "unset", // p2p_client_result_url
2609 false, // expected_allow_p2p_for_downloading
2610 true, // expected_allow_p2p_for_sharing
2611 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002612}
2613
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002614TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002615 P2PTest(true, // initial_allow_p2p_for_downloading
2616 true, // initial_allow_p2p_for_sharing
2617 false, // omaha_disable_p2p_for_downloading
2618 true, // omaha_disable_p2p_for_sharing
2619 true, // payload_state_allow_p2p_attempt
2620 true, // expect_p2p_client_lookup
2621 "http://1.3.5.7/p2p", // p2p_client_result_url
2622 true, // expected_allow_p2p_for_downloading
2623 false, // expected_allow_p2p_for_sharing
2624 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002625}
2626
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002627TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002628 P2PTest(true, // initial_allow_p2p_for_downloading
2629 true, // initial_allow_p2p_for_sharing
2630 true, // omaha_disable_p2p_for_downloading
2631 true, // omaha_disable_p2p_for_sharing
2632 true, // payload_state_allow_p2p_attempt
2633 false, // expect_p2p_client_lookup
2634 "unset", // p2p_client_result_url
2635 false, // expected_allow_p2p_for_downloading
2636 false, // expected_allow_p2p_for_sharing
2637 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002638}
2639
Alex Deymof329b932014-10-30 01:37:48 -07002640bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2641 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002642 fake_update_response_.elapsed_days = elapsed_days;
Marton Hunyady2abda312018-04-24 18:21:49 +02002643 return TestUpdateCheck(fake_update_response_.GetUpdateResponse(),
2644 -1,
2645 false, // ping_only
2646 ErrorCode::kSuccess,
2647 metrics::CheckResult::kUpdateAvailable,
2648 metrics::CheckReaction::kUpdating,
2649 metrics::DownloadErrorCode::kUnset,
2650 response,
2651 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002652}
2653
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002654TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002655 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002656
Kevin Cernekee2494e282016-03-29 18:03:53 -07002657 // Simulate a successful update check that happens during OOBE. The
2658 // deadline in the response is needed to force the update attempt to
2659 // occur; responses without a deadline seen during OOBE will normally
2660 // return ErrorCode::kNonCriticalUpdateInOOBE.
2661 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2662 fake_update_response_.deadline = "20101020";
2663
David Zeuthen639aa362014-02-03 16:23:44 -08002664 // Check that we parse elapsed_days in the Omaha Response correctly.
2665 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002666 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2667 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002668 EXPECT_TRUE(response.update_exists);
2669 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002670 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002671 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002672 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002673 EXPECT_EQ(prefs_days, 42);
2674
2675 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002676 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002677 EXPECT_TRUE(response.update_exists);
2678 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002679 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002680 EXPECT_EQ(prefs_days, 42);
2681
2682 // Note that elapsed_days is not necessarily divisible by 7 so check
2683 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002684 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2685 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002686 EXPECT_TRUE(response.update_exists);
2687 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002688 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002689 EXPECT_EQ(prefs_days, 21);
2690
2691 // Check that we correctly handle elapsed_days not being included in
2692 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002693 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002694 EXPECT_TRUE(response.update_exists);
2695 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002696}
2697
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002698// If there is no prefs and OOBE is not complete, we should not
2699// report anything to Omaha.
2700TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002701 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002702 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2703 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2704}
David Zeuthen639aa362014-02-03 16:23:44 -08002705
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002706// If OOBE is complete and happened on a valid date (e.g. after Jan
2707// 1 2007 0:00 PST), that date should be used and written to
2708// prefs. However, first try with an invalid date and check we do
2709// nothing.
2710TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2711 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2712 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2713 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2714 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2715}
David Zeuthen639aa362014-02-03 16:23:44 -08002716
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002717// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2718// should yield an InstallDate of 14.
2719TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2720 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2721 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2722 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2723 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002724
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002725 int64_t prefs_days;
2726 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2727 EXPECT_EQ(prefs_days, 14);
2728}
David Zeuthen639aa362014-02-03 16:23:44 -08002729
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002730// Now that we have a valid date in prefs, check that we keep using
2731// that even if OOBE date reports something else. The date Jan 30,
2732// 2007 0:00 PST should yield an InstallDate of 28... but since
2733// there's a prefs file, we should still get 14.
2734TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2735 // Set a valid date in the prefs first.
2736 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002737
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002738 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2739 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2740 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002741
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002742 int64_t prefs_days;
2743 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2744 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002745
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002746 // If we delete the prefs file, we should get 28 days.
2747 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2748 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2749 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2750 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002751}
2752
Zentaro Kavanagh1f899d52018-02-27 15:02:47 -08002753// Verifies that a device with no device policy, and is not a consumer
2754// device sets the max kernel key version to the current version.
2755// ie. the same behavior as if rollback is enabled.
2756TEST_F(OmahaRequestActionTest, NoPolicyEnterpriseDevicesSetMaxRollback) {
2757 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2758
2759 // Setup and verify some initial default values for the kernel TPM
2760 // values that control verified boot and rollback.
2761 const int min_kernel_version = 4;
2762 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2763 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2764 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2765 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2766
2767 TestRollbackCheck(false /* is_consumer_device */,
2768 3 /* rollback_allowed_milestones */,
2769 false /* is_policy_loaded */);
2770
2771 // Verify max_kernel_rollforward was set to the current minimum
2772 // kernel key version. This has the effect of freezing roll
2773 // forwards indefinitely. This will hold the rollback window
2774 // open until a future change will be able to move this forward
2775 // relative the configured window.
2776 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2777 EXPECT_EQ(min_kernel_version, fake_hw->GetMaxKernelKeyRollforward());
2778}
2779
2780// Verifies that a conmsumer device with no device policy sets the
2781// max kernel key version to the current version. ie. the same
2782// behavior as if rollback is enabled.
2783TEST_F(OmahaRequestActionTest, NoPolicyConsumerDevicesSetMaxRollback) {
2784 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2785
2786 // Setup and verify some initial default values for the kernel TPM
2787 // values that control verified boot and rollback.
2788 const int min_kernel_version = 3;
2789 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2790 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2791 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2792 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2793
2794 TestRollbackCheck(true /* is_consumer_device */,
2795 3 /* rollback_allowed_milestones */,
2796 false /* is_policy_loaded */);
2797
2798 // Verify that with rollback disabled that max_kernel_rollforward
2799 // was set to logical infinity. This is the expected behavior for
2800 // consumer devices and matches the existing behavior prior to the
2801 // rollback features.
2802 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2803 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2804}
2805
2806// Verifies that a device with rollback enabled sets max_kernel_rollforward
2807// in the TPM to prevent roll forward.
2808TEST_F(OmahaRequestActionTest, RollbackEnabledDevicesSetMaxRollback) {
2809 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2810
2811 // Setup and verify some initial default values for the kernel TPM
2812 // values that control verified boot and rollback.
2813 const int allowed_milestones = 4;
2814 const int min_kernel_version = 3;
2815 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2816 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2817 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2818 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2819
2820 TestRollbackCheck(false /* is_consumer_device */,
2821 allowed_milestones,
2822 true /* is_policy_loaded */);
2823
2824 // Verify that with rollback enabled that max_kernel_rollforward
2825 // was set to the current minimum kernel key version. This has
2826 // the effect of freezing roll forwards indefinitely. This will
2827 // hold the rollback window open until a future change will
2828 // be able to move this forward relative the configured window.
2829 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2830 EXPECT_EQ(min_kernel_version, fake_hw->GetMaxKernelKeyRollforward());
2831}
2832
2833// Verifies that a device with rollback disabled sets max_kernel_rollforward
2834// in the TPM to logical infinity, to allow roll forward.
2835TEST_F(OmahaRequestActionTest, RollbackDisabledDevicesSetMaxRollback) {
2836 FakeHardware* fake_hw = fake_system_state_.fake_hardware();
2837
2838 // Setup and verify some initial default values for the kernel TPM
2839 // values that control verified boot and rollback.
2840 const int allowed_milestones = 0;
2841 const int min_kernel_version = 3;
2842 fake_hw->SetMinKernelKeyVersion(min_kernel_version);
2843 fake_hw->SetMaxKernelKeyRollforward(kRollforwardInfinity);
2844 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2845 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2846
2847 TestRollbackCheck(false /* is_consumer_device */,
2848 allowed_milestones,
2849 true /* is_policy_loaded */);
2850
2851 // Verify that with rollback disabled that max_kernel_rollforward
2852 // was set to logical infinity.
2853 EXPECT_EQ(min_kernel_version, fake_hw->GetMinKernelKeyVersion());
2854 EXPECT_EQ(kRollforwardInfinity, fake_hw->GetMaxKernelKeyRollforward());
2855}
2856
Darin Petkov6a5b3222010-07-13 14:55:28 -07002857} // namespace chromeos_update_engine