blob: 8f4dc87ab7f5133b40023a98195ef239d7800d73 [file] [log] [blame]
Mike Frysinger8155d082012-04-06 15:23:18 -04001// Copyright (c) 2012 The Chromium OS Authors. All rights reserved.
Darin Petkov6a5b3222010-07-13 14:55:28 -07002// Use of this source code is governed by a BSD-style license that can be
3// found in the LICENSE file.
4
Alex Deymo8427b4a2014-11-05 14:00:32 -08005#include "update_engine/omaha_request_action.h"
6
Ben Chan9abb7632014-08-07 00:10:53 -07007#include <glib.h>
8#include <stdint.h>
9
Darin Petkov6a5b3222010-07-13 14:55:28 -070010#include <string>
11#include <vector>
Darin Petkov0dc8e9a2010-07-14 14:51:57 -070012
Alex Deymo60ca1a72015-06-18 18:19:15 -070013#include <base/bind.h>
Alex Deymo8e18f932015-03-27 16:16:59 -070014#include <base/strings/string_number_conversions.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070015#include <base/strings/string_util.h>
16#include <base/strings/stringprintf.h>
17#include <base/time/time.h>
Alex Deymo60ca1a72015-06-18 18:19:15 -070018#include <chromeos/bind_lambda.h>
Chris Sosa77f79e82014-06-02 18:16:24 -070019#include <chromeos/dbus/service_constants.h>
Alex Deymo60ca1a72015-06-18 18:19:15 -070020#include <chromeos/message_loops/fake_message_loop.h>
21#include <chromeos/message_loops/message_loop.h>
22#include <chromeos/message_loops/message_loop_utils.h>
Alex Deymoe1e3afe2014-10-30 13:02:49 -070023#include <gtest/gtest.h>
Jay Srinivasand29695d2013-04-08 15:08:05 -070024
Darin Petkov6a5b3222010-07-13 14:55:28 -070025#include "update_engine/action_pipe.h"
Jay Srinivasand29695d2013-04-08 15:08:05 -070026#include "update_engine/constants.h"
Alex Deymoe1e3afe2014-10-30 13:02:49 -070027#include "update_engine/fake_prefs.h"
Chris Sosa77f79e82014-06-02 18:16:24 -070028#include "update_engine/mock_connection_manager.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070029#include "update_engine/mock_http_fetcher.h"
Gilad Arnold74b5f552014-10-07 08:17:16 -070030#include "update_engine/mock_payload_state.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070031#include "update_engine/omaha_hash_calculator.h"
Darin Petkova4a8a8c2010-07-15 22:21:12 -070032#include "update_engine/omaha_request_params.h"
Jay Srinivasan480ddfa2012-06-01 19:15:26 -070033#include "update_engine/prefs.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070034#include "update_engine/test_utils.h"
Jay Srinivasan480ddfa2012-06-01 19:15:26 -070035#include "update_engine/utils.h"
Darin Petkov6a5b3222010-07-13 14:55:28 -070036
Darin Petkov1cbd78f2010-07-29 12:38:34 -070037using base::Time;
38using base::TimeDelta;
Alex Deymo10875d92014-11-10 21:52:57 -080039using chromeos_update_engine::test_utils::System;
40using chromeos_update_engine::test_utils::WriteFileString;
Darin Petkov6a5b3222010-07-13 14:55:28 -070041using std::string;
42using std::vector;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070043using testing::AllOf;
Alex Deymof329b932014-10-30 01:37:48 -070044using testing::AnyNumber;
Jay Srinivasan34b5d862012-07-23 11:43:22 -070045using testing::DoAll;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070046using testing::Ge;
47using testing::Le;
Darin Petkov9c096d62010-11-17 14:49:04 -080048using testing::NiceMock;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070049using testing::Return;
Gilad Arnold74b5f552014-10-07 08:17:16 -070050using testing::ReturnPointee;
51using testing::SaveArg;
Darin Petkov1cbd78f2010-07-29 12:38:34 -070052using testing::SetArgumentPointee;
Alex Deymof329b932014-10-30 01:37:48 -070053using testing::_;
Darin Petkov6a5b3222010-07-13 14:55:28 -070054
Alex Deymo8e18f932015-03-27 16:16:59 -070055namespace {
56
57// This is a helper struct to allow unit tests build an update response with the
58// values they care about.
59struct FakeUpdateResponse {
60 string GetNoUpdateResponse() const {
61 string entity_str;
62 if (include_entity)
63 entity_str = "<!DOCTYPE response [<!ENTITY CrOS \"ChromeOS\">]>";
Alex Deymo00d79ac2015-06-29 15:41:49 -070064 return
65 "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" +
66 entity_str + "<response protocol=\"3.0\">"
Alex Deymo8e18f932015-03-27 16:16:59 -070067 "<daystart elapsed_seconds=\"100\"/>"
Alex Deymo00d79ac2015-06-29 15:41:49 -070068 "<app appid=\"" + app_id + "\" " +
69 (include_cohorts ? "cohort=\"" + cohort + "\" cohorthint=\"" +
70 cohorthint + "\" cohortname=\"" + cohortname + "\" " : "") +
71 " status=\"ok\">"
72 "<ping status=\"ok\"/>"
73 "<updatecheck status=\"noupdate\"/></app></response>";
Alex Deymo8e18f932015-03-27 16:16:59 -070074 }
75
76 string GetUpdateResponse() const {
77 return
78 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
79 "protocol=\"3.0\">"
80 "<daystart elapsed_seconds=\"100\"" +
81 (elapsed_days.empty() ? "" : (" elapsed_days=\"" + elapsed_days + "\""))
82 + "/>"
83 "<app appid=\"" + app_id + "\" " +
84 (include_cohorts ? "cohort=\"" + cohort + "\" cohorthint=\"" +
85 cohorthint + "\" cohortname=\"" + cohortname + "\" " : "") +
86 " status=\"ok\">"
87 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
88 "<urls><url codebase=\"" + codebase + "\"/></urls>"
89 "<manifest version=\"" + version + "\">"
90 "<packages><package hash=\"not-used\" name=\"" + filename + "\" "
91 "size=\"" + base::Int64ToString(size) + "\"/></packages>"
92 "<actions><action event=\"postinstall\" "
93 "ChromeOSVersion=\"" + version + "\" "
94 "MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt + "\" "
95 "IsDelta=\"true\" "
96 "IsDeltaPayload=\"true\" "
97 "MaxDaysToScatter=\"" + max_days_to_scatter + "\" "
98 "sha256=\"" + hash + "\" "
99 "needsadmin=\"" + needsadmin + "\" " +
100 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
101 (disable_p2p_for_downloading ?
102 "DisableP2PForDownloading=\"true\" " : "") +
103 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
104 "/></actions></manifest></updatecheck></app></response>";
105 }
106
107 // Return the payload URL, which is split in two fields in the XML response.
108 string GetPayloadUrl() {
109 return codebase + filename;
110 }
111
112 string app_id = chromeos_update_engine::OmahaRequestParams::kAppId;
113 string version = "1.2.3.4";
114 string more_info_url = "http://more/info";
115 string prompt = "true";
116 string codebase = "http://code/base/";
117 string filename = "file.signed";
118 string hash = "HASH1234=";
119 string needsadmin = "false";
120 int64_t size = 123;
121 string deadline = "";
122 string max_days_to_scatter = "7";
123 string elapsed_days = "42";
124
125 // P2P setting defaults to allowed.
126 bool disable_p2p_for_downloading = false;
127 bool disable_p2p_for_sharing = false;
128
129 // Omaha cohorts settings.
130 bool include_cohorts = false;
131 string cohort = "";
132 string cohorthint = "";
133 string cohortname = "";
134
135 // Whether to include the CrOS <!ENTITY> in the XML response.
136 bool include_entity = false;
137};
138
139} // namespace
140
Darin Petkov6a5b3222010-07-13 14:55:28 -0700141namespace chromeos_update_engine {
142
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700143class OmahaRequestActionTest : public ::testing::Test {
144 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800145 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700146 fake_system_state_.set_request_params(&request_params_);
147 fake_system_state_.set_prefs(&fake_prefs_);
148 }
149
150 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800151 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700152 // is used. |payload_state| may be null, in which case a local mock is used.
153 // |p2p_manager| may be null, in which case a local mock is used.
154 // |connection_manager| may be null, in which case a local mock is used.
155 // out_response may be null. If |fail_http_response_code| is non-negative,
156 // the transfer will fail with that code. |ping_only| is passed through to the
157 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
158 // post-data received by the mock HttpFetcher is returned.
159 //
160 // The |expected_check_result|, |expected_check_reaction| and
161 // |expected_error_code| parameters are for checking expectations
162 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
163 // UMA statistics. Use the appropriate ::kUnset value to specify that
164 // the given metric should not be reported.
165 bool TestUpdateCheck(OmahaRequestParams* request_params,
166 const string& http_response,
167 int fail_http_response_code,
168 bool ping_only,
169 ErrorCode expected_code,
170 metrics::CheckResult expected_check_result,
171 metrics::CheckReaction expected_check_reaction,
172 metrics::DownloadErrorCode expected_download_error_code,
173 OmahaResponse* out_response,
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800174 chromeos::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700175
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800176 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700177 // only a ping or also an updatecheck.
178 void PingTest(bool ping_only);
179
180 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700181 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700182 OmahaResponse *response);
183
184 // P2P test helper function.
185 void P2PTest(
186 bool initial_allow_p2p_for_downloading,
187 bool initial_allow_p2p_for_sharing,
188 bool omaha_disable_p2p_for_downloading,
189 bool omaha_disable_p2p_for_sharing,
190 bool payload_state_allow_p2p_attempt,
191 bool expect_p2p_client_lookup,
192 const string& p2p_client_result_url,
193 bool expected_allow_p2p_for_downloading,
194 bool expected_allow_p2p_for_sharing,
195 const string& expected_p2p_url);
196
197 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700198 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700199
200 // By default, all tests use these objects unless they replace them in the
201 // fake_system_state_.
202 OmahaRequestParams request_params_ = OmahaRequestParams{
203 &fake_system_state_,
204 OmahaRequestParams::kOsPlatform,
205 OmahaRequestParams::kOsVersion,
206 "service_pack",
207 "x86-generic",
208 OmahaRequestParams::kAppId,
209 "0.1.0.0",
210 "en-US",
211 "unittest",
212 "OEM MODEL 09235 7471",
213 "ChromeOSFirmware.1.0",
214 "0X0A1",
215 false, // delta okay
216 false, // interactive
217 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700218 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700219
220 FakePrefs fake_prefs_;
221};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700222
223namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700224class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
225 public:
226 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700227 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800228 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700229 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800230 void ProcessingDone(const ActionProcessor* processor,
231 ErrorCode code) override {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700232 chromeos::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700233 }
234
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800235 void ActionCompleted(ActionProcessor* processor,
236 AbstractAction* action,
237 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700238 // make sure actions always succeed
239 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700240 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700241 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700242 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700243 }
David Zeuthena99981f2013-04-29 13:42:47 -0700244 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700245};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700246} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700247
248class OutputObjectCollectorAction;
249
250template<>
251class ActionTraits<OutputObjectCollectorAction> {
252 public:
253 // Does not take an object for input
254 typedef OmahaResponse InputObjectType;
255 // On success, puts the output path on output
256 typedef NoneType OutputObjectType;
257};
258
259class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
260 public:
261 OutputObjectCollectorAction() : has_input_object_(false) {}
262 void PerformAction() {
263 // copy input object
264 has_input_object_ = HasInputObject();
265 if (has_input_object_)
266 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700267 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700268 }
269 // Should never be called
270 void TerminateProcessing() {
271 CHECK(false);
272 }
273 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700274 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700275 return "OutputObjectCollectorAction";
276 }
Alex Deymof329b932014-10-30 01:37:48 -0700277 string Type() const { return StaticType(); }
Darin Petkov6a5b3222010-07-13 14:55:28 -0700278 bool has_input_object_;
279 OmahaResponse omaha_response_;
280};
281
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700282bool OmahaRequestActionTest::TestUpdateCheck(
283 OmahaRequestParams* request_params,
284 const string& http_response,
285 int fail_http_response_code,
286 bool ping_only,
287 ErrorCode expected_code,
288 metrics::CheckResult expected_check_result,
289 metrics::CheckReaction expected_check_reaction,
290 metrics::DownloadErrorCode expected_download_error_code,
291 OmahaResponse* out_response,
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800292 chromeos::Blob* out_post_data) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700293 chromeos::FakeMessageLoop loop(nullptr);
294 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700295 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800296 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700297 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700298 if (fail_http_response_code >= 0) {
299 fetcher->FailTransfer(fail_http_response_code);
300 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700301 if (request_params)
302 fake_system_state_.set_request_params(request_params);
303 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700304 nullptr,
Thieu Le116fda32011-04-19 11:01:54 -0700305 fetcher,
Darin Petkov265f2902011-05-09 15:17:40 -0700306 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700307 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700308 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700309
Darin Petkov6a5b3222010-07-13 14:55:28 -0700310 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700311 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700312 processor.EnqueueAction(&action);
313
314 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700315 BondActions(&action, &collector_action);
316 processor.EnqueueAction(&collector_action);
317
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700318 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800319 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700320 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800321 SendEnumToUMA(metrics::kMetricCheckResult,
322 static_cast<int>(expected_check_result),
323 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
324 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700325 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800326 SendEnumToUMA(metrics::kMetricCheckReaction,
327 static_cast<int>(expected_check_reaction),
328 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
329 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700330 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700331 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
332 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800333 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
334 ? 0 : 1);
335
Alex Deymo60ca1a72015-06-18 18:19:15 -0700336 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
337 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
338 loop.Run();
339 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
340 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700341 if (collector_action.has_input_object_ && out_response)
342 *out_response = collector_action.omaha_response_;
343 if (out_post_data)
344 *out_post_data = fetcher->post_data();
345 return collector_action.has_input_object_;
346}
347
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700348// Tests Event requests -- they should always succeed. |out_post_data|
349// may be null; if non-null, the post-data received by the mock
350// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700351void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700352 OmahaEvent* event,
353 const string& http_response,
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800354 chromeos::Blob* out_post_data) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700355 chromeos::FakeMessageLoop loop(nullptr);
356 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700357 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800358 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700359 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700360 FakeSystemState fake_system_state;
361 fake_system_state.set_request_params(&params);
362 OmahaRequestAction action(&fake_system_state, event, fetcher, false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700363 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700364 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700365 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700366 processor.EnqueueAction(&action);
367
Alex Deymo60ca1a72015-06-18 18:19:15 -0700368 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
369 loop.Run();
370
371 // This test should schedule a callback to notify the crash reporter if
372 // the passed event is an error.
373 EXPECT_EQ(event->result == OmahaEvent::kResultError, loop.PendingTasks());
374
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700375 if (out_post_data)
376 *out_post_data = fetcher->post_data();
377}
378
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700379TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400380 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700381 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400382 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700383 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700384 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400385 -1,
386 false, // ping_only
387 ErrorCode::kOmahaRequestXMLHasEntityDecl,
388 metrics::CheckResult::kParsingError,
389 metrics::CheckReaction::kUnset,
390 metrics::DownloadErrorCode::kUnset,
391 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700392 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400393 EXPECT_FALSE(response.update_exists);
394}
395
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700396TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700397 OmahaResponse response;
398 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700399 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700400 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700401 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700402 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700403 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800404 metrics::CheckResult::kNoUpdateAvailable,
405 metrics::CheckReaction::kUnset,
406 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700407 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700408 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700409 EXPECT_FALSE(response.update_exists);
410}
411
Alex Deymo8e18f932015-03-27 16:16:59 -0700412// Test that all the values in the response are parsed in a normal update
413// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700414TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700415 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700416 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700417 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700418 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700419 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700420 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700421 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700422 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800423 metrics::CheckResult::kUpdateAvailable,
424 metrics::CheckReaction::kUpdating,
425 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700426 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700427 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700428 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -0700429 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700430 EXPECT_EQ(fake_update_response_.version, response.version);
431 EXPECT_EQ(fake_update_response_.GetPayloadUrl(), response.payload_urls[0]);
432 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
433 EXPECT_EQ(fake_update_response_.hash, response.hash);
434 EXPECT_EQ(fake_update_response_.size, response.size);
435 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
436 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
437 // Omaha cohort attribets are not set in the response, so they should not be
438 // persisted.
439 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
440 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
441 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700442}
443
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700444TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700445 OmahaResponse response;
446 // Set up a connection manager that doesn't allow a valid update over
447 // the current ethernet connection.
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700448 MockConnectionManager mock_cm(nullptr);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700449 fake_system_state_.set_connection_manager(&mock_cm);
450
Alex Deymo75eac7e2015-07-29 13:39:14 -0700451 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _, _)).WillRepeatedly(
452 DoAll(SetArgumentPointee<1>(NetworkConnectionType::kEthernet),
453 SetArgumentPointee<2>(NetworkTethering::kUnknown),
454 Return(true)));
455 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(NetworkConnectionType::kEthernet, _))
Chris Sosa77f79e82014-06-02 18:16:24 -0700456 .WillRepeatedly(Return(false));
Alex Deymo75eac7e2015-07-29 13:39:14 -0700457 EXPECT_CALL(mock_cm,
458 StringForConnectionType(NetworkConnectionType::kEthernet))
Chris Sosa77f79e82014-06-02 18:16:24 -0700459 .WillRepeatedly(Return(shill::kTypeEthernet));
460
461 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700462 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700463 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700464 -1,
465 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700466 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700467 metrics::CheckResult::kUpdateAvailable,
468 metrics::CheckReaction::kIgnored,
469 metrics::DownloadErrorCode::kUnset,
470 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700471 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700472 EXPECT_FALSE(response.update_exists);
473}
474
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700475TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700476 string rollback_version = "1234.0.0";
477 OmahaResponse response;
478
479 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700480 fake_system_state_.set_payload_state(&mock_payload_state);
481
Chris Sosa77f79e82014-06-02 18:16:24 -0700482 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
483 .WillRepeatedly(Return(rollback_version));
484
Alex Deymo8e18f932015-03-27 16:16:59 -0700485 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700486 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700487 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700488 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700489 -1,
490 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700491 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700492 metrics::CheckResult::kUpdateAvailable,
493 metrics::CheckReaction::kIgnored,
494 metrics::DownloadErrorCode::kUnset,
495 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700496 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700497 EXPECT_FALSE(response.update_exists);
498}
499
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700500TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700501 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700502 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700503 params.set_wall_clock_based_wait_enabled(true);
504 params.set_update_check_count_wait_enabled(false);
505 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700506
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700507 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700508 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700509 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700510 -1,
511 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700512 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800513 metrics::CheckResult::kUpdateAvailable,
514 metrics::CheckReaction::kDeferring,
515 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700516 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700517 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700518 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700519
520 // Verify if we are interactive check we don't defer.
521 params.set_interactive(true);
522 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700523 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700524 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700525 -1,
526 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700527 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800528 metrics::CheckResult::kUpdateAvailable,
529 metrics::CheckReaction::kUpdating,
530 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700531 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700532 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700533 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700534}
535
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700536TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700537 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700538 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700539 params.set_wall_clock_based_wait_enabled(false);
540 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700541
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700542 params.set_update_check_count_wait_enabled(true);
543 params.set_min_update_checks_needed(1);
544 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700545
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700546 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700547 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700548 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700549 -1,
550 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700551 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800552 metrics::CheckResult::kUpdateAvailable,
553 metrics::CheckReaction::kUpdating,
554 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700555 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700556 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700557 EXPECT_TRUE(response.update_exists);
558}
559
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700560TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700561 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700562 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700563 params.set_wall_clock_based_wait_enabled(true);
564 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700565
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700566 params.set_update_check_count_wait_enabled(true);
567 params.set_min_update_checks_needed(1);
568 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700569
Alex Deymo8e18f932015-03-27 16:16:59 -0700570 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700571 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700572 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700573 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700574 -1,
575 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700576 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800577 metrics::CheckResult::kUpdateAvailable,
578 metrics::CheckReaction::kUpdating,
579 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700580 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700581 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700582 EXPECT_TRUE(response.update_exists);
583}
584
585
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700586TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700587 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700588 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700589 params.set_wall_clock_based_wait_enabled(true);
590 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700591
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700592 params.set_update_check_count_wait_enabled(true);
593 params.set_min_update_checks_needed(0);
594 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700595
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700596 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700597 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700598 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700599 -1,
600 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700601 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800602 metrics::CheckResult::kUpdateAvailable,
603 metrics::CheckReaction::kUpdating,
604 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700605 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700606 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700607
Ben Chan9abb7632014-08-07 00:10:53 -0700608 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700609 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700610 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700611 EXPECT_TRUE(response.update_exists);
612}
613
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700614TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700615 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700616 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700617 params.set_wall_clock_based_wait_enabled(true);
618 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700619
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700620 params.set_update_check_count_wait_enabled(true);
621 params.set_min_update_checks_needed(1);
622 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700623
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700624 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700625 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700626 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700627 -1,
628 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700629 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800630 metrics::CheckResult::kUpdateAvailable,
631 metrics::CheckReaction::kDeferring,
632 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700633 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700634 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700635
Ben Chan9abb7632014-08-07 00:10:53 -0700636 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700637 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700638 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700639 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700640
641 // Verify if we are interactive check we don't defer.
642 params.set_interactive(true);
643 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700644 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700645 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700646 -1,
647 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700648 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800649 metrics::CheckResult::kUpdateAvailable,
650 metrics::CheckReaction::kUpdating,
651 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700652 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700653 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700654 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700655}
656
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700657TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700658 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700659 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700660 params.set_wall_clock_based_wait_enabled(true);
661 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700662
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700663 params.set_update_check_count_wait_enabled(true);
664 params.set_min_update_checks_needed(1);
665 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700666
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700667 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700668
669 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700670 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700671 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700672 -1,
673 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700674 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800675 metrics::CheckResult::kUpdateAvailable,
676 metrics::CheckReaction::kDeferring,
677 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700678 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700679 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700680
Ben Chan9abb7632014-08-07 00:10:53 -0700681 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700682 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700683 // count remains the same, as the decrementing happens in update_attempter
684 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700685 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700686 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700687
688 // Verify if we are interactive check we don't defer.
689 params.set_interactive(true);
690 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700691 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700692 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700693 -1,
694 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700695 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800696 metrics::CheckResult::kUpdateAvailable,
697 metrics::CheckReaction::kUpdating,
698 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700699 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700700 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700701 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700702}
Jay Srinivasan0a708742012-03-20 11:26:12 -0700703
Alex Deymo8e18f932015-03-27 16:16:59 -0700704TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
705 OmahaResponse response;
706 OmahaRequestParams params = request_params_;
707 fake_update_response_.include_cohorts = true;
708 fake_update_response_.cohort = "s/154454/8479665";
709 fake_update_response_.cohorthint = "please-put-me-on-beta";
710 fake_update_response_.cohortname = "stable";
711
712 ASSERT_TRUE(TestUpdateCheck(&params,
713 fake_update_response_.GetUpdateResponse(),
714 -1,
715 false, // ping_only
716 ErrorCode::kSuccess,
717 metrics::CheckResult::kUpdateAvailable,
718 metrics::CheckReaction::kUpdating,
719 metrics::DownloadErrorCode::kUnset,
720 &response,
721 nullptr));
722
723 string value;
724 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
725 EXPECT_EQ(fake_update_response_.cohort, value);
726
727 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
728 EXPECT_EQ(fake_update_response_.cohorthint, value);
729
730 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
731 EXPECT_EQ(fake_update_response_.cohortname, value);
732}
733
734TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
735 OmahaResponse response;
736 OmahaRequestParams params = request_params_;
737 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
738 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
739 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
740 fake_update_response_.include_cohorts = true;
741 fake_update_response_.cohort = "s/154454/8479665";
742 fake_update_response_.cohorthint = "please-put-me-on-beta";
743 fake_update_response_.cohortname = "";
744
745 ASSERT_TRUE(TestUpdateCheck(&params,
746 fake_update_response_.GetUpdateResponse(),
747 -1,
748 false, // ping_only
749 ErrorCode::kSuccess,
750 metrics::CheckResult::kUpdateAvailable,
751 metrics::CheckReaction::kUpdating,
752 metrics::DownloadErrorCode::kUnset,
753 &response,
754 nullptr));
755
756 string value;
757 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
758 EXPECT_EQ(fake_update_response_.cohort, value);
759
760 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
761 EXPECT_EQ(fake_update_response_.cohorthint, value);
762
763 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
764}
765
766TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
767 OmahaResponse response;
768 OmahaRequestParams params = request_params_;
769 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
770
771 ASSERT_TRUE(TestUpdateCheck(&params,
772 fake_update_response_.GetUpdateResponse(),
773 -1,
774 false, // ping_only
775 ErrorCode::kSuccess,
776 metrics::CheckResult::kUpdateAvailable,
777 metrics::CheckReaction::kUpdating,
778 metrics::DownloadErrorCode::kUnset,
779 &response,
780 nullptr));
781
782 string value;
783 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
784 EXPECT_EQ("old_value", value);
785
786 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
787 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
788}
789
Alex Deymo00d79ac2015-06-29 15:41:49 -0700790TEST_F(OmahaRequestActionTest, CohortsArePersistedWhenNoUpdate) {
791 OmahaResponse response;
792 OmahaRequestParams params = request_params_;
793 fake_update_response_.include_cohorts = true;
794 fake_update_response_.cohort = "s/154454/8479665";
795 fake_update_response_.cohorthint = "please-put-me-on-beta";
796 fake_update_response_.cohortname = "stable";
797
798 ASSERT_TRUE(TestUpdateCheck(&params,
799 fake_update_response_.GetNoUpdateResponse(),
800 -1,
801 false, // ping_only
802 ErrorCode::kSuccess,
803 metrics::CheckResult::kNoUpdateAvailable,
804 metrics::CheckReaction::kUnset,
805 metrics::DownloadErrorCode::kUnset,
806 &response,
807 nullptr));
808
809 string value;
810 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
811 EXPECT_EQ(fake_update_response_.cohort, value);
812
813 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
814 EXPECT_EQ(fake_update_response_.cohorthint, value);
815
816 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
817 EXPECT_EQ(fake_update_response_.cohortname, value);
818}
819
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700820TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -0700821 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700822
Alex Deymo60ca1a72015-06-18 18:19:15 -0700823 chromeos::FakeMessageLoop loop(nullptr);
824 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700825
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700826 OmahaRequestParams params = request_params_;
827 fake_system_state_.set_request_params(&params);
828 OmahaRequestAction action(&fake_system_state_, nullptr,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700829 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800830 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700831 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -0700832 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700833 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700834 ActionProcessor processor;
835 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700836 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700837
Alex Deymo60ca1a72015-06-18 18:19:15 -0700838 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
839 loop.Run();
840 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700841 EXPECT_FALSE(processor.IsRunning());
842}
843
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700844TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700845 OmahaResponse response;
846 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700847 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700848 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700849 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700850 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700851 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800852 metrics::CheckResult::kParsingError,
853 metrics::CheckReaction::kUnset,
854 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -0700855 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700856 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -0700857 EXPECT_FALSE(response.update_exists);
858}
859
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700860TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -0700861 OmahaResponse response;
862 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700863 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -0700864 "",
865 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700866 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700867 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800868 metrics::CheckResult::kParsingError,
869 metrics::CheckReaction::kUnset,
870 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700871 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700872 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700873 EXPECT_FALSE(response.update_exists);
874}
875
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700876TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700877 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700878 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700879 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700880 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
881 "<daystart elapsed_seconds=\"100\"/>"
882 "<app appid=\"foo\" status=\"ok\">"
883 "<ping status=\"ok\"/>"
884 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700885 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700886 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700887 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800888 metrics::CheckResult::kParsingError,
889 metrics::CheckReaction::kUnset,
890 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700891 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700892 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700893 EXPECT_FALSE(response.update_exists);
894}
895
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700896TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700897 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700898 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700899 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700900 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
901 "<daystart elapsed_seconds=\"100\"/>"
902 "<app appid=\"foo\" status=\"ok\">"
903 "<ping status=\"ok\"/>"
904 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700905 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700906 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700907 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800908 metrics::CheckResult::kParsingError,
909 metrics::CheckReaction::kUnset,
910 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700911 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700912 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700913 EXPECT_FALSE(response.update_exists);
914}
915
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700916TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700917 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700918 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700919 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700920 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
921 "<daystart elapsed_seconds=\"100\"/>"
922 "<app appid=\"foo\" status=\"ok\">"
923 "<ping status=\"ok\"/>"
924 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700925 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700926 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700927 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800928 metrics::CheckResult::kParsingError,
929 metrics::CheckReaction::kUnset,
930 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700931 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700932 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700933 EXPECT_FALSE(response.update_exists);
934}
935
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700936TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700937 string input_response =
938 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
939 "<daystart elapsed_seconds=\"100\"/>"
940 "<app appid=\"xyz\" status=\"ok\">"
941 "<updatecheck status=\"ok\">"
942 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -0700943 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700944 "<packages><package hash=\"not-used\" name=\"f\" "
945 "size=\"587\"/></packages>"
946 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700947 "ChromeOSVersion=\"10.2.3.4\" "
948 "Prompt=\"false\" "
949 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -0800950 "IsDeltaPayload=\"false\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700951 "sha256=\"lkq34j5345\" "
952 "needsadmin=\"true\" "
953 "/></actions></manifest></updatecheck></app></response>";
954 LOG(INFO) << "Input Response = " << input_response;
955
Darin Petkov6a5b3222010-07-13 14:55:28 -0700956 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700957 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700958 input_response,
Darin Petkovedc522e2010-11-05 09:35:17 -0700959 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700960 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700961 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800962 metrics::CheckResult::kUpdateAvailable,
963 metrics::CheckReaction::kUpdating,
964 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700965 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700966 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700967 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -0700968 EXPECT_EQ("10.2.3.4", response.version);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800969 EXPECT_EQ("http://missing/field/test/f", response.payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700970 EXPECT_EQ("", response.more_info_url);
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700971 EXPECT_EQ("lkq34j5345", response.hash);
972 EXPECT_EQ(587, response.size);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700973 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -0700974 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700975}
976
977namespace {
978class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
979 public:
980 void ProcessingStopped(const ActionProcessor* processor) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700981 chromeos::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700982 }
Darin Petkov6a5b3222010-07-13 14:55:28 -0700983};
984
Alex Deymo60ca1a72015-06-18 18:19:15 -0700985void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700986 processor->StartProcessing();
987 CHECK(processor->IsRunning());
988 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700989}
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700990} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700991
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700992TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700993 chromeos::FakeMessageLoop loop(nullptr);
994 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700995
Alex Deymo60ca1a72015-06-18 18:19:15 -0700996 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700997 OmahaRequestAction action(&fake_system_state_, nullptr,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700998 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800999 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001000 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001001 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001002 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001003 ActionProcessor processor;
1004 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001005 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -07001006
Alex Deymo60ca1a72015-06-18 18:19:15 -07001007 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
1008 loop.Run();
1009 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -07001010}
1011
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001012TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -07001013 string output;
1014 EXPECT_TRUE(XmlEncode("ab", &output));
1015 EXPECT_EQ("ab", output);
1016 EXPECT_TRUE(XmlEncode("a<b", &output));
1017 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -07001018 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
1019 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -07001020 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
1021 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -07001022 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -07001023 EXPECT_FALSE(XmlEncode("\xc2", &output));
1024 // Fail with invalid ASCII-7 chars.
1025 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
1026}
Darin Petkov6a5b3222010-07-13 14:55:28 -07001027
Alex Deymob0d74eb2015-03-30 17:59:17 -07001028TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
1029 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
1030 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
1031}
1032
1033TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001034 chromeos::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001035
1036 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001037 OmahaRequestParams params(&fake_system_state_,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001038 OmahaRequestParams::kOsPlatform,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001039 OmahaRequestParams::kOsVersion,
1040 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001041 "x86 generic<id",
Darin Petkov6a5b3222010-07-13 14:55:28 -07001042 OmahaRequestParams::kAppId,
1043 "0.1.0.0",
1044 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001045 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001046 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001047 "ChromeOSFirmware.1.0",
1048 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001049 false, // delta okay
1050 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001051 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001052 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001053 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1054 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1055 fake_prefs_.SetString(kPrefsOmahaCohortName,
1056 JoinString(vector<string>(100, "My spoon is too big."),
1057 ' '));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001058 OmahaResponse response;
1059 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001060 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001061 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001062 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001063 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001064 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001065 metrics::CheckResult::kParsingError,
1066 metrics::CheckReaction::kUnset,
1067 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001068 &response,
1069 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001070 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001071 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001072 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1073 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1074 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1075 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1076 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1077 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1078 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1079 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1080 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1081 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1082 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1083 // Values from Prefs that are too big are removed from the XML instead of
1084 // encoded.
1085 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001086}
1087
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001088TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001089 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001090 fake_update_response_.deadline = "&lt;20110101";
1091 fake_update_response_.more_info_url = "testthe&lt;url";
1092 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001093 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001094 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001095 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001096 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001097 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001098 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001099 metrics::CheckResult::kUpdateAvailable,
1100 metrics::CheckReaction::kUpdating,
1101 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001102 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001103 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001104
1105 EXPECT_EQ(response.more_info_url, "testthe<url");
Jay Srinivasan6f6ea002012-12-14 11:26:28 -08001106 EXPECT_EQ(response.payload_urls[0], "testthe&codebase/file.signed");
Darin Petkov6c118642010-10-21 12:06:30 -07001107 EXPECT_EQ(response.deadline, "<20110101");
Darin Petkov6a5b3222010-07-13 14:55:28 -07001108}
1109
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001110TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001111 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001112 // overflows int32_t:
1113 fake_update_response_.size = 123123123123123ll;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001114 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001115 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001116 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001117 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001118 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001119 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001120 metrics::CheckResult::kUpdateAvailable,
1121 metrics::CheckReaction::kUpdating,
1122 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001123 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001124 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001125
1126 EXPECT_EQ(response.size, 123123123123123ll);
1127}
1128
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001129TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001130 chromeos::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001131 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001132 fake_system_state_.set_prefs(&prefs);
1133
Darin Petkov95508da2011-01-05 12:42:29 -08001134 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1135 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001136 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001137 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001138 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001139 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001140 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001141 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001142 metrics::CheckResult::kParsingError,
1143 metrics::CheckReaction::kUnset,
1144 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001145 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001146 &post_data));
1147 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001148 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001149 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001150 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001151 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001152 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001153 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1154 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001155 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1156 string::npos);
1157 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1158 string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001159}
1160
Jay Srinivasan0a708742012-03-20 11:26:12 -07001161
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001162TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001163 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001164 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001165 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1166 "invalid xml>",
1167 &post_data);
1168 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001169 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001170 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001171 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001172 OmahaEvent::kTypeUpdateDownloadStarted,
1173 OmahaEvent::kResultSuccess);
1174 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001175 EXPECT_EQ(post_str.find("ping"), string::npos);
1176 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001177}
1178
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001179TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001180 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001181 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001182 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1183 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001184 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001185 "invalid xml>",
1186 &post_data);
1187 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001188 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001189 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001190 " <event eventtype=\"%d\" eventresult=\"%d\" "
1191 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001192 OmahaEvent::kTypeDownloadComplete,
1193 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001194 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001195 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001196 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001197}
1198
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001199TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001200 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001201 // Create a copy of the OmahaRequestParams to reuse it later.
1202 OmahaRequestParams params = request_params_;
1203 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001204 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001205 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001206 nullptr,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001207 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -08001208 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001209 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001210 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001211 EXPECT_FALSE(update_check_action.IsEvent());
1212
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001213 params = request_params_;
1214 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001215 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001216 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001217 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001218 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -08001219 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001220 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001221 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001222 EXPECT_TRUE(event_action.IsEvent());
1223}
1224
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001225TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001226 for (int i = 0; i < 2; i++) {
1227 bool delta_okay = i == 1;
1228 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001229 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001230 OmahaRequestParams params(&fake_system_state_,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001231 OmahaRequestParams::kOsPlatform,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001232 OmahaRequestParams::kOsVersion,
1233 "service_pack",
1234 "x86-generic",
1235 OmahaRequestParams::kAppId,
1236 "0.1.0.0",
1237 "en-US",
1238 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001239 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001240 "ChromeOSFirmware.1.0",
1241 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001242 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001243 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001244 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001245 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001246 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001247 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001248 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001249 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001250 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001251 metrics::CheckResult::kParsingError,
1252 metrics::CheckReaction::kUnset,
1253 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001254 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001255 &post_data));
1256 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001257 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001258 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1259 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001260 string::npos)
1261 << "i = " << i;
1262 }
1263}
1264
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001265TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001266 for (int i = 0; i < 2; i++) {
1267 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001268 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001269 chromeos::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001270 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001271 OmahaRequestParams params(&fake_system_state_,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001272 OmahaRequestParams::kOsPlatform,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001273 OmahaRequestParams::kOsVersion,
1274 "service_pack",
1275 "x86-generic",
1276 OmahaRequestParams::kAppId,
1277 "0.1.0.0",
1278 "en-US",
1279 "unittest_track",
1280 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001281 "ChromeOSFirmware.1.0",
1282 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001283 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001284 interactive,
1285 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001286 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001287 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001288 "invalid xml>",
1289 -1,
1290 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001291 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001292 metrics::CheckResult::kParsingError,
1293 metrics::CheckReaction::kUnset,
1294 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001295 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001296 &post_data));
1297 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001298 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001299 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1300 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001301 string::npos)
1302 << "i = " << i;
1303 }
1304}
1305
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001306TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001307 OmahaEvent default_event;
1308 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1309 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001310 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001311
1312 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1313 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1314 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001315 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001316
1317 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1318 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001319 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001320 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1321 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001322 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001323}
1324
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001325void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001326 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001327 fake_system_state_.set_prefs(&prefs);
1328 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1329 .Times(AnyNumber());
1330 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1331 // Add a few hours to the day difference to test no rounding, etc.
1332 int64_t five_days_ago =
1333 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1334 int64_t six_days_ago =
1335 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1336 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1337 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1338 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1339 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1340 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1341 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001342 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001343 ASSERT_TRUE(
1344 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001345 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001346 -1,
1347 ping_only,
1348 ErrorCode::kSuccess,
1349 metrics::CheckResult::kUnset,
1350 metrics::CheckReaction::kUnset,
1351 metrics::DownloadErrorCode::kUnset,
1352 nullptr,
1353 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001354 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001355 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1356 string::npos);
1357 if (ping_only) {
1358 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1359 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1360 } else {
1361 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1362 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001363 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001364}
1365
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001366TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1367 PingTest(true /* ping_only */);
1368}
1369
1370TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1371 PingTest(false /* ping_only */);
1372}
1373
1374TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001375 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001376 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001377 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1378 .Times(AnyNumber());
1379 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001380 int64_t three_days_ago =
1381 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1382 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001383 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1384 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001385 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1386 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1387 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1388 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001389 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001390 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001391 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001392 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001393 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001394 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001395 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001396 metrics::CheckResult::kNoUpdateAvailable,
1397 metrics::CheckReaction::kUnset,
1398 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001399 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001400 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001401 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001402 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001403 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001404}
1405
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001406TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001407 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001408 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001409 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1410 .Times(AnyNumber());
1411 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001412 int64_t four_days_ago =
1413 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1414 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001415 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1416 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001417 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1418 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1419 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1420 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001421 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001422 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001423 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001424 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001425 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001426 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001427 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001428 metrics::CheckResult::kNoUpdateAvailable,
1429 metrics::CheckReaction::kUnset,
1430 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001431 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001432 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001433 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001434 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001435 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001436}
1437
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001438TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001439 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001440 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001441 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1442 .Times(AnyNumber());
1443 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001444 int64_t one_hour_ago =
1445 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001446 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1447 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001448 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1449 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1450 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1451 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001452 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1453 // send a ping.
1454 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1455 .WillOnce(Return(true));
1456 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1457 .WillOnce(Return(true));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001458 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001459 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001460 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001461 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001462 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001463 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001464 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001465 metrics::CheckResult::kNoUpdateAvailable,
1466 metrics::CheckReaction::kUnset,
1467 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001468 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001469 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001470 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001471 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001472}
1473
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001474TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001475 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001476 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001477 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001478 int64_t now = Time::Now().ToInternalValue();
1479 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1480 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1481 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1482 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1483 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1484 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001485 chromeos::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001486 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001487 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001488 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001489 -1,
1490 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001491 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001492 metrics::CheckResult::kUnset,
1493 metrics::CheckReaction::kUnset,
1494 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001495 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001496 &post_data));
1497 EXPECT_EQ(post_data.size(), 0);
1498}
1499
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001500TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001501 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001502 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001503 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1504 .Times(AnyNumber());
1505 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001506 int64_t future =
1507 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001508 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1509 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001510 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1511 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1512 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1513 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1514 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1515 .WillOnce(Return(true));
1516 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1517 .WillOnce(Return(true));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001518 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001519 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001520 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001521 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1522 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001523 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001524 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001525 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001526 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001527 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001528 metrics::CheckResult::kNoUpdateAvailable,
1529 metrics::CheckReaction::kUnset,
1530 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001531 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001532 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001533 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001534 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001535}
1536
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001537TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001538 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001539 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001540 // may fail if it runs for longer than 5 seconds. It shouldn't run
1541 // that long though.
1542 int64_t midnight =
1543 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1544 int64_t midnight_slack =
1545 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001546 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001547 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001548 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1549 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001550 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1551 AllOf(Ge(midnight), Le(midnight_slack))))
1552 .WillOnce(Return(true));
1553 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1554 AllOf(Ge(midnight), Le(midnight_slack))))
1555 .WillOnce(Return(true));
1556 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001557 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001558 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1559 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001560 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001561 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001562 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001563 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001564 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001565 metrics::CheckResult::kNoUpdateAvailable,
1566 metrics::CheckReaction::kUnset,
1567 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001568 nullptr,
1569 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001570}
1571
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001572TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001573 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001574 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001575 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1576 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001577 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1578 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1579 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001580 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001581 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1582 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001583 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001584 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001585 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001586 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001587 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001588 metrics::CheckResult::kNoUpdateAvailable,
1589 metrics::CheckReaction::kUnset,
1590 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001591 nullptr,
1592 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001593}
1594
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001595TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001596 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001597 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001598 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1599 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001600 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1601 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1602 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001603 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001604 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1605 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001606 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001607 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001608 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001609 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001610 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001611 metrics::CheckResult::kNoUpdateAvailable,
1612 metrics::CheckReaction::kUnset,
1613 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001614 nullptr,
1615 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001616}
1617
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001618TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001619 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001620 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07001621 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001622 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001623 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001624 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001625 metrics::CheckResult::kParsingError,
1626 metrics::CheckReaction::kUnset,
1627 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001628 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07001629 &post_data));
1630 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001631 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07001632 EXPECT_EQ(post_str.find("machineid="), string::npos);
1633 EXPECT_EQ(post_str.find("userid="), string::npos);
1634}
1635
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001636TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001637 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001638 const int http_error_code =
1639 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07001640 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001641 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001642 "",
1643 501,
Darin Petkov265f2902011-05-09 15:17:40 -07001644 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001645 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001646 metrics::CheckResult::kDownloadError,
1647 metrics::CheckReaction::kUnset,
1648 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07001649 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001650 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001651 EXPECT_FALSE(response.update_exists);
1652}
1653
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001654TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001655 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001656 const int http_error_code =
1657 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07001658 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001659 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001660 "",
1661 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07001662 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001663 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001664 metrics::CheckResult::kDownloadError,
1665 metrics::CheckReaction::kUnset,
1666 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07001667 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001668 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001669 EXPECT_FALSE(response.update_exists);
1670}
1671
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001672TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001673 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001674 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001675 params.set_wall_clock_based_wait_enabled(true);
1676 params.set_waiting_period(TimeDelta().FromDays(1));
1677 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001678
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001679 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001680 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001681 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001682 -1,
1683 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001684 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001685 metrics::CheckResult::kUpdateAvailable,
1686 metrics::CheckReaction::kDeferring,
1687 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001688 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001689 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001690
Ben Chan9abb7632014-08-07 00:10:53 -07001691 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001692 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001693 ASSERT_GT(timestamp, 0);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001694 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001695
1696 // Verify if we are interactive check we don't defer.
1697 params.set_interactive(true);
1698 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001699 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001700 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001701 -1,
1702 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001703 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001704 metrics::CheckResult::kUpdateAvailable,
1705 metrics::CheckReaction::kUpdating,
1706 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001707 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001708 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001709 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001710}
1711
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001712TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001713 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001714 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001715 params.set_wall_clock_based_wait_enabled(true);
1716 params.set_waiting_period(TimeDelta().FromDays(1));
1717 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001718
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001719 // Set the timestamp to a very old value such that it exceeds the
1720 // waiting period set above.
1721 Time t1;
1722 Time::FromString("1/1/2012", &t1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001723 ASSERT_TRUE(fake_prefs_.SetInt64(
1724 kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001725 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001726 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001727 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001728 -1,
1729 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001730 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001731 metrics::CheckResult::kUpdateAvailable,
1732 metrics::CheckReaction::kUpdating,
1733 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001734 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001735 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001736
1737 EXPECT_TRUE(response.update_exists);
1738
1739 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07001740 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001741 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001742 ASSERT_TRUE(timestamp == t1.ToInternalValue());
1743}
1744
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001745TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001746 // Create a uniquely named test directory.
1747 string test_dir;
1748 ASSERT_TRUE(utils::MakeTempDirectory(
1749 "omaha_request_action-test-XXXXXX", &test_dir));
1750
1751 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1752 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001753 kStatefulPartition + "/etc"));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001754 chromeos::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001755 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001756 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001757 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001758 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001759 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1760 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1761 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
1762 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001763 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001764 "CHROMEOS_IS_POWERWASH_ALLOWED=true\n"
1765 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001766 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001767 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001768 params.SetLockDown(false);
1769 params.Init("1.2.3.4", "", 0);
1770 EXPECT_EQ("canary-channel", params.current_channel());
1771 EXPECT_EQ("stable-channel", params.target_channel());
1772 EXPECT_TRUE(params.to_more_stable_channel());
1773 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001774 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001775 "invalid xml>",
1776 -1,
1777 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001778 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001779 metrics::CheckResult::kParsingError,
1780 metrics::CheckReaction::kUnset,
1781 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001782 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001783 &post_data));
1784 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001785 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001786 EXPECT_NE(string::npos, post_str.find(
1787 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
1788 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
1789 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001790
Alex Deymo10875d92014-11-10 21:52:57 -08001791 ASSERT_TRUE(test_utils::RecursiveUnlinkDir(test_dir));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001792}
1793
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001794TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001795 // Create a uniquely named test directory.
1796 string test_dir;
1797 ASSERT_TRUE(utils::MakeTempDirectory(
1798 "omaha_request_action-test-XXXXXX", &test_dir));
1799
1800 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1801 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001802 kStatefulPartition + "/etc"));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001803 chromeos::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001804 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001805 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001806 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001807 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001808 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1809 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1810 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
1811 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001812 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001813 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001814 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001815 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001816 params.SetLockDown(false);
1817 params.Init("5.6.7.8", "", 0);
1818 EXPECT_EQ("stable-channel", params.current_channel());
1819 EXPECT_EQ("canary-channel", params.target_channel());
1820 EXPECT_FALSE(params.to_more_stable_channel());
1821 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001822 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001823 "invalid xml>",
1824 -1,
1825 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001826 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001827 metrics::CheckResult::kParsingError,
1828 metrics::CheckReaction::kUnset,
1829 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001830 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001831 &post_data));
1832 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001833 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001834 EXPECT_NE(string::npos, post_str.find(
1835 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
1836 "version=\"5.6.7.8\" "
1837 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001838 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001839}
1840
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001841// Checks that the initial ping with a=-1 r=-1 is not send when the device
1842// was powerwashed.
1843TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
1844 fake_prefs_.SetString(kPrefsPreviousVersion, "");
1845
1846 // Flag that the device was powerwashed in the past.
1847 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
1848
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001849 chromeos::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001850 ASSERT_TRUE(
1851 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001852 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001853 -1,
1854 false, // ping_only
1855 ErrorCode::kSuccess,
1856 metrics::CheckResult::kNoUpdateAvailable,
1857 metrics::CheckReaction::kUnset,
1858 metrics::DownloadErrorCode::kUnset,
1859 nullptr,
1860 &post_data));
1861 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001862 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001863 EXPECT_EQ(string::npos, post_str.find("<ping"));
1864}
1865
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001866void OmahaRequestActionTest::P2PTest(
1867 bool initial_allow_p2p_for_downloading,
1868 bool initial_allow_p2p_for_sharing,
1869 bool omaha_disable_p2p_for_downloading,
1870 bool omaha_disable_p2p_for_sharing,
1871 bool payload_state_allow_p2p_attempt,
1872 bool expect_p2p_client_lookup,
1873 const string& p2p_client_result_url,
1874 bool expected_allow_p2p_for_downloading,
1875 bool expected_allow_p2p_for_sharing,
1876 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07001877 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001878 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07001879 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
1880 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
1881 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07001882
1883 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001884 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07001885 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
1886 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07001887 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
1888 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
1889 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
1890 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
1891 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
1892 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
1893 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
1894 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
1895 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
1896 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
1897
David Zeuthen8f191b22013-08-06 12:27:50 -07001898 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001899 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07001900 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
1901
David Zeuthen4cc5ed22014-01-15 12:35:03 -08001902 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
1903 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07001904 .Times(expect_p2p_client_lookup ? 1 : 0);
1905
Alex Deymo8e18f932015-03-27 16:16:59 -07001906 fake_update_response_.disable_p2p_for_downloading =
1907 omaha_disable_p2p_for_downloading;
1908 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07001909 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001910 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001911 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07001912 -1,
1913 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001914 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001915 metrics::CheckResult::kUpdateAvailable,
1916 metrics::CheckReaction::kUpdating,
1917 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07001918 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001919 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07001920 EXPECT_TRUE(response.update_exists);
1921
Gilad Arnold74b5f552014-10-07 08:17:16 -07001922 EXPECT_EQ(omaha_disable_p2p_for_downloading,
1923 response.disable_p2p_for_downloading);
1924 EXPECT_EQ(omaha_disable_p2p_for_sharing,
1925 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07001926
Gilad Arnold74b5f552014-10-07 08:17:16 -07001927 EXPECT_EQ(expected_allow_p2p_for_downloading,
1928 actual_allow_p2p_for_downloading);
1929 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
1930 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07001931}
1932
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001933TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001934 P2PTest(true, // initial_allow_p2p_for_downloading
1935 true, // initial_allow_p2p_for_sharing
1936 false, // omaha_disable_p2p_for_downloading
1937 false, // omaha_disable_p2p_for_sharing
1938 true, // payload_state_allow_p2p_attempt
1939 true, // expect_p2p_client_lookup
1940 "http://1.3.5.7/p2p", // p2p_client_result_url
1941 true, // expected_allow_p2p_for_downloading
1942 true, // expected_allow_p2p_for_sharing
1943 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001944}
1945
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001946TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001947 P2PTest(true, // initial_allow_p2p_for_downloading
1948 true, // initial_allow_p2p_for_sharing
1949 false, // omaha_disable_p2p_for_downloading
1950 false, // omaha_disable_p2p_for_sharing
1951 true, // payload_state_allow_p2p_attempt
1952 true, // expect_p2p_client_lookup
1953 "", // p2p_client_result_url
1954 false, // expected_allow_p2p_for_downloading
1955 true, // expected_allow_p2p_for_sharing
1956 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001957}
1958
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001959TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001960 P2PTest(false, // initial_allow_p2p_for_downloading
1961 true, // initial_allow_p2p_for_sharing
1962 false, // omaha_disable_p2p_for_downloading
1963 false, // omaha_disable_p2p_for_sharing
1964 true, // payload_state_allow_p2p_attempt
1965 false, // expect_p2p_client_lookup
1966 "unset", // p2p_client_result_url
1967 false, // expected_allow_p2p_for_downloading
1968 true, // expected_allow_p2p_for_sharing
1969 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001970}
1971
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001972TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001973 P2PTest(true, // initial_allow_p2p_for_downloading
1974 true, // initial_allow_p2p_for_sharing
1975 true, // omaha_disable_p2p_for_downloading
1976 false, // omaha_disable_p2p_for_sharing
1977 true, // payload_state_allow_p2p_attempt
1978 false, // expect_p2p_client_lookup
1979 "unset", // p2p_client_result_url
1980 false, // expected_allow_p2p_for_downloading
1981 true, // expected_allow_p2p_for_sharing
1982 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001983}
1984
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001985TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001986 P2PTest(true, // initial_allow_p2p_for_downloading
1987 true, // initial_allow_p2p_for_sharing
1988 false, // omaha_disable_p2p_for_downloading
1989 true, // omaha_disable_p2p_for_sharing
1990 true, // payload_state_allow_p2p_attempt
1991 true, // expect_p2p_client_lookup
1992 "http://1.3.5.7/p2p", // p2p_client_result_url
1993 true, // expected_allow_p2p_for_downloading
1994 false, // expected_allow_p2p_for_sharing
1995 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001996}
1997
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001998TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001999 P2PTest(true, // initial_allow_p2p_for_downloading
2000 true, // initial_allow_p2p_for_sharing
2001 true, // omaha_disable_p2p_for_downloading
2002 true, // omaha_disable_p2p_for_sharing
2003 true, // payload_state_allow_p2p_attempt
2004 false, // expect_p2p_client_lookup
2005 "unset", // p2p_client_result_url
2006 false, // expected_allow_p2p_for_downloading
2007 false, // expected_allow_p2p_for_sharing
2008 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07002009}
2010
Alex Deymof329b932014-10-30 01:37:48 -07002011bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
2012 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07002013 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08002014 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002015 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07002016 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08002017 -1,
2018 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07002019 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08002020 metrics::CheckResult::kUpdateAvailable,
2021 metrics::CheckReaction::kUpdating,
2022 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08002023 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07002024 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08002025}
2026
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002027TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08002028 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08002029
2030 // Check that we parse elapsed_days in the Omaha Response correctly.
2031 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002032 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2033 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002034 EXPECT_TRUE(response.update_exists);
2035 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002036 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002037 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002038 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002039 EXPECT_EQ(prefs_days, 42);
2040
2041 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002042 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002043 EXPECT_TRUE(response.update_exists);
2044 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002045 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002046 EXPECT_EQ(prefs_days, 42);
2047
2048 // Note that elapsed_days is not necessarily divisible by 7 so check
2049 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002050 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2051 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002052 EXPECT_TRUE(response.update_exists);
2053 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002054 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002055 EXPECT_EQ(prefs_days, 21);
2056
2057 // Check that we correctly handle elapsed_days not being included in
2058 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002059 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002060 EXPECT_TRUE(response.update_exists);
2061 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002062}
2063
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002064// If there is no prefs and OOBE is not complete, we should not
2065// report anything to Omaha.
2066TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
2067 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2068 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2069}
David Zeuthen639aa362014-02-03 16:23:44 -08002070
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002071// If OOBE is complete and happened on a valid date (e.g. after Jan
2072// 1 2007 0:00 PST), that date should be used and written to
2073// prefs. However, first try with an invalid date and check we do
2074// nothing.
2075TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2076 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2077 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2078 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2079 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2080}
David Zeuthen639aa362014-02-03 16:23:44 -08002081
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002082// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2083// should yield an InstallDate of 14.
2084TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2085 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2086 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2087 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2088 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002089
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002090 int64_t prefs_days;
2091 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2092 EXPECT_EQ(prefs_days, 14);
2093}
David Zeuthen639aa362014-02-03 16:23:44 -08002094
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002095// Now that we have a valid date in prefs, check that we keep using
2096// that even if OOBE date reports something else. The date Jan 30,
2097// 2007 0:00 PST should yield an InstallDate of 28... but since
2098// there's a prefs file, we should still get 14.
2099TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2100 // Set a valid date in the prefs first.
2101 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002102
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002103 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2104 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2105 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002106
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002107 int64_t prefs_days;
2108 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2109 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002110
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002111 // If we delete the prefs file, we should get 28 days.
2112 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2113 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2114 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2115 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002116}
2117
Darin Petkov6a5b3222010-07-13 14:55:28 -07002118} // namespace chromeos_update_engine