blob: 90910317b65a8e15a2b21f5f2dc90a27a05b7d9b [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";
71
Alex Deymo8e18f932015-03-27 16:16:59 -070072// This is a helper struct to allow unit tests build an update response with the
73// values they care about.
74struct FakeUpdateResponse {
75 string GetNoUpdateResponse() const {
76 string entity_str;
77 if (include_entity)
78 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Sen Jiang81259682017-03-30 15:11:30 -070079 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" + entity_str +
80 "<response protocol=\"3.0\">"
81 "<daystart elapsed_seconds=\"100\"/>"
82 "<app appid=\"" +
83 app_id + "\" " +
84 (include_cohorts
85 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
86 "\" cohortname=\"" + cohortname + "\" "
87 : "") +
88 " status=\"ok\">"
89 "<ping status=\"ok\"/>"
90 "<updatecheck status=\"noupdate\"/></app>" +
91 (multi_app_no_update
92 ? "<app><updatecheck status=\"noupdate\"/></app>"
93 : "") +
94 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -070095 }
96
97 string GetUpdateResponse() const {
Sen Jiang2703ef42017-03-16 13:36:21 -070098 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
99 "protocol=\"3.0\">"
100 "<daystart elapsed_seconds=\"100\"" +
101 (elapsed_days.empty() ? ""
102 : (" elapsed_days=\"" + elapsed_days + "\"")) +
103 "/>"
104 "<app appid=\"" +
105 app_id + "\" " +
106 (include_cohorts
107 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
108 "\" cohortname=\"" + cohortname + "\" "
109 : "") +
110 " status=\"ok\">"
111 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
112 "<urls><url codebase=\"" +
113 codebase +
114 "\"/></urls>"
115 "<manifest version=\"" +
116 version +
117 "\">"
118 "<packages><package hash=\"not-used\" name=\"" +
119 filename + "\" size=\"" + base::Int64ToString(size) +
Sen Jiang0affc2c2017-02-10 15:55:05 -0800120 "\" hash_sha256=\"" + hash + "\"/>" +
121 (multi_package ? "<package name=\"package2\" size=\"222\" "
122 "hash_sha256=\"hash2\"/>"
123 : "") +
124 "</packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700125 "<actions><action event=\"postinstall\" MetadataSize=\"11" +
126 (multi_package ? ":22" : "") + "\" ChromeOSVersion=\"" + version +
127 "\" MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt +
Sen Jiang2703ef42017-03-16 13:36:21 -0700128 "\" "
129 "IsDelta=\"true\" "
Sen Jiangcdd52062017-05-18 15:33:10 -0700130 "IsDeltaPayload=\"true" +
131 (multi_package ? ":false" : "") +
132 "\" "
Sen Jiang2703ef42017-03-16 13:36:21 -0700133 "MaxDaysToScatter=\"" +
134 max_days_to_scatter +
135 "\" "
136 "sha256=\"not-used\" "
137 "needsadmin=\"" +
138 needsadmin + "\" " +
139 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
140 (disable_p2p_for_downloading ? "DisableP2PForDownloading=\"true\" "
141 : "") +
142 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
Sen Jiang81259682017-03-30 15:11:30 -0700143 "/></actions></manifest></updatecheck></app>" +
144 (multi_app
145 ? "<app><updatecheck status=\"ok\"><urls><url codebase=\"" +
146 codebase2 +
147 "\"/></urls><manifest><packages>"
148 "<package name=\"package3\" size=\"333\" "
149 "hash_sha256=\"hash3\"/></packages>"
Sen Jiang00adf7b2017-06-26 15:57:29 -0700150 "<actions><action event=\"postinstall\" " +
151 (multi_app_self_update
152 ? "noupdate=\"true\" IsDeltaPayload=\"true\" "
153 : "IsDeltaPayload=\"false\" ") +
154 "MetadataSize=\"33\"/></actions>"
Sen Jiang81259682017-03-30 15:11:30 -0700155 "</manifest></updatecheck></app>"
156 : "") +
157 (multi_app_no_update
158 ? "<app><updatecheck status=\"noupdate\"/></app>"
159 : "") +
160 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700161 }
162
163 // Return the payload URL, which is split in two fields in the XML response.
164 string GetPayloadUrl() {
165 return codebase + filename;
166 }
167
Alex Deymo85616652015-10-15 18:48:31 -0700168 string app_id = kTestAppId;
Alex Deymo8e18f932015-03-27 16:16:59 -0700169 string version = "1.2.3.4";
170 string more_info_url = "http://more/info";
171 string prompt = "true";
172 string codebase = "http://code/base/";
Sen Jiang81259682017-03-30 15:11:30 -0700173 string codebase2 = "http://code/base/2/";
Alex Deymo8e18f932015-03-27 16:16:59 -0700174 string filename = "file.signed";
Sen Jiang2703ef42017-03-16 13:36:21 -0700175 string hash = "4841534831323334";
Alex Deymo8e18f932015-03-27 16:16:59 -0700176 string needsadmin = "false";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800177 uint64_t size = 123;
Alex Deymo8e18f932015-03-27 16:16:59 -0700178 string deadline = "";
179 string max_days_to_scatter = "7";
180 string elapsed_days = "42";
181
182 // P2P setting defaults to allowed.
183 bool disable_p2p_for_downloading = false;
184 bool disable_p2p_for_sharing = false;
185
186 // Omaha cohorts settings.
187 bool include_cohorts = false;
188 string cohort = "";
189 string cohorthint = "";
190 string cohortname = "";
191
192 // Whether to include the CrOS <!ENTITY> in the XML response.
193 bool include_entity = false;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800194
Sen Jiang81259682017-03-30 15:11:30 -0700195 // Whether to include more than one app.
196 bool multi_app = false;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700197 // Whether to include an app with noupdate="true".
198 bool multi_app_self_update = false;
199 // Whether to include an additional app with status="noupdate".
Sen Jiang81259682017-03-30 15:11:30 -0700200 bool multi_app_no_update = false;
201 // Whether to include more than one package in an app.
Sen Jiang0affc2c2017-02-10 15:55:05 -0800202 bool multi_package = false;
Alex Deymo8e18f932015-03-27 16:16:59 -0700203};
204
205} // namespace
206
Darin Petkov6a5b3222010-07-13 14:55:28 -0700207namespace chromeos_update_engine {
208
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700209class OmahaRequestActionTest : public ::testing::Test {
210 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800211 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700212 fake_system_state_.set_request_params(&request_params_);
213 fake_system_state_.set_prefs(&fake_prefs_);
214 }
215
216 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800217 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700218 // is used. |payload_state| may be null, in which case a local mock is used.
219 // |p2p_manager| may be null, in which case a local mock is used.
220 // |connection_manager| may be null, in which case a local mock is used.
221 // out_response may be null. If |fail_http_response_code| is non-negative,
222 // the transfer will fail with that code. |ping_only| is passed through to the
223 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
224 // post-data received by the mock HttpFetcher is returned.
225 //
226 // The |expected_check_result|, |expected_check_reaction| and
227 // |expected_error_code| parameters are for checking expectations
228 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
229 // UMA statistics. Use the appropriate ::kUnset value to specify that
230 // the given metric should not be reported.
231 bool TestUpdateCheck(OmahaRequestParams* request_params,
232 const string& http_response,
233 int fail_http_response_code,
234 bool ping_only,
235 ErrorCode expected_code,
236 metrics::CheckResult expected_check_result,
237 metrics::CheckReaction expected_check_reaction,
238 metrics::DownloadErrorCode expected_download_error_code,
239 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700240 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700241
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800242 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700243 // only a ping or also an updatecheck.
244 void PingTest(bool ping_only);
245
246 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700247 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700248 OmahaResponse *response);
249
250 // P2P test helper function.
251 void P2PTest(
252 bool initial_allow_p2p_for_downloading,
253 bool initial_allow_p2p_for_sharing,
254 bool omaha_disable_p2p_for_downloading,
255 bool omaha_disable_p2p_for_sharing,
256 bool payload_state_allow_p2p_attempt,
257 bool expect_p2p_client_lookup,
258 const string& p2p_client_result_url,
259 bool expected_allow_p2p_for_downloading,
260 bool expected_allow_p2p_for_sharing,
261 const string& expected_p2p_url);
262
263 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700264 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700265
266 // By default, all tests use these objects unless they replace them in the
267 // fake_system_state_.
268 OmahaRequestParams request_params_ = OmahaRequestParams{
269 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700270 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700271 OmahaRequestParams::kOsVersion,
272 "service_pack",
273 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700274 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700275 "0.1.0.0",
276 "en-US",
277 "unittest",
278 "OEM MODEL 09235 7471",
279 "ChromeOSFirmware.1.0",
280 "0X0A1",
281 false, // delta okay
282 false, // interactive
283 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700284 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700285
286 FakePrefs fake_prefs_;
287};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700288
289namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700290class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
291 public:
292 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700293 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800294 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700295 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800296 void ProcessingDone(const ActionProcessor* processor,
297 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700298 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700299 }
300
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800301 void ActionCompleted(ActionProcessor* processor,
302 AbstractAction* action,
303 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700304 // make sure actions always succeed
305 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700306 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700307 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700308 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700309 }
David Zeuthena99981f2013-04-29 13:42:47 -0700310 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700311};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700312} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700313
314class OutputObjectCollectorAction;
315
316template<>
317class ActionTraits<OutputObjectCollectorAction> {
318 public:
319 // Does not take an object for input
320 typedef OmahaResponse InputObjectType;
321 // On success, puts the output path on output
322 typedef NoneType OutputObjectType;
323};
324
325class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
326 public:
327 OutputObjectCollectorAction() : has_input_object_(false) {}
328 void PerformAction() {
329 // copy input object
330 has_input_object_ = HasInputObject();
331 if (has_input_object_)
332 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700333 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700334 }
335 // Should never be called
336 void TerminateProcessing() {
337 CHECK(false);
338 }
339 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700340 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700341 return "OutputObjectCollectorAction";
342 }
Alex Deymof329b932014-10-30 01:37:48 -0700343 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800344 using InputObjectType =
345 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
346 using OutputObjectType =
347 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700348 bool has_input_object_;
349 OmahaResponse omaha_response_;
350};
351
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700352bool OmahaRequestActionTest::TestUpdateCheck(
353 OmahaRequestParams* request_params,
354 const string& http_response,
355 int fail_http_response_code,
356 bool ping_only,
357 ErrorCode expected_code,
358 metrics::CheckResult expected_check_result,
359 metrics::CheckReaction expected_check_reaction,
360 metrics::DownloadErrorCode expected_download_error_code,
361 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700362 brillo::Blob* out_post_data) {
363 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700364 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700365 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800366 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700367 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700368 if (fail_http_response_code >= 0) {
369 fetcher->FailTransfer(fail_http_response_code);
370 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700371 if (request_params)
372 fake_system_state_.set_request_params(request_params);
373 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700374 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300375 brillo::make_unique_ptr(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700376 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700377 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700378 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700379
Darin Petkov6a5b3222010-07-13 14:55:28 -0700380 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700381 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700382 processor.EnqueueAction(&action);
383
384 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700385 BondActions(&action, &collector_action);
386 processor.EnqueueAction(&collector_action);
387
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700388 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800389 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700390 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800391 SendEnumToUMA(metrics::kMetricCheckResult,
392 static_cast<int>(expected_check_result),
393 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
394 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
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::kMetricCheckReaction,
397 static_cast<int>(expected_check_reaction),
398 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
399 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700400 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700401 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
402 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800403 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
404 ? 0 : 1);
405
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700406 loop.PostTask(base::Bind(
407 [](ActionProcessor* processor) { processor->StartProcessing(); },
408 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700409 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700410 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700411 if (collector_action.has_input_object_ && out_response)
412 *out_response = collector_action.omaha_response_;
413 if (out_post_data)
414 *out_post_data = fetcher->post_data();
415 return collector_action.has_input_object_;
416}
417
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700418// Tests Event requests -- they should always succeed. |out_post_data|
419// may be null; if non-null, the post-data received by the mock
420// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700421void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700422 OmahaEvent* event,
423 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700424 brillo::Blob* out_post_data) {
425 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700426 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700427 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800428 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700429 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700430 FakeSystemState fake_system_state;
431 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300432 OmahaRequestAction action(&fake_system_state,
433 event,
434 brillo::make_unique_ptr(fetcher),
435 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700436 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700437 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700438 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700439 processor.EnqueueAction(&action);
440
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700441 loop.PostTask(base::Bind(
442 [](ActionProcessor* processor) { processor->StartProcessing(); },
443 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700444 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700445 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700446
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700447 if (out_post_data)
448 *out_post_data = fetcher->post_data();
449}
450
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700451TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400452 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700453 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400454 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700455 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700456 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400457 -1,
458 false, // ping_only
459 ErrorCode::kOmahaRequestXMLHasEntityDecl,
460 metrics::CheckResult::kParsingError,
461 metrics::CheckReaction::kUnset,
462 metrics::DownloadErrorCode::kUnset,
463 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700464 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400465 EXPECT_FALSE(response.update_exists);
466}
467
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700468TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700469 OmahaResponse response;
470 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700471 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700472 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700473 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700474 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700475 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800476 metrics::CheckResult::kNoUpdateAvailable,
477 metrics::CheckReaction::kUnset,
478 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700479 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700480 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700481 EXPECT_FALSE(response.update_exists);
482}
483
Sen Jiang81259682017-03-30 15:11:30 -0700484TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
485 OmahaResponse response;
486 fake_update_response_.multi_app_no_update = true;
487 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
488 fake_update_response_.GetNoUpdateResponse(),
489 -1,
490 false, // ping_only
491 ErrorCode::kSuccess,
492 metrics::CheckResult::kNoUpdateAvailable,
493 metrics::CheckReaction::kUnset,
494 metrics::DownloadErrorCode::kUnset,
495 &response,
496 nullptr));
497 EXPECT_FALSE(response.update_exists);
498}
499
Sen Jiang00adf7b2017-06-26 15:57:29 -0700500TEST_F(OmahaRequestActionTest, MultiAppNoPartialUpdateTest) {
501 OmahaResponse response;
502 fake_update_response_.multi_app_no_update = true;
503 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
504 fake_update_response_.GetUpdateResponse(),
505 -1,
506 false, // ping_only
507 ErrorCode::kSuccess,
508 metrics::CheckResult::kNoUpdateAvailable,
509 metrics::CheckReaction::kUnset,
510 metrics::DownloadErrorCode::kUnset,
511 &response,
512 nullptr));
513 EXPECT_FALSE(response.update_exists);
514}
515
516TEST_F(OmahaRequestActionTest, NoSelfUpdateTest) {
517 OmahaResponse response;
518 ASSERT_TRUE(TestUpdateCheck(
519 nullptr, // request_params
520 "<response><app><updatecheck status=\"ok\"><manifest><actions><action "
521 "event=\"postinstall\" noupdate=\"true\"/></actions>"
522 "</manifest></updatecheck></app></response>",
523 -1,
524 false, // ping_only
525 ErrorCode::kSuccess,
526 metrics::CheckResult::kNoUpdateAvailable,
527 metrics::CheckReaction::kUnset,
528 metrics::DownloadErrorCode::kUnset,
529 &response,
530 nullptr));
531 EXPECT_FALSE(response.update_exists);
532}
533
Alex Deymo8e18f932015-03-27 16:16:59 -0700534// Test that all the values in the response are parsed in a normal update
535// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700536TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700537 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700538 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700539 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700540 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700541 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700542 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700543 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700544 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800545 metrics::CheckResult::kUpdateAvailable,
546 metrics::CheckReaction::kUpdating,
547 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700548 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700549 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700550 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700551 EXPECT_EQ(fake_update_response_.version, response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800552 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
553 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700554 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800555 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
556 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700557 EXPECT_EQ(true, response.packages[0].is_delta);
Alex Deymo8e18f932015-03-27 16:16:59 -0700558 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
559 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
560 // Omaha cohort attribets are not set in the response, so they should not be
561 // persisted.
562 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
563 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
564 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700565}
566
Sen Jiang0affc2c2017-02-10 15:55:05 -0800567TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
568 OmahaResponse response;
569 fake_update_response_.multi_package = true;
570 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
571 fake_update_response_.GetUpdateResponse(),
572 -1,
573 false, // ping_only
574 ErrorCode::kSuccess,
575 metrics::CheckResult::kUpdateAvailable,
576 metrics::CheckReaction::kUpdating,
577 metrics::DownloadErrorCode::kUnset,
578 &response,
579 nullptr));
580 EXPECT_TRUE(response.update_exists);
581 EXPECT_EQ(fake_update_response_.version, response.version);
582 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
583 response.packages[0].payload_urls[0]);
584 EXPECT_EQ(fake_update_response_.codebase + "package2",
585 response.packages[1].payload_urls[0]);
586 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
587 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700588 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700589 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800590 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700591 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800592 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700593 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700594 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700595}
596
597TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
598 OmahaResponse response;
599 fake_update_response_.multi_app = true;
600 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
601 fake_update_response_.GetUpdateResponse(),
602 -1,
603 false, // ping_only
604 ErrorCode::kSuccess,
605 metrics::CheckResult::kUpdateAvailable,
606 metrics::CheckReaction::kUpdating,
607 metrics::DownloadErrorCode::kUnset,
608 &response,
609 nullptr));
610 EXPECT_TRUE(response.update_exists);
611 EXPECT_EQ(fake_update_response_.version, response.version);
612 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
613 response.packages[0].payload_urls[0]);
614 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
615 response.packages[1].payload_urls[0]);
616 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
617 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
618 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700619 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700620 ASSERT_EQ(2u, response.packages.size());
621 EXPECT_EQ(string("hash3"), response.packages[1].hash);
622 EXPECT_EQ(333u, response.packages[1].size);
623 EXPECT_EQ(33u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700624 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700625}
626
627TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
628 OmahaResponse response;
Sen Jiang00adf7b2017-06-26 15:57:29 -0700629 fake_update_response_.multi_app = true;
630 fake_update_response_.multi_app_self_update = true;
Sen Jiang81259682017-03-30 15:11:30 -0700631 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
632 fake_update_response_.GetUpdateResponse(),
633 -1,
634 false, // ping_only
635 ErrorCode::kSuccess,
636 metrics::CheckResult::kUpdateAvailable,
637 metrics::CheckReaction::kUpdating,
638 metrics::DownloadErrorCode::kUnset,
639 &response,
640 nullptr));
641 EXPECT_TRUE(response.update_exists);
642 EXPECT_EQ(fake_update_response_.version, response.version);
643 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
644 response.packages[0].payload_urls[0]);
645 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
646 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
647 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang00adf7b2017-06-26 15:57:29 -0700648 ASSERT_EQ(2u, response.packages.size());
649 EXPECT_EQ(string("hash3"), response.packages[1].hash);
650 EXPECT_EQ(333u, response.packages[1].size);
651 EXPECT_EQ(33u, response.packages[1].metadata_size);
652 EXPECT_EQ(true, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700653}
654
655TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
656 OmahaResponse response;
657 fake_update_response_.multi_app = true;
658 fake_update_response_.multi_package = true;
659 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
660 fake_update_response_.GetUpdateResponse(),
661 -1,
662 false, // ping_only
663 ErrorCode::kSuccess,
664 metrics::CheckResult::kUpdateAvailable,
665 metrics::CheckReaction::kUpdating,
666 metrics::DownloadErrorCode::kUnset,
667 &response,
668 nullptr));
669 EXPECT_TRUE(response.update_exists);
670 EXPECT_EQ(fake_update_response_.version, response.version);
671 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
672 response.packages[0].payload_urls[0]);
673 EXPECT_EQ(fake_update_response_.codebase + "package2",
674 response.packages[1].payload_urls[0]);
675 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
676 response.packages[2].payload_urls[0]);
677 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
678 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
679 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700680 EXPECT_EQ(true, response.packages[0].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700681 ASSERT_EQ(3u, response.packages.size());
682 EXPECT_EQ(string("hash2"), response.packages[1].hash);
683 EXPECT_EQ(222u, response.packages[1].size);
684 EXPECT_EQ(22u, response.packages[1].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700685 EXPECT_EQ(false, response.packages[1].is_delta);
Sen Jiang81259682017-03-30 15:11:30 -0700686 EXPECT_EQ(string("hash3"), response.packages[2].hash);
687 EXPECT_EQ(333u, response.packages[2].size);
688 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiangcdd52062017-05-18 15:33:10 -0700689 EXPECT_EQ(false, response.packages[2].is_delta);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800690}
691
Alex Deymo14ad88e2016-06-29 12:30:14 -0700692TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
693 const string http_response = "<?xml invalid response";
694 request_params_.set_interactive(true);
695
696 brillo::FakeMessageLoop loop(nullptr);
697 loop.SetAsCurrent();
698
699 MockHttpFetcher* fetcher =
700 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
701 OmahaRequestAction action(
702 &fake_system_state_, nullptr, brillo::make_unique_ptr(fetcher), false);
703 ActionProcessor processor;
704 processor.EnqueueAction(&action);
705
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700706 loop.PostTask(base::Bind(
707 [](ActionProcessor* processor) { processor->StartProcessing(); },
708 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700709 loop.Run();
710 EXPECT_FALSE(loop.PendingTasks());
711
712 // Check that the headers were set in the fetcher during the action. Note that
713 // we set this request as "interactive".
714 EXPECT_EQ("fg", fetcher->GetHeader("X-GoogleUpdate-Interactivity"));
715 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-GoogleUpdate-AppId"));
716 EXPECT_NE("", fetcher->GetHeader("X-GoogleUpdate-Updater"));
717}
718
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700719TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700720 OmahaResponse response;
721 // Set up a connection manager that doesn't allow a valid update over
722 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700723 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700724 fake_system_state_.set_connection_manager(&mock_cm);
725
Alex Deymo30534502015-07-20 15:06:33 -0700726 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
727 .WillRepeatedly(
Sen Jiang255e22b2016-05-20 16:15:29 -0700728 DoAll(SetArgumentPointee<0>(ConnectionType::kEthernet),
729 SetArgumentPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700730 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700731 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
732 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700733
734 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700735 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700736 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700737 -1,
738 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700739 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700740 metrics::CheckResult::kUpdateAvailable,
741 metrics::CheckReaction::kIgnored,
742 metrics::DownloadErrorCode::kUnset,
743 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700744 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700745 EXPECT_FALSE(response.update_exists);
746}
747
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700748TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700749 string rollback_version = "1234.0.0";
750 OmahaResponse response;
751
752 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700753 fake_system_state_.set_payload_state(&mock_payload_state);
754
Chris Sosa77f79e82014-06-02 18:16:24 -0700755 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
756 .WillRepeatedly(Return(rollback_version));
757
Alex Deymo8e18f932015-03-27 16:16:59 -0700758 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700759 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700760 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700761 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700762 -1,
763 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700764 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700765 metrics::CheckResult::kUpdateAvailable,
766 metrics::CheckReaction::kIgnored,
767 metrics::DownloadErrorCode::kUnset,
768 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700769 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700770 EXPECT_FALSE(response.update_exists);
771}
772
Kevin Cernekee2494e282016-03-29 18:03:53 -0700773// Verify that update checks called during OOBE will only try to download
774// an update if the response includes a non-empty deadline field.
775TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
776 OmahaResponse response;
777
778 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
779 ASSERT_FALSE(
780 TestUpdateCheck(nullptr, // request_params
781 fake_update_response_.GetUpdateResponse(),
782 -1,
783 false, // ping_only
784 ErrorCode::kNonCriticalUpdateInOOBE,
785 metrics::CheckResult::kUnset,
786 metrics::CheckReaction::kUnset,
787 metrics::DownloadErrorCode::kUnset,
788 &response,
789 nullptr));
790 EXPECT_FALSE(response.update_exists);
791
Alex Deymo46a9aae2016-05-04 20:20:11 -0700792 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
793 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
794 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
795 fake_update_response_.GetUpdateResponse(),
796 -1,
797 false, // ping_only
798 ErrorCode::kSuccess,
799 metrics::CheckResult::kUpdateAvailable,
800 metrics::CheckReaction::kUpdating,
801 metrics::DownloadErrorCode::kUnset,
802 &response,
803 nullptr));
804 EXPECT_TRUE(response.update_exists);
805 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
806
807 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700808 fake_update_response_.deadline = "20101020";
809 ASSERT_TRUE(
810 TestUpdateCheck(nullptr, // request_params
811 fake_update_response_.GetUpdateResponse(),
812 -1,
813 false, // ping_only
814 ErrorCode::kSuccess,
815 metrics::CheckResult::kUpdateAvailable,
816 metrics::CheckReaction::kUpdating,
817 metrics::DownloadErrorCode::kUnset,
818 &response,
819 nullptr));
820 EXPECT_TRUE(response.update_exists);
821}
822
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700823TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700824 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700825 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700826 params.set_wall_clock_based_wait_enabled(true);
827 params.set_update_check_count_wait_enabled(false);
828 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700829
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700830 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700831 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700832 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700833 -1,
834 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700835 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800836 metrics::CheckResult::kUpdateAvailable,
837 metrics::CheckReaction::kDeferring,
838 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700839 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700840 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700841 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700842
843 // Verify if we are interactive check we don't defer.
844 params.set_interactive(true);
845 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700846 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700847 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700848 -1,
849 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700850 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800851 metrics::CheckResult::kUpdateAvailable,
852 metrics::CheckReaction::kUpdating,
853 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700854 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700855 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700856 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700857}
858
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700859TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700860 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700861 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700862 params.set_wall_clock_based_wait_enabled(false);
863 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700864
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700865 params.set_update_check_count_wait_enabled(true);
866 params.set_min_update_checks_needed(1);
867 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700868
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700869 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700870 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700871 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700872 -1,
873 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700874 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800875 metrics::CheckResult::kUpdateAvailable,
876 metrics::CheckReaction::kUpdating,
877 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700878 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700879 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700880 EXPECT_TRUE(response.update_exists);
881}
882
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700883TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700884 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700885 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700886 params.set_wall_clock_based_wait_enabled(true);
887 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700888
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700889 params.set_update_check_count_wait_enabled(true);
890 params.set_min_update_checks_needed(1);
891 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700892
Alex Deymo8e18f932015-03-27 16:16:59 -0700893 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700894 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700895 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700896 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700897 -1,
898 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700899 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800900 metrics::CheckResult::kUpdateAvailable,
901 metrics::CheckReaction::kUpdating,
902 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700903 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700904 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700905 EXPECT_TRUE(response.update_exists);
906}
907
908
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700909TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700910 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700911 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700912 params.set_wall_clock_based_wait_enabled(true);
913 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700914
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700915 params.set_update_check_count_wait_enabled(true);
916 params.set_min_update_checks_needed(0);
917 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700918
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700919 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700920 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700921 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700922 -1,
923 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700924 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800925 metrics::CheckResult::kUpdateAvailable,
926 metrics::CheckReaction::kUpdating,
927 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700928 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700929 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700930
Ben Chan9abb7632014-08-07 00:10:53 -0700931 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700932 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700933 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700934 EXPECT_TRUE(response.update_exists);
935}
936
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700937TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700938 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700939 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700940 params.set_wall_clock_based_wait_enabled(true);
941 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700942
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700943 params.set_update_check_count_wait_enabled(true);
944 params.set_min_update_checks_needed(1);
945 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700946
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700947 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700948 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700949 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700950 -1,
951 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700952 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800953 metrics::CheckResult::kUpdateAvailable,
954 metrics::CheckReaction::kDeferring,
955 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700956 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700957 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700958
Ben Chan9abb7632014-08-07 00:10:53 -0700959 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700960 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700961 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700962 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700963
964 // Verify if we are interactive check we don't defer.
965 params.set_interactive(true);
966 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700967 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700968 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700969 -1,
970 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700971 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800972 metrics::CheckResult::kUpdateAvailable,
973 metrics::CheckReaction::kUpdating,
974 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700975 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700976 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700977 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700978}
979
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700980TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700981 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700982 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700983 params.set_wall_clock_based_wait_enabled(true);
984 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700985
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700986 params.set_update_check_count_wait_enabled(true);
987 params.set_min_update_checks_needed(1);
988 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700989
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700990 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700991
992 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700993 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700994 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700995 -1,
996 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700997 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800998 metrics::CheckResult::kUpdateAvailable,
999 metrics::CheckReaction::kDeferring,
1000 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001001 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001002 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001003
Ben Chan9abb7632014-08-07 00:10:53 -07001004 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001005 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001006 // count remains the same, as the decrementing happens in update_attempter
1007 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001008 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001009 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001010
1011 // Verify if we are interactive check we don't defer.
1012 params.set_interactive(true);
1013 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001014 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001015 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001016 -1,
1017 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001018 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001019 metrics::CheckResult::kUpdateAvailable,
1020 metrics::CheckReaction::kUpdating,
1021 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001022 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001023 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001024 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -07001025}
Jay Srinivasan0a708742012-03-20 11:26:12 -07001026
Alex Deymo8e18f932015-03-27 16:16:59 -07001027TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
1028 OmahaResponse response;
1029 OmahaRequestParams params = request_params_;
1030 fake_update_response_.include_cohorts = true;
1031 fake_update_response_.cohort = "s/154454/8479665";
1032 fake_update_response_.cohorthint = "please-put-me-on-beta";
1033 fake_update_response_.cohortname = "stable";
1034
1035 ASSERT_TRUE(TestUpdateCheck(&params,
1036 fake_update_response_.GetUpdateResponse(),
1037 -1,
1038 false, // ping_only
1039 ErrorCode::kSuccess,
1040 metrics::CheckResult::kUpdateAvailable,
1041 metrics::CheckReaction::kUpdating,
1042 metrics::DownloadErrorCode::kUnset,
1043 &response,
1044 nullptr));
1045
1046 string value;
1047 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1048 EXPECT_EQ(fake_update_response_.cohort, value);
1049
1050 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1051 EXPECT_EQ(fake_update_response_.cohorthint, value);
1052
1053 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1054 EXPECT_EQ(fake_update_response_.cohortname, value);
1055}
1056
1057TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1058 OmahaResponse response;
1059 OmahaRequestParams params = request_params_;
1060 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1061 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1062 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1063 fake_update_response_.include_cohorts = true;
1064 fake_update_response_.cohort = "s/154454/8479665";
1065 fake_update_response_.cohorthint = "please-put-me-on-beta";
1066 fake_update_response_.cohortname = "";
1067
1068 ASSERT_TRUE(TestUpdateCheck(&params,
1069 fake_update_response_.GetUpdateResponse(),
1070 -1,
1071 false, // ping_only
1072 ErrorCode::kSuccess,
1073 metrics::CheckResult::kUpdateAvailable,
1074 metrics::CheckReaction::kUpdating,
1075 metrics::DownloadErrorCode::kUnset,
1076 &response,
1077 nullptr));
1078
1079 string value;
1080 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1081 EXPECT_EQ(fake_update_response_.cohort, value);
1082
1083 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1084 EXPECT_EQ(fake_update_response_.cohorthint, value);
1085
1086 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1087}
1088
1089TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1090 OmahaResponse response;
1091 OmahaRequestParams params = request_params_;
1092 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1093
1094 ASSERT_TRUE(TestUpdateCheck(&params,
1095 fake_update_response_.GetUpdateResponse(),
1096 -1,
1097 false, // ping_only
1098 ErrorCode::kSuccess,
1099 metrics::CheckResult::kUpdateAvailable,
1100 metrics::CheckReaction::kUpdating,
1101 metrics::DownloadErrorCode::kUnset,
1102 &response,
1103 nullptr));
1104
1105 string value;
1106 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1107 EXPECT_EQ("old_value", value);
1108
1109 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1110 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1111}
1112
Alex Deymo00d79ac2015-06-29 15:41:49 -07001113TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1114 OmahaResponse response;
1115 OmahaRequestParams params = request_params_;
1116 fake_update_response_.include_cohorts = true;
1117 fake_update_response_.cohort = "s/154454/8479665";
1118 fake_update_response_.cohorthint = "please-put-me-on-beta";
1119 fake_update_response_.cohortname = "stable";
1120
1121 ASSERT_TRUE(TestUpdateCheck(&params,
1122 fake_update_response_.GetNoUpdateResponse(),
1123 -1,
1124 false, // ping_only
1125 ErrorCode::kSuccess,
1126 metrics::CheckResult::kNoUpdateAvailable,
1127 metrics::CheckReaction::kUnset,
1128 metrics::DownloadErrorCode::kUnset,
1129 &response,
1130 nullptr));
1131
1132 string value;
1133 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1134 EXPECT_EQ(fake_update_response_.cohort, value);
1135
1136 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1137 EXPECT_EQ(fake_update_response_.cohorthint, value);
1138
1139 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1140 EXPECT_EQ(fake_update_response_.cohortname, value);
1141}
1142
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001143TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001144 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001145
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001146 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001147 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001148
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001149 OmahaRequestParams params = request_params_;
1150 fake_system_state_.set_request_params(&params);
1151 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001152 brillo::make_unique_ptr(
1153 new MockHttpFetcher(http_response.data(),
1154 http_response.size(),
1155 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001156 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001157 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001158 ActionProcessor processor;
1159 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001160 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001161
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001162 loop.PostTask(base::Bind(
1163 [](ActionProcessor* processor) { processor->StartProcessing(); },
1164 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001165 loop.Run();
1166 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001167 EXPECT_FALSE(processor.IsRunning());
1168}
1169
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001170TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001171 OmahaResponse response;
1172 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001173 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001174 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001175 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001176 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001177 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001178 metrics::CheckResult::kParsingError,
1179 metrics::CheckReaction::kUnset,
1180 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001181 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001182 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001183 EXPECT_FALSE(response.update_exists);
1184}
1185
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001186TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001187 OmahaResponse response;
1188 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001189 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001190 "",
1191 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001192 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001193 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001194 metrics::CheckResult::kParsingError,
1195 metrics::CheckReaction::kUnset,
1196 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001197 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001198 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001199 EXPECT_FALSE(response.update_exists);
1200}
1201
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001202TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001203 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001204 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001205 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001206 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1207 "<daystart elapsed_seconds=\"100\"/>"
1208 "<app appid=\"foo\" status=\"ok\">"
1209 "<ping status=\"ok\"/>"
1210 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001211 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001212 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001213 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001214 metrics::CheckResult::kParsingError,
1215 metrics::CheckReaction::kUnset,
1216 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001217 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001218 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001219 EXPECT_FALSE(response.update_exists);
1220}
1221
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001222TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001223 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001224 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001225 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001226 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1227 "<daystart elapsed_seconds=\"100\"/>"
1228 "<app appid=\"foo\" status=\"ok\">"
1229 "<ping status=\"ok\"/>"
1230 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001231 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001232 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001233 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001234 metrics::CheckResult::kParsingError,
1235 metrics::CheckReaction::kUnset,
1236 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001237 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001238 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001239 EXPECT_FALSE(response.update_exists);
1240}
1241
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001242TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001243 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001244 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001245 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001246 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1247 "<daystart elapsed_seconds=\"100\"/>"
1248 "<app appid=\"foo\" status=\"ok\">"
1249 "<ping status=\"ok\"/>"
1250 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001251 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001252 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001253 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001254 metrics::CheckResult::kParsingError,
1255 metrics::CheckReaction::kUnset,
1256 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001257 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001258 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001259 EXPECT_FALSE(response.update_exists);
1260}
1261
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001262TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001263 string input_response =
1264 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1265 "<daystart elapsed_seconds=\"100\"/>"
1266 "<app appid=\"xyz\" status=\"ok\">"
1267 "<updatecheck status=\"ok\">"
1268 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001269 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001270 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001271 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001272 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001273 "ChromeOSVersion=\"10.2.3.4\" "
1274 "Prompt=\"false\" "
1275 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001276 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001277 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001278 "needsadmin=\"true\" "
1279 "/></actions></manifest></updatecheck></app></response>";
1280 LOG(INFO) << "Input Response = " << input_response;
1281
Darin Petkov6a5b3222010-07-13 14:55:28 -07001282 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001283 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001284 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001285 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001286 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001287 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001288 metrics::CheckResult::kUpdateAvailable,
1289 metrics::CheckReaction::kUpdating,
1290 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001291 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001292 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001293 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001294 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001295 EXPECT_EQ("http://missing/field/test/f",
1296 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001297 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001298 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1299 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001300 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001301 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001302}
1303
1304namespace {
1305class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1306 public:
1307 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001308 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001309 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001310};
1311
Alex Deymo60ca1a72015-06-18 18:19:15 -07001312void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001313 processor->StartProcessing();
1314 CHECK(processor->IsRunning());
1315 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001316}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001317} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001318
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001319TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001320 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001321 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001322
Alex Deymo60ca1a72015-06-18 18:19:15 -07001323 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001324 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001325 brillo::make_unique_ptr(
1326 new MockHttpFetcher(http_response.data(),
1327 http_response.size(),
1328 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001329 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001330 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001331 ActionProcessor processor;
1332 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001333 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001334
Alex Deymo60ca1a72015-06-18 18:19:15 -07001335 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1336 loop.Run();
1337 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001338}
1339
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001340TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001341 string output;
1342 EXPECT_TRUE(XmlEncode("ab", &output));
1343 EXPECT_EQ("ab", output);
1344 EXPECT_TRUE(XmlEncode("a<b", &output));
1345 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001346 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1347 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001348 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1349 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001350 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001351 EXPECT_FALSE(XmlEncode("\xc2", &output));
1352 // Fail with invalid ASCII-7 chars.
1353 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1354}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001355
Alex Deymob0d74eb2015-03-30 17:59:17 -07001356TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1357 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1358 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1359}
1360
1361TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001362 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001363
1364 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001365 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001366 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001367 OmahaRequestParams::kOsVersion,
1368 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001369 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001370 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001371 "0.1.0.0",
1372 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001373 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001374 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001375 "ChromeOSFirmware.1.0",
1376 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001377 false, // delta okay
1378 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001379 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001380 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001381 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1382 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1383 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001384 base::JoinString(
1385 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001386 OmahaResponse response;
1387 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001388 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001389 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001390 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001391 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001392 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001393 metrics::CheckResult::kParsingError,
1394 metrics::CheckReaction::kUnset,
1395 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001396 &response,
1397 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001398 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001399 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001400 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1401 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1402 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1403 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1404 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1405 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1406 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1407 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1408 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1409 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1410 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1411 // Values from Prefs that are too big are removed from the XML instead of
1412 // encoded.
1413 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001414}
1415
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001416TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001417 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001418 fake_update_response_.deadline = "&lt;20110101";
1419 fake_update_response_.more_info_url = "testthe&lt;url";
1420 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001421 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001422 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001423 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001424 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001425 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001426 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001427 metrics::CheckResult::kUpdateAvailable,
1428 metrics::CheckReaction::kUpdating,
1429 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001430 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001431 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001432
Sen Jiang0affc2c2017-02-10 15:55:05 -08001433 EXPECT_EQ("testthe<url", response.more_info_url);
1434 EXPECT_EQ("testthe&codebase/file.signed",
1435 response.packages[0].payload_urls[0]);
1436 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001437}
1438
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001439TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001440 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001441 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001442 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001443 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001444 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001445 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001446 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001447 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001448 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001449 metrics::CheckResult::kUpdateAvailable,
1450 metrics::CheckReaction::kUpdating,
1451 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001452 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001453 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001454
Sen Jiang0affc2c2017-02-10 15:55:05 -08001455 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001456}
1457
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001458TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001459 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001460 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001461 fake_system_state_.set_prefs(&prefs);
1462
Darin Petkov95508da2011-01-05 12:42:29 -08001463 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1464 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001465 // An existing but empty previous version means that we didn't reboot to a new
1466 // update, therefore, no need to update the previous version.
1467 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001468 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001469 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001470 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001471 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001472 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001473 metrics::CheckResult::kParsingError,
1474 metrics::CheckReaction::kUnset,
1475 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001476 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001477 &post_data));
1478 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001479 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001480 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001481 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001482 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001483 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001484 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1485 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001486 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1487 string::npos);
1488 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1489 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001490 // No <event> tag should be sent if we didn't reboot to an update.
1491 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001492}
1493
Jay Srinivasan0a708742012-03-20 11:26:12 -07001494
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001495TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001496 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001497 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001498 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1499 "invalid xml>",
1500 &post_data);
1501 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001502 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001503 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001504 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001505 OmahaEvent::kTypeUpdateDownloadStarted,
1506 OmahaEvent::kResultSuccess);
1507 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001508 EXPECT_EQ(post_str.find("ping"), string::npos);
1509 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001510}
1511
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001512TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001513 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001514 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001515 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1516 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001517 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001518 "invalid xml>",
1519 &post_data);
1520 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001521 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001522 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001523 " <event eventtype=\"%d\" eventresult=\"%d\" "
1524 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001525 OmahaEvent::kTypeDownloadComplete,
1526 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001527 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001528 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001529 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001530}
1531
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001532TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001533 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001534 // Create a copy of the OmahaRequestParams to reuse it later.
1535 OmahaRequestParams params = request_params_;
1536 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001537 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001538 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001539 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001540 brillo::make_unique_ptr(
1541 new MockHttpFetcher(http_response.data(),
1542 http_response.size(),
1543 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001544 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001545 EXPECT_FALSE(update_check_action.IsEvent());
1546
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001547 params = request_params_;
1548 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001549 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001550 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001551 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001552 brillo::make_unique_ptr(
1553 new MockHttpFetcher(http_response.data(),
1554 http_response.size(),
1555 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001556 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001557 EXPECT_TRUE(event_action.IsEvent());
1558}
1559
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001560TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001561 for (int i = 0; i < 2; i++) {
1562 bool delta_okay = i == 1;
1563 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001564 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001565 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001566 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001567 OmahaRequestParams::kOsVersion,
1568 "service_pack",
1569 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001570 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001571 "0.1.0.0",
1572 "en-US",
1573 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001574 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001575 "ChromeOSFirmware.1.0",
1576 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001577 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001578 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001579 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001580 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001581 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001582 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001583 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001584 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001585 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001586 metrics::CheckResult::kParsingError,
1587 metrics::CheckReaction::kUnset,
1588 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001589 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001590 &post_data));
1591 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001592 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001593 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1594 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001595 string::npos)
1596 << "i = " << i;
1597 }
1598}
1599
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001600TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001601 for (int i = 0; i < 2; i++) {
1602 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001603 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001604 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001605 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001606 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001607 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001608 OmahaRequestParams::kOsVersion,
1609 "service_pack",
1610 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001611 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001612 "0.1.0.0",
1613 "en-US",
1614 "unittest_track",
1615 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001616 "ChromeOSFirmware.1.0",
1617 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001618 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001619 interactive,
1620 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001621 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001622 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001623 "invalid xml>",
1624 -1,
1625 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001626 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001627 metrics::CheckResult::kParsingError,
1628 metrics::CheckReaction::kUnset,
1629 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001630 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001631 &post_data));
1632 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001633 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001634 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1635 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001636 string::npos)
1637 << "i = " << i;
1638 }
1639}
1640
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001641TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001642 OmahaEvent default_event;
1643 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1644 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001645 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001646
1647 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1648 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1649 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001650 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001651
1652 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1653 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001654 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001655 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1656 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001657 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001658}
1659
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001660void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001661 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001662 fake_system_state_.set_prefs(&prefs);
1663 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1664 .Times(AnyNumber());
1665 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1666 // Add a few hours to the day difference to test no rounding, etc.
1667 int64_t five_days_ago =
1668 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1669 int64_t six_days_ago =
1670 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1671 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1672 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1673 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1674 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1675 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1676 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001677 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001678 ASSERT_TRUE(
1679 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001680 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001681 -1,
1682 ping_only,
1683 ErrorCode::kSuccess,
1684 metrics::CheckResult::kUnset,
1685 metrics::CheckReaction::kUnset,
1686 metrics::DownloadErrorCode::kUnset,
1687 nullptr,
1688 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001689 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001690 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1691 string::npos);
1692 if (ping_only) {
1693 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1694 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1695 } else {
1696 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1697 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001698 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001699}
1700
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001701TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1702 PingTest(true /* ping_only */);
1703}
1704
1705TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1706 PingTest(false /* ping_only */);
1707}
1708
1709TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001710 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001711 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001712 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1713 .Times(AnyNumber());
1714 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001715 int64_t three_days_ago =
1716 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1717 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001718 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1719 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001720 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1721 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1722 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1723 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001724 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001725 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001726 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001727 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001728 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001729 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001730 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001731 metrics::CheckResult::kNoUpdateAvailable,
1732 metrics::CheckReaction::kUnset,
1733 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001734 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001735 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001736 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001737 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001738 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001739}
1740
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001741TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001742 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001743 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001744 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1745 .Times(AnyNumber());
1746 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001747 int64_t four_days_ago =
1748 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1749 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001750 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1751 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001752 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1753 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1754 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1755 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001756 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001757 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001758 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001759 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001760 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001761 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001762 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001763 metrics::CheckResult::kNoUpdateAvailable,
1764 metrics::CheckReaction::kUnset,
1765 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001766 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001767 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001768 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001769 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001770 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001771}
1772
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001773TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001774 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001775 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001776 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1777 .Times(AnyNumber());
1778 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001779 int64_t one_hour_ago =
1780 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001781 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1782 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001783 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1784 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1785 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1786 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001787 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1788 // send a ping.
1789 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1790 .WillOnce(Return(true));
1791 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1792 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001793 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001794 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001795 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001796 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001797 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001798 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001799 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001800 metrics::CheckResult::kNoUpdateAvailable,
1801 metrics::CheckReaction::kUnset,
1802 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001803 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001804 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001805 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001806 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001807}
1808
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001809TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001810 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001811 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001812 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001813 int64_t now = Time::Now().ToInternalValue();
1814 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1815 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1816 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1817 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1818 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1819 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001820 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001821 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001822 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001823 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001824 -1,
1825 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001826 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001827 metrics::CheckResult::kUnset,
1828 metrics::CheckReaction::kUnset,
1829 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001830 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001831 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001832 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001833}
1834
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001835TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001836 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001837 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001838 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1839 .Times(AnyNumber());
1840 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001841 int64_t future =
1842 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001843 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1844 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001845 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1846 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1847 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1848 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1849 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1850 .WillOnce(Return(true));
1851 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1852 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001853 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001854 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001855 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001856 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1857 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001858 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001859 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001860 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001861 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001862 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001863 metrics::CheckResult::kNoUpdateAvailable,
1864 metrics::CheckReaction::kUnset,
1865 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001866 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001867 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001868 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001869 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001870}
1871
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001872TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001873 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001874 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001875 // may fail if it runs for longer than 5 seconds. It shouldn't run
1876 // that long though.
1877 int64_t midnight =
1878 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1879 int64_t midnight_slack =
1880 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
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(_, _)).Times(AnyNumber());
1884 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001885 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1886 AllOf(Ge(midnight), Le(midnight_slack))))
1887 .WillOnce(Return(true));
1888 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1889 AllOf(Ge(midnight), Le(midnight_slack))))
1890 .WillOnce(Return(true));
1891 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001892 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001893 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1894 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001895 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001896 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001897 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001898 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001899 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001900 metrics::CheckResult::kNoUpdateAvailable,
1901 metrics::CheckReaction::kUnset,
1902 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001903 nullptr,
1904 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001905}
1906
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001907TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001908 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001909 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001910 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1911 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001912 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1913 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1914 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001915 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001916 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1917 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001918 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001919 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001920 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001921 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001922 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001923 metrics::CheckResult::kNoUpdateAvailable,
1924 metrics::CheckReaction::kUnset,
1925 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001926 nullptr,
1927 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001928}
1929
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001930TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001931 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001932 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001933 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1934 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001935 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1936 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1937 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001938 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001939 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1940 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001941 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001942 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001943 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001944 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001945 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001946 metrics::CheckResult::kNoUpdateAvailable,
1947 metrics::CheckReaction::kUnset,
1948 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001949 nullptr,
1950 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001951}
1952
Alex Deymob3fa53b2016-04-18 19:57:58 -07001953TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
1954 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
1955 // the "eol" attribute.
1956 ASSERT_TRUE(
1957 TestUpdateCheck(nullptr, // request_params
1958 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1959 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
1960 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
1961 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
1962 "</app></response>",
1963 -1,
1964 false, // ping_only
1965 ErrorCode::kSuccess,
1966 metrics::CheckResult::kNoUpdateAvailable,
1967 metrics::CheckReaction::kUnset,
1968 metrics::DownloadErrorCode::kUnset,
1969 nullptr,
1970 nullptr));
1971 string eol_pref;
1972 EXPECT_TRUE(
1973 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
1974 // Note that the eol="eol" attribute should be ignored and the _eol should be
1975 // used instead.
1976 EXPECT_EQ("security-only", eol_pref);
1977}
1978
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001979TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001980 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001981 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07001982 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001983 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001984 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001985 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001986 metrics::CheckResult::kParsingError,
1987 metrics::CheckReaction::kUnset,
1988 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001989 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07001990 &post_data));
1991 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001992 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07001993 EXPECT_EQ(post_str.find("machineid="), string::npos);
1994 EXPECT_EQ(post_str.find("userid="), string::npos);
1995}
1996
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001997TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001998 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001999 const int http_error_code =
2000 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07002001 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002002 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002003 "",
2004 501,
Darin Petkov265f2902011-05-09 15:17:40 -07002005 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002006 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002007 metrics::CheckResult::kDownloadError,
2008 metrics::CheckReaction::kUnset,
2009 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07002010 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002011 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002012 EXPECT_FALSE(response.update_exists);
2013}
2014
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002015TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07002016 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002017 const int http_error_code =
2018 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07002019 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002020 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07002021 "",
2022 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07002023 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002024 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08002025 metrics::CheckResult::kDownloadError,
2026 metrics::CheckReaction::kUnset,
2027 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07002028 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002029 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07002030 EXPECT_FALSE(response.update_exists);
2031}
2032
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002033TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002034 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002035 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002036 params.set_wall_clock_based_wait_enabled(true);
2037 params.set_waiting_period(TimeDelta().FromDays(1));
2038 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002039
Sen Jiang7c1171e2016-06-23 11:35:40 -07002040 Time arbitrary_date;
2041 Time::FromString("6/4/1989", &arbitrary_date);
2042 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
2043 ASSERT_FALSE(TestUpdateCheck(&params,
2044 fake_update_response_.GetUpdateResponse(),
2045 -1,
2046 false, // ping_only
2047 ErrorCode::kOmahaUpdateDeferredPerPolicy,
2048 metrics::CheckResult::kUpdateAvailable,
2049 metrics::CheckReaction::kDeferring,
2050 metrics::DownloadErrorCode::kUnset,
2051 &response,
2052 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002053
Ben Chan9abb7632014-08-07 00:10:53 -07002054 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002055 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002056 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002057 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002058
2059 // Verify if we are interactive check we don't defer.
2060 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002061 ASSERT_TRUE(TestUpdateCheck(&params,
2062 fake_update_response_.GetUpdateResponse(),
2063 -1,
2064 false, // ping_only
2065 ErrorCode::kSuccess,
2066 metrics::CheckResult::kUpdateAvailable,
2067 metrics::CheckReaction::kUpdating,
2068 metrics::DownloadErrorCode::kUnset,
2069 &response,
2070 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002071 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002072}
2073
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002074TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002075 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002076 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002077 params.set_wall_clock_based_wait_enabled(true);
2078 params.set_waiting_period(TimeDelta().FromDays(1));
2079 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002080
Sen Jiang7c1171e2016-06-23 11:35:40 -07002081 Time t1, t2;
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002082 Time::FromString("1/1/2012", &t1);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002083 Time::FromString("1/3/2012", &t2);
2084 ASSERT_TRUE(
2085 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2086 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2087 ASSERT_TRUE(TestUpdateCheck(&params,
2088 fake_update_response_.GetUpdateResponse(),
2089 -1,
2090 false, // ping_only
2091 ErrorCode::kSuccess,
2092 metrics::CheckResult::kUpdateAvailable,
2093 metrics::CheckReaction::kUpdating,
2094 metrics::DownloadErrorCode::kUnset,
2095 &response,
2096 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002097
2098 EXPECT_TRUE(response.update_exists);
2099
2100 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002101 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002102 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002103 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2104}
2105
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002106TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002107 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002108 base::ScopedTempDir tempdir;
2109 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002110
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002111 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002112 OmahaRequestParams params(&fake_system_state_);
2113 params.set_root(tempdir.path().value());
2114 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2115 params.set_app_version("1.2.3.4");
2116 params.set_current_channel("canary-channel");
2117 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2118 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002119 EXPECT_TRUE(params.to_more_stable_channel());
2120 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002121 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002122 "invalid xml>",
2123 -1,
2124 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002125 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002126 metrics::CheckResult::kParsingError,
2127 metrics::CheckReaction::kUnset,
2128 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002129 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002130 &post_data));
2131 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002132 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002133 EXPECT_NE(string::npos, post_str.find(
2134 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2135 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2136 "track=\"stable-channel\" from_track=\"canary-channel\" "));
2137}
2138
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002139TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002140 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002141 base::ScopedTempDir tempdir;
2142 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002143
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002144 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002145 OmahaRequestParams params(&fake_system_state_);
2146 params.set_root(tempdir.path().value());
2147 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2148 params.set_app_version("5.6.7.8");
2149 params.set_current_channel("stable-channel");
2150 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2151 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002152 EXPECT_FALSE(params.to_more_stable_channel());
2153 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002154 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002155 "invalid xml>",
2156 -1,
2157 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002158 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002159 metrics::CheckResult::kParsingError,
2160 metrics::CheckReaction::kUnset,
2161 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002162 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002163 &post_data));
2164 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002165 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002166 EXPECT_NE(string::npos, post_str.find(
2167 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2168 "version=\"5.6.7.8\" "
2169 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002170 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002171}
2172
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002173// Checks that the initial ping with a=-1 r=-1 is not send when the device
2174// was powerwashed.
2175TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2176 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2177
2178 // Flag that the device was powerwashed in the past.
2179 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2180
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002181 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002182 ASSERT_TRUE(
2183 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002184 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002185 -1,
2186 false, // ping_only
2187 ErrorCode::kSuccess,
2188 metrics::CheckResult::kNoUpdateAvailable,
2189 metrics::CheckReaction::kUnset,
2190 metrics::DownloadErrorCode::kUnset,
2191 nullptr,
2192 &post_data));
2193 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002194 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002195 EXPECT_EQ(string::npos, post_str.find("<ping"));
2196}
2197
Alex Deymo9fded1e2015-11-05 12:31:19 -08002198// Checks that the event 54 is sent on a reboot to a new update.
2199TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2200 // Flag that the device was updated in a previous boot.
2201 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2202
2203 brillo::Blob post_data;
2204 ASSERT_TRUE(
2205 TestUpdateCheck(nullptr, // request_params
2206 fake_update_response_.GetNoUpdateResponse(),
2207 -1,
2208 false, // ping_only
2209 ErrorCode::kSuccess,
2210 metrics::CheckResult::kNoUpdateAvailable,
2211 metrics::CheckReaction::kUnset,
2212 metrics::DownloadErrorCode::kUnset,
2213 nullptr,
2214 &post_data));
2215 string post_str(post_data.begin(), post_data.end());
2216
2217 // An event 54 is included and has the right version.
2218 EXPECT_NE(string::npos,
2219 post_str.find(base::StringPrintf(
2220 "<event eventtype=\"%d\"",
2221 OmahaEvent::kTypeRebootedAfterUpdate)));
2222 EXPECT_NE(string::npos,
2223 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2224
2225 // The previous version flag should have been removed.
2226 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2227 string prev_version;
2228 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2229 EXPECT_TRUE(prev_version.empty());
2230}
2231
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002232void OmahaRequestActionTest::P2PTest(
2233 bool initial_allow_p2p_for_downloading,
2234 bool initial_allow_p2p_for_sharing,
2235 bool omaha_disable_p2p_for_downloading,
2236 bool omaha_disable_p2p_for_sharing,
2237 bool payload_state_allow_p2p_attempt,
2238 bool expect_p2p_client_lookup,
2239 const string& p2p_client_result_url,
2240 bool expected_allow_p2p_for_downloading,
2241 bool expected_allow_p2p_for_sharing,
2242 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002243 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002244 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002245 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2246 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2247 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002248
2249 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002250 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002251 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2252 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002253 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2254 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2255 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2256 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2257 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2258 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2259 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2260 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2261 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2262 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2263
David Zeuthen8f191b22013-08-06 12:27:50 -07002264 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002265 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002266 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2267
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002268 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2269 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002270 .Times(expect_p2p_client_lookup ? 1 : 0);
2271
Alex Deymo8e18f932015-03-27 16:16:59 -07002272 fake_update_response_.disable_p2p_for_downloading =
2273 omaha_disable_p2p_for_downloading;
2274 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002275 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002276 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002277 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002278 -1,
2279 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002280 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002281 metrics::CheckResult::kUpdateAvailable,
2282 metrics::CheckReaction::kUpdating,
2283 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002284 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002285 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002286 EXPECT_TRUE(response.update_exists);
2287
Gilad Arnold74b5f552014-10-07 08:17:16 -07002288 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2289 response.disable_p2p_for_downloading);
2290 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2291 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002292
Gilad Arnold74b5f552014-10-07 08:17:16 -07002293 EXPECT_EQ(expected_allow_p2p_for_downloading,
2294 actual_allow_p2p_for_downloading);
2295 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2296 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002297}
2298
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002299TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002300 P2PTest(true, // initial_allow_p2p_for_downloading
2301 true, // initial_allow_p2p_for_sharing
2302 false, // omaha_disable_p2p_for_downloading
2303 false, // omaha_disable_p2p_for_sharing
2304 true, // payload_state_allow_p2p_attempt
2305 true, // expect_p2p_client_lookup
2306 "http://1.3.5.7/p2p", // p2p_client_result_url
2307 true, // expected_allow_p2p_for_downloading
2308 true, // expected_allow_p2p_for_sharing
2309 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002310}
2311
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002312TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002313 P2PTest(true, // initial_allow_p2p_for_downloading
2314 true, // initial_allow_p2p_for_sharing
2315 false, // omaha_disable_p2p_for_downloading
2316 false, // omaha_disable_p2p_for_sharing
2317 true, // payload_state_allow_p2p_attempt
2318 true, // expect_p2p_client_lookup
2319 "", // p2p_client_result_url
2320 false, // expected_allow_p2p_for_downloading
2321 true, // expected_allow_p2p_for_sharing
2322 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002323}
2324
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002325TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002326 P2PTest(false, // initial_allow_p2p_for_downloading
2327 true, // initial_allow_p2p_for_sharing
2328 false, // omaha_disable_p2p_for_downloading
2329 false, // omaha_disable_p2p_for_sharing
2330 true, // payload_state_allow_p2p_attempt
2331 false, // expect_p2p_client_lookup
2332 "unset", // p2p_client_result_url
2333 false, // expected_allow_p2p_for_downloading
2334 true, // expected_allow_p2p_for_sharing
2335 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002336}
2337
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002338TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002339 P2PTest(true, // initial_allow_p2p_for_downloading
2340 true, // initial_allow_p2p_for_sharing
2341 true, // omaha_disable_p2p_for_downloading
2342 false, // omaha_disable_p2p_for_sharing
2343 true, // payload_state_allow_p2p_attempt
2344 false, // expect_p2p_client_lookup
2345 "unset", // p2p_client_result_url
2346 false, // expected_allow_p2p_for_downloading
2347 true, // expected_allow_p2p_for_sharing
2348 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002349}
2350
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002351TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002352 P2PTest(true, // initial_allow_p2p_for_downloading
2353 true, // initial_allow_p2p_for_sharing
2354 false, // omaha_disable_p2p_for_downloading
2355 true, // omaha_disable_p2p_for_sharing
2356 true, // payload_state_allow_p2p_attempt
2357 true, // expect_p2p_client_lookup
2358 "http://1.3.5.7/p2p", // p2p_client_result_url
2359 true, // expected_allow_p2p_for_downloading
2360 false, // expected_allow_p2p_for_sharing
2361 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002362}
2363
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002364TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002365 P2PTest(true, // initial_allow_p2p_for_downloading
2366 true, // initial_allow_p2p_for_sharing
2367 true, // omaha_disable_p2p_for_downloading
2368 true, // omaha_disable_p2p_for_sharing
2369 true, // payload_state_allow_p2p_attempt
2370 false, // expect_p2p_client_lookup
2371 "unset", // p2p_client_result_url
2372 false, // expected_allow_p2p_for_downloading
2373 false, // expected_allow_p2p_for_sharing
2374 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002375}
2376
Alex Deymof329b932014-10-30 01:37:48 -07002377bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2378 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002379 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002380 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002381 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002382 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002383 -1,
2384 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002385 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002386 metrics::CheckResult::kUpdateAvailable,
2387 metrics::CheckReaction::kUpdating,
2388 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002389 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002390 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002391}
2392
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002393TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002394 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002395
Kevin Cernekee2494e282016-03-29 18:03:53 -07002396 // Simulate a successful update check that happens during OOBE. The
2397 // deadline in the response is needed to force the update attempt to
2398 // occur; responses without a deadline seen during OOBE will normally
2399 // return ErrorCode::kNonCriticalUpdateInOOBE.
2400 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2401 fake_update_response_.deadline = "20101020";
2402
David Zeuthen639aa362014-02-03 16:23:44 -08002403 // Check that we parse elapsed_days in the Omaha Response correctly.
2404 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002405 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2406 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002407 EXPECT_TRUE(response.update_exists);
2408 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002409 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002410 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002411 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002412 EXPECT_EQ(prefs_days, 42);
2413
2414 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002415 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002416 EXPECT_TRUE(response.update_exists);
2417 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002418 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002419 EXPECT_EQ(prefs_days, 42);
2420
2421 // Note that elapsed_days is not necessarily divisible by 7 so check
2422 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002423 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2424 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002425 EXPECT_TRUE(response.update_exists);
2426 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002427 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002428 EXPECT_EQ(prefs_days, 21);
2429
2430 // Check that we correctly handle elapsed_days not being included in
2431 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002432 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002433 EXPECT_TRUE(response.update_exists);
2434 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002435}
2436
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002437// If there is no prefs and OOBE is not complete, we should not
2438// report anything to Omaha.
2439TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002440 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002441 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2442 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2443}
David Zeuthen639aa362014-02-03 16:23:44 -08002444
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002445// If OOBE is complete and happened on a valid date (e.g. after Jan
2446// 1 2007 0:00 PST), that date should be used and written to
2447// prefs. However, first try with an invalid date and check we do
2448// nothing.
2449TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2450 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2451 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2452 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2453 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2454}
David Zeuthen639aa362014-02-03 16:23:44 -08002455
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002456// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2457// should yield an InstallDate of 14.
2458TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2459 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2460 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2461 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2462 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002463
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002464 int64_t prefs_days;
2465 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2466 EXPECT_EQ(prefs_days, 14);
2467}
David Zeuthen639aa362014-02-03 16:23:44 -08002468
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002469// Now that we have a valid date in prefs, check that we keep using
2470// that even if OOBE date reports something else. The date Jan 30,
2471// 2007 0:00 PST should yield an InstallDate of 28... but since
2472// there's a prefs file, we should still get 14.
2473TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2474 // Set a valid date in the prefs first.
2475 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002476
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002477 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2478 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2479 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002480
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002481 int64_t prefs_days;
2482 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2483 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002484
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002485 // If we delete the prefs file, we should get 28 days.
2486 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2487 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2488 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2489 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002490}
2491
Darin Petkov6a5b3222010-07-13 14:55:28 -07002492} // namespace chromeos_update_engine