blob: 9a01a1324598a86911d2462a99011e9c699890e6 [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>
23#include <vector>
Darin Petkov0dc8e9a2010-07-14 14:51:57 -070024
Alex Deymo60ca1a72015-06-18 18:19:15 -070025#include <base/bind.h>
Alex Deymo110e0302015-10-19 20:35:21 -070026#include <base/files/file_util.h>
Sen Jiang297e5832016-03-17 14:45:51 -070027#include <base/files/scoped_temp_dir.h>
Ben Chan5c02c132017-06-27 07:10:36 -070028#include <base/memory/ptr_util.h>
Alex Deymo8e18f932015-03-27 16:16:59 -070029#include <base/strings/string_number_conversions.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070030#include <base/strings/string_util.h>
31#include <base/strings/stringprintf.h>
32#include <base/time/time.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070033#include <brillo/bind_lambda.h>
34#include <brillo/message_loops/fake_message_loop.h>
35#include <brillo/message_loops/message_loop.h>
36#include <brillo/message_loops/message_loop_utils.h>
Alex Deymoe1e3afe2014-10-30 13:02:49 -070037#include <gtest/gtest.h>
Jay Srinivasand29695d2013-04-08 15:08:05 -070038
Alex Deymo39910dc2015-11-09 17:04:30 -080039#include "update_engine/common/action_pipe.h"
40#include "update_engine/common/constants.h"
41#include "update_engine/common/fake_prefs.h"
42#include "update_engine/common/hash_calculator.h"
43#include "update_engine/common/mock_http_fetcher.h"
44#include "update_engine/common/platform_constants.h"
45#include "update_engine/common/prefs.h"
46#include "update_engine/common/test_utils.h"
Alex Deymoc1c17b42015-11-23 03:53:15 -030047#include "update_engine/fake_system_state.h"
Tianjie Xu282aa1f2017-09-05 13:42:45 -070048#include "update_engine/metrics_reporter_interface.h"
Chris Sosa77f79e82014-06-02 18:16:24 -070049#include "update_engine/mock_connection_manager.h"
Gilad Arnold74b5f552014-10-07 08:17:16 -070050#include "update_engine/mock_payload_state.h"
Darin Petkova4a8a8c2010-07-15 22:21:12 -070051#include "update_engine/omaha_request_params.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070052
Darin Petkov1cbd78f2010-07-29 12:38:34 -070053using base::Time;
54using base::TimeDelta;
Darin Petkov6a5b3222010-07-13 14:55:28 -070055using std::string;
56using std::vector;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070057using testing::AllOf;
Alex Deymof329b932014-10-30 01:37:48 -070058using testing::AnyNumber;
Jay Srinivasan34b5d862012-07-23 11:43:22 -070059using testing::DoAll;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070060using testing::Ge;
61using testing::Le;
Darin Petkov9c096d62010-11-17 14:49:04 -080062using testing::NiceMock;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070063using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070064using testing::ReturnPointee;
65using testing::SaveArg;
Ben Chan672c1f52017-10-23 15:41:39 -070066using testing::SetArgPointee;
Alex Deymof329b932014-10-30 01:37:48 -070067using testing::_;
Darin Petkov6a5b3222010-07-13 14:55:28 -070068
Alex Deymo8e18f932015-03-27 16:16:59 -070069namespace {
70
Alex Deymo85616652015-10-15 18:48:31 -070071const char kTestAppId[] = "test-app-id";
Aaron Wood7dcdedf2017-09-06 17:17:41 -070072const char kTestAppId2[] = "test-app2-id";
Alex Deymo85616652015-10-15 18:48:31 -070073
Alex Deymo8e18f932015-03-27 16:16:59 -070074// This is a helper struct to allow unit tests build an update response with the
75// values they care about.
76struct FakeUpdateResponse {
77 string GetNoUpdateResponse() const {
78 string entity_str;
79 if (include_entity)
80 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Sen Jiang81259682017-03-30 15:11:30 -070081 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" + entity_str +
82 "<response protocol=\"3.0\">"
83 "<daystart elapsed_seconds=\"100\"/>"
84 "<app appid=\"" +
85 app_id + "\" " +
86 (include_cohorts
87 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
88 "\" cohortname=\"" + cohortname + "\" "
89 : "") +
90 " status=\"ok\">"
91 "<ping status=\"ok\"/>"
92 "<updatecheck status=\"noupdate\"/></app>" +
93 (multi_app_no_update
Aaron Wood7dcdedf2017-09-06 17:17:41 -070094 ? "<app appid=\"" + app_id2 +
95 "\"><updatecheck status=\"noupdate\"/></app>"
Sen Jiang81259682017-03-30 15:11:30 -070096 : "") +
97 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -070098 }
99
100 string GetUpdateResponse() const {
Sen Jiang2703ef42017-03-16 13:36:21 -0700101 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
102 "protocol=\"3.0\">"
103 "<daystart elapsed_seconds=\"100\"" +
104 (elapsed_days.empty() ? ""
105 : (" elapsed_days=\"" + elapsed_days + "\"")) +
106 "/>"
107 "<app appid=\"" +
108 app_id + "\" " +
109 (include_cohorts
110 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
111 "\" cohortname=\"" + cohortname + "\" "
112 : "") +
113 " status=\"ok\">"
114 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
115 "<urls><url codebase=\"" +
116 codebase +
117 "\"/></urls>"
118 "<manifest version=\"" +
119 version +
120 "\">"
121 "<packages><package hash=\"not-used\" name=\"" +
122 filename + "\" size=\"" + base::Int64ToString(size) +
Sen Jiang0affc2c2017-02-10 15:55:05 -0800123 "\" hash_sha256=\"" + hash + "\"/>" +
124 (multi_package ? "<package name=\"package2\" size=\"222\" "
125 "hash_sha256=\"hash2\"/>"
126 : "") +
127 "</packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700128 "<actions><action event=\"postinstall\" MetadataSize=\"11" +
129 (multi_package ? ":22" : "") + "\" ChromeOSVersion=\"" + version +
130 "\" MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt +
Sen Jiang2703ef42017-03-16 13:36:21 -0700131 "\" "
132 "IsDelta=\"true\" "
Sen Jiangcdd52062017-05-18 15:33:10 -0700133 "IsDeltaPayload=\"true" +
134 (multi_package ? ":false" : "") +
135 "\" "
Sen Jiang2703ef42017-03-16 13:36:21 -0700136 "MaxDaysToScatter=\"" +
137 max_days_to_scatter +
138 "\" "
139 "sha256=\"not-used\" "
140 "needsadmin=\"" +
141 needsadmin + "\" " +
142 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
143 (disable_p2p_for_downloading ? "DisableP2PForDownloading=\"true\" "
144 : "") +
145 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
Sen Jiangfe284402018-03-21 14:03:50 -0700146 (powerwash ? "Powerwash=\"true\" " : "") +
Sen Jiang81259682017-03-30 15:11:30 -0700147 "/></actions></manifest></updatecheck></app>" +
148 (multi_app
Sen Jiangb1e063a2017-09-15 17:44:31 -0700149 ? "<app appid=\"" + app_id2 + "\"" +
150 (include_cohorts ? " cohort=\"cohort2\"" : "") +
151 "><updatecheck status=\"ok\"><urls><url codebase=\"" +
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700152 codebase2 + "\"/></urls><manifest version=\"" + version2 +
153 "\"><packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700154 "<package name=\"package3\" size=\"333\" "
155 "hash_sha256=\"hash3\"/></packages>"
Sen Jiang00adf7b2017-06-26 15:57:29 -0700156 "<actions><action event=\"postinstall\" " +
157 (multi_app_self_update
158 ? "noupdate=\"true\" IsDeltaPayload=\"true\" "
159 : "IsDeltaPayload=\"false\" ") +
160 "MetadataSize=\"33\"/></actions>"
Sen Jiang81259682017-03-30 15:11:30 -0700161 "</manifest></updatecheck></app>"
162 : "") +
163 (multi_app_no_update
164 ? "<app><updatecheck status=\"noupdate\"/></app>"
165 : "") +
166 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700167 }
168
169 // Return the payload URL, which is split in two fields in the XML response.
170 string GetPayloadUrl() {
171 return codebase + filename;
172 }
173
Alex Deymo85616652015-10-15 18:48:31 -0700174 string app_id = kTestAppId;
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700175 string app_id2 = kTestAppId2;
Alex Deymo8e18f932015-03-27 16:16:59 -0700176 string version = "1.2.3.4";
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700177 string version2 = "2.3.4.5";
Alex Deymo8e18f932015-03-27 16:16:59 -0700178 string more_info_url = "http://more/info";
179 string prompt = "true";
180 string codebase = "http://code/base/";
Sen Jiang81259682017-03-30 15:11:30 -0700181 string codebase2 = "http://code/base/2/";
Alex Deymo8e18f932015-03-27 16:16:59 -0700182 string filename = "file.signed";
Sen Jiang2703ef42017-03-16 13:36:21 -0700183 string hash = "4841534831323334";
Alex Deymo8e18f932015-03-27 16:16:59 -0700184 string needsadmin = "false";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800185 uint64_t size = 123;
Alex Deymo8e18f932015-03-27 16:16:59 -0700186 string deadline = "";
187 string max_days_to_scatter = "7";
188 string elapsed_days = "42";
189
190 // P2P setting defaults to allowed.
191 bool disable_p2p_for_downloading = false;
192 bool disable_p2p_for_sharing = false;
193
Sen Jiangfe284402018-03-21 14:03:50 -0700194 bool powerwash = false;
195
Alex Deymo8e18f932015-03-27 16:16:59 -0700196 // Omaha cohorts settings.
197 bool include_cohorts = false;
198 string cohort = "";
199 string cohorthint = "";
200 string cohortname = "";
201
202 // Whether to include the CrOS <!ENTITY> in the XML response.
203 bool include_entity = false;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800204
Sen Jiang81259682017-03-30 15:11:30 -0700205 // Whether to include more than one app.
206 bool multi_app = false;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700207 // Whether to include an app with noupdate="true".
208 bool multi_app_self_update = false;
209 // Whether to include an additional app with status="noupdate".
Sen Jiang81259682017-03-30 15:11:30 -0700210 bool multi_app_no_update = false;
211 // Whether to include more than one package in an app.
Sen Jiang0affc2c2017-02-10 15:55:05 -0800212 bool multi_package = false;
Alex Deymo8e18f932015-03-27 16:16:59 -0700213};
214
215} // namespace
216
Darin Petkov6a5b3222010-07-13 14:55:28 -0700217namespace chromeos_update_engine {
218
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700219class OmahaRequestActionTest : public ::testing::Test {
220 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800221 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700222 fake_system_state_.set_request_params(&request_params_);
223 fake_system_state_.set_prefs(&fake_prefs_);
224 }
225
226 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800227 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700228 // is used. |payload_state| may be null, in which case a local mock is used.
229 // |p2p_manager| may be null, in which case a local mock is used.
230 // |connection_manager| may be null, in which case a local mock is used.
231 // out_response may be null. If |fail_http_response_code| is non-negative,
232 // the transfer will fail with that code. |ping_only| is passed through to the
233 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
234 // post-data received by the mock HttpFetcher is returned.
235 //
236 // The |expected_check_result|, |expected_check_reaction| and
237 // |expected_error_code| parameters are for checking expectations
238 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
239 // UMA statistics. Use the appropriate ::kUnset value to specify that
240 // the given metric should not be reported.
241 bool TestUpdateCheck(OmahaRequestParams* request_params,
242 const string& http_response,
243 int fail_http_response_code,
244 bool ping_only,
245 ErrorCode expected_code,
246 metrics::CheckResult expected_check_result,
247 metrics::CheckReaction expected_check_reaction,
248 metrics::DownloadErrorCode expected_download_error_code,
249 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700250 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700251
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800252 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700253 // only a ping or also an updatecheck.
254 void PingTest(bool ping_only);
255
256 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700257 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700258 OmahaResponse *response);
259
260 // P2P test helper function.
261 void P2PTest(
262 bool initial_allow_p2p_for_downloading,
263 bool initial_allow_p2p_for_sharing,
264 bool omaha_disable_p2p_for_downloading,
265 bool omaha_disable_p2p_for_sharing,
266 bool payload_state_allow_p2p_attempt,
267 bool expect_p2p_client_lookup,
268 const string& p2p_client_result_url,
269 bool expected_allow_p2p_for_downloading,
270 bool expected_allow_p2p_for_sharing,
271 const string& expected_p2p_url);
272
273 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700274 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700275
276 // By default, all tests use these objects unless they replace them in the
277 // fake_system_state_.
278 OmahaRequestParams request_params_ = OmahaRequestParams{
279 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700280 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700281 OmahaRequestParams::kOsVersion,
282 "service_pack",
283 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700284 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700285 "0.1.0.0",
286 "en-US",
287 "unittest",
288 "OEM MODEL 09235 7471",
289 "ChromeOSFirmware.1.0",
290 "0X0A1",
291 false, // delta okay
292 false, // interactive
293 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700294 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700295
296 FakePrefs fake_prefs_;
297};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700298
299namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700300class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
301 public:
302 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700303 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800304 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700305 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800306 void ProcessingDone(const ActionProcessor* processor,
307 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700308 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700309 }
310
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800311 void ActionCompleted(ActionProcessor* processor,
312 AbstractAction* action,
313 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700314 // make sure actions always succeed
315 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700316 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700317 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700318 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700319 }
David Zeuthena99981f2013-04-29 13:42:47 -0700320 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700321};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700322} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700323
324class OutputObjectCollectorAction;
325
326template<>
327class ActionTraits<OutputObjectCollectorAction> {
328 public:
329 // Does not take an object for input
330 typedef OmahaResponse InputObjectType;
331 // On success, puts the output path on output
332 typedef NoneType OutputObjectType;
333};
334
335class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
336 public:
337 OutputObjectCollectorAction() : has_input_object_(false) {}
338 void PerformAction() {
339 // copy input object
340 has_input_object_ = HasInputObject();
341 if (has_input_object_)
342 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700343 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700344 }
345 // Should never be called
346 void TerminateProcessing() {
347 CHECK(false);
348 }
349 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700350 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700351 return "OutputObjectCollectorAction";
352 }
Alex Deymof329b932014-10-30 01:37:48 -0700353 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800354 using InputObjectType =
355 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
356 using OutputObjectType =
357 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700358 bool has_input_object_;
359 OmahaResponse omaha_response_;
360};
361
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700362bool OmahaRequestActionTest::TestUpdateCheck(
363 OmahaRequestParams* request_params,
364 const string& http_response,
365 int fail_http_response_code,
366 bool ping_only,
367 ErrorCode expected_code,
368 metrics::CheckResult expected_check_result,
369 metrics::CheckReaction expected_check_reaction,
370 metrics::DownloadErrorCode expected_download_error_code,
371 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700372 brillo::Blob* out_post_data) {
373 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700374 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700375 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800376 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700377 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700378 if (fail_http_response_code >= 0) {
379 fetcher->FailTransfer(fail_http_response_code);
380 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700381 if (request_params)
382 fake_system_state_.set_request_params(request_params);
383 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700384 nullptr,
Ben Chan5c02c132017-06-27 07:10:36 -0700385 base::WrapUnique(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700386 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700387 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700388 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700389
Darin Petkov6a5b3222010-07-13 14:55:28 -0700390 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700391 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700392 processor.EnqueueAction(&action);
393
394 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700395 BondActions(&action, &collector_action);
396 processor.EnqueueAction(&collector_action);
397
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700398 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
399 ReportUpdateCheckMetrics(_, _, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800400 .Times(AnyNumber());
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700401
402 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
403 ReportUpdateCheckMetrics(_,
404 expected_check_result,
405 expected_check_reaction,
406 expected_download_error_code))
407 .Times(ping_only ? 0 : 1);
David Zeuthen33bae492014-02-25 16:16:18 -0800408
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700409 loop.PostTask(base::Bind(
410 [](ActionProcessor* processor) { processor->StartProcessing(); },
411 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700412 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700413 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700414 if (collector_action.has_input_object_ && out_response)
415 *out_response = collector_action.omaha_response_;
416 if (out_post_data)
417 *out_post_data = fetcher->post_data();
418 return collector_action.has_input_object_;
419}
420
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700421// Tests Event requests -- they should always succeed. |out_post_data|
422// may be null; if non-null, the post-data received by the mock
423// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700424void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700425 OmahaEvent* event,
426 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700427 brillo::Blob* out_post_data) {
428 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700429 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700430 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800431 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700432 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700433 FakeSystemState fake_system_state;
434 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300435 OmahaRequestAction action(&fake_system_state,
436 event,
Ben Chan5c02c132017-06-27 07:10:36 -0700437 base::WrapUnique(fetcher),
Alex Deymoc1c17b42015-11-23 03:53:15 -0300438 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700439 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700440 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700441 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700442 processor.EnqueueAction(&action);
443
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700444 loop.PostTask(base::Bind(
445 [](ActionProcessor* processor) { processor->StartProcessing(); },
446 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700447 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700448 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700449
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700450 if (out_post_data)
451 *out_post_data = fetcher->post_data();
452}
453
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700454TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400455 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700456 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400457 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700458 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700459 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400460 -1,
461 false, // ping_only
462 ErrorCode::kOmahaRequestXMLHasEntityDecl,
463 metrics::CheckResult::kParsingError,
464 metrics::CheckReaction::kUnset,
465 metrics::DownloadErrorCode::kUnset,
466 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700467 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400468 EXPECT_FALSE(response.update_exists);
469}
470
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700471TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700472 OmahaResponse response;
473 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700474 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700475 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700476 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700477 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700478 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800479 metrics::CheckResult::kNoUpdateAvailable,
480 metrics::CheckReaction::kUnset,
481 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700482 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700483 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700484 EXPECT_FALSE(response.update_exists);
485}
486
Sen Jiang81259682017-03-30 15:11:30 -0700487TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
488 OmahaResponse response;
489 fake_update_response_.multi_app_no_update = true;
490 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
491 fake_update_response_.GetNoUpdateResponse(),
492 -1,
493 false, // ping_only
494 ErrorCode::kSuccess,
495 metrics::CheckResult::kNoUpdateAvailable,
496 metrics::CheckReaction::kUnset,
497 metrics::DownloadErrorCode::kUnset,
498 &response,
499 nullptr));
500 EXPECT_FALSE(response.update_exists);
501}
502
Sen Jiang00adf7b2017-06-26 15:57:29 -0700503TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
504 OmahaResponse response;
505 fake_update_response_.multi_app_no_update = true;
506 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
507 fake_update_response_.GetUpdateResponse(),
508 -1,
509 false, // ping_only
510 ErrorCode::kSuccess,
511 metrics::CheckResult::kNoUpdateAvailable,
512 metrics::CheckReaction::kUnset,
513 metrics::DownloadErrorCode::kUnset,
514 &response,
515 nullptr));
516 EXPECT_FALSE(response.update_exists);
517}
518
519TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
520 OmahaResponse response;
521 ASSERT_TRUE(TestUpdateCheck(
522 nullptr, // request_params
523 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
524 "event=\"postinstall\" noupdate=\"true\"/></actions>"
525 "</manifest></updatecheck></app></response>",
526 -1,
527 false, // ping_only
528 ErrorCode::kSuccess,
529 metrics::CheckResult::kNoUpdateAvailable,
530 metrics::CheckReaction::kUnset,
531 metrics::DownloadErrorCode::kUnset,
532 &response,
533 nullptr));
534 EXPECT_FALSE(response.update_exists);
535}
536
Alex Deymo8e18f932015-03-27 16:16:59 -0700537// Test that all the values in the response are parsed in a normal update
538// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700539TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700540 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700541 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700542 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700543 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700544 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700545 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700546 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700547 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800548 metrics::CheckResult::kUpdateAvailable,
549 metrics::CheckReaction::kUpdating,
550 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700551 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700552 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700553 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700554 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700555 EXPECT_EQ("", response.system_version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800556 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
557 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700558 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800559 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
560 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700561 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700562 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
563 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
Sen Jiangfe284402018-03-21 14:03:50 -0700564 EXPECT_FALSE(response.powerwash_required);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700565 // Omaha cohort attributes are not set in the response, so they should not be
Alex Deymo8e18f932015-03-27 16:16:59 -0700566 // persisted.
567 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
568 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
569 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700570}
571
Sen Jiang0affc2c2017-02-10 15:55:05 -0800572TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
573 OmahaResponse response;
574 fake_update_response_.multi_package = true;
575 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
576 fake_update_response_.GetUpdateResponse(),
577 -1,
578 false, // ping_only
579 ErrorCode::kSuccess,
580 metrics::CheckResult::kUpdateAvailable,
581 metrics::CheckReaction::kUpdating,
582 metrics::DownloadErrorCode::kUnset,
583 &response,
584 nullptr));
585 EXPECT_TRUE(response.update_exists);
586 EXPECT_EQ(fake_update_response_.version, response.version);
587 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
588 response.packages[0].payload_urls[0]);
589 EXPECT_EQ(fake_update_response_.codebase + "package2",
590 response.packages[1].payload_urls[0]);
591 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
592 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700593 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700594 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800595 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700596 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800597 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700598 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700599 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700600}
601
602TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
603 OmahaResponse response;
604 fake_update_response_.multi_app = true;
605 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
606 fake_update_response_.GetUpdateResponse(),
607 -1,
608 false, // ping_only
609 ErrorCode::kSuccess,
610 metrics::CheckResult::kUpdateAvailable,
611 metrics::CheckReaction::kUpdating,
612 metrics::DownloadErrorCode::kUnset,
613 &response,
614 nullptr));
615 EXPECT_TRUE(response.update_exists);
616 EXPECT_EQ(fake_update_response_.version, response.version);
617 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
618 response.packages[0].payload_urls[0]);
619 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
620 response.packages[1].payload_urls[0]);
621 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
622 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
623 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700624 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700625 ASSERT_EQ(2u, response.packages.size());
626 EXPECT_EQ(string("hash3"), response.packages[1].hash);
627 EXPECT_EQ(333u, response.packages[1].size);
628 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700629 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700630}
631
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700632TEST_F(OmahaRequestActionTest, MultiAppAndSystemUpdateTest) {
633 OmahaResponse response;
634 fake_update_response_.multi_app = true;
635 // trigger the lining up of the app and system versions
636 request_params_.set_system_app_id(fake_update_response_.app_id2);
637
638 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
639 fake_update_response_.GetUpdateResponse(),
640 -1,
641 false, // ping_only
642 ErrorCode::kSuccess,
643 metrics::CheckResult::kUpdateAvailable,
644 metrics::CheckReaction::kUpdating,
645 metrics::DownloadErrorCode::kUnset,
646 &response,
647 nullptr));
648 EXPECT_TRUE(response.update_exists);
649 EXPECT_EQ(fake_update_response_.version, response.version);
650 EXPECT_EQ(fake_update_response_.version2, response.system_version);
651 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
652 response.packages[0].payload_urls[0]);
653 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
654 response.packages[1].payload_urls[0]);
655 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
656 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
657 EXPECT_EQ(11u, response.packages[0].metadata_size);
658 EXPECT_EQ(true, response.packages[0].is_delta);
659 ASSERT_EQ(2u, response.packages.size());
660 EXPECT_EQ(string("hash3"), response.packages[1].hash);
661 EXPECT_EQ(333u, response.packages[1].size);
662 EXPECT_EQ(33u, response.packages[1].metadata_size);
663 EXPECT_EQ(false, response.packages[1].is_delta);
664}
665
Sen Jiang81259682017-03-30 15:11:30 -0700666TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
667 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700668 fake_update_response_.multi_app = true;
669 fake_update_response_.multi_app_self_update = true;
Sen Jiang81259682017-03-30 15:11:30 -0700670 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
671 fake_update_response_.GetUpdateResponse(),
672 -1,
673 false, // ping_only
674 ErrorCode::kSuccess,
675 metrics::CheckResult::kUpdateAvailable,
676 metrics::CheckReaction::kUpdating,
677 metrics::DownloadErrorCode::kUnset,
678 &response,
679 nullptr));
680 EXPECT_TRUE(response.update_exists);
681 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700682 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700683 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
684 response.packages[0].payload_urls[0]);
685 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
686 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
687 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700688 ASSERT_EQ(2u, response.packages.size());
689 EXPECT_EQ(string("hash3"), response.packages[1].hash);
690 EXPECT_EQ(333u, response.packages[1].size);
691 EXPECT_EQ(33u, response.packages[1].metadata_size);
692 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700693}
694
695TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
696 OmahaResponse response;
697 fake_update_response_.multi_app = true;
698 fake_update_response_.multi_package = true;
699 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
700 fake_update_response_.GetUpdateResponse(),
701 -1,
702 false, // ping_only
703 ErrorCode::kSuccess,
704 metrics::CheckResult::kUpdateAvailable,
705 metrics::CheckReaction::kUpdating,
706 metrics::DownloadErrorCode::kUnset,
707 &response,
708 nullptr));
709 EXPECT_TRUE(response.update_exists);
710 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700711 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700712 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
713 response.packages[0].payload_urls[0]);
714 EXPECT_EQ(fake_update_response_.codebase + "package2",
715 response.packages[1].payload_urls[0]);
716 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
717 response.packages[2].payload_urls[0]);
718 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
719 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
720 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700721 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700722 ASSERT_EQ(3u, response.packages.size());
723 EXPECT_EQ(string("hash2"), response.packages[1].hash);
724 EXPECT_EQ(222u, response.packages[1].size);
725 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700726 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700727 EXPECT_EQ(string("hash3"), response.packages[2].hash);
728 EXPECT_EQ(333u, response.packages[2].size);
729 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700730 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800731}
732
Sen Jiangfe284402018-03-21 14:03:50 -0700733TEST_F(OmahaRequestActionTest, PowerwashTest) {
734 OmahaResponse response;
735 fake_update_response_.powerwash = true;
736 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
737 fake_update_response_.GetUpdateResponse(),
738 -1,
739 false, // ping_only
740 ErrorCode::kSuccess,
741 metrics::CheckResult::kUpdateAvailable,
742 metrics::CheckReaction::kUpdating,
743 metrics::DownloadErrorCode::kUnset,
744 &response,
745 nullptr));
746 EXPECT_TRUE(response.update_exists);
747 EXPECT_TRUE(response.powerwash_required);
748}
749
Alex Deymo14ad88e2016-06-29 12:30:14 -0700750TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
751 const string http_response = "<?xml invalid response";
752 request_params_.set_interactive(true);
753
754 brillo::FakeMessageLoop loop(nullptr);
755 loop.SetAsCurrent();
756
757 MockHttpFetcher* fetcher =
758 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
Ben Chan5c02c132017-06-27 07:10:36 -0700759 OmahaRequestAction action(&fake_system_state_, nullptr,
760 base::WrapUnique(fetcher), false);
Alex Deymo14ad88e2016-06-29 12:30:14 -0700761 ActionProcessor processor;
762 processor.EnqueueAction(&action);
763
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700764 loop.PostTask(base::Bind(
765 [](ActionProcessor* processor) { processor->StartProcessing(); },
766 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700767 loop.Run();
768 EXPECT_FALSE(loop.PendingTasks());
769
770 // Check that the headers were set in the fetcher during the action. Note that
771 // we set this request as "interactive".
Sen Jiang42fa45e2018-03-12 11:02:14 -0700772 EXPECT_EQ("fg", fetcher->GetHeader("X-Goog-Update-Interactivity"));
773 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-Goog-Update-AppId"));
774 EXPECT_NE("", fetcher->GetHeader("X-Goog-Update-Updater"));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700775}
776
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700777TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700778 OmahaResponse response;
779 // Set up a connection manager that doesn't allow a valid update over
780 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700781 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700782 fake_system_state_.set_connection_manager(&mock_cm);
783
Alex Deymo30534502015-07-20 15:06:33 -0700784 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
785 .WillRepeatedly(
Ben Chan672c1f52017-10-23 15:41:39 -0700786 DoAll(SetArgPointee<0>(ConnectionType::kEthernet),
787 SetArgPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700788 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700789 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
790 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700791
792 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700793 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700794 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700795 -1,
796 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700797 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700798 metrics::CheckResult::kUpdateAvailable,
799 metrics::CheckReaction::kIgnored,
800 metrics::DownloadErrorCode::kUnset,
801 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700802 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700803 EXPECT_FALSE(response.update_exists);
804}
805
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700806TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700807 string rollback_version = "1234.0.0";
808 OmahaResponse response;
809
810 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700811 fake_system_state_.set_payload_state(&mock_payload_state);
812
Chris Sosa77f79e82014-06-02 18:16:24 -0700813 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
814 .WillRepeatedly(Return(rollback_version));
815
Alex Deymo8e18f932015-03-27 16:16:59 -0700816 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700817 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700818 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700819 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700820 -1,
821 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700822 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700823 metrics::CheckResult::kUpdateAvailable,
824 metrics::CheckReaction::kIgnored,
825 metrics::DownloadErrorCode::kUnset,
826 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700827 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700828 EXPECT_FALSE(response.update_exists);
829}
830
Kevin Cernekee2494e282016-03-29 18:03:53 -0700831// Verify that update checks called during OOBE will only try to download
832// an update if the response includes a non-empty deadline field.
833TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
834 OmahaResponse response;
835
Sen Jiang8cd42342018-01-31 12:06:59 -0800836 // TODO(senj): set better default value for metrics::checkresult in
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700837 // OmahaRequestAction::ActionCompleted.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700838 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700839 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
840 fake_update_response_.GetUpdateResponse(),
841 -1,
842 false, // ping_only
843 ErrorCode::kNonCriticalUpdateInOOBE,
844 metrics::CheckResult::kParsingError,
845 metrics::CheckReaction::kUnset,
846 metrics::DownloadErrorCode::kUnset,
847 &response,
848 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -0700849 EXPECT_FALSE(response.update_exists);
850
Alex Deymo46a9aae2016-05-04 20:20:11 -0700851 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
852 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
853 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
854 fake_update_response_.GetUpdateResponse(),
855 -1,
856 false, // ping_only
857 ErrorCode::kSuccess,
858 metrics::CheckResult::kUpdateAvailable,
859 metrics::CheckReaction::kUpdating,
860 metrics::DownloadErrorCode::kUnset,
861 &response,
862 nullptr));
863 EXPECT_TRUE(response.update_exists);
864 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
865
866 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700867 fake_update_response_.deadline = "20101020";
868 ASSERT_TRUE(
869 TestUpdateCheck(nullptr, // request_params
870 fake_update_response_.GetUpdateResponse(),
871 -1,
872 false, // ping_only
873 ErrorCode::kSuccess,
874 metrics::CheckResult::kUpdateAvailable,
875 metrics::CheckReaction::kUpdating,
876 metrics::DownloadErrorCode::kUnset,
877 &response,
878 nullptr));
879 EXPECT_TRUE(response.update_exists);
880}
881
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700882TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700883 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700884 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700885 params.set_wall_clock_based_wait_enabled(true);
886 params.set_update_check_count_wait_enabled(false);
887 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700888
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700889 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700890 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700891 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700892 -1,
893 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700894 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800895 metrics::CheckResult::kUpdateAvailable,
896 metrics::CheckReaction::kDeferring,
897 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700898 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700899 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700900 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700901
902 // Verify if we are interactive check we don't defer.
903 params.set_interactive(true);
904 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700905 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700906 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700907 -1,
908 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700909 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800910 metrics::CheckResult::kUpdateAvailable,
911 metrics::CheckReaction::kUpdating,
912 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700913 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700914 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700915 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700916}
917
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700918TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700919 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700920 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700921 params.set_wall_clock_based_wait_enabled(false);
922 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700923
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700924 params.set_update_check_count_wait_enabled(true);
925 params.set_min_update_checks_needed(1);
926 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700927
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700928 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700929 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700930 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700931 -1,
932 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700933 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800934 metrics::CheckResult::kUpdateAvailable,
935 metrics::CheckReaction::kUpdating,
936 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700937 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700938 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700939 EXPECT_TRUE(response.update_exists);
940}
941
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700942TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700943 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700944 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700945 params.set_wall_clock_based_wait_enabled(true);
946 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700947
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700948 params.set_update_check_count_wait_enabled(true);
949 params.set_min_update_checks_needed(1);
950 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700951
Alex Deymo8e18f932015-03-27 16:16:59 -0700952 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700953 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700954 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700955 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700956 -1,
957 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700958 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800959 metrics::CheckResult::kUpdateAvailable,
960 metrics::CheckReaction::kUpdating,
961 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700962 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700963 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700964 EXPECT_TRUE(response.update_exists);
965}
966
967
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700968TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700969 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700970 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700971 params.set_wall_clock_based_wait_enabled(true);
972 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700973
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700974 params.set_update_check_count_wait_enabled(true);
975 params.set_min_update_checks_needed(0);
976 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700977
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700978 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700979 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700980 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700981 -1,
982 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700983 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800984 metrics::CheckResult::kUpdateAvailable,
985 metrics::CheckReaction::kUpdating,
986 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700987 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700988 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700989
Ben Chan9abb7632014-08-07 00:10:53 -0700990 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700991 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700992 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700993 EXPECT_TRUE(response.update_exists);
994}
995
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700996TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700997 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700998 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700999 params.set_wall_clock_based_wait_enabled(true);
1000 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001001
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001002 params.set_update_check_count_wait_enabled(true);
1003 params.set_min_update_checks_needed(1);
1004 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001005
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001006 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001007 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001008 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001009 -1,
1010 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001011 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001012 metrics::CheckResult::kUpdateAvailable,
1013 metrics::CheckReaction::kDeferring,
1014 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001015 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001016 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001017
Ben Chan9abb7632014-08-07 00:10:53 -07001018 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001019 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001020 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001021 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001022
1023 // Verify if we are interactive check we don't defer.
1024 params.set_interactive(true);
1025 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001026 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001027 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001028 -1,
1029 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001030 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001031 metrics::CheckResult::kUpdateAvailable,
1032 metrics::CheckReaction::kUpdating,
1033 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001034 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001035 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001036 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001037}
1038
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001039TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001040 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001041 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001042 params.set_wall_clock_based_wait_enabled(true);
1043 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001044
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001045 params.set_update_check_count_wait_enabled(true);
1046 params.set_min_update_checks_needed(1);
1047 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001048
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001049 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001050
1051 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001052 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001053 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001054 -1,
1055 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001056 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001057 metrics::CheckResult::kUpdateAvailable,
1058 metrics::CheckReaction::kDeferring,
1059 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001060 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001061 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001062
Ben Chan9abb7632014-08-07 00:10:53 -07001063 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001064 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001065 // count remains the same, as the decrementing happens in update_attempter
1066 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001067 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001068 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001069
1070 // Verify if we are interactive check we don't defer.
1071 params.set_interactive(true);
1072 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001073 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001074 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001075 -1,
1076 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001077 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001078 metrics::CheckResult::kUpdateAvailable,
1079 metrics::CheckReaction::kUpdating,
1080 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001081 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001082 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001083 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001084}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001085
Alex Deymo8e18f932015-03-27 16:16:59 -07001086TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1087 OmahaResponse response;
1088 OmahaRequestParams params = request_params_;
1089 fake_update_response_.include_cohorts = true;
1090 fake_update_response_.cohort = "s/154454/8479665";
1091 fake_update_response_.cohorthint = "please-put-me-on-beta";
1092 fake_update_response_.cohortname = "stable";
1093
1094 ASSERT_TRUE(TestUpdateCheck(&params,
1095 fake_update_response_.GetUpdateResponse(),
1096 -1,
1097 false, // ping_only
1098 ErrorCode::kSuccess,
1099 metrics::CheckResult::kUpdateAvailable,
1100 metrics::CheckReaction::kUpdating,
1101 metrics::DownloadErrorCode::kUnset,
1102 &response,
1103 nullptr));
1104
1105 string value;
1106 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1107 EXPECT_EQ(fake_update_response_.cohort, value);
1108
1109 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1110 EXPECT_EQ(fake_update_response_.cohorthint, value);
1111
1112 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1113 EXPECT_EQ(fake_update_response_.cohortname, value);
1114}
1115
1116TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1117 OmahaResponse response;
1118 OmahaRequestParams params = request_params_;
1119 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1120 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1121 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1122 fake_update_response_.include_cohorts = true;
1123 fake_update_response_.cohort = "s/154454/8479665";
1124 fake_update_response_.cohorthint = "please-put-me-on-beta";
1125 fake_update_response_.cohortname = "";
1126
1127 ASSERT_TRUE(TestUpdateCheck(&params,
1128 fake_update_response_.GetUpdateResponse(),
1129 -1,
1130 false, // ping_only
1131 ErrorCode::kSuccess,
1132 metrics::CheckResult::kUpdateAvailable,
1133 metrics::CheckReaction::kUpdating,
1134 metrics::DownloadErrorCode::kUnset,
1135 &response,
1136 nullptr));
1137
1138 string value;
1139 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1140 EXPECT_EQ(fake_update_response_.cohort, value);
1141
1142 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1143 EXPECT_EQ(fake_update_response_.cohorthint, value);
1144
1145 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1146}
1147
1148TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1149 OmahaResponse response;
1150 OmahaRequestParams params = request_params_;
1151 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1152
1153 ASSERT_TRUE(TestUpdateCheck(&params,
1154 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));
1163
1164 string value;
1165 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1166 EXPECT_EQ("old_value", value);
1167
1168 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1169 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1170}
1171
Alex Deymo00d79ac2015-06-29 15:41:49 -07001172TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1173 OmahaResponse response;
1174 OmahaRequestParams params = request_params_;
1175 fake_update_response_.include_cohorts = true;
1176 fake_update_response_.cohort = "s/154454/8479665";
1177 fake_update_response_.cohorthint = "please-put-me-on-beta";
1178 fake_update_response_.cohortname = "stable";
1179
1180 ASSERT_TRUE(TestUpdateCheck(&params,
1181 fake_update_response_.GetNoUpdateResponse(),
1182 -1,
1183 false, // ping_only
1184 ErrorCode::kSuccess,
1185 metrics::CheckResult::kNoUpdateAvailable,
1186 metrics::CheckReaction::kUnset,
1187 metrics::DownloadErrorCode::kUnset,
1188 &response,
1189 nullptr));
1190
1191 string value;
1192 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1193 EXPECT_EQ(fake_update_response_.cohort, value);
1194
1195 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1196 EXPECT_EQ(fake_update_response_.cohorthint, value);
1197
1198 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1199 EXPECT_EQ(fake_update_response_.cohortname, value);
1200}
1201
Sen Jiangb1e063a2017-09-15 17:44:31 -07001202TEST_F(OmahaRequestActionTest, MultiAppCohortTest) {
1203 OmahaResponse response;
1204 OmahaRequestParams params = request_params_;
1205 fake_update_response_.multi_app = true;
1206 fake_update_response_.include_cohorts = true;
1207 fake_update_response_.cohort = "s/154454/8479665";
1208 fake_update_response_.cohorthint = "please-put-me-on-beta";
1209 fake_update_response_.cohortname = "stable";
1210
1211 ASSERT_TRUE(TestUpdateCheck(&params,
1212 fake_update_response_.GetUpdateResponse(),
1213 -1,
1214 false, // ping_only
1215 ErrorCode::kSuccess,
1216 metrics::CheckResult::kUpdateAvailable,
1217 metrics::CheckReaction::kUpdating,
1218 metrics::DownloadErrorCode::kUnset,
1219 &response,
1220 nullptr));
1221
1222 string value;
1223 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1224 EXPECT_EQ(fake_update_response_.cohort, value);
1225
1226 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1227 EXPECT_EQ(fake_update_response_.cohorthint, value);
1228
1229 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1230 EXPECT_EQ(fake_update_response_.cohortname, value);
1231}
1232
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001233TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001234 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001235
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001236 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001237 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001238
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001239 OmahaRequestParams params = request_params_;
1240 fake_system_state_.set_request_params(&params);
Ben Chan5c02c132017-06-27 07:10:36 -07001241 OmahaRequestAction action(
1242 &fake_system_state_,
1243 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001244 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001245 http_response.size(),
1246 nullptr),
1247 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001248 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001249 ActionProcessor processor;
1250 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001251 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001252
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001253 loop.PostTask(base::Bind(
1254 [](ActionProcessor* processor) { processor->StartProcessing(); },
1255 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001256 loop.Run();
1257 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001258 EXPECT_FALSE(processor.IsRunning());
1259}
1260
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001261TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001262 OmahaResponse response;
1263 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001264 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001265 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001266 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001267 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001268 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001269 metrics::CheckResult::kParsingError,
1270 metrics::CheckReaction::kUnset,
1271 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001272 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001273 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001274 EXPECT_FALSE(response.update_exists);
1275}
1276
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001277TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001278 OmahaResponse response;
1279 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001280 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001281 "",
1282 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001283 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001284 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001285 metrics::CheckResult::kParsingError,
1286 metrics::CheckReaction::kUnset,
1287 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001288 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001289 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001290 EXPECT_FALSE(response.update_exists);
1291}
1292
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001293TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001294 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001295 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001296 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001297 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1298 "<daystart elapsed_seconds=\"100\"/>"
1299 "<app appid=\"foo\" status=\"ok\">"
1300 "<ping status=\"ok\"/>"
1301 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001302 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001303 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001304 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001305 metrics::CheckResult::kParsingError,
1306 metrics::CheckReaction::kUnset,
1307 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001308 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001309 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001310 EXPECT_FALSE(response.update_exists);
1311}
1312
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001313TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001314 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001315 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001316 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001317 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1318 "<daystart elapsed_seconds=\"100\"/>"
1319 "<app appid=\"foo\" status=\"ok\">"
1320 "<ping status=\"ok\"/>"
1321 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001322 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001323 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001324 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001325 metrics::CheckResult::kParsingError,
1326 metrics::CheckReaction::kUnset,
1327 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001328 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001329 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001330 EXPECT_FALSE(response.update_exists);
1331}
1332
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001333TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001334 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001335 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001336 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001337 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1338 "<daystart elapsed_seconds=\"100\"/>"
1339 "<app appid=\"foo\" status=\"ok\">"
1340 "<ping status=\"ok\"/>"
1341 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001342 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001343 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001344 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001345 metrics::CheckResult::kParsingError,
1346 metrics::CheckReaction::kUnset,
1347 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001348 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001349 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001350 EXPECT_FALSE(response.update_exists);
1351}
1352
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001353TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001354 string input_response =
1355 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1356 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001357 // the appid needs to match that in the request params
1358 "<app appid=\"" +
1359 fake_update_response_.app_id +
1360 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001361 "<updatecheck status=\"ok\">"
1362 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001363 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001364 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001365 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001366 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001367 "ChromeOSVersion=\"10.2.3.4\" "
1368 "Prompt=\"false\" "
1369 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001370 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001371 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001372 "needsadmin=\"true\" "
1373 "/></actions></manifest></updatecheck></app></response>";
1374 LOG(INFO) << "Input Response = " << input_response;
1375
Darin Petkov6a5b3222010-07-13 14:55:28 -07001376 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001377 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001378 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001379 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001380 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001381 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001382 metrics::CheckResult::kUpdateAvailable,
1383 metrics::CheckReaction::kUpdating,
1384 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001385 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001386 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001387 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001388 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001389 EXPECT_EQ("http://missing/field/test/f",
1390 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001391 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001392 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1393 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001394 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001395 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001396}
1397
1398namespace {
1399class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1400 public:
1401 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001402 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001403 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001404};
1405
Alex Deymo60ca1a72015-06-18 18:19:15 -07001406void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001407 processor->StartProcessing();
1408 CHECK(processor->IsRunning());
1409 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001410}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001411} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001412
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001413TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001414 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001415 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001416
Alex Deymo60ca1a72015-06-18 18:19:15 -07001417 string http_response("doesn't matter");
Ben Chan5c02c132017-06-27 07:10:36 -07001418 OmahaRequestAction action(
1419 &fake_system_state_,
1420 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001421 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001422 http_response.size(),
1423 nullptr),
1424 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001425 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001426 ActionProcessor processor;
1427 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001428 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001429
Alex Deymo60ca1a72015-06-18 18:19:15 -07001430 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1431 loop.Run();
1432 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001433}
1434
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001435TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001436 string output;
1437 EXPECT_TRUE(XmlEncode("ab", &output));
1438 EXPECT_EQ("ab", output);
1439 EXPECT_TRUE(XmlEncode("a<b", &output));
1440 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001441 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1442 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001443 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1444 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001445 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001446 EXPECT_FALSE(XmlEncode("\xc2", &output));
1447 // Fail with invalid ASCII-7 chars.
1448 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1449}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001450
Alex Deymob0d74eb2015-03-30 17:59:17 -07001451TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1452 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1453 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1454}
1455
1456TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001457 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001458
1459 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001460 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001461 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001462 OmahaRequestParams::kOsVersion,
1463 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001464 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001465 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001466 "0.1.0.0",
1467 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001468 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001469 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001470 "ChromeOSFirmware.1.0",
1471 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001472 false, // delta okay
1473 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001474 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001475 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001476 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1477 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1478 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001479 base::JoinString(
1480 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001481 OmahaResponse response;
1482 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001483 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001484 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001485 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001486 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001487 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001488 metrics::CheckResult::kParsingError,
1489 metrics::CheckReaction::kUnset,
1490 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001491 &response,
1492 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001493 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001494 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001495 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1496 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1497 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1498 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1499 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1500 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1501 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1502 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1503 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1504 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1505 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1506 // Values from Prefs that are too big are removed from the XML instead of
1507 // encoded.
1508 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001509}
1510
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001511TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001512 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001513 fake_update_response_.deadline = "&lt;20110101";
1514 fake_update_response_.more_info_url = "testthe&lt;url";
1515 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001516 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001517 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001518 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001519 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001520 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001521 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001522 metrics::CheckResult::kUpdateAvailable,
1523 metrics::CheckReaction::kUpdating,
1524 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001525 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001526 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001527
Sen Jiang0affc2c2017-02-10 15:55:05 -08001528 EXPECT_EQ("testthe<url", response.more_info_url);
1529 EXPECT_EQ("testthe&codebase/file.signed",
1530 response.packages[0].payload_urls[0]);
1531 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001532}
1533
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001534TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001535 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001536 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001537 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001538 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001539 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001540 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001541 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001542 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001543 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001544 metrics::CheckResult::kUpdateAvailable,
1545 metrics::CheckReaction::kUpdating,
1546 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001547 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001548 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001549
Sen Jiang0affc2c2017-02-10 15:55:05 -08001550 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001551}
1552
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001553TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001554 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001555 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001556 fake_system_state_.set_prefs(&prefs);
1557
Darin Petkov95508da2011-01-05 12:42:29 -08001558 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001559 .WillOnce(DoAll(SetArgPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001560 // An existing but empty previous version means that we didn't reboot to a new
1561 // update, therefore, no need to update the previous version.
1562 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001563 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001564 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001565 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001566 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001567 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001568 metrics::CheckResult::kParsingError,
1569 metrics::CheckReaction::kUnset,
1570 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001571 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001572 &post_data));
1573 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001574 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001575 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001576 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001577 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001578 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001579 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1580 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001581 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1582 string::npos);
1583 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1584 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001585 // No <event> tag should be sent if we didn't reboot to an update.
1586 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001587}
1588
Jay Srinivasan0a708742012-03-20 11:26:12 -07001589
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001590TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001591 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001592 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001593 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1594 "invalid xml>",
1595 &post_data);
1596 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001597 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001598 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001599 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001600 OmahaEvent::kTypeUpdateDownloadStarted,
1601 OmahaEvent::kResultSuccess);
1602 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001603 EXPECT_EQ(post_str.find("ping"), string::npos);
1604 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001605}
1606
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001607TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001608 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001609 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001610 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1611 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001612 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001613 "invalid xml>",
1614 &post_data);
1615 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001616 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001617 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001618 " <event eventtype=\"%d\" eventresult=\"%d\" "
1619 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001620 OmahaEvent::kTypeDownloadComplete,
1621 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001622 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001623 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001624 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001625}
1626
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001627TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001628 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001629 // Create a copy of the OmahaRequestParams to reuse it later.
1630 OmahaRequestParams params = request_params_;
1631 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001632 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001633 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001634 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001635 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001636 http_response.size(),
1637 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001638 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001639 EXPECT_FALSE(update_check_action.IsEvent());
1640
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001641 params = request_params_;
1642 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001643 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001644 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001645 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Ben Chanab5a0af2017-10-12 14:57:50 -07001646 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001647 http_response.size(),
1648 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001649 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001650 EXPECT_TRUE(event_action.IsEvent());
1651}
1652
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001653TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001654 for (int i = 0; i < 2; i++) {
1655 bool delta_okay = i == 1;
1656 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001657 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001658 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001659 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001660 OmahaRequestParams::kOsVersion,
1661 "service_pack",
1662 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001663 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001664 "0.1.0.0",
1665 "en-US",
1666 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001667 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001668 "ChromeOSFirmware.1.0",
1669 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001670 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001671 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001672 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001673 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001674 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001675 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001676 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001677 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001678 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001679 metrics::CheckResult::kParsingError,
1680 metrics::CheckReaction::kUnset,
1681 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001682 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001683 &post_data));
1684 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001685 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001686 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1687 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001688 string::npos)
1689 << "i = " << i;
1690 }
1691}
1692
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001693TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001694 for (int i = 0; i < 2; i++) {
1695 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001696 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001697 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001698 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001699 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001700 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001701 OmahaRequestParams::kOsVersion,
1702 "service_pack",
1703 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001704 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001705 "0.1.0.0",
1706 "en-US",
1707 "unittest_track",
1708 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001709 "ChromeOSFirmware.1.0",
1710 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001711 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001712 interactive,
1713 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001714 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001715 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001716 "invalid xml>",
1717 -1,
1718 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001719 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001720 metrics::CheckResult::kParsingError,
1721 metrics::CheckReaction::kUnset,
1722 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001723 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001724 &post_data));
1725 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001726 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001727 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1728 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001729 string::npos)
1730 << "i = " << i;
1731 }
1732}
1733
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001734TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001735 OmahaEvent default_event;
1736 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1737 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001738 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001739
1740 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1741 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1742 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001743 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001744
1745 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1746 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001747 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001748 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1749 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001750 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001751}
1752
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001753void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001754 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001755 fake_system_state_.set_prefs(&prefs);
1756 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1757 .Times(AnyNumber());
1758 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1759 // Add a few hours to the day difference to test no rounding, etc.
1760 int64_t five_days_ago =
1761 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1762 int64_t six_days_ago =
1763 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1764 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001765 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001766 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001767 .WillOnce(DoAll(SetArgPointee<1>(six_days_ago), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001768 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001769 .WillOnce(DoAll(SetArgPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001770 brillo::Blob post_data;
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001771 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
1772 fake_update_response_.GetNoUpdateResponse(),
1773 -1,
1774 ping_only,
1775 ErrorCode::kSuccess,
1776 metrics::CheckResult::kNoUpdateAvailable,
1777 metrics::CheckReaction::kUnset,
1778 metrics::DownloadErrorCode::kUnset,
1779 nullptr,
1780 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001781 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001782 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1783 string::npos);
1784 if (ping_only) {
1785 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1786 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1787 } else {
1788 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1789 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001790 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001791}
1792
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001793TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1794 PingTest(true /* ping_only */);
1795}
1796
1797TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1798 PingTest(false /* ping_only */);
1799}
1800
1801TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001802 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001803 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001804 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1805 .Times(AnyNumber());
1806 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001807 int64_t three_days_ago =
1808 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1809 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001810 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001811 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001812 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001813 .WillOnce(DoAll(SetArgPointee<1>(three_days_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001814 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001815 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001816 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001817 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001818 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001819 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001820 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001821 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001822 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001823 metrics::CheckResult::kNoUpdateAvailable,
1824 metrics::CheckReaction::kUnset,
1825 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001826 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001827 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001828 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001829 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001830 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001831}
1832
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001833TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001834 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001835 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001836 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1837 .Times(AnyNumber());
1838 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001839 int64_t four_days_ago =
1840 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1841 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001842 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001843 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001844 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001845 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001846 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001847 .WillOnce(DoAll(SetArgPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001848 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001849 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001850 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001851 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001852 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001853 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001854 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001855 metrics::CheckResult::kNoUpdateAvailable,
1856 metrics::CheckReaction::kUnset,
1857 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001858 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001859 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001860 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001861 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001862 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001863}
1864
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001865TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001866 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001867 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001868 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1869 .Times(AnyNumber());
1870 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001871 int64_t one_hour_ago =
1872 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001873 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001874 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001875 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001876 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001877 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001878 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001879 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1880 // send a ping.
1881 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1882 .WillOnce(Return(true));
1883 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1884 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001885 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001886 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001887 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001888 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001889 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001890 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001891 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001892 metrics::CheckResult::kNoUpdateAvailable,
1893 metrics::CheckReaction::kUnset,
1894 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001895 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001896 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001897 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001898 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001899}
1900
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001901TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001902 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001903 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001904 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001905 int64_t now = Time::Now().ToInternalValue();
1906 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001907 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07001908 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001909 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07001910 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1911 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001912 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001913 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001914 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001915 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001916 -1,
1917 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001918 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001919 metrics::CheckResult::kUnset,
1920 metrics::CheckReaction::kUnset,
1921 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001922 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001923 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001924 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001925}
1926
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001927TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001928 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001929 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001930 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1931 .Times(AnyNumber());
1932 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001933 int64_t future =
1934 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001935 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001936 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001937 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001938 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001939 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001940 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001941 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1942 .WillOnce(Return(true));
1943 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1944 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001945 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001946 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001947 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001948 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1949 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001950 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001951 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001952 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001953 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001954 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001955 metrics::CheckResult::kNoUpdateAvailable,
1956 metrics::CheckReaction::kUnset,
1957 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001958 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001959 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001960 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001961 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001962}
1963
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001964TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001965 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001966 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001967 // may fail if it runs for longer than 5 seconds. It shouldn't run
1968 // that long though.
1969 int64_t midnight =
1970 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1971 int64_t midnight_slack =
1972 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001973 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001974 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001975 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1976 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001977 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1978 AllOf(Ge(midnight), Le(midnight_slack))))
1979 .WillOnce(Return(true));
1980 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1981 AllOf(Ge(midnight), Le(midnight_slack))))
1982 .WillOnce(Return(true));
1983 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001984 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001985 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1986 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001987 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001988 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001989 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001990 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001991 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001992 metrics::CheckResult::kNoUpdateAvailable,
1993 metrics::CheckReaction::kUnset,
1994 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001995 nullptr,
1996 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001997}
1998
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001999TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002000 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002001 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002002 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2003 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002004 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2005 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2006 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002007 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002008 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2009 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002010 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002011 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002012 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002013 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002014 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002015 metrics::CheckResult::kNoUpdateAvailable,
2016 metrics::CheckReaction::kUnset,
2017 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002018 nullptr,
2019 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002020}
2021
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002022TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002023 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002024 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002025 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2026 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002027 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2028 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2029 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002030 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002031 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2032 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002033 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002034 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002035 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002036 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002037 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002038 metrics::CheckResult::kNoUpdateAvailable,
2039 metrics::CheckReaction::kUnset,
2040 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002041 nullptr,
2042 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002043}
2044
Alex Deymob3fa53b2016-04-18 19:57:58 -07002045TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
2046 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2047 // the "eol" attribute.
2048 ASSERT_TRUE(
2049 TestUpdateCheck(nullptr, // request_params
2050 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2051 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2052 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2053 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2054 "</app></response>",
2055 -1,
2056 false, // ping_only
2057 ErrorCode::kSuccess,
2058 metrics::CheckResult::kNoUpdateAvailable,
2059 metrics::CheckReaction::kUnset,
2060 metrics::DownloadErrorCode::kUnset,
2061 nullptr,
2062 nullptr));
2063 string eol_pref;
2064 EXPECT_TRUE(
2065 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2066 // Note that the eol="eol" attribute should be ignored and the _eol should be
2067 // used instead.
2068 EXPECT_EQ("security-only", eol_pref);
2069}
2070
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002071TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002072 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002073 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07002074 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002075 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002076 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002077 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002078 metrics::CheckResult::kParsingError,
2079 metrics::CheckReaction::kUnset,
2080 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002081 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002082 &post_data));
2083 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002084 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002085 EXPECT_EQ(post_str.find("machineid="), string::npos);
2086 EXPECT_EQ(post_str.find("userid="), string::npos);
2087}
2088
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002089TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002090 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002091 const int http_error_code =
2092 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07002093 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002094 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002095 "",
2096 501,
Darin Petkov265f2902011-05-09 15:17:40 -07002097 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002098 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002099 metrics::CheckResult::kDownloadError,
2100 metrics::CheckReaction::kUnset,
2101 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07002102 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002103 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002104 EXPECT_FALSE(response.update_exists);
2105}
2106
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002107TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002108 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002109 const int http_error_code =
2110 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07002111 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002112 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002113 "",
2114 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07002115 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002116 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002117 metrics::CheckResult::kDownloadError,
2118 metrics::CheckReaction::kUnset,
2119 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07002120 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002121 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002122 EXPECT_FALSE(response.update_exists);
2123}
2124
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002125TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002126 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002127 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002128 params.set_wall_clock_based_wait_enabled(true);
2129 params.set_waiting_period(TimeDelta().FromDays(1));
2130 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002131
Sen Jiang7c1171e2016-06-23 11:35:40 -07002132 Time arbitrary_date;
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002133 ASSERT_TRUE(Time::FromString("6/4/1989", &arbitrary_date));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002134 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
2135 ASSERT_FALSE(TestUpdateCheck(&params,
2136 fake_update_response_.GetUpdateResponse(),
2137 -1,
2138 false, // ping_only
2139 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2140 metrics::CheckResult::kUpdateAvailable,
2141 metrics::CheckReaction::kDeferring,
2142 metrics::DownloadErrorCode::kUnset,
2143 &response,
2144 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002145
Ben Chan9abb7632014-08-07 00:10:53 -07002146 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002147 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002148 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002149 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002150
2151 // Verify if we are interactive check we don't defer.
2152 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002153 ASSERT_TRUE(TestUpdateCheck(&params,
2154 fake_update_response_.GetUpdateResponse(),
2155 -1,
2156 false, // ping_only
2157 ErrorCode::kSuccess,
2158 metrics::CheckResult::kUpdateAvailable,
2159 metrics::CheckReaction::kUpdating,
2160 metrics::DownloadErrorCode::kUnset,
2161 &response,
2162 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002163 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002164}
2165
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002166TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002167 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002168 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002169 params.set_wall_clock_based_wait_enabled(true);
2170 params.set_waiting_period(TimeDelta().FromDays(1));
2171 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002172
Sen Jiang7c1171e2016-06-23 11:35:40 -07002173 Time t1, t2;
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002174 ASSERT_TRUE(Time::FromString("1/1/2012", &t1));
2175 ASSERT_TRUE(Time::FromString("1/3/2012", &t2));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002176 ASSERT_TRUE(
2177 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2178 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2179 ASSERT_TRUE(TestUpdateCheck(&params,
2180 fake_update_response_.GetUpdateResponse(),
2181 -1,
2182 false, // ping_only
2183 ErrorCode::kSuccess,
2184 metrics::CheckResult::kUpdateAvailable,
2185 metrics::CheckReaction::kUpdating,
2186 metrics::DownloadErrorCode::kUnset,
2187 &response,
2188 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002189
2190 EXPECT_TRUE(response.update_exists);
2191
2192 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002193 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002194 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002195 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2196}
2197
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002198TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002199 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002200 base::ScopedTempDir tempdir;
2201 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002202
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002203 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002204 OmahaRequestParams params(&fake_system_state_);
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002205 params.set_root(tempdir.GetPath().value());
Sen Jiang297e5832016-03-17 14:45:51 -07002206 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2207 params.set_app_version("1.2.3.4");
Sen Jiang8cd42342018-01-31 12:06:59 -08002208 params.set_product_components("o.bundle=1");
Sen Jiang297e5832016-03-17 14:45:51 -07002209 params.set_current_channel("canary-channel");
2210 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2211 params.UpdateDownloadChannel();
Sen Jiang8500d3a2018-02-08 12:04:05 -08002212 EXPECT_TRUE(params.ShouldPowerwash());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002213 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002214 "invalid xml>",
2215 -1,
2216 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002217 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002218 metrics::CheckResult::kParsingError,
2219 metrics::CheckReaction::kUnset,
2220 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002221 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002222 &post_data));
2223 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002224 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002225 EXPECT_NE(string::npos, post_str.find(
2226 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2227 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2228 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Sen Jiang8cd42342018-01-31 12:06:59 -08002229 EXPECT_EQ(string::npos, post_str.find("o.bundle"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002230}
2231
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002232TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002233 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002234 base::ScopedTempDir tempdir;
2235 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002236
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002237 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002238 OmahaRequestParams params(&fake_system_state_);
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002239 params.set_root(tempdir.GetPath().value());
Sen Jiang297e5832016-03-17 14:45:51 -07002240 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2241 params.set_app_version("5.6.7.8");
Sen Jiang8cd42342018-01-31 12:06:59 -08002242 params.set_product_components("o.bundle=1");
Sen Jiang297e5832016-03-17 14:45:51 -07002243 params.set_current_channel("stable-channel");
2244 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2245 params.UpdateDownloadChannel();
Sen Jiang8500d3a2018-02-08 12:04:05 -08002246 EXPECT_FALSE(params.ShouldPowerwash());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002247 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002248 "invalid xml>",
2249 -1,
2250 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002251 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002252 metrics::CheckResult::kParsingError,
2253 metrics::CheckReaction::kUnset,
2254 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002255 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002256 &post_data));
2257 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002258 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002259 EXPECT_NE(string::npos, post_str.find(
2260 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2261 "version=\"5.6.7.8\" "
2262 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002263 EXPECT_EQ(string::npos, post_str.find("from_version"));
Sen Jiang8cd42342018-01-31 12:06:59 -08002264 EXPECT_NE(string::npos, post_str.find("o.bundle.version=\"1\""));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002265}
2266
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002267// Checks that the initial ping with a=-1 r=-1 is not send when the device
2268// was powerwashed.
2269TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2270 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2271
2272 // Flag that the device was powerwashed in the past.
2273 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2274
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002275 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002276 ASSERT_TRUE(
2277 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002278 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002279 -1,
2280 false, // ping_only
2281 ErrorCode::kSuccess,
2282 metrics::CheckResult::kNoUpdateAvailable,
2283 metrics::CheckReaction::kUnset,
2284 metrics::DownloadErrorCode::kUnset,
2285 nullptr,
2286 &post_data));
2287 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002288 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002289 EXPECT_EQ(string::npos, post_str.find("<ping"));
2290}
2291
Amin Hassani1677e812017-06-21 13:36:36 -07002292// Checks that the initial ping with a=-1 r=-1 is not send when the device
2293// first_active_omaha_ping_sent is set.
2294TEST_F(OmahaRequestActionTest, PingWhenFirstActiveOmahaPingIsSent) {
2295 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2296
2297 // Flag that the device was not powerwashed in the past.
2298 fake_system_state_.fake_hardware()->SetPowerwashCount(0);
2299
2300 // Flag that the device has sent first active ping in the past.
2301 fake_system_state_.fake_hardware()->SetFirstActiveOmahaPingSent();
2302
2303 brillo::Blob post_data;
2304 ASSERT_TRUE(
2305 TestUpdateCheck(nullptr, // request_params
2306 fake_update_response_.GetNoUpdateResponse(),
2307 -1,
2308 false, // ping_only
2309 ErrorCode::kSuccess,
2310 metrics::CheckResult::kNoUpdateAvailable,
2311 metrics::CheckReaction::kUnset,
2312 metrics::DownloadErrorCode::kUnset,
2313 nullptr,
2314 &post_data));
2315 // We shouldn't send a ping in this case since
2316 // first_active_omaha_ping_sent=true
2317 string post_str(post_data.begin(), post_data.end());
2318 EXPECT_EQ(string::npos, post_str.find("<ping"));
2319}
2320
Alex Deymo9fded1e2015-11-05 12:31:19 -08002321// Checks that the event 54 is sent on a reboot to a new update.
2322TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2323 // Flag that the device was updated in a previous boot.
2324 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2325
2326 brillo::Blob post_data;
2327 ASSERT_TRUE(
2328 TestUpdateCheck(nullptr, // request_params
2329 fake_update_response_.GetNoUpdateResponse(),
2330 -1,
2331 false, // ping_only
2332 ErrorCode::kSuccess,
2333 metrics::CheckResult::kNoUpdateAvailable,
2334 metrics::CheckReaction::kUnset,
2335 metrics::DownloadErrorCode::kUnset,
2336 nullptr,
2337 &post_data));
2338 string post_str(post_data.begin(), post_data.end());
2339
2340 // An event 54 is included and has the right version.
2341 EXPECT_NE(string::npos,
2342 post_str.find(base::StringPrintf(
2343 "<event eventtype=\"%d\"",
2344 OmahaEvent::kTypeRebootedAfterUpdate)));
2345 EXPECT_NE(string::npos,
2346 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2347
2348 // The previous version flag should have been removed.
2349 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2350 string prev_version;
2351 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2352 EXPECT_TRUE(prev_version.empty());
2353}
2354
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002355void OmahaRequestActionTest::P2PTest(
2356 bool initial_allow_p2p_for_downloading,
2357 bool initial_allow_p2p_for_sharing,
2358 bool omaha_disable_p2p_for_downloading,
2359 bool omaha_disable_p2p_for_sharing,
2360 bool payload_state_allow_p2p_attempt,
2361 bool expect_p2p_client_lookup,
2362 const string& p2p_client_result_url,
2363 bool expected_allow_p2p_for_downloading,
2364 bool expected_allow_p2p_for_sharing,
2365 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002366 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002367 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002368 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2369 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2370 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002371
2372 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002373 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002374 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2375 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002376 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2377 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2378 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2379 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2380 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2381 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2382 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2383 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2384 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2385 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2386
David Zeuthen8f191b22013-08-06 12:27:50 -07002387 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002388 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002389 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2390
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002391 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2392 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002393 .Times(expect_p2p_client_lookup ? 1 : 0);
2394
Alex Deymo8e18f932015-03-27 16:16:59 -07002395 fake_update_response_.disable_p2p_for_downloading =
2396 omaha_disable_p2p_for_downloading;
2397 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002398 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002399 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002400 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002401 -1,
2402 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002403 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002404 metrics::CheckResult::kUpdateAvailable,
2405 metrics::CheckReaction::kUpdating,
2406 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002407 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002408 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002409 EXPECT_TRUE(response.update_exists);
2410
Gilad Arnold74b5f552014-10-07 08:17:16 -07002411 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2412 response.disable_p2p_for_downloading);
2413 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2414 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002415
Gilad Arnold74b5f552014-10-07 08:17:16 -07002416 EXPECT_EQ(expected_allow_p2p_for_downloading,
2417 actual_allow_p2p_for_downloading);
2418 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2419 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002420}
2421
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002422TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002423 P2PTest(true, // initial_allow_p2p_for_downloading
2424 true, // initial_allow_p2p_for_sharing
2425 false, // omaha_disable_p2p_for_downloading
2426 false, // omaha_disable_p2p_for_sharing
2427 true, // payload_state_allow_p2p_attempt
2428 true, // expect_p2p_client_lookup
2429 "http://1.3.5.7/p2p", // p2p_client_result_url
2430 true, // expected_allow_p2p_for_downloading
2431 true, // expected_allow_p2p_for_sharing
2432 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002433}
2434
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002435TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002436 P2PTest(true, // initial_allow_p2p_for_downloading
2437 true, // initial_allow_p2p_for_sharing
2438 false, // omaha_disable_p2p_for_downloading
2439 false, // omaha_disable_p2p_for_sharing
2440 true, // payload_state_allow_p2p_attempt
2441 true, // expect_p2p_client_lookup
2442 "", // p2p_client_result_url
2443 false, // expected_allow_p2p_for_downloading
2444 true, // expected_allow_p2p_for_sharing
2445 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002446}
2447
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002448TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002449 P2PTest(false, // initial_allow_p2p_for_downloading
2450 true, // initial_allow_p2p_for_sharing
2451 false, // omaha_disable_p2p_for_downloading
2452 false, // omaha_disable_p2p_for_sharing
2453 true, // payload_state_allow_p2p_attempt
2454 false, // expect_p2p_client_lookup
2455 "unset", // p2p_client_result_url
2456 false, // expected_allow_p2p_for_downloading
2457 true, // expected_allow_p2p_for_sharing
2458 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002459}
2460
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002461TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002462 P2PTest(true, // initial_allow_p2p_for_downloading
2463 true, // initial_allow_p2p_for_sharing
2464 true, // omaha_disable_p2p_for_downloading
2465 false, // omaha_disable_p2p_for_sharing
2466 true, // payload_state_allow_p2p_attempt
2467 false, // expect_p2p_client_lookup
2468 "unset", // p2p_client_result_url
2469 false, // expected_allow_p2p_for_downloading
2470 true, // expected_allow_p2p_for_sharing
2471 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002472}
2473
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002474TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002475 P2PTest(true, // initial_allow_p2p_for_downloading
2476 true, // initial_allow_p2p_for_sharing
2477 false, // omaha_disable_p2p_for_downloading
2478 true, // omaha_disable_p2p_for_sharing
2479 true, // payload_state_allow_p2p_attempt
2480 true, // expect_p2p_client_lookup
2481 "http://1.3.5.7/p2p", // p2p_client_result_url
2482 true, // expected_allow_p2p_for_downloading
2483 false, // expected_allow_p2p_for_sharing
2484 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002485}
2486
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002487TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002488 P2PTest(true, // initial_allow_p2p_for_downloading
2489 true, // initial_allow_p2p_for_sharing
2490 true, // omaha_disable_p2p_for_downloading
2491 true, // omaha_disable_p2p_for_sharing
2492 true, // payload_state_allow_p2p_attempt
2493 false, // expect_p2p_client_lookup
2494 "unset", // p2p_client_result_url
2495 false, // expected_allow_p2p_for_downloading
2496 false, // expected_allow_p2p_for_sharing
2497 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002498}
2499
Alex Deymof329b932014-10-30 01:37:48 -07002500bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2501 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002502 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002503 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002504 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002505 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002506 -1,
2507 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002508 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002509 metrics::CheckResult::kUpdateAvailable,
2510 metrics::CheckReaction::kUpdating,
2511 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002512 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002513 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002514}
2515
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002516TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002517 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002518
Kevin Cernekee2494e282016-03-29 18:03:53 -07002519 // Simulate a successful update check that happens during OOBE. The
2520 // deadline in the response is needed to force the update attempt to
2521 // occur; responses without a deadline seen during OOBE will normally
2522 // return ErrorCode::kNonCriticalUpdateInOOBE.
2523 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2524 fake_update_response_.deadline = "20101020";
2525
David Zeuthen639aa362014-02-03 16:23:44 -08002526 // Check that we parse elapsed_days in the Omaha Response correctly.
2527 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002528 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2529 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002530 EXPECT_TRUE(response.update_exists);
2531 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002532 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002533 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002534 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002535 EXPECT_EQ(prefs_days, 42);
2536
2537 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002538 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002539 EXPECT_TRUE(response.update_exists);
2540 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002541 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002542 EXPECT_EQ(prefs_days, 42);
2543
2544 // Note that elapsed_days is not necessarily divisible by 7 so check
2545 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002546 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2547 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002548 EXPECT_TRUE(response.update_exists);
2549 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002550 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002551 EXPECT_EQ(prefs_days, 21);
2552
2553 // Check that we correctly handle elapsed_days not being included in
2554 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002555 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002556 EXPECT_TRUE(response.update_exists);
2557 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002558}
2559
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002560// If there is no prefs and OOBE is not complete, we should not
2561// report anything to Omaha.
2562TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002563 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002564 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2565 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2566}
David Zeuthen639aa362014-02-03 16:23:44 -08002567
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002568// If OOBE is complete and happened on a valid date (e.g. after Jan
2569// 1 2007 0:00 PST), that date should be used and written to
2570// prefs. However, first try with an invalid date and check we do
2571// nothing.
2572TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2573 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2574 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2575 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2576 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2577}
David Zeuthen639aa362014-02-03 16:23:44 -08002578
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002579// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2580// should yield an InstallDate of 14.
2581TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2582 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2583 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2584 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2585 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002586
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002587 int64_t prefs_days;
2588 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2589 EXPECT_EQ(prefs_days, 14);
2590}
David Zeuthen639aa362014-02-03 16:23:44 -08002591
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002592// Now that we have a valid date in prefs, check that we keep using
2593// that even if OOBE date reports something else. The date Jan 30,
2594// 2007 0:00 PST should yield an InstallDate of 28... but since
2595// there's a prefs file, we should still get 14.
2596TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2597 // Set a valid date in the prefs first.
2598 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002599
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002600 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2601 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2602 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002603
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002604 int64_t prefs_days;
2605 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2606 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002607
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002608 // If we delete the prefs file, we should get 28 days.
2609 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2610 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2611 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2612 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002613}
2614
Darin Petkov6a5b3222010-07-13 14:55:28 -07002615} // namespace chromeos_update_engine