blob: c513baf7b26ab9e91e92273a29d1f182607acd18 [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".
Sen Jiang42fa45e2018-03-12 11:02:14 -0700751 EXPECT_EQ("fg", fetcher->GetHeader("X-Goog-Update-Interactivity"));
752 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-Goog-Update-AppId"));
753 EXPECT_NE("", fetcher->GetHeader("X-Goog-Update-Updater"));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700754}
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
Weidong Guo421ff332017-04-17 10:08:38 -0700785TEST_F(OmahaRequestActionTest, ValidUpdateOverCellularAllowedByDevicePolicy) {
786 // This test tests that update over cellular is allowed as device policy
787 // says yes.
788 OmahaResponse response;
789 MockConnectionManager mock_cm;
790
791 fake_system_state_.set_connection_manager(&mock_cm);
792
793 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
794 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
795 SetArgPointee<1>(ConnectionTethering::kUnknown),
796 Return(true)));
797 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
798 .WillRepeatedly(Return(true));
799 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
800 .WillRepeatedly(Return(true));
801
802 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
803 fake_update_response_.GetUpdateResponse(),
804 -1,
805 false, // ping_only
806 ErrorCode::kSuccess,
807 metrics::CheckResult::kUpdateAvailable,
808 metrics::CheckReaction::kUpdating,
809 metrics::DownloadErrorCode::kUnset,
810 &response,
811 nullptr));
812 EXPECT_TRUE(response.update_exists);
813}
814
815TEST_F(OmahaRequestActionTest, ValidUpdateOverCellularBlockedByDevicePolicy) {
816 // This test tests that update over cellular is blocked as device policy
817 // says no.
818 OmahaResponse response;
819 MockConnectionManager mock_cm;
820
821 fake_system_state_.set_connection_manager(&mock_cm);
822
823 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
824 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
825 SetArgPointee<1>(ConnectionTethering::kUnknown),
826 Return(true)));
827 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
828 .WillRepeatedly(Return(true));
829 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
830 .WillRepeatedly(Return(false));
831
832 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
833 fake_update_response_.GetUpdateResponse(),
834 -1,
835 false, // ping_only
836 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
837 metrics::CheckResult::kUpdateAvailable,
838 metrics::CheckReaction::kIgnored,
839 metrics::DownloadErrorCode::kUnset,
840 &response,
841 nullptr));
842 EXPECT_FALSE(response.update_exists);
843}
844
845TEST_F(OmahaRequestActionTest,
846 ValidUpdateOverCellularAllowedByUserPermissionTrue) {
847 // This test tests that, when device policy is not set, update over cellular
848 // is allowed as permission for update over cellular is set to true.
849 OmahaResponse response;
850 MockConnectionManager mock_cm;
851
852 fake_prefs_.SetBoolean(kPrefsUpdateOverCellularPermission, true);
853 fake_system_state_.set_connection_manager(&mock_cm);
854
855 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
856 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
857 SetArgPointee<1>(ConnectionTethering::kUnknown),
858 Return(true)));
859 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
860 .WillRepeatedly(Return(false));
861 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
862 .WillRepeatedly(Return(true));
863
864 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
865 fake_update_response_.GetUpdateResponse(),
866 -1,
867 false, // ping_only
868 ErrorCode::kSuccess,
869 metrics::CheckResult::kUpdateAvailable,
870 metrics::CheckReaction::kUpdating,
871 metrics::DownloadErrorCode::kUnset,
872 &response,
873 nullptr));
874 EXPECT_TRUE(response.update_exists);
875}
876
877TEST_F(OmahaRequestActionTest,
878 ValidUpdateOverCellularBlockedByUpdateTargetNotMatch) {
879 // This test tests that, when device policy is not set and permission for
880 // update over cellular is set to false or does not exist, update over
881 // cellular is blocked as update target does not match the omaha response.
882 OmahaResponse response;
883 MockConnectionManager mock_cm;
884 // A version different from the version in omaha response.
885 string diff_version = "99.99.99";
886 // A size different from the size in omaha response.
887 int64_t diff_size = 999;
888
889 fake_prefs_.SetString(kPrefsUpdateOverCellularTargetVersion, diff_version);
890 fake_prefs_.SetInt64(kPrefsUpdateOverCellularTargetSize, diff_size);
891 // This test tests cellular (3G) being the only connection type being allowed.
892 fake_system_state_.set_connection_manager(&mock_cm);
893
894 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
895 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
896 SetArgPointee<1>(ConnectionTethering::kUnknown),
897 Return(true)));
898 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
899 .WillRepeatedly(Return(false));
900 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
901 .WillRepeatedly(Return(true));
902
903 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
904 fake_update_response_.GetUpdateResponse(),
905 -1,
906 false, // ping_only
907 ErrorCode::kOmahaUpdateIgnoredOverCellular,
908 metrics::CheckResult::kUpdateAvailable,
909 metrics::CheckReaction::kIgnored,
910 metrics::DownloadErrorCode::kUnset,
911 &response,
912 nullptr));
913 EXPECT_FALSE(response.update_exists);
914}
915
916TEST_F(OmahaRequestActionTest,
917 ValidUpdateOverCellularAllowedByUpdateTargetMatch) {
918 // This test tests that, when device policy is not set and permission for
919 // update over cellular is set to false or does not exist, update over
920 // cellular is allowed as update target matches the omaha response.
921 OmahaResponse response;
922 MockConnectionManager mock_cm;
923 // A version same as the version in omaha response.
924 string new_version = fake_update_response_.version;
925 // A size same as the size in omaha response.
926 int64_t new_size = fake_update_response_.size;
927
928 fake_prefs_.SetString(kPrefsUpdateOverCellularTargetVersion, new_version);
929 fake_prefs_.SetInt64(kPrefsUpdateOverCellularTargetSize, new_size);
930 fake_system_state_.set_connection_manager(&mock_cm);
931
932 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
933 .WillRepeatedly(DoAll(SetArgPointee<0>(ConnectionType::kCellular),
934 SetArgPointee<1>(ConnectionTethering::kUnknown),
935 Return(true)));
936 EXPECT_CALL(mock_cm, IsAllowedConnectionTypesForUpdateSet())
937 .WillRepeatedly(Return(false));
938 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kCellular, _))
939 .WillRepeatedly(Return(true));
940
941 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
942 fake_update_response_.GetUpdateResponse(),
943 -1,
944 false, // ping_only
945 ErrorCode::kSuccess,
946 metrics::CheckResult::kUpdateAvailable,
947 metrics::CheckReaction::kUpdating,
948 metrics::DownloadErrorCode::kUnset,
949 &response,
950 nullptr));
951 EXPECT_TRUE(response.update_exists);
952}
953
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700954TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700955 string rollback_version = "1234.0.0";
956 OmahaResponse response;
957
958 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700959 fake_system_state_.set_payload_state(&mock_payload_state);
960
Chris Sosa77f79e82014-06-02 18:16:24 -0700961 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
962 .WillRepeatedly(Return(rollback_version));
963
Alex Deymo8e18f932015-03-27 16:16:59 -0700964 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700965 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700966 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700967 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700968 -1,
969 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700970 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700971 metrics::CheckResult::kUpdateAvailable,
972 metrics::CheckReaction::kIgnored,
973 metrics::DownloadErrorCode::kUnset,
974 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700975 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700976 EXPECT_FALSE(response.update_exists);
977}
978
Kevin Cernekee2494e282016-03-29 18:03:53 -0700979// Verify that update checks called during OOBE will only try to download
980// an update if the response includes a non-empty deadline field.
981TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
982 OmahaResponse response;
983
Sen Jiang8cd42342018-01-31 12:06:59 -0800984 // TODO(senj): set better default value for metrics::checkresult in
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700985 // OmahaRequestAction::ActionCompleted.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700986 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700987 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
988 fake_update_response_.GetUpdateResponse(),
989 -1,
990 false, // ping_only
991 ErrorCode::kNonCriticalUpdateInOOBE,
992 metrics::CheckResult::kParsingError,
993 metrics::CheckReaction::kUnset,
994 metrics::DownloadErrorCode::kUnset,
995 &response,
996 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -0700997 EXPECT_FALSE(response.update_exists);
998
Alex Deymo46a9aae2016-05-04 20:20:11 -0700999 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
1000 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
1001 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
1002 fake_update_response_.GetUpdateResponse(),
1003 -1,
1004 false, // ping_only
1005 ErrorCode::kSuccess,
1006 metrics::CheckResult::kUpdateAvailable,
1007 metrics::CheckReaction::kUpdating,
1008 metrics::DownloadErrorCode::kUnset,
1009 &response,
1010 nullptr));
1011 EXPECT_TRUE(response.update_exists);
1012 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
1013
1014 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -07001015 fake_update_response_.deadline = "20101020";
1016 ASSERT_TRUE(
1017 TestUpdateCheck(nullptr, // request_params
1018 fake_update_response_.GetUpdateResponse(),
1019 -1,
1020 false, // ping_only
1021 ErrorCode::kSuccess,
1022 metrics::CheckResult::kUpdateAvailable,
1023 metrics::CheckReaction::kUpdating,
1024 metrics::DownloadErrorCode::kUnset,
1025 &response,
1026 nullptr));
1027 EXPECT_TRUE(response.update_exists);
1028}
1029
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001030TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001031 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001032 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001033 params.set_wall_clock_based_wait_enabled(true);
1034 params.set_update_check_count_wait_enabled(false);
1035 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001036
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001037 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001038 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001039 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001040 -1,
1041 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001042 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001043 metrics::CheckResult::kUpdateAvailable,
1044 metrics::CheckReaction::kDeferring,
1045 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001046 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001047 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001048 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001049
1050 // Verify if we are interactive check we don't defer.
1051 params.set_interactive(true);
1052 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001053 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001054 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001055 -1,
1056 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001057 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001058 metrics::CheckResult::kUpdateAvailable,
1059 metrics::CheckReaction::kUpdating,
1060 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001061 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001062 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001063 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001064}
1065
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001066TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001067 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001068 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001069 params.set_wall_clock_based_wait_enabled(false);
1070 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001071
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001072 params.set_update_check_count_wait_enabled(true);
1073 params.set_min_update_checks_needed(1);
1074 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001075
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001076 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001077 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001078 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001079 -1,
1080 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001081 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001082 metrics::CheckResult::kUpdateAvailable,
1083 metrics::CheckReaction::kUpdating,
1084 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001085 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001086 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001087 EXPECT_TRUE(response.update_exists);
1088}
1089
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001090TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001091 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001092 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001093 params.set_wall_clock_based_wait_enabled(true);
1094 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001095
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001096 params.set_update_check_count_wait_enabled(true);
1097 params.set_min_update_checks_needed(1);
1098 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001099
Alex Deymo8e18f932015-03-27 16:16:59 -07001100 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001101 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001102 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001103 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001104 -1,
1105 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001106 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001107 metrics::CheckResult::kUpdateAvailable,
1108 metrics::CheckReaction::kUpdating,
1109 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001110 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001111 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001112 EXPECT_TRUE(response.update_exists);
1113}
1114
1115
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001116TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001117 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001118 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001119 params.set_wall_clock_based_wait_enabled(true);
1120 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001121
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001122 params.set_update_check_count_wait_enabled(true);
1123 params.set_min_update_checks_needed(0);
1124 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001125
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001126 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001127 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001128 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001129 -1,
1130 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001131 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001132 metrics::CheckResult::kUpdateAvailable,
1133 metrics::CheckReaction::kUpdating,
1134 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001135 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001136 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001137
Ben Chan9abb7632014-08-07 00:10:53 -07001138 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001139 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001140 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001141 EXPECT_TRUE(response.update_exists);
1142}
1143
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001144TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001145 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001146 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001147 params.set_wall_clock_based_wait_enabled(true);
1148 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001149
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001150 params.set_update_check_count_wait_enabled(true);
1151 params.set_min_update_checks_needed(1);
1152 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001153
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001154 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001155 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001156 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001157 -1,
1158 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001159 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001160 metrics::CheckResult::kUpdateAvailable,
1161 metrics::CheckReaction::kDeferring,
1162 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001163 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001164 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001165
Ben Chan9abb7632014-08-07 00:10:53 -07001166 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001167 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001168 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001169 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001170
1171 // Verify if we are interactive check we don't defer.
1172 params.set_interactive(true);
1173 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001174 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001175 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001176 -1,
1177 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001178 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001179 metrics::CheckResult::kUpdateAvailable,
1180 metrics::CheckReaction::kUpdating,
1181 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001182 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001183 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001184 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001185}
1186
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001187TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001188 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001189 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001190 params.set_wall_clock_based_wait_enabled(true);
1191 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001192
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001193 params.set_update_check_count_wait_enabled(true);
1194 params.set_min_update_checks_needed(1);
1195 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001196
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001197 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001198
1199 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001200 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001201 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001202 -1,
1203 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001204 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001205 metrics::CheckResult::kUpdateAvailable,
1206 metrics::CheckReaction::kDeferring,
1207 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001208 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001209 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001210
Ben Chan9abb7632014-08-07 00:10:53 -07001211 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001212 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001213 // count remains the same, as the decrementing happens in update_attempter
1214 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001215 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001216 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001217
1218 // Verify if we are interactive check we don't defer.
1219 params.set_interactive(true);
1220 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001221 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001222 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001223 -1,
1224 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001225 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001226 metrics::CheckResult::kUpdateAvailable,
1227 metrics::CheckReaction::kUpdating,
1228 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001229 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001230 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001231 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001232}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001233
Alex Deymo8e18f932015-03-27 16:16:59 -07001234TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1235 OmahaResponse response;
1236 OmahaRequestParams params = request_params_;
1237 fake_update_response_.include_cohorts = true;
1238 fake_update_response_.cohort = "s/154454/8479665";
1239 fake_update_response_.cohorthint = "please-put-me-on-beta";
1240 fake_update_response_.cohortname = "stable";
1241
1242 ASSERT_TRUE(TestUpdateCheck(&params,
1243 fake_update_response_.GetUpdateResponse(),
1244 -1,
1245 false, // ping_only
1246 ErrorCode::kSuccess,
1247 metrics::CheckResult::kUpdateAvailable,
1248 metrics::CheckReaction::kUpdating,
1249 metrics::DownloadErrorCode::kUnset,
1250 &response,
1251 nullptr));
1252
1253 string value;
1254 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1255 EXPECT_EQ(fake_update_response_.cohort, value);
1256
1257 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1258 EXPECT_EQ(fake_update_response_.cohorthint, value);
1259
1260 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1261 EXPECT_EQ(fake_update_response_.cohortname, value);
1262}
1263
1264TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1265 OmahaResponse response;
1266 OmahaRequestParams params = request_params_;
1267 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1268 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1269 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1270 fake_update_response_.include_cohorts = true;
1271 fake_update_response_.cohort = "s/154454/8479665";
1272 fake_update_response_.cohorthint = "please-put-me-on-beta";
1273 fake_update_response_.cohortname = "";
1274
1275 ASSERT_TRUE(TestUpdateCheck(&params,
1276 fake_update_response_.GetUpdateResponse(),
1277 -1,
1278 false, // ping_only
1279 ErrorCode::kSuccess,
1280 metrics::CheckResult::kUpdateAvailable,
1281 metrics::CheckReaction::kUpdating,
1282 metrics::DownloadErrorCode::kUnset,
1283 &response,
1284 nullptr));
1285
1286 string value;
1287 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1288 EXPECT_EQ(fake_update_response_.cohort, value);
1289
1290 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1291 EXPECT_EQ(fake_update_response_.cohorthint, value);
1292
1293 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1294}
1295
1296TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1297 OmahaResponse response;
1298 OmahaRequestParams params = request_params_;
1299 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1300
1301 ASSERT_TRUE(TestUpdateCheck(&params,
1302 fake_update_response_.GetUpdateResponse(),
1303 -1,
1304 false, // ping_only
1305 ErrorCode::kSuccess,
1306 metrics::CheckResult::kUpdateAvailable,
1307 metrics::CheckReaction::kUpdating,
1308 metrics::DownloadErrorCode::kUnset,
1309 &response,
1310 nullptr));
1311
1312 string value;
1313 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1314 EXPECT_EQ("old_value", value);
1315
1316 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1317 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1318}
1319
Alex Deymo00d79ac2015-06-29 15:41:49 -07001320TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1321 OmahaResponse response;
1322 OmahaRequestParams params = request_params_;
1323 fake_update_response_.include_cohorts = true;
1324 fake_update_response_.cohort = "s/154454/8479665";
1325 fake_update_response_.cohorthint = "please-put-me-on-beta";
1326 fake_update_response_.cohortname = "stable";
1327
1328 ASSERT_TRUE(TestUpdateCheck(&params,
1329 fake_update_response_.GetNoUpdateResponse(),
1330 -1,
1331 false, // ping_only
1332 ErrorCode::kSuccess,
1333 metrics::CheckResult::kNoUpdateAvailable,
1334 metrics::CheckReaction::kUnset,
1335 metrics::DownloadErrorCode::kUnset,
1336 &response,
1337 nullptr));
1338
1339 string value;
1340 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1341 EXPECT_EQ(fake_update_response_.cohort, value);
1342
1343 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1344 EXPECT_EQ(fake_update_response_.cohorthint, value);
1345
1346 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1347 EXPECT_EQ(fake_update_response_.cohortname, value);
1348}
1349
Sen Jiangb1e063a2017-09-15 17:44:31 -07001350TEST_F(OmahaRequestActionTest, MultiAppCohortTest) {
1351 OmahaResponse response;
1352 OmahaRequestParams params = request_params_;
1353 fake_update_response_.multi_app = true;
1354 fake_update_response_.include_cohorts = true;
1355 fake_update_response_.cohort = "s/154454/8479665";
1356 fake_update_response_.cohorthint = "please-put-me-on-beta";
1357 fake_update_response_.cohortname = "stable";
1358
1359 ASSERT_TRUE(TestUpdateCheck(&params,
1360 fake_update_response_.GetUpdateResponse(),
1361 -1,
1362 false, // ping_only
1363 ErrorCode::kSuccess,
1364 metrics::CheckResult::kUpdateAvailable,
1365 metrics::CheckReaction::kUpdating,
1366 metrics::DownloadErrorCode::kUnset,
1367 &response,
1368 nullptr));
1369
1370 string value;
1371 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1372 EXPECT_EQ(fake_update_response_.cohort, value);
1373
1374 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1375 EXPECT_EQ(fake_update_response_.cohorthint, value);
1376
1377 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1378 EXPECT_EQ(fake_update_response_.cohortname, value);
1379}
1380
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001381TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001382 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001383
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001384 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001385 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001386
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001387 OmahaRequestParams params = request_params_;
1388 fake_system_state_.set_request_params(&params);
Ben Chan5c02c132017-06-27 07:10:36 -07001389 OmahaRequestAction action(
1390 &fake_system_state_,
1391 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001392 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001393 http_response.size(),
1394 nullptr),
1395 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001396 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001397 ActionProcessor processor;
1398 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001399 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001400
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001401 loop.PostTask(base::Bind(
1402 [](ActionProcessor* processor) { processor->StartProcessing(); },
1403 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001404 loop.Run();
1405 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001406 EXPECT_FALSE(processor.IsRunning());
1407}
1408
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001409TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001410 OmahaResponse response;
1411 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001412 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001413 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001414 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001415 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001416 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001417 metrics::CheckResult::kParsingError,
1418 metrics::CheckReaction::kUnset,
1419 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001420 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001421 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001422 EXPECT_FALSE(response.update_exists);
1423}
1424
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001425TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001426 OmahaResponse response;
1427 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001428 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001429 "",
1430 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001431 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001432 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001433 metrics::CheckResult::kParsingError,
1434 metrics::CheckReaction::kUnset,
1435 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001436 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001437 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001438 EXPECT_FALSE(response.update_exists);
1439}
1440
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001441TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001442 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001443 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001444 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001445 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1446 "<daystart elapsed_seconds=\"100\"/>"
1447 "<app appid=\"foo\" status=\"ok\">"
1448 "<ping status=\"ok\"/>"
1449 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001450 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001451 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001452 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001453 metrics::CheckResult::kParsingError,
1454 metrics::CheckReaction::kUnset,
1455 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001456 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001457 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001458 EXPECT_FALSE(response.update_exists);
1459}
1460
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001461TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001462 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001463 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001464 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001465 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1466 "<daystart elapsed_seconds=\"100\"/>"
1467 "<app appid=\"foo\" status=\"ok\">"
1468 "<ping status=\"ok\"/>"
1469 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001470 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001471 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001472 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001473 metrics::CheckResult::kParsingError,
1474 metrics::CheckReaction::kUnset,
1475 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001476 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001477 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001478 EXPECT_FALSE(response.update_exists);
1479}
1480
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001481TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001482 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001483 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001484 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001485 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1486 "<daystart elapsed_seconds=\"100\"/>"
1487 "<app appid=\"foo\" status=\"ok\">"
1488 "<ping status=\"ok\"/>"
1489 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001490 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001491 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001492 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001493 metrics::CheckResult::kParsingError,
1494 metrics::CheckReaction::kUnset,
1495 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001496 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001497 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001498 EXPECT_FALSE(response.update_exists);
1499}
1500
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001501TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001502 string input_response =
1503 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1504 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001505 // the appid needs to match that in the request params
1506 "<app appid=\"" +
1507 fake_update_response_.app_id +
1508 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001509 "<updatecheck status=\"ok\">"
1510 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001511 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001512 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001513 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001514 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001515 "ChromeOSVersion=\"10.2.3.4\" "
1516 "Prompt=\"false\" "
1517 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001518 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001519 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001520 "needsadmin=\"true\" "
1521 "/></actions></manifest></updatecheck></app></response>";
1522 LOG(INFO) << "Input Response = " << input_response;
1523
Darin Petkov6a5b3222010-07-13 14:55:28 -07001524 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001525 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001526 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001527 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001528 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001529 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001530 metrics::CheckResult::kUpdateAvailable,
1531 metrics::CheckReaction::kUpdating,
1532 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001533 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001534 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001535 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001536 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001537 EXPECT_EQ("http://missing/field/test/f",
1538 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001539 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001540 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1541 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001542 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001543 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001544}
1545
1546namespace {
1547class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1548 public:
1549 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001550 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001551 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001552};
1553
Alex Deymo60ca1a72015-06-18 18:19:15 -07001554void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001555 processor->StartProcessing();
1556 CHECK(processor->IsRunning());
1557 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001558}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001559} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001560
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001561TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001562 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001563 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001564
Alex Deymo60ca1a72015-06-18 18:19:15 -07001565 string http_response("doesn't matter");
Ben Chan5c02c132017-06-27 07:10:36 -07001566 OmahaRequestAction action(
1567 &fake_system_state_,
1568 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001569 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001570 http_response.size(),
1571 nullptr),
1572 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001573 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001574 ActionProcessor processor;
1575 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001576 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001577
Alex Deymo60ca1a72015-06-18 18:19:15 -07001578 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1579 loop.Run();
1580 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001581}
1582
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001583TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001584 string output;
1585 EXPECT_TRUE(XmlEncode("ab", &output));
1586 EXPECT_EQ("ab", output);
1587 EXPECT_TRUE(XmlEncode("a<b", &output));
1588 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001589 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1590 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001591 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1592 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001593 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001594 EXPECT_FALSE(XmlEncode("\xc2", &output));
1595 // Fail with invalid ASCII-7 chars.
1596 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1597}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001598
Alex Deymob0d74eb2015-03-30 17:59:17 -07001599TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1600 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1601 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1602}
1603
1604TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001605 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001606
1607 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001608 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001609 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001610 OmahaRequestParams::kOsVersion,
1611 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001612 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001613 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001614 "0.1.0.0",
1615 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001616 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001617 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001618 "ChromeOSFirmware.1.0",
1619 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001620 false, // delta okay
1621 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001622 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001623 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001624 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1625 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1626 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001627 base::JoinString(
1628 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001629 OmahaResponse response;
1630 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001631 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001632 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001633 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001634 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001635 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001636 metrics::CheckResult::kParsingError,
1637 metrics::CheckReaction::kUnset,
1638 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001639 &response,
1640 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001641 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001642 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001643 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1644 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1645 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1646 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1647 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1648 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1649 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1650 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1651 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1652 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1653 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1654 // Values from Prefs that are too big are removed from the XML instead of
1655 // encoded.
1656 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001657}
1658
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001659TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001660 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001661 fake_update_response_.deadline = "&lt;20110101";
1662 fake_update_response_.more_info_url = "testthe&lt;url";
1663 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001664 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001665 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001666 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001667 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001668 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001669 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001670 metrics::CheckResult::kUpdateAvailable,
1671 metrics::CheckReaction::kUpdating,
1672 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001673 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001674 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001675
Sen Jiang0affc2c2017-02-10 15:55:05 -08001676 EXPECT_EQ("testthe<url", response.more_info_url);
1677 EXPECT_EQ("testthe&codebase/file.signed",
1678 response.packages[0].payload_urls[0]);
1679 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001680}
1681
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001682TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001683 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001684 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001685 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001686 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001687 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001688 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001689 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001690 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001691 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001692 metrics::CheckResult::kUpdateAvailable,
1693 metrics::CheckReaction::kUpdating,
1694 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001695 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001696 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001697
Sen Jiang0affc2c2017-02-10 15:55:05 -08001698 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001699}
1700
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001701TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001702 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001703 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001704 fake_system_state_.set_prefs(&prefs);
1705
Darin Petkov95508da2011-01-05 12:42:29 -08001706 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001707 .WillOnce(DoAll(SetArgPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001708 // An existing but empty previous version means that we didn't reboot to a new
1709 // update, therefore, no need to update the previous version.
1710 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001711 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001712 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001713 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001714 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001715 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001716 metrics::CheckResult::kParsingError,
1717 metrics::CheckReaction::kUnset,
1718 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001719 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001720 &post_data));
1721 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001722 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001723 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001724 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001725 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001726 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001727 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1728 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001729 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1730 string::npos);
1731 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1732 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001733 // No <event> tag should be sent if we didn't reboot to an update.
1734 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001735}
1736
Jay Srinivasan0a708742012-03-20 11:26:12 -07001737
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001738TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001739 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001740 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001741 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1742 "invalid xml>",
1743 &post_data);
1744 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001745 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001746 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001747 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001748 OmahaEvent::kTypeUpdateDownloadStarted,
1749 OmahaEvent::kResultSuccess);
1750 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001751 EXPECT_EQ(post_str.find("ping"), string::npos);
1752 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001753}
1754
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001755TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001756 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001757 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001758 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1759 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001760 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001761 "invalid xml>",
1762 &post_data);
1763 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001764 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001765 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001766 " <event eventtype=\"%d\" eventresult=\"%d\" "
1767 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001768 OmahaEvent::kTypeDownloadComplete,
1769 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001770 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001771 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001772 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001773}
1774
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001775TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001776 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001777 // Create a copy of the OmahaRequestParams to reuse it later.
1778 OmahaRequestParams params = request_params_;
1779 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001780 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001781 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001782 nullptr,
Ben Chanab5a0af2017-10-12 14:57:50 -07001783 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001784 http_response.size(),
1785 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001786 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001787 EXPECT_FALSE(update_check_action.IsEvent());
1788
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001789 params = request_params_;
1790 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001791 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001792 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001793 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Ben Chanab5a0af2017-10-12 14:57:50 -07001794 std::make_unique<MockHttpFetcher>(http_response.data(),
Ben Chan5c02c132017-06-27 07:10:36 -07001795 http_response.size(),
1796 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001797 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001798 EXPECT_TRUE(event_action.IsEvent());
1799}
1800
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001801TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001802 for (int i = 0; i < 2; i++) {
1803 bool delta_okay = i == 1;
1804 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001805 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001806 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001807 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001808 OmahaRequestParams::kOsVersion,
1809 "service_pack",
1810 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001811 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001812 "0.1.0.0",
1813 "en-US",
1814 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001815 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001816 "ChromeOSFirmware.1.0",
1817 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001818 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001819 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001820 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001821 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001822 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001823 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001824 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001825 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001826 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001827 metrics::CheckResult::kParsingError,
1828 metrics::CheckReaction::kUnset,
1829 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001830 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001831 &post_data));
1832 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001833 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001834 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1835 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001836 string::npos)
1837 << "i = " << i;
1838 }
1839}
1840
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001841TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001842 for (int i = 0; i < 2; i++) {
1843 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001844 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001845 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001846 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001847 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001848 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001849 OmahaRequestParams::kOsVersion,
1850 "service_pack",
1851 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001852 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001853 "0.1.0.0",
1854 "en-US",
1855 "unittest_track",
1856 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001857 "ChromeOSFirmware.1.0",
1858 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001859 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001860 interactive,
1861 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001862 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001863 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001864 "invalid xml>",
1865 -1,
1866 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001867 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001868 metrics::CheckResult::kParsingError,
1869 metrics::CheckReaction::kUnset,
1870 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001871 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001872 &post_data));
1873 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001874 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001875 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1876 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001877 string::npos)
1878 << "i = " << i;
1879 }
1880}
1881
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001882TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001883 OmahaEvent default_event;
1884 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1885 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001886 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001887
1888 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1889 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1890 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001891 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001892
1893 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1894 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001895 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001896 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1897 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001898 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001899}
1900
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001901void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001902 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001903 fake_system_state_.set_prefs(&prefs);
1904 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1905 .Times(AnyNumber());
1906 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1907 // Add a few hours to the day difference to test no rounding, etc.
1908 int64_t five_days_ago =
1909 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1910 int64_t six_days_ago =
1911 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1912 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001913 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001914 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001915 .WillOnce(DoAll(SetArgPointee<1>(six_days_ago), Return(true)));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001916 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001917 .WillOnce(DoAll(SetArgPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001918 brillo::Blob post_data;
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001919 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
1920 fake_update_response_.GetNoUpdateResponse(),
1921 -1,
1922 ping_only,
1923 ErrorCode::kSuccess,
1924 metrics::CheckResult::kNoUpdateAvailable,
1925 metrics::CheckReaction::kUnset,
1926 metrics::DownloadErrorCode::kUnset,
1927 nullptr,
1928 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001929 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001930 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1931 string::npos);
1932 if (ping_only) {
1933 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1934 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1935 } else {
1936 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1937 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001938 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001939}
1940
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001941TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1942 PingTest(true /* ping_only */);
1943}
1944
1945TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1946 PingTest(false /* ping_only */);
1947}
1948
1949TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001950 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001951 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001952 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1953 .Times(AnyNumber());
1954 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001955 int64_t three_days_ago =
1956 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1957 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001958 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001959 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001960 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001961 .WillOnce(DoAll(SetArgPointee<1>(three_days_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001962 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001963 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001964 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001965 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001966 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001967 fake_update_response_.GetNoUpdateResponse(),
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,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001975 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001976 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001977 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001978 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001979}
1980
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001981TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001982 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001983 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001984 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1985 .Times(AnyNumber());
1986 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001987 int64_t four_days_ago =
1988 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1989 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001990 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001991 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001992 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001993 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001994 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07001995 .WillOnce(DoAll(SetArgPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001996 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001997 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001998 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001999 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07002000 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002001 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002002 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002003 metrics::CheckResult::kNoUpdateAvailable,
2004 metrics::CheckReaction::kUnset,
2005 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002006 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002007 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002008 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002009 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07002010 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002011}
2012
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002013TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002014 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002015 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002016 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
2017 .Times(AnyNumber());
2018 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002019 int64_t one_hour_ago =
2020 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08002021 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002022 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002023 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002024 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002025 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002026 .WillOnce(DoAll(SetArgPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002027 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
2028 // send a ping.
2029 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
2030 .WillOnce(Return(true));
2031 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
2032 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002033 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002034 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002035 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002036 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07002037 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002038 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002039 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002040 metrics::CheckResult::kNoUpdateAvailable,
2041 metrics::CheckReaction::kUnset,
2042 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002043 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002044 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002045 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002046 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002047}
2048
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002049TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07002050 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08002051 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002052 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07002053 int64_t now = Time::Now().ToInternalValue();
2054 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002055 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07002056 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002057 .WillOnce(DoAll(SetArgPointee<1>(now), Return(true)));
Thieu Leb44e9e82011-06-06 14:34:04 -07002058 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2059 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002060 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07002061 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002062 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002063 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07002064 -1,
2065 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002066 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002067 metrics::CheckResult::kUnset,
2068 metrics::CheckReaction::kUnset,
2069 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002070 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07002071 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08002072 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07002073}
2074
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002075TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002076 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002077 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002078 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
2079 .Times(AnyNumber());
2080 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002081 int64_t future =
2082 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08002083 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002084 .WillOnce(DoAll(SetArgPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002085 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002086 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002087 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
Ben Chan672c1f52017-10-23 15:41:39 -07002088 .WillOnce(DoAll(SetArgPointee<1>(future), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002089 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
2090 .WillOnce(Return(true));
2091 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
2092 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002093 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002094 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002095 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002096 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2097 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002098 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002099 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002100 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002101 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002102 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002103 metrics::CheckResult::kNoUpdateAvailable,
2104 metrics::CheckReaction::kUnset,
2105 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002106 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002107 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002108 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002109 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002110}
2111
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002112TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002113 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07002114 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002115 // may fail if it runs for longer than 5 seconds. It shouldn't run
2116 // that long though.
2117 int64_t midnight =
2118 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
2119 int64_t midnight_slack =
2120 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08002121 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002122 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002123 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2124 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002125 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
2126 AllOf(Ge(midnight), Le(midnight_slack))))
2127 .WillOnce(Return(true));
2128 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
2129 AllOf(Ge(midnight), Le(midnight_slack))))
2130 .WillOnce(Return(true));
2131 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002132 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002133 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2134 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002135 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002136 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002137 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002138 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002139 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002140 metrics::CheckResult::kNoUpdateAvailable,
2141 metrics::CheckReaction::kUnset,
2142 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002143 nullptr,
2144 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002145}
2146
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002147TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002148 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002149 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002150 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2151 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002152 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2153 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2154 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002155 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002156 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2157 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002158 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002159 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002160 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002161 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002162 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002163 metrics::CheckResult::kNoUpdateAvailable,
2164 metrics::CheckReaction::kUnset,
2165 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002166 nullptr,
2167 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002168}
2169
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002170TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002171 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002172 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002173 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2174 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002175 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2176 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2177 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002178 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002179 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2180 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002181 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002182 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002183 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002184 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002185 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002186 metrics::CheckResult::kNoUpdateAvailable,
2187 metrics::CheckReaction::kUnset,
2188 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002189 nullptr,
2190 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002191}
2192
Alex Deymob3fa53b2016-04-18 19:57:58 -07002193TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
2194 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2195 // the "eol" attribute.
2196 ASSERT_TRUE(
2197 TestUpdateCheck(nullptr, // request_params
2198 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2199 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2200 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2201 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2202 "</app></response>",
2203 -1,
2204 false, // ping_only
2205 ErrorCode::kSuccess,
2206 metrics::CheckResult::kNoUpdateAvailable,
2207 metrics::CheckReaction::kUnset,
2208 metrics::DownloadErrorCode::kUnset,
2209 nullptr,
2210 nullptr));
2211 string eol_pref;
2212 EXPECT_TRUE(
2213 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2214 // Note that the eol="eol" attribute should be ignored and the _eol should be
2215 // used instead.
2216 EXPECT_EQ("security-only", eol_pref);
2217}
2218
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002219TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002220 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002221 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07002222 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002223 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002224 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002225 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002226 metrics::CheckResult::kParsingError,
2227 metrics::CheckReaction::kUnset,
2228 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002229 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002230 &post_data));
2231 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002232 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002233 EXPECT_EQ(post_str.find("machineid="), string::npos);
2234 EXPECT_EQ(post_str.find("userid="), string::npos);
2235}
2236
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002237TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002238 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002239 const int http_error_code =
2240 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07002241 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002242 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002243 "",
2244 501,
Darin Petkov265f2902011-05-09 15:17:40 -07002245 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002246 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002247 metrics::CheckResult::kDownloadError,
2248 metrics::CheckReaction::kUnset,
2249 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07002250 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002251 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002252 EXPECT_FALSE(response.update_exists);
2253}
2254
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002255TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002256 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002257 const int http_error_code =
2258 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07002259 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002260 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002261 "",
2262 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07002263 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002264 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002265 metrics::CheckResult::kDownloadError,
2266 metrics::CheckReaction::kUnset,
2267 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07002268 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002269 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002270 EXPECT_FALSE(response.update_exists);
2271}
2272
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002273TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002274 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002275 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002276 params.set_wall_clock_based_wait_enabled(true);
2277 params.set_waiting_period(TimeDelta().FromDays(1));
2278 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002279
Sen Jiang7c1171e2016-06-23 11:35:40 -07002280 Time arbitrary_date;
Amin Hassani71818c82018-03-06 13:25:40 -08002281 Time::FromString("6/4/1989", &arbitrary_date);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002282 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
2283 ASSERT_FALSE(TestUpdateCheck(&params,
2284 fake_update_response_.GetUpdateResponse(),
2285 -1,
2286 false, // ping_only
2287 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2288 metrics::CheckResult::kUpdateAvailable,
2289 metrics::CheckReaction::kDeferring,
2290 metrics::DownloadErrorCode::kUnset,
2291 &response,
2292 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002293
Ben Chan9abb7632014-08-07 00:10:53 -07002294 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002295 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002296 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002297 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002298
2299 // Verify if we are interactive check we don't defer.
2300 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002301 ASSERT_TRUE(TestUpdateCheck(&params,
2302 fake_update_response_.GetUpdateResponse(),
2303 -1,
2304 false, // ping_only
2305 ErrorCode::kSuccess,
2306 metrics::CheckResult::kUpdateAvailable,
2307 metrics::CheckReaction::kUpdating,
2308 metrics::DownloadErrorCode::kUnset,
2309 &response,
2310 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002311 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002312}
2313
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002314TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002315 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002316 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002317 params.set_wall_clock_based_wait_enabled(true);
2318 params.set_waiting_period(TimeDelta().FromDays(1));
2319 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002320
Sen Jiang7c1171e2016-06-23 11:35:40 -07002321 Time t1, t2;
Amin Hassani71818c82018-03-06 13:25:40 -08002322 Time::FromString("1/1/2012", &t1);
2323 Time::FromString("1/3/2012", &t2);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002324 ASSERT_TRUE(
2325 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2326 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2327 ASSERT_TRUE(TestUpdateCheck(&params,
2328 fake_update_response_.GetUpdateResponse(),
2329 -1,
2330 false, // ping_only
2331 ErrorCode::kSuccess,
2332 metrics::CheckResult::kUpdateAvailable,
2333 metrics::CheckReaction::kUpdating,
2334 metrics::DownloadErrorCode::kUnset,
2335 &response,
2336 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002337
2338 EXPECT_TRUE(response.update_exists);
2339
2340 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002341 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002342 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002343 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2344}
2345
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002346TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002347 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002348 base::ScopedTempDir tempdir;
2349 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002350
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002351 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002352 OmahaRequestParams params(&fake_system_state_);
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002353 params.set_root(tempdir.GetPath().value());
Sen Jiang297e5832016-03-17 14:45:51 -07002354 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2355 params.set_app_version("1.2.3.4");
Sen Jiang8cd42342018-01-31 12:06:59 -08002356 params.set_product_components("o.bundle=1");
Sen Jiang297e5832016-03-17 14:45:51 -07002357 params.set_current_channel("canary-channel");
2358 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2359 params.UpdateDownloadChannel();
Sen Jiang8500d3a2018-02-08 12:04:05 -08002360 EXPECT_TRUE(params.ShouldPowerwash());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002361 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002362 "invalid xml>",
2363 -1,
2364 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002365 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002366 metrics::CheckResult::kParsingError,
2367 metrics::CheckReaction::kUnset,
2368 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002369 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002370 &post_data));
2371 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002372 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002373 EXPECT_NE(string::npos, post_str.find(
2374 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2375 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2376 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Sen Jiang8cd42342018-01-31 12:06:59 -08002377 EXPECT_EQ(string::npos, post_str.find("o.bundle"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002378}
2379
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002380TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002381 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002382 base::ScopedTempDir tempdir;
2383 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002384
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002385 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002386 OmahaRequestParams params(&fake_system_state_);
Hidehiko Abe2b9d2412017-12-13 18:56:18 +09002387 params.set_root(tempdir.GetPath().value());
Sen Jiang297e5832016-03-17 14:45:51 -07002388 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2389 params.set_app_version("5.6.7.8");
Sen Jiang8cd42342018-01-31 12:06:59 -08002390 params.set_product_components("o.bundle=1");
Sen Jiang297e5832016-03-17 14:45:51 -07002391 params.set_current_channel("stable-channel");
2392 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2393 params.UpdateDownloadChannel();
Sen Jiang8500d3a2018-02-08 12:04:05 -08002394 EXPECT_FALSE(params.ShouldPowerwash());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002395 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002396 "invalid xml>",
2397 -1,
2398 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002399 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002400 metrics::CheckResult::kParsingError,
2401 metrics::CheckReaction::kUnset,
2402 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002403 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002404 &post_data));
2405 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002406 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002407 EXPECT_NE(string::npos, post_str.find(
2408 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2409 "version=\"5.6.7.8\" "
2410 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002411 EXPECT_EQ(string::npos, post_str.find("from_version"));
Sen Jiang8cd42342018-01-31 12:06:59 -08002412 EXPECT_NE(string::npos, post_str.find("o.bundle.version=\"1\""));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002413}
2414
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002415// Checks that the initial ping with a=-1 r=-1 is not send when the device
2416// was powerwashed.
2417TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2418 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2419
2420 // Flag that the device was powerwashed in the past.
2421 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2422
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002423 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002424 ASSERT_TRUE(
2425 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002426 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002427 -1,
2428 false, // ping_only
2429 ErrorCode::kSuccess,
2430 metrics::CheckResult::kNoUpdateAvailable,
2431 metrics::CheckReaction::kUnset,
2432 metrics::DownloadErrorCode::kUnset,
2433 nullptr,
2434 &post_data));
2435 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002436 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002437 EXPECT_EQ(string::npos, post_str.find("<ping"));
2438}
2439
Amin Hassani1677e812017-06-21 13:36:36 -07002440// Checks that the initial ping with a=-1 r=-1 is not send when the device
2441// first_active_omaha_ping_sent is set.
2442TEST_F(OmahaRequestActionTest, PingWhenFirstActiveOmahaPingIsSent) {
2443 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2444
2445 // Flag that the device was not powerwashed in the past.
2446 fake_system_state_.fake_hardware()->SetPowerwashCount(0);
2447
2448 // Flag that the device has sent first active ping in the past.
2449 fake_system_state_.fake_hardware()->SetFirstActiveOmahaPingSent();
2450
2451 brillo::Blob post_data;
2452 ASSERT_TRUE(
2453 TestUpdateCheck(nullptr, // request_params
2454 fake_update_response_.GetNoUpdateResponse(),
2455 -1,
2456 false, // ping_only
2457 ErrorCode::kSuccess,
2458 metrics::CheckResult::kNoUpdateAvailable,
2459 metrics::CheckReaction::kUnset,
2460 metrics::DownloadErrorCode::kUnset,
2461 nullptr,
2462 &post_data));
2463 // We shouldn't send a ping in this case since
2464 // first_active_omaha_ping_sent=true
2465 string post_str(post_data.begin(), post_data.end());
2466 EXPECT_EQ(string::npos, post_str.find("<ping"));
2467}
2468
Alex Deymo9fded1e2015-11-05 12:31:19 -08002469// Checks that the event 54 is sent on a reboot to a new update.
2470TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2471 // Flag that the device was updated in a previous boot.
2472 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2473
2474 brillo::Blob post_data;
2475 ASSERT_TRUE(
2476 TestUpdateCheck(nullptr, // request_params
2477 fake_update_response_.GetNoUpdateResponse(),
2478 -1,
2479 false, // ping_only
2480 ErrorCode::kSuccess,
2481 metrics::CheckResult::kNoUpdateAvailable,
2482 metrics::CheckReaction::kUnset,
2483 metrics::DownloadErrorCode::kUnset,
2484 nullptr,
2485 &post_data));
2486 string post_str(post_data.begin(), post_data.end());
2487
2488 // An event 54 is included and has the right version.
2489 EXPECT_NE(string::npos,
2490 post_str.find(base::StringPrintf(
2491 "<event eventtype=\"%d\"",
2492 OmahaEvent::kTypeRebootedAfterUpdate)));
2493 EXPECT_NE(string::npos,
2494 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2495
2496 // The previous version flag should have been removed.
2497 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2498 string prev_version;
2499 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2500 EXPECT_TRUE(prev_version.empty());
2501}
2502
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002503void OmahaRequestActionTest::P2PTest(
2504 bool initial_allow_p2p_for_downloading,
2505 bool initial_allow_p2p_for_sharing,
2506 bool omaha_disable_p2p_for_downloading,
2507 bool omaha_disable_p2p_for_sharing,
2508 bool payload_state_allow_p2p_attempt,
2509 bool expect_p2p_client_lookup,
2510 const string& p2p_client_result_url,
2511 bool expected_allow_p2p_for_downloading,
2512 bool expected_allow_p2p_for_sharing,
2513 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002514 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002515 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002516 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2517 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2518 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002519
2520 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002521 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002522 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2523 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002524 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2525 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2526 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2527 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2528 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2529 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2530 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2531 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2532 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2533 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2534
David Zeuthen8f191b22013-08-06 12:27:50 -07002535 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002536 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002537 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2538
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002539 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2540 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002541 .Times(expect_p2p_client_lookup ? 1 : 0);
2542
Alex Deymo8e18f932015-03-27 16:16:59 -07002543 fake_update_response_.disable_p2p_for_downloading =
2544 omaha_disable_p2p_for_downloading;
2545 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002546 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002547 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002548 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002549 -1,
2550 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002551 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002552 metrics::CheckResult::kUpdateAvailable,
2553 metrics::CheckReaction::kUpdating,
2554 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002555 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002556 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002557 EXPECT_TRUE(response.update_exists);
2558
Gilad Arnold74b5f552014-10-07 08:17:16 -07002559 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2560 response.disable_p2p_for_downloading);
2561 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2562 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002563
Gilad Arnold74b5f552014-10-07 08:17:16 -07002564 EXPECT_EQ(expected_allow_p2p_for_downloading,
2565 actual_allow_p2p_for_downloading);
2566 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2567 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002568}
2569
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002570TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002571 P2PTest(true, // initial_allow_p2p_for_downloading
2572 true, // initial_allow_p2p_for_sharing
2573 false, // omaha_disable_p2p_for_downloading
2574 false, // omaha_disable_p2p_for_sharing
2575 true, // payload_state_allow_p2p_attempt
2576 true, // expect_p2p_client_lookup
2577 "http://1.3.5.7/p2p", // p2p_client_result_url
2578 true, // expected_allow_p2p_for_downloading
2579 true, // expected_allow_p2p_for_sharing
2580 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002581}
2582
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002583TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002584 P2PTest(true, // initial_allow_p2p_for_downloading
2585 true, // initial_allow_p2p_for_sharing
2586 false, // omaha_disable_p2p_for_downloading
2587 false, // omaha_disable_p2p_for_sharing
2588 true, // payload_state_allow_p2p_attempt
2589 true, // expect_p2p_client_lookup
2590 "", // p2p_client_result_url
2591 false, // expected_allow_p2p_for_downloading
2592 true, // expected_allow_p2p_for_sharing
2593 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002594}
2595
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002596TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002597 P2PTest(false, // initial_allow_p2p_for_downloading
2598 true, // initial_allow_p2p_for_sharing
2599 false, // omaha_disable_p2p_for_downloading
2600 false, // omaha_disable_p2p_for_sharing
2601 true, // payload_state_allow_p2p_attempt
2602 false, // expect_p2p_client_lookup
2603 "unset", // p2p_client_result_url
2604 false, // expected_allow_p2p_for_downloading
2605 true, // expected_allow_p2p_for_sharing
2606 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002607}
2608
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002609TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002610 P2PTest(true, // initial_allow_p2p_for_downloading
2611 true, // initial_allow_p2p_for_sharing
2612 true, // omaha_disable_p2p_for_downloading
2613 false, // omaha_disable_p2p_for_sharing
2614 true, // payload_state_allow_p2p_attempt
2615 false, // expect_p2p_client_lookup
2616 "unset", // p2p_client_result_url
2617 false, // expected_allow_p2p_for_downloading
2618 true, // expected_allow_p2p_for_sharing
2619 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002620}
2621
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002622TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002623 P2PTest(true, // initial_allow_p2p_for_downloading
2624 true, // initial_allow_p2p_for_sharing
2625 false, // omaha_disable_p2p_for_downloading
2626 true, // omaha_disable_p2p_for_sharing
2627 true, // payload_state_allow_p2p_attempt
2628 true, // expect_p2p_client_lookup
2629 "http://1.3.5.7/p2p", // p2p_client_result_url
2630 true, // expected_allow_p2p_for_downloading
2631 false, // expected_allow_p2p_for_sharing
2632 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002633}
2634
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002635TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002636 P2PTest(true, // initial_allow_p2p_for_downloading
2637 true, // initial_allow_p2p_for_sharing
2638 true, // omaha_disable_p2p_for_downloading
2639 true, // omaha_disable_p2p_for_sharing
2640 true, // payload_state_allow_p2p_attempt
2641 false, // expect_p2p_client_lookup
2642 "unset", // p2p_client_result_url
2643 false, // expected_allow_p2p_for_downloading
2644 false, // expected_allow_p2p_for_sharing
2645 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002646}
2647
Alex Deymof329b932014-10-30 01:37:48 -07002648bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2649 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002650 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002651 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002652 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002653 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002654 -1,
2655 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002656 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002657 metrics::CheckResult::kUpdateAvailable,
2658 metrics::CheckReaction::kUpdating,
2659 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002660 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002661 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002662}
2663
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002664TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002665 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002666
Kevin Cernekee2494e282016-03-29 18:03:53 -07002667 // Simulate a successful update check that happens during OOBE. The
2668 // deadline in the response is needed to force the update attempt to
2669 // occur; responses without a deadline seen during OOBE will normally
2670 // return ErrorCode::kNonCriticalUpdateInOOBE.
2671 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2672 fake_update_response_.deadline = "20101020";
2673
David Zeuthen639aa362014-02-03 16:23:44 -08002674 // Check that we parse elapsed_days in the Omaha Response correctly.
2675 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002676 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2677 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002678 EXPECT_TRUE(response.update_exists);
2679 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002680 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002681 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002682 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002683 EXPECT_EQ(prefs_days, 42);
2684
2685 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002686 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002687 EXPECT_TRUE(response.update_exists);
2688 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002689 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002690 EXPECT_EQ(prefs_days, 42);
2691
2692 // Note that elapsed_days is not necessarily divisible by 7 so check
2693 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002694 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2695 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002696 EXPECT_TRUE(response.update_exists);
2697 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002698 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002699 EXPECT_EQ(prefs_days, 21);
2700
2701 // Check that we correctly handle elapsed_days not being included in
2702 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002703 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002704 EXPECT_TRUE(response.update_exists);
2705 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002706}
2707
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002708// If there is no prefs and OOBE is not complete, we should not
2709// report anything to Omaha.
2710TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002711 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002712 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2713 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2714}
David Zeuthen639aa362014-02-03 16:23:44 -08002715
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002716// If OOBE is complete and happened on a valid date (e.g. after Jan
2717// 1 2007 0:00 PST), that date should be used and written to
2718// prefs. However, first try with an invalid date and check we do
2719// nothing.
2720TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2721 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2722 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2723 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2724 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2725}
David Zeuthen639aa362014-02-03 16:23:44 -08002726
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002727// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2728// should yield an InstallDate of 14.
2729TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2730 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2731 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2732 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2733 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002734
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002735 int64_t prefs_days;
2736 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2737 EXPECT_EQ(prefs_days, 14);
2738}
David Zeuthen639aa362014-02-03 16:23:44 -08002739
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002740// Now that we have a valid date in prefs, check that we keep using
2741// that even if OOBE date reports something else. The date Jan 30,
2742// 2007 0:00 PST should yield an InstallDate of 28... but since
2743// there's a prefs file, we should still get 14.
2744TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2745 // Set a valid date in the prefs first.
2746 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002747
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002748 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2749 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2750 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002751
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002752 int64_t prefs_days;
2753 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2754 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002755
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002756 // If we delete the prefs file, we should get 28 days.
2757 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2758 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2759 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2760 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002761}
2762
Darin Petkov6a5b3222010-07-13 14:55:28 -07002763} // namespace chromeos_update_engine