blob: 409165c46a409aa5e7807d8d64b4d3cf882ddf6e [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"
Alex Deymo38429cf2015-11-11 18:27:22 -080047#include "update_engine/metrics.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
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700394 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800395 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700396 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800397 SendEnumToUMA(metrics::kMetricCheckResult,
398 static_cast<int>(expected_check_result),
399 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
400 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700401 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800402 SendEnumToUMA(metrics::kMetricCheckReaction,
403 static_cast<int>(expected_check_reaction),
404 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
405 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700406 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700407 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
408 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800409 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
410 ? 0 : 1);
411
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700412 loop.PostTask(base::Bind(
413 [](ActionProcessor* processor) { processor->StartProcessing(); },
414 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700415 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700416 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700417 if (collector_action.has_input_object_ && out_response)
418 *out_response = collector_action.omaha_response_;
419 if (out_post_data)
420 *out_post_data = fetcher->post_data();
421 return collector_action.has_input_object_;
422}
423
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700424// Tests Event requests -- they should always succeed. |out_post_data|
425// may be null; if non-null, the post-data received by the mock
426// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700427void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700428 OmahaEvent* event,
429 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700430 brillo::Blob* out_post_data) {
431 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700432 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700433 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800434 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700435 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700436 FakeSystemState fake_system_state;
437 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300438 OmahaRequestAction action(&fake_system_state,
439 event,
440 brillo::make_unique_ptr(fetcher),
441 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700442 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700443 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700444 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700445 processor.EnqueueAction(&action);
446
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700447 loop.PostTask(base::Bind(
448 [](ActionProcessor* processor) { processor->StartProcessing(); },
449 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700450 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700451 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700452
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700453 if (out_post_data)
454 *out_post_data = fetcher->post_data();
455}
456
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700457TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400458 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700459 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400460 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700461 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700462 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400463 -1,
464 false, // ping_only
465 ErrorCode::kOmahaRequestXMLHasEntityDecl,
466 metrics::CheckResult::kParsingError,
467 metrics::CheckReaction::kUnset,
468 metrics::DownloadErrorCode::kUnset,
469 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700470 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400471 EXPECT_FALSE(response.update_exists);
472}
473
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700474TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700475 OmahaResponse response;
476 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700477 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700478 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700479 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700480 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700481 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800482 metrics::CheckResult::kNoUpdateAvailable,
483 metrics::CheckReaction::kUnset,
484 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700485 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700486 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700487 EXPECT_FALSE(response.update_exists);
488}
489
Sen Jiang81259682017-03-30 15:11:30 -0700490TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
491 OmahaResponse response;
492 fake_update_response_.multi_app_no_update = true;
493 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
494 fake_update_response_.GetNoUpdateResponse(),
495 -1,
496 false, // ping_only
497 ErrorCode::kSuccess,
498 metrics::CheckResult::kNoUpdateAvailable,
499 metrics::CheckReaction::kUnset,
500 metrics::DownloadErrorCode::kUnset,
501 &response,
502 nullptr));
503 EXPECT_FALSE(response.update_exists);
504}
505
Sen Jiang00adf7b2017-06-26 15:57:29 -0700506TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
507 OmahaResponse response;
508 fake_update_response_.multi_app_no_update = true;
509 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
510 fake_update_response_.GetUpdateResponse(),
511 -1,
512 false, // ping_only
513 ErrorCode::kSuccess,
514 metrics::CheckResult::kNoUpdateAvailable,
515 metrics::CheckReaction::kUnset,
516 metrics::DownloadErrorCode::kUnset,
517 &response,
518 nullptr));
519 EXPECT_FALSE(response.update_exists);
520}
521
522TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
523 OmahaResponse response;
524 ASSERT_TRUE(TestUpdateCheck(
525 nullptr, // request_params
526 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
527 "event=\"postinstall\" noupdate=\"true\"/></actions>"
528 "</manifest></updatecheck></app></response>",
529 -1,
530 false, // ping_only
531 ErrorCode::kSuccess,
532 metrics::CheckResult::kNoUpdateAvailable,
533 metrics::CheckReaction::kUnset,
534 metrics::DownloadErrorCode::kUnset,
535 &response,
536 nullptr));
537 EXPECT_FALSE(response.update_exists);
538}
539
Alex Deymo8e18f932015-03-27 16:16:59 -0700540// Test that all the values in the response are parsed in a normal update
541// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700542TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700543 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700544 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700545 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700546 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700547 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700548 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700549 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700550 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800551 metrics::CheckResult::kUpdateAvailable,
552 metrics::CheckReaction::kUpdating,
553 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700554 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700555 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700556 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700557 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700558 EXPECT_EQ("", response.system_version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800559 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
560 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700561 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800562 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
563 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700564 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700565 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
566 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700567 // Omaha cohort attributes are not set in the response, so they should not be
Alex Deymo8e18f932015-03-27 16:16:59 -0700568 // persisted.
569 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
570 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
571 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700572}
573
Sen Jiang0affc2c2017-02-10 15:55:05 -0800574TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
575 OmahaResponse response;
576 fake_update_response_.multi_package = true;
577 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
578 fake_update_response_.GetUpdateResponse(),
579 -1,
580 false, // ping_only
581 ErrorCode::kSuccess,
582 metrics::CheckResult::kUpdateAvailable,
583 metrics::CheckReaction::kUpdating,
584 metrics::DownloadErrorCode::kUnset,
585 &response,
586 nullptr));
587 EXPECT_TRUE(response.update_exists);
588 EXPECT_EQ(fake_update_response_.version, response.version);
589 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
590 response.packages[0].payload_urls[0]);
591 EXPECT_EQ(fake_update_response_.codebase + "package2",
592 response.packages[1].payload_urls[0]);
593 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
594 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700595 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700596 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800597 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700598 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800599 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700600 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700601 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700602}
603
604TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
605 OmahaResponse response;
606 fake_update_response_.multi_app = true;
607 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
608 fake_update_response_.GetUpdateResponse(),
609 -1,
610 false, // ping_only
611 ErrorCode::kSuccess,
612 metrics::CheckResult::kUpdateAvailable,
613 metrics::CheckReaction::kUpdating,
614 metrics::DownloadErrorCode::kUnset,
615 &response,
616 nullptr));
617 EXPECT_TRUE(response.update_exists);
618 EXPECT_EQ(fake_update_response_.version, response.version);
619 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
620 response.packages[0].payload_urls[0]);
621 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
622 response.packages[1].payload_urls[0]);
623 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
624 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
625 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700626 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700627 ASSERT_EQ(2u, response.packages.size());
628 EXPECT_EQ(string("hash3"), response.packages[1].hash);
629 EXPECT_EQ(333u, response.packages[1].size);
630 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700631 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700632}
633
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700634TEST_F(OmahaRequestActionTest, MultiAppAndSystemUpdateTest) {
635 OmahaResponse response;
636 fake_update_response_.multi_app = true;
637 // trigger the lining up of the app and system versions
638 request_params_.set_system_app_id(fake_update_response_.app_id2);
639
640 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
641 fake_update_response_.GetUpdateResponse(),
642 -1,
643 false, // ping_only
644 ErrorCode::kSuccess,
645 metrics::CheckResult::kUpdateAvailable,
646 metrics::CheckReaction::kUpdating,
647 metrics::DownloadErrorCode::kUnset,
648 &response,
649 nullptr));
650 EXPECT_TRUE(response.update_exists);
651 EXPECT_EQ(fake_update_response_.version, response.version);
652 EXPECT_EQ(fake_update_response_.version2, response.system_version);
653 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
654 response.packages[0].payload_urls[0]);
655 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
656 response.packages[1].payload_urls[0]);
657 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
658 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
659 EXPECT_EQ(11u, response.packages[0].metadata_size);
660 EXPECT_EQ(true, response.packages[0].is_delta);
661 ASSERT_EQ(2u, response.packages.size());
662 EXPECT_EQ(string("hash3"), response.packages[1].hash);
663 EXPECT_EQ(333u, response.packages[1].size);
664 EXPECT_EQ(33u, response.packages[1].metadata_size);
665 EXPECT_EQ(false, response.packages[1].is_delta);
666}
667
Sen Jiang81259682017-03-30 15:11:30 -0700668TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
669 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700670 fake_update_response_.multi_app = true;
671 fake_update_response_.multi_app_self_update = true;
Sen Jiang81259682017-03-30 15:11:30 -0700672 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
673 fake_update_response_.GetUpdateResponse(),
674 -1,
675 false, // ping_only
676 ErrorCode::kSuccess,
677 metrics::CheckResult::kUpdateAvailable,
678 metrics::CheckReaction::kUpdating,
679 metrics::DownloadErrorCode::kUnset,
680 &response,
681 nullptr));
682 EXPECT_TRUE(response.update_exists);
683 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700684 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700685 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
686 response.packages[0].payload_urls[0]);
687 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
688 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
689 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700690 ASSERT_EQ(2u, response.packages.size());
691 EXPECT_EQ(string("hash3"), response.packages[1].hash);
692 EXPECT_EQ(333u, response.packages[1].size);
693 EXPECT_EQ(33u, response.packages[1].metadata_size);
694 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700695}
696
697TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
698 OmahaResponse response;
699 fake_update_response_.multi_app = true;
700 fake_update_response_.multi_package = true;
701 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
702 fake_update_response_.GetUpdateResponse(),
703 -1,
704 false, // ping_only
705 ErrorCode::kSuccess,
706 metrics::CheckResult::kUpdateAvailable,
707 metrics::CheckReaction::kUpdating,
708 metrics::DownloadErrorCode::kUnset,
709 &response,
710 nullptr));
711 EXPECT_TRUE(response.update_exists);
712 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700713 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700714 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
715 response.packages[0].payload_urls[0]);
716 EXPECT_EQ(fake_update_response_.codebase + "package2",
717 response.packages[1].payload_urls[0]);
718 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
719 response.packages[2].payload_urls[0]);
720 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
721 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
722 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700723 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700724 ASSERT_EQ(3u, response.packages.size());
725 EXPECT_EQ(string("hash2"), response.packages[1].hash);
726 EXPECT_EQ(222u, response.packages[1].size);
727 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700728 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700729 EXPECT_EQ(string("hash3"), response.packages[2].hash);
730 EXPECT_EQ(333u, response.packages[2].size);
731 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700732 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800733}
734
Alex Deymo14ad88e2016-06-29 12:30:14 -0700735TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
736 const string http_response = "<?xml invalid response";
737 request_params_.set_interactive(true);
738
739 brillo::FakeMessageLoop loop(nullptr);
740 loop.SetAsCurrent();
741
742 MockHttpFetcher* fetcher =
743 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
744 OmahaRequestAction action(
745 &fake_system_state_, nullptr, brillo::make_unique_ptr(fetcher), false);
746 ActionProcessor processor;
747 processor.EnqueueAction(&action);
748
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700749 loop.PostTask(base::Bind(
750 [](ActionProcessor* processor) { processor->StartProcessing(); },
751 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700752 loop.Run();
753 EXPECT_FALSE(loop.PendingTasks());
754
755 // Check that the headers were set in the fetcher during the action. Note that
756 // we set this request as "interactive".
757 EXPECT_EQ("fg", fetcher->GetHeader("X-GoogleUpdate-Interactivity"));
758 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-GoogleUpdate-AppId"));
759 EXPECT_NE("", fetcher->GetHeader("X-GoogleUpdate-Updater"));
760}
761
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700762TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700763 OmahaResponse response;
764 // Set up a connection manager that doesn't allow a valid update over
765 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700766 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700767 fake_system_state_.set_connection_manager(&mock_cm);
768
Alex Deymo30534502015-07-20 15:06:33 -0700769 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
770 .WillRepeatedly(
Sen Jiang255e22b2016-05-20 16:15:29 -0700771 DoAll(SetArgumentPointee<0>(ConnectionType::kEthernet),
772 SetArgumentPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700773 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700774 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
775 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700776
777 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700778 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700779 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700780 -1,
781 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700782 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700783 metrics::CheckResult::kUpdateAvailable,
784 metrics::CheckReaction::kIgnored,
785 metrics::DownloadErrorCode::kUnset,
786 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700787 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700788 EXPECT_FALSE(response.update_exists);
789}
790
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700791TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700792 string rollback_version = "1234.0.0";
793 OmahaResponse response;
794
795 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700796 fake_system_state_.set_payload_state(&mock_payload_state);
797
Chris Sosa77f79e82014-06-02 18:16:24 -0700798 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
799 .WillRepeatedly(Return(rollback_version));
800
Alex Deymo8e18f932015-03-27 16:16:59 -0700801 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700802 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700803 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700804 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700805 -1,
806 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700807 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700808 metrics::CheckResult::kUpdateAvailable,
809 metrics::CheckReaction::kIgnored,
810 metrics::DownloadErrorCode::kUnset,
811 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700812 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700813 EXPECT_FALSE(response.update_exists);
814}
815
Kevin Cernekee2494e282016-03-29 18:03:53 -0700816// Verify that update checks called during OOBE will only try to download
817// an update if the response includes a non-empty deadline field.
818TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
819 OmahaResponse response;
820
821 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
822 ASSERT_FALSE(
823 TestUpdateCheck(nullptr, // request_params
824 fake_update_response_.GetUpdateResponse(),
825 -1,
826 false, // ping_only
827 ErrorCode::kNonCriticalUpdateInOOBE,
828 metrics::CheckResult::kUnset,
829 metrics::CheckReaction::kUnset,
830 metrics::DownloadErrorCode::kUnset,
831 &response,
832 nullptr));
833 EXPECT_FALSE(response.update_exists);
834
Alex Deymo46a9aae2016-05-04 20:20:11 -0700835 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
836 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
837 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
838 fake_update_response_.GetUpdateResponse(),
839 -1,
840 false, // ping_only
841 ErrorCode::kSuccess,
842 metrics::CheckResult::kUpdateAvailable,
843 metrics::CheckReaction::kUpdating,
844 metrics::DownloadErrorCode::kUnset,
845 &response,
846 nullptr));
847 EXPECT_TRUE(response.update_exists);
848 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
849
850 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700851 fake_update_response_.deadline = "20101020";
852 ASSERT_TRUE(
853 TestUpdateCheck(nullptr, // request_params
854 fake_update_response_.GetUpdateResponse(),
855 -1,
856 false, // ping_only
857 ErrorCode::kSuccess,
858 metrics::CheckResult::kUpdateAvailable,
859 metrics::CheckReaction::kUpdating,
860 metrics::DownloadErrorCode::kUnset,
861 &response,
862 nullptr));
863 EXPECT_TRUE(response.update_exists);
864}
865
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700866TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700867 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700868 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700869 params.set_wall_clock_based_wait_enabled(true);
870 params.set_update_check_count_wait_enabled(false);
871 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700872
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700873 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700874 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700875 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700876 -1,
877 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700878 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800879 metrics::CheckResult::kUpdateAvailable,
880 metrics::CheckReaction::kDeferring,
881 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700882 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700883 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700884 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700885
886 // Verify if we are interactive check we don't defer.
887 params.set_interactive(true);
888 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700889 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700890 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700891 -1,
892 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700893 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800894 metrics::CheckResult::kUpdateAvailable,
895 metrics::CheckReaction::kUpdating,
896 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700897 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700898 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700899 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700900}
901
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700902TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700903 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700904 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700905 params.set_wall_clock_based_wait_enabled(false);
906 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700907
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700908 params.set_update_check_count_wait_enabled(true);
909 params.set_min_update_checks_needed(1);
910 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700911
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700912 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700913 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700914 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700915 -1,
916 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700917 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800918 metrics::CheckResult::kUpdateAvailable,
919 metrics::CheckReaction::kUpdating,
920 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700921 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700922 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700923 EXPECT_TRUE(response.update_exists);
924}
925
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700926TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700927 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700928 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700929 params.set_wall_clock_based_wait_enabled(true);
930 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700931
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700932 params.set_update_check_count_wait_enabled(true);
933 params.set_min_update_checks_needed(1);
934 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700935
Alex Deymo8e18f932015-03-27 16:16:59 -0700936 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700937 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700938 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700939 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700940 -1,
941 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700942 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800943 metrics::CheckResult::kUpdateAvailable,
944 metrics::CheckReaction::kUpdating,
945 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700946 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700947 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700948 EXPECT_TRUE(response.update_exists);
949}
950
951
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700952TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700953 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700954 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700955 params.set_wall_clock_based_wait_enabled(true);
956 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700957
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700958 params.set_update_check_count_wait_enabled(true);
959 params.set_min_update_checks_needed(0);
960 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700961
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700962 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700963 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700964 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700965 -1,
966 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700967 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800968 metrics::CheckResult::kUpdateAvailable,
969 metrics::CheckReaction::kUpdating,
970 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700971 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700972 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700973
Ben Chan9abb7632014-08-07 00:10:53 -0700974 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700975 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700976 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700977 EXPECT_TRUE(response.update_exists);
978}
979
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700980TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700981 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700982 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700983 params.set_wall_clock_based_wait_enabled(true);
984 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700985
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700986 params.set_update_check_count_wait_enabled(true);
987 params.set_min_update_checks_needed(1);
988 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700989
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700990 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700991 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700992 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700993 -1,
994 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700995 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800996 metrics::CheckResult::kUpdateAvailable,
997 metrics::CheckReaction::kDeferring,
998 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700999 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001000 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001001
Ben Chan9abb7632014-08-07 00:10:53 -07001002 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001003 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001004 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001005 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001006
1007 // Verify if we are interactive check we don't defer.
1008 params.set_interactive(true);
1009 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001010 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001011 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001012 -1,
1013 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001014 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001015 metrics::CheckResult::kUpdateAvailable,
1016 metrics::CheckReaction::kUpdating,
1017 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001018 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001019 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001020 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001021}
1022
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001023TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001024 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001025 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001026 params.set_wall_clock_based_wait_enabled(true);
1027 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001028
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001029 params.set_update_check_count_wait_enabled(true);
1030 params.set_min_update_checks_needed(1);
1031 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001032
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001033 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001034
1035 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001036 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001037 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001038 -1,
1039 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001040 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001041 metrics::CheckResult::kUpdateAvailable,
1042 metrics::CheckReaction::kDeferring,
1043 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001044 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001045 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001046
Ben Chan9abb7632014-08-07 00:10:53 -07001047 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001048 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001049 // count remains the same, as the decrementing happens in update_attempter
1050 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001051 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001052 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001053
1054 // Verify if we are interactive check we don't defer.
1055 params.set_interactive(true);
1056 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001057 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001058 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001059 -1,
1060 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001061 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001062 metrics::CheckResult::kUpdateAvailable,
1063 metrics::CheckReaction::kUpdating,
1064 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001065 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001066 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001067 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001068}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001069
Alex Deymo8e18f932015-03-27 16:16:59 -07001070TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1071 OmahaResponse response;
1072 OmahaRequestParams params = request_params_;
1073 fake_update_response_.include_cohorts = true;
1074 fake_update_response_.cohort = "s/154454/8479665";
1075 fake_update_response_.cohorthint = "please-put-me-on-beta";
1076 fake_update_response_.cohortname = "stable";
1077
1078 ASSERT_TRUE(TestUpdateCheck(&params,
1079 fake_update_response_.GetUpdateResponse(),
1080 -1,
1081 false, // ping_only
1082 ErrorCode::kSuccess,
1083 metrics::CheckResult::kUpdateAvailable,
1084 metrics::CheckReaction::kUpdating,
1085 metrics::DownloadErrorCode::kUnset,
1086 &response,
1087 nullptr));
1088
1089 string value;
1090 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1091 EXPECT_EQ(fake_update_response_.cohort, value);
1092
1093 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1094 EXPECT_EQ(fake_update_response_.cohorthint, value);
1095
1096 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1097 EXPECT_EQ(fake_update_response_.cohortname, value);
1098}
1099
1100TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1101 OmahaResponse response;
1102 OmahaRequestParams params = request_params_;
1103 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1104 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1105 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1106 fake_update_response_.include_cohorts = true;
1107 fake_update_response_.cohort = "s/154454/8479665";
1108 fake_update_response_.cohorthint = "please-put-me-on-beta";
1109 fake_update_response_.cohortname = "";
1110
1111 ASSERT_TRUE(TestUpdateCheck(&params,
1112 fake_update_response_.GetUpdateResponse(),
1113 -1,
1114 false, // ping_only
1115 ErrorCode::kSuccess,
1116 metrics::CheckResult::kUpdateAvailable,
1117 metrics::CheckReaction::kUpdating,
1118 metrics::DownloadErrorCode::kUnset,
1119 &response,
1120 nullptr));
1121
1122 string value;
1123 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1124 EXPECT_EQ(fake_update_response_.cohort, value);
1125
1126 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1127 EXPECT_EQ(fake_update_response_.cohorthint, value);
1128
1129 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1130}
1131
1132TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1133 OmahaResponse response;
1134 OmahaRequestParams params = request_params_;
1135 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1136
1137 ASSERT_TRUE(TestUpdateCheck(&params,
1138 fake_update_response_.GetUpdateResponse(),
1139 -1,
1140 false, // ping_only
1141 ErrorCode::kSuccess,
1142 metrics::CheckResult::kUpdateAvailable,
1143 metrics::CheckReaction::kUpdating,
1144 metrics::DownloadErrorCode::kUnset,
1145 &response,
1146 nullptr));
1147
1148 string value;
1149 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1150 EXPECT_EQ("old_value", value);
1151
1152 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1153 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1154}
1155
Alex Deymo00d79ac2015-06-29 15:41:49 -07001156TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1157 OmahaResponse response;
1158 OmahaRequestParams params = request_params_;
1159 fake_update_response_.include_cohorts = true;
1160 fake_update_response_.cohort = "s/154454/8479665";
1161 fake_update_response_.cohorthint = "please-put-me-on-beta";
1162 fake_update_response_.cohortname = "stable";
1163
1164 ASSERT_TRUE(TestUpdateCheck(&params,
1165 fake_update_response_.GetNoUpdateResponse(),
1166 -1,
1167 false, // ping_only
1168 ErrorCode::kSuccess,
1169 metrics::CheckResult::kNoUpdateAvailable,
1170 metrics::CheckReaction::kUnset,
1171 metrics::DownloadErrorCode::kUnset,
1172 &response,
1173 nullptr));
1174
1175 string value;
1176 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1177 EXPECT_EQ(fake_update_response_.cohort, value);
1178
1179 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1180 EXPECT_EQ(fake_update_response_.cohorthint, value);
1181
1182 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1183 EXPECT_EQ(fake_update_response_.cohortname, value);
1184}
1185
Sen Jiangb1e063a2017-09-15 17:44:31 -07001186TEST_F(OmahaRequestActionTest, MultiAppCohortTest) {
1187 OmahaResponse response;
1188 OmahaRequestParams params = request_params_;
1189 fake_update_response_.multi_app = true;
1190 fake_update_response_.include_cohorts = true;
1191 fake_update_response_.cohort = "s/154454/8479665";
1192 fake_update_response_.cohorthint = "please-put-me-on-beta";
1193 fake_update_response_.cohortname = "stable";
1194
1195 ASSERT_TRUE(TestUpdateCheck(&params,
1196 fake_update_response_.GetUpdateResponse(),
1197 -1,
1198 false, // ping_only
1199 ErrorCode::kSuccess,
1200 metrics::CheckResult::kUpdateAvailable,
1201 metrics::CheckReaction::kUpdating,
1202 metrics::DownloadErrorCode::kUnset,
1203 &response,
1204 nullptr));
1205
1206 string value;
1207 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1208 EXPECT_EQ(fake_update_response_.cohort, value);
1209
1210 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1211 EXPECT_EQ(fake_update_response_.cohorthint, value);
1212
1213 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1214 EXPECT_EQ(fake_update_response_.cohortname, value);
1215}
1216
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001217TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001218 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001219
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001220 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001221 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001222
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001223 OmahaRequestParams params = request_params_;
1224 fake_system_state_.set_request_params(&params);
1225 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001226 brillo::make_unique_ptr(
1227 new MockHttpFetcher(http_response.data(),
1228 http_response.size(),
1229 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001230 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001231 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001232 ActionProcessor processor;
1233 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001234 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001235
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001236 loop.PostTask(base::Bind(
1237 [](ActionProcessor* processor) { processor->StartProcessing(); },
1238 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001239 loop.Run();
1240 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001241 EXPECT_FALSE(processor.IsRunning());
1242}
1243
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001244TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001245 OmahaResponse response;
1246 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001247 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001248 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001249 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001250 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001251 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001252 metrics::CheckResult::kParsingError,
1253 metrics::CheckReaction::kUnset,
1254 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001255 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001256 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001257 EXPECT_FALSE(response.update_exists);
1258}
1259
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001260TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001261 OmahaResponse response;
1262 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001263 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001264 "",
1265 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001266 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001267 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001268 metrics::CheckResult::kParsingError,
1269 metrics::CheckReaction::kUnset,
1270 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001271 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001272 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001273 EXPECT_FALSE(response.update_exists);
1274}
1275
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001276TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001277 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001278 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001279 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001280 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1281 "<daystart elapsed_seconds=\"100\"/>"
1282 "<app appid=\"foo\" status=\"ok\">"
1283 "<ping status=\"ok\"/>"
1284 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001285 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001286 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001287 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001288 metrics::CheckResult::kParsingError,
1289 metrics::CheckReaction::kUnset,
1290 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001291 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001292 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001293 EXPECT_FALSE(response.update_exists);
1294}
1295
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001296TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001297 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001298 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001299 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001300 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1301 "<daystart elapsed_seconds=\"100\"/>"
1302 "<app appid=\"foo\" status=\"ok\">"
1303 "<ping status=\"ok\"/>"
1304 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001305 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001306 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001307 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001308 metrics::CheckResult::kParsingError,
1309 metrics::CheckReaction::kUnset,
1310 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001311 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001312 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001313 EXPECT_FALSE(response.update_exists);
1314}
1315
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001316TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001317 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001318 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001319 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001320 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1321 "<daystart elapsed_seconds=\"100\"/>"
1322 "<app appid=\"foo\" status=\"ok\">"
1323 "<ping status=\"ok\"/>"
1324 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001325 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001326 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001327 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001328 metrics::CheckResult::kParsingError,
1329 metrics::CheckReaction::kUnset,
1330 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001331 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001332 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001333 EXPECT_FALSE(response.update_exists);
1334}
1335
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001336TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001337 string input_response =
1338 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1339 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001340 // the appid needs to match that in the request params
1341 "<app appid=\"" +
1342 fake_update_response_.app_id +
1343 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001344 "<updatecheck status=\"ok\">"
1345 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001346 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001347 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001348 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001349 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001350 "ChromeOSVersion=\"10.2.3.4\" "
1351 "Prompt=\"false\" "
1352 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001353 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001354 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001355 "needsadmin=\"true\" "
1356 "/></actions></manifest></updatecheck></app></response>";
1357 LOG(INFO) << "Input Response = " << input_response;
1358
Darin Petkov6a5b3222010-07-13 14:55:28 -07001359 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001360 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001361 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001362 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001363 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001364 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001365 metrics::CheckResult::kUpdateAvailable,
1366 metrics::CheckReaction::kUpdating,
1367 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001368 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001369 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001370 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001371 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001372 EXPECT_EQ("http://missing/field/test/f",
1373 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001374 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001375 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1376 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001377 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001378 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001379}
1380
1381namespace {
1382class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1383 public:
1384 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001385 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001386 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001387};
1388
Alex Deymo60ca1a72015-06-18 18:19:15 -07001389void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001390 processor->StartProcessing();
1391 CHECK(processor->IsRunning());
1392 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001393}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001394} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001395
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001396TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001397 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001398 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001399
Alex Deymo60ca1a72015-06-18 18:19:15 -07001400 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001401 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001402 brillo::make_unique_ptr(
1403 new MockHttpFetcher(http_response.data(),
1404 http_response.size(),
1405 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001406 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001407 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001408 ActionProcessor processor;
1409 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001410 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001411
Alex Deymo60ca1a72015-06-18 18:19:15 -07001412 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1413 loop.Run();
1414 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001415}
1416
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001417TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001418 string output;
1419 EXPECT_TRUE(XmlEncode("ab", &output));
1420 EXPECT_EQ("ab", output);
1421 EXPECT_TRUE(XmlEncode("a<b", &output));
1422 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001423 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1424 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001425 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1426 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001427 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001428 EXPECT_FALSE(XmlEncode("\xc2", &output));
1429 // Fail with invalid ASCII-7 chars.
1430 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1431}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001432
Alex Deymob0d74eb2015-03-30 17:59:17 -07001433TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1434 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1435 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1436}
1437
1438TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001439 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001440
1441 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001442 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001443 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001444 OmahaRequestParams::kOsVersion,
1445 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001446 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001447 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001448 "0.1.0.0",
1449 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001450 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001451 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001452 "ChromeOSFirmware.1.0",
1453 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001454 false, // delta okay
1455 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001456 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001457 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001458 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1459 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1460 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001461 base::JoinString(
1462 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001463 OmahaResponse response;
1464 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001465 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001466 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001467 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001468 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001469 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001470 metrics::CheckResult::kParsingError,
1471 metrics::CheckReaction::kUnset,
1472 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001473 &response,
1474 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001475 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001476 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001477 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1478 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1479 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1480 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1481 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1482 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1483 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1484 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1485 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1486 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1487 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1488 // Values from Prefs that are too big are removed from the XML instead of
1489 // encoded.
1490 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001491}
1492
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001493TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001494 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001495 fake_update_response_.deadline = "&lt;20110101";
1496 fake_update_response_.more_info_url = "testthe&lt;url";
1497 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001498 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001499 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001500 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001501 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001502 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001503 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001504 metrics::CheckResult::kUpdateAvailable,
1505 metrics::CheckReaction::kUpdating,
1506 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001507 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001508 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001509
Sen Jiang0affc2c2017-02-10 15:55:05 -08001510 EXPECT_EQ("testthe<url", response.more_info_url);
1511 EXPECT_EQ("testthe&codebase/file.signed",
1512 response.packages[0].payload_urls[0]);
1513 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001514}
1515
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001516TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001517 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001518 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001519 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001520 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001521 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001522 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001523 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001524 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001525 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001526 metrics::CheckResult::kUpdateAvailable,
1527 metrics::CheckReaction::kUpdating,
1528 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001529 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001530 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001531
Sen Jiang0affc2c2017-02-10 15:55:05 -08001532 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001533}
1534
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001535TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001536 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001537 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001538 fake_system_state_.set_prefs(&prefs);
1539
Darin Petkov95508da2011-01-05 12:42:29 -08001540 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1541 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001542 // An existing but empty previous version means that we didn't reboot to a new
1543 // update, therefore, no need to update the previous version.
1544 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001545 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001546 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001547 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001548 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001549 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001550 metrics::CheckResult::kParsingError,
1551 metrics::CheckReaction::kUnset,
1552 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001553 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001554 &post_data));
1555 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001556 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001557 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001558 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001559 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001560 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001561 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1562 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001563 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1564 string::npos);
1565 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1566 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001567 // No <event> tag should be sent if we didn't reboot to an update.
1568 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001569}
1570
Jay Srinivasan0a708742012-03-20 11:26:12 -07001571
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001572TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001573 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001574 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001575 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1576 "invalid xml>",
1577 &post_data);
1578 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001579 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001580 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001581 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001582 OmahaEvent::kTypeUpdateDownloadStarted,
1583 OmahaEvent::kResultSuccess);
1584 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001585 EXPECT_EQ(post_str.find("ping"), string::npos);
1586 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001587}
1588
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001589TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001590 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001591 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001592 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1593 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001594 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001595 "invalid xml>",
1596 &post_data);
1597 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001598 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001599 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001600 " <event eventtype=\"%d\" eventresult=\"%d\" "
1601 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001602 OmahaEvent::kTypeDownloadComplete,
1603 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001604 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001605 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001606 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001607}
1608
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001609TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001610 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001611 // Create a copy of the OmahaRequestParams to reuse it later.
1612 OmahaRequestParams params = request_params_;
1613 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001614 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001615 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001616 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001617 brillo::make_unique_ptr(
1618 new MockHttpFetcher(http_response.data(),
1619 http_response.size(),
1620 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001621 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001622 EXPECT_FALSE(update_check_action.IsEvent());
1623
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001624 params = request_params_;
1625 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001626 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001627 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001628 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001629 brillo::make_unique_ptr(
1630 new MockHttpFetcher(http_response.data(),
1631 http_response.size(),
1632 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001633 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001634 EXPECT_TRUE(event_action.IsEvent());
1635}
1636
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001637TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001638 for (int i = 0; i < 2; i++) {
1639 bool delta_okay = i == 1;
1640 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001641 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001642 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001643 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001644 OmahaRequestParams::kOsVersion,
1645 "service_pack",
1646 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001647 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001648 "0.1.0.0",
1649 "en-US",
1650 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001651 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001652 "ChromeOSFirmware.1.0",
1653 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001654 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001655 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001656 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001657 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001658 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001659 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001660 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001661 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001662 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001663 metrics::CheckResult::kParsingError,
1664 metrics::CheckReaction::kUnset,
1665 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001666 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001667 &post_data));
1668 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001669 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001670 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1671 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001672 string::npos)
1673 << "i = " << i;
1674 }
1675}
1676
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001677TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001678 for (int i = 0; i < 2; i++) {
1679 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001680 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001681 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001682 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001683 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001684 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001685 OmahaRequestParams::kOsVersion,
1686 "service_pack",
1687 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001688 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001689 "0.1.0.0",
1690 "en-US",
1691 "unittest_track",
1692 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001693 "ChromeOSFirmware.1.0",
1694 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001695 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001696 interactive,
1697 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001698 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001699 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001700 "invalid xml>",
1701 -1,
1702 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001703 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001704 metrics::CheckResult::kParsingError,
1705 metrics::CheckReaction::kUnset,
1706 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001707 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001708 &post_data));
1709 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001710 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001711 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1712 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001713 string::npos)
1714 << "i = " << i;
1715 }
1716}
1717
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001718TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001719 OmahaEvent default_event;
1720 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1721 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001722 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001723
1724 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1725 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1726 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001727 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001728
1729 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1730 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001731 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001732 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1733 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001734 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001735}
1736
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001737void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001738 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001739 fake_system_state_.set_prefs(&prefs);
1740 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1741 .Times(AnyNumber());
1742 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1743 // Add a few hours to the day difference to test no rounding, etc.
1744 int64_t five_days_ago =
1745 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1746 int64_t six_days_ago =
1747 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1748 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1749 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1750 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1751 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1752 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1753 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001754 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001755 ASSERT_TRUE(
1756 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001757 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001758 -1,
1759 ping_only,
1760 ErrorCode::kSuccess,
1761 metrics::CheckResult::kUnset,
1762 metrics::CheckReaction::kUnset,
1763 metrics::DownloadErrorCode::kUnset,
1764 nullptr,
1765 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001766 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001767 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1768 string::npos);
1769 if (ping_only) {
1770 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1771 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1772 } else {
1773 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1774 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001775 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001776}
1777
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001778TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1779 PingTest(true /* ping_only */);
1780}
1781
1782TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1783 PingTest(false /* ping_only */);
1784}
1785
1786TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001787 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001788 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001789 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1790 .Times(AnyNumber());
1791 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001792 int64_t three_days_ago =
1793 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1794 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001795 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1796 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001797 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1798 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1799 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1800 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001801 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001802 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001803 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001804 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001805 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001806 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001807 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001808 metrics::CheckResult::kNoUpdateAvailable,
1809 metrics::CheckReaction::kUnset,
1810 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001811 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001812 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001813 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001814 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001815 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001816}
1817
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001818TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001819 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001820 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001821 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1822 .Times(AnyNumber());
1823 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001824 int64_t four_days_ago =
1825 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1826 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001827 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1828 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001829 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1830 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1831 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1832 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001833 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001834 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001835 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001836 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001837 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001838 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001839 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001840 metrics::CheckResult::kNoUpdateAvailable,
1841 metrics::CheckReaction::kUnset,
1842 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001843 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001844 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001845 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001846 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001847 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001848}
1849
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001850TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001851 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001852 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001853 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1854 .Times(AnyNumber());
1855 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001856 int64_t one_hour_ago =
1857 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001858 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1859 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001860 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1861 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1862 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1863 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001864 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1865 // send a ping.
1866 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1867 .WillOnce(Return(true));
1868 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1869 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001870 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001871 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001872 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001873 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001874 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001875 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001876 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001877 metrics::CheckResult::kNoUpdateAvailable,
1878 metrics::CheckReaction::kUnset,
1879 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001880 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001881 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001882 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001883 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001884}
1885
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001886TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001887 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001888 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001889 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001890 int64_t now = Time::Now().ToInternalValue();
1891 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1892 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1893 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1894 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1895 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1896 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001897 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001898 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001899 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001900 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001901 -1,
1902 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001903 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001904 metrics::CheckResult::kUnset,
1905 metrics::CheckReaction::kUnset,
1906 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001907 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001908 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001909 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001910}
1911
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001912TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001913 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001914 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001915 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1916 .Times(AnyNumber());
1917 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001918 int64_t future =
1919 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001920 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1921 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001922 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1923 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1924 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1925 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1926 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1927 .WillOnce(Return(true));
1928 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1929 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001930 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001931 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001932 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001933 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1934 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001935 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001936 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001937 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001938 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001939 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001940 metrics::CheckResult::kNoUpdateAvailable,
1941 metrics::CheckReaction::kUnset,
1942 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001943 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001944 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001945 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001946 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001947}
1948
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001949TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001950 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001951 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001952 // may fail if it runs for longer than 5 seconds. It shouldn't run
1953 // that long though.
1954 int64_t midnight =
1955 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1956 int64_t midnight_slack =
1957 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001958 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001959 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001960 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1961 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001962 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1963 AllOf(Ge(midnight), Le(midnight_slack))))
1964 .WillOnce(Return(true));
1965 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1966 AllOf(Ge(midnight), Le(midnight_slack))))
1967 .WillOnce(Return(true));
1968 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001969 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001970 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1971 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001972 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001973 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001974 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001975 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001976 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001977 metrics::CheckResult::kNoUpdateAvailable,
1978 metrics::CheckReaction::kUnset,
1979 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001980 nullptr,
1981 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001982}
1983
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001984TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001985 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001986 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001987 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1988 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001989 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1990 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1991 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001992 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001993 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1994 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001995 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001996 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001997 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001998 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001999 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002000 metrics::CheckResult::kNoUpdateAvailable,
2001 metrics::CheckReaction::kUnset,
2002 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002003 nullptr,
2004 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002005}
2006
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002007TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08002008 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002009 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08002010 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
2011 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002012 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
2013 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
2014 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002015 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002016 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2017 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002018 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07002019 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002020 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002021 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002022 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002023 metrics::CheckResult::kNoUpdateAvailable,
2024 metrics::CheckReaction::kUnset,
2025 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002026 nullptr,
2027 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07002028}
2029
Alex Deymob3fa53b2016-04-18 19:57:58 -07002030TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
2031 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2032 // the "eol" attribute.
2033 ASSERT_TRUE(
2034 TestUpdateCheck(nullptr, // request_params
2035 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2036 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2037 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2038 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2039 "</app></response>",
2040 -1,
2041 false, // ping_only
2042 ErrorCode::kSuccess,
2043 metrics::CheckResult::kNoUpdateAvailable,
2044 metrics::CheckReaction::kUnset,
2045 metrics::DownloadErrorCode::kUnset,
2046 nullptr,
2047 nullptr));
2048 string eol_pref;
2049 EXPECT_TRUE(
2050 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2051 // Note that the eol="eol" attribute should be ignored and the _eol should be
2052 // used instead.
2053 EXPECT_EQ("security-only", eol_pref);
2054}
2055
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002056TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002057 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002058 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07002059 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002060 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002061 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002062 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002063 metrics::CheckResult::kParsingError,
2064 metrics::CheckReaction::kUnset,
2065 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002066 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002067 &post_data));
2068 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002069 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002070 EXPECT_EQ(post_str.find("machineid="), string::npos);
2071 EXPECT_EQ(post_str.find("userid="), string::npos);
2072}
2073
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002074TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002075 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002076 const int http_error_code =
2077 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07002078 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002079 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002080 "",
2081 501,
Darin Petkov265f2902011-05-09 15:17:40 -07002082 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002083 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002084 metrics::CheckResult::kDownloadError,
2085 metrics::CheckReaction::kUnset,
2086 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07002087 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002088 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002089 EXPECT_FALSE(response.update_exists);
2090}
2091
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002092TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002093 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002094 const int http_error_code =
2095 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07002096 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002097 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002098 "",
2099 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07002100 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002101 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002102 metrics::CheckResult::kDownloadError,
2103 metrics::CheckReaction::kUnset,
2104 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07002105 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002106 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002107 EXPECT_FALSE(response.update_exists);
2108}
2109
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002110TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002111 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002112 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002113 params.set_wall_clock_based_wait_enabled(true);
2114 params.set_waiting_period(TimeDelta().FromDays(1));
2115 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002116
Sen Jiang7c1171e2016-06-23 11:35:40 -07002117 Time arbitrary_date;
2118 Time::FromString("6/4/1989", &arbitrary_date);
2119 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
2120 ASSERT_FALSE(TestUpdateCheck(&params,
2121 fake_update_response_.GetUpdateResponse(),
2122 -1,
2123 false, // ping_only
2124 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2125 metrics::CheckResult::kUpdateAvailable,
2126 metrics::CheckReaction::kDeferring,
2127 metrics::DownloadErrorCode::kUnset,
2128 &response,
2129 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002130
Ben Chan9abb7632014-08-07 00:10:53 -07002131 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002132 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002133 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002134 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002135
2136 // Verify if we are interactive check we don't defer.
2137 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002138 ASSERT_TRUE(TestUpdateCheck(&params,
2139 fake_update_response_.GetUpdateResponse(),
2140 -1,
2141 false, // ping_only
2142 ErrorCode::kSuccess,
2143 metrics::CheckResult::kUpdateAvailable,
2144 metrics::CheckReaction::kUpdating,
2145 metrics::DownloadErrorCode::kUnset,
2146 &response,
2147 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002148 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002149}
2150
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002151TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002152 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002153 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002154 params.set_wall_clock_based_wait_enabled(true);
2155 params.set_waiting_period(TimeDelta().FromDays(1));
2156 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002157
Sen Jiang7c1171e2016-06-23 11:35:40 -07002158 Time t1, t2;
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002159 Time::FromString("1/1/2012", &t1);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002160 Time::FromString("1/3/2012", &t2);
2161 ASSERT_TRUE(
2162 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2163 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2164 ASSERT_TRUE(TestUpdateCheck(&params,
2165 fake_update_response_.GetUpdateResponse(),
2166 -1,
2167 false, // ping_only
2168 ErrorCode::kSuccess,
2169 metrics::CheckResult::kUpdateAvailable,
2170 metrics::CheckReaction::kUpdating,
2171 metrics::DownloadErrorCode::kUnset,
2172 &response,
2173 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002174
2175 EXPECT_TRUE(response.update_exists);
2176
2177 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002178 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002179 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002180 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2181}
2182
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002183TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002184 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002185 base::ScopedTempDir tempdir;
2186 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002187
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002188 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002189 OmahaRequestParams params(&fake_system_state_);
2190 params.set_root(tempdir.path().value());
2191 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2192 params.set_app_version("1.2.3.4");
2193 params.set_current_channel("canary-channel");
2194 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2195 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002196 EXPECT_TRUE(params.to_more_stable_channel());
2197 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002198 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002199 "invalid xml>",
2200 -1,
2201 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002202 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002203 metrics::CheckResult::kParsingError,
2204 metrics::CheckReaction::kUnset,
2205 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002206 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002207 &post_data));
2208 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002209 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002210 EXPECT_NE(string::npos, post_str.find(
2211 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2212 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2213 "track=\"stable-channel\" from_track=\"canary-channel\" "));
2214}
2215
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002216TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002217 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002218 base::ScopedTempDir tempdir;
2219 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002220
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002221 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002222 OmahaRequestParams params(&fake_system_state_);
2223 params.set_root(tempdir.path().value());
2224 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2225 params.set_app_version("5.6.7.8");
2226 params.set_current_channel("stable-channel");
2227 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2228 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002229 EXPECT_FALSE(params.to_more_stable_channel());
2230 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002231 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002232 "invalid xml>",
2233 -1,
2234 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002235 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002236 metrics::CheckResult::kParsingError,
2237 metrics::CheckReaction::kUnset,
2238 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002239 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002240 &post_data));
2241 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002242 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002243 EXPECT_NE(string::npos, post_str.find(
2244 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2245 "version=\"5.6.7.8\" "
2246 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002247 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002248}
2249
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002250// Checks that the initial ping with a=-1 r=-1 is not send when the device
2251// was powerwashed.
2252TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2253 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2254
2255 // Flag that the device was powerwashed in the past.
2256 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2257
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002258 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002259 ASSERT_TRUE(
2260 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002261 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002262 -1,
2263 false, // ping_only
2264 ErrorCode::kSuccess,
2265 metrics::CheckResult::kNoUpdateAvailable,
2266 metrics::CheckReaction::kUnset,
2267 metrics::DownloadErrorCode::kUnset,
2268 nullptr,
2269 &post_data));
2270 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002271 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002272 EXPECT_EQ(string::npos, post_str.find("<ping"));
2273}
2274
Alex Deymo9fded1e2015-11-05 12:31:19 -08002275// Checks that the event 54 is sent on a reboot to a new update.
2276TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2277 // Flag that the device was updated in a previous boot.
2278 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2279
2280 brillo::Blob post_data;
2281 ASSERT_TRUE(
2282 TestUpdateCheck(nullptr, // request_params
2283 fake_update_response_.GetNoUpdateResponse(),
2284 -1,
2285 false, // ping_only
2286 ErrorCode::kSuccess,
2287 metrics::CheckResult::kNoUpdateAvailable,
2288 metrics::CheckReaction::kUnset,
2289 metrics::DownloadErrorCode::kUnset,
2290 nullptr,
2291 &post_data));
2292 string post_str(post_data.begin(), post_data.end());
2293
2294 // An event 54 is included and has the right version.
2295 EXPECT_NE(string::npos,
2296 post_str.find(base::StringPrintf(
2297 "<event eventtype=\"%d\"",
2298 OmahaEvent::kTypeRebootedAfterUpdate)));
2299 EXPECT_NE(string::npos,
2300 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2301
2302 // The previous version flag should have been removed.
2303 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2304 string prev_version;
2305 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2306 EXPECT_TRUE(prev_version.empty());
2307}
2308
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002309void OmahaRequestActionTest::P2PTest(
2310 bool initial_allow_p2p_for_downloading,
2311 bool initial_allow_p2p_for_sharing,
2312 bool omaha_disable_p2p_for_downloading,
2313 bool omaha_disable_p2p_for_sharing,
2314 bool payload_state_allow_p2p_attempt,
2315 bool expect_p2p_client_lookup,
2316 const string& p2p_client_result_url,
2317 bool expected_allow_p2p_for_downloading,
2318 bool expected_allow_p2p_for_sharing,
2319 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002320 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002321 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002322 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2323 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2324 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002325
2326 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002327 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002328 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2329 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002330 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2331 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2332 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2333 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2334 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2335 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2336 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2337 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2338 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2339 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2340
David Zeuthen8f191b22013-08-06 12:27:50 -07002341 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002342 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002343 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2344
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002345 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2346 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002347 .Times(expect_p2p_client_lookup ? 1 : 0);
2348
Alex Deymo8e18f932015-03-27 16:16:59 -07002349 fake_update_response_.disable_p2p_for_downloading =
2350 omaha_disable_p2p_for_downloading;
2351 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002352 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002353 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002354 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002355 -1,
2356 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002357 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002358 metrics::CheckResult::kUpdateAvailable,
2359 metrics::CheckReaction::kUpdating,
2360 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002361 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002362 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002363 EXPECT_TRUE(response.update_exists);
2364
Gilad Arnold74b5f552014-10-07 08:17:16 -07002365 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2366 response.disable_p2p_for_downloading);
2367 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2368 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002369
Gilad Arnold74b5f552014-10-07 08:17:16 -07002370 EXPECT_EQ(expected_allow_p2p_for_downloading,
2371 actual_allow_p2p_for_downloading);
2372 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2373 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002374}
2375
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002376TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002377 P2PTest(true, // initial_allow_p2p_for_downloading
2378 true, // initial_allow_p2p_for_sharing
2379 false, // omaha_disable_p2p_for_downloading
2380 false, // omaha_disable_p2p_for_sharing
2381 true, // payload_state_allow_p2p_attempt
2382 true, // expect_p2p_client_lookup
2383 "http://1.3.5.7/p2p", // p2p_client_result_url
2384 true, // expected_allow_p2p_for_downloading
2385 true, // expected_allow_p2p_for_sharing
2386 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002387}
2388
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002389TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002390 P2PTest(true, // initial_allow_p2p_for_downloading
2391 true, // initial_allow_p2p_for_sharing
2392 false, // omaha_disable_p2p_for_downloading
2393 false, // omaha_disable_p2p_for_sharing
2394 true, // payload_state_allow_p2p_attempt
2395 true, // expect_p2p_client_lookup
2396 "", // p2p_client_result_url
2397 false, // expected_allow_p2p_for_downloading
2398 true, // expected_allow_p2p_for_sharing
2399 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002400}
2401
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002402TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002403 P2PTest(false, // initial_allow_p2p_for_downloading
2404 true, // initial_allow_p2p_for_sharing
2405 false, // omaha_disable_p2p_for_downloading
2406 false, // omaha_disable_p2p_for_sharing
2407 true, // payload_state_allow_p2p_attempt
2408 false, // expect_p2p_client_lookup
2409 "unset", // p2p_client_result_url
2410 false, // expected_allow_p2p_for_downloading
2411 true, // expected_allow_p2p_for_sharing
2412 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002413}
2414
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002415TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002416 P2PTest(true, // initial_allow_p2p_for_downloading
2417 true, // initial_allow_p2p_for_sharing
2418 true, // omaha_disable_p2p_for_downloading
2419 false, // omaha_disable_p2p_for_sharing
2420 true, // payload_state_allow_p2p_attempt
2421 false, // expect_p2p_client_lookup
2422 "unset", // p2p_client_result_url
2423 false, // expected_allow_p2p_for_downloading
2424 true, // expected_allow_p2p_for_sharing
2425 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002426}
2427
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002428TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002429 P2PTest(true, // initial_allow_p2p_for_downloading
2430 true, // initial_allow_p2p_for_sharing
2431 false, // omaha_disable_p2p_for_downloading
2432 true, // omaha_disable_p2p_for_sharing
2433 true, // payload_state_allow_p2p_attempt
2434 true, // expect_p2p_client_lookup
2435 "http://1.3.5.7/p2p", // p2p_client_result_url
2436 true, // expected_allow_p2p_for_downloading
2437 false, // expected_allow_p2p_for_sharing
2438 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002439}
2440
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002441TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002442 P2PTest(true, // initial_allow_p2p_for_downloading
2443 true, // initial_allow_p2p_for_sharing
2444 true, // omaha_disable_p2p_for_downloading
2445 true, // omaha_disable_p2p_for_sharing
2446 true, // payload_state_allow_p2p_attempt
2447 false, // expect_p2p_client_lookup
2448 "unset", // p2p_client_result_url
2449 false, // expected_allow_p2p_for_downloading
2450 false, // expected_allow_p2p_for_sharing
2451 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002452}
2453
Alex Deymof329b932014-10-30 01:37:48 -07002454bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2455 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002456 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002457 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002458 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002459 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002460 -1,
2461 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002462 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002463 metrics::CheckResult::kUpdateAvailable,
2464 metrics::CheckReaction::kUpdating,
2465 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002466 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002467 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002468}
2469
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002470TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002471 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002472
Kevin Cernekee2494e282016-03-29 18:03:53 -07002473 // Simulate a successful update check that happens during OOBE. The
2474 // deadline in the response is needed to force the update attempt to
2475 // occur; responses without a deadline seen during OOBE will normally
2476 // return ErrorCode::kNonCriticalUpdateInOOBE.
2477 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2478 fake_update_response_.deadline = "20101020";
2479
David Zeuthen639aa362014-02-03 16:23:44 -08002480 // Check that we parse elapsed_days in the Omaha Response correctly.
2481 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002482 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2483 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002484 EXPECT_TRUE(response.update_exists);
2485 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002486 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002487 int64_t prefs_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 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002492 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002493 EXPECT_TRUE(response.update_exists);
2494 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002495 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002496 EXPECT_EQ(prefs_days, 42);
2497
2498 // Note that elapsed_days is not necessarily divisible by 7 so check
2499 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002500 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2501 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002502 EXPECT_TRUE(response.update_exists);
2503 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002504 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002505 EXPECT_EQ(prefs_days, 21);
2506
2507 // Check that we correctly handle elapsed_days not being included in
2508 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002509 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002510 EXPECT_TRUE(response.update_exists);
2511 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002512}
2513
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002514// If there is no prefs and OOBE is not complete, we should not
2515// report anything to Omaha.
2516TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002517 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002518 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2519 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2520}
David Zeuthen639aa362014-02-03 16:23:44 -08002521
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002522// If OOBE is complete and happened on a valid date (e.g. after Jan
2523// 1 2007 0:00 PST), that date should be used and written to
2524// prefs. However, first try with an invalid date and check we do
2525// nothing.
2526TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2527 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2528 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2529 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2530 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2531}
David Zeuthen639aa362014-02-03 16:23:44 -08002532
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002533// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2534// should yield an InstallDate of 14.
2535TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2536 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2537 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2538 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2539 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002540
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002541 int64_t prefs_days;
2542 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2543 EXPECT_EQ(prefs_days, 14);
2544}
David Zeuthen639aa362014-02-03 16:23:44 -08002545
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002546// Now that we have a valid date in prefs, check that we keep using
2547// that even if OOBE date reports something else. The date Jan 30,
2548// 2007 0:00 PST should yield an InstallDate of 28... but since
2549// there's a prefs file, we should still get 14.
2550TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2551 // Set a valid date in the prefs first.
2552 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002553
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002554 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2555 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2556 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002557
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002558 int64_t prefs_days;
2559 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2560 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002561
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002562 // If we delete the prefs file, we should get 28 days.
2563 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2564 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2565 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2566 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002567}
2568
Darin Petkov6a5b3222010-07-13 14:55:28 -07002569} // namespace chromeos_update_engine