blob: 82f9ae86f564332c49e246f7fcc2be5e4e4286e7 [file] [log] [blame]
Alex Deymoaea4c1c2015-08-19 20:24:43 -07001//
2// Copyright (C) 2012 The Android Open Source Project
3//
4// Licensed under the Apache License, Version 2.0 (the "License");
5// you may not use this file except in compliance with the License.
6// You may obtain a copy of the License at
7//
8// http://www.apache.org/licenses/LICENSE-2.0
9//
10// Unless required by applicable law or agreed to in writing, software
11// distributed under the License is distributed on an "AS IS" BASIS,
12// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13// See the License for the specific language governing permissions and
14// limitations under the License.
15//
Darin Petkov6a5b3222010-07-13 14:55:28 -070016
Alex Deymo8427b4a2014-11-05 14:00:32 -080017#include "update_engine/omaha_request_action.h"
18
Ben Chan9abb7632014-08-07 00:10:53 -070019#include <stdint.h>
20
Darin Petkov6a5b3222010-07-13 14:55:28 -070021#include <string>
22#include <vector>
Darin Petkov0dc8e9a2010-07-14 14:51:57 -070023
Alex Deymo60ca1a72015-06-18 18:19:15 -070024#include <base/bind.h>
Alex Deymo110e0302015-10-19 20:35:21 -070025#include <base/files/file_util.h>
Sen Jiang297e5832016-03-17 14:45:51 -070026#include <base/files/scoped_temp_dir.h>
Alex Deymo8e18f932015-03-27 16:16:59 -070027#include <base/strings/string_number_conversions.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070028#include <base/strings/string_util.h>
29#include <base/strings/stringprintf.h>
30#include <base/time/time.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070031#include <brillo/bind_lambda.h>
Alex Deymoc1c17b42015-11-23 03:53:15 -030032#include <brillo/make_unique_ptr.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070033#include <brillo/message_loops/fake_message_loop.h>
34#include <brillo/message_loops/message_loop.h>
35#include <brillo/message_loops/message_loop_utils.h>
Alex Deymoe1e3afe2014-10-30 13:02:49 -070036#include <gtest/gtest.h>
Jay Srinivasand29695d2013-04-08 15:08:05 -070037
Alex Deymo39910dc2015-11-09 17:04:30 -080038#include "update_engine/common/action_pipe.h"
39#include "update_engine/common/constants.h"
40#include "update_engine/common/fake_prefs.h"
41#include "update_engine/common/hash_calculator.h"
42#include "update_engine/common/mock_http_fetcher.h"
43#include "update_engine/common/platform_constants.h"
44#include "update_engine/common/prefs.h"
45#include "update_engine/common/test_utils.h"
Alex Deymoc1c17b42015-11-23 03:53:15 -030046#include "update_engine/fake_system_state.h"
Alex Deymo38429cf2015-11-11 18:27:22 -080047#include "update_engine/metrics.h"
Chris Sosa77f79e82014-06-02 18:16:24 -070048#include "update_engine/mock_connection_manager.h"
Gilad Arnold74b5f552014-10-07 08:17:16 -070049#include "update_engine/mock_payload_state.h"
Darin Petkova4a8a8c2010-07-15 22:21:12 -070050#include "update_engine/omaha_request_params.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070051
Darin Petkov1cbd78f2010-07-29 12:38:34 -070052using base::Time;
53using base::TimeDelta;
Darin Petkov6a5b3222010-07-13 14:55:28 -070054using std::string;
55using std::vector;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070056using testing::AllOf;
Alex Deymof329b932014-10-30 01:37:48 -070057using testing::AnyNumber;
Jay Srinivasan34b5d862012-07-23 11:43:22 -070058using testing::DoAll;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070059using testing::Ge;
60using testing::Le;
Darin Petkov9c096d62010-11-17 14:49:04 -080061using testing::NiceMock;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070062using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070063using testing::ReturnPointee;
64using testing::SaveArg;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070065using testing::SetArgumentPointee;
Alex Deymof329b932014-10-30 01:37:48 -070066using testing::_;
Darin Petkov6a5b3222010-07-13 14:55:28 -070067
Alex Deymo8e18f932015-03-27 16:16:59 -070068namespace {
69
Alex Deymo85616652015-10-15 18:48:31 -070070const char kTestAppId[] = "test-app-id";
71
Alex Deymo8e18f932015-03-27 16:16:59 -070072// This is a helper struct to allow unit tests build an update response with the
73// values they care about.
74struct FakeUpdateResponse {
75 string GetNoUpdateResponse() const {
76 string entity_str;
77 if (include_entity)
78 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Alex Deymo00d79ac2015-06-29 15:41:49 -070079 return
80 "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" +
81 entity_str + "<response protocol=\"3.0\">"
Alex Deymo8e18f932015-03-27 16:16:59 -070082 "<daystart elapsed_seconds=\"100\"/>"
Alex Deymo00d79ac2015-06-29 15:41:49 -070083 "<app appid=\"" + app_id + "\" " +
84 (include_cohorts ? "cohort=\"" + cohort + "\" cohorthint=\"" +
85 cohorthint + "\" cohortname=\"" + cohortname + "\" " : "") +
86 " status=\"ok\">"
87 "<ping status=\"ok\"/>"
88 "<updatecheck status=\"noupdate\"/></app></response>";
Alex Deymo8e18f932015-03-27 16:16:59 -070089 }
90
91 string GetUpdateResponse() const {
92 return
93 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
94 "protocol=\"3.0\">"
95 "<daystart elapsed_seconds=\"100\"" +
96 (elapsed_days.empty() ? "" : (" elapsed_days=\"" + elapsed_days + "\""))
97 + "/>"
98 "<app appid=\"" + app_id + "\" " +
99 (include_cohorts ? "cohort=\"" + cohort + "\" cohorthint=\"" +
100 cohorthint + "\" cohortname=\"" + cohortname + "\" " : "") +
101 " status=\"ok\">"
102 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
103 "<urls><url codebase=\"" + codebase + "\"/></urls>"
104 "<manifest version=\"" + version + "\">"
105 "<packages><package hash=\"not-used\" name=\"" + filename + "\" "
106 "size=\"" + base::Int64ToString(size) + "\"/></packages>"
107 "<actions><action event=\"postinstall\" "
108 "ChromeOSVersion=\"" + version + "\" "
109 "MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt + "\" "
110 "IsDelta=\"true\" "
111 "IsDeltaPayload=\"true\" "
112 "MaxDaysToScatter=\"" + max_days_to_scatter + "\" "
113 "sha256=\"" + hash + "\" "
114 "needsadmin=\"" + needsadmin + "\" " +
115 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
116 (disable_p2p_for_downloading ?
117 "DisableP2PForDownloading=\"true\" " : "") +
118 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
119 "/></actions></manifest></updatecheck></app></response>";
120 }
121
122 // Return the payload URL, which is split in two fields in the XML response.
123 string GetPayloadUrl() {
124 return codebase + filename;
125 }
126
Alex Deymo85616652015-10-15 18:48:31 -0700127 string app_id = kTestAppId;
Alex Deymo8e18f932015-03-27 16:16:59 -0700128 string version = "1.2.3.4";
129 string more_info_url = "http://more/info";
130 string prompt = "true";
131 string codebase = "http://code/base/";
132 string filename = "file.signed";
133 string hash = "HASH1234=";
134 string needsadmin = "false";
135 int64_t size = 123;
136 string deadline = "";
137 string max_days_to_scatter = "7";
138 string elapsed_days = "42";
139
140 // P2P setting defaults to allowed.
141 bool disable_p2p_for_downloading = false;
142 bool disable_p2p_for_sharing = false;
143
144 // Omaha cohorts settings.
145 bool include_cohorts = false;
146 string cohort = "";
147 string cohorthint = "";
148 string cohortname = "";
149
150 // Whether to include the CrOS <!ENTITY> in the XML response.
151 bool include_entity = false;
152};
153
154} // namespace
155
Darin Petkov6a5b3222010-07-13 14:55:28 -0700156namespace chromeos_update_engine {
157
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700158class OmahaRequestActionTest : public ::testing::Test {
159 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800160 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700161 fake_system_state_.set_request_params(&request_params_);
162 fake_system_state_.set_prefs(&fake_prefs_);
163 }
164
165 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800166 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700167 // is used. |payload_state| may be null, in which case a local mock is used.
168 // |p2p_manager| may be null, in which case a local mock is used.
169 // |connection_manager| may be null, in which case a local mock is used.
170 // out_response may be null. If |fail_http_response_code| is non-negative,
171 // the transfer will fail with that code. |ping_only| is passed through to the
172 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
173 // post-data received by the mock HttpFetcher is returned.
174 //
175 // The |expected_check_result|, |expected_check_reaction| and
176 // |expected_error_code| parameters are for checking expectations
177 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
178 // UMA statistics. Use the appropriate ::kUnset value to specify that
179 // the given metric should not be reported.
180 bool TestUpdateCheck(OmahaRequestParams* request_params,
181 const string& http_response,
182 int fail_http_response_code,
183 bool ping_only,
184 ErrorCode expected_code,
185 metrics::CheckResult expected_check_result,
186 metrics::CheckReaction expected_check_reaction,
187 metrics::DownloadErrorCode expected_download_error_code,
188 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700189 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700190
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800191 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700192 // only a ping or also an updatecheck.
193 void PingTest(bool ping_only);
194
195 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700196 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700197 OmahaResponse *response);
198
199 // P2P test helper function.
200 void P2PTest(
201 bool initial_allow_p2p_for_downloading,
202 bool initial_allow_p2p_for_sharing,
203 bool omaha_disable_p2p_for_downloading,
204 bool omaha_disable_p2p_for_sharing,
205 bool payload_state_allow_p2p_attempt,
206 bool expect_p2p_client_lookup,
207 const string& p2p_client_result_url,
208 bool expected_allow_p2p_for_downloading,
209 bool expected_allow_p2p_for_sharing,
210 const string& expected_p2p_url);
211
212 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700213 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700214
215 // By default, all tests use these objects unless they replace them in the
216 // fake_system_state_.
217 OmahaRequestParams request_params_ = OmahaRequestParams{
218 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700219 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700220 OmahaRequestParams::kOsVersion,
221 "service_pack",
222 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700223 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700224 "0.1.0.0",
225 "en-US",
226 "unittest",
227 "OEM MODEL 09235 7471",
228 "ChromeOSFirmware.1.0",
229 "0X0A1",
230 false, // delta okay
231 false, // interactive
232 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700233 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700234
235 FakePrefs fake_prefs_;
236};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700237
238namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700239class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
240 public:
241 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700242 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800243 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700244 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800245 void ProcessingDone(const ActionProcessor* processor,
246 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700247 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700248 }
249
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800250 void ActionCompleted(ActionProcessor* processor,
251 AbstractAction* action,
252 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700253 // make sure actions always succeed
254 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700255 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700256 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700257 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700258 }
David Zeuthena99981f2013-04-29 13:42:47 -0700259 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700260};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700261} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700262
263class OutputObjectCollectorAction;
264
265template<>
266class ActionTraits<OutputObjectCollectorAction> {
267 public:
268 // Does not take an object for input
269 typedef OmahaResponse InputObjectType;
270 // On success, puts the output path on output
271 typedef NoneType OutputObjectType;
272};
273
274class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
275 public:
276 OutputObjectCollectorAction() : has_input_object_(false) {}
277 void PerformAction() {
278 // copy input object
279 has_input_object_ = HasInputObject();
280 if (has_input_object_)
281 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700282 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700283 }
284 // Should never be called
285 void TerminateProcessing() {
286 CHECK(false);
287 }
288 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700289 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700290 return "OutputObjectCollectorAction";
291 }
Alex Deymof329b932014-10-30 01:37:48 -0700292 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800293 using InputObjectType =
294 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
295 using OutputObjectType =
296 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700297 bool has_input_object_;
298 OmahaResponse omaha_response_;
299};
300
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700301bool OmahaRequestActionTest::TestUpdateCheck(
302 OmahaRequestParams* request_params,
303 const string& http_response,
304 int fail_http_response_code,
305 bool ping_only,
306 ErrorCode expected_code,
307 metrics::CheckResult expected_check_result,
308 metrics::CheckReaction expected_check_reaction,
309 metrics::DownloadErrorCode expected_download_error_code,
310 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700311 brillo::Blob* out_post_data) {
312 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700313 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700314 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800315 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700316 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700317 if (fail_http_response_code >= 0) {
318 fetcher->FailTransfer(fail_http_response_code);
319 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700320 if (request_params)
321 fake_system_state_.set_request_params(request_params);
322 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700323 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300324 brillo::make_unique_ptr(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700325 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700326 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700327 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700328
Darin Petkov6a5b3222010-07-13 14:55:28 -0700329 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700330 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700331 processor.EnqueueAction(&action);
332
333 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700334 BondActions(&action, &collector_action);
335 processor.EnqueueAction(&collector_action);
336
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700337 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800338 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700339 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800340 SendEnumToUMA(metrics::kMetricCheckResult,
341 static_cast<int>(expected_check_result),
342 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
343 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700344 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800345 SendEnumToUMA(metrics::kMetricCheckReaction,
346 static_cast<int>(expected_check_reaction),
347 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
348 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700349 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700350 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
351 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800352 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
353 ? 0 : 1);
354
Alex Deymo60ca1a72015-06-18 18:19:15 -0700355 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700356 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700357 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();
Alex Deymo72a25672016-03-23 15:44:39 -0700390 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700391
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700392 if (out_post_data)
393 *out_post_data = fetcher->post_data();
394}
395
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700396TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400397 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700398 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400399 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700400 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700401 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400402 -1,
403 false, // ping_only
404 ErrorCode::kOmahaRequestXMLHasEntityDecl,
405 metrics::CheckResult::kParsingError,
406 metrics::CheckReaction::kUnset,
407 metrics::DownloadErrorCode::kUnset,
408 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700409 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400410 EXPECT_FALSE(response.update_exists);
411}
412
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700413TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700414 OmahaResponse response;
415 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700416 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700417 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700418 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700419 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700420 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800421 metrics::CheckResult::kNoUpdateAvailable,
422 metrics::CheckReaction::kUnset,
423 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700424 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700425 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700426 EXPECT_FALSE(response.update_exists);
427}
428
Alex Deymo8e18f932015-03-27 16:16:59 -0700429// Test that all the values in the response are parsed in a normal update
430// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700431TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700432 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700433 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700434 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700435 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700436 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700437 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700438 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700439 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800440 metrics::CheckResult::kUpdateAvailable,
441 metrics::CheckReaction::kUpdating,
442 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700443 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700444 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700445 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -0700446 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700447 EXPECT_EQ(fake_update_response_.version, response.version);
448 EXPECT_EQ(fake_update_response_.GetPayloadUrl(), response.payload_urls[0]);
449 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
450 EXPECT_EQ(fake_update_response_.hash, response.hash);
451 EXPECT_EQ(fake_update_response_.size, response.size);
452 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
453 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
454 // Omaha cohort attribets are not set in the response, so they should not be
455 // persisted.
456 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
457 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
458 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700459}
460
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700461TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700462 OmahaResponse response;
463 // Set up a connection manager that doesn't allow a valid update over
464 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700465 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700466 fake_system_state_.set_connection_manager(&mock_cm);
467
Alex Deymo30534502015-07-20 15:06:33 -0700468 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
469 .WillRepeatedly(
Sen Jiang255e22b2016-05-20 16:15:29 -0700470 DoAll(SetArgumentPointee<0>(ConnectionType::kEthernet),
471 SetArgumentPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700472 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700473 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
474 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700475
476 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700477 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700478 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700479 -1,
480 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700481 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700482 metrics::CheckResult::kUpdateAvailable,
483 metrics::CheckReaction::kIgnored,
484 metrics::DownloadErrorCode::kUnset,
485 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700486 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700487 EXPECT_FALSE(response.update_exists);
488}
489
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700490TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700491 string rollback_version = "1234.0.0";
492 OmahaResponse response;
493
494 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700495 fake_system_state_.set_payload_state(&mock_payload_state);
496
Chris Sosa77f79e82014-06-02 18:16:24 -0700497 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
498 .WillRepeatedly(Return(rollback_version));
499
Alex Deymo8e18f932015-03-27 16:16:59 -0700500 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700501 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700502 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700503 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700504 -1,
505 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700506 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700507 metrics::CheckResult::kUpdateAvailable,
508 metrics::CheckReaction::kIgnored,
509 metrics::DownloadErrorCode::kUnset,
510 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700511 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700512 EXPECT_FALSE(response.update_exists);
513}
514
Kevin Cernekee2494e282016-03-29 18:03:53 -0700515// Verify that update checks called during OOBE will only try to download
516// an update if the response includes a non-empty deadline field.
517TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
518 OmahaResponse response;
519
520 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
521 ASSERT_FALSE(
522 TestUpdateCheck(nullptr, // request_params
523 fake_update_response_.GetUpdateResponse(),
524 -1,
525 false, // ping_only
526 ErrorCode::kNonCriticalUpdateInOOBE,
527 metrics::CheckResult::kUnset,
528 metrics::CheckReaction::kUnset,
529 metrics::DownloadErrorCode::kUnset,
530 &response,
531 nullptr));
532 EXPECT_FALSE(response.update_exists);
533
Alex Deymo46a9aae2016-05-04 20:20:11 -0700534 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
535 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
536 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
537 fake_update_response_.GetUpdateResponse(),
538 -1,
539 false, // ping_only
540 ErrorCode::kSuccess,
541 metrics::CheckResult::kUpdateAvailable,
542 metrics::CheckReaction::kUpdating,
543 metrics::DownloadErrorCode::kUnset,
544 &response,
545 nullptr));
546 EXPECT_TRUE(response.update_exists);
547 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
548
549 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700550 fake_update_response_.deadline = "20101020";
551 ASSERT_TRUE(
552 TestUpdateCheck(nullptr, // request_params
553 fake_update_response_.GetUpdateResponse(),
554 -1,
555 false, // ping_only
556 ErrorCode::kSuccess,
557 metrics::CheckResult::kUpdateAvailable,
558 metrics::CheckReaction::kUpdating,
559 metrics::DownloadErrorCode::kUnset,
560 &response,
561 nullptr));
562 EXPECT_TRUE(response.update_exists);
563}
564
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700565TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700566 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700567 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700568 params.set_wall_clock_based_wait_enabled(true);
569 params.set_update_check_count_wait_enabled(false);
570 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700571
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700572 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700573 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700574 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700575 -1,
576 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700577 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800578 metrics::CheckResult::kUpdateAvailable,
579 metrics::CheckReaction::kDeferring,
580 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700581 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700582 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700583 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700584
585 // Verify if we are interactive check we don't defer.
586 params.set_interactive(true);
587 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700588 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700589 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700590 -1,
591 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700592 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800593 metrics::CheckResult::kUpdateAvailable,
594 metrics::CheckReaction::kUpdating,
595 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700596 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700597 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700598 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700599}
600
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700601TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700602 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700603 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700604 params.set_wall_clock_based_wait_enabled(false);
605 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700606
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700607 params.set_update_check_count_wait_enabled(true);
608 params.set_min_update_checks_needed(1);
609 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700610
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700611 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700612 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700613 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700614 -1,
615 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700616 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800617 metrics::CheckResult::kUpdateAvailable,
618 metrics::CheckReaction::kUpdating,
619 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700620 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700621 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700622 EXPECT_TRUE(response.update_exists);
623}
624
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700625TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700626 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700627 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700628 params.set_wall_clock_based_wait_enabled(true);
629 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700630
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700631 params.set_update_check_count_wait_enabled(true);
632 params.set_min_update_checks_needed(1);
633 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700634
Alex Deymo8e18f932015-03-27 16:16:59 -0700635 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700636 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700637 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700638 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700639 -1,
640 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700641 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800642 metrics::CheckResult::kUpdateAvailable,
643 metrics::CheckReaction::kUpdating,
644 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700645 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700646 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700647 EXPECT_TRUE(response.update_exists);
648}
649
650
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700651TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700652 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700653 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700654 params.set_wall_clock_based_wait_enabled(true);
655 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700656
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700657 params.set_update_check_count_wait_enabled(true);
658 params.set_min_update_checks_needed(0);
659 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700660
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700661 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700662 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700663 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -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,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700670 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700671 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700672
Ben Chan9abb7632014-08-07 00:10:53 -0700673 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700674 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700675 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700676 EXPECT_TRUE(response.update_exists);
677}
678
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700679TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700680 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700681 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700682 params.set_wall_clock_based_wait_enabled(true);
683 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700684
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700685 params.set_update_check_count_wait_enabled(true);
686 params.set_min_update_checks_needed(1);
687 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700688
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700689 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700690 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700691 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700692 -1,
693 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700694 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800695 metrics::CheckResult::kUpdateAvailable,
696 metrics::CheckReaction::kDeferring,
697 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700698 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700699 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700700
Ben Chan9abb7632014-08-07 00:10:53 -0700701 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700702 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700703 ASSERT_GT(count, 0);
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}
721
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700722TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700723 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700724 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700725 params.set_wall_clock_based_wait_enabled(true);
726 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700727
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700728 params.set_update_check_count_wait_enabled(true);
729 params.set_min_update_checks_needed(1);
730 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700731
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700732 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700733
734 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700735 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700736 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700737 -1,
738 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700739 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800740 metrics::CheckResult::kUpdateAvailable,
741 metrics::CheckReaction::kDeferring,
742 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700743 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700744 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700745
Ben Chan9abb7632014-08-07 00:10:53 -0700746 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700747 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700748 // count remains the same, as the decrementing happens in update_attempter
749 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700750 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700751 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700752
753 // Verify if we are interactive check we don't defer.
754 params.set_interactive(true);
755 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700756 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700757 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700758 -1,
759 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700760 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800761 metrics::CheckResult::kUpdateAvailable,
762 metrics::CheckReaction::kUpdating,
763 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700764 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700765 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700766 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700767}
Jay Srinivasan0a708742012-03-20 11:26:12 -0700768
Alex Deymo8e18f932015-03-27 16:16:59 -0700769TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
770 OmahaResponse response;
771 OmahaRequestParams params = request_params_;
772 fake_update_response_.include_cohorts = true;
773 fake_update_response_.cohort = "s/154454/8479665";
774 fake_update_response_.cohorthint = "please-put-me-on-beta";
775 fake_update_response_.cohortname = "stable";
776
777 ASSERT_TRUE(TestUpdateCheck(&params,
778 fake_update_response_.GetUpdateResponse(),
779 -1,
780 false, // ping_only
781 ErrorCode::kSuccess,
782 metrics::CheckResult::kUpdateAvailable,
783 metrics::CheckReaction::kUpdating,
784 metrics::DownloadErrorCode::kUnset,
785 &response,
786 nullptr));
787
788 string value;
789 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
790 EXPECT_EQ(fake_update_response_.cohort, value);
791
792 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
793 EXPECT_EQ(fake_update_response_.cohorthint, value);
794
795 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
796 EXPECT_EQ(fake_update_response_.cohortname, value);
797}
798
799TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
800 OmahaResponse response;
801 OmahaRequestParams params = request_params_;
802 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
803 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
804 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
805 fake_update_response_.include_cohorts = true;
806 fake_update_response_.cohort = "s/154454/8479665";
807 fake_update_response_.cohorthint = "please-put-me-on-beta";
808 fake_update_response_.cohortname = "";
809
810 ASSERT_TRUE(TestUpdateCheck(&params,
811 fake_update_response_.GetUpdateResponse(),
812 -1,
813 false, // ping_only
814 ErrorCode::kSuccess,
815 metrics::CheckResult::kUpdateAvailable,
816 metrics::CheckReaction::kUpdating,
817 metrics::DownloadErrorCode::kUnset,
818 &response,
819 nullptr));
820
821 string value;
822 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
823 EXPECT_EQ(fake_update_response_.cohort, value);
824
825 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
826 EXPECT_EQ(fake_update_response_.cohorthint, value);
827
828 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
829}
830
831TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
832 OmahaResponse response;
833 OmahaRequestParams params = request_params_;
834 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
835
836 ASSERT_TRUE(TestUpdateCheck(&params,
837 fake_update_response_.GetUpdateResponse(),
838 -1,
839 false, // ping_only
840 ErrorCode::kSuccess,
841 metrics::CheckResult::kUpdateAvailable,
842 metrics::CheckReaction::kUpdating,
843 metrics::DownloadErrorCode::kUnset,
844 &response,
845 nullptr));
846
847 string value;
848 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
849 EXPECT_EQ("old_value", value);
850
851 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
852 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
853}
854
Alex Deymo00d79ac2015-06-29 15:41:49 -0700855TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
856 OmahaResponse response;
857 OmahaRequestParams params = request_params_;
858 fake_update_response_.include_cohorts = true;
859 fake_update_response_.cohort = "s/154454/8479665";
860 fake_update_response_.cohorthint = "please-put-me-on-beta";
861 fake_update_response_.cohortname = "stable";
862
863 ASSERT_TRUE(TestUpdateCheck(&params,
864 fake_update_response_.GetNoUpdateResponse(),
865 -1,
866 false, // ping_only
867 ErrorCode::kSuccess,
868 metrics::CheckResult::kNoUpdateAvailable,
869 metrics::CheckReaction::kUnset,
870 metrics::DownloadErrorCode::kUnset,
871 &response,
872 nullptr));
873
874 string value;
875 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
876 EXPECT_EQ(fake_update_response_.cohort, value);
877
878 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
879 EXPECT_EQ(fake_update_response_.cohorthint, value);
880
881 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
882 EXPECT_EQ(fake_update_response_.cohortname, value);
883}
884
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700885TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -0700886 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700887
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700888 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700889 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700890
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700891 OmahaRequestParams params = request_params_;
892 fake_system_state_.set_request_params(&params);
893 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300894 brillo::make_unique_ptr(
895 new MockHttpFetcher(http_response.data(),
896 http_response.size(),
897 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -0700898 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700899 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700900 ActionProcessor processor;
901 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700902 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700903
Alex Deymo60ca1a72015-06-18 18:19:15 -0700904 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
905 loop.Run();
906 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700907 EXPECT_FALSE(processor.IsRunning());
908}
909
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700910TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700911 OmahaResponse response;
912 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700913 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700914 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700915 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700916 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700917 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800918 metrics::CheckResult::kParsingError,
919 metrics::CheckReaction::kUnset,
920 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -0700921 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700922 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -0700923 EXPECT_FALSE(response.update_exists);
924}
925
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700926TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -0700927 OmahaResponse response;
928 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700929 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -0700930 "",
931 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700932 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700933 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800934 metrics::CheckResult::kParsingError,
935 metrics::CheckReaction::kUnset,
936 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700937 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700938 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700939 EXPECT_FALSE(response.update_exists);
940}
941
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700942TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700943 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700944 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700945 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700946 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
947 "<daystart elapsed_seconds=\"100\"/>"
948 "<app appid=\"foo\" status=\"ok\">"
949 "<ping status=\"ok\"/>"
950 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700951 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700952 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700953 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800954 metrics::CheckResult::kParsingError,
955 metrics::CheckReaction::kUnset,
956 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700957 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700958 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700959 EXPECT_FALSE(response.update_exists);
960}
961
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700962TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700963 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700964 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700965 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700966 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
967 "<daystart elapsed_seconds=\"100\"/>"
968 "<app appid=\"foo\" status=\"ok\">"
969 "<ping status=\"ok\"/>"
970 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700971 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700972 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700973 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800974 metrics::CheckResult::kParsingError,
975 metrics::CheckReaction::kUnset,
976 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700977 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700978 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700979 EXPECT_FALSE(response.update_exists);
980}
981
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700982TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700983 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700984 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700985 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700986 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
987 "<daystart elapsed_seconds=\"100\"/>"
988 "<app appid=\"foo\" status=\"ok\">"
989 "<ping status=\"ok\"/>"
990 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700991 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700992 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700993 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800994 metrics::CheckResult::kParsingError,
995 metrics::CheckReaction::kUnset,
996 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700997 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700998 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700999 EXPECT_FALSE(response.update_exists);
1000}
1001
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001002TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001003 string input_response =
1004 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1005 "<daystart elapsed_seconds=\"100\"/>"
1006 "<app appid=\"xyz\" status=\"ok\">"
1007 "<updatecheck status=\"ok\">"
1008 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001009 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001010 "<packages><package hash=\"not-used\" name=\"f\" "
1011 "size=\"587\"/></packages>"
1012 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001013 "ChromeOSVersion=\"10.2.3.4\" "
1014 "Prompt=\"false\" "
1015 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001016 "IsDeltaPayload=\"false\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001017 "sha256=\"lkq34j5345\" "
1018 "needsadmin=\"true\" "
1019 "/></actions></manifest></updatecheck></app></response>";
1020 LOG(INFO) << "Input Response = " << input_response;
1021
Darin Petkov6a5b3222010-07-13 14:55:28 -07001022 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001023 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001024 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001025 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001026 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001027 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001028 metrics::CheckResult::kUpdateAvailable,
1029 metrics::CheckReaction::kUpdating,
1030 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001031 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001032 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001033 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001034 EXPECT_EQ("10.2.3.4", response.version);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -08001035 EXPECT_EQ("http://missing/field/test/f", response.payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001036 EXPECT_EQ("", response.more_info_url);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001037 EXPECT_EQ("lkq34j5345", response.hash);
1038 EXPECT_EQ(587, response.size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001039 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001040 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001041}
1042
1043namespace {
1044class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1045 public:
1046 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001047 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001048 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001049};
1050
Alex Deymo60ca1a72015-06-18 18:19:15 -07001051void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001052 processor->StartProcessing();
1053 CHECK(processor->IsRunning());
1054 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001055}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001056} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001057
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001058TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001059 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001060 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001061
Alex Deymo60ca1a72015-06-18 18:19:15 -07001062 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001063 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001064 brillo::make_unique_ptr(
1065 new MockHttpFetcher(http_response.data(),
1066 http_response.size(),
1067 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001068 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001069 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001070 ActionProcessor processor;
1071 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001072 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001073
Alex Deymo60ca1a72015-06-18 18:19:15 -07001074 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1075 loop.Run();
1076 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001077}
1078
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001079TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001080 string output;
1081 EXPECT_TRUE(XmlEncode("ab", &output));
1082 EXPECT_EQ("ab", output);
1083 EXPECT_TRUE(XmlEncode("a<b", &output));
1084 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001085 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1086 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001087 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1088 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001089 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001090 EXPECT_FALSE(XmlEncode("\xc2", &output));
1091 // Fail with invalid ASCII-7 chars.
1092 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1093}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001094
Alex Deymob0d74eb2015-03-30 17:59:17 -07001095TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1096 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1097 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1098}
1099
1100TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001101 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001102
1103 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001104 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001105 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001106 OmahaRequestParams::kOsVersion,
1107 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001108 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001109 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001110 "0.1.0.0",
1111 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001112 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001113 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001114 "ChromeOSFirmware.1.0",
1115 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001116 false, // delta okay
1117 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001118 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001119 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001120 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1121 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1122 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001123 base::JoinString(
1124 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001125 OmahaResponse response;
1126 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001127 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001128 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001129 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001130 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001131 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001132 metrics::CheckResult::kParsingError,
1133 metrics::CheckReaction::kUnset,
1134 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001135 &response,
1136 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001137 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001138 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001139 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1140 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1141 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1142 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1143 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1144 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1145 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1146 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1147 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1148 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1149 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1150 // Values from Prefs that are too big are removed from the XML instead of
1151 // encoded.
1152 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001153}
1154
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001155TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001156 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001157 fake_update_response_.deadline = "&lt;20110101";
1158 fake_update_response_.more_info_url = "testthe&lt;url";
1159 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001160 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001161 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001162 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001163 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001164 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001165 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001166 metrics::CheckResult::kUpdateAvailable,
1167 metrics::CheckReaction::kUpdating,
1168 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001169 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001170 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001171
1172 EXPECT_EQ(response.more_info_url, "testthe<url");
Jay Srinivasan6f6ea002012-12-14 11:26:28 -08001173 EXPECT_EQ(response.payload_urls[0], "testthe&codebase/file.signed");
Darin Petkov6c118642010-10-21 12:06:30 -07001174 EXPECT_EQ(response.deadline, "<20110101");
Darin Petkov6a5b3222010-07-13 14:55:28 -07001175}
1176
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001177TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001178 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001179 // overflows int32_t:
1180 fake_update_response_.size = 123123123123123ll;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001181 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001182 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001183 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001184 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001185 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001186 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001187 metrics::CheckResult::kUpdateAvailable,
1188 metrics::CheckReaction::kUpdating,
1189 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001190 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001191 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001192
1193 EXPECT_EQ(response.size, 123123123123123ll);
1194}
1195
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001196TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001197 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001198 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001199 fake_system_state_.set_prefs(&prefs);
1200
Darin Petkov95508da2011-01-05 12:42:29 -08001201 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1202 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001203 // An existing but empty previous version means that we didn't reboot to a new
1204 // update, therefore, no need to update the previous version.
1205 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001206 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001207 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001208 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001209 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001210 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001211 metrics::CheckResult::kParsingError,
1212 metrics::CheckReaction::kUnset,
1213 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001214 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001215 &post_data));
1216 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001217 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001218 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001219 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001220 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001221 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001222 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1223 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001224 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1225 string::npos);
1226 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1227 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001228 // No <event> tag should be sent if we didn't reboot to an update.
1229 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001230}
1231
Jay Srinivasan0a708742012-03-20 11:26:12 -07001232
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001233TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001234 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001235 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001236 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1237 "invalid xml>",
1238 &post_data);
1239 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001240 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001241 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001242 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001243 OmahaEvent::kTypeUpdateDownloadStarted,
1244 OmahaEvent::kResultSuccess);
1245 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001246 EXPECT_EQ(post_str.find("ping"), string::npos);
1247 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001248}
1249
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001250TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001251 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001252 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001253 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1254 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001255 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001256 "invalid xml>",
1257 &post_data);
1258 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001259 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001260 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001261 " <event eventtype=\"%d\" eventresult=\"%d\" "
1262 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001263 OmahaEvent::kTypeDownloadComplete,
1264 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001265 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001266 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001267 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001268}
1269
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001270TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001271 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001272 // Create a copy of the OmahaRequestParams to reuse it later.
1273 OmahaRequestParams params = request_params_;
1274 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001275 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001276 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001277 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001278 brillo::make_unique_ptr(
1279 new MockHttpFetcher(http_response.data(),
1280 http_response.size(),
1281 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001282 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001283 EXPECT_FALSE(update_check_action.IsEvent());
1284
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001285 params = request_params_;
1286 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001287 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001288 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001289 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001290 brillo::make_unique_ptr(
1291 new MockHttpFetcher(http_response.data(),
1292 http_response.size(),
1293 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001294 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001295 EXPECT_TRUE(event_action.IsEvent());
1296}
1297
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001298TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001299 for (int i = 0; i < 2; i++) {
1300 bool delta_okay = i == 1;
1301 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001302 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001303 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001304 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001305 OmahaRequestParams::kOsVersion,
1306 "service_pack",
1307 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001308 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001309 "0.1.0.0",
1310 "en-US",
1311 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001312 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001313 "ChromeOSFirmware.1.0",
1314 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001315 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001316 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001317 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001318 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001319 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001320 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001321 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001322 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001323 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001324 metrics::CheckResult::kParsingError,
1325 metrics::CheckReaction::kUnset,
1326 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001327 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001328 &post_data));
1329 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001330 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001331 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1332 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001333 string::npos)
1334 << "i = " << i;
1335 }
1336}
1337
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001338TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001339 for (int i = 0; i < 2; i++) {
1340 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001341 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001342 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001343 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001344 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001345 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001346 OmahaRequestParams::kOsVersion,
1347 "service_pack",
1348 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001349 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001350 "0.1.0.0",
1351 "en-US",
1352 "unittest_track",
1353 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001354 "ChromeOSFirmware.1.0",
1355 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001356 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001357 interactive,
1358 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001359 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001360 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001361 "invalid xml>",
1362 -1,
1363 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001364 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001365 metrics::CheckResult::kParsingError,
1366 metrics::CheckReaction::kUnset,
1367 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001368 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001369 &post_data));
1370 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001371 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001372 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1373 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001374 string::npos)
1375 << "i = " << i;
1376 }
1377}
1378
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001379TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001380 OmahaEvent default_event;
1381 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1382 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001383 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001384
1385 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1386 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1387 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001388 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001389
1390 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1391 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001392 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001393 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1394 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001395 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001396}
1397
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001398void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001399 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001400 fake_system_state_.set_prefs(&prefs);
1401 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1402 .Times(AnyNumber());
1403 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1404 // Add a few hours to the day difference to test no rounding, etc.
1405 int64_t five_days_ago =
1406 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1407 int64_t six_days_ago =
1408 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1409 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1410 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1411 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1412 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1413 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1414 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001415 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001416 ASSERT_TRUE(
1417 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001418 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001419 -1,
1420 ping_only,
1421 ErrorCode::kSuccess,
1422 metrics::CheckResult::kUnset,
1423 metrics::CheckReaction::kUnset,
1424 metrics::DownloadErrorCode::kUnset,
1425 nullptr,
1426 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001427 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001428 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1429 string::npos);
1430 if (ping_only) {
1431 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1432 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1433 } else {
1434 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1435 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001436 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001437}
1438
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001439TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1440 PingTest(true /* ping_only */);
1441}
1442
1443TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1444 PingTest(false /* ping_only */);
1445}
1446
1447TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001448 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001449 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001450 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1451 .Times(AnyNumber());
1452 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001453 int64_t three_days_ago =
1454 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1455 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001456 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1457 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001458 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1459 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1460 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1461 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001462 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001463 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001464 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001465 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001466 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001467 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001468 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001469 metrics::CheckResult::kNoUpdateAvailable,
1470 metrics::CheckReaction::kUnset,
1471 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001472 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001473 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001474 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001475 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001476 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001477}
1478
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001479TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001480 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001481 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001482 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1483 .Times(AnyNumber());
1484 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001485 int64_t four_days_ago =
1486 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1487 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001488 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1489 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001490 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1491 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1492 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1493 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001494 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001495 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001496 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001497 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001498 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001499 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001500 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001501 metrics::CheckResult::kNoUpdateAvailable,
1502 metrics::CheckReaction::kUnset,
1503 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001504 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001505 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001506 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001507 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001508 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001509}
1510
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001511TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001512 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001513 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001514 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1515 .Times(AnyNumber());
1516 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001517 int64_t one_hour_ago =
1518 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001519 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1520 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001521 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1522 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1523 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1524 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001525 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1526 // send a ping.
1527 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1528 .WillOnce(Return(true));
1529 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1530 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001531 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001532 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001533 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001534 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001535 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001536 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001537 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001538 metrics::CheckResult::kNoUpdateAvailable,
1539 metrics::CheckReaction::kUnset,
1540 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001541 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001542 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001543 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001544 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001545}
1546
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001547TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001548 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001549 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001550 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001551 int64_t now = Time::Now().ToInternalValue();
1552 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1553 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1554 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1555 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1556 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1557 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001558 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001559 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001560 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001561 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001562 -1,
1563 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001564 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001565 metrics::CheckResult::kUnset,
1566 metrics::CheckReaction::kUnset,
1567 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001568 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001569 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001570 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001571}
1572
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001573TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001574 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001575 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001576 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1577 .Times(AnyNumber());
1578 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001579 int64_t future =
1580 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001581 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1582 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001583 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1584 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1585 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1586 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1587 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1588 .WillOnce(Return(true));
1589 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1590 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001591 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001592 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001593 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001594 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1595 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001596 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001597 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001598 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001599 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001600 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001601 metrics::CheckResult::kNoUpdateAvailable,
1602 metrics::CheckReaction::kUnset,
1603 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001604 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001605 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001606 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001607 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001608}
1609
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001610TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001611 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001612 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001613 // may fail if it runs for longer than 5 seconds. It shouldn't run
1614 // that long though.
1615 int64_t midnight =
1616 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1617 int64_t midnight_slack =
1618 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001619 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001620 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001621 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1622 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001623 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1624 AllOf(Ge(midnight), Le(midnight_slack))))
1625 .WillOnce(Return(true));
1626 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1627 AllOf(Ge(midnight), Le(midnight_slack))))
1628 .WillOnce(Return(true));
1629 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001630 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001631 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1632 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001633 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001634 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001635 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001636 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001637 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001638 metrics::CheckResult::kNoUpdateAvailable,
1639 metrics::CheckReaction::kUnset,
1640 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001641 nullptr,
1642 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001643}
1644
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001645TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001646 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001647 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001648 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1649 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001650 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1651 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1652 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001653 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001654 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1655 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001656 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001657 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001658 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001659 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001660 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001661 metrics::CheckResult::kNoUpdateAvailable,
1662 metrics::CheckReaction::kUnset,
1663 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001664 nullptr,
1665 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001666}
1667
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001668TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001669 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001670 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001671 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1672 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001673 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1674 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1675 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001676 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001677 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1678 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001679 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001680 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001681 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001682 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001683 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001684 metrics::CheckResult::kNoUpdateAvailable,
1685 metrics::CheckReaction::kUnset,
1686 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001687 nullptr,
1688 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001689}
1690
Alex Deymob3fa53b2016-04-18 19:57:58 -07001691TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
1692 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
1693 // the "eol" attribute.
1694 ASSERT_TRUE(
1695 TestUpdateCheck(nullptr, // request_params
1696 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1697 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
1698 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
1699 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
1700 "</app></response>",
1701 -1,
1702 false, // ping_only
1703 ErrorCode::kSuccess,
1704 metrics::CheckResult::kNoUpdateAvailable,
1705 metrics::CheckReaction::kUnset,
1706 metrics::DownloadErrorCode::kUnset,
1707 nullptr,
1708 nullptr));
1709 string eol_pref;
1710 EXPECT_TRUE(
1711 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
1712 // Note that the eol="eol" attribute should be ignored and the _eol should be
1713 // used instead.
1714 EXPECT_EQ("security-only", eol_pref);
1715}
1716
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001717TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001718 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001719 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07001720 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001721 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001722 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001723 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001724 metrics::CheckResult::kParsingError,
1725 metrics::CheckReaction::kUnset,
1726 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001727 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07001728 &post_data));
1729 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001730 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07001731 EXPECT_EQ(post_str.find("machineid="), string::npos);
1732 EXPECT_EQ(post_str.find("userid="), string::npos);
1733}
1734
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001735TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001736 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001737 const int http_error_code =
1738 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07001739 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001740 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001741 "",
1742 501,
Darin Petkov265f2902011-05-09 15:17:40 -07001743 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001744 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001745 metrics::CheckResult::kDownloadError,
1746 metrics::CheckReaction::kUnset,
1747 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07001748 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001749 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001750 EXPECT_FALSE(response.update_exists);
1751}
1752
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001753TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001754 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001755 const int http_error_code =
1756 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07001757 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001758 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001759 "",
1760 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07001761 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001762 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001763 metrics::CheckResult::kDownloadError,
1764 metrics::CheckReaction::kUnset,
1765 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07001766 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001767 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001768 EXPECT_FALSE(response.update_exists);
1769}
1770
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001771TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001772 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001773 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001774 params.set_wall_clock_based_wait_enabled(true);
1775 params.set_waiting_period(TimeDelta().FromDays(1));
1776 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001777
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001778 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001779 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001780 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001781 -1,
1782 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001783 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001784 metrics::CheckResult::kUpdateAvailable,
1785 metrics::CheckReaction::kDeferring,
1786 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001787 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001788 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001789
Ben Chan9abb7632014-08-07 00:10:53 -07001790 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001791 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001792 ASSERT_GT(timestamp, 0);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001793 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001794
1795 // Verify if we are interactive check we don't defer.
1796 params.set_interactive(true);
1797 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001798 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001799 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001800 -1,
1801 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001802 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001803 metrics::CheckResult::kUpdateAvailable,
1804 metrics::CheckReaction::kUpdating,
1805 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001806 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001807 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001808 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001809}
1810
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001811TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001812 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001813 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001814 params.set_wall_clock_based_wait_enabled(true);
1815 params.set_waiting_period(TimeDelta().FromDays(1));
1816 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001817
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001818 // Set the timestamp to a very old value such that it exceeds the
1819 // waiting period set above.
1820 Time t1;
1821 Time::FromString("1/1/2012", &t1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001822 ASSERT_TRUE(fake_prefs_.SetInt64(
1823 kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001824 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001825 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001826 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001827 -1,
1828 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001829 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001830 metrics::CheckResult::kUpdateAvailable,
1831 metrics::CheckReaction::kUpdating,
1832 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001833 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001834 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001835
1836 EXPECT_TRUE(response.update_exists);
1837
1838 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07001839 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001840 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001841 ASSERT_TRUE(timestamp == t1.ToInternalValue());
1842}
1843
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001844TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001845 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07001846 base::ScopedTempDir tempdir;
1847 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001848
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001849 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07001850 OmahaRequestParams params(&fake_system_state_);
1851 params.set_root(tempdir.path().value());
1852 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
1853 params.set_app_version("1.2.3.4");
1854 params.set_current_channel("canary-channel");
1855 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
1856 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001857 EXPECT_TRUE(params.to_more_stable_channel());
1858 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001859 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001860 "invalid xml>",
1861 -1,
1862 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001863 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001864 metrics::CheckResult::kParsingError,
1865 metrics::CheckReaction::kUnset,
1866 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001867 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001868 &post_data));
1869 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001870 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001871 EXPECT_NE(string::npos, post_str.find(
1872 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
1873 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
1874 "track=\"stable-channel\" from_track=\"canary-channel\" "));
1875}
1876
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001877TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001878 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07001879 base::ScopedTempDir tempdir;
1880 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001881
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001882 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07001883 OmahaRequestParams params(&fake_system_state_);
1884 params.set_root(tempdir.path().value());
1885 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
1886 params.set_app_version("5.6.7.8");
1887 params.set_current_channel("stable-channel");
1888 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
1889 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001890 EXPECT_FALSE(params.to_more_stable_channel());
1891 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001892 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001893 "invalid xml>",
1894 -1,
1895 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001896 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001897 metrics::CheckResult::kParsingError,
1898 metrics::CheckReaction::kUnset,
1899 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001900 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001901 &post_data));
1902 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001903 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001904 EXPECT_NE(string::npos, post_str.find(
1905 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
1906 "version=\"5.6.7.8\" "
1907 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001908 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001909}
1910
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001911// Checks that the initial ping with a=-1 r=-1 is not send when the device
1912// was powerwashed.
1913TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
1914 fake_prefs_.SetString(kPrefsPreviousVersion, "");
1915
1916 // Flag that the device was powerwashed in the past.
1917 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
1918
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001919 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001920 ASSERT_TRUE(
1921 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001922 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001923 -1,
1924 false, // ping_only
1925 ErrorCode::kSuccess,
1926 metrics::CheckResult::kNoUpdateAvailable,
1927 metrics::CheckReaction::kUnset,
1928 metrics::DownloadErrorCode::kUnset,
1929 nullptr,
1930 &post_data));
1931 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001932 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001933 EXPECT_EQ(string::npos, post_str.find("<ping"));
1934}
1935
Alex Deymo9fded1e2015-11-05 12:31:19 -08001936// Checks that the event 54 is sent on a reboot to a new update.
1937TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
1938 // Flag that the device was updated in a previous boot.
1939 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
1940
1941 brillo::Blob post_data;
1942 ASSERT_TRUE(
1943 TestUpdateCheck(nullptr, // request_params
1944 fake_update_response_.GetNoUpdateResponse(),
1945 -1,
1946 false, // ping_only
1947 ErrorCode::kSuccess,
1948 metrics::CheckResult::kNoUpdateAvailable,
1949 metrics::CheckReaction::kUnset,
1950 metrics::DownloadErrorCode::kUnset,
1951 nullptr,
1952 &post_data));
1953 string post_str(post_data.begin(), post_data.end());
1954
1955 // An event 54 is included and has the right version.
1956 EXPECT_NE(string::npos,
1957 post_str.find(base::StringPrintf(
1958 "<event eventtype=\"%d\"",
1959 OmahaEvent::kTypeRebootedAfterUpdate)));
1960 EXPECT_NE(string::npos,
1961 post_str.find("previousversion=\"1.2.3.4\"></event>"));
1962
1963 // The previous version flag should have been removed.
1964 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
1965 string prev_version;
1966 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
1967 EXPECT_TRUE(prev_version.empty());
1968}
1969
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001970void OmahaRequestActionTest::P2PTest(
1971 bool initial_allow_p2p_for_downloading,
1972 bool initial_allow_p2p_for_sharing,
1973 bool omaha_disable_p2p_for_downloading,
1974 bool omaha_disable_p2p_for_sharing,
1975 bool payload_state_allow_p2p_attempt,
1976 bool expect_p2p_client_lookup,
1977 const string& p2p_client_result_url,
1978 bool expected_allow_p2p_for_downloading,
1979 bool expected_allow_p2p_for_sharing,
1980 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07001981 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001982 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07001983 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
1984 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
1985 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07001986
1987 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001988 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07001989 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
1990 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07001991 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
1992 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
1993 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
1994 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
1995 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
1996 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
1997 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
1998 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
1999 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2000 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2001
David Zeuthen8f191b22013-08-06 12:27:50 -07002002 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002003 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002004 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2005
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002006 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2007 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002008 .Times(expect_p2p_client_lookup ? 1 : 0);
2009
Alex Deymo8e18f932015-03-27 16:16:59 -07002010 fake_update_response_.disable_p2p_for_downloading =
2011 omaha_disable_p2p_for_downloading;
2012 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002013 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002014 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002015 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002016 -1,
2017 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002018 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002019 metrics::CheckResult::kUpdateAvailable,
2020 metrics::CheckReaction::kUpdating,
2021 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002022 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002023 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002024 EXPECT_TRUE(response.update_exists);
2025
Gilad Arnold74b5f552014-10-07 08:17:16 -07002026 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2027 response.disable_p2p_for_downloading);
2028 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2029 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002030
Gilad Arnold74b5f552014-10-07 08:17:16 -07002031 EXPECT_EQ(expected_allow_p2p_for_downloading,
2032 actual_allow_p2p_for_downloading);
2033 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2034 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002035}
2036
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002037TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002038 P2PTest(true, // initial_allow_p2p_for_downloading
2039 true, // initial_allow_p2p_for_sharing
2040 false, // omaha_disable_p2p_for_downloading
2041 false, // omaha_disable_p2p_for_sharing
2042 true, // payload_state_allow_p2p_attempt
2043 true, // expect_p2p_client_lookup
2044 "http://1.3.5.7/p2p", // p2p_client_result_url
2045 true, // expected_allow_p2p_for_downloading
2046 true, // expected_allow_p2p_for_sharing
2047 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002048}
2049
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002050TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002051 P2PTest(true, // initial_allow_p2p_for_downloading
2052 true, // initial_allow_p2p_for_sharing
2053 false, // omaha_disable_p2p_for_downloading
2054 false, // omaha_disable_p2p_for_sharing
2055 true, // payload_state_allow_p2p_attempt
2056 true, // expect_p2p_client_lookup
2057 "", // p2p_client_result_url
2058 false, // expected_allow_p2p_for_downloading
2059 true, // expected_allow_p2p_for_sharing
2060 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002061}
2062
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002063TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002064 P2PTest(false, // initial_allow_p2p_for_downloading
2065 true, // initial_allow_p2p_for_sharing
2066 false, // omaha_disable_p2p_for_downloading
2067 false, // omaha_disable_p2p_for_sharing
2068 true, // payload_state_allow_p2p_attempt
2069 false, // expect_p2p_client_lookup
2070 "unset", // p2p_client_result_url
2071 false, // expected_allow_p2p_for_downloading
2072 true, // expected_allow_p2p_for_sharing
2073 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002074}
2075
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002076TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002077 P2PTest(true, // initial_allow_p2p_for_downloading
2078 true, // initial_allow_p2p_for_sharing
2079 true, // omaha_disable_p2p_for_downloading
2080 false, // omaha_disable_p2p_for_sharing
2081 true, // payload_state_allow_p2p_attempt
2082 false, // expect_p2p_client_lookup
2083 "unset", // p2p_client_result_url
2084 false, // expected_allow_p2p_for_downloading
2085 true, // expected_allow_p2p_for_sharing
2086 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002087}
2088
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002089TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002090 P2PTest(true, // initial_allow_p2p_for_downloading
2091 true, // initial_allow_p2p_for_sharing
2092 false, // omaha_disable_p2p_for_downloading
2093 true, // omaha_disable_p2p_for_sharing
2094 true, // payload_state_allow_p2p_attempt
2095 true, // expect_p2p_client_lookup
2096 "http://1.3.5.7/p2p", // p2p_client_result_url
2097 true, // expected_allow_p2p_for_downloading
2098 false, // expected_allow_p2p_for_sharing
2099 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002100}
2101
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002102TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002103 P2PTest(true, // initial_allow_p2p_for_downloading
2104 true, // initial_allow_p2p_for_sharing
2105 true, // omaha_disable_p2p_for_downloading
2106 true, // omaha_disable_p2p_for_sharing
2107 true, // payload_state_allow_p2p_attempt
2108 false, // expect_p2p_client_lookup
2109 "unset", // p2p_client_result_url
2110 false, // expected_allow_p2p_for_downloading
2111 false, // expected_allow_p2p_for_sharing
2112 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002113}
2114
Alex Deymof329b932014-10-30 01:37:48 -07002115bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2116 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002117 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002118 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002119 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002120 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002121 -1,
2122 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002123 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002124 metrics::CheckResult::kUpdateAvailable,
2125 metrics::CheckReaction::kUpdating,
2126 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002127 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002128 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002129}
2130
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002131TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002132 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002133
Kevin Cernekee2494e282016-03-29 18:03:53 -07002134 // Simulate a successful update check that happens during OOBE. The
2135 // deadline in the response is needed to force the update attempt to
2136 // occur; responses without a deadline seen during OOBE will normally
2137 // return ErrorCode::kNonCriticalUpdateInOOBE.
2138 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2139 fake_update_response_.deadline = "20101020";
2140
David Zeuthen639aa362014-02-03 16:23:44 -08002141 // Check that we parse elapsed_days in the Omaha Response correctly.
2142 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002143 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2144 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002145 EXPECT_TRUE(response.update_exists);
2146 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002147 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002148 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002149 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002150 EXPECT_EQ(prefs_days, 42);
2151
2152 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002153 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002154 EXPECT_TRUE(response.update_exists);
2155 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002156 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002157 EXPECT_EQ(prefs_days, 42);
2158
2159 // Note that elapsed_days is not necessarily divisible by 7 so check
2160 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002161 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2162 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002163 EXPECT_TRUE(response.update_exists);
2164 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002165 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002166 EXPECT_EQ(prefs_days, 21);
2167
2168 // Check that we correctly handle elapsed_days not being included in
2169 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002170 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002171 EXPECT_TRUE(response.update_exists);
2172 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002173}
2174
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002175// If there is no prefs and OOBE is not complete, we should not
2176// report anything to Omaha.
2177TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002178 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002179 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2180 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2181}
David Zeuthen639aa362014-02-03 16:23:44 -08002182
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002183// If OOBE is complete and happened on a valid date (e.g. after Jan
2184// 1 2007 0:00 PST), that date should be used and written to
2185// prefs. However, first try with an invalid date and check we do
2186// nothing.
2187TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2188 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2189 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2190 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2191 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2192}
David Zeuthen639aa362014-02-03 16:23:44 -08002193
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002194// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2195// should yield an InstallDate of 14.
2196TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2197 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2198 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2199 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2200 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002201
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002202 int64_t prefs_days;
2203 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2204 EXPECT_EQ(prefs_days, 14);
2205}
David Zeuthen639aa362014-02-03 16:23:44 -08002206
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002207// Now that we have a valid date in prefs, check that we keep using
2208// that even if OOBE date reports something else. The date Jan 30,
2209// 2007 0:00 PST should yield an InstallDate of 28... but since
2210// there's a prefs file, we should still get 14.
2211TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2212 // Set a valid date in the prefs first.
2213 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002214
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002215 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2216 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2217 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002218
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002219 int64_t prefs_days;
2220 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2221 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002222
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002223 // If we delete the prefs file, we should get 28 days.
2224 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2225 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2226 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2227 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002228}
2229
Darin Petkov6a5b3222010-07-13 14:55:28 -07002230} // namespace chromeos_update_engine