blob: 084d3aabfaccc901f011e4d26f2f0d9008084395 [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>
Alex Deymo8e18f932015-03-27 16:16:59 -070026#include <base/strings/string_number_conversions.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070027#include <base/strings/string_util.h>
28#include <base/strings/stringprintf.h>
29#include <base/time/time.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070030#include <brillo/bind_lambda.h>
Alex Deymoc1c17b42015-11-23 03:53:15 -030031#include <brillo/make_unique_ptr.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070032#include <brillo/message_loops/fake_message_loop.h>
33#include <brillo/message_loops/message_loop.h>
34#include <brillo/message_loops/message_loop_utils.h>
Alex Deymoe1e3afe2014-10-30 13:02:49 -070035#include <gtest/gtest.h>
Jay Srinivasand29695d2013-04-08 15:08:05 -070036
Alex Deymo39910dc2015-11-09 17:04:30 -080037#include "update_engine/common/action_pipe.h"
38#include "update_engine/common/constants.h"
39#include "update_engine/common/fake_prefs.h"
40#include "update_engine/common/hash_calculator.h"
41#include "update_engine/common/mock_http_fetcher.h"
42#include "update_engine/common/platform_constants.h"
43#include "update_engine/common/prefs.h"
44#include "update_engine/common/test_utils.h"
45#include "update_engine/common/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;
Alex Deymo10875d92014-11-10 21:52:57 -080054using chromeos_update_engine::test_utils::System;
55using chromeos_update_engine::test_utils::WriteFileString;
Darin Petkov6a5b3222010-07-13 14:55:28 -070056using std::string;
57using std::vector;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070058using testing::AllOf;
Alex Deymof329b932014-10-30 01:37:48 -070059using testing::AnyNumber;
Jay Srinivasan34b5d862012-07-23 11:43:22 -070060using testing::DoAll;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070061using testing::Ge;
62using testing::Le;
Darin Petkov9c096d62010-11-17 14:49:04 -080063using testing::NiceMock;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070064using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070065using testing::ReturnPointee;
66using testing::SaveArg;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070067using testing::SetArgumentPointee;
Alex Deymof329b932014-10-30 01:37:48 -070068using testing::_;
Darin Petkov6a5b3222010-07-13 14:55:28 -070069
Alex Deymo8e18f932015-03-27 16:16:59 -070070namespace {
71
Alex Deymo85616652015-10-15 18:48:31 -070072const char kTestAppId[] = "test-app-id";
73
Alex Deymo8e18f932015-03-27 16:16:59 -070074// This is a helper struct to allow unit tests build an update response with the
75// values they care about.
76struct FakeUpdateResponse {
77 string GetNoUpdateResponse() const {
78 string entity_str;
79 if (include_entity)
80 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Alex Deymo00d79ac2015-06-29 15:41:49 -070081 return
82 "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" +
83 entity_str + "<response protocol=\"3.0\">"
Alex Deymo8e18f932015-03-27 16:16:59 -070084 "<daystart elapsed_seconds=\"100\"/>"
Alex Deymo00d79ac2015-06-29 15:41:49 -070085 "<app appid=\"" + app_id + "\" " +
86 (include_cohorts ? "cohort=\"" + cohort + "\" cohorthint=\"" +
87 cohorthint + "\" cohortname=\"" + cohortname + "\" " : "") +
88 " status=\"ok\">"
89 "<ping status=\"ok\"/>"
90 "<updatecheck status=\"noupdate\"/></app></response>";
Alex Deymo8e18f932015-03-27 16:16:59 -070091 }
92
93 string GetUpdateResponse() const {
94 return
95 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
96 "protocol=\"3.0\">"
97 "<daystart elapsed_seconds=\"100\"" +
98 (elapsed_days.empty() ? "" : (" elapsed_days=\"" + elapsed_days + "\""))
99 + "/>"
100 "<app appid=\"" + app_id + "\" " +
101 (include_cohorts ? "cohort=\"" + cohort + "\" cohorthint=\"" +
102 cohorthint + "\" cohortname=\"" + cohortname + "\" " : "") +
103 " status=\"ok\">"
104 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
105 "<urls><url codebase=\"" + codebase + "\"/></urls>"
106 "<manifest version=\"" + version + "\">"
107 "<packages><package hash=\"not-used\" name=\"" + filename + "\" "
108 "size=\"" + base::Int64ToString(size) + "\"/></packages>"
109 "<actions><action event=\"postinstall\" "
110 "ChromeOSVersion=\"" + version + "\" "
111 "MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt + "\" "
112 "IsDelta=\"true\" "
113 "IsDeltaPayload=\"true\" "
114 "MaxDaysToScatter=\"" + max_days_to_scatter + "\" "
115 "sha256=\"" + hash + "\" "
116 "needsadmin=\"" + needsadmin + "\" " +
117 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
118 (disable_p2p_for_downloading ?
119 "DisableP2PForDownloading=\"true\" " : "") +
120 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
121 "/></actions></manifest></updatecheck></app></response>";
122 }
123
124 // Return the payload URL, which is split in two fields in the XML response.
125 string GetPayloadUrl() {
126 return codebase + filename;
127 }
128
Alex Deymo85616652015-10-15 18:48:31 -0700129 string app_id = kTestAppId;
Alex Deymo8e18f932015-03-27 16:16:59 -0700130 string version = "1.2.3.4";
131 string more_info_url = "http://more/info";
132 string prompt = "true";
133 string codebase = "http://code/base/";
134 string filename = "file.signed";
135 string hash = "HASH1234=";
136 string needsadmin = "false";
137 int64_t size = 123;
138 string deadline = "";
139 string max_days_to_scatter = "7";
140 string elapsed_days = "42";
141
142 // P2P setting defaults to allowed.
143 bool disable_p2p_for_downloading = false;
144 bool disable_p2p_for_sharing = false;
145
146 // Omaha cohorts settings.
147 bool include_cohorts = false;
148 string cohort = "";
149 string cohorthint = "";
150 string cohortname = "";
151
152 // Whether to include the CrOS <!ENTITY> in the XML response.
153 bool include_entity = false;
154};
155
156} // namespace
157
Darin Petkov6a5b3222010-07-13 14:55:28 -0700158namespace chromeos_update_engine {
159
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700160class OmahaRequestActionTest : public ::testing::Test {
161 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800162 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700163 fake_system_state_.set_request_params(&request_params_);
164 fake_system_state_.set_prefs(&fake_prefs_);
165 }
166
167 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800168 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700169 // is used. |payload_state| may be null, in which case a local mock is used.
170 // |p2p_manager| may be null, in which case a local mock is used.
171 // |connection_manager| may be null, in which case a local mock is used.
172 // out_response may be null. If |fail_http_response_code| is non-negative,
173 // the transfer will fail with that code. |ping_only| is passed through to the
174 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
175 // post-data received by the mock HttpFetcher is returned.
176 //
177 // The |expected_check_result|, |expected_check_reaction| and
178 // |expected_error_code| parameters are for checking expectations
179 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
180 // UMA statistics. Use the appropriate ::kUnset value to specify that
181 // the given metric should not be reported.
182 bool TestUpdateCheck(OmahaRequestParams* request_params,
183 const string& http_response,
184 int fail_http_response_code,
185 bool ping_only,
186 ErrorCode expected_code,
187 metrics::CheckResult expected_check_result,
188 metrics::CheckReaction expected_check_reaction,
189 metrics::DownloadErrorCode expected_download_error_code,
190 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700191 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700192
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800193 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700194 // only a ping or also an updatecheck.
195 void PingTest(bool ping_only);
196
197 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700198 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700199 OmahaResponse *response);
200
201 // P2P test helper function.
202 void P2PTest(
203 bool initial_allow_p2p_for_downloading,
204 bool initial_allow_p2p_for_sharing,
205 bool omaha_disable_p2p_for_downloading,
206 bool omaha_disable_p2p_for_sharing,
207 bool payload_state_allow_p2p_attempt,
208 bool expect_p2p_client_lookup,
209 const string& p2p_client_result_url,
210 bool expected_allow_p2p_for_downloading,
211 bool expected_allow_p2p_for_sharing,
212 const string& expected_p2p_url);
213
214 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700215 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700216
217 // By default, all tests use these objects unless they replace them in the
218 // fake_system_state_.
219 OmahaRequestParams request_params_ = OmahaRequestParams{
220 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700221 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700222 OmahaRequestParams::kOsVersion,
223 "service_pack",
224 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700225 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700226 "0.1.0.0",
227 "en-US",
228 "unittest",
229 "OEM MODEL 09235 7471",
230 "ChromeOSFirmware.1.0",
231 "0X0A1",
232 false, // delta okay
233 false, // interactive
234 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700235 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700236
237 FakePrefs fake_prefs_;
238};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700239
240namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700241class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
242 public:
243 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700244 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800245 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700246 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800247 void ProcessingDone(const ActionProcessor* processor,
248 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700249 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700250 }
251
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800252 void ActionCompleted(ActionProcessor* processor,
253 AbstractAction* action,
254 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700255 // make sure actions always succeed
256 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700257 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700258 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700259 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700260 }
David Zeuthena99981f2013-04-29 13:42:47 -0700261 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700262};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700263} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700264
265class OutputObjectCollectorAction;
266
267template<>
268class ActionTraits<OutputObjectCollectorAction> {
269 public:
270 // Does not take an object for input
271 typedef OmahaResponse InputObjectType;
272 // On success, puts the output path on output
273 typedef NoneType OutputObjectType;
274};
275
276class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
277 public:
278 OutputObjectCollectorAction() : has_input_object_(false) {}
279 void PerformAction() {
280 // copy input object
281 has_input_object_ = HasInputObject();
282 if (has_input_object_)
283 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700284 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700285 }
286 // Should never be called
287 void TerminateProcessing() {
288 CHECK(false);
289 }
290 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700291 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700292 return "OutputObjectCollectorAction";
293 }
Alex Deymof329b932014-10-30 01:37:48 -0700294 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800295 using InputObjectType =
296 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
297 using OutputObjectType =
298 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700299 bool has_input_object_;
300 OmahaResponse omaha_response_;
301};
302
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700303bool OmahaRequestActionTest::TestUpdateCheck(
304 OmahaRequestParams* request_params,
305 const string& http_response,
306 int fail_http_response_code,
307 bool ping_only,
308 ErrorCode expected_code,
309 metrics::CheckResult expected_check_result,
310 metrics::CheckReaction expected_check_reaction,
311 metrics::DownloadErrorCode expected_download_error_code,
312 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700313 brillo::Blob* out_post_data) {
314 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700315 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700316 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800317 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700318 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700319 if (fail_http_response_code >= 0) {
320 fetcher->FailTransfer(fail_http_response_code);
321 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700322 if (request_params)
323 fake_system_state_.set_request_params(request_params);
324 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700325 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300326 brillo::make_unique_ptr(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700327 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700328 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700329 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700330
Darin Petkov6a5b3222010-07-13 14:55:28 -0700331 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700332 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700333 processor.EnqueueAction(&action);
334
335 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700336 BondActions(&action, &collector_action);
337 processor.EnqueueAction(&collector_action);
338
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700339 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800340 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700341 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800342 SendEnumToUMA(metrics::kMetricCheckResult,
343 static_cast<int>(expected_check_result),
344 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
345 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700346 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800347 SendEnumToUMA(metrics::kMetricCheckReaction,
348 static_cast<int>(expected_check_reaction),
349 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
350 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700351 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700352 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
353 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800354 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
355 ? 0 : 1);
356
Alex Deymo60ca1a72015-06-18 18:19:15 -0700357 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
358 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
359 loop.Run();
360 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
361 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700362 if (collector_action.has_input_object_ && out_response)
363 *out_response = collector_action.omaha_response_;
364 if (out_post_data)
365 *out_post_data = fetcher->post_data();
366 return collector_action.has_input_object_;
367}
368
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700369// Tests Event requests -- they should always succeed. |out_post_data|
370// may be null; if non-null, the post-data received by the mock
371// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700372void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700373 OmahaEvent* event,
374 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700375 brillo::Blob* out_post_data) {
376 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700377 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700378 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800379 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700380 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700381 FakeSystemState fake_system_state;
382 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300383 OmahaRequestAction action(&fake_system_state,
384 event,
385 brillo::make_unique_ptr(fetcher),
386 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700387 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700388 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700389 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700390 processor.EnqueueAction(&action);
391
Alex Deymo60ca1a72015-06-18 18:19:15 -0700392 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
393 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700394 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700395
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700396 if (out_post_data)
397 *out_post_data = fetcher->post_data();
398}
399
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700400TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400401 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700402 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400403 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700404 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700405 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400406 -1,
407 false, // ping_only
408 ErrorCode::kOmahaRequestXMLHasEntityDecl,
409 metrics::CheckResult::kParsingError,
410 metrics::CheckReaction::kUnset,
411 metrics::DownloadErrorCode::kUnset,
412 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700413 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400414 EXPECT_FALSE(response.update_exists);
415}
416
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700417TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700418 OmahaResponse response;
419 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700420 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700421 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700422 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700423 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700424 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800425 metrics::CheckResult::kNoUpdateAvailable,
426 metrics::CheckReaction::kUnset,
427 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700428 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700429 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700430 EXPECT_FALSE(response.update_exists);
431}
432
Alex Deymo8e18f932015-03-27 16:16:59 -0700433// Test that all the values in the response are parsed in a normal update
434// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700435TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700436 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700437 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700438 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700439 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700440 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700441 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700442 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700443 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800444 metrics::CheckResult::kUpdateAvailable,
445 metrics::CheckReaction::kUpdating,
446 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700447 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700448 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700449 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -0700450 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700451 EXPECT_EQ(fake_update_response_.version, response.version);
452 EXPECT_EQ(fake_update_response_.GetPayloadUrl(), response.payload_urls[0]);
453 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
454 EXPECT_EQ(fake_update_response_.hash, response.hash);
455 EXPECT_EQ(fake_update_response_.size, response.size);
456 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
457 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
458 // Omaha cohort attribets are not set in the response, so they should not be
459 // persisted.
460 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
461 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
462 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700463}
464
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700465TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700466 OmahaResponse response;
467 // Set up a connection manager that doesn't allow a valid update over
468 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700469 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700470 fake_system_state_.set_connection_manager(&mock_cm);
471
Alex Deymo30534502015-07-20 15:06:33 -0700472 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
473 .WillRepeatedly(
474 DoAll(SetArgumentPointee<0>(NetworkConnectionType::kEthernet),
475 SetArgumentPointee<1>(NetworkTethering::kUnknown),
476 Return(true)));
Alex Deymo75eac7e2015-07-29 13:39:14 -0700477 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(NetworkConnectionType::kEthernet, _))
Chris Sosa77f79e82014-06-02 18:16:24 -0700478 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700479
480 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700481 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700482 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700483 -1,
484 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700485 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700486 metrics::CheckResult::kUpdateAvailable,
487 metrics::CheckReaction::kIgnored,
488 metrics::DownloadErrorCode::kUnset,
489 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700490 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700491 EXPECT_FALSE(response.update_exists);
492}
493
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700494TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700495 string rollback_version = "1234.0.0";
496 OmahaResponse response;
497
498 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700499 fake_system_state_.set_payload_state(&mock_payload_state);
500
Chris Sosa77f79e82014-06-02 18:16:24 -0700501 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
502 .WillRepeatedly(Return(rollback_version));
503
Alex Deymo8e18f932015-03-27 16:16:59 -0700504 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700505 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700506 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700507 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700508 -1,
509 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700510 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700511 metrics::CheckResult::kUpdateAvailable,
512 metrics::CheckReaction::kIgnored,
513 metrics::DownloadErrorCode::kUnset,
514 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700515 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700516 EXPECT_FALSE(response.update_exists);
517}
518
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700519TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700520 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700521 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700522 params.set_wall_clock_based_wait_enabled(true);
523 params.set_update_check_count_wait_enabled(false);
524 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700525
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700526 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700527 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700528 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700529 -1,
530 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700531 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800532 metrics::CheckResult::kUpdateAvailable,
533 metrics::CheckReaction::kDeferring,
534 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700535 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700536 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700537 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700538
539 // Verify if we are interactive check we don't defer.
540 params.set_interactive(true);
541 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700542 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700543 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700544 -1,
545 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700546 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800547 metrics::CheckResult::kUpdateAvailable,
548 metrics::CheckReaction::kUpdating,
549 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700550 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700551 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700552 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700553}
554
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700555TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700556 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700557 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700558 params.set_wall_clock_based_wait_enabled(false);
559 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700560
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700561 params.set_update_check_count_wait_enabled(true);
562 params.set_min_update_checks_needed(1);
563 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700564
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700565 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700566 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700567 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700568 -1,
569 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700570 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800571 metrics::CheckResult::kUpdateAvailable,
572 metrics::CheckReaction::kUpdating,
573 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700574 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700575 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700576 EXPECT_TRUE(response.update_exists);
577}
578
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700579TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700580 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700581 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700582 params.set_wall_clock_based_wait_enabled(true);
583 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700584
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700585 params.set_update_check_count_wait_enabled(true);
586 params.set_min_update_checks_needed(1);
587 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700588
Alex Deymo8e18f932015-03-27 16:16:59 -0700589 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700590 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700591 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700592 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700593 -1,
594 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700595 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800596 metrics::CheckResult::kUpdateAvailable,
597 metrics::CheckReaction::kUpdating,
598 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700599 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700600 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700601 EXPECT_TRUE(response.update_exists);
602}
603
604
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700605TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700606 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700607 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700608 params.set_wall_clock_based_wait_enabled(true);
609 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700610
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700611 params.set_update_check_count_wait_enabled(true);
612 params.set_min_update_checks_needed(0);
613 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700614
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700615 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700616 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700617 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700618 -1,
619 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700620 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800621 metrics::CheckResult::kUpdateAvailable,
622 metrics::CheckReaction::kUpdating,
623 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700624 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700625 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700626
Ben Chan9abb7632014-08-07 00:10:53 -0700627 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700628 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700629 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700630 EXPECT_TRUE(response.update_exists);
631}
632
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700633TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700634 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700635 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700636 params.set_wall_clock_based_wait_enabled(true);
637 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700638
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700639 params.set_update_check_count_wait_enabled(true);
640 params.set_min_update_checks_needed(1);
641 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700642
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700643 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700644 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700645 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700646 -1,
647 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700648 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800649 metrics::CheckResult::kUpdateAvailable,
650 metrics::CheckReaction::kDeferring,
651 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700652 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700653 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700654
Ben Chan9abb7632014-08-07 00:10:53 -0700655 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700656 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700657 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700658 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700659
660 // Verify if we are interactive check we don't defer.
661 params.set_interactive(true);
662 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700663 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700664 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700665 -1,
666 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700667 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800668 metrics::CheckResult::kUpdateAvailable,
669 metrics::CheckReaction::kUpdating,
670 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700671 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700672 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700673 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700674}
675
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700676TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700677 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700678 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700679 params.set_wall_clock_based_wait_enabled(true);
680 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700681
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700682 params.set_update_check_count_wait_enabled(true);
683 params.set_min_update_checks_needed(1);
684 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700685
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700686 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700687
688 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700689 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700690 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700691 -1,
692 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700693 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800694 metrics::CheckResult::kUpdateAvailable,
695 metrics::CheckReaction::kDeferring,
696 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700697 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700698 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700699
Ben Chan9abb7632014-08-07 00:10:53 -0700700 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700701 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700702 // count remains the same, as the decrementing happens in update_attempter
703 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700704 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700705 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700706
707 // Verify if we are interactive check we don't defer.
708 params.set_interactive(true);
709 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700710 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700711 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700712 -1,
713 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700714 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800715 metrics::CheckResult::kUpdateAvailable,
716 metrics::CheckReaction::kUpdating,
717 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700718 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700719 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700720 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700721}
Jay Srinivasan0a708742012-03-20 11:26:12 -0700722
Alex Deymo8e18f932015-03-27 16:16:59 -0700723TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
724 OmahaResponse response;
725 OmahaRequestParams params = request_params_;
726 fake_update_response_.include_cohorts = true;
727 fake_update_response_.cohort = "s/154454/8479665";
728 fake_update_response_.cohorthint = "please-put-me-on-beta";
729 fake_update_response_.cohortname = "stable";
730
731 ASSERT_TRUE(TestUpdateCheck(&params,
732 fake_update_response_.GetUpdateResponse(),
733 -1,
734 false, // ping_only
735 ErrorCode::kSuccess,
736 metrics::CheckResult::kUpdateAvailable,
737 metrics::CheckReaction::kUpdating,
738 metrics::DownloadErrorCode::kUnset,
739 &response,
740 nullptr));
741
742 string value;
743 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
744 EXPECT_EQ(fake_update_response_.cohort, value);
745
746 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
747 EXPECT_EQ(fake_update_response_.cohorthint, value);
748
749 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
750 EXPECT_EQ(fake_update_response_.cohortname, value);
751}
752
753TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
754 OmahaResponse response;
755 OmahaRequestParams params = request_params_;
756 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
757 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
758 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
759 fake_update_response_.include_cohorts = true;
760 fake_update_response_.cohort = "s/154454/8479665";
761 fake_update_response_.cohorthint = "please-put-me-on-beta";
762 fake_update_response_.cohortname = "";
763
764 ASSERT_TRUE(TestUpdateCheck(&params,
765 fake_update_response_.GetUpdateResponse(),
766 -1,
767 false, // ping_only
768 ErrorCode::kSuccess,
769 metrics::CheckResult::kUpdateAvailable,
770 metrics::CheckReaction::kUpdating,
771 metrics::DownloadErrorCode::kUnset,
772 &response,
773 nullptr));
774
775 string value;
776 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
777 EXPECT_EQ(fake_update_response_.cohort, value);
778
779 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
780 EXPECT_EQ(fake_update_response_.cohorthint, value);
781
782 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
783}
784
785TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
786 OmahaResponse response;
787 OmahaRequestParams params = request_params_;
788 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
789
790 ASSERT_TRUE(TestUpdateCheck(&params,
791 fake_update_response_.GetUpdateResponse(),
792 -1,
793 false, // ping_only
794 ErrorCode::kSuccess,
795 metrics::CheckResult::kUpdateAvailable,
796 metrics::CheckReaction::kUpdating,
797 metrics::DownloadErrorCode::kUnset,
798 &response,
799 nullptr));
800
801 string value;
802 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
803 EXPECT_EQ("old_value", value);
804
805 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
806 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
807}
808
Alex Deymo00d79ac2015-06-29 15:41:49 -0700809TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
810 OmahaResponse response;
811 OmahaRequestParams params = request_params_;
812 fake_update_response_.include_cohorts = true;
813 fake_update_response_.cohort = "s/154454/8479665";
814 fake_update_response_.cohorthint = "please-put-me-on-beta";
815 fake_update_response_.cohortname = "stable";
816
817 ASSERT_TRUE(TestUpdateCheck(&params,
818 fake_update_response_.GetNoUpdateResponse(),
819 -1,
820 false, // ping_only
821 ErrorCode::kSuccess,
822 metrics::CheckResult::kNoUpdateAvailable,
823 metrics::CheckReaction::kUnset,
824 metrics::DownloadErrorCode::kUnset,
825 &response,
826 nullptr));
827
828 string value;
829 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
830 EXPECT_EQ(fake_update_response_.cohort, value);
831
832 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
833 EXPECT_EQ(fake_update_response_.cohorthint, value);
834
835 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
836 EXPECT_EQ(fake_update_response_.cohortname, value);
837}
838
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700839TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -0700840 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700841
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700842 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700843 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700844
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700845 OmahaRequestParams params = request_params_;
846 fake_system_state_.set_request_params(&params);
847 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300848 brillo::make_unique_ptr(
849 new MockHttpFetcher(http_response.data(),
850 http_response.size(),
851 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -0700852 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700853 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700854 ActionProcessor processor;
855 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700856 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700857
Alex Deymo60ca1a72015-06-18 18:19:15 -0700858 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
859 loop.Run();
860 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700861 EXPECT_FALSE(processor.IsRunning());
862}
863
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700864TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700865 OmahaResponse response;
866 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700867 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700868 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700869 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700870 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700871 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800872 metrics::CheckResult::kParsingError,
873 metrics::CheckReaction::kUnset,
874 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -0700875 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700876 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -0700877 EXPECT_FALSE(response.update_exists);
878}
879
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700880TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -0700881 OmahaResponse response;
882 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700883 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -0700884 "",
885 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700886 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700887 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800888 metrics::CheckResult::kParsingError,
889 metrics::CheckReaction::kUnset,
890 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700891 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700892 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700893 EXPECT_FALSE(response.update_exists);
894}
895
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700896TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700897 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700898 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700899 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700900 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
901 "<daystart elapsed_seconds=\"100\"/>"
902 "<app appid=\"foo\" status=\"ok\">"
903 "<ping status=\"ok\"/>"
904 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700905 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700906 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700907 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800908 metrics::CheckResult::kParsingError,
909 metrics::CheckReaction::kUnset,
910 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700911 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700912 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700913 EXPECT_FALSE(response.update_exists);
914}
915
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700916TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700917 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700918 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700919 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700920 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
921 "<daystart elapsed_seconds=\"100\"/>"
922 "<app appid=\"foo\" status=\"ok\">"
923 "<ping status=\"ok\"/>"
924 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700925 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700926 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700927 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800928 metrics::CheckResult::kParsingError,
929 metrics::CheckReaction::kUnset,
930 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700931 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700932 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700933 EXPECT_FALSE(response.update_exists);
934}
935
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700936TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700937 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700938 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700939 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700940 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
941 "<daystart elapsed_seconds=\"100\"/>"
942 "<app appid=\"foo\" status=\"ok\">"
943 "<ping status=\"ok\"/>"
944 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700945 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700946 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700947 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800948 metrics::CheckResult::kParsingError,
949 metrics::CheckReaction::kUnset,
950 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700951 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700952 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700953 EXPECT_FALSE(response.update_exists);
954}
955
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700956TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700957 string input_response =
958 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
959 "<daystart elapsed_seconds=\"100\"/>"
960 "<app appid=\"xyz\" status=\"ok\">"
961 "<updatecheck status=\"ok\">"
962 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -0700963 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700964 "<packages><package hash=\"not-used\" name=\"f\" "
965 "size=\"587\"/></packages>"
966 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700967 "ChromeOSVersion=\"10.2.3.4\" "
968 "Prompt=\"false\" "
969 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -0800970 "IsDeltaPayload=\"false\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700971 "sha256=\"lkq34j5345\" "
972 "needsadmin=\"true\" "
973 "/></actions></manifest></updatecheck></app></response>";
974 LOG(INFO) << "Input Response = " << input_response;
975
Darin Petkov6a5b3222010-07-13 14:55:28 -0700976 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700977 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700978 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -0700979 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700980 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700981 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800982 metrics::CheckResult::kUpdateAvailable,
983 metrics::CheckReaction::kUpdating,
984 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700985 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700986 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700987 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -0700988 EXPECT_EQ("10.2.3.4", response.version);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800989 EXPECT_EQ("http://missing/field/test/f", response.payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700990 EXPECT_EQ("", response.more_info_url);
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700991 EXPECT_EQ("lkq34j5345", response.hash);
992 EXPECT_EQ(587, response.size);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700993 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -0700994 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700995}
996
997namespace {
998class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
999 public:
1000 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001001 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001002 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001003};
1004
Alex Deymo60ca1a72015-06-18 18:19:15 -07001005void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001006 processor->StartProcessing();
1007 CHECK(processor->IsRunning());
1008 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001009}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001010} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001011
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001012TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001013 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001014 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001015
Alex Deymo60ca1a72015-06-18 18:19:15 -07001016 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001017 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001018 brillo::make_unique_ptr(
1019 new MockHttpFetcher(http_response.data(),
1020 http_response.size(),
1021 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001022 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001023 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001024 ActionProcessor processor;
1025 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001026 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001027
Alex Deymo60ca1a72015-06-18 18:19:15 -07001028 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1029 loop.Run();
1030 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001031}
1032
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001033TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001034 string output;
1035 EXPECT_TRUE(XmlEncode("ab", &output));
1036 EXPECT_EQ("ab", output);
1037 EXPECT_TRUE(XmlEncode("a<b", &output));
1038 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001039 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1040 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001041 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1042 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001043 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001044 EXPECT_FALSE(XmlEncode("\xc2", &output));
1045 // Fail with invalid ASCII-7 chars.
1046 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1047}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001048
Alex Deymob0d74eb2015-03-30 17:59:17 -07001049TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1050 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1051 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1052}
1053
1054TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001055 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001056
1057 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001058 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001059 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001060 OmahaRequestParams::kOsVersion,
1061 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001062 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001063 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001064 "0.1.0.0",
1065 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001066 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001067 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001068 "ChromeOSFirmware.1.0",
1069 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001070 false, // delta okay
1071 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001072 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001073 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001074 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1075 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1076 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001077 base::JoinString(
1078 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001079 OmahaResponse response;
1080 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001081 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001082 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001083 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001084 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001085 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001086 metrics::CheckResult::kParsingError,
1087 metrics::CheckReaction::kUnset,
1088 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001089 &response,
1090 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001091 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001092 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001093 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1094 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1095 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1096 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1097 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1098 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1099 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1100 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1101 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1102 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1103 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1104 // Values from Prefs that are too big are removed from the XML instead of
1105 // encoded.
1106 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001107}
1108
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001109TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001110 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001111 fake_update_response_.deadline = "&lt;20110101";
1112 fake_update_response_.more_info_url = "testthe&lt;url";
1113 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001114 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001115 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001116 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001117 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001118 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001119 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001120 metrics::CheckResult::kUpdateAvailable,
1121 metrics::CheckReaction::kUpdating,
1122 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001123 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001124 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001125
1126 EXPECT_EQ(response.more_info_url, "testthe<url");
Jay Srinivasan6f6ea002012-12-14 11:26:28 -08001127 EXPECT_EQ(response.payload_urls[0], "testthe&codebase/file.signed");
Darin Petkov6c118642010-10-21 12:06:30 -07001128 EXPECT_EQ(response.deadline, "<20110101");
Darin Petkov6a5b3222010-07-13 14:55:28 -07001129}
1130
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001131TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001132 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001133 // overflows int32_t:
1134 fake_update_response_.size = 123123123123123ll;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001135 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001136 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001137 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001138 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001139 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001140 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001141 metrics::CheckResult::kUpdateAvailable,
1142 metrics::CheckReaction::kUpdating,
1143 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001144 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001145 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001146
1147 EXPECT_EQ(response.size, 123123123123123ll);
1148}
1149
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001150TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001151 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001152 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001153 fake_system_state_.set_prefs(&prefs);
1154
Darin Petkov95508da2011-01-05 12:42:29 -08001155 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1156 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001157 // An existing but empty previous version means that we didn't reboot to a new
1158 // update, therefore, no need to update the previous version.
1159 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001160 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001161 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001162 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001163 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001164 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001165 metrics::CheckResult::kParsingError,
1166 metrics::CheckReaction::kUnset,
1167 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001168 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001169 &post_data));
1170 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001171 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001172 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001173 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001174 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001175 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001176 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1177 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001178 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1179 string::npos);
1180 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1181 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001182 // No <event> tag should be sent if we didn't reboot to an update.
1183 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001184}
1185
Jay Srinivasan0a708742012-03-20 11:26:12 -07001186
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001187TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001188 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001189 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001190 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1191 "invalid xml>",
1192 &post_data);
1193 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001194 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001195 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001196 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001197 OmahaEvent::kTypeUpdateDownloadStarted,
1198 OmahaEvent::kResultSuccess);
1199 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001200 EXPECT_EQ(post_str.find("ping"), string::npos);
1201 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001202}
1203
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001204TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001205 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001206 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001207 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1208 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001209 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001210 "invalid xml>",
1211 &post_data);
1212 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001213 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001214 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001215 " <event eventtype=\"%d\" eventresult=\"%d\" "
1216 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001217 OmahaEvent::kTypeDownloadComplete,
1218 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001219 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001220 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001221 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001222}
1223
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001224TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001225 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001226 // Create a copy of the OmahaRequestParams to reuse it later.
1227 OmahaRequestParams params = request_params_;
1228 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001229 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001230 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001231 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001232 brillo::make_unique_ptr(
1233 new MockHttpFetcher(http_response.data(),
1234 http_response.size(),
1235 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001236 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001237 EXPECT_FALSE(update_check_action.IsEvent());
1238
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001239 params = request_params_;
1240 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001241 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001242 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001243 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001244 brillo::make_unique_ptr(
1245 new MockHttpFetcher(http_response.data(),
1246 http_response.size(),
1247 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001248 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001249 EXPECT_TRUE(event_action.IsEvent());
1250}
1251
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001252TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001253 for (int i = 0; i < 2; i++) {
1254 bool delta_okay = i == 1;
1255 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001256 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001257 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001258 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001259 OmahaRequestParams::kOsVersion,
1260 "service_pack",
1261 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001262 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001263 "0.1.0.0",
1264 "en-US",
1265 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001266 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001267 "ChromeOSFirmware.1.0",
1268 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001269 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001270 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001271 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001272 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001273 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001274 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001275 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001276 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001277 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001278 metrics::CheckResult::kParsingError,
1279 metrics::CheckReaction::kUnset,
1280 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001281 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001282 &post_data));
1283 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001284 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001285 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1286 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001287 string::npos)
1288 << "i = " << i;
1289 }
1290}
1291
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001292TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001293 for (int i = 0; i < 2; i++) {
1294 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001295 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001296 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001297 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001298 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001299 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001300 OmahaRequestParams::kOsVersion,
1301 "service_pack",
1302 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001303 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001304 "0.1.0.0",
1305 "en-US",
1306 "unittest_track",
1307 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001308 "ChromeOSFirmware.1.0",
1309 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001310 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001311 interactive,
1312 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001313 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001314 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001315 "invalid xml>",
1316 -1,
1317 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001318 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001319 metrics::CheckResult::kParsingError,
1320 metrics::CheckReaction::kUnset,
1321 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001322 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001323 &post_data));
1324 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001325 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001326 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1327 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001328 string::npos)
1329 << "i = " << i;
1330 }
1331}
1332
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001333TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001334 OmahaEvent default_event;
1335 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1336 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001337 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001338
1339 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1340 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1341 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001342 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001343
1344 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1345 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001346 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001347 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1348 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001349 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001350}
1351
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001352void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001353 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001354 fake_system_state_.set_prefs(&prefs);
1355 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1356 .Times(AnyNumber());
1357 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1358 // Add a few hours to the day difference to test no rounding, etc.
1359 int64_t five_days_ago =
1360 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1361 int64_t six_days_ago =
1362 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1363 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1364 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1365 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1366 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1367 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1368 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001369 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001370 ASSERT_TRUE(
1371 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001372 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001373 -1,
1374 ping_only,
1375 ErrorCode::kSuccess,
1376 metrics::CheckResult::kUnset,
1377 metrics::CheckReaction::kUnset,
1378 metrics::DownloadErrorCode::kUnset,
1379 nullptr,
1380 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001381 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001382 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1383 string::npos);
1384 if (ping_only) {
1385 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1386 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1387 } else {
1388 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1389 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001390 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001391}
1392
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001393TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1394 PingTest(true /* ping_only */);
1395}
1396
1397TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1398 PingTest(false /* ping_only */);
1399}
1400
1401TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001402 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001403 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001404 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1405 .Times(AnyNumber());
1406 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001407 int64_t three_days_ago =
1408 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1409 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001410 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1411 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001412 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1413 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1414 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1415 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001416 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001417 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001418 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001419 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001420 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001421 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001422 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001423 metrics::CheckResult::kNoUpdateAvailable,
1424 metrics::CheckReaction::kUnset,
1425 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001426 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001427 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001428 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001429 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001430 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001431}
1432
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001433TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001434 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001435 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001436 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1437 .Times(AnyNumber());
1438 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001439 int64_t four_days_ago =
1440 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1441 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001442 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1443 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001444 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1445 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1446 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1447 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001448 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001449 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001450 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001451 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001452 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001453 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001454 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001455 metrics::CheckResult::kNoUpdateAvailable,
1456 metrics::CheckReaction::kUnset,
1457 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001458 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001459 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001460 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001461 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001462 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001463}
1464
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001465TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001466 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001467 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001468 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1469 .Times(AnyNumber());
1470 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001471 int64_t one_hour_ago =
1472 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001473 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1474 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001475 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1476 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1477 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1478 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001479 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1480 // send a ping.
1481 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1482 .WillOnce(Return(true));
1483 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1484 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001485 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001486 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001487 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001488 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001489 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001490 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001491 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001492 metrics::CheckResult::kNoUpdateAvailable,
1493 metrics::CheckReaction::kUnset,
1494 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001495 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001496 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001497 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001498 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001499}
1500
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001501TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001502 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001503 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001504 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001505 int64_t now = Time::Now().ToInternalValue();
1506 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1507 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1508 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1509 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1510 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1511 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001512 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001513 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001514 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001515 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001516 -1,
1517 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001518 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001519 metrics::CheckResult::kUnset,
1520 metrics::CheckReaction::kUnset,
1521 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001522 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001523 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001524 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001525}
1526
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001527TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001528 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001529 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001530 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1531 .Times(AnyNumber());
1532 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001533 int64_t future =
1534 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001535 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1536 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001537 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1538 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1539 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1540 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1541 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1542 .WillOnce(Return(true));
1543 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1544 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001545 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001546 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001547 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001548 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1549 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001550 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001551 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001552 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001553 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001554 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001555 metrics::CheckResult::kNoUpdateAvailable,
1556 metrics::CheckReaction::kUnset,
1557 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001558 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001559 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001560 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001561 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001562}
1563
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001564TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001565 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001566 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001567 // may fail if it runs for longer than 5 seconds. It shouldn't run
1568 // that long though.
1569 int64_t midnight =
1570 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1571 int64_t midnight_slack =
1572 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001573 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001574 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001575 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1576 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001577 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1578 AllOf(Ge(midnight), Le(midnight_slack))))
1579 .WillOnce(Return(true));
1580 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1581 AllOf(Ge(midnight), Le(midnight_slack))))
1582 .WillOnce(Return(true));
1583 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001584 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001585 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1586 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001587 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001588 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001589 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001590 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001591 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001592 metrics::CheckResult::kNoUpdateAvailable,
1593 metrics::CheckReaction::kUnset,
1594 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001595 nullptr,
1596 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001597}
1598
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001599TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001600 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001601 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001602 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1603 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001604 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1605 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1606 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001607 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001608 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1609 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001610 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001611 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001612 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001613 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001614 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001615 metrics::CheckResult::kNoUpdateAvailable,
1616 metrics::CheckReaction::kUnset,
1617 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001618 nullptr,
1619 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001620}
1621
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001622TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001623 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001624 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001625 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1626 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001627 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1628 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1629 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001630 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001631 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1632 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001633 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001634 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001635 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001636 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001637 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001638 metrics::CheckResult::kNoUpdateAvailable,
1639 metrics::CheckReaction::kUnset,
1640 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001641 nullptr,
1642 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001643}
1644
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001645TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001646 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001647 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07001648 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001649 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001650 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001651 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001652 metrics::CheckResult::kParsingError,
1653 metrics::CheckReaction::kUnset,
1654 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001655 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07001656 &post_data));
1657 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001658 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07001659 EXPECT_EQ(post_str.find("machineid="), string::npos);
1660 EXPECT_EQ(post_str.find("userid="), string::npos);
1661}
1662
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001663TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001664 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001665 const int http_error_code =
1666 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07001667 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001668 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001669 "",
1670 501,
Darin Petkov265f2902011-05-09 15:17:40 -07001671 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001672 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001673 metrics::CheckResult::kDownloadError,
1674 metrics::CheckReaction::kUnset,
1675 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07001676 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001677 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001678 EXPECT_FALSE(response.update_exists);
1679}
1680
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001681TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001682 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001683 const int http_error_code =
1684 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07001685 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001686 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001687 "",
1688 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07001689 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001690 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001691 metrics::CheckResult::kDownloadError,
1692 metrics::CheckReaction::kUnset,
1693 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07001694 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001695 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001696 EXPECT_FALSE(response.update_exists);
1697}
1698
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001699TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001700 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001701 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001702 params.set_wall_clock_based_wait_enabled(true);
1703 params.set_waiting_period(TimeDelta().FromDays(1));
1704 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001705
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001706 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001707 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001708 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001709 -1,
1710 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001711 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001712 metrics::CheckResult::kUpdateAvailable,
1713 metrics::CheckReaction::kDeferring,
1714 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001715 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001716 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001717
Ben Chan9abb7632014-08-07 00:10:53 -07001718 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001719 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001720 ASSERT_GT(timestamp, 0);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001721 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001722
1723 // Verify if we are interactive check we don't defer.
1724 params.set_interactive(true);
1725 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001726 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001727 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001728 -1,
1729 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001730 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001731 metrics::CheckResult::kUpdateAvailable,
1732 metrics::CheckReaction::kUpdating,
1733 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001734 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001735 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001736 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001737}
1738
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001739TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001740 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001741 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001742 params.set_wall_clock_based_wait_enabled(true);
1743 params.set_waiting_period(TimeDelta().FromDays(1));
1744 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001745
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001746 // Set the timestamp to a very old value such that it exceeds the
1747 // waiting period set above.
1748 Time t1;
1749 Time::FromString("1/1/2012", &t1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001750 ASSERT_TRUE(fake_prefs_.SetInt64(
1751 kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001752 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001753 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001754 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001755 -1,
1756 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001757 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001758 metrics::CheckResult::kUpdateAvailable,
1759 metrics::CheckReaction::kUpdating,
1760 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001761 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001762 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001763
1764 EXPECT_TRUE(response.update_exists);
1765
1766 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07001767 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001768 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001769 ASSERT_TRUE(timestamp == t1.ToInternalValue());
1770}
1771
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001772TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001773 // Create a uniquely named test directory.
1774 string test_dir;
1775 ASSERT_TRUE(utils::MakeTempDirectory(
1776 "omaha_request_action-test-XXXXXX", &test_dir));
1777
1778 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1779 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001780 kStatefulPartition + "/etc"));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001781 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001782 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001783 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001784 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001785 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001786 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1787 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1788 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
1789 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001790 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001791 "CHROMEOS_IS_POWERWASH_ALLOWED=true\n"
1792 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001793 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001794 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001795 params.Init("1.2.3.4", "", 0);
1796 EXPECT_EQ("canary-channel", params.current_channel());
1797 EXPECT_EQ("stable-channel", params.target_channel());
1798 EXPECT_TRUE(params.to_more_stable_channel());
1799 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001800 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001801 "invalid xml>",
1802 -1,
1803 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001804 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001805 metrics::CheckResult::kParsingError,
1806 metrics::CheckReaction::kUnset,
1807 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001808 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001809 &post_data));
1810 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001811 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001812 EXPECT_NE(string::npos, post_str.find(
1813 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
1814 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
1815 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001816
Alex Deymo110e0302015-10-19 20:35:21 -07001817 ASSERT_TRUE(base::DeleteFile(base::FilePath(test_dir), true));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001818}
1819
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001820TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001821 // Create a uniquely named test directory.
1822 string test_dir;
1823 ASSERT_TRUE(utils::MakeTempDirectory(
1824 "omaha_request_action-test-XXXXXX", &test_dir));
1825
1826 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1827 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001828 kStatefulPartition + "/etc"));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001829 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001830 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001831 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001832 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001833 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001834 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1835 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1836 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
1837 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001838 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001839 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001840 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001841 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001842 params.Init("5.6.7.8", "", 0);
1843 EXPECT_EQ("stable-channel", params.current_channel());
1844 EXPECT_EQ("canary-channel", params.target_channel());
1845 EXPECT_FALSE(params.to_more_stable_channel());
1846 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001847 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001848 "invalid xml>",
1849 -1,
1850 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001851 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001852 metrics::CheckResult::kParsingError,
1853 metrics::CheckReaction::kUnset,
1854 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001855 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001856 &post_data));
1857 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001858 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001859 EXPECT_NE(string::npos, post_str.find(
1860 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
1861 "version=\"5.6.7.8\" "
1862 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001863 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001864}
1865
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001866// Checks that the initial ping with a=-1 r=-1 is not send when the device
1867// was powerwashed.
1868TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
1869 fake_prefs_.SetString(kPrefsPreviousVersion, "");
1870
1871 // Flag that the device was powerwashed in the past.
1872 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
1873
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001874 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001875 ASSERT_TRUE(
1876 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001877 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001878 -1,
1879 false, // ping_only
1880 ErrorCode::kSuccess,
1881 metrics::CheckResult::kNoUpdateAvailable,
1882 metrics::CheckReaction::kUnset,
1883 metrics::DownloadErrorCode::kUnset,
1884 nullptr,
1885 &post_data));
1886 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001887 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001888 EXPECT_EQ(string::npos, post_str.find("<ping"));
1889}
1890
Alex Deymo9fded1e2015-11-05 12:31:19 -08001891// Checks that the event 54 is sent on a reboot to a new update.
1892TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
1893 // Flag that the device was updated in a previous boot.
1894 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
1895
1896 brillo::Blob post_data;
1897 ASSERT_TRUE(
1898 TestUpdateCheck(nullptr, // request_params
1899 fake_update_response_.GetNoUpdateResponse(),
1900 -1,
1901 false, // ping_only
1902 ErrorCode::kSuccess,
1903 metrics::CheckResult::kNoUpdateAvailable,
1904 metrics::CheckReaction::kUnset,
1905 metrics::DownloadErrorCode::kUnset,
1906 nullptr,
1907 &post_data));
1908 string post_str(post_data.begin(), post_data.end());
1909
1910 // An event 54 is included and has the right version.
1911 EXPECT_NE(string::npos,
1912 post_str.find(base::StringPrintf(
1913 "<event eventtype=\"%d\"",
1914 OmahaEvent::kTypeRebootedAfterUpdate)));
1915 EXPECT_NE(string::npos,
1916 post_str.find("previousversion=\"1.2.3.4\"></event>"));
1917
1918 // The previous version flag should have been removed.
1919 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
1920 string prev_version;
1921 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
1922 EXPECT_TRUE(prev_version.empty());
1923}
1924
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001925void OmahaRequestActionTest::P2PTest(
1926 bool initial_allow_p2p_for_downloading,
1927 bool initial_allow_p2p_for_sharing,
1928 bool omaha_disable_p2p_for_downloading,
1929 bool omaha_disable_p2p_for_sharing,
1930 bool payload_state_allow_p2p_attempt,
1931 bool expect_p2p_client_lookup,
1932 const string& p2p_client_result_url,
1933 bool expected_allow_p2p_for_downloading,
1934 bool expected_allow_p2p_for_sharing,
1935 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07001936 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001937 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07001938 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
1939 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
1940 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07001941
1942 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001943 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07001944 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
1945 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07001946 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
1947 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
1948 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
1949 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
1950 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
1951 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
1952 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
1953 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
1954 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
1955 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
1956
David Zeuthen8f191b22013-08-06 12:27:50 -07001957 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001958 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07001959 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
1960
David Zeuthen4cc5ed22014-01-15 12:35:03 -08001961 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
1962 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07001963 .Times(expect_p2p_client_lookup ? 1 : 0);
1964
Alex Deymo8e18f932015-03-27 16:16:59 -07001965 fake_update_response_.disable_p2p_for_downloading =
1966 omaha_disable_p2p_for_downloading;
1967 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07001968 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001969 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001970 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07001971 -1,
1972 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001973 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001974 metrics::CheckResult::kUpdateAvailable,
1975 metrics::CheckReaction::kUpdating,
1976 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07001977 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001978 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07001979 EXPECT_TRUE(response.update_exists);
1980
Gilad Arnold74b5f552014-10-07 08:17:16 -07001981 EXPECT_EQ(omaha_disable_p2p_for_downloading,
1982 response.disable_p2p_for_downloading);
1983 EXPECT_EQ(omaha_disable_p2p_for_sharing,
1984 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07001985
Gilad Arnold74b5f552014-10-07 08:17:16 -07001986 EXPECT_EQ(expected_allow_p2p_for_downloading,
1987 actual_allow_p2p_for_downloading);
1988 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
1989 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07001990}
1991
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001992TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001993 P2PTest(true, // initial_allow_p2p_for_downloading
1994 true, // initial_allow_p2p_for_sharing
1995 false, // omaha_disable_p2p_for_downloading
1996 false, // omaha_disable_p2p_for_sharing
1997 true, // payload_state_allow_p2p_attempt
1998 true, // expect_p2p_client_lookup
1999 "http://1.3.5.7/p2p", // p2p_client_result_url
2000 true, // expected_allow_p2p_for_downloading
2001 true, // expected_allow_p2p_for_sharing
2002 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002003}
2004
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002005TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002006 P2PTest(true, // initial_allow_p2p_for_downloading
2007 true, // initial_allow_p2p_for_sharing
2008 false, // omaha_disable_p2p_for_downloading
2009 false, // omaha_disable_p2p_for_sharing
2010 true, // payload_state_allow_p2p_attempt
2011 true, // expect_p2p_client_lookup
2012 "", // p2p_client_result_url
2013 false, // expected_allow_p2p_for_downloading
2014 true, // expected_allow_p2p_for_sharing
2015 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002016}
2017
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002018TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002019 P2PTest(false, // initial_allow_p2p_for_downloading
2020 true, // initial_allow_p2p_for_sharing
2021 false, // omaha_disable_p2p_for_downloading
2022 false, // omaha_disable_p2p_for_sharing
2023 true, // payload_state_allow_p2p_attempt
2024 false, // expect_p2p_client_lookup
2025 "unset", // p2p_client_result_url
2026 false, // expected_allow_p2p_for_downloading
2027 true, // expected_allow_p2p_for_sharing
2028 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002029}
2030
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002031TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002032 P2PTest(true, // initial_allow_p2p_for_downloading
2033 true, // initial_allow_p2p_for_sharing
2034 true, // omaha_disable_p2p_for_downloading
2035 false, // omaha_disable_p2p_for_sharing
2036 true, // payload_state_allow_p2p_attempt
2037 false, // expect_p2p_client_lookup
2038 "unset", // p2p_client_result_url
2039 false, // expected_allow_p2p_for_downloading
2040 true, // expected_allow_p2p_for_sharing
2041 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002042}
2043
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002044TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002045 P2PTest(true, // initial_allow_p2p_for_downloading
2046 true, // initial_allow_p2p_for_sharing
2047 false, // omaha_disable_p2p_for_downloading
2048 true, // omaha_disable_p2p_for_sharing
2049 true, // payload_state_allow_p2p_attempt
2050 true, // expect_p2p_client_lookup
2051 "http://1.3.5.7/p2p", // p2p_client_result_url
2052 true, // expected_allow_p2p_for_downloading
2053 false, // expected_allow_p2p_for_sharing
2054 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002055}
2056
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002057TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002058 P2PTest(true, // initial_allow_p2p_for_downloading
2059 true, // initial_allow_p2p_for_sharing
2060 true, // omaha_disable_p2p_for_downloading
2061 true, // omaha_disable_p2p_for_sharing
2062 true, // payload_state_allow_p2p_attempt
2063 false, // expect_p2p_client_lookup
2064 "unset", // p2p_client_result_url
2065 false, // expected_allow_p2p_for_downloading
2066 false, // expected_allow_p2p_for_sharing
2067 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002068}
2069
Alex Deymof329b932014-10-30 01:37:48 -07002070bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2071 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002072 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002073 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002074 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002075 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002076 -1,
2077 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002078 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002079 metrics::CheckResult::kUpdateAvailable,
2080 metrics::CheckReaction::kUpdating,
2081 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002082 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002083 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002084}
2085
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002086TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002087 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002088
2089 // Check that we parse elapsed_days in the Omaha Response correctly.
2090 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002091 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2092 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002093 EXPECT_TRUE(response.update_exists);
2094 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002095 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002096 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002097 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002098 EXPECT_EQ(prefs_days, 42);
2099
2100 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002101 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002102 EXPECT_TRUE(response.update_exists);
2103 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002104 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002105 EXPECT_EQ(prefs_days, 42);
2106
2107 // Note that elapsed_days is not necessarily divisible by 7 so check
2108 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002109 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2110 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002111 EXPECT_TRUE(response.update_exists);
2112 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002113 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002114 EXPECT_EQ(prefs_days, 21);
2115
2116 // Check that we correctly handle elapsed_days not being included in
2117 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002118 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002119 EXPECT_TRUE(response.update_exists);
2120 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002121}
2122
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002123// If there is no prefs and OOBE is not complete, we should not
2124// report anything to Omaha.
2125TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
2126 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2127 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2128}
David Zeuthen639aa362014-02-03 16:23:44 -08002129
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002130// If OOBE is complete and happened on a valid date (e.g. after Jan
2131// 1 2007 0:00 PST), that date should be used and written to
2132// prefs. However, first try with an invalid date and check we do
2133// nothing.
2134TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2135 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2136 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2137 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2138 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2139}
David Zeuthen639aa362014-02-03 16:23:44 -08002140
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002141// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2142// should yield an InstallDate of 14.
2143TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2144 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2145 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2146 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2147 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002148
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002149 int64_t prefs_days;
2150 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2151 EXPECT_EQ(prefs_days, 14);
2152}
David Zeuthen639aa362014-02-03 16:23:44 -08002153
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002154// Now that we have a valid date in prefs, check that we keep using
2155// that even if OOBE date reports something else. The date Jan 30,
2156// 2007 0:00 PST should yield an InstallDate of 28... but since
2157// there's a prefs file, we should still get 14.
2158TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2159 // Set a valid date in the prefs first.
2160 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002161
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002162 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2163 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2164 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002165
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002166 int64_t prefs_days;
2167 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2168 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002169
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002170 // If we delete the prefs file, we should get 28 days.
2171 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2172 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2173 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2174 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002175}
2176
Darin Petkov6a5b3222010-07-13 14:55:28 -07002177} // namespace chromeos_update_engine