blob: 0398281360c3de39d758b0d0dde420a26c3e5a84 [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
Darin Petkov6a5b3222010-07-13 14:55:28 -070021#include <string>
22#include <vector>
Darin Petkov0dc8e9a2010-07-14 14:51:57 -070023
Alex Deymo60ca1a72015-06-18 18:19:15 -070024#include <base/bind.h>
Alex Deymo110e0302015-10-19 20:35:21 -070025#include <base/files/file_util.h>
Sen Jiang297e5832016-03-17 14:45:51 -070026#include <base/files/scoped_temp_dir.h>
Alex Deymo8e18f932015-03-27 16:16:59 -070027#include <base/strings/string_number_conversions.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070028#include <base/strings/string_util.h>
29#include <base/strings/stringprintf.h>
30#include <base/time/time.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070031#include <brillo/bind_lambda.h>
Alex Deymoc1c17b42015-11-23 03:53:15 -030032#include <brillo/make_unique_ptr.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070033#include <brillo/message_loops/fake_message_loop.h>
34#include <brillo/message_loops/message_loop.h>
35#include <brillo/message_loops/message_loop_utils.h>
Alex Deymoe1e3afe2014-10-30 13:02:49 -070036#include <gtest/gtest.h>
Jay Srinivasand29695d2013-04-08 15:08:05 -070037
Alex Deymo39910dc2015-11-09 17:04:30 -080038#include "update_engine/common/action_pipe.h"
39#include "update_engine/common/constants.h"
40#include "update_engine/common/fake_prefs.h"
41#include "update_engine/common/hash_calculator.h"
42#include "update_engine/common/mock_http_fetcher.h"
43#include "update_engine/common/platform_constants.h"
44#include "update_engine/common/prefs.h"
45#include "update_engine/common/test_utils.h"
Alex Deymoc1c17b42015-11-23 03:53:15 -030046#include "update_engine/fake_system_state.h"
Tianjie Xu282aa1f2017-09-05 13:42:45 -070047#include "update_engine/metrics_reporter_interface.h"
Chris Sosa77f79e82014-06-02 18:16:24 -070048#include "update_engine/mock_connection_manager.h"
Gilad Arnold74b5f552014-10-07 08:17:16 -070049#include "update_engine/mock_payload_state.h"
Darin Petkova4a8a8c2010-07-15 22:21:12 -070050#include "update_engine/omaha_request_params.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070051
Darin Petkov1cbd78f2010-07-29 12:38:34 -070052using base::Time;
53using base::TimeDelta;
Darin Petkov6a5b3222010-07-13 14:55:28 -070054using std::string;
55using std::vector;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070056using testing::AllOf;
Alex Deymof329b932014-10-30 01:37:48 -070057using testing::AnyNumber;
Jay Srinivasan34b5d862012-07-23 11:43:22 -070058using testing::DoAll;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070059using testing::Ge;
60using testing::Le;
Darin Petkov9c096d62010-11-17 14:49:04 -080061using testing::NiceMock;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070062using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070063using testing::ReturnPointee;
64using testing::SaveArg;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070065using testing::SetArgumentPointee;
Alex Deymof329b932014-10-30 01:37:48 -070066using testing::_;
Darin Petkov6a5b3222010-07-13 14:55:28 -070067
Alex Deymo8e18f932015-03-27 16:16:59 -070068namespace {
69
Alex Deymo85616652015-10-15 18:48:31 -070070const char kTestAppId[] = "test-app-id";
Aaron Wood7dcdedf2017-09-06 17:17:41 -070071const char kTestAppId2[] = "test-app2-id";
Alex Deymo85616652015-10-15 18:48:31 -070072
Alex Deymo8e18f932015-03-27 16:16:59 -070073// This is a helper struct to allow unit tests build an update response with the
74// values they care about.
75struct FakeUpdateResponse {
76 string GetNoUpdateResponse() const {
77 string entity_str;
78 if (include_entity)
79 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Sen Jiang81259682017-03-30 15:11:30 -070080 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" + entity_str +
81 "<response protocol=\"3.0\">"
82 "<daystart elapsed_seconds=\"100\"/>"
83 "<app appid=\"" +
84 app_id + "\" " +
85 (include_cohorts
86 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
87 "\" cohortname=\"" + cohortname + "\" "
88 : "") +
89 " status=\"ok\">"
90 "<ping status=\"ok\"/>"
91 "<updatecheck status=\"noupdate\"/></app>" +
92 (multi_app_no_update
Aaron Wood7dcdedf2017-09-06 17:17:41 -070093 ? "<app appid=\"" + app_id2 +
94 "\"><updatecheck status=\"noupdate\"/></app>"
Sen Jiang81259682017-03-30 15:11:30 -070095 : "") +
96 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -070097 }
98
99 string GetUpdateResponse() const {
Sen Jiang2703ef42017-03-16 13:36:21 -0700100 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
101 "protocol=\"3.0\">"
102 "<daystart elapsed_seconds=\"100\"" +
103 (elapsed_days.empty() ? ""
104 : (" elapsed_days=\"" + elapsed_days + "\"")) +
105 "/>"
106 "<app appid=\"" +
107 app_id + "\" " +
108 (include_cohorts
109 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
110 "\" cohortname=\"" + cohortname + "\" "
111 : "") +
112 " status=\"ok\">"
113 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
114 "<urls><url codebase=\"" +
115 codebase +
116 "\"/></urls>"
117 "<manifest version=\"" +
118 version +
119 "\">"
120 "<packages><package hash=\"not-used\" name=\"" +
121 filename + "\" size=\"" + base::Int64ToString(size) +
Sen Jiang0affc2c2017-02-10 15:55:05 -0800122 "\" hash_sha256=\"" + hash + "\"/>" +
123 (multi_package ? "<package name=\"package2\" size=\"222\" "
124 "hash_sha256=\"hash2\"/>"
125 : "") +
126 "</packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700127 "<actions><action event=\"postinstall\" MetadataSize=\"11" +
128 (multi_package ? ":22" : "") + "\" ChromeOSVersion=\"" + version +
129 "\" MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt +
Sen Jiang2703ef42017-03-16 13:36:21 -0700130 "\" "
131 "IsDelta=\"true\" "
Sen Jiangcdd52062017-05-18 15:33:10 -0700132 "IsDeltaPayload=\"true" +
133 (multi_package ? ":false" : "") +
134 "\" "
Sen Jiang2703ef42017-03-16 13:36:21 -0700135 "MaxDaysToScatter=\"" +
136 max_days_to_scatter +
137 "\" "
138 "sha256=\"not-used\" "
139 "needsadmin=\"" +
140 needsadmin + "\" " +
141 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
142 (disable_p2p_for_downloading ? "DisableP2PForDownloading=\"true\" "
143 : "") +
144 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
Sen Jiang81259682017-03-30 15:11:30 -0700145 "/></actions></manifest></updatecheck></app>" +
146 (multi_app
Sen Jiangb1e063a2017-09-15 17:44:31 -0700147 ? "<app appid=\"" + app_id2 + "\"" +
148 (include_cohorts ? " cohort=\"cohort2\"" : "") +
149 "><updatecheck status=\"ok\"><urls><url codebase=\"" +
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700150 codebase2 + "\"/></urls><manifest version=\"" + version2 +
151 "\"><packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700152 "<package name=\"package3\" size=\"333\" "
153 "hash_sha256=\"hash3\"/></packages>"
Sen Jiang00adf7b2017-06-26 15:57:29 -0700154 "<actions><action event=\"postinstall\" " +
155 (multi_app_self_update
156 ? "noupdate=\"true\" IsDeltaPayload=\"true\" "
157 : "IsDeltaPayload=\"false\" ") +
158 "MetadataSize=\"33\"/></actions>"
Sen Jiang81259682017-03-30 15:11:30 -0700159 "</manifest></updatecheck></app>"
160 : "") +
161 (multi_app_no_update
162 ? "<app><updatecheck status=\"noupdate\"/></app>"
163 : "") +
164 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700165 }
166
167 // Return the payload URL, which is split in two fields in the XML response.
168 string GetPayloadUrl() {
169 return codebase + filename;
170 }
171
Alex Deymo85616652015-10-15 18:48:31 -0700172 string app_id = kTestAppId;
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700173 string app_id2 = kTestAppId2;
Alex Deymo8e18f932015-03-27 16:16:59 -0700174 string version = "1.2.3.4";
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700175 string version2 = "2.3.4.5";
Alex Deymo8e18f932015-03-27 16:16:59 -0700176 string more_info_url = "http://more/info";
177 string prompt = "true";
178 string codebase = "http://code/base/";
Sen Jiang81259682017-03-30 15:11:30 -0700179 string codebase2 = "http://code/base/2/";
Alex Deymo8e18f932015-03-27 16:16:59 -0700180 string filename = "file.signed";
Sen Jiang2703ef42017-03-16 13:36:21 -0700181 string hash = "4841534831323334";
Alex Deymo8e18f932015-03-27 16:16:59 -0700182 string needsadmin = "false";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800183 uint64_t size = 123;
Alex Deymo8e18f932015-03-27 16:16:59 -0700184 string deadline = "";
185 string max_days_to_scatter = "7";
186 string elapsed_days = "42";
187
188 // P2P setting defaults to allowed.
189 bool disable_p2p_for_downloading = false;
190 bool disable_p2p_for_sharing = false;
191
192 // Omaha cohorts settings.
193 bool include_cohorts = false;
194 string cohort = "";
195 string cohorthint = "";
196 string cohortname = "";
197
198 // Whether to include the CrOS <!ENTITY> in the XML response.
199 bool include_entity = false;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800200
Sen Jiang81259682017-03-30 15:11:30 -0700201 // Whether to include more than one app.
202 bool multi_app = false;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700203 // Whether to include an app with noupdate="true".
204 bool multi_app_self_update = false;
205 // Whether to include an additional app with status="noupdate".
Sen Jiang81259682017-03-30 15:11:30 -0700206 bool multi_app_no_update = false;
207 // Whether to include more than one package in an app.
Sen Jiang0affc2c2017-02-10 15:55:05 -0800208 bool multi_package = false;
Alex Deymo8e18f932015-03-27 16:16:59 -0700209};
210
211} // namespace
212
Darin Petkov6a5b3222010-07-13 14:55:28 -0700213namespace chromeos_update_engine {
214
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700215class OmahaRequestActionTest : public ::testing::Test {
216 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800217 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700218 fake_system_state_.set_request_params(&request_params_);
219 fake_system_state_.set_prefs(&fake_prefs_);
220 }
221
222 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800223 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700224 // is used. |payload_state| may be null, in which case a local mock is used.
225 // |p2p_manager| may be null, in which case a local mock is used.
226 // |connection_manager| may be null, in which case a local mock is used.
227 // out_response may be null. If |fail_http_response_code| is non-negative,
228 // the transfer will fail with that code. |ping_only| is passed through to the
229 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
230 // post-data received by the mock HttpFetcher is returned.
231 //
232 // The |expected_check_result|, |expected_check_reaction| and
233 // |expected_error_code| parameters are for checking expectations
234 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
235 // UMA statistics. Use the appropriate ::kUnset value to specify that
236 // the given metric should not be reported.
237 bool TestUpdateCheck(OmahaRequestParams* request_params,
238 const string& http_response,
239 int fail_http_response_code,
240 bool ping_only,
241 ErrorCode expected_code,
242 metrics::CheckResult expected_check_result,
243 metrics::CheckReaction expected_check_reaction,
244 metrics::DownloadErrorCode expected_download_error_code,
245 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700246 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700247
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800248 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700249 // only a ping or also an updatecheck.
250 void PingTest(bool ping_only);
251
252 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700253 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700254 OmahaResponse *response);
255
256 // P2P test helper function.
257 void P2PTest(
258 bool initial_allow_p2p_for_downloading,
259 bool initial_allow_p2p_for_sharing,
260 bool omaha_disable_p2p_for_downloading,
261 bool omaha_disable_p2p_for_sharing,
262 bool payload_state_allow_p2p_attempt,
263 bool expect_p2p_client_lookup,
264 const string& p2p_client_result_url,
265 bool expected_allow_p2p_for_downloading,
266 bool expected_allow_p2p_for_sharing,
267 const string& expected_p2p_url);
268
269 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700270 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700271
272 // By default, all tests use these objects unless they replace them in the
273 // fake_system_state_.
274 OmahaRequestParams request_params_ = OmahaRequestParams{
275 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700276 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700277 OmahaRequestParams::kOsVersion,
278 "service_pack",
279 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700280 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700281 "0.1.0.0",
282 "en-US",
283 "unittest",
284 "OEM MODEL 09235 7471",
285 "ChromeOSFirmware.1.0",
286 "0X0A1",
287 false, // delta okay
288 false, // interactive
289 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700290 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700291
292 FakePrefs fake_prefs_;
293};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700294
295namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700296class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
297 public:
298 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700299 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800300 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700301 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800302 void ProcessingDone(const ActionProcessor* processor,
303 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700304 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700305 }
306
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800307 void ActionCompleted(ActionProcessor* processor,
308 AbstractAction* action,
309 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700310 // make sure actions always succeed
311 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700312 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700313 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700314 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700315 }
David Zeuthena99981f2013-04-29 13:42:47 -0700316 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700317};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700318} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700319
320class OutputObjectCollectorAction;
321
322template<>
323class ActionTraits<OutputObjectCollectorAction> {
324 public:
325 // Does not take an object for input
326 typedef OmahaResponse InputObjectType;
327 // On success, puts the output path on output
328 typedef NoneType OutputObjectType;
329};
330
331class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
332 public:
333 OutputObjectCollectorAction() : has_input_object_(false) {}
334 void PerformAction() {
335 // copy input object
336 has_input_object_ = HasInputObject();
337 if (has_input_object_)
338 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700339 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700340 }
341 // Should never be called
342 void TerminateProcessing() {
343 CHECK(false);
344 }
345 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700346 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700347 return "OutputObjectCollectorAction";
348 }
Alex Deymof329b932014-10-30 01:37:48 -0700349 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800350 using InputObjectType =
351 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
352 using OutputObjectType =
353 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700354 bool has_input_object_;
355 OmahaResponse omaha_response_;
356};
357
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700358bool OmahaRequestActionTest::TestUpdateCheck(
359 OmahaRequestParams* request_params,
360 const string& http_response,
361 int fail_http_response_code,
362 bool ping_only,
363 ErrorCode expected_code,
364 metrics::CheckResult expected_check_result,
365 metrics::CheckReaction expected_check_reaction,
366 metrics::DownloadErrorCode expected_download_error_code,
367 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700368 brillo::Blob* out_post_data) {
369 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700370 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700371 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800372 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700373 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700374 if (fail_http_response_code >= 0) {
375 fetcher->FailTransfer(fail_http_response_code);
376 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700377 if (request_params)
378 fake_system_state_.set_request_params(request_params);
379 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700380 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300381 brillo::make_unique_ptr(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700382 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700383 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700384 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700385
Darin Petkov6a5b3222010-07-13 14:55:28 -0700386 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700387 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700388 processor.EnqueueAction(&action);
389
390 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700391 BondActions(&action, &collector_action);
392 processor.EnqueueAction(&collector_action);
393
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700394 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
395 ReportUpdateCheckMetrics(_, _, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800396 .Times(AnyNumber());
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700397
398 EXPECT_CALL(*fake_system_state_.mock_metrics_reporter(),
399 ReportUpdateCheckMetrics(_,
400 expected_check_result,
401 expected_check_reaction,
402 expected_download_error_code))
403 .Times(ping_only ? 0 : 1);
David Zeuthen33bae492014-02-25 16:16:18 -0800404
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700405 loop.PostTask(base::Bind(
406 [](ActionProcessor* processor) { processor->StartProcessing(); },
407 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700408 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700409 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700410 if (collector_action.has_input_object_ && out_response)
411 *out_response = collector_action.omaha_response_;
412 if (out_post_data)
413 *out_post_data = fetcher->post_data();
414 return collector_action.has_input_object_;
415}
416
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700417// Tests Event requests -- they should always succeed. |out_post_data|
418// may be null; if non-null, the post-data received by the mock
419// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700420void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700421 OmahaEvent* event,
422 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700423 brillo::Blob* out_post_data) {
424 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700425 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700426 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800427 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700428 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700429 FakeSystemState fake_system_state;
430 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300431 OmahaRequestAction action(&fake_system_state,
432 event,
433 brillo::make_unique_ptr(fetcher),
434 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700435 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700436 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700437 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700438 processor.EnqueueAction(&action);
439
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700440 loop.PostTask(base::Bind(
441 [](ActionProcessor* processor) { processor->StartProcessing(); },
442 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700443 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700444 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700445
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700446 if (out_post_data)
447 *out_post_data = fetcher->post_data();
448}
449
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700450TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400451 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700452 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400453 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700454 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700455 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400456 -1,
457 false, // ping_only
458 ErrorCode::kOmahaRequestXMLHasEntityDecl,
459 metrics::CheckResult::kParsingError,
460 metrics::CheckReaction::kUnset,
461 metrics::DownloadErrorCode::kUnset,
462 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700463 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400464 EXPECT_FALSE(response.update_exists);
465}
466
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700467TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700468 OmahaResponse response;
469 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700470 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700471 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700472 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700473 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700474 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800475 metrics::CheckResult::kNoUpdateAvailable,
476 metrics::CheckReaction::kUnset,
477 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700478 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700479 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700480 EXPECT_FALSE(response.update_exists);
481}
482
Sen Jiang81259682017-03-30 15:11:30 -0700483TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
484 OmahaResponse response;
485 fake_update_response_.multi_app_no_update = true;
486 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
487 fake_update_response_.GetNoUpdateResponse(),
488 -1,
489 false, // ping_only
490 ErrorCode::kSuccess,
491 metrics::CheckResult::kNoUpdateAvailable,
492 metrics::CheckReaction::kUnset,
493 metrics::DownloadErrorCode::kUnset,
494 &response,
495 nullptr));
496 EXPECT_FALSE(response.update_exists);
497}
498
Sen Jiang00adf7b2017-06-26 15:57:29 -0700499TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
500 OmahaResponse response;
501 fake_update_response_.multi_app_no_update = true;
502 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
503 fake_update_response_.GetUpdateResponse(),
504 -1,
505 false, // ping_only
506 ErrorCode::kSuccess,
507 metrics::CheckResult::kNoUpdateAvailable,
508 metrics::CheckReaction::kUnset,
509 metrics::DownloadErrorCode::kUnset,
510 &response,
511 nullptr));
512 EXPECT_FALSE(response.update_exists);
513}
514
515TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
516 OmahaResponse response;
517 ASSERT_TRUE(TestUpdateCheck(
518 nullptr, // request_params
519 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
520 "event=\"postinstall\" noupdate=\"true\"/></actions>"
521 "</manifest></updatecheck></app></response>",
522 -1,
523 false, // ping_only
524 ErrorCode::kSuccess,
525 metrics::CheckResult::kNoUpdateAvailable,
526 metrics::CheckReaction::kUnset,
527 metrics::DownloadErrorCode::kUnset,
528 &response,
529 nullptr));
530 EXPECT_FALSE(response.update_exists);
531}
532
Alex Deymo8e18f932015-03-27 16:16:59 -0700533// Test that all the values in the response are parsed in a normal update
534// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700535TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700536 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700537 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700538 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700539 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700540 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700541 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700542 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700543 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800544 metrics::CheckResult::kUpdateAvailable,
545 metrics::CheckReaction::kUpdating,
546 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700547 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700548 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700549 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700550 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700551 EXPECT_EQ("", response.system_version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800552 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
553 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700554 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800555 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
556 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700557 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700558 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
559 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700560 // Omaha cohort attributes are not set in the response, so they should not be
Alex Deymo8e18f932015-03-27 16:16:59 -0700561 // persisted.
562 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
563 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
564 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700565}
566
Sen Jiang0affc2c2017-02-10 15:55:05 -0800567TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
568 OmahaResponse response;
569 fake_update_response_.multi_package = true;
570 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
571 fake_update_response_.GetUpdateResponse(),
572 -1,
573 false, // ping_only
574 ErrorCode::kSuccess,
575 metrics::CheckResult::kUpdateAvailable,
576 metrics::CheckReaction::kUpdating,
577 metrics::DownloadErrorCode::kUnset,
578 &response,
579 nullptr));
580 EXPECT_TRUE(response.update_exists);
581 EXPECT_EQ(fake_update_response_.version, response.version);
582 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
583 response.packages[0].payload_urls[0]);
584 EXPECT_EQ(fake_update_response_.codebase + "package2",
585 response.packages[1].payload_urls[0]);
586 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
587 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700588 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700589 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800590 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700591 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800592 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700593 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700594 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700595}
596
597TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
598 OmahaResponse response;
599 fake_update_response_.multi_app = true;
600 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
601 fake_update_response_.GetUpdateResponse(),
602 -1,
603 false, // ping_only
604 ErrorCode::kSuccess,
605 metrics::CheckResult::kUpdateAvailable,
606 metrics::CheckReaction::kUpdating,
607 metrics::DownloadErrorCode::kUnset,
608 &response,
609 nullptr));
610 EXPECT_TRUE(response.update_exists);
611 EXPECT_EQ(fake_update_response_.version, response.version);
612 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
613 response.packages[0].payload_urls[0]);
614 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
615 response.packages[1].payload_urls[0]);
616 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
617 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
618 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700619 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700620 ASSERT_EQ(2u, response.packages.size());
621 EXPECT_EQ(string("hash3"), response.packages[1].hash);
622 EXPECT_EQ(333u, response.packages[1].size);
623 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700624 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700625}
626
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700627TEST_F(OmahaRequestActionTest, MultiAppAndSystemUpdateTest) {
628 OmahaResponse response;
629 fake_update_response_.multi_app = true;
630 // trigger the lining up of the app and system versions
631 request_params_.set_system_app_id(fake_update_response_.app_id2);
632
633 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
634 fake_update_response_.GetUpdateResponse(),
635 -1,
636 false, // ping_only
637 ErrorCode::kSuccess,
638 metrics::CheckResult::kUpdateAvailable,
639 metrics::CheckReaction::kUpdating,
640 metrics::DownloadErrorCode::kUnset,
641 &response,
642 nullptr));
643 EXPECT_TRUE(response.update_exists);
644 EXPECT_EQ(fake_update_response_.version, response.version);
645 EXPECT_EQ(fake_update_response_.version2, response.system_version);
646 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
647 response.packages[0].payload_urls[0]);
648 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
649 response.packages[1].payload_urls[0]);
650 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
651 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
652 EXPECT_EQ(11u, response.packages[0].metadata_size);
653 EXPECT_EQ(true, response.packages[0].is_delta);
654 ASSERT_EQ(2u, response.packages.size());
655 EXPECT_EQ(string("hash3"), response.packages[1].hash);
656 EXPECT_EQ(333u, response.packages[1].size);
657 EXPECT_EQ(33u, response.packages[1].metadata_size);
658 EXPECT_EQ(false, response.packages[1].is_delta);
659}
660
Sen Jiang81259682017-03-30 15:11:30 -0700661TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
662 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700663 fake_update_response_.multi_app = true;
664 fake_update_response_.multi_app_self_update = true;
Sen Jiang81259682017-03-30 15:11:30 -0700665 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
666 fake_update_response_.GetUpdateResponse(),
667 -1,
668 false, // ping_only
669 ErrorCode::kSuccess,
670 metrics::CheckResult::kUpdateAvailable,
671 metrics::CheckReaction::kUpdating,
672 metrics::DownloadErrorCode::kUnset,
673 &response,
674 nullptr));
675 EXPECT_TRUE(response.update_exists);
676 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700677 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700678 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
679 response.packages[0].payload_urls[0]);
680 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
681 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
682 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700683 ASSERT_EQ(2u, response.packages.size());
684 EXPECT_EQ(string("hash3"), response.packages[1].hash);
685 EXPECT_EQ(333u, response.packages[1].size);
686 EXPECT_EQ(33u, response.packages[1].metadata_size);
687 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700688}
689
690TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
691 OmahaResponse response;
692 fake_update_response_.multi_app = true;
693 fake_update_response_.multi_package = true;
694 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
695 fake_update_response_.GetUpdateResponse(),
696 -1,
697 false, // ping_only
698 ErrorCode::kSuccess,
699 metrics::CheckResult::kUpdateAvailable,
700 metrics::CheckReaction::kUpdating,
701 metrics::DownloadErrorCode::kUnset,
702 &response,
703 nullptr));
704 EXPECT_TRUE(response.update_exists);
705 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700706 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700707 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
708 response.packages[0].payload_urls[0]);
709 EXPECT_EQ(fake_update_response_.codebase + "package2",
710 response.packages[1].payload_urls[0]);
711 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
712 response.packages[2].payload_urls[0]);
713 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
714 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
715 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700716 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700717 ASSERT_EQ(3u, response.packages.size());
718 EXPECT_EQ(string("hash2"), response.packages[1].hash);
719 EXPECT_EQ(222u, response.packages[1].size);
720 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700721 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700722 EXPECT_EQ(string("hash3"), response.packages[2].hash);
723 EXPECT_EQ(333u, response.packages[2].size);
724 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700725 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800726}
727
Alex Deymo14ad88e2016-06-29 12:30:14 -0700728TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
729 const string http_response = "<?xml invalid response";
730 request_params_.set_interactive(true);
731
732 brillo::FakeMessageLoop loop(nullptr);
733 loop.SetAsCurrent();
734
735 MockHttpFetcher* fetcher =
736 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
737 OmahaRequestAction action(
738 &fake_system_state_, nullptr, brillo::make_unique_ptr(fetcher), false);
739 ActionProcessor processor;
740 processor.EnqueueAction(&action);
741
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700742 loop.PostTask(base::Bind(
743 [](ActionProcessor* processor) { processor->StartProcessing(); },
744 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700745 loop.Run();
746 EXPECT_FALSE(loop.PendingTasks());
747
748 // Check that the headers were set in the fetcher during the action. Note that
749 // we set this request as "interactive".
750 EXPECT_EQ("fg", fetcher->GetHeader("X-GoogleUpdate-Interactivity"));
751 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-GoogleUpdate-AppId"));
752 EXPECT_NE("", fetcher->GetHeader("X-GoogleUpdate-Updater"));
753}
754
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700755TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700756 OmahaResponse response;
757 // Set up a connection manager that doesn't allow a valid update over
758 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700759 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700760 fake_system_state_.set_connection_manager(&mock_cm);
761
Alex Deymo30534502015-07-20 15:06:33 -0700762 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
763 .WillRepeatedly(
Sen Jiang255e22b2016-05-20 16:15:29 -0700764 DoAll(SetArgumentPointee<0>(ConnectionType::kEthernet),
765 SetArgumentPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700766 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700767 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
768 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700769
770 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700771 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700772 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700773 -1,
774 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700775 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700776 metrics::CheckResult::kUpdateAvailable,
777 metrics::CheckReaction::kIgnored,
778 metrics::DownloadErrorCode::kUnset,
779 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700780 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700781 EXPECT_FALSE(response.update_exists);
782}
783
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700784TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700785 string rollback_version = "1234.0.0";
786 OmahaResponse response;
787
788 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700789 fake_system_state_.set_payload_state(&mock_payload_state);
790
Chris Sosa77f79e82014-06-02 18:16:24 -0700791 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
792 .WillRepeatedly(Return(rollback_version));
793
Alex Deymo8e18f932015-03-27 16:16:59 -0700794 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700795 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700796 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700797 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700798 -1,
799 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700800 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700801 metrics::CheckResult::kUpdateAvailable,
802 metrics::CheckReaction::kIgnored,
803 metrics::DownloadErrorCode::kUnset,
804 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700805 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700806 EXPECT_FALSE(response.update_exists);
807}
808
Kevin Cernekee2494e282016-03-29 18:03:53 -0700809// Verify that update checks called during OOBE will only try to download
810// an update if the response includes a non-empty deadline field.
811TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
812 OmahaResponse response;
813
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700814 // TODO set better default value for metrics::checkresult in
815 // OmahaRequestAction::ActionCompleted.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700816 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Tianjie Xu282aa1f2017-09-05 13:42:45 -0700817 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
818 fake_update_response_.GetUpdateResponse(),
819 -1,
820 false, // ping_only
821 ErrorCode::kNonCriticalUpdateInOOBE,
822 metrics::CheckResult::kParsingError,
823 metrics::CheckReaction::kUnset,
824 metrics::DownloadErrorCode::kUnset,
825 &response,
826 nullptr));
Kevin Cernekee2494e282016-03-29 18:03:53 -0700827 EXPECT_FALSE(response.update_exists);
828
Alex Deymo46a9aae2016-05-04 20:20:11 -0700829 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
830 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
831 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
832 fake_update_response_.GetUpdateResponse(),
833 -1,
834 false, // ping_only
835 ErrorCode::kSuccess,
836 metrics::CheckResult::kUpdateAvailable,
837 metrics::CheckReaction::kUpdating,
838 metrics::DownloadErrorCode::kUnset,
839 &response,
840 nullptr));
841 EXPECT_TRUE(response.update_exists);
842 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
843
844 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700845 fake_update_response_.deadline = "20101020";
846 ASSERT_TRUE(
847 TestUpdateCheck(nullptr, // request_params
848 fake_update_response_.GetUpdateResponse(),
849 -1,
850 false, // ping_only
851 ErrorCode::kSuccess,
852 metrics::CheckResult::kUpdateAvailable,
853 metrics::CheckReaction::kUpdating,
854 metrics::DownloadErrorCode::kUnset,
855 &response,
856 nullptr));
857 EXPECT_TRUE(response.update_exists);
858}
859
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700860TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700861 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700862 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700863 params.set_wall_clock_based_wait_enabled(true);
864 params.set_update_check_count_wait_enabled(false);
865 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700866
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700867 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700868 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700869 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700870 -1,
871 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700872 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800873 metrics::CheckResult::kUpdateAvailable,
874 metrics::CheckReaction::kDeferring,
875 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700876 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700877 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700878 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700879
880 // Verify if we are interactive check we don't defer.
881 params.set_interactive(true);
882 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700883 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700884 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700885 -1,
886 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700887 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800888 metrics::CheckResult::kUpdateAvailable,
889 metrics::CheckReaction::kUpdating,
890 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700891 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700892 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700893 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700894}
895
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700896TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700897 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700898 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700899 params.set_wall_clock_based_wait_enabled(false);
900 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700901
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700902 params.set_update_check_count_wait_enabled(true);
903 params.set_min_update_checks_needed(1);
904 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700905
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700906 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700907 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700908 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700909 -1,
910 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700911 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800912 metrics::CheckResult::kUpdateAvailable,
913 metrics::CheckReaction::kUpdating,
914 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700915 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700916 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700917 EXPECT_TRUE(response.update_exists);
918}
919
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700920TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700921 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700922 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700923 params.set_wall_clock_based_wait_enabled(true);
924 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700925
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700926 params.set_update_check_count_wait_enabled(true);
927 params.set_min_update_checks_needed(1);
928 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700929
Alex Deymo8e18f932015-03-27 16:16:59 -0700930 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700931 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700932 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700933 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700934 -1,
935 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700936 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800937 metrics::CheckResult::kUpdateAvailable,
938 metrics::CheckReaction::kUpdating,
939 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700940 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700941 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700942 EXPECT_TRUE(response.update_exists);
943}
944
945
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700946TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700947 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700948 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700949 params.set_wall_clock_based_wait_enabled(true);
950 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700951
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700952 params.set_update_check_count_wait_enabled(true);
953 params.set_min_update_checks_needed(0);
954 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700955
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700956 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700957 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700958 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700959 -1,
960 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700961 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800962 metrics::CheckResult::kUpdateAvailable,
963 metrics::CheckReaction::kUpdating,
964 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700965 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700966 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700967
Ben Chan9abb7632014-08-07 00:10:53 -0700968 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700969 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700970 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700971 EXPECT_TRUE(response.update_exists);
972}
973
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700974TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700975 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700976 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700977 params.set_wall_clock_based_wait_enabled(true);
978 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700979
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700980 params.set_update_check_count_wait_enabled(true);
981 params.set_min_update_checks_needed(1);
982 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700983
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700984 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700985 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700986 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700987 -1,
988 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700989 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800990 metrics::CheckResult::kUpdateAvailable,
991 metrics::CheckReaction::kDeferring,
992 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700993 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700994 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700995
Ben Chan9abb7632014-08-07 00:10:53 -0700996 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700997 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700998 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700999 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001000
1001 // Verify if we are interactive check we don't defer.
1002 params.set_interactive(true);
1003 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001004 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001005 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001006 -1,
1007 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001008 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001009 metrics::CheckResult::kUpdateAvailable,
1010 metrics::CheckReaction::kUpdating,
1011 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001012 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001013 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001014 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001015}
1016
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001017TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001018 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001019 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001020 params.set_wall_clock_based_wait_enabled(true);
1021 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001022
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001023 params.set_update_check_count_wait_enabled(true);
1024 params.set_min_update_checks_needed(1);
1025 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001026
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001027 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001028
1029 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001030 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001031 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001032 -1,
1033 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001034 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001035 metrics::CheckResult::kUpdateAvailable,
1036 metrics::CheckReaction::kDeferring,
1037 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001038 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001039 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001040
Ben Chan9abb7632014-08-07 00:10:53 -07001041 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001042 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001043 // count remains the same, as the decrementing happens in update_attempter
1044 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001045 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001046 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001047
1048 // Verify if we are interactive check we don't defer.
1049 params.set_interactive(true);
1050 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001051 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001052 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001053 -1,
1054 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001055 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001056 metrics::CheckResult::kUpdateAvailable,
1057 metrics::CheckReaction::kUpdating,
1058 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001059 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001060 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001061 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001062}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001063
Alex Deymo8e18f932015-03-27 16:16:59 -07001064TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1065 OmahaResponse response;
1066 OmahaRequestParams params = request_params_;
1067 fake_update_response_.include_cohorts = true;
1068 fake_update_response_.cohort = "s/154454/8479665";
1069 fake_update_response_.cohorthint = "please-put-me-on-beta";
1070 fake_update_response_.cohortname = "stable";
1071
1072 ASSERT_TRUE(TestUpdateCheck(&params,
1073 fake_update_response_.GetUpdateResponse(),
1074 -1,
1075 false, // ping_only
1076 ErrorCode::kSuccess,
1077 metrics::CheckResult::kUpdateAvailable,
1078 metrics::CheckReaction::kUpdating,
1079 metrics::DownloadErrorCode::kUnset,
1080 &response,
1081 nullptr));
1082
1083 string value;
1084 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1085 EXPECT_EQ(fake_update_response_.cohort, value);
1086
1087 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1088 EXPECT_EQ(fake_update_response_.cohorthint, value);
1089
1090 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1091 EXPECT_EQ(fake_update_response_.cohortname, value);
1092}
1093
1094TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1095 OmahaResponse response;
1096 OmahaRequestParams params = request_params_;
1097 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1098 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1099 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1100 fake_update_response_.include_cohorts = true;
1101 fake_update_response_.cohort = "s/154454/8479665";
1102 fake_update_response_.cohorthint = "please-put-me-on-beta";
1103 fake_update_response_.cohortname = "";
1104
1105 ASSERT_TRUE(TestUpdateCheck(&params,
1106 fake_update_response_.GetUpdateResponse(),
1107 -1,
1108 false, // ping_only
1109 ErrorCode::kSuccess,
1110 metrics::CheckResult::kUpdateAvailable,
1111 metrics::CheckReaction::kUpdating,
1112 metrics::DownloadErrorCode::kUnset,
1113 &response,
1114 nullptr));
1115
1116 string value;
1117 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1118 EXPECT_EQ(fake_update_response_.cohort, value);
1119
1120 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1121 EXPECT_EQ(fake_update_response_.cohorthint, value);
1122
1123 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1124}
1125
1126TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1127 OmahaResponse response;
1128 OmahaRequestParams params = request_params_;
1129 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1130
1131 ASSERT_TRUE(TestUpdateCheck(&params,
1132 fake_update_response_.GetUpdateResponse(),
1133 -1,
1134 false, // ping_only
1135 ErrorCode::kSuccess,
1136 metrics::CheckResult::kUpdateAvailable,
1137 metrics::CheckReaction::kUpdating,
1138 metrics::DownloadErrorCode::kUnset,
1139 &response,
1140 nullptr));
1141
1142 string value;
1143 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1144 EXPECT_EQ("old_value", value);
1145
1146 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1147 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1148}
1149
Alex Deymo00d79ac2015-06-29 15:41:49 -07001150TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1151 OmahaResponse response;
1152 OmahaRequestParams params = request_params_;
1153 fake_update_response_.include_cohorts = true;
1154 fake_update_response_.cohort = "s/154454/8479665";
1155 fake_update_response_.cohorthint = "please-put-me-on-beta";
1156 fake_update_response_.cohortname = "stable";
1157
1158 ASSERT_TRUE(TestUpdateCheck(&params,
1159 fake_update_response_.GetNoUpdateResponse(),
1160 -1,
1161 false, // ping_only
1162 ErrorCode::kSuccess,
1163 metrics::CheckResult::kNoUpdateAvailable,
1164 metrics::CheckReaction::kUnset,
1165 metrics::DownloadErrorCode::kUnset,
1166 &response,
1167 nullptr));
1168
1169 string value;
1170 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1171 EXPECT_EQ(fake_update_response_.cohort, value);
1172
1173 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1174 EXPECT_EQ(fake_update_response_.cohorthint, value);
1175
1176 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1177 EXPECT_EQ(fake_update_response_.cohortname, value);
1178}
1179
Sen Jiangb1e063a2017-09-15 17:44:31 -07001180TEST_F(OmahaRequestActionTest, MultiAppCohortTest) {
1181 OmahaResponse response;
1182 OmahaRequestParams params = request_params_;
1183 fake_update_response_.multi_app = true;
1184 fake_update_response_.include_cohorts = true;
1185 fake_update_response_.cohort = "s/154454/8479665";
1186 fake_update_response_.cohorthint = "please-put-me-on-beta";
1187 fake_update_response_.cohortname = "stable";
1188
1189 ASSERT_TRUE(TestUpdateCheck(&params,
1190 fake_update_response_.GetUpdateResponse(),
1191 -1,
1192 false, // ping_only
1193 ErrorCode::kSuccess,
1194 metrics::CheckResult::kUpdateAvailable,
1195 metrics::CheckReaction::kUpdating,
1196 metrics::DownloadErrorCode::kUnset,
1197 &response,
1198 nullptr));
1199
1200 string value;
1201 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1202 EXPECT_EQ(fake_update_response_.cohort, value);
1203
1204 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1205 EXPECT_EQ(fake_update_response_.cohorthint, value);
1206
1207 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1208 EXPECT_EQ(fake_update_response_.cohortname, value);
1209}
1210
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001211TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001212 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001213
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001214 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001215 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001216
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001217 OmahaRequestParams params = request_params_;
1218 fake_system_state_.set_request_params(&params);
1219 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001220 brillo::make_unique_ptr(
1221 new MockHttpFetcher(http_response.data(),
1222 http_response.size(),
1223 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001224 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001225 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001226 ActionProcessor processor;
1227 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001228 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001229
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001230 loop.PostTask(base::Bind(
1231 [](ActionProcessor* processor) { processor->StartProcessing(); },
1232 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001233 loop.Run();
1234 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001235 EXPECT_FALSE(processor.IsRunning());
1236}
1237
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001238TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001239 OmahaResponse response;
1240 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001241 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001242 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001243 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001244 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001245 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001246 metrics::CheckResult::kParsingError,
1247 metrics::CheckReaction::kUnset,
1248 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001249 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001250 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001251 EXPECT_FALSE(response.update_exists);
1252}
1253
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001254TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001255 OmahaResponse response;
1256 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001257 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001258 "",
1259 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001260 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001261 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001262 metrics::CheckResult::kParsingError,
1263 metrics::CheckReaction::kUnset,
1264 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001265 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001266 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001267 EXPECT_FALSE(response.update_exists);
1268}
1269
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001270TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001271 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001272 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001273 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001274 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1275 "<daystart elapsed_seconds=\"100\"/>"
1276 "<app appid=\"foo\" status=\"ok\">"
1277 "<ping status=\"ok\"/>"
1278 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001279 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001280 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001281 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001282 metrics::CheckResult::kParsingError,
1283 metrics::CheckReaction::kUnset,
1284 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001285 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001286 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001287 EXPECT_FALSE(response.update_exists);
1288}
1289
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001290TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001291 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001292 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001293 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001294 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1295 "<daystart elapsed_seconds=\"100\"/>"
1296 "<app appid=\"foo\" status=\"ok\">"
1297 "<ping status=\"ok\"/>"
1298 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001299 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001300 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001301 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001302 metrics::CheckResult::kParsingError,
1303 metrics::CheckReaction::kUnset,
1304 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001305 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001306 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001307 EXPECT_FALSE(response.update_exists);
1308}
1309
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001310TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001311 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001312 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001313 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001314 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1315 "<daystart elapsed_seconds=\"100\"/>"
1316 "<app appid=\"foo\" status=\"ok\">"
1317 "<ping status=\"ok\"/>"
1318 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001319 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001320 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001321 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001322 metrics::CheckResult::kParsingError,
1323 metrics::CheckReaction::kUnset,
1324 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001325 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001326 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001327 EXPECT_FALSE(response.update_exists);
1328}
1329
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001330TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001331 string input_response =
1332 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1333 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001334 // the appid needs to match that in the request params
1335 "<app appid=\"" +
1336 fake_update_response_.app_id +
1337 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001338 "<updatecheck status=\"ok\">"
1339 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001340 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001341 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001342 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001343 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001344 "ChromeOSVersion=\"10.2.3.4\" "
1345 "Prompt=\"false\" "
1346 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001347 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001348 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001349 "needsadmin=\"true\" "
1350 "/></actions></manifest></updatecheck></app></response>";
1351 LOG(INFO) << "Input Response = " << input_response;
1352
Darin Petkov6a5b3222010-07-13 14:55:28 -07001353 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001354 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001355 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001356 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001357 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001358 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001359 metrics::CheckResult::kUpdateAvailable,
1360 metrics::CheckReaction::kUpdating,
1361 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001362 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001363 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001364 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001365 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001366 EXPECT_EQ("http://missing/field/test/f",
1367 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001368 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001369 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1370 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001371 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001372 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001373}
1374
1375namespace {
1376class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1377 public:
1378 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001379 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001380 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001381};
1382
Alex Deymo60ca1a72015-06-18 18:19:15 -07001383void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001384 processor->StartProcessing();
1385 CHECK(processor->IsRunning());
1386 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001387}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001388} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001389
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001390TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001391 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001392 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001393
Alex Deymo60ca1a72015-06-18 18:19:15 -07001394 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001395 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001396 brillo::make_unique_ptr(
1397 new MockHttpFetcher(http_response.data(),
1398 http_response.size(),
1399 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001400 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001401 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001402 ActionProcessor processor;
1403 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001404 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001405
Alex Deymo60ca1a72015-06-18 18:19:15 -07001406 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1407 loop.Run();
1408 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001409}
1410
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001411TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001412 string output;
1413 EXPECT_TRUE(XmlEncode("ab", &output));
1414 EXPECT_EQ("ab", output);
1415 EXPECT_TRUE(XmlEncode("a<b", &output));
1416 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001417 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1418 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001419 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1420 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001421 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001422 EXPECT_FALSE(XmlEncode("\xc2", &output));
1423 // Fail with invalid ASCII-7 chars.
1424 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1425}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001426
Alex Deymob0d74eb2015-03-30 17:59:17 -07001427TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1428 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1429 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1430}
1431
1432TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001433 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001434
1435 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001436 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001437 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001438 OmahaRequestParams::kOsVersion,
1439 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001440 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001441 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001442 "0.1.0.0",
1443 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001444 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001445 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001446 "ChromeOSFirmware.1.0",
1447 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001448 false, // delta okay
1449 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001450 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001451 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001452 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1453 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1454 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001455 base::JoinString(
1456 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001457 OmahaResponse response;
1458 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001459 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001460 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001461 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001462 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001463 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001464 metrics::CheckResult::kParsingError,
1465 metrics::CheckReaction::kUnset,
1466 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001467 &response,
1468 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001469 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001470 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001471 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1472 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1473 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1474 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1475 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1476 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1477 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1478 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1479 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1480 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1481 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1482 // Values from Prefs that are too big are removed from the XML instead of
1483 // encoded.
1484 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001485}
1486
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001487TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001488 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001489 fake_update_response_.deadline = "&lt;20110101";
1490 fake_update_response_.more_info_url = "testthe&lt;url";
1491 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001492 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001493 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001494 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001495 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001496 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001497 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001498 metrics::CheckResult::kUpdateAvailable,
1499 metrics::CheckReaction::kUpdating,
1500 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001501 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001502 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001503
Sen Jiang0affc2c2017-02-10 15:55:05 -08001504 EXPECT_EQ("testthe<url", response.more_info_url);
1505 EXPECT_EQ("testthe&codebase/file.signed",
1506 response.packages[0].payload_urls[0]);
1507 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001508}
1509
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001510TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001511 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001512 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001513 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001514 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001515 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001516 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001517 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001518 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001519 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001520 metrics::CheckResult::kUpdateAvailable,
1521 metrics::CheckReaction::kUpdating,
1522 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001523 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001524 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001525
Sen Jiang0affc2c2017-02-10 15:55:05 -08001526 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001527}
1528
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001529TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001530 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001531 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001532 fake_system_state_.set_prefs(&prefs);
1533
Darin Petkov95508da2011-01-05 12:42:29 -08001534 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1535 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001536 // An existing but empty previous version means that we didn't reboot to a new
1537 // update, therefore, no need to update the previous version.
1538 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001539 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001540 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001541 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001542 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001543 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001544 metrics::CheckResult::kParsingError,
1545 metrics::CheckReaction::kUnset,
1546 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001547 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001548 &post_data));
1549 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001550 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001551 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001552 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001553 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001554 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001555 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1556 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001557 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1558 string::npos);
1559 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1560 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001561 // No <event> tag should be sent if we didn't reboot to an update.
1562 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001563}
1564
Jay Srinivasan0a708742012-03-20 11:26:12 -07001565
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001566TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001567 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001568 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001569 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1570 "invalid xml>",
1571 &post_data);
1572 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001573 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001574 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001575 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001576 OmahaEvent::kTypeUpdateDownloadStarted,
1577 OmahaEvent::kResultSuccess);
1578 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001579 EXPECT_EQ(post_str.find("ping"), string::npos);
1580 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001581}
1582
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001583TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001584 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001585 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001586 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1587 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001588 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001589 "invalid xml>",
1590 &post_data);
1591 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001592 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001593 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001594 " <event eventtype=\"%d\" eventresult=\"%d\" "
1595 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001596 OmahaEvent::kTypeDownloadComplete,
1597 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001598 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001599 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001600 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001601}
1602
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001603TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001604 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001605 // Create a copy of the OmahaRequestParams to reuse it later.
1606 OmahaRequestParams params = request_params_;
1607 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001608 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001609 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001610 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001611 brillo::make_unique_ptr(
1612 new MockHttpFetcher(http_response.data(),
1613 http_response.size(),
1614 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001615 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001616 EXPECT_FALSE(update_check_action.IsEvent());
1617
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001618 params = request_params_;
1619 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001620 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001621 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001622 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001623 brillo::make_unique_ptr(
1624 new MockHttpFetcher(http_response.data(),
1625 http_response.size(),
1626 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001627 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001628 EXPECT_TRUE(event_action.IsEvent());
1629}
1630
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001631TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001632 for (int i = 0; i < 2; i++) {
1633 bool delta_okay = i == 1;
1634 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001635 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001636 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001637 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001638 OmahaRequestParams::kOsVersion,
1639 "service_pack",
1640 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001641 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001642 "0.1.0.0",
1643 "en-US",
1644 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001645 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001646 "ChromeOSFirmware.1.0",
1647 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001648 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001649 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001650 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001651 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001652 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001653 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001654 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001655 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001656 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001657 metrics::CheckResult::kParsingError,
1658 metrics::CheckReaction::kUnset,
1659 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001660 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001661 &post_data));
1662 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001663 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001664 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1665 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001666 string::npos)
1667 << "i = " << i;
1668 }
1669}
1670
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001671TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001672 for (int i = 0; i < 2; i++) {
1673 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001674 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001675 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001676 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001677 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001678 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001679 OmahaRequestParams::kOsVersion,
1680 "service_pack",
1681 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001682 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001683 "0.1.0.0",
1684 "en-US",
1685 "unittest_track",
1686 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001687 "ChromeOSFirmware.1.0",
1688 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001689 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001690 interactive,
1691 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001692 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001693 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001694 "invalid xml>",
1695 -1,
1696 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001697 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001698 metrics::CheckResult::kParsingError,
1699 metrics::CheckReaction::kUnset,
1700 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001701 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001702 &post_data));
1703 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001704 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001705 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1706 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001707 string::npos)
1708 << "i = " << i;
1709 }
1710}
1711
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001712TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001713 OmahaEvent default_event;
1714 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1715 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001716 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001717
1718 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1719 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1720 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001721 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001722
1723 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1724 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001725 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001726 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1727 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001728 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001729}
1730
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001731void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001732 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001733 fake_system_state_.set_prefs(&prefs);
1734 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1735 .Times(AnyNumber());
1736 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1737 // Add a few hours to the day difference to test no rounding, etc.
1738 int64_t five_days_ago =
1739 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1740 int64_t six_days_ago =
1741 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1742 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1743 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1744 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1745 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1746 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1747 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001748 brillo::Blob post_data;
Tianjie Xu282aa1f2017-09-05 13:42:45 -07001749 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
1750 fake_update_response_.GetNoUpdateResponse(),
1751 -1,
1752 ping_only,
1753 ErrorCode::kSuccess,
1754 metrics::CheckResult::kNoUpdateAvailable,
1755 metrics::CheckReaction::kUnset,
1756 metrics::DownloadErrorCode::kUnset,
1757 nullptr,
1758 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001759 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001760 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1761 string::npos);
1762 if (ping_only) {
1763 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1764 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1765 } else {
1766 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1767 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001768 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001769}
1770
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001771TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1772 PingTest(true /* ping_only */);
1773}
1774
1775TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1776 PingTest(false /* ping_only */);
1777}
1778
1779TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001780 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001781 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001782 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1783 .Times(AnyNumber());
1784 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001785 int64_t three_days_ago =
1786 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1787 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001788 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1789 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001790 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1791 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1792 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1793 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001794 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001795 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001796 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001797 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001798 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001799 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001800 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001801 metrics::CheckResult::kNoUpdateAvailable,
1802 metrics::CheckReaction::kUnset,
1803 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001804 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001805 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001806 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001807 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001808 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001809}
1810
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001811TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001812 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001813 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001814 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1815 .Times(AnyNumber());
1816 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001817 int64_t four_days_ago =
1818 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1819 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001820 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1821 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001822 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1823 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1824 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1825 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001826 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001827 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001828 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001829 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001830 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001831 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001832 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001833 metrics::CheckResult::kNoUpdateAvailable,
1834 metrics::CheckReaction::kUnset,
1835 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001836 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001837 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001838 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001839 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001840 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001841}
1842
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001843TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001844 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001845 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001846 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1847 .Times(AnyNumber());
1848 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001849 int64_t one_hour_ago =
1850 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001851 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1852 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001853 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1854 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1855 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1856 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001857 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1858 // send a ping.
1859 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1860 .WillOnce(Return(true));
1861 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1862 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001863 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001864 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001865 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001866 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001867 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001868 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001869 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001870 metrics::CheckResult::kNoUpdateAvailable,
1871 metrics::CheckReaction::kUnset,
1872 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001873 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001874 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001875 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001876 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001877}
1878
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001879TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001880 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001881 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001882 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001883 int64_t now = Time::Now().ToInternalValue();
1884 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1885 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1886 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1887 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1888 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1889 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001890 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001891 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001892 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001893 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001894 -1,
1895 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001896 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001897 metrics::CheckResult::kUnset,
1898 metrics::CheckReaction::kUnset,
1899 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001900 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001901 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001902 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001903}
1904
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001905TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001906 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001907 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001908 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1909 .Times(AnyNumber());
1910 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001911 int64_t future =
1912 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001913 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1914 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001915 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1916 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1917 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1918 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1919 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1920 .WillOnce(Return(true));
1921 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1922 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001923 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001924 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001925 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001926 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1927 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001928 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001929 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001930 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001931 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001932 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001933 metrics::CheckResult::kNoUpdateAvailable,
1934 metrics::CheckReaction::kUnset,
1935 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001936 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001937 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001938 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001939 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001940}
1941
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001942TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001943 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001944 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001945 // may fail if it runs for longer than 5 seconds. It shouldn't run
1946 // that long though.
1947 int64_t midnight =
1948 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1949 int64_t midnight_slack =
1950 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001951 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001952 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001953 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1954 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001955 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1956 AllOf(Ge(midnight), Le(midnight_slack))))
1957 .WillOnce(Return(true));
1958 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1959 AllOf(Ge(midnight), Le(midnight_slack))))
1960 .WillOnce(Return(true));
1961 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001962 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001963 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1964 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001965 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001966 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001967 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001968 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001969 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001970 metrics::CheckResult::kNoUpdateAvailable,
1971 metrics::CheckReaction::kUnset,
1972 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001973 nullptr,
1974 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001975}
1976
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001977TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001978 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001979 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001980 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1981 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001982 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1983 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1984 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001985 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001986 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1987 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001988 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001989 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001990 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001991 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001992 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001993 metrics::CheckResult::kNoUpdateAvailable,
1994 metrics::CheckReaction::kUnset,
1995 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001996 nullptr,
1997 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001998}
1999
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002000TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002001 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002002 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002003 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2004 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002005 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2006 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2007 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002008 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002009 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2010 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002011 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002012 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002013 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002014 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002015 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002016 metrics::CheckResult::kNoUpdateAvailable,
2017 metrics::CheckReaction::kUnset,
2018 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002019 nullptr,
2020 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002021}
2022
Alex Deymob3fa53b2016-04-18 19:57:58 -07002023TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
2024 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2025 // the "eol" attribute.
2026 ASSERT_TRUE(
2027 TestUpdateCheck(nullptr, // request_params
2028 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2029 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2030 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2031 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2032 "</app></response>",
2033 -1,
2034 false, // ping_only
2035 ErrorCode::kSuccess,
2036 metrics::CheckResult::kNoUpdateAvailable,
2037 metrics::CheckReaction::kUnset,
2038 metrics::DownloadErrorCode::kUnset,
2039 nullptr,
2040 nullptr));
2041 string eol_pref;
2042 EXPECT_TRUE(
2043 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2044 // Note that the eol="eol" attribute should be ignored and the _eol should be
2045 // used instead.
2046 EXPECT_EQ("security-only", eol_pref);
2047}
2048
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002049TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002050 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002051 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07002052 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002053 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002054 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002055 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002056 metrics::CheckResult::kParsingError,
2057 metrics::CheckReaction::kUnset,
2058 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002059 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002060 &post_data));
2061 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002062 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002063 EXPECT_EQ(post_str.find("machineid="), string::npos);
2064 EXPECT_EQ(post_str.find("userid="), string::npos);
2065}
2066
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002067TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002068 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002069 const int http_error_code =
2070 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07002071 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002072 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002073 "",
2074 501,
Darin Petkov265f2902011-05-09 15:17:40 -07002075 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002076 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002077 metrics::CheckResult::kDownloadError,
2078 metrics::CheckReaction::kUnset,
2079 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07002080 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002081 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002082 EXPECT_FALSE(response.update_exists);
2083}
2084
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002085TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002086 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002087 const int http_error_code =
2088 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07002089 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002090 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002091 "",
2092 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07002093 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002094 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002095 metrics::CheckResult::kDownloadError,
2096 metrics::CheckReaction::kUnset,
2097 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07002098 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002099 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002100 EXPECT_FALSE(response.update_exists);
2101}
2102
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002103TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002104 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002105 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002106 params.set_wall_clock_based_wait_enabled(true);
2107 params.set_waiting_period(TimeDelta().FromDays(1));
2108 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002109
Sen Jiang7c1171e2016-06-23 11:35:40 -07002110 Time arbitrary_date;
2111 Time::FromString("6/4/1989", &arbitrary_date);
2112 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
2113 ASSERT_FALSE(TestUpdateCheck(&params,
2114 fake_update_response_.GetUpdateResponse(),
2115 -1,
2116 false, // ping_only
2117 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2118 metrics::CheckResult::kUpdateAvailable,
2119 metrics::CheckReaction::kDeferring,
2120 metrics::DownloadErrorCode::kUnset,
2121 &response,
2122 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002123
Ben Chan9abb7632014-08-07 00:10:53 -07002124 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002125 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002126 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002127 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002128
2129 // Verify if we are interactive check we don't defer.
2130 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002131 ASSERT_TRUE(TestUpdateCheck(&params,
2132 fake_update_response_.GetUpdateResponse(),
2133 -1,
2134 false, // ping_only
2135 ErrorCode::kSuccess,
2136 metrics::CheckResult::kUpdateAvailable,
2137 metrics::CheckReaction::kUpdating,
2138 metrics::DownloadErrorCode::kUnset,
2139 &response,
2140 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002141 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002142}
2143
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002144TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002145 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002146 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002147 params.set_wall_clock_based_wait_enabled(true);
2148 params.set_waiting_period(TimeDelta().FromDays(1));
2149 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002150
Sen Jiang7c1171e2016-06-23 11:35:40 -07002151 Time t1, t2;
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002152 Time::FromString("1/1/2012", &t1);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002153 Time::FromString("1/3/2012", &t2);
2154 ASSERT_TRUE(
2155 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2156 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2157 ASSERT_TRUE(TestUpdateCheck(&params,
2158 fake_update_response_.GetUpdateResponse(),
2159 -1,
2160 false, // ping_only
2161 ErrorCode::kSuccess,
2162 metrics::CheckResult::kUpdateAvailable,
2163 metrics::CheckReaction::kUpdating,
2164 metrics::DownloadErrorCode::kUnset,
2165 &response,
2166 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002167
2168 EXPECT_TRUE(response.update_exists);
2169
2170 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002171 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002172 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002173 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2174}
2175
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002176TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002177 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002178 base::ScopedTempDir tempdir;
2179 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002180
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002181 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002182 OmahaRequestParams params(&fake_system_state_);
2183 params.set_root(tempdir.path().value());
2184 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2185 params.set_app_version("1.2.3.4");
2186 params.set_current_channel("canary-channel");
2187 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2188 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002189 EXPECT_TRUE(params.to_more_stable_channel());
2190 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002191 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002192 "invalid xml>",
2193 -1,
2194 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002195 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002196 metrics::CheckResult::kParsingError,
2197 metrics::CheckReaction::kUnset,
2198 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002199 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002200 &post_data));
2201 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002202 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002203 EXPECT_NE(string::npos, post_str.find(
2204 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2205 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2206 "track=\"stable-channel\" from_track=\"canary-channel\" "));
2207}
2208
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002209TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002210 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002211 base::ScopedTempDir tempdir;
2212 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002213
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002214 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002215 OmahaRequestParams params(&fake_system_state_);
2216 params.set_root(tempdir.path().value());
2217 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2218 params.set_app_version("5.6.7.8");
2219 params.set_current_channel("stable-channel");
2220 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2221 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002222 EXPECT_FALSE(params.to_more_stable_channel());
2223 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002224 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002225 "invalid xml>",
2226 -1,
2227 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002228 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002229 metrics::CheckResult::kParsingError,
2230 metrics::CheckReaction::kUnset,
2231 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002232 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002233 &post_data));
2234 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002235 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002236 EXPECT_NE(string::npos, post_str.find(
2237 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2238 "version=\"5.6.7.8\" "
2239 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002240 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002241}
2242
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002243// Checks that the initial ping with a=-1 r=-1 is not send when the device
2244// was powerwashed.
2245TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2246 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2247
2248 // Flag that the device was powerwashed in the past.
2249 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2250
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002251 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002252 ASSERT_TRUE(
2253 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002254 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002255 -1,
2256 false, // ping_only
2257 ErrorCode::kSuccess,
2258 metrics::CheckResult::kNoUpdateAvailable,
2259 metrics::CheckReaction::kUnset,
2260 metrics::DownloadErrorCode::kUnset,
2261 nullptr,
2262 &post_data));
2263 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002264 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002265 EXPECT_EQ(string::npos, post_str.find("<ping"));
2266}
2267
Alex Deymo9fded1e2015-11-05 12:31:19 -08002268// Checks that the event 54 is sent on a reboot to a new update.
2269TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2270 // Flag that the device was updated in a previous boot.
2271 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2272
2273 brillo::Blob post_data;
2274 ASSERT_TRUE(
2275 TestUpdateCheck(nullptr, // request_params
2276 fake_update_response_.GetNoUpdateResponse(),
2277 -1,
2278 false, // ping_only
2279 ErrorCode::kSuccess,
2280 metrics::CheckResult::kNoUpdateAvailable,
2281 metrics::CheckReaction::kUnset,
2282 metrics::DownloadErrorCode::kUnset,
2283 nullptr,
2284 &post_data));
2285 string post_str(post_data.begin(), post_data.end());
2286
2287 // An event 54 is included and has the right version.
2288 EXPECT_NE(string::npos,
2289 post_str.find(base::StringPrintf(
2290 "<event eventtype=\"%d\"",
2291 OmahaEvent::kTypeRebootedAfterUpdate)));
2292 EXPECT_NE(string::npos,
2293 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2294
2295 // The previous version flag should have been removed.
2296 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2297 string prev_version;
2298 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2299 EXPECT_TRUE(prev_version.empty());
2300}
2301
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002302void OmahaRequestActionTest::P2PTest(
2303 bool initial_allow_p2p_for_downloading,
2304 bool initial_allow_p2p_for_sharing,
2305 bool omaha_disable_p2p_for_downloading,
2306 bool omaha_disable_p2p_for_sharing,
2307 bool payload_state_allow_p2p_attempt,
2308 bool expect_p2p_client_lookup,
2309 const string& p2p_client_result_url,
2310 bool expected_allow_p2p_for_downloading,
2311 bool expected_allow_p2p_for_sharing,
2312 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002313 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002314 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002315 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2316 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2317 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002318
2319 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002320 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002321 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2322 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002323 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2324 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2325 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2326 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2327 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2328 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2329 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2330 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2331 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2332 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2333
David Zeuthen8f191b22013-08-06 12:27:50 -07002334 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002335 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002336 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2337
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002338 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2339 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002340 .Times(expect_p2p_client_lookup ? 1 : 0);
2341
Alex Deymo8e18f932015-03-27 16:16:59 -07002342 fake_update_response_.disable_p2p_for_downloading =
2343 omaha_disable_p2p_for_downloading;
2344 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002345 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002346 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002347 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002348 -1,
2349 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002350 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002351 metrics::CheckResult::kUpdateAvailable,
2352 metrics::CheckReaction::kUpdating,
2353 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002354 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002355 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002356 EXPECT_TRUE(response.update_exists);
2357
Gilad Arnold74b5f552014-10-07 08:17:16 -07002358 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2359 response.disable_p2p_for_downloading);
2360 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2361 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002362
Gilad Arnold74b5f552014-10-07 08:17:16 -07002363 EXPECT_EQ(expected_allow_p2p_for_downloading,
2364 actual_allow_p2p_for_downloading);
2365 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2366 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002367}
2368
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002369TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002370 P2PTest(true, // initial_allow_p2p_for_downloading
2371 true, // initial_allow_p2p_for_sharing
2372 false, // omaha_disable_p2p_for_downloading
2373 false, // omaha_disable_p2p_for_sharing
2374 true, // payload_state_allow_p2p_attempt
2375 true, // expect_p2p_client_lookup
2376 "http://1.3.5.7/p2p", // p2p_client_result_url
2377 true, // expected_allow_p2p_for_downloading
2378 true, // expected_allow_p2p_for_sharing
2379 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002380}
2381
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002382TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002383 P2PTest(true, // initial_allow_p2p_for_downloading
2384 true, // initial_allow_p2p_for_sharing
2385 false, // omaha_disable_p2p_for_downloading
2386 false, // omaha_disable_p2p_for_sharing
2387 true, // payload_state_allow_p2p_attempt
2388 true, // expect_p2p_client_lookup
2389 "", // p2p_client_result_url
2390 false, // expected_allow_p2p_for_downloading
2391 true, // expected_allow_p2p_for_sharing
2392 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002393}
2394
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002395TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002396 P2PTest(false, // initial_allow_p2p_for_downloading
2397 true, // initial_allow_p2p_for_sharing
2398 false, // omaha_disable_p2p_for_downloading
2399 false, // omaha_disable_p2p_for_sharing
2400 true, // payload_state_allow_p2p_attempt
2401 false, // expect_p2p_client_lookup
2402 "unset", // p2p_client_result_url
2403 false, // expected_allow_p2p_for_downloading
2404 true, // expected_allow_p2p_for_sharing
2405 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002406}
2407
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002408TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002409 P2PTest(true, // initial_allow_p2p_for_downloading
2410 true, // initial_allow_p2p_for_sharing
2411 true, // omaha_disable_p2p_for_downloading
2412 false, // omaha_disable_p2p_for_sharing
2413 true, // payload_state_allow_p2p_attempt
2414 false, // expect_p2p_client_lookup
2415 "unset", // p2p_client_result_url
2416 false, // expected_allow_p2p_for_downloading
2417 true, // expected_allow_p2p_for_sharing
2418 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002419}
2420
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002421TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002422 P2PTest(true, // initial_allow_p2p_for_downloading
2423 true, // initial_allow_p2p_for_sharing
2424 false, // omaha_disable_p2p_for_downloading
2425 true, // omaha_disable_p2p_for_sharing
2426 true, // payload_state_allow_p2p_attempt
2427 true, // expect_p2p_client_lookup
2428 "http://1.3.5.7/p2p", // p2p_client_result_url
2429 true, // expected_allow_p2p_for_downloading
2430 false, // expected_allow_p2p_for_sharing
2431 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002432}
2433
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002434TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002435 P2PTest(true, // initial_allow_p2p_for_downloading
2436 true, // initial_allow_p2p_for_sharing
2437 true, // omaha_disable_p2p_for_downloading
2438 true, // omaha_disable_p2p_for_sharing
2439 true, // payload_state_allow_p2p_attempt
2440 false, // expect_p2p_client_lookup
2441 "unset", // p2p_client_result_url
2442 false, // expected_allow_p2p_for_downloading
2443 false, // expected_allow_p2p_for_sharing
2444 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002445}
2446
Alex Deymof329b932014-10-30 01:37:48 -07002447bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2448 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002449 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002450 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002451 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002452 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002453 -1,
2454 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002455 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002456 metrics::CheckResult::kUpdateAvailable,
2457 metrics::CheckReaction::kUpdating,
2458 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002459 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002460 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002461}
2462
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002463TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002464 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002465
Kevin Cernekee2494e282016-03-29 18:03:53 -07002466 // Simulate a successful update check that happens during OOBE. The
2467 // deadline in the response is needed to force the update attempt to
2468 // occur; responses without a deadline seen during OOBE will normally
2469 // return ErrorCode::kNonCriticalUpdateInOOBE.
2470 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2471 fake_update_response_.deadline = "20101020";
2472
David Zeuthen639aa362014-02-03 16:23:44 -08002473 // Check that we parse elapsed_days in the Omaha Response correctly.
2474 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002475 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2476 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002477 EXPECT_TRUE(response.update_exists);
2478 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002479 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002480 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002481 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002482 EXPECT_EQ(prefs_days, 42);
2483
2484 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002485 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002486 EXPECT_TRUE(response.update_exists);
2487 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002488 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002489 EXPECT_EQ(prefs_days, 42);
2490
2491 // Note that elapsed_days is not necessarily divisible by 7 so check
2492 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002493 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2494 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002495 EXPECT_TRUE(response.update_exists);
2496 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002497 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002498 EXPECT_EQ(prefs_days, 21);
2499
2500 // Check that we correctly handle elapsed_days not being included in
2501 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002502 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002503 EXPECT_TRUE(response.update_exists);
2504 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002505}
2506
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002507// If there is no prefs and OOBE is not complete, we should not
2508// report anything to Omaha.
2509TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002510 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002511 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2512 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2513}
David Zeuthen639aa362014-02-03 16:23:44 -08002514
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002515// If OOBE is complete and happened on a valid date (e.g. after Jan
2516// 1 2007 0:00 PST), that date should be used and written to
2517// prefs. However, first try with an invalid date and check we do
2518// nothing.
2519TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2520 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2521 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2522 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2523 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2524}
David Zeuthen639aa362014-02-03 16:23:44 -08002525
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002526// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2527// should yield an InstallDate of 14.
2528TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2529 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2530 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2531 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2532 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002533
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002534 int64_t prefs_days;
2535 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2536 EXPECT_EQ(prefs_days, 14);
2537}
David Zeuthen639aa362014-02-03 16:23:44 -08002538
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002539// Now that we have a valid date in prefs, check that we keep using
2540// that even if OOBE date reports something else. The date Jan 30,
2541// 2007 0:00 PST should yield an InstallDate of 28... but since
2542// there's a prefs file, we should still get 14.
2543TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2544 // Set a valid date in the prefs first.
2545 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002546
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002547 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2548 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2549 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002550
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002551 int64_t prefs_days;
2552 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2553 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002554
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002555 // If we delete the prefs file, we should get 28 days.
2556 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2557 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2558 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2559 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002560}
2561
Darin Petkov6a5b3222010-07-13 14:55:28 -07002562} // namespace chromeos_update_engine