blob: 87dc404a80d3c3a9bcc357b112c4edb69c16f4e3 [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 Jiang81259682017-03-30 15:11:30 -0700146 "/></actions></manifest></updatecheck></app>" +
147 (multi_app
Sen Jiangb1e063a2017-09-15 17:44:31 -0700148 ? "<app appid=\"" + app_id2 + "\"" +
149 (include_cohorts ? " cohort=\"cohort2\"" : "") +
150 "><updatecheck status=\"ok\"><urls><url codebase=\"" +
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700151 codebase2 + "\"/></urls><manifest version=\"" + version2 +
152 "\"><packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700153 "<package name=\"package3\" size=\"333\" "
154 "hash_sha256=\"hash3\"/></packages>"
Sen Jiang00adf7b2017-06-26 15:57:29 -0700155 "<actions><action event=\"postinstall\" " +
156 (multi_app_self_update
157 ? "noupdate=\"true\" IsDeltaPayload=\"true\" "
158 : "IsDeltaPayload=\"false\" ") +
159 "MetadataSize=\"33\"/></actions>"
Sen Jiang81259682017-03-30 15:11:30 -0700160 "</manifest></updatecheck></app>"
161 : "") +
162 (multi_app_no_update
163 ? "<app><updatecheck status=\"noupdate\"/></app>"
164 : "") +
165 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700166 }
167
168 // Return the payload URL, which is split in two fields in the XML response.
169 string GetPayloadUrl() {
170 return codebase + filename;
171 }
172
Alex Deymo85616652015-10-15 18:48:31 -0700173 string app_id = kTestAppId;
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700174 string app_id2 = kTestAppId2;
Alex Deymo8e18f932015-03-27 16:16:59 -0700175 string version = "1.2.3.4";
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700176 string version2 = "2.3.4.5";
Alex Deymo8e18f932015-03-27 16:16:59 -0700177 string more_info_url = "http://more/info";
178 string prompt = "true";
179 string codebase = "http://code/base/";
Sen Jiang81259682017-03-30 15:11:30 -0700180 string codebase2 = "http://code/base/2/";
Alex Deymo8e18f932015-03-27 16:16:59 -0700181 string filename = "file.signed";
Sen Jiang2703ef42017-03-16 13:36:21 -0700182 string hash = "4841534831323334";
Alex Deymo8e18f932015-03-27 16:16:59 -0700183 string needsadmin = "false";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800184 uint64_t size = 123;
Alex Deymo8e18f932015-03-27 16:16:59 -0700185 string deadline = "";
186 string max_days_to_scatter = "7";
187 string elapsed_days = "42";
188
189 // P2P setting defaults to allowed.
190 bool disable_p2p_for_downloading = false;
191 bool disable_p2p_for_sharing = false;
192
193 // Omaha cohorts settings.
194 bool include_cohorts = false;
195 string cohort = "";
196 string cohorthint = "";
197 string cohortname = "";
198
199 // Whether to include the CrOS <!ENTITY> in the XML response.
200 bool include_entity = false;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800201
Sen Jiang81259682017-03-30 15:11:30 -0700202 // Whether to include more than one app.
203 bool multi_app = false;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700204 // Whether to include an app with noupdate="true".
205 bool multi_app_self_update = false;
206 // Whether to include an additional app with status="noupdate".
Sen Jiang81259682017-03-30 15:11:30 -0700207 bool multi_app_no_update = false;
208 // Whether to include more than one package in an app.
Sen Jiang0affc2c2017-02-10 15:55:05 -0800209 bool multi_package = false;
Alex Deymo8e18f932015-03-27 16:16:59 -0700210};
211
212} // namespace
213
Darin Petkov6a5b3222010-07-13 14:55:28 -0700214namespace chromeos_update_engine {
215
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700216class OmahaRequestActionTest : public ::testing::Test {
217 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800218 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700219 fake_system_state_.set_request_params(&request_params_);
220 fake_system_state_.set_prefs(&fake_prefs_);
221 }
222
223 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800224 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700225 // is used. |payload_state| may be null, in which case a local mock is used.
226 // |p2p_manager| may be null, in which case a local mock is used.
227 // |connection_manager| may be null, in which case a local mock is used.
228 // out_response may be null. If |fail_http_response_code| is non-negative,
229 // the transfer will fail with that code. |ping_only| is passed through to the
230 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
231 // post-data received by the mock HttpFetcher is returned.
232 //
233 // The |expected_check_result|, |expected_check_reaction| and
234 // |expected_error_code| parameters are for checking expectations
235 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
236 // UMA statistics. Use the appropriate ::kUnset value to specify that
237 // the given metric should not be reported.
238 bool TestUpdateCheck(OmahaRequestParams* request_params,
239 const string& http_response,
240 int fail_http_response_code,
241 bool ping_only,
242 ErrorCode expected_code,
243 metrics::CheckResult expected_check_result,
244 metrics::CheckReaction expected_check_reaction,
245 metrics::DownloadErrorCode expected_download_error_code,
246 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700247 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700248
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800249 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700250 // only a ping or also an updatecheck.
251 void PingTest(bool ping_only);
252
253 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700254 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700255 OmahaResponse *response);
256
257 // P2P test helper function.
258 void P2PTest(
259 bool initial_allow_p2p_for_downloading,
260 bool initial_allow_p2p_for_sharing,
261 bool omaha_disable_p2p_for_downloading,
262 bool omaha_disable_p2p_for_sharing,
263 bool payload_state_allow_p2p_attempt,
264 bool expect_p2p_client_lookup,
265 const string& p2p_client_result_url,
266 bool expected_allow_p2p_for_downloading,
267 bool expected_allow_p2p_for_sharing,
268 const string& expected_p2p_url);
269
270 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700271 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700272
273 // By default, all tests use these objects unless they replace them in the
274 // fake_system_state_.
275 OmahaRequestParams request_params_ = OmahaRequestParams{
276 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700277 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700278 OmahaRequestParams::kOsVersion,
279 "service_pack",
280 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700281 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700282 "0.1.0.0",
283 "en-US",
284 "unittest",
285 "OEM MODEL 09235 7471",
286 "ChromeOSFirmware.1.0",
287 "0X0A1",
288 false, // delta okay
289 false, // interactive
290 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700291 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700292
293 FakePrefs fake_prefs_;
294};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700295
296namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700297class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
298 public:
299 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700300 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800301 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700302 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800303 void ProcessingDone(const ActionProcessor* processor,
304 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700305 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700306 }
307
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800308 void ActionCompleted(ActionProcessor* processor,
309 AbstractAction* action,
310 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700311 // make sure actions always succeed
312 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700313 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700314 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700315 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700316 }
David Zeuthena99981f2013-04-29 13:42:47 -0700317 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700318};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700319} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700320
321class OutputObjectCollectorAction;
322
323template<>
324class ActionTraits<OutputObjectCollectorAction> {
325 public:
326 // Does not take an object for input
327 typedef OmahaResponse InputObjectType;
328 // On success, puts the output path on output
329 typedef NoneType OutputObjectType;
330};
331
332class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
333 public:
334 OutputObjectCollectorAction() : has_input_object_(false) {}
335 void PerformAction() {
336 // copy input object
337 has_input_object_ = HasInputObject();
338 if (has_input_object_)
339 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700340 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700341 }
342 // Should never be called
343 void TerminateProcessing() {
344 CHECK(false);
345 }
346 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700347 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700348 return "OutputObjectCollectorAction";
349 }
Alex Deymof329b932014-10-30 01:37:48 -0700350 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800351 using InputObjectType =
352 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
353 using OutputObjectType =
354 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700355 bool has_input_object_;
356 OmahaResponse omaha_response_;
357};
358
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700359bool OmahaRequestActionTest::TestUpdateCheck(
360 OmahaRequestParams* request_params,
361 const string& http_response,
362 int fail_http_response_code,
363 bool ping_only,
364 ErrorCode expected_code,
365 metrics::CheckResult expected_check_result,
366 metrics::CheckReaction expected_check_reaction,
367 metrics::DownloadErrorCode expected_download_error_code,
368 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700369 brillo::Blob* out_post_data) {
370 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700371 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700372 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800373 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700374 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700375 if (fail_http_response_code >= 0) {
376 fetcher->FailTransfer(fail_http_response_code);
377 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700378 if (request_params)
379 fake_system_state_.set_request_params(request_params);
380 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700381 nullptr,
Ben Chan5c02c132017-06-27 07:10:36 -0700382 base::WrapUnique(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700383 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700384 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700385 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700386
Darin Petkov6a5b3222010-07-13 14:55:28 -0700387 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700388 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700389 processor.EnqueueAction(&action);
390
391 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700392 BondActions(&action, &collector_action);
393 processor.EnqueueAction(&collector_action);
394
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700395 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
396 ReportUpdateCheckMetrics(_, _, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800397 .Times(AnyNumber());
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700398
399 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
400 ReportUpdateCheckMetrics(_,
401 expected_check_result,
402 expected_check_reaction,
403 expected_download_error_code))
404 .Times(ping_only ? 0 : 1);
David Zeuthen33bae492014-02-25 16:16:18 -0800405
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700406 loop.PostTask(base::Bind(
407 [](ActionProcessor* processor) { processor->StartProcessing(); },
408 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700409 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700410 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700411 if (collector_action.has_input_object_ && out_response)
412 *out_response = collector_action.omaha_response_;
413 if (out_post_data)
414 *out_post_data = fetcher->post_data();
415 return collector_action.has_input_object_;
416}
417
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700418// Tests Event requests -- they should always succeed. |out_post_data|
419// may be null; if non-null, the post-data received by the mock
420// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700421void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700422 OmahaEvent* event,
423 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700424 brillo::Blob* out_post_data) {
425 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700426 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700427 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800428 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700429 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700430 FakeSystemState fake_system_state;
431 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300432 OmahaRequestAction action(&fake_system_state,
433 event,
Ben Chan5c02c132017-06-27 07:10:36 -0700434 base::WrapUnique(fetcher),
Alex Deymoc1c17b42015-11-23 03:53:15 -0300435 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700436 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700437 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700438 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700439 processor.EnqueueAction(&action);
440
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700441 loop.PostTask(base::Bind(
442 [](ActionProcessor* processor) { processor->StartProcessing(); },
443 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700444 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700445 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700446
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700447 if (out_post_data)
448 *out_post_data = fetcher->post_data();
449}
450
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700451TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400452 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700453 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400454 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700455 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700456 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400457 -1,
458 false, // ping_only
459 ErrorCode::kOmahaRequestXMLHasEntityDecl,
460 metrics::CheckResult::kParsingError,
461 metrics::CheckReaction::kUnset,
462 metrics::DownloadErrorCode::kUnset,
463 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700464 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400465 EXPECT_FALSE(response.update_exists);
466}
467
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700468TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700469 OmahaResponse response;
470 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700471 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700472 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700473 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700474 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700475 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800476 metrics::CheckResult::kNoUpdateAvailable,
477 metrics::CheckReaction::kUnset,
478 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700479 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700480 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700481 EXPECT_FALSE(response.update_exists);
482}
483
Sen Jiang81259682017-03-30 15:11:30 -0700484TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
485 OmahaResponse response;
486 fake_update_response_.multi_app_no_update = true;
487 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
488 fake_update_response_.GetNoUpdateResponse(),
489 -1,
490 false, // ping_only
491 ErrorCode::kSuccess,
492 metrics::CheckResult::kNoUpdateAvailable,
493 metrics::CheckReaction::kUnset,
494 metrics::DownloadErrorCode::kUnset,
495 &response,
496 nullptr));
497 EXPECT_FALSE(response.update_exists);
498}
499
Sen Jiang00adf7b2017-06-26 15:57:29 -0700500TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
501 OmahaResponse response;
502 fake_update_response_.multi_app_no_update = true;
503 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
504 fake_update_response_.GetUpdateResponse(),
505 -1,
506 false, // ping_only
507 ErrorCode::kSuccess,
508 metrics::CheckResult::kNoUpdateAvailable,
509 metrics::CheckReaction::kUnset,
510 metrics::DownloadErrorCode::kUnset,
511 &response,
512 nullptr));
513 EXPECT_FALSE(response.update_exists);
514}
515
516TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
517 OmahaResponse response;
518 ASSERT_TRUE(TestUpdateCheck(
519 nullptr, // request_params
520 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
521 "event=\"postinstall\" noupdate=\"true\"/></actions>"
522 "</manifest></updatecheck></app></response>",
523 -1,
524 false, // ping_only
525 ErrorCode::kSuccess,
526 metrics::CheckResult::kNoUpdateAvailable,
527 metrics::CheckReaction::kUnset,
528 metrics::DownloadErrorCode::kUnset,
529 &response,
530 nullptr));
531 EXPECT_FALSE(response.update_exists);
532}
533
Alex Deymo8e18f932015-03-27 16:16:59 -0700534// Test that all the values in the response are parsed in a normal update
535// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700536TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700537 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700538 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700539 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700540 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700541 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700542 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700543 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700544 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800545 metrics::CheckResult::kUpdateAvailable,
546 metrics::CheckReaction::kUpdating,
547 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700548 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700549 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700550 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700551 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700552 EXPECT_EQ("", response.system_version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800553 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
554 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700555 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800556 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
557 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700558 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700559 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
560 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700561 // Omaha cohort attributes are not set in the response, so they should not be
Alex Deymo8e18f932015-03-27 16:16:59 -0700562 // persisted.
563 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
564 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
565 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700566}
567
Sen Jiang0affc2c2017-02-10 15:55:05 -0800568TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
569 OmahaResponse response;
570 fake_update_response_.multi_package = true;
571 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
572 fake_update_response_.GetUpdateResponse(),
573 -1,
574 false, // ping_only
575 ErrorCode::kSuccess,
576 metrics::CheckResult::kUpdateAvailable,
577 metrics::CheckReaction::kUpdating,
578 metrics::DownloadErrorCode::kUnset,
579 &response,
580 nullptr));
581 EXPECT_TRUE(response.update_exists);
582 EXPECT_EQ(fake_update_response_.version, response.version);
583 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
584 response.packages[0].payload_urls[0]);
585 EXPECT_EQ(fake_update_response_.codebase + "package2",
586 response.packages[1].payload_urls[0]);
587 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
588 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700589 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700590 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800591 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700592 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800593 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700594 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700595 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700596}
597
598TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
599 OmahaResponse response;
600 fake_update_response_.multi_app = true;
601 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
602 fake_update_response_.GetUpdateResponse(),
603 -1,
604 false, // ping_only
605 ErrorCode::kSuccess,
606 metrics::CheckResult::kUpdateAvailable,
607 metrics::CheckReaction::kUpdating,
608 metrics::DownloadErrorCode::kUnset,
609 &response,
610 nullptr));
611 EXPECT_TRUE(response.update_exists);
612 EXPECT_EQ(fake_update_response_.version, response.version);
613 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
614 response.packages[0].payload_urls[0]);
615 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
616 response.packages[1].payload_urls[0]);
617 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
618 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
619 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700620 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700621 ASSERT_EQ(2u, response.packages.size());
622 EXPECT_EQ(string("hash3"), response.packages[1].hash);
623 EXPECT_EQ(333u, response.packages[1].size);
624 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700625 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700626}
627
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700628TEST_F(OmahaRequestActionTest, MultiAppAndSystemUpdateTest) {
629 OmahaResponse response;
630 fake_update_response_.multi_app = true;
631 // trigger the lining up of the app and system versions
632 request_params_.set_system_app_id(fake_update_response_.app_id2);
633
634 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
635 fake_update_response_.GetUpdateResponse(),
636 -1,
637 false, // ping_only
638 ErrorCode::kSuccess,
639 metrics::CheckResult::kUpdateAvailable,
640 metrics::CheckReaction::kUpdating,
641 metrics::DownloadErrorCode::kUnset,
642 &response,
643 nullptr));
644 EXPECT_TRUE(response.update_exists);
645 EXPECT_EQ(fake_update_response_.version, response.version);
646 EXPECT_EQ(fake_update_response_.version2, response.system_version);
647 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
648 response.packages[0].payload_urls[0]);
649 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
650 response.packages[1].payload_urls[0]);
651 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
652 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
653 EXPECT_EQ(11u, response.packages[0].metadata_size);
654 EXPECT_EQ(true, response.packages[0].is_delta);
655 ASSERT_EQ(2u, response.packages.size());
656 EXPECT_EQ(string("hash3"), response.packages[1].hash);
657 EXPECT_EQ(333u, response.packages[1].size);
658 EXPECT_EQ(33u, response.packages[1].metadata_size);
659 EXPECT_EQ(false, response.packages[1].is_delta);
660}
661
Sen Jiang81259682017-03-30 15:11:30 -0700662TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
663 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700664 fake_update_response_.multi_app = true;
665 fake_update_response_.multi_app_self_update = true;
Sen Jiang81259682017-03-30 15:11:30 -0700666 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
667 fake_update_response_.GetUpdateResponse(),
668 -1,
669 false, // ping_only
670 ErrorCode::kSuccess,
671 metrics::CheckResult::kUpdateAvailable,
672 metrics::CheckReaction::kUpdating,
673 metrics::DownloadErrorCode::kUnset,
674 &response,
675 nullptr));
676 EXPECT_TRUE(response.update_exists);
677 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700678 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700679 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
680 response.packages[0].payload_urls[0]);
681 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
682 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
683 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700684 ASSERT_EQ(2u, response.packages.size());
685 EXPECT_EQ(string("hash3"), response.packages[1].hash);
686 EXPECT_EQ(333u, response.packages[1].size);
687 EXPECT_EQ(33u, response.packages[1].metadata_size);
688 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700689}
690
691TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
692 OmahaResponse response;
693 fake_update_response_.multi_app = true;
694 fake_update_response_.multi_package = true;
695 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
696 fake_update_response_.GetUpdateResponse(),
697 -1,
698 false, // ping_only
699 ErrorCode::kSuccess,
700 metrics::CheckResult::kUpdateAvailable,
701 metrics::CheckReaction::kUpdating,
702 metrics::DownloadErrorCode::kUnset,
703 &response,
704 nullptr));
705 EXPECT_TRUE(response.update_exists);
706 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700707 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700708 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
709 response.packages[0].payload_urls[0]);
710 EXPECT_EQ(fake_update_response_.codebase + "package2",
711 response.packages[1].payload_urls[0]);
712 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
713 response.packages[2].payload_urls[0]);
714 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
715 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
716 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700717 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700718 ASSERT_EQ(3u, response.packages.size());
719 EXPECT_EQ(string("hash2"), response.packages[1].hash);
720 EXPECT_EQ(222u, response.packages[1].size);
721 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700722 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700723 EXPECT_EQ(string("hash3"), response.packages[2].hash);
724 EXPECT_EQ(333u, response.packages[2].size);
725 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700726 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800727}
728
Alex Deymo14ad88e2016-06-29 12:30:14 -0700729TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
730 const string http_response = "<?xml invalid response";
731 request_params_.set_interactive(true);
732
733 brillo::FakeMessageLoop loop(nullptr);
734 loop.SetAsCurrent();
735
736 MockHttpFetcher* fetcher =
737 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
Ben Chan5c02c132017-06-27 07:10:36 -0700738 OmahaRequestAction action(&fake_system_state_, nullptr,
739 base::WrapUnique(fetcher), false);
Alex Deymo14ad88e2016-06-29 12:30:14 -0700740 ActionProcessor processor;
741 processor.EnqueueAction(&action);
742
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700743 loop.PostTask(base::Bind(
744 [](ActionProcessor* processor) { processor->StartProcessing(); },
745 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700746 loop.Run();
747 EXPECT_FALSE(loop.PendingTasks());
748
749 // Check that the headers were set in the fetcher during the action. Note that
750 // we set this request as "interactive".
751 EXPECT_EQ("fg", fetcher->GetHeader("X-GoogleUpdate-Interactivity"));
752 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-GoogleUpdate-AppId"));
753 EXPECT_NE("", fetcher->GetHeader("X-GoogleUpdate-Updater"));
754}
755
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700756TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700757 OmahaResponse response;
758 // Set up a connection manager that doesn't allow a valid update over
759 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700760 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700761 fake_system_state_.set_connection_manager(&mock_cm);
762
Alex Deymo30534502015-07-20 15:06:33 -0700763 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
764 .WillRepeatedly(
Ben Chan672c1f52017-10-23 15:41:39 -0700765 DoAll(SetArgPointee<0>(ConnectionType::kEthernet),
766 SetArgPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700767 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700768 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
769 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700770
771 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700772 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700773 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700774 -1,
775 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700776 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700777 metrics::CheckResult::kUpdateAvailable,
778 metrics::CheckReaction::kIgnored,
779 metrics::DownloadErrorCode::kUnset,
780 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700781 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700782 EXPECT_FALSE(response.update_exists);
783}
784
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700785TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700786 string rollback_version = "1234.0.0";
787 OmahaResponse response;
788
789 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700790 fake_system_state_.set_payload_state(&mock_payload_state);
791
Chris Sosa77f79e82014-06-02 18:16:24 -0700792 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
793 .WillRepeatedly(Return(rollback_version));
794
Alex Deymo8e18f932015-03-27 16:16:59 -0700795 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700796 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700797 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700798 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700799 -1,
800 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700801 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700802 metrics::CheckResult::kUpdateAvailable,
803 metrics::CheckReaction::kIgnored,
804 metrics::DownloadErrorCode::kUnset,
805 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700806 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700807 EXPECT_FALSE(response.update_exists);
808}
809
Kevin Cernekee2494e282016-03-29 18:03:53 -0700810// Verify that update checks called during OOBE will only try to download
811// an update if the response includes a non-empty deadline field.
812TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
813 OmahaResponse response;
814
Sen Jiang8cd42342018-01-31 12:06:59 -0800815 // TODO(senj): set better default value for metrics::checkresult in
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700816 // OmahaRequestAction::ActionCompleted.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700817 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700818 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
819 fake_update_response_.GetUpdateResponse(),
820 -1,
821 false, // ping_only
822 ErrorCode::kNonCriticalUpdateInOOBE,
823 metrics::CheckResult::kParsingError,
824 metrics::CheckReaction::kUnset,
825 metrics::DownloadErrorCode::kUnset,
826 &response,
827 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -0700828 EXPECT_FALSE(response.update_exists);
829
Alex Deymo46a9aae2016-05-04 20:20:11 -0700830 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
831 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
832 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
833 fake_update_response_.GetUpdateResponse(),
834 -1,
835 false, // ping_only
836 ErrorCode::kSuccess,
837 metrics::CheckResult::kUpdateAvailable,
838 metrics::CheckReaction::kUpdating,
839 metrics::DownloadErrorCode::kUnset,
840 &response,
841 nullptr));
842 EXPECT_TRUE(response.update_exists);
843 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
844
845 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700846 fake_update_response_.deadline = "20101020";
847 ASSERT_TRUE(
848 TestUpdateCheck(nullptr, // request_params
849 fake_update_response_.GetUpdateResponse(),
850 -1,
851 false, // ping_only
852 ErrorCode::kSuccess,
853 metrics::CheckResult::kUpdateAvailable,
854 metrics::CheckReaction::kUpdating,
855 metrics::DownloadErrorCode::kUnset,
856 &response,
857 nullptr));
858 EXPECT_TRUE(response.update_exists);
859}
860
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700861TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700862 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700863 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700864 params.set_wall_clock_based_wait_enabled(true);
865 params.set_update_check_count_wait_enabled(false);
866 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700867
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700868 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700869 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700870 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700871 -1,
872 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700873 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800874 metrics::CheckResult::kUpdateAvailable,
875 metrics::CheckReaction::kDeferring,
876 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700877 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700878 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700879 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700880
881 // Verify if we are interactive check we don't defer.
882 params.set_interactive(true);
883 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700884 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700885 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700886 -1,
887 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700888 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800889 metrics::CheckResult::kUpdateAvailable,
890 metrics::CheckReaction::kUpdating,
891 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700892 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700893 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700894 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700895}
896
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700897TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700898 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700899 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700900 params.set_wall_clock_based_wait_enabled(false);
901 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700902
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700903 params.set_update_check_count_wait_enabled(true);
904 params.set_min_update_checks_needed(1);
905 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700906
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700907 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700908 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700909 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700910 -1,
911 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700912 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800913 metrics::CheckResult::kUpdateAvailable,
914 metrics::CheckReaction::kUpdating,
915 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700916 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700917 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700918 EXPECT_TRUE(response.update_exists);
919}
920
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700921TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700922 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700923 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700924 params.set_wall_clock_based_wait_enabled(true);
925 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700926
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700927 params.set_update_check_count_wait_enabled(true);
928 params.set_min_update_checks_needed(1);
929 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700930
Alex Deymo8e18f932015-03-27 16:16:59 -0700931 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700932 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700933 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700934 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700935 -1,
936 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700937 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800938 metrics::CheckResult::kUpdateAvailable,
939 metrics::CheckReaction::kUpdating,
940 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700941 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700942 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700943 EXPECT_TRUE(response.update_exists);
944}
945
946
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700947TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700948 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700949 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700950 params.set_wall_clock_based_wait_enabled(true);
951 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700952
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700953 params.set_update_check_count_wait_enabled(true);
954 params.set_min_update_checks_needed(0);
955 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700956
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700957 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700958 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700959 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700960 -1,
961 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700962 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800963 metrics::CheckResult::kUpdateAvailable,
964 metrics::CheckReaction::kUpdating,
965 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700966 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700967 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700968
Ben Chan9abb7632014-08-07 00:10:53 -0700969 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700970 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700971 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700972 EXPECT_TRUE(response.update_exists);
973}
974
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700975TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700976 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700977 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700978 params.set_wall_clock_based_wait_enabled(true);
979 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700980
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700981 params.set_update_check_count_wait_enabled(true);
982 params.set_min_update_checks_needed(1);
983 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700984
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700985 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700986 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700987 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700988 -1,
989 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700990 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800991 metrics::CheckResult::kUpdateAvailable,
992 metrics::CheckReaction::kDeferring,
993 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700994 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700995 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700996
Ben Chan9abb7632014-08-07 00:10:53 -0700997 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700998 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700999 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001000 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001001
1002 // Verify if we are interactive check we don't defer.
1003 params.set_interactive(true);
1004 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001005 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001006 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001007 -1,
1008 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001009 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001010 metrics::CheckResult::kUpdateAvailable,
1011 metrics::CheckReaction::kUpdating,
1012 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001013 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001014 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001015 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001016}
1017
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001018TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001019 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001020 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001021 params.set_wall_clock_based_wait_enabled(true);
1022 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001023
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001024 params.set_update_check_count_wait_enabled(true);
1025 params.set_min_update_checks_needed(1);
1026 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001027
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001028 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001029
1030 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001031 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001032 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001033 -1,
1034 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001035 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001036 metrics::CheckResult::kUpdateAvailable,
1037 metrics::CheckReaction::kDeferring,
1038 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001039 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001040 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001041
Ben Chan9abb7632014-08-07 00:10:53 -07001042 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001043 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001044 // count remains the same, as the decrementing happens in update_attempter
1045 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001046 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001047 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001048
1049 // Verify if we are interactive check we don't defer.
1050 params.set_interactive(true);
1051 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001052 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001053 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001054 -1,
1055 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001056 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001057 metrics::CheckResult::kUpdateAvailable,
1058 metrics::CheckReaction::kUpdating,
1059 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001060 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001061 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001062 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001063}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001064
Alex Deymo8e18f932015-03-27 16:16:59 -07001065TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1066 OmahaResponse response;
1067 OmahaRequestParams params = request_params_;
1068 fake_update_response_.include_cohorts = true;
1069 fake_update_response_.cohort = "s/154454/8479665";
1070 fake_update_response_.cohorthint = "please-put-me-on-beta";
1071 fake_update_response_.cohortname = "stable";
1072
1073 ASSERT_TRUE(TestUpdateCheck(&params,
1074 fake_update_response_.GetUpdateResponse(),
1075 -1,
1076 false, // ping_only
1077 ErrorCode::kSuccess,
1078 metrics::CheckResult::kUpdateAvailable,
1079 metrics::CheckReaction::kUpdating,
1080 metrics::DownloadErrorCode::kUnset,
1081 &response,
1082 nullptr));
1083
1084 string value;
1085 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1086 EXPECT_EQ(fake_update_response_.cohort, value);
1087
1088 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1089 EXPECT_EQ(fake_update_response_.cohorthint, value);
1090
1091 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1092 EXPECT_EQ(fake_update_response_.cohortname, value);
1093}
1094
1095TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1096 OmahaResponse response;
1097 OmahaRequestParams params = request_params_;
1098 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1099 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1100 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1101 fake_update_response_.include_cohorts = true;
1102 fake_update_response_.cohort = "s/154454/8479665";
1103 fake_update_response_.cohorthint = "please-put-me-on-beta";
1104 fake_update_response_.cohortname = "";
1105
1106 ASSERT_TRUE(TestUpdateCheck(&params,
1107 fake_update_response_.GetUpdateResponse(),
1108 -1,
1109 false, // ping_only
1110 ErrorCode::kSuccess,
1111 metrics::CheckResult::kUpdateAvailable,
1112 metrics::CheckReaction::kUpdating,
1113 metrics::DownloadErrorCode::kUnset,
1114 &response,
1115 nullptr));
1116
1117 string value;
1118 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1119 EXPECT_EQ(fake_update_response_.cohort, value);
1120
1121 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1122 EXPECT_EQ(fake_update_response_.cohorthint, value);
1123
1124 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1125}
1126
1127TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1128 OmahaResponse response;
1129 OmahaRequestParams params = request_params_;
1130 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1131
1132 ASSERT_TRUE(TestUpdateCheck(&params,
1133 fake_update_response_.GetUpdateResponse(),
1134 -1,
1135 false, // ping_only
1136 ErrorCode::kSuccess,
1137 metrics::CheckResult::kUpdateAvailable,
1138 metrics::CheckReaction::kUpdating,
1139 metrics::DownloadErrorCode::kUnset,
1140 &response,
1141 nullptr));
1142
1143 string value;
1144 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1145 EXPECT_EQ("old_value", value);
1146
1147 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1148 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1149}
1150
Alex Deymo00d79ac2015-06-29 15:41:49 -07001151TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1152 OmahaResponse response;
1153 OmahaRequestParams params = request_params_;
1154 fake_update_response_.include_cohorts = true;
1155 fake_update_response_.cohort = "s/154454/8479665";
1156 fake_update_response_.cohorthint = "please-put-me-on-beta";
1157 fake_update_response_.cohortname = "stable";
1158
1159 ASSERT_TRUE(TestUpdateCheck(&params,
1160 fake_update_response_.GetNoUpdateResponse(),
1161 -1,
1162 false, // ping_only
1163 ErrorCode::kSuccess,
1164 metrics::CheckResult::kNoUpdateAvailable,
1165 metrics::CheckReaction::kUnset,
1166 metrics::DownloadErrorCode::kUnset,
1167 &response,
1168 nullptr));
1169
1170 string value;
1171 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1172 EXPECT_EQ(fake_update_response_.cohort, value);
1173
1174 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1175 EXPECT_EQ(fake_update_response_.cohorthint, value);
1176
1177 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1178 EXPECT_EQ(fake_update_response_.cohortname, value);
1179}
1180
Sen Jiangb1e063a2017-09-15 17:44:31 -07001181TEST_F(OmahaRequestActionTest, MultiAppCohortTest) {
1182 OmahaResponse response;
1183 OmahaRequestParams params = request_params_;
1184 fake_update_response_.multi_app = true;
1185 fake_update_response_.include_cohorts = true;
1186 fake_update_response_.cohort = "s/154454/8479665";
1187 fake_update_response_.cohorthint = "please-put-me-on-beta";
1188 fake_update_response_.cohortname = "stable";
1189
1190 ASSERT_TRUE(TestUpdateCheck(&params,
1191 fake_update_response_.GetUpdateResponse(),
1192 -1,
1193 false, // ping_only
1194 ErrorCode::kSuccess,
1195 metrics::CheckResult::kUpdateAvailable,
1196 metrics::CheckReaction::kUpdating,
1197 metrics::DownloadErrorCode::kUnset,
1198 &response,
1199 nullptr));
1200
1201 string value;
1202 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1203 EXPECT_EQ(fake_update_response_.cohort, value);
1204
1205 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1206 EXPECT_EQ(fake_update_response_.cohorthint, value);
1207
1208 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1209 EXPECT_EQ(fake_update_response_.cohortname, value);
1210}
1211
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001212TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001213 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001214
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001215 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001216 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001217
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001218 OmahaRequestParams params = request_params_;
1219 fake_system_state_.set_request_params(&params);
Ben Chan5c02c132017-06-27 07:10:36 -07001220 OmahaRequestAction action(
1221 &fake_system_state_,
1222 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001223 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001224 http_response.size(),
1225 nullptr),
1226 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001227 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001228 ActionProcessor processor;
1229 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001230 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001231
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001232 loop.PostTask(base::Bind(
1233 [](ActionProcessor* processor) { processor->StartProcessing(); },
1234 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001235 loop.Run();
1236 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001237 EXPECT_FALSE(processor.IsRunning());
1238}
1239
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001240TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001241 OmahaResponse response;
1242 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001243 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001244 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001245 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001246 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001247 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001248 metrics::CheckResult::kParsingError,
1249 metrics::CheckReaction::kUnset,
1250 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001251 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001252 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001253 EXPECT_FALSE(response.update_exists);
1254}
1255
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001256TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001257 OmahaResponse response;
1258 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001259 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001260 "",
1261 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001262 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001263 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001264 metrics::CheckResult::kParsingError,
1265 metrics::CheckReaction::kUnset,
1266 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001267 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001268 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001269 EXPECT_FALSE(response.update_exists);
1270}
1271
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001272TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001273 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001274 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001275 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001276 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1277 "<daystart elapsed_seconds=\"100\"/>"
1278 "<app appid=\"foo\" status=\"ok\">"
1279 "<ping status=\"ok\"/>"
1280 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001281 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001282 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001283 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001284 metrics::CheckResult::kParsingError,
1285 metrics::CheckReaction::kUnset,
1286 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001287 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001288 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001289 EXPECT_FALSE(response.update_exists);
1290}
1291
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001292TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001293 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001294 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001295 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001296 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1297 "<daystart elapsed_seconds=\"100\"/>"
1298 "<app appid=\"foo\" status=\"ok\">"
1299 "<ping status=\"ok\"/>"
1300 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001301 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001302 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001303 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001304 metrics::CheckResult::kParsingError,
1305 metrics::CheckReaction::kUnset,
1306 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001307 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001308 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001309 EXPECT_FALSE(response.update_exists);
1310}
1311
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001312TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001313 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001314 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001315 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001316 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1317 "<daystart elapsed_seconds=\"100\"/>"
1318 "<app appid=\"foo\" status=\"ok\">"
1319 "<ping status=\"ok\"/>"
1320 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001321 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001322 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001323 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001324 metrics::CheckResult::kParsingError,
1325 metrics::CheckReaction::kUnset,
1326 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001327 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001328 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001329 EXPECT_FALSE(response.update_exists);
1330}
1331
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001332TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001333 string input_response =
1334 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1335 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001336 // the appid needs to match that in the request params
1337 "<app appid=\"" +
1338 fake_update_response_.app_id +
1339 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001340 "<updatecheck status=\"ok\">"
1341 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001342 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001343 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001344 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001345 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001346 "ChromeOSVersion=\"10.2.3.4\" "
1347 "Prompt=\"false\" "
1348 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001349 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001350 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001351 "needsadmin=\"true\" "
1352 "/></actions></manifest></updatecheck></app></response>";
1353 LOG(INFO) << "Input Response = " << input_response;
1354
Darin Petkov6a5b3222010-07-13 14:55:28 -07001355 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001356 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001357 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001358 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001359 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001360 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001361 metrics::CheckResult::kUpdateAvailable,
1362 metrics::CheckReaction::kUpdating,
1363 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001364 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001365 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001366 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001367 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001368 EXPECT_EQ("http://missing/field/test/f",
1369 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001370 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001371 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1372 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001373 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001374 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001375}
1376
1377namespace {
1378class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1379 public:
1380 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001381 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001382 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001383};
1384
Alex Deymo60ca1a72015-06-18 18:19:15 -07001385void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001386 processor->StartProcessing();
1387 CHECK(processor->IsRunning());
1388 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001389}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001390} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001391
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001392TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001393 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001394 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001395
Alex Deymo60ca1a72015-06-18 18:19:15 -07001396 string http_response("doesn't matter");
Ben Chan5c02c132017-06-27 07:10:36 -07001397 OmahaRequestAction action(
1398 &fake_system_state_,
1399 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001400 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001401 http_response.size(),
1402 nullptr),
1403 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001404 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001405 ActionProcessor processor;
1406 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001407 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001408
Alex Deymo60ca1a72015-06-18 18:19:15 -07001409 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1410 loop.Run();
1411 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001412}
1413
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001414TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001415 string output;
1416 EXPECT_TRUE(XmlEncode("ab", &output));
1417 EXPECT_EQ("ab", output);
1418 EXPECT_TRUE(XmlEncode("a<b", &output));
1419 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001420 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1421 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001422 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1423 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001424 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001425 EXPECT_FALSE(XmlEncode("\xc2", &output));
1426 // Fail with invalid ASCII-7 chars.
1427 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1428}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001429
Alex Deymob0d74eb2015-03-30 17:59:17 -07001430TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1431 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1432 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1433}
1434
1435TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001436 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001437
1438 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001439 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001440 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001441 OmahaRequestParams::kOsVersion,
1442 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001443 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001444 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001445 "0.1.0.0",
1446 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001447 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001448 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001449 "ChromeOSFirmware.1.0",
1450 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001451 false, // delta okay
1452 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001453 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001454 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001455 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1456 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1457 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001458 base::JoinString(
1459 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001460 OmahaResponse response;
1461 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001462 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001463 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001464 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001465 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001466 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001467 metrics::CheckResult::kParsingError,
1468 metrics::CheckReaction::kUnset,
1469 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001470 &response,
1471 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001472 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001473 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001474 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1475 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1476 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1477 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1478 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1479 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1480 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1481 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1482 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1483 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1484 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1485 // Values from Prefs that are too big are removed from the XML instead of
1486 // encoded.
1487 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001488}
1489
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001490TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001491 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001492 fake_update_response_.deadline = "&lt;20110101";
1493 fake_update_response_.more_info_url = "testthe&lt;url";
1494 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001495 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001496 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001497 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001498 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001499 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001500 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001501 metrics::CheckResult::kUpdateAvailable,
1502 metrics::CheckReaction::kUpdating,
1503 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001504 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001505 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001506
Sen Jiang0affc2c2017-02-10 15:55:05 -08001507 EXPECT_EQ("testthe<url", response.more_info_url);
1508 EXPECT_EQ("testthe&codebase/file.signed",
1509 response.packages[0].payload_urls[0]);
1510 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001511}
1512
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001513TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001514 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001515 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001516 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001517 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001518 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001519 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001520 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001521 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001522 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001523 metrics::CheckResult::kUpdateAvailable,
1524 metrics::CheckReaction::kUpdating,
1525 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001526 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001527 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001528
Sen Jiang0affc2c2017-02-10 15:55:05 -08001529 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001530}
1531
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001532TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001533 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001534 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001535 fake_system_state_.set_prefs(&prefs);
1536
Darin Petkov95508da2011-01-05 12:42:29 -08001537 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001538 .WillOnce(DoAll(SetArgPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001539 // An existing but empty previous version means that we didn't reboot to a new
1540 // update, therefore, no need to update the previous version.
1541 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001542 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001543 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001544 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001545 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001546 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001547 metrics::CheckResult::kParsingError,
1548 metrics::CheckReaction::kUnset,
1549 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001550 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001551 &post_data));
1552 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001553 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001554 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001555 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001556 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001557 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001558 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1559 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001560 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1561 string::npos);
1562 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1563 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001564 // No <event> tag should be sent if we didn't reboot to an update.
1565 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001566}
1567
Jay Srinivasan0a708742012-03-20 11:26:12 -07001568
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001569TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001570 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001571 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001572 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1573 "invalid xml>",
1574 &post_data);
1575 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001576 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001577 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001578 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001579 OmahaEvent::kTypeUpdateDownloadStarted,
1580 OmahaEvent::kResultSuccess);
1581 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001582 EXPECT_EQ(post_str.find("ping"), string::npos);
1583 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001584}
1585
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001586TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001587 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001588 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001589 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1590 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001591 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001592 "invalid xml>",
1593 &post_data);
1594 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001595 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001596 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001597 " <event eventtype=\"%d\" eventresult=\"%d\" "
1598 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001599 OmahaEvent::kTypeDownloadComplete,
1600 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001601 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001602 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001603 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001604}
1605
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001606TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001607 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001608 // Create a copy of the OmahaRequestParams to reuse it later.
1609 OmahaRequestParams params = request_params_;
1610 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001611 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001612 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001613 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001614 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001615 http_response.size(),
1616 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001617 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001618 EXPECT_FALSE(update_check_action.IsEvent());
1619
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001620 params = request_params_;
1621 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001622 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001623 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001624 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Ben Chanab5a0af2017-10-12 14:57:50 -07001625 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001626 http_response.size(),
1627 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001628 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001629 EXPECT_TRUE(event_action.IsEvent());
1630}
1631
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001632TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001633 for (int i = 0; i < 2; i++) {
1634 bool delta_okay = i == 1;
1635 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001636 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001637 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001638 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001639 OmahaRequestParams::kOsVersion,
1640 "service_pack",
1641 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001642 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001643 "0.1.0.0",
1644 "en-US",
1645 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001646 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001647 "ChromeOSFirmware.1.0",
1648 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001649 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001650 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001651 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001652 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001653 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001654 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001655 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001656 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001657 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001658 metrics::CheckResult::kParsingError,
1659 metrics::CheckReaction::kUnset,
1660 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001661 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001662 &post_data));
1663 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001664 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001665 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1666 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001667 string::npos)
1668 << "i = " << i;
1669 }
1670}
1671
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001672TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001673 for (int i = 0; i < 2; i++) {
1674 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001675 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001676 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001677 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001678 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001679 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001680 OmahaRequestParams::kOsVersion,
1681 "service_pack",
1682 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001683 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001684 "0.1.0.0",
1685 "en-US",
1686 "unittest_track",
1687 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001688 "ChromeOSFirmware.1.0",
1689 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001690 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001691 interactive,
1692 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001693 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001694 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001695 "invalid xml>",
1696 -1,
1697 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001698 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001699 metrics::CheckResult::kParsingError,
1700 metrics::CheckReaction::kUnset,
1701 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001702 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001703 &post_data));
1704 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001705 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001706 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1707 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001708 string::npos)
1709 << "i = " << i;
1710 }
1711}
1712
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001713TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001714 OmahaEvent default_event;
1715 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1716 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001717 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001718
1719 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1720 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1721 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001722 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001723
1724 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1725 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001726 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001727 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1728 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001729 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001730}
1731
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001732void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001733 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001734 fake_system_state_.set_prefs(&prefs);
1735 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1736 .Times(AnyNumber());
1737 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1738 // Add a few hours to the day difference to test no rounding, etc.
1739 int64_t five_days_ago =
1740 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1741 int64_t six_days_ago =
1742 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1743 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001744 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001745 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001746 .WillOnce(DoAll(SetArgPointee<1>(six_days_ago), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001747 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001748 .WillOnce(DoAll(SetArgPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001749 brillo::Blob post_data;
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001750 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
1751 fake_update_response_.GetNoUpdateResponse(),
1752 -1,
1753 ping_only,
1754 ErrorCode::kSuccess,
1755 metrics::CheckResult::kNoUpdateAvailable,
1756 metrics::CheckReaction::kUnset,
1757 metrics::DownloadErrorCode::kUnset,
1758 nullptr,
1759 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001760 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001761 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1762 string::npos);
1763 if (ping_only) {
1764 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1765 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1766 } else {
1767 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1768 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001769 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001770}
1771
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001772TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1773 PingTest(true /* ping_only */);
1774}
1775
1776TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1777 PingTest(false /* ping_only */);
1778}
1779
1780TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001781 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001782 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001783 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1784 .Times(AnyNumber());
1785 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001786 int64_t three_days_ago =
1787 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1788 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001789 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001790 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001791 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001792 .WillOnce(DoAll(SetArgPointee<1>(three_days_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001793 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001794 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001795 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001796 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001797 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001798 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001799 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001800 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001801 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001802 metrics::CheckResult::kNoUpdateAvailable,
1803 metrics::CheckReaction::kUnset,
1804 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001805 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001806 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001807 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001808 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001809 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001810}
1811
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001812TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001813 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001814 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001815 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1816 .Times(AnyNumber());
1817 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001818 int64_t four_days_ago =
1819 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1820 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001821 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001822 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001823 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001824 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001825 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001826 .WillOnce(DoAll(SetArgPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001827 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001828 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001829 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001830 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001831 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001832 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001833 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001834 metrics::CheckResult::kNoUpdateAvailable,
1835 metrics::CheckReaction::kUnset,
1836 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001837 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001838 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001839 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001840 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001841 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001842}
1843
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001844TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001845 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001846 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001847 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1848 .Times(AnyNumber());
1849 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001850 int64_t one_hour_ago =
1851 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001852 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001853 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001854 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001855 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001856 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001857 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001858 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1859 // send a ping.
1860 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1861 .WillOnce(Return(true));
1862 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1863 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001864 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001865 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001866 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001867 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001868 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001869 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001870 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001871 metrics::CheckResult::kNoUpdateAvailable,
1872 metrics::CheckReaction::kUnset,
1873 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001874 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001875 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001876 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001877 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001878}
1879
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001880TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001881 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001882 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001883 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001884 int64_t now = Time::Now().ToInternalValue();
1885 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001886 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07001887 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001888 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07001889 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1890 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001891 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001892 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001893 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001894 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001895 -1,
1896 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001897 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001898 metrics::CheckResult::kUnset,
1899 metrics::CheckReaction::kUnset,
1900 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001901 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001902 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001903 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001904}
1905
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001906TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001907 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001908 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001909 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1910 .Times(AnyNumber());
1911 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001912 int64_t future =
1913 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001914 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001915 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001916 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001917 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001918 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001919 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001920 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1921 .WillOnce(Return(true));
1922 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1923 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001924 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001925 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001926 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001927 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1928 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001929 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001930 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001931 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001932 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001933 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001934 metrics::CheckResult::kNoUpdateAvailable,
1935 metrics::CheckReaction::kUnset,
1936 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001937 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001938 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001939 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001940 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001941}
1942
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001943TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001944 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001945 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001946 // may fail if it runs for longer than 5 seconds. It shouldn't run
1947 // that long though.
1948 int64_t midnight =
1949 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1950 int64_t midnight_slack =
1951 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001952 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001953 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001954 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1955 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001956 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1957 AllOf(Ge(midnight), Le(midnight_slack))))
1958 .WillOnce(Return(true));
1959 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1960 AllOf(Ge(midnight), Le(midnight_slack))))
1961 .WillOnce(Return(true));
1962 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001963 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001964 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1965 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001966 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001967 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001968 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001969 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001970 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001971 metrics::CheckResult::kNoUpdateAvailable,
1972 metrics::CheckReaction::kUnset,
1973 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001974 nullptr,
1975 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001976}
1977
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001978TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001979 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001980 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001981 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1982 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001983 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1984 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1985 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001986 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001987 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1988 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001989 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001990 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001991 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001992 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001993 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001994 metrics::CheckResult::kNoUpdateAvailable,
1995 metrics::CheckReaction::kUnset,
1996 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001997 nullptr,
1998 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001999}
2000
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002001TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002002 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002003 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002004 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2005 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002006 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2007 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2008 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002009 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002010 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2011 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002012 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002013 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002014 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002015 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002016 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002017 metrics::CheckResult::kNoUpdateAvailable,
2018 metrics::CheckReaction::kUnset,
2019 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002020 nullptr,
2021 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002022}
2023
Alex Deymob3fa53b2016-04-18 19:57:58 -07002024TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
2025 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2026 // the "eol" attribute.
2027 ASSERT_TRUE(
2028 TestUpdateCheck(nullptr, // request_params
2029 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2030 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2031 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2032 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2033 "</app></response>",
2034 -1,
2035 false, // ping_only
2036 ErrorCode::kSuccess,
2037 metrics::CheckResult::kNoUpdateAvailable,
2038 metrics::CheckReaction::kUnset,
2039 metrics::DownloadErrorCode::kUnset,
2040 nullptr,
2041 nullptr));
2042 string eol_pref;
2043 EXPECT_TRUE(
2044 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2045 // Note that the eol="eol" attribute should be ignored and the _eol should be
2046 // used instead.
2047 EXPECT_EQ("security-only", eol_pref);
2048}
2049
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002050TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002051 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002052 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07002053 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002054 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002055 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002056 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002057 metrics::CheckResult::kParsingError,
2058 metrics::CheckReaction::kUnset,
2059 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002060 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002061 &post_data));
2062 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002063 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002064 EXPECT_EQ(post_str.find("machineid="), string::npos);
2065 EXPECT_EQ(post_str.find("userid="), string::npos);
2066}
2067
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002068TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002069 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002070 const int http_error_code =
2071 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07002072 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002073 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002074 "",
2075 501,
Darin Petkov265f2902011-05-09 15:17:40 -07002076 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002077 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002078 metrics::CheckResult::kDownloadError,
2079 metrics::CheckReaction::kUnset,
2080 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07002081 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002082 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002083 EXPECT_FALSE(response.update_exists);
2084}
2085
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002086TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002087 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002088 const int http_error_code =
2089 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07002090 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002091 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002092 "",
2093 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07002094 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002095 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002096 metrics::CheckResult::kDownloadError,
2097 metrics::CheckReaction::kUnset,
2098 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07002099 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002100 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002101 EXPECT_FALSE(response.update_exists);
2102}
2103
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002104TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002105 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002106 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002107 params.set_wall_clock_based_wait_enabled(true);
2108 params.set_waiting_period(TimeDelta().FromDays(1));
2109 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002110
Sen Jiang7c1171e2016-06-23 11:35:40 -07002111 Time arbitrary_date;
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002112 ASSERT_TRUE(Time::FromString("6/4/1989", &arbitrary_date));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002113 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
2114 ASSERT_FALSE(TestUpdateCheck(&params,
2115 fake_update_response_.GetUpdateResponse(),
2116 -1,
2117 false, // ping_only
2118 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2119 metrics::CheckResult::kUpdateAvailable,
2120 metrics::CheckReaction::kDeferring,
2121 metrics::DownloadErrorCode::kUnset,
2122 &response,
2123 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002124
Ben Chan9abb7632014-08-07 00:10:53 -07002125 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002126 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002127 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002128 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002129
2130 // Verify if we are interactive check we don't defer.
2131 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002132 ASSERT_TRUE(TestUpdateCheck(&params,
2133 fake_update_response_.GetUpdateResponse(),
2134 -1,
2135 false, // ping_only
2136 ErrorCode::kSuccess,
2137 metrics::CheckResult::kUpdateAvailable,
2138 metrics::CheckReaction::kUpdating,
2139 metrics::DownloadErrorCode::kUnset,
2140 &response,
2141 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002142 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002143}
2144
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002145TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002146 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002147 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002148 params.set_wall_clock_based_wait_enabled(true);
2149 params.set_waiting_period(TimeDelta().FromDays(1));
2150 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002151
Sen Jiang7c1171e2016-06-23 11:35:40 -07002152 Time t1, t2;
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002153 ASSERT_TRUE(Time::FromString("1/1/2012", &t1));
2154 ASSERT_TRUE(Time::FromString("1/3/2012", &t2));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002155 ASSERT_TRUE(
2156 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2157 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2158 ASSERT_TRUE(TestUpdateCheck(&params,
2159 fake_update_response_.GetUpdateResponse(),
2160 -1,
2161 false, // ping_only
2162 ErrorCode::kSuccess,
2163 metrics::CheckResult::kUpdateAvailable,
2164 metrics::CheckReaction::kUpdating,
2165 metrics::DownloadErrorCode::kUnset,
2166 &response,
2167 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002168
2169 EXPECT_TRUE(response.update_exists);
2170
2171 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002172 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002173 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002174 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2175}
2176
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002177TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002178 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002179 base::ScopedTempDir tempdir;
2180 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002181
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002182 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002183 OmahaRequestParams params(&fake_system_state_);
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002184 params.set_root(tempdir.GetPath().value());
Sen Jiang297e5832016-03-17 14:45:51 -07002185 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2186 params.set_app_version("1.2.3.4");
Sen Jiang8cd42342018-01-31 12:06:59 -08002187 params.set_product_components("o.bundle=1");
Sen Jiang297e5832016-03-17 14:45:51 -07002188 params.set_current_channel("canary-channel");
2189 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2190 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002191 EXPECT_TRUE(params.to_more_stable_channel());
2192 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002193 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002194 "invalid xml>",
2195 -1,
2196 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002197 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002198 metrics::CheckResult::kParsingError,
2199 metrics::CheckReaction::kUnset,
2200 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002201 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002202 &post_data));
2203 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002204 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002205 EXPECT_NE(string::npos, post_str.find(
2206 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2207 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2208 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Sen Jiang8cd42342018-01-31 12:06:59 -08002209 EXPECT_EQ(string::npos, post_str.find("o.bundle"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002210}
2211
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002212TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002213 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002214 base::ScopedTempDir tempdir;
2215 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002216
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002217 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002218 OmahaRequestParams params(&fake_system_state_);
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002219 params.set_root(tempdir.GetPath().value());
Sen Jiang297e5832016-03-17 14:45:51 -07002220 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2221 params.set_app_version("5.6.7.8");
Sen Jiang8cd42342018-01-31 12:06:59 -08002222 params.set_product_components("o.bundle=1");
Sen Jiang297e5832016-03-17 14:45:51 -07002223 params.set_current_channel("stable-channel");
2224 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2225 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002226 EXPECT_FALSE(params.to_more_stable_channel());
2227 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002228 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002229 "invalid xml>",
2230 -1,
2231 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002232 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002233 metrics::CheckResult::kParsingError,
2234 metrics::CheckReaction::kUnset,
2235 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002236 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002237 &post_data));
2238 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002239 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002240 EXPECT_NE(string::npos, post_str.find(
2241 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2242 "version=\"5.6.7.8\" "
2243 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002244 EXPECT_EQ(string::npos, post_str.find("from_version"));
Sen Jiang8cd42342018-01-31 12:06:59 -08002245 EXPECT_NE(string::npos, post_str.find("o.bundle.version=\"1\""));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002246}
2247
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002248// Checks that the initial ping with a=-1 r=-1 is not send when the device
2249// was powerwashed.
2250TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2251 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2252
2253 // Flag that the device was powerwashed in the past.
2254 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2255
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002256 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002257 ASSERT_TRUE(
2258 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002259 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002260 -1,
2261 false, // ping_only
2262 ErrorCode::kSuccess,
2263 metrics::CheckResult::kNoUpdateAvailable,
2264 metrics::CheckReaction::kUnset,
2265 metrics::DownloadErrorCode::kUnset,
2266 nullptr,
2267 &post_data));
2268 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002269 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002270 EXPECT_EQ(string::npos, post_str.find("<ping"));
2271}
2272
Amin Hassani1677e812017-06-21 13:36:36 -07002273// Checks that the initial ping with a=-1 r=-1 is not send when the device
2274// first_active_omaha_ping_sent is set.
2275TEST_F(OmahaRequestActionTest, PingWhenFirstActiveOmahaPingIsSent) {
2276 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2277
2278 // Flag that the device was not powerwashed in the past.
2279 fake_system_state_.fake_hardware()->SetPowerwashCount(0);
2280
2281 // Flag that the device has sent first active ping in the past.
2282 fake_system_state_.fake_hardware()->SetFirstActiveOmahaPingSent();
2283
2284 brillo::Blob post_data;
2285 ASSERT_TRUE(
2286 TestUpdateCheck(nullptr, // request_params
2287 fake_update_response_.GetNoUpdateResponse(),
2288 -1,
2289 false, // ping_only
2290 ErrorCode::kSuccess,
2291 metrics::CheckResult::kNoUpdateAvailable,
2292 metrics::CheckReaction::kUnset,
2293 metrics::DownloadErrorCode::kUnset,
2294 nullptr,
2295 &post_data));
2296 // We shouldn't send a ping in this case since
2297 // first_active_omaha_ping_sent=true
2298 string post_str(post_data.begin(), post_data.end());
2299 EXPECT_EQ(string::npos, post_str.find("<ping"));
2300}
2301
Alex Deymo9fded1e2015-11-05 12:31:19 -08002302// Checks that the event 54 is sent on a reboot to a new update.
2303TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2304 // Flag that the device was updated in a previous boot.
2305 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2306
2307 brillo::Blob post_data;
2308 ASSERT_TRUE(
2309 TestUpdateCheck(nullptr, // request_params
2310 fake_update_response_.GetNoUpdateResponse(),
2311 -1,
2312 false, // ping_only
2313 ErrorCode::kSuccess,
2314 metrics::CheckResult::kNoUpdateAvailable,
2315 metrics::CheckReaction::kUnset,
2316 metrics::DownloadErrorCode::kUnset,
2317 nullptr,
2318 &post_data));
2319 string post_str(post_data.begin(), post_data.end());
2320
2321 // An event 54 is included and has the right version.
2322 EXPECT_NE(string::npos,
2323 post_str.find(base::StringPrintf(
2324 "<event eventtype=\"%d\"",
2325 OmahaEvent::kTypeRebootedAfterUpdate)));
2326 EXPECT_NE(string::npos,
2327 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2328
2329 // The previous version flag should have been removed.
2330 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2331 string prev_version;
2332 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2333 EXPECT_TRUE(prev_version.empty());
2334}
2335
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002336void OmahaRequestActionTest::P2PTest(
2337 bool initial_allow_p2p_for_downloading,
2338 bool initial_allow_p2p_for_sharing,
2339 bool omaha_disable_p2p_for_downloading,
2340 bool omaha_disable_p2p_for_sharing,
2341 bool payload_state_allow_p2p_attempt,
2342 bool expect_p2p_client_lookup,
2343 const string& p2p_client_result_url,
2344 bool expected_allow_p2p_for_downloading,
2345 bool expected_allow_p2p_for_sharing,
2346 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002347 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002348 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002349 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2350 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2351 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002352
2353 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002354 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002355 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2356 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002357 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2358 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2359 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2360 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2361 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2362 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2363 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2364 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2365 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2366 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2367
David Zeuthen8f191b22013-08-06 12:27:50 -07002368 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002369 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002370 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2371
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002372 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2373 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002374 .Times(expect_p2p_client_lookup ? 1 : 0);
2375
Alex Deymo8e18f932015-03-27 16:16:59 -07002376 fake_update_response_.disable_p2p_for_downloading =
2377 omaha_disable_p2p_for_downloading;
2378 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002379 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002380 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002381 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002382 -1,
2383 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002384 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002385 metrics::CheckResult::kUpdateAvailable,
2386 metrics::CheckReaction::kUpdating,
2387 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002388 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002389 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002390 EXPECT_TRUE(response.update_exists);
2391
Gilad Arnold74b5f552014-10-07 08:17:16 -07002392 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2393 response.disable_p2p_for_downloading);
2394 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2395 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002396
Gilad Arnold74b5f552014-10-07 08:17:16 -07002397 EXPECT_EQ(expected_allow_p2p_for_downloading,
2398 actual_allow_p2p_for_downloading);
2399 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2400 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002401}
2402
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002403TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002404 P2PTest(true, // initial_allow_p2p_for_downloading
2405 true, // initial_allow_p2p_for_sharing
2406 false, // omaha_disable_p2p_for_downloading
2407 false, // omaha_disable_p2p_for_sharing
2408 true, // payload_state_allow_p2p_attempt
2409 true, // expect_p2p_client_lookup
2410 "http://1.3.5.7/p2p", // p2p_client_result_url
2411 true, // expected_allow_p2p_for_downloading
2412 true, // expected_allow_p2p_for_sharing
2413 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002414}
2415
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002416TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002417 P2PTest(true, // initial_allow_p2p_for_downloading
2418 true, // initial_allow_p2p_for_sharing
2419 false, // omaha_disable_p2p_for_downloading
2420 false, // omaha_disable_p2p_for_sharing
2421 true, // payload_state_allow_p2p_attempt
2422 true, // expect_p2p_client_lookup
2423 "", // p2p_client_result_url
2424 false, // expected_allow_p2p_for_downloading
2425 true, // expected_allow_p2p_for_sharing
2426 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002427}
2428
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002429TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002430 P2PTest(false, // initial_allow_p2p_for_downloading
2431 true, // initial_allow_p2p_for_sharing
2432 false, // omaha_disable_p2p_for_downloading
2433 false, // omaha_disable_p2p_for_sharing
2434 true, // payload_state_allow_p2p_attempt
2435 false, // expect_p2p_client_lookup
2436 "unset", // p2p_client_result_url
2437 false, // expected_allow_p2p_for_downloading
2438 true, // expected_allow_p2p_for_sharing
2439 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002440}
2441
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002442TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002443 P2PTest(true, // initial_allow_p2p_for_downloading
2444 true, // initial_allow_p2p_for_sharing
2445 true, // omaha_disable_p2p_for_downloading
2446 false, // omaha_disable_p2p_for_sharing
2447 true, // payload_state_allow_p2p_attempt
2448 false, // expect_p2p_client_lookup
2449 "unset", // p2p_client_result_url
2450 false, // expected_allow_p2p_for_downloading
2451 true, // expected_allow_p2p_for_sharing
2452 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002453}
2454
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002455TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002456 P2PTest(true, // initial_allow_p2p_for_downloading
2457 true, // initial_allow_p2p_for_sharing
2458 false, // omaha_disable_p2p_for_downloading
2459 true, // omaha_disable_p2p_for_sharing
2460 true, // payload_state_allow_p2p_attempt
2461 true, // expect_p2p_client_lookup
2462 "http://1.3.5.7/p2p", // p2p_client_result_url
2463 true, // expected_allow_p2p_for_downloading
2464 false, // expected_allow_p2p_for_sharing
2465 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002466}
2467
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002468TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002469 P2PTest(true, // initial_allow_p2p_for_downloading
2470 true, // initial_allow_p2p_for_sharing
2471 true, // omaha_disable_p2p_for_downloading
2472 true, // omaha_disable_p2p_for_sharing
2473 true, // payload_state_allow_p2p_attempt
2474 false, // expect_p2p_client_lookup
2475 "unset", // p2p_client_result_url
2476 false, // expected_allow_p2p_for_downloading
2477 false, // expected_allow_p2p_for_sharing
2478 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002479}
2480
Alex Deymof329b932014-10-30 01:37:48 -07002481bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2482 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002483 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002484 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002485 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002486 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002487 -1,
2488 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002489 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002490 metrics::CheckResult::kUpdateAvailable,
2491 metrics::CheckReaction::kUpdating,
2492 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002493 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002494 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002495}
2496
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002497TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002498 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002499
Kevin Cernekee2494e282016-03-29 18:03:53 -07002500 // Simulate a successful update check that happens during OOBE. The
2501 // deadline in the response is needed to force the update attempt to
2502 // occur; responses without a deadline seen during OOBE will normally
2503 // return ErrorCode::kNonCriticalUpdateInOOBE.
2504 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2505 fake_update_response_.deadline = "20101020";
2506
David Zeuthen639aa362014-02-03 16:23:44 -08002507 // Check that we parse elapsed_days in the Omaha Response correctly.
2508 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002509 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2510 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002511 EXPECT_TRUE(response.update_exists);
2512 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002513 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002514 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002515 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002516 EXPECT_EQ(prefs_days, 42);
2517
2518 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002519 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002520 EXPECT_TRUE(response.update_exists);
2521 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002522 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002523 EXPECT_EQ(prefs_days, 42);
2524
2525 // Note that elapsed_days is not necessarily divisible by 7 so check
2526 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002527 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2528 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002529 EXPECT_TRUE(response.update_exists);
2530 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002531 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002532 EXPECT_EQ(prefs_days, 21);
2533
2534 // Check that we correctly handle elapsed_days not being included in
2535 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002536 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002537 EXPECT_TRUE(response.update_exists);
2538 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002539}
2540
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002541// If there is no prefs and OOBE is not complete, we should not
2542// report anything to Omaha.
2543TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002544 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002545 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2546 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2547}
David Zeuthen639aa362014-02-03 16:23:44 -08002548
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002549// If OOBE is complete and happened on a valid date (e.g. after Jan
2550// 1 2007 0:00 PST), that date should be used and written to
2551// prefs. However, first try with an invalid date and check we do
2552// nothing.
2553TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2554 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2555 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2556 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2557 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2558}
David Zeuthen639aa362014-02-03 16:23:44 -08002559
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002560// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2561// should yield an InstallDate of 14.
2562TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2563 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2564 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2565 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2566 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002567
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002568 int64_t prefs_days;
2569 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2570 EXPECT_EQ(prefs_days, 14);
2571}
David Zeuthen639aa362014-02-03 16:23:44 -08002572
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002573// Now that we have a valid date in prefs, check that we keep using
2574// that even if OOBE date reports something else. The date Jan 30,
2575// 2007 0:00 PST should yield an InstallDate of 28... but since
2576// there's a prefs file, we should still get 14.
2577TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2578 // Set a valid date in the prefs first.
2579 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002580
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002581 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2582 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2583 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002584
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002585 int64_t prefs_days;
2586 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2587 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002588
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002589 // If we delete the prefs file, we should get 28 days.
2590 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2591 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2592 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2593 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002594}
2595
Darin Petkov6a5b3222010-07-13 14:55:28 -07002596} // namespace chromeos_update_engine