blob: 7e38956dfd3599cb93a9393bbb5908f5b97b446f [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
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700147 ? "<app appid=\"" + app_id2 +
148 "\"><updatecheck status=\"ok\"><urls><url codebase=\"" +
149 codebase2 + "\"/></urls><manifest version=\"" + version2 +
150 "\"><packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700151 "<package name=\"package3\" size=\"333\" "
152 "hash_sha256=\"hash3\"/></packages>"
Sen Jiang00adf7b2017-06-26 15:57:29 -0700153 "<actions><action event=\"postinstall\" " +
154 (multi_app_self_update
155 ? "noupdate=\"true\" IsDeltaPayload=\"true\" "
156 : "IsDeltaPayload=\"false\" ") +
157 "MetadataSize=\"33\"/></actions>"
Sen Jiang81259682017-03-30 15:11:30 -0700158 "</manifest></updatecheck></app>"
159 : "") +
160 (multi_app_no_update
161 ? "<app><updatecheck status=\"noupdate\"/></app>"
162 : "") +
163 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700164 }
165
166 // Return the payload URL, which is split in two fields in the XML response.
167 string GetPayloadUrl() {
168 return codebase + filename;
169 }
170
Alex Deymo85616652015-10-15 18:48:31 -0700171 string app_id = kTestAppId;
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700172 string app_id2 = kTestAppId2;
Alex Deymo8e18f932015-03-27 16:16:59 -0700173 string version = "1.2.3.4";
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700174 string version2 = "2.3.4.5";
Alex Deymo8e18f932015-03-27 16:16:59 -0700175 string more_info_url = "http://more/info";
176 string prompt = "true";
177 string codebase = "http://code/base/";
Sen Jiang81259682017-03-30 15:11:30 -0700178 string codebase2 = "http://code/base/2/";
Alex Deymo8e18f932015-03-27 16:16:59 -0700179 string filename = "file.signed";
Sen Jiang2703ef42017-03-16 13:36:21 -0700180 string hash = "4841534831323334";
Alex Deymo8e18f932015-03-27 16:16:59 -0700181 string needsadmin = "false";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800182 uint64_t size = 123;
Alex Deymo8e18f932015-03-27 16:16:59 -0700183 string deadline = "";
184 string max_days_to_scatter = "7";
185 string elapsed_days = "42";
186
187 // P2P setting defaults to allowed.
188 bool disable_p2p_for_downloading = false;
189 bool disable_p2p_for_sharing = false;
190
191 // Omaha cohorts settings.
192 bool include_cohorts = false;
193 string cohort = "";
194 string cohorthint = "";
195 string cohortname = "";
196
197 // Whether to include the CrOS <!ENTITY> in the XML response.
198 bool include_entity = false;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800199
Sen Jiang81259682017-03-30 15:11:30 -0700200 // Whether to include more than one app.
201 bool multi_app = false;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700202 // Whether to include an app with noupdate="true".
203 bool multi_app_self_update = false;
204 // Whether to include an additional app with status="noupdate".
Sen Jiang81259682017-03-30 15:11:30 -0700205 bool multi_app_no_update = false;
206 // Whether to include more than one package in an app.
Sen Jiang0affc2c2017-02-10 15:55:05 -0800207 bool multi_package = false;
Alex Deymo8e18f932015-03-27 16:16:59 -0700208};
209
210} // namespace
211
Darin Petkov6a5b3222010-07-13 14:55:28 -0700212namespace chromeos_update_engine {
213
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700214class OmahaRequestActionTest : public ::testing::Test {
215 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800216 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700217 fake_system_state_.set_request_params(&request_params_);
218 fake_system_state_.set_prefs(&fake_prefs_);
219 }
220
221 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800222 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700223 // is used. |payload_state| may be null, in which case a local mock is used.
224 // |p2p_manager| may be null, in which case a local mock is used.
225 // |connection_manager| may be null, in which case a local mock is used.
226 // out_response may be null. If |fail_http_response_code| is non-negative,
227 // the transfer will fail with that code. |ping_only| is passed through to the
228 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
229 // post-data received by the mock HttpFetcher is returned.
230 //
231 // The |expected_check_result|, |expected_check_reaction| and
232 // |expected_error_code| parameters are for checking expectations
233 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
234 // UMA statistics. Use the appropriate ::kUnset value to specify that
235 // the given metric should not be reported.
236 bool TestUpdateCheck(OmahaRequestParams* request_params,
237 const string& http_response,
238 int fail_http_response_code,
239 bool ping_only,
240 ErrorCode expected_code,
241 metrics::CheckResult expected_check_result,
242 metrics::CheckReaction expected_check_reaction,
243 metrics::DownloadErrorCode expected_download_error_code,
244 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700245 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700246
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800247 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700248 // only a ping or also an updatecheck.
249 void PingTest(bool ping_only);
250
251 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700252 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700253 OmahaResponse *response);
254
255 // P2P test helper function.
256 void P2PTest(
257 bool initial_allow_p2p_for_downloading,
258 bool initial_allow_p2p_for_sharing,
259 bool omaha_disable_p2p_for_downloading,
260 bool omaha_disable_p2p_for_sharing,
261 bool payload_state_allow_p2p_attempt,
262 bool expect_p2p_client_lookup,
263 const string& p2p_client_result_url,
264 bool expected_allow_p2p_for_downloading,
265 bool expected_allow_p2p_for_sharing,
266 const string& expected_p2p_url);
267
268 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700269 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700270
271 // By default, all tests use these objects unless they replace them in the
272 // fake_system_state_.
273 OmahaRequestParams request_params_ = OmahaRequestParams{
274 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700275 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700276 OmahaRequestParams::kOsVersion,
277 "service_pack",
278 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700279 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700280 "0.1.0.0",
281 "en-US",
282 "unittest",
283 "OEM MODEL 09235 7471",
284 "ChromeOSFirmware.1.0",
285 "0X0A1",
286 false, // delta okay
287 false, // interactive
288 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700289 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700290
291 FakePrefs fake_prefs_;
292};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700293
294namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700295class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
296 public:
297 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700298 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800299 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700300 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800301 void ProcessingDone(const ActionProcessor* processor,
302 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700303 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700304 }
305
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800306 void ActionCompleted(ActionProcessor* processor,
307 AbstractAction* action,
308 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700309 // make sure actions always succeed
310 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700311 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700312 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700313 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700314 }
David Zeuthena99981f2013-04-29 13:42:47 -0700315 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700316};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700317} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700318
319class OutputObjectCollectorAction;
320
321template<>
322class ActionTraits<OutputObjectCollectorAction> {
323 public:
324 // Does not take an object for input
325 typedef OmahaResponse InputObjectType;
326 // On success, puts the output path on output
327 typedef NoneType OutputObjectType;
328};
329
330class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
331 public:
332 OutputObjectCollectorAction() : has_input_object_(false) {}
333 void PerformAction() {
334 // copy input object
335 has_input_object_ = HasInputObject();
336 if (has_input_object_)
337 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700338 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700339 }
340 // Should never be called
341 void TerminateProcessing() {
342 CHECK(false);
343 }
344 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700345 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700346 return "OutputObjectCollectorAction";
347 }
Alex Deymof329b932014-10-30 01:37:48 -0700348 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800349 using InputObjectType =
350 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
351 using OutputObjectType =
352 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700353 bool has_input_object_;
354 OmahaResponse omaha_response_;
355};
356
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700357bool OmahaRequestActionTest::TestUpdateCheck(
358 OmahaRequestParams* request_params,
359 const string& http_response,
360 int fail_http_response_code,
361 bool ping_only,
362 ErrorCode expected_code,
363 metrics::CheckResult expected_check_result,
364 metrics::CheckReaction expected_check_reaction,
365 metrics::DownloadErrorCode expected_download_error_code,
366 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700367 brillo::Blob* out_post_data) {
368 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700369 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700370 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800371 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700372 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700373 if (fail_http_response_code >= 0) {
374 fetcher->FailTransfer(fail_http_response_code);
375 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700376 if (request_params)
377 fake_system_state_.set_request_params(request_params);
378 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700379 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300380 brillo::make_unique_ptr(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700381 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700382 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700383 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700384
Darin Petkov6a5b3222010-07-13 14:55:28 -0700385 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700386 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700387 processor.EnqueueAction(&action);
388
389 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700390 BondActions(&action, &collector_action);
391 processor.EnqueueAction(&collector_action);
392
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700393 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800394 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700395 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800396 SendEnumToUMA(metrics::kMetricCheckResult,
397 static_cast<int>(expected_check_result),
398 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
399 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700400 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800401 SendEnumToUMA(metrics::kMetricCheckReaction,
402 static_cast<int>(expected_check_reaction),
403 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
404 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700405 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700406 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
407 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800408 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
409 ? 0 : 1);
410
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700411 loop.PostTask(base::Bind(
412 [](ActionProcessor* processor) { processor->StartProcessing(); },
413 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700414 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700415 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700416 if (collector_action.has_input_object_ && out_response)
417 *out_response = collector_action.omaha_response_;
418 if (out_post_data)
419 *out_post_data = fetcher->post_data();
420 return collector_action.has_input_object_;
421}
422
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700423// Tests Event requests -- they should always succeed. |out_post_data|
424// may be null; if non-null, the post-data received by the mock
425// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700426void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700427 OmahaEvent* event,
428 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700429 brillo::Blob* out_post_data) {
430 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700431 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700432 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800433 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700434 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700435 FakeSystemState fake_system_state;
436 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300437 OmahaRequestAction action(&fake_system_state,
438 event,
439 brillo::make_unique_ptr(fetcher),
440 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700441 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700442 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700443 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700444 processor.EnqueueAction(&action);
445
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700446 loop.PostTask(base::Bind(
447 [](ActionProcessor* processor) { processor->StartProcessing(); },
448 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700449 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700450 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700451
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700452 if (out_post_data)
453 *out_post_data = fetcher->post_data();
454}
455
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700456TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400457 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700458 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400459 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700460 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700461 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400462 -1,
463 false, // ping_only
464 ErrorCode::kOmahaRequestXMLHasEntityDecl,
465 metrics::CheckResult::kParsingError,
466 metrics::CheckReaction::kUnset,
467 metrics::DownloadErrorCode::kUnset,
468 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700469 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400470 EXPECT_FALSE(response.update_exists);
471}
472
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700473TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700474 OmahaResponse response;
475 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700476 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700477 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700478 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700479 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700480 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800481 metrics::CheckResult::kNoUpdateAvailable,
482 metrics::CheckReaction::kUnset,
483 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700484 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700485 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700486 EXPECT_FALSE(response.update_exists);
487}
488
Sen Jiang81259682017-03-30 15:11:30 -0700489TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
490 OmahaResponse response;
491 fake_update_response_.multi_app_no_update = true;
492 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
493 fake_update_response_.GetNoUpdateResponse(),
494 -1,
495 false, // ping_only
496 ErrorCode::kSuccess,
497 metrics::CheckResult::kNoUpdateAvailable,
498 metrics::CheckReaction::kUnset,
499 metrics::DownloadErrorCode::kUnset,
500 &response,
501 nullptr));
502 EXPECT_FALSE(response.update_exists);
503}
504
Sen Jiang00adf7b2017-06-26 15:57:29 -0700505TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
506 OmahaResponse response;
507 fake_update_response_.multi_app_no_update = true;
508 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
509 fake_update_response_.GetUpdateResponse(),
510 -1,
511 false, // ping_only
512 ErrorCode::kSuccess,
513 metrics::CheckResult::kNoUpdateAvailable,
514 metrics::CheckReaction::kUnset,
515 metrics::DownloadErrorCode::kUnset,
516 &response,
517 nullptr));
518 EXPECT_FALSE(response.update_exists);
519}
520
521TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
522 OmahaResponse response;
523 ASSERT_TRUE(TestUpdateCheck(
524 nullptr, // request_params
525 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
526 "event=\"postinstall\" noupdate=\"true\"/></actions>"
527 "</manifest></updatecheck></app></response>",
528 -1,
529 false, // ping_only
530 ErrorCode::kSuccess,
531 metrics::CheckResult::kNoUpdateAvailable,
532 metrics::CheckReaction::kUnset,
533 metrics::DownloadErrorCode::kUnset,
534 &response,
535 nullptr));
536 EXPECT_FALSE(response.update_exists);
537}
538
Alex Deymo8e18f932015-03-27 16:16:59 -0700539// Test that all the values in the response are parsed in a normal update
540// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700541TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700542 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700543 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700544 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700545 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700546 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700547 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700548 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700549 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800550 metrics::CheckResult::kUpdateAvailable,
551 metrics::CheckReaction::kUpdating,
552 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700553 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700554 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700555 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700556 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700557 EXPECT_EQ("", response.system_version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800558 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
559 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700560 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800561 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
562 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700563 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700564 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
565 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700566 // Omaha cohort attributes are not set in the response, so they should not be
Alex Deymo8e18f932015-03-27 16:16:59 -0700567 // persisted.
568 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
569 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
570 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700571}
572
Sen Jiang0affc2c2017-02-10 15:55:05 -0800573TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
574 OmahaResponse response;
575 fake_update_response_.multi_package = true;
576 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
577 fake_update_response_.GetUpdateResponse(),
578 -1,
579 false, // ping_only
580 ErrorCode::kSuccess,
581 metrics::CheckResult::kUpdateAvailable,
582 metrics::CheckReaction::kUpdating,
583 metrics::DownloadErrorCode::kUnset,
584 &response,
585 nullptr));
586 EXPECT_TRUE(response.update_exists);
587 EXPECT_EQ(fake_update_response_.version, response.version);
588 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
589 response.packages[0].payload_urls[0]);
590 EXPECT_EQ(fake_update_response_.codebase + "package2",
591 response.packages[1].payload_urls[0]);
592 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
593 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700594 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700595 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800596 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700597 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800598 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700599 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700600 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700601}
602
603TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
604 OmahaResponse response;
605 fake_update_response_.multi_app = true;
606 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
607 fake_update_response_.GetUpdateResponse(),
608 -1,
609 false, // ping_only
610 ErrorCode::kSuccess,
611 metrics::CheckResult::kUpdateAvailable,
612 metrics::CheckReaction::kUpdating,
613 metrics::DownloadErrorCode::kUnset,
614 &response,
615 nullptr));
616 EXPECT_TRUE(response.update_exists);
617 EXPECT_EQ(fake_update_response_.version, response.version);
618 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
619 response.packages[0].payload_urls[0]);
620 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
621 response.packages[1].payload_urls[0]);
622 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
623 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
624 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700625 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700626 ASSERT_EQ(2u, response.packages.size());
627 EXPECT_EQ(string("hash3"), response.packages[1].hash);
628 EXPECT_EQ(333u, response.packages[1].size);
629 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700630 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700631}
632
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700633TEST_F(OmahaRequestActionTest, MultiAppAndSystemUpdateTest) {
634 OmahaResponse response;
635 fake_update_response_.multi_app = true;
636 // trigger the lining up of the app and system versions
637 request_params_.set_system_app_id(fake_update_response_.app_id2);
638
639 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
640 fake_update_response_.GetUpdateResponse(),
641 -1,
642 false, // ping_only
643 ErrorCode::kSuccess,
644 metrics::CheckResult::kUpdateAvailable,
645 metrics::CheckReaction::kUpdating,
646 metrics::DownloadErrorCode::kUnset,
647 &response,
648 nullptr));
649 EXPECT_TRUE(response.update_exists);
650 EXPECT_EQ(fake_update_response_.version, response.version);
651 EXPECT_EQ(fake_update_response_.version2, response.system_version);
652 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
653 response.packages[0].payload_urls[0]);
654 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
655 response.packages[1].payload_urls[0]);
656 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
657 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
658 EXPECT_EQ(11u, response.packages[0].metadata_size);
659 EXPECT_EQ(true, response.packages[0].is_delta);
660 ASSERT_EQ(2u, response.packages.size());
661 EXPECT_EQ(string("hash3"), response.packages[1].hash);
662 EXPECT_EQ(333u, response.packages[1].size);
663 EXPECT_EQ(33u, response.packages[1].metadata_size);
664 EXPECT_EQ(false, response.packages[1].is_delta);
665}
666
Sen Jiang81259682017-03-30 15:11:30 -0700667TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
668 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700669 fake_update_response_.multi_app = true;
670 fake_update_response_.multi_app_self_update = true;
Sen Jiang81259682017-03-30 15:11:30 -0700671 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
672 fake_update_response_.GetUpdateResponse(),
673 -1,
674 false, // ping_only
675 ErrorCode::kSuccess,
676 metrics::CheckResult::kUpdateAvailable,
677 metrics::CheckReaction::kUpdating,
678 metrics::DownloadErrorCode::kUnset,
679 &response,
680 nullptr));
681 EXPECT_TRUE(response.update_exists);
682 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700683 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700684 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
685 response.packages[0].payload_urls[0]);
686 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
687 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
688 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700689 ASSERT_EQ(2u, response.packages.size());
690 EXPECT_EQ(string("hash3"), response.packages[1].hash);
691 EXPECT_EQ(333u, response.packages[1].size);
692 EXPECT_EQ(33u, response.packages[1].metadata_size);
693 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700694}
695
696TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
697 OmahaResponse response;
698 fake_update_response_.multi_app = true;
699 fake_update_response_.multi_package = true;
700 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
701 fake_update_response_.GetUpdateResponse(),
702 -1,
703 false, // ping_only
704 ErrorCode::kSuccess,
705 metrics::CheckResult::kUpdateAvailable,
706 metrics::CheckReaction::kUpdating,
707 metrics::DownloadErrorCode::kUnset,
708 &response,
709 nullptr));
710 EXPECT_TRUE(response.update_exists);
711 EXPECT_EQ(fake_update_response_.version, response.version);
Aaron Wood7dcdedf2017-09-06 17:17:41 -0700712 EXPECT_EQ("", response.system_version);
Sen Jiang81259682017-03-30 15:11:30 -0700713 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
714 response.packages[0].payload_urls[0]);
715 EXPECT_EQ(fake_update_response_.codebase + "package2",
716 response.packages[1].payload_urls[0]);
717 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
718 response.packages[2].payload_urls[0]);
719 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
720 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
721 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700722 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700723 ASSERT_EQ(3u, response.packages.size());
724 EXPECT_EQ(string("hash2"), response.packages[1].hash);
725 EXPECT_EQ(222u, response.packages[1].size);
726 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700727 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700728 EXPECT_EQ(string("hash3"), response.packages[2].hash);
729 EXPECT_EQ(333u, response.packages[2].size);
730 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700731 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800732}
733
Alex Deymo14ad88e2016-06-29 12:30:14 -0700734TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
735 const string http_response = "<?xml invalid response";
736 request_params_.set_interactive(true);
737
738 brillo::FakeMessageLoop loop(nullptr);
739 loop.SetAsCurrent();
740
741 MockHttpFetcher* fetcher =
742 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
743 OmahaRequestAction action(
744 &fake_system_state_, nullptr, brillo::make_unique_ptr(fetcher), false);
745 ActionProcessor processor;
746 processor.EnqueueAction(&action);
747
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700748 loop.PostTask(base::Bind(
749 [](ActionProcessor* processor) { processor->StartProcessing(); },
750 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700751 loop.Run();
752 EXPECT_FALSE(loop.PendingTasks());
753
754 // Check that the headers were set in the fetcher during the action. Note that
755 // we set this request as "interactive".
756 EXPECT_EQ("fg", fetcher->GetHeader("X-GoogleUpdate-Interactivity"));
757 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-GoogleUpdate-AppId"));
758 EXPECT_NE("", fetcher->GetHeader("X-GoogleUpdate-Updater"));
759}
760
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700761TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700762 OmahaResponse response;
763 // Set up a connection manager that doesn't allow a valid update over
764 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700765 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700766 fake_system_state_.set_connection_manager(&mock_cm);
767
Alex Deymo30534502015-07-20 15:06:33 -0700768 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
769 .WillRepeatedly(
Sen Jiang255e22b2016-05-20 16:15:29 -0700770 DoAll(SetArgumentPointee<0>(ConnectionType::kEthernet),
771 SetArgumentPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700772 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700773 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
774 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700775
776 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700777 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700778 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700779 -1,
780 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700781 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700782 metrics::CheckResult::kUpdateAvailable,
783 metrics::CheckReaction::kIgnored,
784 metrics::DownloadErrorCode::kUnset,
785 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700786 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700787 EXPECT_FALSE(response.update_exists);
788}
789
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700790TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700791 string rollback_version = "1234.0.0";
792 OmahaResponse response;
793
794 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700795 fake_system_state_.set_payload_state(&mock_payload_state);
796
Chris Sosa77f79e82014-06-02 18:16:24 -0700797 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
798 .WillRepeatedly(Return(rollback_version));
799
Alex Deymo8e18f932015-03-27 16:16:59 -0700800 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700801 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700802 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700803 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700804 -1,
805 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700806 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700807 metrics::CheckResult::kUpdateAvailable,
808 metrics::CheckReaction::kIgnored,
809 metrics::DownloadErrorCode::kUnset,
810 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700811 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700812 EXPECT_FALSE(response.update_exists);
813}
814
Kevin Cernekee2494e282016-03-29 18:03:53 -0700815// Verify that update checks called during OOBE will only try to download
816// an update if the response includes a non-empty deadline field.
817TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
818 OmahaResponse response;
819
820 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
821 ASSERT_FALSE(
822 TestUpdateCheck(nullptr, // request_params
823 fake_update_response_.GetUpdateResponse(),
824 -1,
825 false, // ping_only
826 ErrorCode::kNonCriticalUpdateInOOBE,
827 metrics::CheckResult::kUnset,
828 metrics::CheckReaction::kUnset,
829 metrics::DownloadErrorCode::kUnset,
830 &response,
831 nullptr));
832 EXPECT_FALSE(response.update_exists);
833
Alex Deymo46a9aae2016-05-04 20:20:11 -0700834 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
835 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
836 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
837 fake_update_response_.GetUpdateResponse(),
838 -1,
839 false, // ping_only
840 ErrorCode::kSuccess,
841 metrics::CheckResult::kUpdateAvailable,
842 metrics::CheckReaction::kUpdating,
843 metrics::DownloadErrorCode::kUnset,
844 &response,
845 nullptr));
846 EXPECT_TRUE(response.update_exists);
847 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
848
849 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700850 fake_update_response_.deadline = "20101020";
851 ASSERT_TRUE(
852 TestUpdateCheck(nullptr, // request_params
853 fake_update_response_.GetUpdateResponse(),
854 -1,
855 false, // ping_only
856 ErrorCode::kSuccess,
857 metrics::CheckResult::kUpdateAvailable,
858 metrics::CheckReaction::kUpdating,
859 metrics::DownloadErrorCode::kUnset,
860 &response,
861 nullptr));
862 EXPECT_TRUE(response.update_exists);
863}
864
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700865TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700866 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700867 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700868 params.set_wall_clock_based_wait_enabled(true);
869 params.set_update_check_count_wait_enabled(false);
870 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700871
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700872 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700873 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700874 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700875 -1,
876 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700877 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800878 metrics::CheckResult::kUpdateAvailable,
879 metrics::CheckReaction::kDeferring,
880 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700881 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700882 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700883 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700884
885 // Verify if we are interactive check we don't defer.
886 params.set_interactive(true);
887 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700888 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700889 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700890 -1,
891 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700892 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800893 metrics::CheckResult::kUpdateAvailable,
894 metrics::CheckReaction::kUpdating,
895 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700896 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700897 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700898 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700899}
900
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700901TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700902 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700903 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700904 params.set_wall_clock_based_wait_enabled(false);
905 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700906
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700907 params.set_update_check_count_wait_enabled(true);
908 params.set_min_update_checks_needed(1);
909 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700910
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700911 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700912 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700913 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700914 -1,
915 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700916 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800917 metrics::CheckResult::kUpdateAvailable,
918 metrics::CheckReaction::kUpdating,
919 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700920 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700921 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700922 EXPECT_TRUE(response.update_exists);
923}
924
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700925TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700926 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700927 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700928 params.set_wall_clock_based_wait_enabled(true);
929 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700930
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700931 params.set_update_check_count_wait_enabled(true);
932 params.set_min_update_checks_needed(1);
933 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700934
Alex Deymo8e18f932015-03-27 16:16:59 -0700935 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700936 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700937 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700938 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700939 -1,
940 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700941 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800942 metrics::CheckResult::kUpdateAvailable,
943 metrics::CheckReaction::kUpdating,
944 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700945 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700946 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700947 EXPECT_TRUE(response.update_exists);
948}
949
950
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700951TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700952 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700953 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700954 params.set_wall_clock_based_wait_enabled(true);
955 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700956
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700957 params.set_update_check_count_wait_enabled(true);
958 params.set_min_update_checks_needed(0);
959 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700960
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700961 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700962 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700963 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700964 -1,
965 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700966 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800967 metrics::CheckResult::kUpdateAvailable,
968 metrics::CheckReaction::kUpdating,
969 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700970 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700971 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700972
Ben Chan9abb7632014-08-07 00:10:53 -0700973 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700974 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700975 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700976 EXPECT_TRUE(response.update_exists);
977}
978
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700979TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700980 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700981 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700982 params.set_wall_clock_based_wait_enabled(true);
983 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700984
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700985 params.set_update_check_count_wait_enabled(true);
986 params.set_min_update_checks_needed(1);
987 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700988
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700989 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700990 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700991 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700992 -1,
993 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700994 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800995 metrics::CheckResult::kUpdateAvailable,
996 metrics::CheckReaction::kDeferring,
997 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700998 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700999 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001000
Ben Chan9abb7632014-08-07 00:10:53 -07001001 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001002 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001003 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001004 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001005
1006 // Verify if we are interactive check we don't defer.
1007 params.set_interactive(true);
1008 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001009 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001010 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001011 -1,
1012 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001013 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001014 metrics::CheckResult::kUpdateAvailable,
1015 metrics::CheckReaction::kUpdating,
1016 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001017 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001018 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001019 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001020}
1021
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001022TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001023 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001024 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001025 params.set_wall_clock_based_wait_enabled(true);
1026 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001027
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001028 params.set_update_check_count_wait_enabled(true);
1029 params.set_min_update_checks_needed(1);
1030 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001031
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001032 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001033
1034 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001035 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001036 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001037 -1,
1038 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001039 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001040 metrics::CheckResult::kUpdateAvailable,
1041 metrics::CheckReaction::kDeferring,
1042 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001043 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001044 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001045
Ben Chan9abb7632014-08-07 00:10:53 -07001046 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001047 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001048 // count remains the same, as the decrementing happens in update_attempter
1049 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001050 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001051 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001052
1053 // Verify if we are interactive check we don't defer.
1054 params.set_interactive(true);
1055 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001056 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001057 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001058 -1,
1059 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001060 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001061 metrics::CheckResult::kUpdateAvailable,
1062 metrics::CheckReaction::kUpdating,
1063 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001064 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001065 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001066 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001067}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001068
Alex Deymo8e18f932015-03-27 16:16:59 -07001069TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1070 OmahaResponse response;
1071 OmahaRequestParams params = request_params_;
1072 fake_update_response_.include_cohorts = true;
1073 fake_update_response_.cohort = "s/154454/8479665";
1074 fake_update_response_.cohorthint = "please-put-me-on-beta";
1075 fake_update_response_.cohortname = "stable";
1076
1077 ASSERT_TRUE(TestUpdateCheck(&params,
1078 fake_update_response_.GetUpdateResponse(),
1079 -1,
1080 false, // ping_only
1081 ErrorCode::kSuccess,
1082 metrics::CheckResult::kUpdateAvailable,
1083 metrics::CheckReaction::kUpdating,
1084 metrics::DownloadErrorCode::kUnset,
1085 &response,
1086 nullptr));
1087
1088 string value;
1089 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1090 EXPECT_EQ(fake_update_response_.cohort, value);
1091
1092 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1093 EXPECT_EQ(fake_update_response_.cohorthint, value);
1094
1095 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1096 EXPECT_EQ(fake_update_response_.cohortname, value);
1097}
1098
1099TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1100 OmahaResponse response;
1101 OmahaRequestParams params = request_params_;
1102 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1103 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1104 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1105 fake_update_response_.include_cohorts = true;
1106 fake_update_response_.cohort = "s/154454/8479665";
1107 fake_update_response_.cohorthint = "please-put-me-on-beta";
1108 fake_update_response_.cohortname = "";
1109
1110 ASSERT_TRUE(TestUpdateCheck(&params,
1111 fake_update_response_.GetUpdateResponse(),
1112 -1,
1113 false, // ping_only
1114 ErrorCode::kSuccess,
1115 metrics::CheckResult::kUpdateAvailable,
1116 metrics::CheckReaction::kUpdating,
1117 metrics::DownloadErrorCode::kUnset,
1118 &response,
1119 nullptr));
1120
1121 string value;
1122 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1123 EXPECT_EQ(fake_update_response_.cohort, value);
1124
1125 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1126 EXPECT_EQ(fake_update_response_.cohorthint, value);
1127
1128 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1129}
1130
1131TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1132 OmahaResponse response;
1133 OmahaRequestParams params = request_params_;
1134 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1135
1136 ASSERT_TRUE(TestUpdateCheck(&params,
1137 fake_update_response_.GetUpdateResponse(),
1138 -1,
1139 false, // ping_only
1140 ErrorCode::kSuccess,
1141 metrics::CheckResult::kUpdateAvailable,
1142 metrics::CheckReaction::kUpdating,
1143 metrics::DownloadErrorCode::kUnset,
1144 &response,
1145 nullptr));
1146
1147 string value;
1148 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1149 EXPECT_EQ("old_value", value);
1150
1151 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1152 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1153}
1154
Alex Deymo00d79ac2015-06-29 15:41:49 -07001155TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1156 OmahaResponse response;
1157 OmahaRequestParams params = request_params_;
1158 fake_update_response_.include_cohorts = true;
1159 fake_update_response_.cohort = "s/154454/8479665";
1160 fake_update_response_.cohorthint = "please-put-me-on-beta";
1161 fake_update_response_.cohortname = "stable";
1162
1163 ASSERT_TRUE(TestUpdateCheck(&params,
1164 fake_update_response_.GetNoUpdateResponse(),
1165 -1,
1166 false, // ping_only
1167 ErrorCode::kSuccess,
1168 metrics::CheckResult::kNoUpdateAvailable,
1169 metrics::CheckReaction::kUnset,
1170 metrics::DownloadErrorCode::kUnset,
1171 &response,
1172 nullptr));
1173
1174 string value;
1175 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1176 EXPECT_EQ(fake_update_response_.cohort, value);
1177
1178 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1179 EXPECT_EQ(fake_update_response_.cohorthint, value);
1180
1181 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1182 EXPECT_EQ(fake_update_response_.cohortname, value);
1183}
1184
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001185TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001186 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001187
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001188 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001189 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001190
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001191 OmahaRequestParams params = request_params_;
1192 fake_system_state_.set_request_params(&params);
1193 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001194 brillo::make_unique_ptr(
1195 new MockHttpFetcher(http_response.data(),
1196 http_response.size(),
1197 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001198 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001199 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001200 ActionProcessor processor;
1201 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001202 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001203
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001204 loop.PostTask(base::Bind(
1205 [](ActionProcessor* processor) { processor->StartProcessing(); },
1206 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001207 loop.Run();
1208 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001209 EXPECT_FALSE(processor.IsRunning());
1210}
1211
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001212TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001213 OmahaResponse response;
1214 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001215 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001216 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001217 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001218 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001219 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001220 metrics::CheckResult::kParsingError,
1221 metrics::CheckReaction::kUnset,
1222 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001223 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001224 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001225 EXPECT_FALSE(response.update_exists);
1226}
1227
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001228TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001229 OmahaResponse response;
1230 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001231 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001232 "",
1233 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001234 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001235 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001236 metrics::CheckResult::kParsingError,
1237 metrics::CheckReaction::kUnset,
1238 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001239 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001240 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001241 EXPECT_FALSE(response.update_exists);
1242}
1243
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001244TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001245 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001246 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001247 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001248 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1249 "<daystart elapsed_seconds=\"100\"/>"
1250 "<app appid=\"foo\" status=\"ok\">"
1251 "<ping status=\"ok\"/>"
1252 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001253 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001254 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001255 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001256 metrics::CheckResult::kParsingError,
1257 metrics::CheckReaction::kUnset,
1258 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001259 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001260 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001261 EXPECT_FALSE(response.update_exists);
1262}
1263
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001264TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001265 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001266 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001267 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001268 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1269 "<daystart elapsed_seconds=\"100\"/>"
1270 "<app appid=\"foo\" status=\"ok\">"
1271 "<ping status=\"ok\"/>"
1272 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001273 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001274 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001275 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001276 metrics::CheckResult::kParsingError,
1277 metrics::CheckReaction::kUnset,
1278 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001279 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001280 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001281 EXPECT_FALSE(response.update_exists);
1282}
1283
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001284TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001285 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001286 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001287 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001288 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1289 "<daystart elapsed_seconds=\"100\"/>"
1290 "<app appid=\"foo\" status=\"ok\">"
1291 "<ping status=\"ok\"/>"
1292 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001293 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001294 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001295 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001296 metrics::CheckResult::kParsingError,
1297 metrics::CheckReaction::kUnset,
1298 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001299 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001300 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001301 EXPECT_FALSE(response.update_exists);
1302}
1303
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001304TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001305 string input_response =
1306 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1307 "<daystart elapsed_seconds=\"100\"/>"
Aaron Wood7dcdedf2017-09-06 17:17:41 -07001308 // the appid needs to match that in the request params
1309 "<app appid=\"" +
1310 fake_update_response_.app_id +
1311 "\" status=\"ok\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001312 "<updatecheck status=\"ok\">"
1313 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001314 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001315 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001316 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001317 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001318 "ChromeOSVersion=\"10.2.3.4\" "
1319 "Prompt=\"false\" "
1320 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001321 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001322 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001323 "needsadmin=\"true\" "
1324 "/></actions></manifest></updatecheck></app></response>";
1325 LOG(INFO) << "Input Response = " << input_response;
1326
Darin Petkov6a5b3222010-07-13 14:55:28 -07001327 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001328 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001329 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001330 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001331 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001332 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001333 metrics::CheckResult::kUpdateAvailable,
1334 metrics::CheckReaction::kUpdating,
1335 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001336 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001337 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001338 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001339 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001340 EXPECT_EQ("http://missing/field/test/f",
1341 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001342 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001343 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1344 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001345 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001346 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001347}
1348
1349namespace {
1350class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1351 public:
1352 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001353 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001354 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001355};
1356
Alex Deymo60ca1a72015-06-18 18:19:15 -07001357void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001358 processor->StartProcessing();
1359 CHECK(processor->IsRunning());
1360 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001361}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001362} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001363
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001364TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001365 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001366 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001367
Alex Deymo60ca1a72015-06-18 18:19:15 -07001368 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001369 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001370 brillo::make_unique_ptr(
1371 new MockHttpFetcher(http_response.data(),
1372 http_response.size(),
1373 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001374 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001375 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001376 ActionProcessor processor;
1377 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001378 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001379
Alex Deymo60ca1a72015-06-18 18:19:15 -07001380 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1381 loop.Run();
1382 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001383}
1384
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001385TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001386 string output;
1387 EXPECT_TRUE(XmlEncode("ab", &output));
1388 EXPECT_EQ("ab", output);
1389 EXPECT_TRUE(XmlEncode("a<b", &output));
1390 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001391 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1392 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001393 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1394 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001395 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001396 EXPECT_FALSE(XmlEncode("\xc2", &output));
1397 // Fail with invalid ASCII-7 chars.
1398 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1399}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001400
Alex Deymob0d74eb2015-03-30 17:59:17 -07001401TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1402 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1403 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1404}
1405
1406TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001407 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001408
1409 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001410 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001411 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001412 OmahaRequestParams::kOsVersion,
1413 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001414 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001415 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001416 "0.1.0.0",
1417 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001418 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001419 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001420 "ChromeOSFirmware.1.0",
1421 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001422 false, // delta okay
1423 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001424 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001425 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001426 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1427 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1428 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001429 base::JoinString(
1430 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001431 OmahaResponse response;
1432 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001433 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001434 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001435 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001436 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001437 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001438 metrics::CheckResult::kParsingError,
1439 metrics::CheckReaction::kUnset,
1440 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001441 &response,
1442 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001443 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001444 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001445 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1446 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1447 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1448 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1449 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1450 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1451 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1452 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1453 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1454 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1455 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1456 // Values from Prefs that are too big are removed from the XML instead of
1457 // encoded.
1458 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001459}
1460
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001461TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001462 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001463 fake_update_response_.deadline = "&lt;20110101";
1464 fake_update_response_.more_info_url = "testthe&lt;url";
1465 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001466 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001467 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001468 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001469 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001470 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001471 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001472 metrics::CheckResult::kUpdateAvailable,
1473 metrics::CheckReaction::kUpdating,
1474 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001475 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001476 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001477
Sen Jiang0affc2c2017-02-10 15:55:05 -08001478 EXPECT_EQ("testthe<url", response.more_info_url);
1479 EXPECT_EQ("testthe&codebase/file.signed",
1480 response.packages[0].payload_urls[0]);
1481 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001482}
1483
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001484TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001485 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001486 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001487 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001488 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001489 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001490 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001491 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001492 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001493 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001494 metrics::CheckResult::kUpdateAvailable,
1495 metrics::CheckReaction::kUpdating,
1496 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001497 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001498 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001499
Sen Jiang0affc2c2017-02-10 15:55:05 -08001500 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001501}
1502
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001503TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001504 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001505 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001506 fake_system_state_.set_prefs(&prefs);
1507
Darin Petkov95508da2011-01-05 12:42:29 -08001508 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1509 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001510 // An existing but empty previous version means that we didn't reboot to a new
1511 // update, therefore, no need to update the previous version.
1512 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001513 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001514 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001515 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001516 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001517 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001518 metrics::CheckResult::kParsingError,
1519 metrics::CheckReaction::kUnset,
1520 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001521 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001522 &post_data));
1523 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001524 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001525 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001526 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001527 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001528 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001529 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1530 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001531 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1532 string::npos);
1533 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1534 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001535 // No <event> tag should be sent if we didn't reboot to an update.
1536 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001537}
1538
Jay Srinivasan0a708742012-03-20 11:26:12 -07001539
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001540TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001541 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001542 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001543 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1544 "invalid xml>",
1545 &post_data);
1546 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001547 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001548 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001549 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001550 OmahaEvent::kTypeUpdateDownloadStarted,
1551 OmahaEvent::kResultSuccess);
1552 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001553 EXPECT_EQ(post_str.find("ping"), string::npos);
1554 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001555}
1556
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001557TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001558 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001559 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001560 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1561 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001562 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001563 "invalid xml>",
1564 &post_data);
1565 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001566 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001567 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001568 " <event eventtype=\"%d\" eventresult=\"%d\" "
1569 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001570 OmahaEvent::kTypeDownloadComplete,
1571 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001572 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001573 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001574 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001575}
1576
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001577TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001578 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001579 // Create a copy of the OmahaRequestParams to reuse it later.
1580 OmahaRequestParams params = request_params_;
1581 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001582 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001583 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001584 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001585 brillo::make_unique_ptr(
1586 new MockHttpFetcher(http_response.data(),
1587 http_response.size(),
1588 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001589 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001590 EXPECT_FALSE(update_check_action.IsEvent());
1591
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001592 params = request_params_;
1593 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001594 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001595 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001596 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001597 brillo::make_unique_ptr(
1598 new MockHttpFetcher(http_response.data(),
1599 http_response.size(),
1600 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001601 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001602 EXPECT_TRUE(event_action.IsEvent());
1603}
1604
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001605TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001606 for (int i = 0; i < 2; i++) {
1607 bool delta_okay = i == 1;
1608 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001609 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001610 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001611 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001612 OmahaRequestParams::kOsVersion,
1613 "service_pack",
1614 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001615 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001616 "0.1.0.0",
1617 "en-US",
1618 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001619 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001620 "ChromeOSFirmware.1.0",
1621 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001622 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001623 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001624 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001625 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001626 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001627 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001628 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001629 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001630 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001631 metrics::CheckResult::kParsingError,
1632 metrics::CheckReaction::kUnset,
1633 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001634 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001635 &post_data));
1636 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001637 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001638 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1639 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001640 string::npos)
1641 << "i = " << i;
1642 }
1643}
1644
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001645TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001646 for (int i = 0; i < 2; i++) {
1647 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001648 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001649 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001650 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001651 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001652 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001653 OmahaRequestParams::kOsVersion,
1654 "service_pack",
1655 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001656 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001657 "0.1.0.0",
1658 "en-US",
1659 "unittest_track",
1660 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001661 "ChromeOSFirmware.1.0",
1662 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001663 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001664 interactive,
1665 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001666 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001667 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001668 "invalid xml>",
1669 -1,
1670 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001671 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001672 metrics::CheckResult::kParsingError,
1673 metrics::CheckReaction::kUnset,
1674 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001675 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001676 &post_data));
1677 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001678 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001679 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1680 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001681 string::npos)
1682 << "i = " << i;
1683 }
1684}
1685
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001686TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001687 OmahaEvent default_event;
1688 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1689 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001690 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001691
1692 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1693 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1694 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001695 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001696
1697 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1698 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001699 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001700 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1701 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001702 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001703}
1704
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001705void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001706 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001707 fake_system_state_.set_prefs(&prefs);
1708 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1709 .Times(AnyNumber());
1710 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1711 // Add a few hours to the day difference to test no rounding, etc.
1712 int64_t five_days_ago =
1713 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1714 int64_t six_days_ago =
1715 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1716 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1717 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1718 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1719 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1720 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1721 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001722 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001723 ASSERT_TRUE(
1724 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001725 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001726 -1,
1727 ping_only,
1728 ErrorCode::kSuccess,
1729 metrics::CheckResult::kUnset,
1730 metrics::CheckReaction::kUnset,
1731 metrics::DownloadErrorCode::kUnset,
1732 nullptr,
1733 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001734 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001735 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1736 string::npos);
1737 if (ping_only) {
1738 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1739 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1740 } else {
1741 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1742 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001743 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001744}
1745
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001746TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1747 PingTest(true /* ping_only */);
1748}
1749
1750TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1751 PingTest(false /* ping_only */);
1752}
1753
1754TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001755 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001756 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001757 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1758 .Times(AnyNumber());
1759 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001760 int64_t three_days_ago =
1761 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1762 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001763 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1764 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001765 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1766 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1767 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1768 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001769 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001770 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001771 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001772 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001773 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001774 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001775 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001776 metrics::CheckResult::kNoUpdateAvailable,
1777 metrics::CheckReaction::kUnset,
1778 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001779 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001780 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001781 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001782 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001783 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001784}
1785
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001786TEST_F(OmahaRequestActionTest, RollCallPingTest) {
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 four_days_ago =
1793 (Time::Now() - TimeDelta::FromHours(4 * 24)).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>(now), Return(true)));
1799 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1800 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), 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\" r=\"4\"></ping>\n"),
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, NoPingTest) {
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 one_hour_ago =
1825 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001826 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1827 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001828 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1829 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1830 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1831 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001832 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1833 // send a ping.
1834 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1835 .WillOnce(Return(true));
1836 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1837 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001838 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001839 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001840 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001841 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001842 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001843 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001844 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001845 metrics::CheckResult::kNoUpdateAvailable,
1846 metrics::CheckReaction::kUnset,
1847 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001848 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001849 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001850 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001851 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001852}
1853
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001854TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001855 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001856 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001857 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001858 int64_t now = Time::Now().ToInternalValue();
1859 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1860 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1861 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1862 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1863 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1864 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001865 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001866 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001867 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001868 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001869 -1,
1870 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001871 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001872 metrics::CheckResult::kUnset,
1873 metrics::CheckReaction::kUnset,
1874 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001875 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001876 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001877 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001878}
1879
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001880TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001881 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001882 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001883 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1884 .Times(AnyNumber());
1885 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001886 int64_t future =
1887 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001888 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1889 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001890 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1891 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1892 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1893 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1894 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1895 .WillOnce(Return(true));
1896 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1897 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001898 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001899 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001900 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001901 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1902 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001903 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001904 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001905 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001906 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001907 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001908 metrics::CheckResult::kNoUpdateAvailable,
1909 metrics::CheckReaction::kUnset,
1910 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001911 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001912 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001913 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001914 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001915}
1916
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001917TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001918 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001919 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001920 // may fail if it runs for longer than 5 seconds. It shouldn't run
1921 // that long though.
1922 int64_t midnight =
1923 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1924 int64_t midnight_slack =
1925 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001926 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001927 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001928 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1929 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001930 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1931 AllOf(Ge(midnight), Le(midnight_slack))))
1932 .WillOnce(Return(true));
1933 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1934 AllOf(Ge(midnight), Le(midnight_slack))))
1935 .WillOnce(Return(true));
1936 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001937 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001938 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1939 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001940 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001941 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001942 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001943 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001944 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001945 metrics::CheckResult::kNoUpdateAvailable,
1946 metrics::CheckReaction::kUnset,
1947 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001948 nullptr,
1949 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001950}
1951
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001952TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001953 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001954 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001955 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1956 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001957 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1958 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1959 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001960 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001961 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1962 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001963 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001964 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001965 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001966 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001967 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001968 metrics::CheckResult::kNoUpdateAvailable,
1969 metrics::CheckReaction::kUnset,
1970 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001971 nullptr,
1972 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001973}
1974
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001975TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001976 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001977 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001978 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1979 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001980 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1981 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1982 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001983 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001984 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1985 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001986 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001987 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001988 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001989 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001990 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001991 metrics::CheckResult::kNoUpdateAvailable,
1992 metrics::CheckReaction::kUnset,
1993 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001994 nullptr,
1995 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001996}
1997
Alex Deymob3fa53b2016-04-18 19:57:58 -07001998TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
1999 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
2000 // the "eol" attribute.
2001 ASSERT_TRUE(
2002 TestUpdateCheck(nullptr, // request_params
2003 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
2004 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
2005 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
2006 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
2007 "</app></response>",
2008 -1,
2009 false, // ping_only
2010 ErrorCode::kSuccess,
2011 metrics::CheckResult::kNoUpdateAvailable,
2012 metrics::CheckReaction::kUnset,
2013 metrics::DownloadErrorCode::kUnset,
2014 nullptr,
2015 nullptr));
2016 string eol_pref;
2017 EXPECT_TRUE(
2018 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
2019 // Note that the eol="eol" attribute should be ignored and the _eol should be
2020 // used instead.
2021 EXPECT_EQ("security-only", eol_pref);
2022}
2023
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002024TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002025 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002026 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07002027 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07002028 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07002029 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002030 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002031 metrics::CheckResult::kParsingError,
2032 metrics::CheckReaction::kUnset,
2033 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002034 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07002035 &post_data));
2036 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002037 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07002038 EXPECT_EQ(post_str.find("machineid="), string::npos);
2039 EXPECT_EQ(post_str.find("userid="), string::npos);
2040}
2041
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002042TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002043 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002044 const int http_error_code =
2045 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07002046 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002047 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002048 "",
2049 501,
Darin Petkov265f2902011-05-09 15:17:40 -07002050 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002051 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002052 metrics::CheckResult::kDownloadError,
2053 metrics::CheckReaction::kUnset,
2054 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07002055 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002056 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002057 EXPECT_FALSE(response.update_exists);
2058}
2059
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002060TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002061 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002062 const int http_error_code =
2063 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07002064 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002065 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002066 "",
2067 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07002068 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002069 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002070 metrics::CheckResult::kDownloadError,
2071 metrics::CheckReaction::kUnset,
2072 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07002073 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002074 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002075 EXPECT_FALSE(response.update_exists);
2076}
2077
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002078TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002079 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002080 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002081 params.set_wall_clock_based_wait_enabled(true);
2082 params.set_waiting_period(TimeDelta().FromDays(1));
2083 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002084
Sen Jiang7c1171e2016-06-23 11:35:40 -07002085 Time arbitrary_date;
2086 Time::FromString("6/4/1989", &arbitrary_date);
2087 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
2088 ASSERT_FALSE(TestUpdateCheck(&params,
2089 fake_update_response_.GetUpdateResponse(),
2090 -1,
2091 false, // ping_only
2092 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2093 metrics::CheckResult::kUpdateAvailable,
2094 metrics::CheckReaction::kDeferring,
2095 metrics::DownloadErrorCode::kUnset,
2096 &response,
2097 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002098
Ben Chan9abb7632014-08-07 00:10:53 -07002099 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002100 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002101 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002102 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002103
2104 // Verify if we are interactive check we don't defer.
2105 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002106 ASSERT_TRUE(TestUpdateCheck(&params,
2107 fake_update_response_.GetUpdateResponse(),
2108 -1,
2109 false, // ping_only
2110 ErrorCode::kSuccess,
2111 metrics::CheckResult::kUpdateAvailable,
2112 metrics::CheckReaction::kUpdating,
2113 metrics::DownloadErrorCode::kUnset,
2114 &response,
2115 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002116 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002117}
2118
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002119TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002120 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002121 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002122 params.set_wall_clock_based_wait_enabled(true);
2123 params.set_waiting_period(TimeDelta().FromDays(1));
2124 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002125
Sen Jiang7c1171e2016-06-23 11:35:40 -07002126 Time t1, t2;
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002127 Time::FromString("1/1/2012", &t1);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002128 Time::FromString("1/3/2012", &t2);
2129 ASSERT_TRUE(
2130 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2131 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2132 ASSERT_TRUE(TestUpdateCheck(&params,
2133 fake_update_response_.GetUpdateResponse(),
2134 -1,
2135 false, // ping_only
2136 ErrorCode::kSuccess,
2137 metrics::CheckResult::kUpdateAvailable,
2138 metrics::CheckReaction::kUpdating,
2139 metrics::DownloadErrorCode::kUnset,
2140 &response,
2141 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002142
2143 EXPECT_TRUE(response.update_exists);
2144
2145 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002146 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002147 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002148 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2149}
2150
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002151TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002152 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002153 base::ScopedTempDir tempdir;
2154 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002155
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002156 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002157 OmahaRequestParams params(&fake_system_state_);
2158 params.set_root(tempdir.path().value());
2159 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2160 params.set_app_version("1.2.3.4");
2161 params.set_current_channel("canary-channel");
2162 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2163 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002164 EXPECT_TRUE(params.to_more_stable_channel());
2165 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002166 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002167 "invalid xml>",
2168 -1,
2169 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002170 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002171 metrics::CheckResult::kParsingError,
2172 metrics::CheckReaction::kUnset,
2173 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002174 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002175 &post_data));
2176 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002177 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002178 EXPECT_NE(string::npos, post_str.find(
2179 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2180 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2181 "track=\"stable-channel\" from_track=\"canary-channel\" "));
2182}
2183
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002184TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002185 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002186 base::ScopedTempDir tempdir;
2187 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002188
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002189 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002190 OmahaRequestParams params(&fake_system_state_);
2191 params.set_root(tempdir.path().value());
2192 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2193 params.set_app_version("5.6.7.8");
2194 params.set_current_channel("stable-channel");
2195 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2196 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002197 EXPECT_FALSE(params.to_more_stable_channel());
2198 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002199 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002200 "invalid xml>",
2201 -1,
2202 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002203 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002204 metrics::CheckResult::kParsingError,
2205 metrics::CheckReaction::kUnset,
2206 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002207 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002208 &post_data));
2209 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002210 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002211 EXPECT_NE(string::npos, post_str.find(
2212 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2213 "version=\"5.6.7.8\" "
2214 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002215 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002216}
2217
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002218// Checks that the initial ping with a=-1 r=-1 is not send when the device
2219// was powerwashed.
2220TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2221 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2222
2223 // Flag that the device was powerwashed in the past.
2224 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2225
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002226 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002227 ASSERT_TRUE(
2228 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002229 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002230 -1,
2231 false, // ping_only
2232 ErrorCode::kSuccess,
2233 metrics::CheckResult::kNoUpdateAvailable,
2234 metrics::CheckReaction::kUnset,
2235 metrics::DownloadErrorCode::kUnset,
2236 nullptr,
2237 &post_data));
2238 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002239 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002240 EXPECT_EQ(string::npos, post_str.find("<ping"));
2241}
2242
Alex Deymo9fded1e2015-11-05 12:31:19 -08002243// Checks that the event 54 is sent on a reboot to a new update.
2244TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2245 // Flag that the device was updated in a previous boot.
2246 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2247
2248 brillo::Blob post_data;
2249 ASSERT_TRUE(
2250 TestUpdateCheck(nullptr, // request_params
2251 fake_update_response_.GetNoUpdateResponse(),
2252 -1,
2253 false, // ping_only
2254 ErrorCode::kSuccess,
2255 metrics::CheckResult::kNoUpdateAvailable,
2256 metrics::CheckReaction::kUnset,
2257 metrics::DownloadErrorCode::kUnset,
2258 nullptr,
2259 &post_data));
2260 string post_str(post_data.begin(), post_data.end());
2261
2262 // An event 54 is included and has the right version.
2263 EXPECT_NE(string::npos,
2264 post_str.find(base::StringPrintf(
2265 "<event eventtype=\"%d\"",
2266 OmahaEvent::kTypeRebootedAfterUpdate)));
2267 EXPECT_NE(string::npos,
2268 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2269
2270 // The previous version flag should have been removed.
2271 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2272 string prev_version;
2273 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2274 EXPECT_TRUE(prev_version.empty());
2275}
2276
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002277void OmahaRequestActionTest::P2PTest(
2278 bool initial_allow_p2p_for_downloading,
2279 bool initial_allow_p2p_for_sharing,
2280 bool omaha_disable_p2p_for_downloading,
2281 bool omaha_disable_p2p_for_sharing,
2282 bool payload_state_allow_p2p_attempt,
2283 bool expect_p2p_client_lookup,
2284 const string& p2p_client_result_url,
2285 bool expected_allow_p2p_for_downloading,
2286 bool expected_allow_p2p_for_sharing,
2287 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002288 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002289 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002290 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2291 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2292 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002293
2294 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002295 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002296 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2297 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002298 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2299 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2300 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2301 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2302 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2303 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2304 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2305 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2306 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2307 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2308
David Zeuthen8f191b22013-08-06 12:27:50 -07002309 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002310 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002311 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2312
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002313 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2314 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002315 .Times(expect_p2p_client_lookup ? 1 : 0);
2316
Alex Deymo8e18f932015-03-27 16:16:59 -07002317 fake_update_response_.disable_p2p_for_downloading =
2318 omaha_disable_p2p_for_downloading;
2319 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002320 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002321 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002322 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002323 -1,
2324 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002325 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002326 metrics::CheckResult::kUpdateAvailable,
2327 metrics::CheckReaction::kUpdating,
2328 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002329 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002330 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002331 EXPECT_TRUE(response.update_exists);
2332
Gilad Arnold74b5f552014-10-07 08:17:16 -07002333 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2334 response.disable_p2p_for_downloading);
2335 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2336 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002337
Gilad Arnold74b5f552014-10-07 08:17:16 -07002338 EXPECT_EQ(expected_allow_p2p_for_downloading,
2339 actual_allow_p2p_for_downloading);
2340 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2341 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002342}
2343
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002344TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002345 P2PTest(true, // initial_allow_p2p_for_downloading
2346 true, // initial_allow_p2p_for_sharing
2347 false, // omaha_disable_p2p_for_downloading
2348 false, // omaha_disable_p2p_for_sharing
2349 true, // payload_state_allow_p2p_attempt
2350 true, // expect_p2p_client_lookup
2351 "http://1.3.5.7/p2p", // p2p_client_result_url
2352 true, // expected_allow_p2p_for_downloading
2353 true, // expected_allow_p2p_for_sharing
2354 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002355}
2356
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002357TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002358 P2PTest(true, // initial_allow_p2p_for_downloading
2359 true, // initial_allow_p2p_for_sharing
2360 false, // omaha_disable_p2p_for_downloading
2361 false, // omaha_disable_p2p_for_sharing
2362 true, // payload_state_allow_p2p_attempt
2363 true, // expect_p2p_client_lookup
2364 "", // p2p_client_result_url
2365 false, // expected_allow_p2p_for_downloading
2366 true, // expected_allow_p2p_for_sharing
2367 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002368}
2369
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002370TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002371 P2PTest(false, // initial_allow_p2p_for_downloading
2372 true, // initial_allow_p2p_for_sharing
2373 false, // omaha_disable_p2p_for_downloading
2374 false, // omaha_disable_p2p_for_sharing
2375 true, // payload_state_allow_p2p_attempt
2376 false, // expect_p2p_client_lookup
2377 "unset", // p2p_client_result_url
2378 false, // expected_allow_p2p_for_downloading
2379 true, // expected_allow_p2p_for_sharing
2380 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002381}
2382
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002383TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002384 P2PTest(true, // initial_allow_p2p_for_downloading
2385 true, // initial_allow_p2p_for_sharing
2386 true, // omaha_disable_p2p_for_downloading
2387 false, // omaha_disable_p2p_for_sharing
2388 true, // payload_state_allow_p2p_attempt
2389 false, // expect_p2p_client_lookup
2390 "unset", // p2p_client_result_url
2391 false, // expected_allow_p2p_for_downloading
2392 true, // expected_allow_p2p_for_sharing
2393 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002394}
2395
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002396TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002397 P2PTest(true, // initial_allow_p2p_for_downloading
2398 true, // initial_allow_p2p_for_sharing
2399 false, // omaha_disable_p2p_for_downloading
2400 true, // omaha_disable_p2p_for_sharing
2401 true, // payload_state_allow_p2p_attempt
2402 true, // expect_p2p_client_lookup
2403 "http://1.3.5.7/p2p", // p2p_client_result_url
2404 true, // expected_allow_p2p_for_downloading
2405 false, // expected_allow_p2p_for_sharing
2406 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002407}
2408
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002409TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002410 P2PTest(true, // initial_allow_p2p_for_downloading
2411 true, // initial_allow_p2p_for_sharing
2412 true, // omaha_disable_p2p_for_downloading
2413 true, // omaha_disable_p2p_for_sharing
2414 true, // payload_state_allow_p2p_attempt
2415 false, // expect_p2p_client_lookup
2416 "unset", // p2p_client_result_url
2417 false, // expected_allow_p2p_for_downloading
2418 false, // expected_allow_p2p_for_sharing
2419 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002420}
2421
Alex Deymof329b932014-10-30 01:37:48 -07002422bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2423 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002424 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002425 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002426 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002427 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002428 -1,
2429 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002430 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002431 metrics::CheckResult::kUpdateAvailable,
2432 metrics::CheckReaction::kUpdating,
2433 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002434 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002435 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002436}
2437
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002438TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002439 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002440
Kevin Cernekee2494e282016-03-29 18:03:53 -07002441 // Simulate a successful update check that happens during OOBE. The
2442 // deadline in the response is needed to force the update attempt to
2443 // occur; responses without a deadline seen during OOBE will normally
2444 // return ErrorCode::kNonCriticalUpdateInOOBE.
2445 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2446 fake_update_response_.deadline = "20101020";
2447
David Zeuthen639aa362014-02-03 16:23:44 -08002448 // Check that we parse elapsed_days in the Omaha Response correctly.
2449 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002450 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2451 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002452 EXPECT_TRUE(response.update_exists);
2453 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002454 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002455 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002456 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002457 EXPECT_EQ(prefs_days, 42);
2458
2459 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002460 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002461 EXPECT_TRUE(response.update_exists);
2462 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002463 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002464 EXPECT_EQ(prefs_days, 42);
2465
2466 // Note that elapsed_days is not necessarily divisible by 7 so check
2467 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002468 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2469 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002470 EXPECT_TRUE(response.update_exists);
2471 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002472 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002473 EXPECT_EQ(prefs_days, 21);
2474
2475 // Check that we correctly handle elapsed_days not being included in
2476 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002477 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002478 EXPECT_TRUE(response.update_exists);
2479 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002480}
2481
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002482// If there is no prefs and OOBE is not complete, we should not
2483// report anything to Omaha.
2484TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002485 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002486 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2487 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2488}
David Zeuthen639aa362014-02-03 16:23:44 -08002489
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002490// If OOBE is complete and happened on a valid date (e.g. after Jan
2491// 1 2007 0:00 PST), that date should be used and written to
2492// prefs. However, first try with an invalid date and check we do
2493// nothing.
2494TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2495 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2496 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2497 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2498 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2499}
David Zeuthen639aa362014-02-03 16:23:44 -08002500
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002501// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2502// should yield an InstallDate of 14.
2503TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2504 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2505 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2506 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2507 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002508
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002509 int64_t prefs_days;
2510 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2511 EXPECT_EQ(prefs_days, 14);
2512}
David Zeuthen639aa362014-02-03 16:23:44 -08002513
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002514// Now that we have a valid date in prefs, check that we keep using
2515// that even if OOBE date reports something else. The date Jan 30,
2516// 2007 0:00 PST should yield an InstallDate of 28... but since
2517// there's a prefs file, we should still get 14.
2518TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2519 // Set a valid date in the prefs first.
2520 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002521
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002522 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2523 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2524 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002525
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002526 int64_t prefs_days;
2527 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2528 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002529
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002530 // If we delete the prefs file, we should get 28 days.
2531 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2532 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2533 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2534 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002535}
2536
Darin Petkov6a5b3222010-07-13 14:55:28 -07002537} // namespace chromeos_update_engine