blob: 6d566033b648c3cfc1aaeca42954bb8569e0e7da [file] [log] [blame]
Alex Deymoaea4c1c2015-08-19 20:24:43 -07001//
2// Copyright (C) 2012 The Android Open Source Project
3//
4// Licensed under the Apache License, Version 2.0 (the "License");
5// you may not use this file except in compliance with the License.
6// You may obtain a copy of the License at
7//
8// http://www.apache.org/licenses/LICENSE-2.0
9//
10// Unless required by applicable law or agreed to in writing, software
11// distributed under the License is distributed on an "AS IS" BASIS,
12// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13// See the License for the specific language governing permissions and
14// limitations under the License.
15//
Darin Petkov6a5b3222010-07-13 14:55:28 -070016
Alex Deymo8427b4a2014-11-05 14:00:32 -080017#include "update_engine/omaha_request_action.h"
18
Ben Chan9abb7632014-08-07 00:10:53 -070019#include <stdint.h>
20
Darin Petkov6a5b3222010-07-13 14:55:28 -070021#include <string>
22#include <vector>
Darin Petkov0dc8e9a2010-07-14 14:51:57 -070023
Alex Deymo60ca1a72015-06-18 18:19:15 -070024#include <base/bind.h>
Alex Deymo110e0302015-10-19 20:35:21 -070025#include <base/files/file_util.h>
Sen Jiang297e5832016-03-17 14:45:51 -070026#include <base/files/scoped_temp_dir.h>
Alex Deymo8e18f932015-03-27 16:16:59 -070027#include <base/strings/string_number_conversions.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070028#include <base/strings/string_util.h>
29#include <base/strings/stringprintf.h>
30#include <base/time/time.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070031#include <brillo/bind_lambda.h>
Alex Deymoc1c17b42015-11-23 03:53:15 -030032#include <brillo/make_unique_ptr.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070033#include <brillo/message_loops/fake_message_loop.h>
34#include <brillo/message_loops/message_loop.h>
35#include <brillo/message_loops/message_loop_utils.h>
Alex Deymoe1e3afe2014-10-30 13:02:49 -070036#include <gtest/gtest.h>
Jay Srinivasand29695d2013-04-08 15:08:05 -070037
Alex Deymo39910dc2015-11-09 17:04:30 -080038#include "update_engine/common/action_pipe.h"
39#include "update_engine/common/constants.h"
40#include "update_engine/common/fake_prefs.h"
41#include "update_engine/common/hash_calculator.h"
42#include "update_engine/common/mock_http_fetcher.h"
43#include "update_engine/common/platform_constants.h"
44#include "update_engine/common/prefs.h"
45#include "update_engine/common/test_utils.h"
Alex Deymoc1c17b42015-11-23 03:53:15 -030046#include "update_engine/fake_system_state.h"
Alex Deymo38429cf2015-11-11 18:27:22 -080047#include "update_engine/metrics.h"
Chris Sosa77f79e82014-06-02 18:16:24 -070048#include "update_engine/mock_connection_manager.h"
Gilad Arnold74b5f552014-10-07 08:17:16 -070049#include "update_engine/mock_payload_state.h"
Darin Petkova4a8a8c2010-07-15 22:21:12 -070050#include "update_engine/omaha_request_params.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070051
Darin Petkov1cbd78f2010-07-29 12:38:34 -070052using base::Time;
53using base::TimeDelta;
Darin Petkov6a5b3222010-07-13 14:55:28 -070054using std::string;
55using std::vector;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070056using testing::AllOf;
Alex Deymof329b932014-10-30 01:37:48 -070057using testing::AnyNumber;
Jay Srinivasan34b5d862012-07-23 11:43:22 -070058using testing::DoAll;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070059using testing::Ge;
60using testing::Le;
Darin Petkov9c096d62010-11-17 14:49:04 -080061using testing::NiceMock;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070062using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070063using testing::ReturnPointee;
64using testing::SaveArg;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070065using testing::SetArgumentPointee;
Alex Deymof329b932014-10-30 01:37:48 -070066using testing::_;
Darin Petkov6a5b3222010-07-13 14:55:28 -070067
Alex Deymo8e18f932015-03-27 16:16:59 -070068namespace {
69
Alex Deymo85616652015-10-15 18:48:31 -070070const char kTestAppId[] = "test-app-id";
71
Alex Deymo8e18f932015-03-27 16:16:59 -070072// This is a helper struct to allow unit tests build an update response with the
73// values they care about.
74struct FakeUpdateResponse {
75 string GetNoUpdateResponse() const {
76 string entity_str;
77 if (include_entity)
78 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Sen Jiang81259682017-03-30 15:11:30 -070079 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" + entity_str +
80 "<response protocol=\"3.0\">"
81 "<daystart elapsed_seconds=\"100\"/>"
82 "<app appid=\"" +
83 app_id + "\" " +
84 (include_cohorts
85 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
86 "\" cohortname=\"" + cohortname + "\" "
87 : "") +
88 " status=\"ok\">"
89 "<ping status=\"ok\"/>"
90 "<updatecheck status=\"noupdate\"/></app>" +
91 (multi_app_no_update
92 ? "<app><updatecheck status=\"noupdate\"/></app>"
93 : "") +
94 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -070095 }
96
97 string GetUpdateResponse() const {
Sen Jiang2703ef42017-03-16 13:36:21 -070098 return "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
99 "protocol=\"3.0\">"
100 "<daystart elapsed_seconds=\"100\"" +
101 (elapsed_days.empty() ? ""
102 : (" elapsed_days=\"" + elapsed_days + "\"")) +
103 "/>"
104 "<app appid=\"" +
105 app_id + "\" " +
106 (include_cohorts
107 ? "cohort=\"" + cohort + "\" cohorthint=\"" + cohorthint +
108 "\" cohortname=\"" + cohortname + "\" "
109 : "") +
110 " status=\"ok\">"
111 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
112 "<urls><url codebase=\"" +
113 codebase +
114 "\"/></urls>"
115 "<manifest version=\"" +
116 version +
117 "\">"
118 "<packages><package hash=\"not-used\" name=\"" +
119 filename + "\" size=\"" + base::Int64ToString(size) +
Sen Jiang0affc2c2017-02-10 15:55:05 -0800120 "\" hash_sha256=\"" + hash + "\"/>" +
121 (multi_package ? "<package name=\"package2\" size=\"222\" "
122 "hash_sha256=\"hash2\"/>"
123 : "") +
124 "</packages>"
Sen Jiang81259682017-03-30 15:11:30 -0700125 "<actions><action event=\"postinstall\" MetadataSize=\"11" +
126 (multi_package ? ":22" : "") + "\" ChromeOSVersion=\"" + version +
127 "\" MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt +
Sen Jiang2703ef42017-03-16 13:36:21 -0700128 "\" "
129 "IsDelta=\"true\" "
130 "IsDeltaPayload=\"true\" "
131 "MaxDaysToScatter=\"" +
132 max_days_to_scatter +
133 "\" "
134 "sha256=\"not-used\" "
135 "needsadmin=\"" +
136 needsadmin + "\" " +
137 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
138 (disable_p2p_for_downloading ? "DisableP2PForDownloading=\"true\" "
139 : "") +
140 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
Sen Jiang81259682017-03-30 15:11:30 -0700141 "/></actions></manifest></updatecheck></app>" +
142 (multi_app
143 ? "<app><updatecheck status=\"ok\"><urls><url codebase=\"" +
144 codebase2 +
145 "\"/></urls><manifest><packages>"
146 "<package name=\"package3\" size=\"333\" "
147 "hash_sha256=\"hash3\"/></packages>"
148 "<actions><action event=\"postinstall\" "
149 "MetadataSize=\"33\"/></actions>"
150 "</manifest></updatecheck></app>"
151 : "") +
152 (multi_app_no_update
153 ? "<app><updatecheck status=\"noupdate\"/></app>"
154 : "") +
155 "</response>";
Alex Deymo8e18f932015-03-27 16:16:59 -0700156 }
157
158 // Return the payload URL, which is split in two fields in the XML response.
159 string GetPayloadUrl() {
160 return codebase + filename;
161 }
162
Alex Deymo85616652015-10-15 18:48:31 -0700163 string app_id = kTestAppId;
Alex Deymo8e18f932015-03-27 16:16:59 -0700164 string version = "1.2.3.4";
165 string more_info_url = "http://more/info";
166 string prompt = "true";
167 string codebase = "http://code/base/";
Sen Jiang81259682017-03-30 15:11:30 -0700168 string codebase2 = "http://code/base/2/";
Alex Deymo8e18f932015-03-27 16:16:59 -0700169 string filename = "file.signed";
Sen Jiang2703ef42017-03-16 13:36:21 -0700170 string hash = "4841534831323334";
Alex Deymo8e18f932015-03-27 16:16:59 -0700171 string needsadmin = "false";
Sen Jiang0affc2c2017-02-10 15:55:05 -0800172 uint64_t size = 123;
Alex Deymo8e18f932015-03-27 16:16:59 -0700173 string deadline = "";
174 string max_days_to_scatter = "7";
175 string elapsed_days = "42";
176
177 // P2P setting defaults to allowed.
178 bool disable_p2p_for_downloading = false;
179 bool disable_p2p_for_sharing = false;
180
181 // Omaha cohorts settings.
182 bool include_cohorts = false;
183 string cohort = "";
184 string cohorthint = "";
185 string cohortname = "";
186
187 // Whether to include the CrOS <!ENTITY> in the XML response.
188 bool include_entity = false;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800189
Sen Jiang81259682017-03-30 15:11:30 -0700190 // Whether to include more than one app.
191 bool multi_app = false;
192 // Whether to include an additional app with no update.
193 bool multi_app_no_update = false;
194 // Whether to include more than one package in an app.
Sen Jiang0affc2c2017-02-10 15:55:05 -0800195 bool multi_package = false;
Alex Deymo8e18f932015-03-27 16:16:59 -0700196};
197
198} // namespace
199
Darin Petkov6a5b3222010-07-13 14:55:28 -0700200namespace chromeos_update_engine {
201
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700202class OmahaRequestActionTest : public ::testing::Test {
203 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800204 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700205 fake_system_state_.set_request_params(&request_params_);
206 fake_system_state_.set_prefs(&fake_prefs_);
207 }
208
209 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800210 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700211 // is used. |payload_state| may be null, in which case a local mock is used.
212 // |p2p_manager| may be null, in which case a local mock is used.
213 // |connection_manager| may be null, in which case a local mock is used.
214 // out_response may be null. If |fail_http_response_code| is non-negative,
215 // the transfer will fail with that code. |ping_only| is passed through to the
216 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
217 // post-data received by the mock HttpFetcher is returned.
218 //
219 // The |expected_check_result|, |expected_check_reaction| and
220 // |expected_error_code| parameters are for checking expectations
221 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
222 // UMA statistics. Use the appropriate ::kUnset value to specify that
223 // the given metric should not be reported.
224 bool TestUpdateCheck(OmahaRequestParams* request_params,
225 const string& http_response,
226 int fail_http_response_code,
227 bool ping_only,
228 ErrorCode expected_code,
229 metrics::CheckResult expected_check_result,
230 metrics::CheckReaction expected_check_reaction,
231 metrics::DownloadErrorCode expected_download_error_code,
232 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700233 brillo::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700234
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800235 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700236 // only a ping or also an updatecheck.
237 void PingTest(bool ping_only);
238
239 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700240 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700241 OmahaResponse *response);
242
243 // P2P test helper function.
244 void P2PTest(
245 bool initial_allow_p2p_for_downloading,
246 bool initial_allow_p2p_for_sharing,
247 bool omaha_disable_p2p_for_downloading,
248 bool omaha_disable_p2p_for_sharing,
249 bool payload_state_allow_p2p_attempt,
250 bool expect_p2p_client_lookup,
251 const string& p2p_client_result_url,
252 bool expected_allow_p2p_for_downloading,
253 bool expected_allow_p2p_for_sharing,
254 const string& expected_p2p_url);
255
256 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700257 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700258
259 // By default, all tests use these objects unless they replace them in the
260 // fake_system_state_.
261 OmahaRequestParams request_params_ = OmahaRequestParams{
262 &fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -0700263 constants::kOmahaPlatformName,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700264 OmahaRequestParams::kOsVersion,
265 "service_pack",
266 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -0700267 kTestAppId,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700268 "0.1.0.0",
269 "en-US",
270 "unittest",
271 "OEM MODEL 09235 7471",
272 "ChromeOSFirmware.1.0",
273 "0X0A1",
274 false, // delta okay
275 false, // interactive
276 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700277 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700278
279 FakePrefs fake_prefs_;
280};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700281
282namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700283class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
284 public:
285 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700286 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800287 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700288 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800289 void ProcessingDone(const ActionProcessor* processor,
290 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700291 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700292 }
293
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800294 void ActionCompleted(ActionProcessor* processor,
295 AbstractAction* action,
296 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700297 // make sure actions always succeed
298 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700299 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700300 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700301 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700302 }
David Zeuthena99981f2013-04-29 13:42:47 -0700303 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700304};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700305} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700306
307class OutputObjectCollectorAction;
308
309template<>
310class ActionTraits<OutputObjectCollectorAction> {
311 public:
312 // Does not take an object for input
313 typedef OmahaResponse InputObjectType;
314 // On success, puts the output path on output
315 typedef NoneType OutputObjectType;
316};
317
318class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
319 public:
320 OutputObjectCollectorAction() : has_input_object_(false) {}
321 void PerformAction() {
322 // copy input object
323 has_input_object_ = HasInputObject();
324 if (has_input_object_)
325 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700326 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700327 }
328 // Should never be called
329 void TerminateProcessing() {
330 CHECK(false);
331 }
332 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700333 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700334 return "OutputObjectCollectorAction";
335 }
Alex Deymof329b932014-10-30 01:37:48 -0700336 string Type() const { return StaticType(); }
Alex Vakulenko0057daa2016-01-23 16:22:50 -0800337 using InputObjectType =
338 ActionTraits<OutputObjectCollectorAction>::InputObjectType;
339 using OutputObjectType =
340 ActionTraits<OutputObjectCollectorAction>::OutputObjectType;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700341 bool has_input_object_;
342 OmahaResponse omaha_response_;
343};
344
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700345bool OmahaRequestActionTest::TestUpdateCheck(
346 OmahaRequestParams* request_params,
347 const string& http_response,
348 int fail_http_response_code,
349 bool ping_only,
350 ErrorCode expected_code,
351 metrics::CheckResult expected_check_result,
352 metrics::CheckReaction expected_check_reaction,
353 metrics::DownloadErrorCode expected_download_error_code,
354 OmahaResponse* out_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700355 brillo::Blob* out_post_data) {
356 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700357 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700358 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800359 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700360 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700361 if (fail_http_response_code >= 0) {
362 fetcher->FailTransfer(fail_http_response_code);
363 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700364 if (request_params)
365 fake_system_state_.set_request_params(request_params);
366 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700367 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -0300368 brillo::make_unique_ptr(fetcher),
Darin Petkov265f2902011-05-09 15:17:40 -0700369 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700370 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700371 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700372
Darin Petkov6a5b3222010-07-13 14:55:28 -0700373 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700374 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700375 processor.EnqueueAction(&action);
376
377 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700378 BondActions(&action, &collector_action);
379 processor.EnqueueAction(&collector_action);
380
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700381 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800382 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700383 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800384 SendEnumToUMA(metrics::kMetricCheckResult,
385 static_cast<int>(expected_check_result),
386 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
387 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700388 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800389 SendEnumToUMA(metrics::kMetricCheckReaction,
390 static_cast<int>(expected_check_reaction),
391 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
392 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700393 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700394 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
395 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800396 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
397 ? 0 : 1);
398
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700399 loop.PostTask(base::Bind(
400 [](ActionProcessor* processor) { processor->StartProcessing(); },
401 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700402 loop.Run();
Alex Deymo60ca1a72015-06-18 18:19:15 -0700403 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700404 if (collector_action.has_input_object_ && out_response)
405 *out_response = collector_action.omaha_response_;
406 if (out_post_data)
407 *out_post_data = fetcher->post_data();
408 return collector_action.has_input_object_;
409}
410
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700411// Tests Event requests -- they should always succeed. |out_post_data|
412// may be null; if non-null, the post-data received by the mock
413// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700414void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700415 OmahaEvent* event,
416 const string& http_response,
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700417 brillo::Blob* out_post_data) {
418 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700419 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700420 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800421 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700422 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700423 FakeSystemState fake_system_state;
424 fake_system_state.set_request_params(&params);
Alex Deymoc1c17b42015-11-23 03:53:15 -0300425 OmahaRequestAction action(&fake_system_state,
426 event,
427 brillo::make_unique_ptr(fetcher),
428 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700429 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700430 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700431 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700432 processor.EnqueueAction(&action);
433
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700434 loop.PostTask(base::Bind(
435 [](ActionProcessor* processor) { processor->StartProcessing(); },
436 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700437 loop.Run();
Alex Deymo72a25672016-03-23 15:44:39 -0700438 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo60ca1a72015-06-18 18:19:15 -0700439
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700440 if (out_post_data)
441 *out_post_data = fetcher->post_data();
442}
443
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700444TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400445 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700446 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400447 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700448 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700449 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400450 -1,
451 false, // ping_only
452 ErrorCode::kOmahaRequestXMLHasEntityDecl,
453 metrics::CheckResult::kParsingError,
454 metrics::CheckReaction::kUnset,
455 metrics::DownloadErrorCode::kUnset,
456 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700457 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400458 EXPECT_FALSE(response.update_exists);
459}
460
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700461TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700462 OmahaResponse response;
463 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700464 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700465 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700466 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700467 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700468 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800469 metrics::CheckResult::kNoUpdateAvailable,
470 metrics::CheckReaction::kUnset,
471 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700472 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700473 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700474 EXPECT_FALSE(response.update_exists);
475}
476
Sen Jiang81259682017-03-30 15:11:30 -0700477TEST_F(OmahaRequestActionTest, MultiAppNoUpdateTest) {
478 OmahaResponse response;
479 fake_update_response_.multi_app_no_update = true;
480 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
481 fake_update_response_.GetNoUpdateResponse(),
482 -1,
483 false, // ping_only
484 ErrorCode::kSuccess,
485 metrics::CheckResult::kNoUpdateAvailable,
486 metrics::CheckReaction::kUnset,
487 metrics::DownloadErrorCode::kUnset,
488 &response,
489 nullptr));
490 EXPECT_FALSE(response.update_exists);
491}
492
Alex Deymo8e18f932015-03-27 16:16:59 -0700493// Test that all the values in the response are parsed in a normal update
494// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700495TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700496 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700497 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700498 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700499 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700500 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700501 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700502 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700503 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800504 metrics::CheckResult::kUpdateAvailable,
505 metrics::CheckReaction::kUpdating,
506 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700507 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700508 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700509 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700510 EXPECT_EQ(fake_update_response_.version, response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800511 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
512 response.packages[0].payload_urls[0]);
Alex Deymo8e18f932015-03-27 16:16:59 -0700513 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800514 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
515 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Alex Deymo8e18f932015-03-27 16:16:59 -0700516 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
517 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
518 // Omaha cohort attribets are not set in the response, so they should not be
519 // persisted.
520 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
521 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
522 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700523}
524
Sen Jiang0affc2c2017-02-10 15:55:05 -0800525TEST_F(OmahaRequestActionTest, MultiPackageUpdateTest) {
526 OmahaResponse response;
527 fake_update_response_.multi_package = true;
528 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
529 fake_update_response_.GetUpdateResponse(),
530 -1,
531 false, // ping_only
532 ErrorCode::kSuccess,
533 metrics::CheckResult::kUpdateAvailable,
534 metrics::CheckReaction::kUpdating,
535 metrics::DownloadErrorCode::kUnset,
536 &response,
537 nullptr));
538 EXPECT_TRUE(response.update_exists);
539 EXPECT_EQ(fake_update_response_.version, response.version);
540 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
541 response.packages[0].payload_urls[0]);
542 EXPECT_EQ(fake_update_response_.codebase + "package2",
543 response.packages[1].payload_urls[0]);
544 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
545 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Sen Jiang81259682017-03-30 15:11:30 -0700546 EXPECT_EQ(11u, response.packages[0].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800547 ASSERT_EQ(2u, response.packages.size());
Sen Jiang81259682017-03-30 15:11:30 -0700548 EXPECT_EQ(string("hash2"), response.packages[1].hash);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800549 EXPECT_EQ(222u, response.packages[1].size);
Sen Jiang81259682017-03-30 15:11:30 -0700550 EXPECT_EQ(22u, response.packages[1].metadata_size);
551}
552
553TEST_F(OmahaRequestActionTest, MultiAppUpdateTest) {
554 OmahaResponse response;
555 fake_update_response_.multi_app = true;
556 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
557 fake_update_response_.GetUpdateResponse(),
558 -1,
559 false, // ping_only
560 ErrorCode::kSuccess,
561 metrics::CheckResult::kUpdateAvailable,
562 metrics::CheckReaction::kUpdating,
563 metrics::DownloadErrorCode::kUnset,
564 &response,
565 nullptr));
566 EXPECT_TRUE(response.update_exists);
567 EXPECT_EQ(fake_update_response_.version, response.version);
568 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
569 response.packages[0].payload_urls[0]);
570 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
571 response.packages[1].payload_urls[0]);
572 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
573 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
574 EXPECT_EQ(11u, response.packages[0].metadata_size);
575 ASSERT_EQ(2u, response.packages.size());
576 EXPECT_EQ(string("hash3"), response.packages[1].hash);
577 EXPECT_EQ(333u, response.packages[1].size);
578 EXPECT_EQ(33u, response.packages[1].metadata_size);
579}
580
581TEST_F(OmahaRequestActionTest, MultiAppPartialUpdateTest) {
582 OmahaResponse response;
583 fake_update_response_.multi_app_no_update = true;
584 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
585 fake_update_response_.GetUpdateResponse(),
586 -1,
587 false, // ping_only
588 ErrorCode::kSuccess,
589 metrics::CheckResult::kUpdateAvailable,
590 metrics::CheckReaction::kUpdating,
591 metrics::DownloadErrorCode::kUnset,
592 &response,
593 nullptr));
594 EXPECT_TRUE(response.update_exists);
595 EXPECT_EQ(fake_update_response_.version, response.version);
596 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
597 response.packages[0].payload_urls[0]);
598 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
599 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
600 EXPECT_EQ(11u, response.packages[0].metadata_size);
601 ASSERT_EQ(1u, response.packages.size());
602}
603
604TEST_F(OmahaRequestActionTest, MultiAppMultiPackageUpdateTest) {
605 OmahaResponse response;
606 fake_update_response_.multi_app = true;
607 fake_update_response_.multi_package = true;
608 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
609 fake_update_response_.GetUpdateResponse(),
610 -1,
611 false, // ping_only
612 ErrorCode::kSuccess,
613 metrics::CheckResult::kUpdateAvailable,
614 metrics::CheckReaction::kUpdating,
615 metrics::DownloadErrorCode::kUnset,
616 &response,
617 nullptr));
618 EXPECT_TRUE(response.update_exists);
619 EXPECT_EQ(fake_update_response_.version, response.version);
620 EXPECT_EQ(fake_update_response_.GetPayloadUrl(),
621 response.packages[0].payload_urls[0]);
622 EXPECT_EQ(fake_update_response_.codebase + "package2",
623 response.packages[1].payload_urls[0]);
624 EXPECT_EQ(fake_update_response_.codebase2 + "package3",
625 response.packages[2].payload_urls[0]);
626 EXPECT_EQ(fake_update_response_.hash, response.packages[0].hash);
627 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
628 EXPECT_EQ(11u, response.packages[0].metadata_size);
629 ASSERT_EQ(3u, response.packages.size());
630 EXPECT_EQ(string("hash2"), response.packages[1].hash);
631 EXPECT_EQ(222u, response.packages[1].size);
632 EXPECT_EQ(22u, response.packages[1].metadata_size);
633 EXPECT_EQ(string("hash3"), response.packages[2].hash);
634 EXPECT_EQ(333u, response.packages[2].size);
635 EXPECT_EQ(33u, response.packages[2].metadata_size);
Sen Jiang0affc2c2017-02-10 15:55:05 -0800636}
637
Alex Deymo14ad88e2016-06-29 12:30:14 -0700638TEST_F(OmahaRequestActionTest, ExtraHeadersSentTest) {
639 const string http_response = "<?xml invalid response";
640 request_params_.set_interactive(true);
641
642 brillo::FakeMessageLoop loop(nullptr);
643 loop.SetAsCurrent();
644
645 MockHttpFetcher* fetcher =
646 new MockHttpFetcher(http_response.data(), http_response.size(), nullptr);
647 OmahaRequestAction action(
648 &fake_system_state_, nullptr, brillo::make_unique_ptr(fetcher), false);
649 ActionProcessor processor;
650 processor.EnqueueAction(&action);
651
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700652 loop.PostTask(base::Bind(
653 [](ActionProcessor* processor) { processor->StartProcessing(); },
654 base::Unretained(&processor)));
Alex Deymo14ad88e2016-06-29 12:30:14 -0700655 loop.Run();
656 EXPECT_FALSE(loop.PendingTasks());
657
658 // Check that the headers were set in the fetcher during the action. Note that
659 // we set this request as "interactive".
660 EXPECT_EQ("fg", fetcher->GetHeader("X-GoogleUpdate-Interactivity"));
661 EXPECT_EQ(kTestAppId, fetcher->GetHeader("X-GoogleUpdate-AppId"));
662 EXPECT_NE("", fetcher->GetHeader("X-GoogleUpdate-Updater"));
663}
664
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700665TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700666 OmahaResponse response;
667 // Set up a connection manager that doesn't allow a valid update over
668 // the current ethernet connection.
Alex Deymof6ee0162015-07-31 12:35:22 -0700669 MockConnectionManager mock_cm;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700670 fake_system_state_.set_connection_manager(&mock_cm);
671
Alex Deymo30534502015-07-20 15:06:33 -0700672 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _))
673 .WillRepeatedly(
Sen Jiang255e22b2016-05-20 16:15:29 -0700674 DoAll(SetArgumentPointee<0>(ConnectionType::kEthernet),
675 SetArgumentPointee<1>(ConnectionTethering::kUnknown),
Alex Deymo30534502015-07-20 15:06:33 -0700676 Return(true)));
Sen Jiang255e22b2016-05-20 16:15:29 -0700677 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(ConnectionType::kEthernet, _))
678 .WillRepeatedly(Return(false));
Chris Sosa77f79e82014-06-02 18:16:24 -0700679
680 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700681 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700682 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700683 -1,
684 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700685 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700686 metrics::CheckResult::kUpdateAvailable,
687 metrics::CheckReaction::kIgnored,
688 metrics::DownloadErrorCode::kUnset,
689 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700690 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700691 EXPECT_FALSE(response.update_exists);
692}
693
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700694TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700695 string rollback_version = "1234.0.0";
696 OmahaResponse response;
697
698 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700699 fake_system_state_.set_payload_state(&mock_payload_state);
700
Chris Sosa77f79e82014-06-02 18:16:24 -0700701 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
702 .WillRepeatedly(Return(rollback_version));
703
Alex Deymo8e18f932015-03-27 16:16:59 -0700704 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700705 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700706 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700707 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700708 -1,
709 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700710 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700711 metrics::CheckResult::kUpdateAvailable,
712 metrics::CheckReaction::kIgnored,
713 metrics::DownloadErrorCode::kUnset,
714 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700715 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700716 EXPECT_FALSE(response.update_exists);
717}
718
Kevin Cernekee2494e282016-03-29 18:03:53 -0700719// Verify that update checks called during OOBE will only try to download
720// an update if the response includes a non-empty deadline field.
721TEST_F(OmahaRequestActionTest, SkipNonCriticalUpdatesBeforeOOBE) {
722 OmahaResponse response;
723
724 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
725 ASSERT_FALSE(
726 TestUpdateCheck(nullptr, // request_params
727 fake_update_response_.GetUpdateResponse(),
728 -1,
729 false, // ping_only
730 ErrorCode::kNonCriticalUpdateInOOBE,
731 metrics::CheckResult::kUnset,
732 metrics::CheckReaction::kUnset,
733 metrics::DownloadErrorCode::kUnset,
734 &response,
735 nullptr));
736 EXPECT_FALSE(response.update_exists);
737
Alex Deymo46a9aae2016-05-04 20:20:11 -0700738 // The IsOOBEComplete() value is ignored when the OOBE flow is not enabled.
739 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(false);
740 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
741 fake_update_response_.GetUpdateResponse(),
742 -1,
743 false, // ping_only
744 ErrorCode::kSuccess,
745 metrics::CheckResult::kUpdateAvailable,
746 metrics::CheckReaction::kUpdating,
747 metrics::DownloadErrorCode::kUnset,
748 &response,
749 nullptr));
750 EXPECT_TRUE(response.update_exists);
751 fake_system_state_.fake_hardware()->SetIsOOBEEnabled(true);
752
753 // The payload is applied when a deadline was set in the response.
Kevin Cernekee2494e282016-03-29 18:03:53 -0700754 fake_update_response_.deadline = "20101020";
755 ASSERT_TRUE(
756 TestUpdateCheck(nullptr, // request_params
757 fake_update_response_.GetUpdateResponse(),
758 -1,
759 false, // ping_only
760 ErrorCode::kSuccess,
761 metrics::CheckResult::kUpdateAvailable,
762 metrics::CheckReaction::kUpdating,
763 metrics::DownloadErrorCode::kUnset,
764 &response,
765 nullptr));
766 EXPECT_TRUE(response.update_exists);
767}
768
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700769TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700770 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700771 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700772 params.set_wall_clock_based_wait_enabled(true);
773 params.set_update_check_count_wait_enabled(false);
774 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700775
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700776 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700777 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700778 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700779 -1,
780 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700781 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800782 metrics::CheckResult::kUpdateAvailable,
783 metrics::CheckReaction::kDeferring,
784 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700785 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700786 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700787 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700788
789 // Verify if we are interactive check we don't defer.
790 params.set_interactive(true);
791 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700792 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700793 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700794 -1,
795 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700796 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800797 metrics::CheckResult::kUpdateAvailable,
798 metrics::CheckReaction::kUpdating,
799 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700800 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700801 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700802 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700803}
804
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700805TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700806 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700807 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700808 params.set_wall_clock_based_wait_enabled(false);
809 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700810
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700811 params.set_update_check_count_wait_enabled(true);
812 params.set_min_update_checks_needed(1);
813 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700814
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700815 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700816 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700817 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700818 -1,
819 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700820 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800821 metrics::CheckResult::kUpdateAvailable,
822 metrics::CheckReaction::kUpdating,
823 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700824 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700825 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700826 EXPECT_TRUE(response.update_exists);
827}
828
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700829TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700830 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700831 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700832 params.set_wall_clock_based_wait_enabled(true);
833 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700834
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700835 params.set_update_check_count_wait_enabled(true);
836 params.set_min_update_checks_needed(1);
837 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700838
Alex Deymo8e18f932015-03-27 16:16:59 -0700839 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700840 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700841 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700842 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700843 -1,
844 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700845 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800846 metrics::CheckResult::kUpdateAvailable,
847 metrics::CheckReaction::kUpdating,
848 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700849 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700850 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700851 EXPECT_TRUE(response.update_exists);
852}
853
854
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700855TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700856 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700857 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700858 params.set_wall_clock_based_wait_enabled(true);
859 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700860
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700861 params.set_update_check_count_wait_enabled(true);
862 params.set_min_update_checks_needed(0);
863 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700864
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700865 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700866 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700867 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700868 -1,
869 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700870 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800871 metrics::CheckResult::kUpdateAvailable,
872 metrics::CheckReaction::kUpdating,
873 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700874 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700875 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700876
Ben Chan9abb7632014-08-07 00:10:53 -0700877 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700878 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700879 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700880 EXPECT_TRUE(response.update_exists);
881}
882
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700883TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700884 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700885 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700886 params.set_wall_clock_based_wait_enabled(true);
887 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700888
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700889 params.set_update_check_count_wait_enabled(true);
890 params.set_min_update_checks_needed(1);
891 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700892
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700893 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700894 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700895 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700896 -1,
897 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700898 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800899 metrics::CheckResult::kUpdateAvailable,
900 metrics::CheckReaction::kDeferring,
901 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700902 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700903 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700904
Ben Chan9abb7632014-08-07 00:10:53 -0700905 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700906 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700907 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700908 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700909
910 // Verify if we are interactive check we don't defer.
911 params.set_interactive(true);
912 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700913 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700914 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700915 -1,
916 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700917 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800918 metrics::CheckResult::kUpdateAvailable,
919 metrics::CheckReaction::kUpdating,
920 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700921 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700922 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700923 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700924}
925
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700926TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700927 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700928 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700929 params.set_wall_clock_based_wait_enabled(true);
930 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700931
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700932 params.set_update_check_count_wait_enabled(true);
933 params.set_min_update_checks_needed(1);
934 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700935
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700936 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700937
938 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700939 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700940 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700941 -1,
942 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700943 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800944 metrics::CheckResult::kUpdateAvailable,
945 metrics::CheckReaction::kDeferring,
946 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700947 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700948 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700949
Ben Chan9abb7632014-08-07 00:10:53 -0700950 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700951 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700952 // count remains the same, as the decrementing happens in update_attempter
953 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700954 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700955 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700956
957 // Verify if we are interactive check we don't defer.
958 params.set_interactive(true);
959 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700960 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700961 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700962 -1,
963 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700964 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800965 metrics::CheckResult::kUpdateAvailable,
966 metrics::CheckReaction::kUpdating,
967 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700968 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700969 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700970 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700971}
Jay Srinivasan0a708742012-03-20 11:26:12 -0700972
Alex Deymo8e18f932015-03-27 16:16:59 -0700973TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
974 OmahaResponse response;
975 OmahaRequestParams params = request_params_;
976 fake_update_response_.include_cohorts = true;
977 fake_update_response_.cohort = "s/154454/8479665";
978 fake_update_response_.cohorthint = "please-put-me-on-beta";
979 fake_update_response_.cohortname = "stable";
980
981 ASSERT_TRUE(TestUpdateCheck(&params,
982 fake_update_response_.GetUpdateResponse(),
983 -1,
984 false, // ping_only
985 ErrorCode::kSuccess,
986 metrics::CheckResult::kUpdateAvailable,
987 metrics::CheckReaction::kUpdating,
988 metrics::DownloadErrorCode::kUnset,
989 &response,
990 nullptr));
991
992 string value;
993 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
994 EXPECT_EQ(fake_update_response_.cohort, value);
995
996 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
997 EXPECT_EQ(fake_update_response_.cohorthint, value);
998
999 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1000 EXPECT_EQ(fake_update_response_.cohortname, value);
1001}
1002
1003TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
1004 OmahaResponse response;
1005 OmahaRequestParams params = request_params_;
1006 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1007 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
1008 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
1009 fake_update_response_.include_cohorts = true;
1010 fake_update_response_.cohort = "s/154454/8479665";
1011 fake_update_response_.cohorthint = "please-put-me-on-beta";
1012 fake_update_response_.cohortname = "";
1013
1014 ASSERT_TRUE(TestUpdateCheck(&params,
1015 fake_update_response_.GetUpdateResponse(),
1016 -1,
1017 false, // ping_only
1018 ErrorCode::kSuccess,
1019 metrics::CheckResult::kUpdateAvailable,
1020 metrics::CheckReaction::kUpdating,
1021 metrics::DownloadErrorCode::kUnset,
1022 &response,
1023 nullptr));
1024
1025 string value;
1026 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1027 EXPECT_EQ(fake_update_response_.cohort, value);
1028
1029 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1030 EXPECT_EQ(fake_update_response_.cohorthint, value);
1031
1032 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1033}
1034
1035TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
1036 OmahaResponse response;
1037 OmahaRequestParams params = request_params_;
1038 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
1039
1040 ASSERT_TRUE(TestUpdateCheck(&params,
1041 fake_update_response_.GetUpdateResponse(),
1042 -1,
1043 false, // ping_only
1044 ErrorCode::kSuccess,
1045 metrics::CheckResult::kUpdateAvailable,
1046 metrics::CheckReaction::kUpdating,
1047 metrics::DownloadErrorCode::kUnset,
1048 &response,
1049 nullptr));
1050
1051 string value;
1052 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1053 EXPECT_EQ("old_value", value);
1054
1055 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1056 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1057}
1058
Alex Deymo00d79ac2015-06-29 15:41:49 -07001059TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
1060 OmahaResponse response;
1061 OmahaRequestParams params = request_params_;
1062 fake_update_response_.include_cohorts = true;
1063 fake_update_response_.cohort = "s/154454/8479665";
1064 fake_update_response_.cohorthint = "please-put-me-on-beta";
1065 fake_update_response_.cohortname = "stable";
1066
1067 ASSERT_TRUE(TestUpdateCheck(&params,
1068 fake_update_response_.GetNoUpdateResponse(),
1069 -1,
1070 false, // ping_only
1071 ErrorCode::kSuccess,
1072 metrics::CheckResult::kNoUpdateAvailable,
1073 metrics::CheckReaction::kUnset,
1074 metrics::DownloadErrorCode::kUnset,
1075 &response,
1076 nullptr));
1077
1078 string value;
1079 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
1080 EXPECT_EQ(fake_update_response_.cohort, value);
1081
1082 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
1083 EXPECT_EQ(fake_update_response_.cohorthint, value);
1084
1085 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
1086 EXPECT_EQ(fake_update_response_.cohortname, value);
1087}
1088
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001089TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001090 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001091
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001092 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001093 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001094
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001095 OmahaRequestParams params = request_params_;
1096 fake_system_state_.set_request_params(&params);
1097 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001098 brillo::make_unique_ptr(
1099 new MockHttpFetcher(http_response.data(),
1100 http_response.size(),
1101 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001102 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001103 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001104 ActionProcessor processor;
1105 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001106 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001107
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -07001108 loop.PostTask(base::Bind(
1109 [](ActionProcessor* processor) { processor->StartProcessing(); },
1110 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -07001111 loop.Run();
1112 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001113 EXPECT_FALSE(processor.IsRunning());
1114}
1115
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001116TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001117 OmahaResponse response;
1118 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001119 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001120 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001121 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001122 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001123 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001124 metrics::CheckResult::kParsingError,
1125 metrics::CheckReaction::kUnset,
1126 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -07001127 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001128 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001129 EXPECT_FALSE(response.update_exists);
1130}
1131
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001132TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001133 OmahaResponse response;
1134 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001135 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001136 "",
1137 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001138 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001139 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001140 metrics::CheckResult::kParsingError,
1141 metrics::CheckReaction::kUnset,
1142 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001143 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001144 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001145 EXPECT_FALSE(response.update_exists);
1146}
1147
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001148TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001149 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001150 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001151 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001152 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1153 "<daystart elapsed_seconds=\"100\"/>"
1154 "<app appid=\"foo\" status=\"ok\">"
1155 "<ping status=\"ok\"/>"
1156 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001157 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001158 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001159 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001160 metrics::CheckResult::kParsingError,
1161 metrics::CheckReaction::kUnset,
1162 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001163 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001164 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001165 EXPECT_FALSE(response.update_exists);
1166}
1167
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001168TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001169 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001170 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001171 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001172 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1173 "<daystart elapsed_seconds=\"100\"/>"
1174 "<app appid=\"foo\" status=\"ok\">"
1175 "<ping status=\"ok\"/>"
1176 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001177 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001178 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001179 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001180 metrics::CheckResult::kParsingError,
1181 metrics::CheckReaction::kUnset,
1182 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001183 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001184 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001185 EXPECT_FALSE(response.update_exists);
1186}
1187
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001188TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001189 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001190 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001191 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001192 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1193 "<daystart elapsed_seconds=\"100\"/>"
1194 "<app appid=\"foo\" status=\"ok\">"
1195 "<ping status=\"ok\"/>"
1196 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001197 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001198 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001199 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -08001200 metrics::CheckResult::kParsingError,
1201 metrics::CheckReaction::kUnset,
1202 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001203 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001204 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001205 EXPECT_FALSE(response.update_exists);
1206}
1207
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001208TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001209 string input_response =
1210 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
1211 "<daystart elapsed_seconds=\"100\"/>"
1212 "<app appid=\"xyz\" status=\"ok\">"
1213 "<updatecheck status=\"ok\">"
1214 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -07001215 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001216 "<packages><package hash=\"not-used\" name=\"f\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001217 "size=\"587\" hash_sha256=\"lkq34j5345\"/></packages>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001218 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001219 "ChromeOSVersion=\"10.2.3.4\" "
1220 "Prompt=\"false\" "
1221 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -08001222 "IsDeltaPayload=\"false\" "
Sen Jiang2703ef42017-03-16 13:36:21 -07001223 "sha256=\"not-used\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001224 "needsadmin=\"true\" "
1225 "/></actions></manifest></updatecheck></app></response>";
1226 LOG(INFO) << "Input Response = " << input_response;
1227
Darin Petkov6a5b3222010-07-13 14:55:28 -07001228 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001229 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001230 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -07001231 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001232 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001233 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001234 metrics::CheckResult::kUpdateAvailable,
1235 metrics::CheckReaction::kUpdating,
1236 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001237 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001238 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001239 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -07001240 EXPECT_EQ("10.2.3.4", response.version);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001241 EXPECT_EQ("http://missing/field/test/f",
1242 response.packages[0].payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001243 EXPECT_EQ("", response.more_info_url);
Sen Jiang0affc2c2017-02-10 15:55:05 -08001244 EXPECT_EQ("lkq34j5345", response.packages[0].hash);
1245 EXPECT_EQ(587u, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001246 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -07001247 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001248}
1249
1250namespace {
1251class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
1252 public:
1253 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001254 brillo::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001255 }
Darin Petkov6a5b3222010-07-13 14:55:28 -07001256};
1257
Alex Deymo60ca1a72015-06-18 18:19:15 -07001258void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001259 processor->StartProcessing();
1260 CHECK(processor->IsRunning());
1261 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001262}
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001263} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -07001264
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001265TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001266 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -07001267 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -07001268
Alex Deymo60ca1a72015-06-18 18:19:15 -07001269 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001270 OmahaRequestAction action(&fake_system_state_, nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001271 brillo::make_unique_ptr(
1272 new MockHttpFetcher(http_response.data(),
1273 http_response.size(),
1274 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001275 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001276 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001277 ActionProcessor processor;
1278 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001279 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001280
Alex Deymo60ca1a72015-06-18 18:19:15 -07001281 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1282 loop.Run();
1283 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001284}
1285
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001286TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001287 string output;
1288 EXPECT_TRUE(XmlEncode("ab", &output));
1289 EXPECT_EQ("ab", output);
1290 EXPECT_TRUE(XmlEncode("a<b", &output));
1291 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001292 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1293 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001294 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1295 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001296 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001297 EXPECT_FALSE(XmlEncode("\xc2", &output));
1298 // Fail with invalid ASCII-7 chars.
1299 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1300}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001301
Alex Deymob0d74eb2015-03-30 17:59:17 -07001302TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1303 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1304 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1305}
1306
1307TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001308 brillo::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001309
1310 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001311 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001312 constants::kOmahaPlatformName,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001313 OmahaRequestParams::kOsVersion,
1314 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001315 "x86 generic<id",
Alex Deymo85616652015-10-15 18:48:31 -07001316 kTestAppId,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001317 "0.1.0.0",
1318 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001319 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001320 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001321 "ChromeOSFirmware.1.0",
1322 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001323 false, // delta okay
1324 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001325 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001326 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001327 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1328 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1329 fake_prefs_.SetString(kPrefsOmahaCohortName,
Alex Vakulenko0103c362016-01-20 07:56:15 -08001330 base::JoinString(
1331 vector<string>(100, "My spoon is too big."), " "));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001332 OmahaResponse response;
1333 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001334 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001335 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001336 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001337 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001338 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001339 metrics::CheckResult::kParsingError,
1340 metrics::CheckReaction::kUnset,
1341 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001342 &response,
1343 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001344 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001345 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001346 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1347 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1348 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1349 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1350 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1351 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1352 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1353 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1354 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1355 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1356 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1357 // Values from Prefs that are too big are removed from the XML instead of
1358 // encoded.
1359 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001360}
1361
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001362TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001363 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001364 fake_update_response_.deadline = "&lt;20110101";
1365 fake_update_response_.more_info_url = "testthe&lt;url";
1366 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001367 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001368 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001369 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001370 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001371 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001372 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001373 metrics::CheckResult::kUpdateAvailable,
1374 metrics::CheckReaction::kUpdating,
1375 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001376 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001377 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001378
Sen Jiang0affc2c2017-02-10 15:55:05 -08001379 EXPECT_EQ("testthe<url", response.more_info_url);
1380 EXPECT_EQ("testthe&codebase/file.signed",
1381 response.packages[0].payload_urls[0]);
1382 EXPECT_EQ("<20110101", response.deadline);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001383}
1384
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001385TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001386 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001387 // overflows int32_t:
Sen Jiang0affc2c2017-02-10 15:55:05 -08001388 fake_update_response_.size = 123123123123123ull;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001389 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001390 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001391 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001392 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001393 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001394 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001395 metrics::CheckResult::kUpdateAvailable,
1396 metrics::CheckReaction::kUpdating,
1397 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001398 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001399 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001400
Sen Jiang0affc2c2017-02-10 15:55:05 -08001401 EXPECT_EQ(fake_update_response_.size, response.packages[0].size);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001402}
1403
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001404TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001405 brillo::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001406 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001407 fake_system_state_.set_prefs(&prefs);
1408
Darin Petkov95508da2011-01-05 12:42:29 -08001409 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1410 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Alex Deymoefb9d832015-11-02 18:39:02 -08001411 // An existing but empty previous version means that we didn't reboot to a new
1412 // update, therefore, no need to update the previous version.
1413 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(0);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001414 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001415 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001416 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001417 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001418 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001419 metrics::CheckResult::kParsingError,
1420 metrics::CheckReaction::kUnset,
1421 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001422 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001423 &post_data));
1424 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001425 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001426 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001427 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001428 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001429 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001430 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1431 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001432 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1433 string::npos);
1434 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1435 string::npos);
Alex Deymoefb9d832015-11-02 18:39:02 -08001436 // No <event> tag should be sent if we didn't reboot to an update.
1437 EXPECT_EQ(post_str.find("<event"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001438}
1439
Jay Srinivasan0a708742012-03-20 11:26:12 -07001440
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001441TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001442 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001443 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001444 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1445 "invalid xml>",
1446 &post_data);
1447 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001448 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001449 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001450 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001451 OmahaEvent::kTypeUpdateDownloadStarted,
1452 OmahaEvent::kResultSuccess);
1453 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001454 EXPECT_EQ(post_str.find("ping"), string::npos);
1455 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001456}
1457
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001458TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001459 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001460 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001461 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1462 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001463 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001464 "invalid xml>",
1465 &post_data);
1466 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001467 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001468 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001469 " <event eventtype=\"%d\" eventresult=\"%d\" "
1470 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001471 OmahaEvent::kTypeDownloadComplete,
1472 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001473 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001474 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001475 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001476}
1477
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001478TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001479 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001480 // Create a copy of the OmahaRequestParams to reuse it later.
1481 OmahaRequestParams params = request_params_;
1482 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001483 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001484 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001485 nullptr,
Alex Deymoc1c17b42015-11-23 03:53:15 -03001486 brillo::make_unique_ptr(
1487 new MockHttpFetcher(http_response.data(),
1488 http_response.size(),
1489 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001490 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001491 EXPECT_FALSE(update_check_action.IsEvent());
1492
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001493 params = request_params_;
1494 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001495 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001496 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001497 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Alex Deymoc1c17b42015-11-23 03:53:15 -03001498 brillo::make_unique_ptr(
1499 new MockHttpFetcher(http_response.data(),
1500 http_response.size(),
1501 nullptr)),
Thieu Le116fda32011-04-19 11:01:54 -07001502 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001503 EXPECT_TRUE(event_action.IsEvent());
1504}
1505
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001506TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001507 for (int i = 0; i < 2; i++) {
1508 bool delta_okay = i == 1;
1509 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001510 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001511 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001512 constants::kOmahaPlatformName,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001513 OmahaRequestParams::kOsVersion,
1514 "service_pack",
1515 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001516 kTestAppId,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001517 "0.1.0.0",
1518 "en-US",
1519 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001520 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001521 "ChromeOSFirmware.1.0",
1522 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001523 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001524 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001525 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001526 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001527 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001528 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001529 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001530 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001531 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001532 metrics::CheckResult::kParsingError,
1533 metrics::CheckReaction::kUnset,
1534 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001535 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001536 &post_data));
1537 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001538 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001539 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1540 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001541 string::npos)
1542 << "i = " << i;
1543 }
1544}
1545
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001546TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001547 for (int i = 0; i < 2; i++) {
1548 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001549 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001550 brillo::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001551 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001552 OmahaRequestParams params(&fake_system_state_,
Alex Deymoac41a822015-09-15 20:52:53 -07001553 constants::kOmahaPlatformName,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001554 OmahaRequestParams::kOsVersion,
1555 "service_pack",
1556 "x86-generic",
Alex Deymo85616652015-10-15 18:48:31 -07001557 kTestAppId,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001558 "0.1.0.0",
1559 "en-US",
1560 "unittest_track",
1561 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001562 "ChromeOSFirmware.1.0",
1563 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001564 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001565 interactive,
1566 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001567 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001568 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001569 "invalid xml>",
1570 -1,
1571 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001572 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001573 metrics::CheckResult::kParsingError,
1574 metrics::CheckReaction::kUnset,
1575 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001576 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001577 &post_data));
1578 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001579 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001580 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1581 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001582 string::npos)
1583 << "i = " << i;
1584 }
1585}
1586
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001587TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001588 OmahaEvent default_event;
1589 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1590 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001591 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001592
1593 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1594 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1595 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001596 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001597
1598 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1599 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001600 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001601 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1602 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001603 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001604}
1605
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001606void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001607 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001608 fake_system_state_.set_prefs(&prefs);
1609 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1610 .Times(AnyNumber());
1611 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1612 // Add a few hours to the day difference to test no rounding, etc.
1613 int64_t five_days_ago =
1614 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1615 int64_t six_days_ago =
1616 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1617 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1618 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1619 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1620 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1621 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1622 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001623 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001624 ASSERT_TRUE(
1625 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001626 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001627 -1,
1628 ping_only,
1629 ErrorCode::kSuccess,
1630 metrics::CheckResult::kUnset,
1631 metrics::CheckReaction::kUnset,
1632 metrics::DownloadErrorCode::kUnset,
1633 nullptr,
1634 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001635 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001636 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1637 string::npos);
1638 if (ping_only) {
1639 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1640 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1641 } else {
1642 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1643 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001644 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001645}
1646
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001647TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1648 PingTest(true /* ping_only */);
1649}
1650
1651TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1652 PingTest(false /* ping_only */);
1653}
1654
1655TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001656 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001657 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001658 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1659 .Times(AnyNumber());
1660 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001661 int64_t three_days_ago =
1662 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1663 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001664 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1665 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001666 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1667 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1668 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1669 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001670 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001671 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001672 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001673 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001674 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001675 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001676 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001677 metrics::CheckResult::kNoUpdateAvailable,
1678 metrics::CheckReaction::kUnset,
1679 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001680 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001681 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001682 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001683 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001684 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001685}
1686
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001687TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001688 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001689 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001690 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1691 .Times(AnyNumber());
1692 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001693 int64_t four_days_ago =
1694 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1695 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001696 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1697 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001698 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1699 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1700 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1701 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001702 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001703 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001704 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001705 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001706 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001707 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001708 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001709 metrics::CheckResult::kNoUpdateAvailable,
1710 metrics::CheckReaction::kUnset,
1711 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001712 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001713 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001714 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001715 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001716 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001717}
1718
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001719TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001720 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001721 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001722 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1723 .Times(AnyNumber());
1724 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001725 int64_t one_hour_ago =
1726 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001727 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1728 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001729 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1730 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1731 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1732 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001733 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1734 // send a ping.
1735 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1736 .WillOnce(Return(true));
1737 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1738 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001739 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001740 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001741 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001742 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001743 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001744 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001745 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001746 metrics::CheckResult::kNoUpdateAvailable,
1747 metrics::CheckReaction::kUnset,
1748 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001749 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001750 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001751 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001752 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001753}
1754
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001755TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001756 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001757 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001758 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001759 int64_t now = Time::Now().ToInternalValue();
1760 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1761 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1762 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1763 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1764 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1765 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001766 brillo::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001767 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001768 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001769 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001770 -1,
1771 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001772 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001773 metrics::CheckResult::kUnset,
1774 metrics::CheckReaction::kUnset,
1775 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001776 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001777 &post_data));
Alex Deymo80f70ff2016-02-10 16:08:11 -08001778 EXPECT_EQ(0U, post_data.size());
Thieu Leb44e9e82011-06-06 14:34:04 -07001779}
1780
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001781TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001782 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001783 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001784 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1785 .Times(AnyNumber());
1786 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001787 int64_t future =
1788 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001789 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1790 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001791 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1792 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1793 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1794 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1795 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1796 .WillOnce(Return(true));
1797 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1798 .WillOnce(Return(true));
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001799 brillo::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001800 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001801 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001802 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1803 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001804 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001805 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001806 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001807 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001808 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001809 metrics::CheckResult::kNoUpdateAvailable,
1810 metrics::CheckReaction::kUnset,
1811 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001812 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001813 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001814 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001815 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001816}
1817
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001818TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001819 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001820 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001821 // may fail if it runs for longer than 5 seconds. It shouldn't run
1822 // that long though.
1823 int64_t midnight =
1824 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1825 int64_t midnight_slack =
1826 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001827 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001828 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001829 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1830 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001831 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1832 AllOf(Ge(midnight), Le(midnight_slack))))
1833 .WillOnce(Return(true));
1834 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1835 AllOf(Ge(midnight), Le(midnight_slack))))
1836 .WillOnce(Return(true));
1837 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001838 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001839 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1840 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001841 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001842 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001843 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001844 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001845 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001846 metrics::CheckResult::kNoUpdateAvailable,
1847 metrics::CheckReaction::kUnset,
1848 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001849 nullptr,
1850 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001851}
1852
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001853TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001854 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001855 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001856 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1857 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001858 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1859 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1860 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001861 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001862 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1863 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001864 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001865 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001866 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001867 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001868 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001869 metrics::CheckResult::kNoUpdateAvailable,
1870 metrics::CheckReaction::kUnset,
1871 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001872 nullptr,
1873 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001874}
1875
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001876TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001877 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001878 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001879 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1880 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001881 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1882 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1883 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001884 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001885 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1886 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001887 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001888 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001889 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001890 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001891 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001892 metrics::CheckResult::kNoUpdateAvailable,
1893 metrics::CheckReaction::kUnset,
1894 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001895 nullptr,
1896 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001897}
1898
Alex Deymob3fa53b2016-04-18 19:57:58 -07001899TEST_F(OmahaRequestActionTest, ParseUpdateCheckAttributesTest) {
1900 // Test that the "eol" flags is only parsed from the "_eol" attribute and not
1901 // the "eol" attribute.
1902 ASSERT_TRUE(
1903 TestUpdateCheck(nullptr, // request_params
1904 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1905 "protocol=\"3.0\"><app appid=\"foo\" status=\"ok\">"
1906 "<ping status=\"ok\"/><updatecheck status=\"noupdate\" "
1907 "_eol=\"security-only\" eol=\"eol\" _foo=\"bar\"/>"
1908 "</app></response>",
1909 -1,
1910 false, // ping_only
1911 ErrorCode::kSuccess,
1912 metrics::CheckResult::kNoUpdateAvailable,
1913 metrics::CheckReaction::kUnset,
1914 metrics::DownloadErrorCode::kUnset,
1915 nullptr,
1916 nullptr));
1917 string eol_pref;
1918 EXPECT_TRUE(
1919 fake_system_state_.prefs()->GetString(kPrefsOmahaEolStatus, &eol_pref));
1920 // Note that the eol="eol" attribute should be ignored and the _eol should be
1921 // used instead.
1922 EXPECT_EQ("security-only", eol_pref);
1923}
1924
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001925TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07001926 brillo::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001927 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07001928 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001929 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001930 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001931 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001932 metrics::CheckResult::kParsingError,
1933 metrics::CheckReaction::kUnset,
1934 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001935 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07001936 &post_data));
1937 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001938 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07001939 EXPECT_EQ(post_str.find("machineid="), string::npos);
1940 EXPECT_EQ(post_str.find("userid="), string::npos);
1941}
1942
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001943TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001944 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001945 const int http_error_code =
1946 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07001947 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001948 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001949 "",
1950 501,
Darin Petkov265f2902011-05-09 15:17:40 -07001951 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001952 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001953 metrics::CheckResult::kDownloadError,
1954 metrics::CheckReaction::kUnset,
1955 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07001956 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001957 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001958 EXPECT_FALSE(response.update_exists);
1959}
1960
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001961TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001962 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001963 const int http_error_code =
1964 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07001965 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001966 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001967 "",
1968 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07001969 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001970 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001971 metrics::CheckResult::kDownloadError,
1972 metrics::CheckReaction::kUnset,
1973 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07001974 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001975 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001976 EXPECT_FALSE(response.update_exists);
1977}
1978
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001979TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001980 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001981 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001982 params.set_wall_clock_based_wait_enabled(true);
1983 params.set_waiting_period(TimeDelta().FromDays(1));
1984 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001985
Sen Jiang7c1171e2016-06-23 11:35:40 -07001986 Time arbitrary_date;
1987 Time::FromString("6/4/1989", &arbitrary_date);
1988 fake_system_state_.fake_clock()->SetWallclockTime(arbitrary_date);
1989 ASSERT_FALSE(TestUpdateCheck(&params,
1990 fake_update_response_.GetUpdateResponse(),
1991 -1,
1992 false, // ping_only
1993 ErrorCode::kOmahaUpdateDeferredPerPolicy,
1994 metrics::CheckResult::kUpdateAvailable,
1995 metrics::CheckReaction::kDeferring,
1996 metrics::DownloadErrorCode::kUnset,
1997 &response,
1998 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001999
Ben Chan9abb7632014-08-07 00:10:53 -07002000 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002001 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Sen Jiang7c1171e2016-06-23 11:35:40 -07002002 EXPECT_EQ(arbitrary_date.ToInternalValue(), timestamp);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002003 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07002004
2005 // Verify if we are interactive check we don't defer.
2006 params.set_interactive(true);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002007 ASSERT_TRUE(TestUpdateCheck(&params,
2008 fake_update_response_.GetUpdateResponse(),
2009 -1,
2010 false, // ping_only
2011 ErrorCode::kSuccess,
2012 metrics::CheckResult::kUpdateAvailable,
2013 metrics::CheckReaction::kUpdating,
2014 metrics::DownloadErrorCode::kUnset,
2015 &response,
2016 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07002017 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002018}
2019
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002020TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002021 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002022 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002023 params.set_wall_clock_based_wait_enabled(true);
2024 params.set_waiting_period(TimeDelta().FromDays(1));
2025 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002026
Sen Jiang7c1171e2016-06-23 11:35:40 -07002027 Time t1, t2;
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002028 Time::FromString("1/1/2012", &t1);
Sen Jiang7c1171e2016-06-23 11:35:40 -07002029 Time::FromString("1/3/2012", &t2);
2030 ASSERT_TRUE(
2031 fake_prefs_.SetInt64(kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
2032 fake_system_state_.fake_clock()->SetWallclockTime(t2);
2033 ASSERT_TRUE(TestUpdateCheck(&params,
2034 fake_update_response_.GetUpdateResponse(),
2035 -1,
2036 false, // ping_only
2037 ErrorCode::kSuccess,
2038 metrics::CheckResult::kUpdateAvailable,
2039 metrics::CheckReaction::kUpdating,
2040 metrics::DownloadErrorCode::kUnset,
2041 &response,
2042 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002043
2044 EXPECT_TRUE(response.update_exists);
2045
2046 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07002047 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002048 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07002049 ASSERT_TRUE(timestamp == t1.ToInternalValue());
2050}
2051
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002052TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002053 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002054 base::ScopedTempDir tempdir;
2055 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002056
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002057 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002058 OmahaRequestParams params(&fake_system_state_);
2059 params.set_root(tempdir.path().value());
2060 params.set_app_id("{22222222-2222-2222-2222-222222222222}");
2061 params.set_app_version("1.2.3.4");
2062 params.set_current_channel("canary-channel");
2063 EXPECT_TRUE(params.SetTargetChannel("stable-channel", true, nullptr));
2064 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002065 EXPECT_TRUE(params.to_more_stable_channel());
2066 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002067 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002068 "invalid xml>",
2069 -1,
2070 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002071 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002072 metrics::CheckResult::kParsingError,
2073 metrics::CheckReaction::kUnset,
2074 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002075 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002076 &post_data));
2077 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002078 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002079 EXPECT_NE(string::npos, post_str.find(
2080 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
2081 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
2082 "track=\"stable-channel\" from_track=\"canary-channel\" "));
2083}
2084
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002085TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002086 // Create a uniquely named test directory.
Sen Jiang297e5832016-03-17 14:45:51 -07002087 base::ScopedTempDir tempdir;
2088 ASSERT_TRUE(tempdir.CreateUniqueTempDir());
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07002089
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002090 brillo::Blob post_data;
Sen Jiang297e5832016-03-17 14:45:51 -07002091 OmahaRequestParams params(&fake_system_state_);
2092 params.set_root(tempdir.path().value());
2093 params.set_app_id("{11111111-1111-1111-1111-111111111111}");
2094 params.set_app_version("5.6.7.8");
2095 params.set_current_channel("stable-channel");
2096 EXPECT_TRUE(params.SetTargetChannel("canary-channel", false, nullptr));
2097 params.UpdateDownloadChannel();
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002098 EXPECT_FALSE(params.to_more_stable_channel());
2099 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002100 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002101 "invalid xml>",
2102 -1,
2103 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002104 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08002105 metrics::CheckResult::kParsingError,
2106 metrics::CheckReaction::kUnset,
2107 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002108 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002109 &post_data));
2110 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002111 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002112 EXPECT_NE(string::npos, post_str.find(
2113 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
2114 "version=\"5.6.7.8\" "
2115 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002116 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07002117}
2118
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002119// Checks that the initial ping with a=-1 r=-1 is not send when the device
2120// was powerwashed.
2121TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
2122 fake_prefs_.SetString(kPrefsPreviousVersion, "");
2123
2124 // Flag that the device was powerwashed in the past.
2125 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
2126
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -07002127 brillo::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002128 ASSERT_TRUE(
2129 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002130 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002131 -1,
2132 false, // ping_only
2133 ErrorCode::kSuccess,
2134 metrics::CheckResult::kNoUpdateAvailable,
2135 metrics::CheckReaction::kUnset,
2136 metrics::DownloadErrorCode::kUnset,
2137 nullptr,
2138 &post_data));
2139 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08002140 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07002141 EXPECT_EQ(string::npos, post_str.find("<ping"));
2142}
2143
Alex Deymo9fded1e2015-11-05 12:31:19 -08002144// Checks that the event 54 is sent on a reboot to a new update.
2145TEST_F(OmahaRequestActionTest, RebootAfterUpdateEvent) {
2146 // Flag that the device was updated in a previous boot.
2147 fake_prefs_.SetString(kPrefsPreviousVersion, "1.2.3.4");
2148
2149 brillo::Blob post_data;
2150 ASSERT_TRUE(
2151 TestUpdateCheck(nullptr, // request_params
2152 fake_update_response_.GetNoUpdateResponse(),
2153 -1,
2154 false, // ping_only
2155 ErrorCode::kSuccess,
2156 metrics::CheckResult::kNoUpdateAvailable,
2157 metrics::CheckReaction::kUnset,
2158 metrics::DownloadErrorCode::kUnset,
2159 nullptr,
2160 &post_data));
2161 string post_str(post_data.begin(), post_data.end());
2162
2163 // An event 54 is included and has the right version.
2164 EXPECT_NE(string::npos,
2165 post_str.find(base::StringPrintf(
2166 "<event eventtype=\"%d\"",
2167 OmahaEvent::kTypeRebootedAfterUpdate)));
2168 EXPECT_NE(string::npos,
2169 post_str.find("previousversion=\"1.2.3.4\"></event>"));
2170
2171 // The previous version flag should have been removed.
2172 EXPECT_TRUE(fake_prefs_.Exists(kPrefsPreviousVersion));
2173 string prev_version;
2174 EXPECT_TRUE(fake_prefs_.GetString(kPrefsPreviousVersion, &prev_version));
2175 EXPECT_TRUE(prev_version.empty());
2176}
2177
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002178void OmahaRequestActionTest::P2PTest(
2179 bool initial_allow_p2p_for_downloading,
2180 bool initial_allow_p2p_for_sharing,
2181 bool omaha_disable_p2p_for_downloading,
2182 bool omaha_disable_p2p_for_sharing,
2183 bool payload_state_allow_p2p_attempt,
2184 bool expect_p2p_client_lookup,
2185 const string& p2p_client_result_url,
2186 bool expected_allow_p2p_for_downloading,
2187 bool expected_allow_p2p_for_sharing,
2188 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07002189 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002190 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07002191 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
2192 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
2193 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07002194
2195 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002196 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07002197 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
2198 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07002199 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
2200 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
2201 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
2202 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
2203 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
2204 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
2205 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
2206 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
2207 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
2208 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
2209
David Zeuthen8f191b22013-08-06 12:27:50 -07002210 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002211 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07002212 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
2213
David Zeuthen4cc5ed22014-01-15 12:35:03 -08002214 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
2215 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07002216 .Times(expect_p2p_client_lookup ? 1 : 0);
2217
Alex Deymo8e18f932015-03-27 16:16:59 -07002218 fake_update_response_.disable_p2p_for_downloading =
2219 omaha_disable_p2p_for_downloading;
2220 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07002221 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002222 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07002223 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07002224 -1,
2225 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002226 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002227 metrics::CheckResult::kUpdateAvailable,
2228 metrics::CheckReaction::kUpdating,
2229 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07002230 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002231 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07002232 EXPECT_TRUE(response.update_exists);
2233
Gilad Arnold74b5f552014-10-07 08:17:16 -07002234 EXPECT_EQ(omaha_disable_p2p_for_downloading,
2235 response.disable_p2p_for_downloading);
2236 EXPECT_EQ(omaha_disable_p2p_for_sharing,
2237 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07002238
Gilad Arnold74b5f552014-10-07 08:17:16 -07002239 EXPECT_EQ(expected_allow_p2p_for_downloading,
2240 actual_allow_p2p_for_downloading);
2241 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
2242 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07002243}
2244
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002245TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002246 P2PTest(true, // initial_allow_p2p_for_downloading
2247 true, // initial_allow_p2p_for_sharing
2248 false, // omaha_disable_p2p_for_downloading
2249 false, // omaha_disable_p2p_for_sharing
2250 true, // payload_state_allow_p2p_attempt
2251 true, // expect_p2p_client_lookup
2252 "http://1.3.5.7/p2p", // p2p_client_result_url
2253 true, // expected_allow_p2p_for_downloading
2254 true, // expected_allow_p2p_for_sharing
2255 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002256}
2257
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002258TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002259 P2PTest(true, // initial_allow_p2p_for_downloading
2260 true, // initial_allow_p2p_for_sharing
2261 false, // omaha_disable_p2p_for_downloading
2262 false, // omaha_disable_p2p_for_sharing
2263 true, // payload_state_allow_p2p_attempt
2264 true, // expect_p2p_client_lookup
2265 "", // p2p_client_result_url
2266 false, // expected_allow_p2p_for_downloading
2267 true, // expected_allow_p2p_for_sharing
2268 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002269}
2270
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002271TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002272 P2PTest(false, // initial_allow_p2p_for_downloading
2273 true, // initial_allow_p2p_for_sharing
2274 false, // omaha_disable_p2p_for_downloading
2275 false, // omaha_disable_p2p_for_sharing
2276 true, // payload_state_allow_p2p_attempt
2277 false, // expect_p2p_client_lookup
2278 "unset", // p2p_client_result_url
2279 false, // expected_allow_p2p_for_downloading
2280 true, // expected_allow_p2p_for_sharing
2281 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002282}
2283
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002284TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002285 P2PTest(true, // initial_allow_p2p_for_downloading
2286 true, // initial_allow_p2p_for_sharing
2287 true, // omaha_disable_p2p_for_downloading
2288 false, // omaha_disable_p2p_for_sharing
2289 true, // payload_state_allow_p2p_attempt
2290 false, // expect_p2p_client_lookup
2291 "unset", // p2p_client_result_url
2292 false, // expected_allow_p2p_for_downloading
2293 true, // expected_allow_p2p_for_sharing
2294 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002295}
2296
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002297TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002298 P2PTest(true, // initial_allow_p2p_for_downloading
2299 true, // initial_allow_p2p_for_sharing
2300 false, // omaha_disable_p2p_for_downloading
2301 true, // omaha_disable_p2p_for_sharing
2302 true, // payload_state_allow_p2p_attempt
2303 true, // expect_p2p_client_lookup
2304 "http://1.3.5.7/p2p", // p2p_client_result_url
2305 true, // expected_allow_p2p_for_downloading
2306 false, // expected_allow_p2p_for_sharing
2307 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002308}
2309
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002310TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07002311 P2PTest(true, // initial_allow_p2p_for_downloading
2312 true, // initial_allow_p2p_for_sharing
2313 true, // omaha_disable_p2p_for_downloading
2314 true, // omaha_disable_p2p_for_sharing
2315 true, // payload_state_allow_p2p_attempt
2316 false, // expect_p2p_client_lookup
2317 "unset", // p2p_client_result_url
2318 false, // expected_allow_p2p_for_downloading
2319 false, // expected_allow_p2p_for_sharing
2320 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002321}
2322
Alex Deymof329b932014-10-30 01:37:48 -07002323bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2324 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002325 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002326 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002327 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002328 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002329 -1,
2330 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002331 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002332 metrics::CheckResult::kUpdateAvailable,
2333 metrics::CheckReaction::kUpdating,
2334 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002335 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002336 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002337}
2338
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002339TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002340 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002341
Kevin Cernekee2494e282016-03-29 18:03:53 -07002342 // Simulate a successful update check that happens during OOBE. The
2343 // deadline in the response is needed to force the update attempt to
2344 // occur; responses without a deadline seen during OOBE will normally
2345 // return ErrorCode::kNonCriticalUpdateInOOBE.
2346 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
2347 fake_update_response_.deadline = "20101020";
2348
David Zeuthen639aa362014-02-03 16:23:44 -08002349 // Check that we parse elapsed_days in the Omaha Response correctly.
2350 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002351 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2352 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002353 EXPECT_TRUE(response.update_exists);
2354 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002355 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002356 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002357 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002358 EXPECT_EQ(prefs_days, 42);
2359
2360 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002361 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002362 EXPECT_TRUE(response.update_exists);
2363 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002364 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002365 EXPECT_EQ(prefs_days, 42);
2366
2367 // Note that elapsed_days is not necessarily divisible by 7 so check
2368 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002369 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2370 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002371 EXPECT_TRUE(response.update_exists);
2372 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002373 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002374 EXPECT_EQ(prefs_days, 21);
2375
2376 // Check that we correctly handle elapsed_days not being included in
2377 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002378 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002379 EXPECT_TRUE(response.update_exists);
2380 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002381}
2382
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002383// If there is no prefs and OOBE is not complete, we should not
2384// report anything to Omaha.
2385TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
Kevin Cernekee2494e282016-03-29 18:03:53 -07002386 fake_system_state_.fake_hardware()->UnsetIsOOBEComplete();
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002387 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2388 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2389}
David Zeuthen639aa362014-02-03 16:23:44 -08002390
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002391// If OOBE is complete and happened on a valid date (e.g. after Jan
2392// 1 2007 0:00 PST), that date should be used and written to
2393// prefs. However, first try with an invalid date and check we do
2394// nothing.
2395TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2396 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2397 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2398 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2399 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2400}
David Zeuthen639aa362014-02-03 16:23:44 -08002401
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002402// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2403// should yield an InstallDate of 14.
2404TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2405 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2406 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2407 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2408 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002409
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002410 int64_t prefs_days;
2411 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2412 EXPECT_EQ(prefs_days, 14);
2413}
David Zeuthen639aa362014-02-03 16:23:44 -08002414
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002415// Now that we have a valid date in prefs, check that we keep using
2416// that even if OOBE date reports something else. The date Jan 30,
2417// 2007 0:00 PST should yield an InstallDate of 28... but since
2418// there's a prefs file, we should still get 14.
2419TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2420 // Set a valid date in the prefs first.
2421 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002422
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002423 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2424 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2425 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002426
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002427 int64_t prefs_days;
2428 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2429 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002430
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002431 // If we delete the prefs file, we should get 28 days.
2432 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2433 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2434 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2435 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002436}
2437
Darin Petkov6a5b3222010-07-13 14:55:28 -07002438} // namespace chromeos_update_engine