blob: 69585c8326dcd72908d0826a6a84a269e2f34e19 [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(); }
Darin Petkov6a5b3222010-07-13 14:55:28 -0700295 bool has_input_object_;
296 OmahaResponse omaha_response_;
297};
298
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700299bool OmahaRequestActionTest::TestUpdateCheck(
300 OmahaRequestParams* request_params,
301 const string& http_response,
302 int fail_http_response_code,
303 bool ping_only,
304 ErrorCode expected_code,
305 metrics::CheckResult expected_check_result,
306 metrics::CheckReaction expected_check_reaction,
307 metrics::DownloadErrorCode expected_download_error_code,
308 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700309 brillo::Blob* out_post_data) {
310 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700311 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700312 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800313 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700314 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700315 if (fail_http_response_code >= 0) {
316 fetcher->FailTransfer(fail_http_response_code);
317 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700318 if (request_params)
319 fake_system_state_.set_request_params(request_params);
320 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700321 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300322 brillo::make_unique_ptr(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700323 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700324 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700325 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700326
Darin Petkov6a5b3222010-07-13 14:55:28 -0700327 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700328 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700329 processor.EnqueueAction(&action);
330
331 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700332 BondActions(&action, &collector_action);
333 processor.EnqueueAction(&collector_action);
334
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700335 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800336 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700337 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800338 SendEnumToUMA(metrics::kMetricCheckResult,
339 static_cast<int>(expected_check_result),
340 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
341 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700342 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800343 SendEnumToUMA(metrics::kMetricCheckReaction,
344 static_cast<int>(expected_check_reaction),
345 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
346 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700347 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700348 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
349 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800350 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
351 ? 0 : 1);
352
Alex Deymo60ca1a72015-06-18 18:19:15 -0700353 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
354 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
355 loop.Run();
356 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
357 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700358 if (collector_action.has_input_object_ && out_response)
359 *out_response = collector_action.omaha_response_;
360 if (out_post_data)
361 *out_post_data = fetcher->post_data();
362 return collector_action.has_input_object_;
363}
364
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700365// Tests Event requests -- they should always succeed. |out_post_data|
366// may be null; if non-null, the post-data received by the mock
367// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700368void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700369 OmahaEvent* event,
370 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700371 brillo::Blob* out_post_data) {
372 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700373 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700374 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800375 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700376 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700377 FakeSystemState fake_system_state;
378 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300379 OmahaRequestAction action(&fake_system_state,
380 event,
381 brillo::make_unique_ptr(fetcher),
382 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700383 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700384 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700385 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700386 processor.EnqueueAction(&action);
387
Alex Deymo60ca1a72015-06-18 18:19:15 -0700388 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
389 loop.Run();
390
391 // This test should schedule a callback to notify the crash reporter if
392 // the passed event is an error.
393 EXPECT_EQ(event->result == OmahaEvent::kResultError, loop.PendingTasks());
394
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700395 if (out_post_data)
396 *out_post_data = fetcher->post_data();
397}
398
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700399TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400400 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700401 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400402 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700403 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700404 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400405 -1,
406 false, // ping_only
407 ErrorCode::kOmahaRequestXMLHasEntityDecl,
408 metrics::CheckResult::kParsingError,
409 metrics::CheckReaction::kUnset,
410 metrics::DownloadErrorCode::kUnset,
411 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700412 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400413 EXPECT_FALSE(response.update_exists);
414}
415
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700416TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700417 OmahaResponse response;
418 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700419 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700420 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700421 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700422 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700423 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800424 metrics::CheckResult::kNoUpdateAvailable,
425 metrics::CheckReaction::kUnset,
426 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700427 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700428 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700429 EXPECT_FALSE(response.update_exists);
430}
431
Alex Deymo8e18f932015-03-27 16:16:59 -0700432// Test that all the values in the response are parsed in a normal update
433// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700434TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700435 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700436 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700437 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700438 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700439 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700440 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700441 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700442 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800443 metrics::CheckResult::kUpdateAvailable,
444 metrics::CheckReaction::kUpdating,
445 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700446 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700447 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700448 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -0700449 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700450 EXPECT_EQ(fake_update_response_.version, response.version);
451 EXPECT_EQ(fake_update_response_.GetPayloadUrl(), response.payload_urls[0]);
452 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
453 EXPECT_EQ(fake_update_response_.hash, response.hash);
454 EXPECT_EQ(fake_update_response_.size, response.size);
455 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
456 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
457 // Omaha cohort attribets are not set in the response, so they should not be
458 // persisted.
459 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
460 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
461 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700462}
463
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700464TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700465 OmahaResponse response;
466 // Set up a connection manager that doesn't allow a valid update over
467 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700468 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700469 fake_system_state_.set_connection_manager(&mock_cm);
470
Alex Deymo30534502015-07-20 15:06:33 -0700471 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
472 .WillRepeatedly(
473 DoAll(SetArgumentPointee<0>(NetworkConnectionType::kEthernet),
474 SetArgumentPointee<1>(NetworkTethering::kUnknown),
475 Return(true)));
Alex Deymo75eac7e2015-07-29 13:39:14 -0700476 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(NetworkConnectionType::kEthernet, _))
Chris Sosa77f79e82014-06-02 18:16:24 -0700477 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700478
479 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700480 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700481 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700482 -1,
483 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700484 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700485 metrics::CheckResult::kUpdateAvailable,
486 metrics::CheckReaction::kIgnored,
487 metrics::DownloadErrorCode::kUnset,
488 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700489 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700490 EXPECT_FALSE(response.update_exists);
491}
492
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700493TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700494 string rollback_version = "1234.0.0";
495 OmahaResponse response;
496
497 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700498 fake_system_state_.set_payload_state(&mock_payload_state);
499
Chris Sosa77f79e82014-06-02 18:16:24 -0700500 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
501 .WillRepeatedly(Return(rollback_version));
502
Alex Deymo8e18f932015-03-27 16:16:59 -0700503 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700504 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700505 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700506 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700507 -1,
508 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700509 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700510 metrics::CheckResult::kUpdateAvailable,
511 metrics::CheckReaction::kIgnored,
512 metrics::DownloadErrorCode::kUnset,
513 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700514 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700515 EXPECT_FALSE(response.update_exists);
516}
517
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700518TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700519 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700520 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700521 params.set_wall_clock_based_wait_enabled(true);
522 params.set_update_check_count_wait_enabled(false);
523 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700524
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700525 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700526 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700527 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700528 -1,
529 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700530 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800531 metrics::CheckResult::kUpdateAvailable,
532 metrics::CheckReaction::kDeferring,
533 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700534 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700535 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700536 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700537
538 // Verify if we are interactive check we don't defer.
539 params.set_interactive(true);
540 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700541 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700542 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700543 -1,
544 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700545 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800546 metrics::CheckResult::kUpdateAvailable,
547 metrics::CheckReaction::kUpdating,
548 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700549 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700550 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700551 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700552}
553
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700554TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700555 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700556 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700557 params.set_wall_clock_based_wait_enabled(false);
558 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700559
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700560 params.set_update_check_count_wait_enabled(true);
561 params.set_min_update_checks_needed(1);
562 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700563
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700564 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700565 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700566 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700567 -1,
568 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700569 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800570 metrics::CheckResult::kUpdateAvailable,
571 metrics::CheckReaction::kUpdating,
572 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700573 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700574 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700575 EXPECT_TRUE(response.update_exists);
576}
577
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700578TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700579 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700580 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700581 params.set_wall_clock_based_wait_enabled(true);
582 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700583
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700584 params.set_update_check_count_wait_enabled(true);
585 params.set_min_update_checks_needed(1);
586 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700587
Alex Deymo8e18f932015-03-27 16:16:59 -0700588 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700589 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700590 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700591 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700592 -1,
593 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700594 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800595 metrics::CheckResult::kUpdateAvailable,
596 metrics::CheckReaction::kUpdating,
597 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700598 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700599 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700600 EXPECT_TRUE(response.update_exists);
601}
602
603
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700604TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700605 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700606 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700607 params.set_wall_clock_based_wait_enabled(true);
608 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700609
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700610 params.set_update_check_count_wait_enabled(true);
611 params.set_min_update_checks_needed(0);
612 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700613
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700614 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700615 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700616 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700617 -1,
618 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700619 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800620 metrics::CheckResult::kUpdateAvailable,
621 metrics::CheckReaction::kUpdating,
622 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700623 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700624 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700625
Ben Chan9abb7632014-08-07 00:10:53 -0700626 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700627 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700628 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700629 EXPECT_TRUE(response.update_exists);
630}
631
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700632TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700633 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700634 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700635 params.set_wall_clock_based_wait_enabled(true);
636 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700637
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700638 params.set_update_check_count_wait_enabled(true);
639 params.set_min_update_checks_needed(1);
640 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700641
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700642 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700643 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700644 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700645 -1,
646 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700647 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800648 metrics::CheckResult::kUpdateAvailable,
649 metrics::CheckReaction::kDeferring,
650 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700651 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700652 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700653
Ben Chan9abb7632014-08-07 00:10:53 -0700654 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700655 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700656 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700657 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700658
659 // Verify if we are interactive check we don't defer.
660 params.set_interactive(true);
661 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700662 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700663 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700664 -1,
665 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700666 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800667 metrics::CheckResult::kUpdateAvailable,
668 metrics::CheckReaction::kUpdating,
669 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700670 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700671 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700672 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700673}
674
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700675TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700676 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700677 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700678 params.set_wall_clock_based_wait_enabled(true);
679 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700680
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700681 params.set_update_check_count_wait_enabled(true);
682 params.set_min_update_checks_needed(1);
683 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700684
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700685 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700686
687 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700688 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700689 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700690 -1,
691 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700692 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800693 metrics::CheckResult::kUpdateAvailable,
694 metrics::CheckReaction::kDeferring,
695 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700696 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700697 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700698
Ben Chan9abb7632014-08-07 00:10:53 -0700699 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700700 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700701 // count remains the same, as the decrementing happens in update_attempter
702 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700703 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700704 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700705
706 // Verify if we are interactive check we don't defer.
707 params.set_interactive(true);
708 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700709 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700710 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700711 -1,
712 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700713 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800714 metrics::CheckResult::kUpdateAvailable,
715 metrics::CheckReaction::kUpdating,
716 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700717 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700718 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700719 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700720}
Jay Srinivasan0a708742012-03-20 11:26:12 -0700721
Alex Deymo8e18f932015-03-27 16:16:59 -0700722TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
723 OmahaResponse response;
724 OmahaRequestParams params = request_params_;
725 fake_update_response_.include_cohorts = true;
726 fake_update_response_.cohort = "s/154454/8479665";
727 fake_update_response_.cohorthint = "please-put-me-on-beta";
728 fake_update_response_.cohortname = "stable";
729
730 ASSERT_TRUE(TestUpdateCheck(&params,
731 fake_update_response_.GetUpdateResponse(),
732 -1,
733 false, // ping_only
734 ErrorCode::kSuccess,
735 metrics::CheckResult::kUpdateAvailable,
736 metrics::CheckReaction::kUpdating,
737 metrics::DownloadErrorCode::kUnset,
738 &response,
739 nullptr));
740
741 string value;
742 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
743 EXPECT_EQ(fake_update_response_.cohort, value);
744
745 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
746 EXPECT_EQ(fake_update_response_.cohorthint, value);
747
748 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
749 EXPECT_EQ(fake_update_response_.cohortname, value);
750}
751
752TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
753 OmahaResponse response;
754 OmahaRequestParams params = request_params_;
755 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
756 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
757 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
758 fake_update_response_.include_cohorts = true;
759 fake_update_response_.cohort = "s/154454/8479665";
760 fake_update_response_.cohorthint = "please-put-me-on-beta";
761 fake_update_response_.cohortname = "";
762
763 ASSERT_TRUE(TestUpdateCheck(&params,
764 fake_update_response_.GetUpdateResponse(),
765 -1,
766 false, // ping_only
767 ErrorCode::kSuccess,
768 metrics::CheckResult::kUpdateAvailable,
769 metrics::CheckReaction::kUpdating,
770 metrics::DownloadErrorCode::kUnset,
771 &response,
772 nullptr));
773
774 string value;
775 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
776 EXPECT_EQ(fake_update_response_.cohort, value);
777
778 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
779 EXPECT_EQ(fake_update_response_.cohorthint, value);
780
781 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
782}
783
784TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
785 OmahaResponse response;
786 OmahaRequestParams params = request_params_;
787 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
788
789 ASSERT_TRUE(TestUpdateCheck(&params,
790 fake_update_response_.GetUpdateResponse(),
791 -1,
792 false, // ping_only
793 ErrorCode::kSuccess,
794 metrics::CheckResult::kUpdateAvailable,
795 metrics::CheckReaction::kUpdating,
796 metrics::DownloadErrorCode::kUnset,
797 &response,
798 nullptr));
799
800 string value;
801 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
802 EXPECT_EQ("old_value", value);
803
804 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
805 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
806}
807
Alex Deymo00d79ac2015-06-29 15:41:49 -0700808TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
809 OmahaResponse response;
810 OmahaRequestParams params = request_params_;
811 fake_update_response_.include_cohorts = true;
812 fake_update_response_.cohort = "s/154454/8479665";
813 fake_update_response_.cohorthint = "please-put-me-on-beta";
814 fake_update_response_.cohortname = "stable";
815
816 ASSERT_TRUE(TestUpdateCheck(&params,
817 fake_update_response_.GetNoUpdateResponse(),
818 -1,
819 false, // ping_only
820 ErrorCode::kSuccess,
821 metrics::CheckResult::kNoUpdateAvailable,
822 metrics::CheckReaction::kUnset,
823 metrics::DownloadErrorCode::kUnset,
824 &response,
825 nullptr));
826
827 string value;
828 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
829 EXPECT_EQ(fake_update_response_.cohort, value);
830
831 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
832 EXPECT_EQ(fake_update_response_.cohorthint, value);
833
834 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
835 EXPECT_EQ(fake_update_response_.cohortname, value);
836}
837
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700838TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -0700839 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700840
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700841 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700842 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700843
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700844 OmahaRequestParams params = request_params_;
845 fake_system_state_.set_request_params(&params);
846 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300847 brillo::make_unique_ptr(
848 new MockHttpFetcher(http_response.data(),
849 http_response.size(),
850 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -0700851 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700852 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700853 ActionProcessor processor;
854 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700855 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700856
Alex Deymo60ca1a72015-06-18 18:19:15 -0700857 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
858 loop.Run();
859 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700860 EXPECT_FALSE(processor.IsRunning());
861}
862
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700863TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700864 OmahaResponse response;
865 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700866 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700867 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700868 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700869 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700870 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800871 metrics::CheckResult::kParsingError,
872 metrics::CheckReaction::kUnset,
873 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -0700874 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700875 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -0700876 EXPECT_FALSE(response.update_exists);
877}
878
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700879TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -0700880 OmahaResponse response;
881 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700882 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -0700883 "",
884 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700885 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700886 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800887 metrics::CheckResult::kParsingError,
888 metrics::CheckReaction::kUnset,
889 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700890 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700891 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700892 EXPECT_FALSE(response.update_exists);
893}
894
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700895TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700896 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700897 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700898 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700899 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
900 "<daystart elapsed_seconds=\"100\"/>"
901 "<app appid=\"foo\" status=\"ok\">"
902 "<ping status=\"ok\"/>"
903 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700904 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700905 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700906 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800907 metrics::CheckResult::kParsingError,
908 metrics::CheckReaction::kUnset,
909 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700910 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700911 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700912 EXPECT_FALSE(response.update_exists);
913}
914
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700915TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700916 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700917 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700918 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700919 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
920 "<daystart elapsed_seconds=\"100\"/>"
921 "<app appid=\"foo\" status=\"ok\">"
922 "<ping status=\"ok\"/>"
923 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700924 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700925 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700926 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800927 metrics::CheckResult::kParsingError,
928 metrics::CheckReaction::kUnset,
929 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700930 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700931 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700932 EXPECT_FALSE(response.update_exists);
933}
934
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700935TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700936 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700937 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700938 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700939 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
940 "<daystart elapsed_seconds=\"100\"/>"
941 "<app appid=\"foo\" status=\"ok\">"
942 "<ping status=\"ok\"/>"
943 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700944 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700945 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700946 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800947 metrics::CheckResult::kParsingError,
948 metrics::CheckReaction::kUnset,
949 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700950 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700951 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700952 EXPECT_FALSE(response.update_exists);
953}
954
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700955TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700956 string input_response =
957 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
958 "<daystart elapsed_seconds=\"100\"/>"
959 "<app appid=\"xyz\" status=\"ok\">"
960 "<updatecheck status=\"ok\">"
961 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -0700962 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700963 "<packages><package hash=\"not-used\" name=\"f\" "
964 "size=\"587\"/></packages>"
965 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700966 "ChromeOSVersion=\"10.2.3.4\" "
967 "Prompt=\"false\" "
968 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -0800969 "IsDeltaPayload=\"false\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700970 "sha256=\"lkq34j5345\" "
971 "needsadmin=\"true\" "
972 "/></actions></manifest></updatecheck></app></response>";
973 LOG(INFO) << "Input Response = " << input_response;
974
Darin Petkov6a5b3222010-07-13 14:55:28 -0700975 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700976 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700977 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -0700978 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700979 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700980 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800981 metrics::CheckResult::kUpdateAvailable,
982 metrics::CheckReaction::kUpdating,
983 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700984 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700985 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700986 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -0700987 EXPECT_EQ("10.2.3.4", response.version);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800988 EXPECT_EQ("http://missing/field/test/f", response.payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700989 EXPECT_EQ("", response.more_info_url);
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700990 EXPECT_EQ("lkq34j5345", response.hash);
991 EXPECT_EQ(587, response.size);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700992 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -0700993 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700994}
995
996namespace {
997class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
998 public:
999 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001000 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001001 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001002};
1003
Alex Deymo60ca1a72015-06-18 18:19:15 -07001004void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001005 processor->StartProcessing();
1006 CHECK(processor->IsRunning());
1007 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001008}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001009} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001010
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001011TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001012 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001013 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001014
Alex Deymo60ca1a72015-06-18 18:19:15 -07001015 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001016 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001017 brillo::make_unique_ptr(
1018 new MockHttpFetcher(http_response.data(),
1019 http_response.size(),
1020 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001021 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001022 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001023 ActionProcessor processor;
1024 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001025 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001026
Alex Deymo60ca1a72015-06-18 18:19:15 -07001027 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1028 loop.Run();
1029 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001030}
1031
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001032TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001033 string output;
1034 EXPECT_TRUE(XmlEncode("ab", &output));
1035 EXPECT_EQ("ab", output);
1036 EXPECT_TRUE(XmlEncode("a<b", &output));
1037 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001038 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1039 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001040 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1041 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001042 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001043 EXPECT_FALSE(XmlEncode("\xc2", &output));
1044 // Fail with invalid ASCII-7 chars.
1045 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1046}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001047
Alex Deymob0d74eb2015-03-30 17:59:17 -07001048TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1049 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1050 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1051}
1052
1053TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001054 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001055
1056 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001057 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001058 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001059 OmahaRequestParams::kOsVersion,
1060 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001061 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001062 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001063 "0.1.0.0",
1064 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001065 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001066 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001067 "ChromeOSFirmware.1.0",
1068 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001069 false, // delta okay
1070 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001071 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001072 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001073 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1074 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1075 fake_prefs_.SetString(kPrefsOmahaCohortName,
1076 JoinString(vector<string>(100, "My spoon is too big."),
1077 ' '));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001078 OmahaResponse response;
1079 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001080 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001081 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001082 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001083 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001084 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001085 metrics::CheckResult::kParsingError,
1086 metrics::CheckReaction::kUnset,
1087 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001088 &response,
1089 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001090 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001091 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001092 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1093 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1094 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1095 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1096 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1097 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1098 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1099 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1100 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1101 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1102 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1103 // Values from Prefs that are too big are removed from the XML instead of
1104 // encoded.
1105 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001106}
1107
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001108TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001109 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001110 fake_update_response_.deadline = "&lt;20110101";
1111 fake_update_response_.more_info_url = "testthe&lt;url";
1112 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001113 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001114 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001115 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001116 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001117 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001118 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001119 metrics::CheckResult::kUpdateAvailable,
1120 metrics::CheckReaction::kUpdating,
1121 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001122 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001123 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001124
1125 EXPECT_EQ(response.more_info_url, "testthe<url");
Jay Srinivasan6f6ea002012-12-14 11:26:28 -08001126 EXPECT_EQ(response.payload_urls[0], "testthe&codebase/file.signed");
Darin Petkov6c118642010-10-21 12:06:30 -07001127 EXPECT_EQ(response.deadline, "<20110101");
Darin Petkov6a5b3222010-07-13 14:55:28 -07001128}
1129
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001130TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001131 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001132 // overflows int32_t:
1133 fake_update_response_.size = 123123123123123ll;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001134 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001135 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001136 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001137 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001138 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001139 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001140 metrics::CheckResult::kUpdateAvailable,
1141 metrics::CheckReaction::kUpdating,
1142 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001143 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001144 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001145
1146 EXPECT_EQ(response.size, 123123123123123ll);
1147}
1148
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001149TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001150 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001151 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001152 fake_system_state_.set_prefs(&prefs);
1153
Darin Petkov95508da2011-01-05 12:42:29 -08001154 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1155 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001156 // An existing but empty previous version means that we didn't reboot to a new
1157 // update, therefore, no need to update the previous version.
1158 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001159 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001160 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001161 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001162 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001163 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001164 metrics::CheckResult::kParsingError,
1165 metrics::CheckReaction::kUnset,
1166 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001167 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001168 &post_data));
1169 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001170 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001171 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001172 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001173 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001174 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001175 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1176 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001177 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1178 string::npos);
1179 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1180 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001181 // No <event> tag should be sent if we didn't reboot to an update.
1182 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001183}
1184
Jay Srinivasan0a708742012-03-20 11:26:12 -07001185
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001186TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001187 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001188 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001189 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1190 "invalid xml>",
1191 &post_data);
1192 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001193 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001194 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001195 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001196 OmahaEvent::kTypeUpdateDownloadStarted,
1197 OmahaEvent::kResultSuccess);
1198 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001199 EXPECT_EQ(post_str.find("ping"), string::npos);
1200 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001201}
1202
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001203TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001204 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001205 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001206 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1207 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001208 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001209 "invalid xml>",
1210 &post_data);
1211 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001212 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001213 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001214 " <event eventtype=\"%d\" eventresult=\"%d\" "
1215 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001216 OmahaEvent::kTypeDownloadComplete,
1217 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001218 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001219 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001220 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001221}
1222
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001223TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001224 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001225 // Create a copy of the OmahaRequestParams to reuse it later.
1226 OmahaRequestParams params = request_params_;
1227 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001228 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001229 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001230 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001231 brillo::make_unique_ptr(
1232 new MockHttpFetcher(http_response.data(),
1233 http_response.size(),
1234 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001235 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001236 EXPECT_FALSE(update_check_action.IsEvent());
1237
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001238 params = request_params_;
1239 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001240 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001241 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001242 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001243 brillo::make_unique_ptr(
1244 new MockHttpFetcher(http_response.data(),
1245 http_response.size(),
1246 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001247 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001248 EXPECT_TRUE(event_action.IsEvent());
1249}
1250
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001251TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001252 for (int i = 0; i < 2; i++) {
1253 bool delta_okay = i == 1;
1254 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001255 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001256 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001257 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001258 OmahaRequestParams::kOsVersion,
1259 "service_pack",
1260 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001261 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001262 "0.1.0.0",
1263 "en-US",
1264 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001265 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001266 "ChromeOSFirmware.1.0",
1267 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001268 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001269 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001270 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001271 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001272 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001273 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001274 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001275 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001276 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001277 metrics::CheckResult::kParsingError,
1278 metrics::CheckReaction::kUnset,
1279 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001280 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001281 &post_data));
1282 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001283 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001284 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1285 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001286 string::npos)
1287 << "i = " << i;
1288 }
1289}
1290
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001291TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001292 for (int i = 0; i < 2; i++) {
1293 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001294 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001295 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001296 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001297 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001298 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001299 OmahaRequestParams::kOsVersion,
1300 "service_pack",
1301 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001302 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001303 "0.1.0.0",
1304 "en-US",
1305 "unittest_track",
1306 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001307 "ChromeOSFirmware.1.0",
1308 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001309 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001310 interactive,
1311 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001312 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001313 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001314 "invalid xml>",
1315 -1,
1316 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001317 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001318 metrics::CheckResult::kParsingError,
1319 metrics::CheckReaction::kUnset,
1320 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001321 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001322 &post_data));
1323 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001324 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001325 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1326 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001327 string::npos)
1328 << "i = " << i;
1329 }
1330}
1331
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001332TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001333 OmahaEvent default_event;
1334 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1335 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001336 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001337
1338 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1339 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1340 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001341 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001342
1343 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1344 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001345 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001346 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1347 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001348 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001349}
1350
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001351void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001352 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001353 fake_system_state_.set_prefs(&prefs);
1354 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1355 .Times(AnyNumber());
1356 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1357 // Add a few hours to the day difference to test no rounding, etc.
1358 int64_t five_days_ago =
1359 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1360 int64_t six_days_ago =
1361 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1362 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1363 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1364 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1365 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1366 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1367 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001368 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001369 ASSERT_TRUE(
1370 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001371 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001372 -1,
1373 ping_only,
1374 ErrorCode::kSuccess,
1375 metrics::CheckResult::kUnset,
1376 metrics::CheckReaction::kUnset,
1377 metrics::DownloadErrorCode::kUnset,
1378 nullptr,
1379 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001380 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001381 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1382 string::npos);
1383 if (ping_only) {
1384 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1385 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1386 } else {
1387 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1388 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001389 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001390}
1391
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001392TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1393 PingTest(true /* ping_only */);
1394}
1395
1396TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1397 PingTest(false /* ping_only */);
1398}
1399
1400TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001401 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001402 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001403 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1404 .Times(AnyNumber());
1405 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001406 int64_t three_days_ago =
1407 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1408 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001409 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1410 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001411 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1412 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1413 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1414 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001415 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001416 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001417 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001418 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001419 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001420 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001421 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001422 metrics::CheckResult::kNoUpdateAvailable,
1423 metrics::CheckReaction::kUnset,
1424 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001425 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001426 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001427 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001428 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001429 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001430}
1431
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001432TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001433 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001434 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001435 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1436 .Times(AnyNumber());
1437 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001438 int64_t four_days_ago =
1439 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1440 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001441 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1442 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001443 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1444 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1445 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1446 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001447 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001448 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001449 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001450 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001451 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001452 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001453 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001454 metrics::CheckResult::kNoUpdateAvailable,
1455 metrics::CheckReaction::kUnset,
1456 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001457 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001458 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001459 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001460 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001461 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001462}
1463
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001464TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001465 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001466 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001467 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1468 .Times(AnyNumber());
1469 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001470 int64_t one_hour_ago =
1471 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001472 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1473 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001474 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1475 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1476 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1477 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001478 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1479 // send a ping.
1480 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1481 .WillOnce(Return(true));
1482 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1483 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001484 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001485 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001486 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001487 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001488 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001489 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001490 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001491 metrics::CheckResult::kNoUpdateAvailable,
1492 metrics::CheckReaction::kUnset,
1493 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001494 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001495 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001496 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001497 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001498}
1499
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001500TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001501 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001502 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001503 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001504 int64_t now = Time::Now().ToInternalValue();
1505 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1506 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1507 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1508 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1509 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1510 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001511 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001512 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001513 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001514 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001515 -1,
1516 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001517 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001518 metrics::CheckResult::kUnset,
1519 metrics::CheckReaction::kUnset,
1520 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001521 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001522 &post_data));
1523 EXPECT_EQ(post_data.size(), 0);
1524}
1525
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001526TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001527 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001528 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001529 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1530 .Times(AnyNumber());
1531 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001532 int64_t future =
1533 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001534 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1535 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001536 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1537 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1538 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1539 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1540 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1541 .WillOnce(Return(true));
1542 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1543 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001544 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001545 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001546 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001547 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1548 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001549 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001550 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001551 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001552 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001553 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001554 metrics::CheckResult::kNoUpdateAvailable,
1555 metrics::CheckReaction::kUnset,
1556 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001557 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001558 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001559 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001560 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001561}
1562
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001563TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001564 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001565 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001566 // may fail if it runs for longer than 5 seconds. It shouldn't run
1567 // that long though.
1568 int64_t midnight =
1569 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1570 int64_t midnight_slack =
1571 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001572 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001573 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001574 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1575 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001576 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1577 AllOf(Ge(midnight), Le(midnight_slack))))
1578 .WillOnce(Return(true));
1579 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1580 AllOf(Ge(midnight), Le(midnight_slack))))
1581 .WillOnce(Return(true));
1582 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001583 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001584 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1585 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001586 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001587 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001588 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001589 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001590 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001591 metrics::CheckResult::kNoUpdateAvailable,
1592 metrics::CheckReaction::kUnset,
1593 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001594 nullptr,
1595 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001596}
1597
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001598TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001599 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001600 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001601 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1602 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001603 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1604 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1605 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001606 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001607 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1608 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001609 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001610 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001611 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001612 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001613 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001614 metrics::CheckResult::kNoUpdateAvailable,
1615 metrics::CheckReaction::kUnset,
1616 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001617 nullptr,
1618 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001619}
1620
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001621TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001622 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001623 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001624 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1625 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001626 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1627 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1628 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001629 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001630 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1631 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001632 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001633 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001634 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001635 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001636 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001637 metrics::CheckResult::kNoUpdateAvailable,
1638 metrics::CheckReaction::kUnset,
1639 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001640 nullptr,
1641 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001642}
1643
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001644TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001645 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001646 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07001647 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001648 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001649 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001650 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001651 metrics::CheckResult::kParsingError,
1652 metrics::CheckReaction::kUnset,
1653 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001654 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07001655 &post_data));
1656 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001657 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07001658 EXPECT_EQ(post_str.find("machineid="), string::npos);
1659 EXPECT_EQ(post_str.find("userid="), string::npos);
1660}
1661
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001662TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001663 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001664 const int http_error_code =
1665 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07001666 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001667 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001668 "",
1669 501,
Darin Petkov265f2902011-05-09 15:17:40 -07001670 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001671 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001672 metrics::CheckResult::kDownloadError,
1673 metrics::CheckReaction::kUnset,
1674 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07001675 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001676 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001677 EXPECT_FALSE(response.update_exists);
1678}
1679
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001680TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001681 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001682 const int http_error_code =
1683 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07001684 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001685 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001686 "",
1687 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07001688 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001689 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001690 metrics::CheckResult::kDownloadError,
1691 metrics::CheckReaction::kUnset,
1692 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07001693 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001694 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001695 EXPECT_FALSE(response.update_exists);
1696}
1697
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001698TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001699 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001700 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001701 params.set_wall_clock_based_wait_enabled(true);
1702 params.set_waiting_period(TimeDelta().FromDays(1));
1703 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001704
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001705 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001706 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001707 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001708 -1,
1709 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001710 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001711 metrics::CheckResult::kUpdateAvailable,
1712 metrics::CheckReaction::kDeferring,
1713 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001714 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001715 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001716
Ben Chan9abb7632014-08-07 00:10:53 -07001717 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001718 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001719 ASSERT_GT(timestamp, 0);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001720 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001721
1722 // Verify if we are interactive check we don't defer.
1723 params.set_interactive(true);
1724 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001725 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001726 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001727 -1,
1728 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001729 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001730 metrics::CheckResult::kUpdateAvailable,
1731 metrics::CheckReaction::kUpdating,
1732 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001733 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001734 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001735 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001736}
1737
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001738TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001739 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001740 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001741 params.set_wall_clock_based_wait_enabled(true);
1742 params.set_waiting_period(TimeDelta().FromDays(1));
1743 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001744
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001745 // Set the timestamp to a very old value such that it exceeds the
1746 // waiting period set above.
1747 Time t1;
1748 Time::FromString("1/1/2012", &t1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001749 ASSERT_TRUE(fake_prefs_.SetInt64(
1750 kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001751 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001752 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001753 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001754 -1,
1755 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001756 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001757 metrics::CheckResult::kUpdateAvailable,
1758 metrics::CheckReaction::kUpdating,
1759 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001760 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001761 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001762
1763 EXPECT_TRUE(response.update_exists);
1764
1765 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07001766 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001767 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001768 ASSERT_TRUE(timestamp == t1.ToInternalValue());
1769}
1770
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001771TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001772 // Create a uniquely named test directory.
1773 string test_dir;
1774 ASSERT_TRUE(utils::MakeTempDirectory(
1775 "omaha_request_action-test-XXXXXX", &test_dir));
1776
1777 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1778 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001779 kStatefulPartition + "/etc"));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001780 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001781 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001782 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001783 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001784 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001785 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1786 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1787 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
1788 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001789 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001790 "CHROMEOS_IS_POWERWASH_ALLOWED=true\n"
1791 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001792 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001793 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001794 params.Init("1.2.3.4", "", 0);
1795 EXPECT_EQ("canary-channel", params.current_channel());
1796 EXPECT_EQ("stable-channel", params.target_channel());
1797 EXPECT_TRUE(params.to_more_stable_channel());
1798 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001799 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001800 "invalid xml>",
1801 -1,
1802 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001803 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001804 metrics::CheckResult::kParsingError,
1805 metrics::CheckReaction::kUnset,
1806 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001807 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001808 &post_data));
1809 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001810 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001811 EXPECT_NE(string::npos, post_str.find(
1812 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
1813 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
1814 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001815
Alex Deymo110e0302015-10-19 20:35:21 -07001816 ASSERT_TRUE(base::DeleteFile(base::FilePath(test_dir), true));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001817}
1818
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001819TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001820 // Create a uniquely named test directory.
1821 string test_dir;
1822 ASSERT_TRUE(utils::MakeTempDirectory(
1823 "omaha_request_action-test-XXXXXX", &test_dir));
1824
1825 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1826 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001827 kStatefulPartition + "/etc"));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001828 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001829 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001830 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001831 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001832 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001833 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1834 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1835 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
1836 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001837 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001838 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001839 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001840 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001841 params.Init("5.6.7.8", "", 0);
1842 EXPECT_EQ("stable-channel", params.current_channel());
1843 EXPECT_EQ("canary-channel", params.target_channel());
1844 EXPECT_FALSE(params.to_more_stable_channel());
1845 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001846 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001847 "invalid xml>",
1848 -1,
1849 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001850 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001851 metrics::CheckResult::kParsingError,
1852 metrics::CheckReaction::kUnset,
1853 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001854 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001855 &post_data));
1856 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001857 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001858 EXPECT_NE(string::npos, post_str.find(
1859 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
1860 "version=\"5.6.7.8\" "
1861 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001862 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001863}
1864
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001865// Checks that the initial ping with a=-1 r=-1 is not send when the device
1866// was powerwashed.
1867TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
1868 fake_prefs_.SetString(kPrefsPreviousVersion, "");
1869
1870 // Flag that the device was powerwashed in the past.
1871 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
1872
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001873 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001874 ASSERT_TRUE(
1875 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001876 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001877 -1,
1878 false, // ping_only
1879 ErrorCode::kSuccess,
1880 metrics::CheckResult::kNoUpdateAvailable,
1881 metrics::CheckReaction::kUnset,
1882 metrics::DownloadErrorCode::kUnset,
1883 nullptr,
1884 &post_data));
1885 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001886 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001887 EXPECT_EQ(string::npos, post_str.find("<ping"));
1888}
1889
Alex Deymo9fded1e2015-11-05 12:31:19 -08001890// Checks that the event 54 is sent on a reboot to a new update.
1891TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
1892 // Flag that the device was updated in a previous boot.
1893 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
1894
1895 brillo::Blob post_data;
1896 ASSERT_TRUE(
1897 TestUpdateCheck(nullptr, // request_params
1898 fake_update_response_.GetNoUpdateResponse(),
1899 -1,
1900 false, // ping_only
1901 ErrorCode::kSuccess,
1902 metrics::CheckResult::kNoUpdateAvailable,
1903 metrics::CheckReaction::kUnset,
1904 metrics::DownloadErrorCode::kUnset,
1905 nullptr,
1906 &post_data));
1907 string post_str(post_data.begin(), post_data.end());
1908
1909 // An event 54 is included and has the right version.
1910 EXPECT_NE(string::npos,
1911 post_str.find(base::StringPrintf(
1912 "<event eventtype=\"%d\"",
1913 OmahaEvent::kTypeRebootedAfterUpdate)));
1914 EXPECT_NE(string::npos,
1915 post_str.find("previousversion=\"1.2.3.4\"></event>"));
1916
1917 // The previous version flag should have been removed.
1918 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
1919 string prev_version;
1920 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
1921 EXPECT_TRUE(prev_version.empty());
1922}
1923
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001924void OmahaRequestActionTest::P2PTest(
1925 bool initial_allow_p2p_for_downloading,
1926 bool initial_allow_p2p_for_sharing,
1927 bool omaha_disable_p2p_for_downloading,
1928 bool omaha_disable_p2p_for_sharing,
1929 bool payload_state_allow_p2p_attempt,
1930 bool expect_p2p_client_lookup,
1931 const string& p2p_client_result_url,
1932 bool expected_allow_p2p_for_downloading,
1933 bool expected_allow_p2p_for_sharing,
1934 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07001935 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001936 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07001937 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
1938 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
1939 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07001940
1941 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001942 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07001943 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
1944 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07001945 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
1946 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
1947 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
1948 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
1949 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
1950 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
1951 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
1952 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
1953 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
1954 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
1955
David Zeuthen8f191b22013-08-06 12:27:50 -07001956 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001957 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07001958 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
1959
David Zeuthen4cc5ed22014-01-15 12:35:03 -08001960 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
1961 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07001962 .Times(expect_p2p_client_lookup ? 1 : 0);
1963
Alex Deymo8e18f932015-03-27 16:16:59 -07001964 fake_update_response_.disable_p2p_for_downloading =
1965 omaha_disable_p2p_for_downloading;
1966 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07001967 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001968 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001969 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07001970 -1,
1971 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001972 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001973 metrics::CheckResult::kUpdateAvailable,
1974 metrics::CheckReaction::kUpdating,
1975 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07001976 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001977 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07001978 EXPECT_TRUE(response.update_exists);
1979
Gilad Arnold74b5f552014-10-07 08:17:16 -07001980 EXPECT_EQ(omaha_disable_p2p_for_downloading,
1981 response.disable_p2p_for_downloading);
1982 EXPECT_EQ(omaha_disable_p2p_for_sharing,
1983 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07001984
Gilad Arnold74b5f552014-10-07 08:17:16 -07001985 EXPECT_EQ(expected_allow_p2p_for_downloading,
1986 actual_allow_p2p_for_downloading);
1987 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
1988 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07001989}
1990
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001991TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001992 P2PTest(true, // initial_allow_p2p_for_downloading
1993 true, // initial_allow_p2p_for_sharing
1994 false, // omaha_disable_p2p_for_downloading
1995 false, // omaha_disable_p2p_for_sharing
1996 true, // payload_state_allow_p2p_attempt
1997 true, // expect_p2p_client_lookup
1998 "http://1.3.5.7/p2p", // p2p_client_result_url
1999 true, // expected_allow_p2p_for_downloading
2000 true, // expected_allow_p2p_for_sharing
2001 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002002}
2003
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002004TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002005 P2PTest(true, // initial_allow_p2p_for_downloading
2006 true, // initial_allow_p2p_for_sharing
2007 false, // omaha_disable_p2p_for_downloading
2008 false, // omaha_disable_p2p_for_sharing
2009 true, // payload_state_allow_p2p_attempt
2010 true, // expect_p2p_client_lookup
2011 "", // p2p_client_result_url
2012 false, // expected_allow_p2p_for_downloading
2013 true, // expected_allow_p2p_for_sharing
2014 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002015}
2016
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002017TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002018 P2PTest(false, // initial_allow_p2p_for_downloading
2019 true, // initial_allow_p2p_for_sharing
2020 false, // omaha_disable_p2p_for_downloading
2021 false, // omaha_disable_p2p_for_sharing
2022 true, // payload_state_allow_p2p_attempt
2023 false, // expect_p2p_client_lookup
2024 "unset", // p2p_client_result_url
2025 false, // expected_allow_p2p_for_downloading
2026 true, // expected_allow_p2p_for_sharing
2027 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002028}
2029
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002030TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002031 P2PTest(true, // initial_allow_p2p_for_downloading
2032 true, // initial_allow_p2p_for_sharing
2033 true, // omaha_disable_p2p_for_downloading
2034 false, // omaha_disable_p2p_for_sharing
2035 true, // payload_state_allow_p2p_attempt
2036 false, // expect_p2p_client_lookup
2037 "unset", // p2p_client_result_url
2038 false, // expected_allow_p2p_for_downloading
2039 true, // expected_allow_p2p_for_sharing
2040 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002041}
2042
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002043TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002044 P2PTest(true, // initial_allow_p2p_for_downloading
2045 true, // initial_allow_p2p_for_sharing
2046 false, // omaha_disable_p2p_for_downloading
2047 true, // omaha_disable_p2p_for_sharing
2048 true, // payload_state_allow_p2p_attempt
2049 true, // expect_p2p_client_lookup
2050 "http://1.3.5.7/p2p", // p2p_client_result_url
2051 true, // expected_allow_p2p_for_downloading
2052 false, // expected_allow_p2p_for_sharing
2053 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002054}
2055
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002056TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002057 P2PTest(true, // initial_allow_p2p_for_downloading
2058 true, // initial_allow_p2p_for_sharing
2059 true, // omaha_disable_p2p_for_downloading
2060 true, // omaha_disable_p2p_for_sharing
2061 true, // payload_state_allow_p2p_attempt
2062 false, // expect_p2p_client_lookup
2063 "unset", // p2p_client_result_url
2064 false, // expected_allow_p2p_for_downloading
2065 false, // expected_allow_p2p_for_sharing
2066 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002067}
2068
Alex Deymof329b932014-10-30 01:37:48 -07002069bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2070 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002071 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002072 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002073 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002074 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002075 -1,
2076 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002077 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002078 metrics::CheckResult::kUpdateAvailable,
2079 metrics::CheckReaction::kUpdating,
2080 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002081 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002082 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002083}
2084
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002085TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002086 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002087
2088 // Check that we parse elapsed_days in the Omaha Response correctly.
2089 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002090 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2091 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002092 EXPECT_TRUE(response.update_exists);
2093 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002094 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002095 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002096 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002097 EXPECT_EQ(prefs_days, 42);
2098
2099 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002100 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002101 EXPECT_TRUE(response.update_exists);
2102 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002103 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002104 EXPECT_EQ(prefs_days, 42);
2105
2106 // Note that elapsed_days is not necessarily divisible by 7 so check
2107 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002108 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2109 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002110 EXPECT_TRUE(response.update_exists);
2111 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002112 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002113 EXPECT_EQ(prefs_days, 21);
2114
2115 // Check that we correctly handle elapsed_days not being included in
2116 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002117 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002118 EXPECT_TRUE(response.update_exists);
2119 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002120}
2121
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002122// If there is no prefs and OOBE is not complete, we should not
2123// report anything to Omaha.
2124TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
2125 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2126 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2127}
David Zeuthen639aa362014-02-03 16:23:44 -08002128
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002129// If OOBE is complete and happened on a valid date (e.g. after Jan
2130// 1 2007 0:00 PST), that date should be used and written to
2131// prefs. However, first try with an invalid date and check we do
2132// nothing.
2133TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2134 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2135 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2136 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2137 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2138}
David Zeuthen639aa362014-02-03 16:23:44 -08002139
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002140// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2141// should yield an InstallDate of 14.
2142TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2143 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2144 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2145 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2146 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002147
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002148 int64_t prefs_days;
2149 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2150 EXPECT_EQ(prefs_days, 14);
2151}
David Zeuthen639aa362014-02-03 16:23:44 -08002152
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002153// Now that we have a valid date in prefs, check that we keep using
2154// that even if OOBE date reports something else. The date Jan 30,
2155// 2007 0:00 PST should yield an InstallDate of 28... but since
2156// there's a prefs file, we should still get 14.
2157TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2158 // Set a valid date in the prefs first.
2159 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002160
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002161 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2162 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2163 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002164
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002165 int64_t prefs_days;
2166 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2167 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002168
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002169 // If we delete the prefs file, we should get 28 days.
2170 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2171 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2172 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2173 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002174}
2175
Darin Petkov6a5b3222010-07-13 14:55:28 -07002176} // namespace chromeos_update_engine