blob: c6355b1798f002d72a5a612c9a44d1783c5e9731 [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\">]>";
64 return base::StringPrintf(
65 "<?xml version=\"1.0\" encoding=\"UTF-8\"?>"
66 "%s<response protocol=\"3.0\">"
67 "<daystart elapsed_seconds=\"100\"/>"
68 "<app appid=\"%s\" status=\"ok\"><ping status=\"ok\"/>"
69 "<updatecheck status=\"noupdate\"/></app></response>",
70 entity_str.c_str(), app_id.c_str());
71 }
72
73 string GetUpdateResponse() const {
74 return
75 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
76 "protocol=\"3.0\">"
77 "<daystart elapsed_seconds=\"100\"" +
78 (elapsed_days.empty() ? "" : (" elapsed_days=\"" + elapsed_days + "\""))
79 + "/>"
80 "<app appid=\"" + app_id + "\" " +
81 (include_cohorts ? "cohort=\"" + cohort + "\" cohorthint=\"" +
82 cohorthint + "\" cohortname=\"" + cohortname + "\" " : "") +
83 " status=\"ok\">"
84 "<ping status=\"ok\"/><updatecheck status=\"ok\">"
85 "<urls><url codebase=\"" + codebase + "\"/></urls>"
86 "<manifest version=\"" + version + "\">"
87 "<packages><package hash=\"not-used\" name=\"" + filename + "\" "
88 "size=\"" + base::Int64ToString(size) + "\"/></packages>"
89 "<actions><action event=\"postinstall\" "
90 "ChromeOSVersion=\"" + version + "\" "
91 "MoreInfo=\"" + more_info_url + "\" Prompt=\"" + prompt + "\" "
92 "IsDelta=\"true\" "
93 "IsDeltaPayload=\"true\" "
94 "MaxDaysToScatter=\"" + max_days_to_scatter + "\" "
95 "sha256=\"" + hash + "\" "
96 "needsadmin=\"" + needsadmin + "\" " +
97 (deadline.empty() ? "" : ("deadline=\"" + deadline + "\" ")) +
98 (disable_p2p_for_downloading ?
99 "DisableP2PForDownloading=\"true\" " : "") +
100 (disable_p2p_for_sharing ? "DisableP2PForSharing=\"true\" " : "") +
101 "/></actions></manifest></updatecheck></app></response>";
102 }
103
104 // Return the payload URL, which is split in two fields in the XML response.
105 string GetPayloadUrl() {
106 return codebase + filename;
107 }
108
109 string app_id = chromeos_update_engine::OmahaRequestParams::kAppId;
110 string version = "1.2.3.4";
111 string more_info_url = "http://more/info";
112 string prompt = "true";
113 string codebase = "http://code/base/";
114 string filename = "file.signed";
115 string hash = "HASH1234=";
116 string needsadmin = "false";
117 int64_t size = 123;
118 string deadline = "";
119 string max_days_to_scatter = "7";
120 string elapsed_days = "42";
121
122 // P2P setting defaults to allowed.
123 bool disable_p2p_for_downloading = false;
124 bool disable_p2p_for_sharing = false;
125
126 // Omaha cohorts settings.
127 bool include_cohorts = false;
128 string cohort = "";
129 string cohorthint = "";
130 string cohortname = "";
131
132 // Whether to include the CrOS <!ENTITY> in the XML response.
133 bool include_entity = false;
134};
135
136} // namespace
137
Darin Petkov6a5b3222010-07-13 14:55:28 -0700138namespace chromeos_update_engine {
139
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700140class OmahaRequestActionTest : public ::testing::Test {
141 protected:
Alex Deymo610277e2014-11-11 21:18:11 -0800142 void SetUp() override {
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700143 fake_system_state_.set_request_params(&request_params_);
144 fake_system_state_.set_prefs(&fake_prefs_);
145 }
146
147 // Returns true iff an output response was obtained from the
Alex Deymo8427b4a2014-11-05 14:00:32 -0800148 // OmahaRequestAction. |prefs| may be null, in which case a local MockPrefs
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700149 // is used. |payload_state| may be null, in which case a local mock is used.
150 // |p2p_manager| may be null, in which case a local mock is used.
151 // |connection_manager| may be null, in which case a local mock is used.
152 // out_response may be null. If |fail_http_response_code| is non-negative,
153 // the transfer will fail with that code. |ping_only| is passed through to the
154 // OmahaRequestAction constructor. out_post_data may be null; if non-null, the
155 // post-data received by the mock HttpFetcher is returned.
156 //
157 // The |expected_check_result|, |expected_check_reaction| and
158 // |expected_error_code| parameters are for checking expectations
159 // about reporting UpdateEngine.Check.{Result,Reaction,DownloadError}
160 // UMA statistics. Use the appropriate ::kUnset value to specify that
161 // the given metric should not be reported.
162 bool TestUpdateCheck(OmahaRequestParams* request_params,
163 const string& http_response,
164 int fail_http_response_code,
165 bool ping_only,
166 ErrorCode expected_code,
167 metrics::CheckResult expected_check_result,
168 metrics::CheckReaction expected_check_reaction,
169 metrics::DownloadErrorCode expected_download_error_code,
170 OmahaResponse* out_response,
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800171 chromeos::Blob* out_post_data);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700172
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800173 // Runs and checks a ping test. |ping_only| indicates whether it should send
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700174 // only a ping or also an updatecheck.
175 void PingTest(bool ping_only);
176
177 // InstallDate test helper function.
Alex Deymof329b932014-10-30 01:37:48 -0700178 bool InstallDateParseHelper(const string &elapsed_days,
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700179 OmahaResponse *response);
180
181 // P2P test helper function.
182 void P2PTest(
183 bool initial_allow_p2p_for_downloading,
184 bool initial_allow_p2p_for_sharing,
185 bool omaha_disable_p2p_for_downloading,
186 bool omaha_disable_p2p_for_sharing,
187 bool payload_state_allow_p2p_attempt,
188 bool expect_p2p_client_lookup,
189 const string& p2p_client_result_url,
190 bool expected_allow_p2p_for_downloading,
191 bool expected_allow_p2p_for_sharing,
192 const string& expected_p2p_url);
193
194 FakeSystemState fake_system_state_;
Alex Deymo8e18f932015-03-27 16:16:59 -0700195 FakeUpdateResponse fake_update_response_;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700196
197 // By default, all tests use these objects unless they replace them in the
198 // fake_system_state_.
199 OmahaRequestParams request_params_ = OmahaRequestParams{
200 &fake_system_state_,
201 OmahaRequestParams::kOsPlatform,
202 OmahaRequestParams::kOsVersion,
203 "service_pack",
204 "x86-generic",
205 OmahaRequestParams::kAppId,
206 "0.1.0.0",
207 "en-US",
208 "unittest",
209 "OEM MODEL 09235 7471",
210 "ChromeOSFirmware.1.0",
211 "0X0A1",
212 false, // delta okay
213 false, // interactive
214 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -0700215 ""}; // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700216
217 FakePrefs fake_prefs_;
218};
Darin Petkov6a5b3222010-07-13 14:55:28 -0700219
220namespace {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700221class OmahaRequestActionTestProcessorDelegate : public ActionProcessorDelegate {
222 public:
223 OmahaRequestActionTestProcessorDelegate()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700224 : expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -0800225 ~OmahaRequestActionTestProcessorDelegate() override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700226 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800227 void ProcessingDone(const ActionProcessor* processor,
228 ErrorCode code) override {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700229 chromeos::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700230 }
231
Yunlian Jiang35866ed2015-01-29 13:09:20 -0800232 void ActionCompleted(ActionProcessor* processor,
233 AbstractAction* action,
234 ErrorCode code) override {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700235 // make sure actions always succeed
236 if (action->Type() == OmahaRequestAction::StaticType())
Darin Petkovc1a8b422010-07-19 11:34:49 -0700237 EXPECT_EQ(expected_code_, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700238 else
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700239 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700240 }
David Zeuthena99981f2013-04-29 13:42:47 -0700241 ErrorCode expected_code_;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700242};
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700243} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700244
245class OutputObjectCollectorAction;
246
247template<>
248class ActionTraits<OutputObjectCollectorAction> {
249 public:
250 // Does not take an object for input
251 typedef OmahaResponse InputObjectType;
252 // On success, puts the output path on output
253 typedef NoneType OutputObjectType;
254};
255
256class OutputObjectCollectorAction : public Action<OutputObjectCollectorAction> {
257 public:
258 OutputObjectCollectorAction() : has_input_object_(false) {}
259 void PerformAction() {
260 // copy input object
261 has_input_object_ = HasInputObject();
262 if (has_input_object_)
263 omaha_response_ = GetInputObject();
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700264 processor_->ActionComplete(this, ErrorCode::kSuccess);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700265 }
266 // Should never be called
267 void TerminateProcessing() {
268 CHECK(false);
269 }
270 // Debugging/logging
Alex Deymof329b932014-10-30 01:37:48 -0700271 static string StaticType() {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700272 return "OutputObjectCollectorAction";
273 }
Alex Deymof329b932014-10-30 01:37:48 -0700274 string Type() const { return StaticType(); }
Darin Petkov6a5b3222010-07-13 14:55:28 -0700275 bool has_input_object_;
276 OmahaResponse omaha_response_;
277};
278
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700279bool OmahaRequestActionTest::TestUpdateCheck(
280 OmahaRequestParams* request_params,
281 const string& http_response,
282 int fail_http_response_code,
283 bool ping_only,
284 ErrorCode expected_code,
285 metrics::CheckResult expected_check_result,
286 metrics::CheckReaction expected_check_reaction,
287 metrics::DownloadErrorCode expected_download_error_code,
288 OmahaResponse* out_response,
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800289 chromeos::Blob* out_post_data) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700290 chromeos::FakeMessageLoop loop(nullptr);
291 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700292 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800293 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700294 nullptr);
Darin Petkovedc522e2010-11-05 09:35:17 -0700295 if (fail_http_response_code >= 0) {
296 fetcher->FailTransfer(fail_http_response_code);
297 }
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700298 if (request_params)
299 fake_system_state_.set_request_params(request_params);
300 OmahaRequestAction action(&fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700301 nullptr,
Thieu Le116fda32011-04-19 11:01:54 -0700302 fetcher,
Darin Petkov265f2902011-05-09 15:17:40 -0700303 ping_only);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700304 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkovc1a8b422010-07-19 11:34:49 -0700305 delegate.expected_code_ = expected_code;
Darin Petkova4a8a8c2010-07-15 22:21:12 -0700306
Darin Petkov6a5b3222010-07-13 14:55:28 -0700307 ActionProcessor processor;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700308 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700309 processor.EnqueueAction(&action);
310
311 OutputObjectCollectorAction collector_action;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700312 BondActions(&action, &collector_action);
313 processor.EnqueueAction(&collector_action);
314
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700315 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(), SendEnumToUMA(_, _, _))
David Zeuthen33bae492014-02-25 16:16:18 -0800316 .Times(AnyNumber());
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700317 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800318 SendEnumToUMA(metrics::kMetricCheckResult,
319 static_cast<int>(expected_check_result),
320 static_cast<int>(metrics::CheckResult::kNumConstants) - 1))
321 .Times(expected_check_result == metrics::CheckResult::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700322 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthen33bae492014-02-25 16:16:18 -0800323 SendEnumToUMA(metrics::kMetricCheckReaction,
324 static_cast<int>(expected_check_reaction),
325 static_cast<int>(metrics::CheckReaction::kNumConstants) - 1))
326 .Times(expected_check_reaction == metrics::CheckReaction::kUnset ? 0 : 1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700327 EXPECT_CALL(*fake_system_state_.mock_metrics_lib(),
David Zeuthenc0dd0212014-04-04 14:49:49 -0700328 SendSparseToUMA(metrics::kMetricCheckDownloadErrorCode,
329 static_cast<int>(expected_download_error_code)))
David Zeuthen33bae492014-02-25 16:16:18 -0800330 .Times(expected_download_error_code == metrics::DownloadErrorCode::kUnset
331 ? 0 : 1);
332
Alex Deymo60ca1a72015-06-18 18:19:15 -0700333 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
334 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
335 loop.Run();
336 LOG(INFO) << "loop.PendingTasks() = " << loop.PendingTasks();
337 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700338 if (collector_action.has_input_object_ && out_response)
339 *out_response = collector_action.omaha_response_;
340 if (out_post_data)
341 *out_post_data = fetcher->post_data();
342 return collector_action.has_input_object_;
343}
344
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700345// Tests Event requests -- they should always succeed. |out_post_data|
346// may be null; if non-null, the post-data received by the mock
347// HttpFetcher is returned.
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700348void TestEvent(OmahaRequestParams params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700349 OmahaEvent* event,
350 const string& http_response,
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800351 chromeos::Blob* out_post_data) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700352 chromeos::FakeMessageLoop loop(nullptr);
353 loop.SetAsCurrent();
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700354 MockHttpFetcher* fetcher = new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800355 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700356 nullptr);
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700357 FakeSystemState fake_system_state;
358 fake_system_state.set_request_params(&params);
359 OmahaRequestAction action(&fake_system_state, event, fetcher, false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700360 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700361 ActionProcessor processor;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700362 processor.set_delegate(&delegate);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700363 processor.EnqueueAction(&action);
364
Alex Deymo60ca1a72015-06-18 18:19:15 -0700365 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
366 loop.Run();
367
368 // This test should schedule a callback to notify the crash reporter if
369 // the passed event is an error.
370 EXPECT_EQ(event->result == OmahaEvent::kResultError, loop.PendingTasks());
371
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700372 if (out_post_data)
373 *out_post_data = fetcher->post_data();
374}
375
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700376TEST_F(OmahaRequestActionTest, RejectEntities) {
David Zeuthenf3e28012014-08-26 18:23:52 -0400377 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700378 fake_update_response_.include_entity = true;
David Zeuthenf3e28012014-08-26 18:23:52 -0400379 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700380 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700381 fake_update_response_.GetNoUpdateResponse(),
David Zeuthenf3e28012014-08-26 18:23:52 -0400382 -1,
383 false, // ping_only
384 ErrorCode::kOmahaRequestXMLHasEntityDecl,
385 metrics::CheckResult::kParsingError,
386 metrics::CheckReaction::kUnset,
387 metrics::DownloadErrorCode::kUnset,
388 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700389 nullptr));
David Zeuthenf3e28012014-08-26 18:23:52 -0400390 EXPECT_FALSE(response.update_exists);
391}
392
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700393TEST_F(OmahaRequestActionTest, NoUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700394 OmahaResponse response;
395 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700396 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700397 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700398 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700399 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700400 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800401 metrics::CheckResult::kNoUpdateAvailable,
402 metrics::CheckReaction::kUnset,
403 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700404 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700405 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700406 EXPECT_FALSE(response.update_exists);
407}
408
Alex Deymo8e18f932015-03-27 16:16:59 -0700409// Test that all the values in the response are parsed in a normal update
410// response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700411TEST_F(OmahaRequestActionTest, ValidUpdateTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700412 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -0700413 fake_update_response_.deadline = "20101020";
Darin Petkov6a5b3222010-07-13 14:55:28 -0700414 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700415 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700416 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -0700417 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700418 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700419 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800420 metrics::CheckResult::kUpdateAvailable,
421 metrics::CheckReaction::kUpdating,
422 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700423 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700424 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700425 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -0700426 EXPECT_TRUE(response.update_exists);
Alex Deymo8e18f932015-03-27 16:16:59 -0700427 EXPECT_EQ(fake_update_response_.version, response.version);
428 EXPECT_EQ(fake_update_response_.GetPayloadUrl(), response.payload_urls[0]);
429 EXPECT_EQ(fake_update_response_.more_info_url, response.more_info_url);
430 EXPECT_EQ(fake_update_response_.hash, response.hash);
431 EXPECT_EQ(fake_update_response_.size, response.size);
432 EXPECT_EQ(fake_update_response_.prompt == "true", response.prompt);
433 EXPECT_EQ(fake_update_response_.deadline, response.deadline);
434 // Omaha cohort attribets are not set in the response, so they should not be
435 // persisted.
436 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohort));
437 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortHint));
438 EXPECT_FALSE(fake_prefs_.Exists(kPrefsOmahaCohortName));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700439}
440
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700441TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByConnection) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700442 OmahaResponse response;
443 // Set up a connection manager that doesn't allow a valid update over
444 // the current ethernet connection.
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700445 MockConnectionManager mock_cm(nullptr);
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700446 fake_system_state_.set_connection_manager(&mock_cm);
447
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700448 EXPECT_CALL(mock_cm, GetConnectionProperties(_, _, _))
Chris Sosa77f79e82014-06-02 18:16:24 -0700449 .WillRepeatedly(DoAll(SetArgumentPointee<1>(kNetEthernet),
450 SetArgumentPointee<2>(NetworkTethering::kUnknown),
451 Return(true)));
452 EXPECT_CALL(mock_cm, IsUpdateAllowedOver(kNetEthernet, _))
453 .WillRepeatedly(Return(false));
454 EXPECT_CALL(mock_cm, StringForConnectionType(kNetEthernet))
455 .WillRepeatedly(Return(shill::kTypeEthernet));
456
457 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700458 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700459 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700460 -1,
461 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700462 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700463 metrics::CheckResult::kUpdateAvailable,
464 metrics::CheckReaction::kIgnored,
465 metrics::DownloadErrorCode::kUnset,
466 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700467 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700468 EXPECT_FALSE(response.update_exists);
469}
470
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700471TEST_F(OmahaRequestActionTest, ValidUpdateBlockedByRollback) {
Chris Sosa77f79e82014-06-02 18:16:24 -0700472 string rollback_version = "1234.0.0";
473 OmahaResponse response;
474
475 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700476 fake_system_state_.set_payload_state(&mock_payload_state);
477
Chris Sosa77f79e82014-06-02 18:16:24 -0700478 EXPECT_CALL(mock_payload_state, GetRollbackVersion())
479 .WillRepeatedly(Return(rollback_version));
480
Alex Deymo8e18f932015-03-27 16:16:59 -0700481 fake_update_response_.version = rollback_version;
Chris Sosa77f79e82014-06-02 18:16:24 -0700482 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700483 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -0700484 fake_update_response_.GetUpdateResponse(),
Chris Sosa77f79e82014-06-02 18:16:24 -0700485 -1,
486 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700487 ErrorCode::kOmahaUpdateIgnoredPerPolicy,
Chris Sosa77f79e82014-06-02 18:16:24 -0700488 metrics::CheckResult::kUpdateAvailable,
489 metrics::CheckReaction::kIgnored,
490 metrics::DownloadErrorCode::kUnset,
491 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700492 nullptr));
Chris Sosa77f79e82014-06-02 18:16:24 -0700493 EXPECT_FALSE(response.update_exists);
494}
495
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700496TEST_F(OmahaRequestActionTest, WallClockBasedWaitAloneCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700497 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700498 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700499 params.set_wall_clock_based_wait_enabled(true);
500 params.set_update_check_count_wait_enabled(false);
501 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700502
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700503 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700504 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700505 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700506 -1,
507 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700508 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800509 metrics::CheckResult::kUpdateAvailable,
510 metrics::CheckReaction::kDeferring,
511 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700512 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700513 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700514 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700515
516 // Verify if we are interactive check we don't defer.
517 params.set_interactive(true);
518 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700519 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700520 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700521 -1,
522 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700523 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800524 metrics::CheckResult::kUpdateAvailable,
525 metrics::CheckReaction::kUpdating,
526 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700527 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700528 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700529 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700530}
531
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700532TEST_F(OmahaRequestActionTest, NoWallClockBasedWaitCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700533 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700534 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700535 params.set_wall_clock_based_wait_enabled(false);
536 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700537
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700538 params.set_update_check_count_wait_enabled(true);
539 params.set_min_update_checks_needed(1);
540 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700541
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700542 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700543 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700544 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700545 -1,
546 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700547 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800548 metrics::CheckResult::kUpdateAvailable,
549 metrics::CheckReaction::kUpdating,
550 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700551 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700552 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700553 EXPECT_TRUE(response.update_exists);
554}
555
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700556TEST_F(OmahaRequestActionTest, ZeroMaxDaysToScatterCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700557 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700558 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700559 params.set_wall_clock_based_wait_enabled(true);
560 params.set_waiting_period(TimeDelta::FromDays(2));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700561
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700562 params.set_update_check_count_wait_enabled(true);
563 params.set_min_update_checks_needed(1);
564 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700565
Alex Deymo8e18f932015-03-27 16:16:59 -0700566 fake_update_response_.max_days_to_scatter = "0";
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700567 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700568 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700569 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700570 -1,
571 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700572 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800573 metrics::CheckResult::kUpdateAvailable,
574 metrics::CheckReaction::kUpdating,
575 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700576 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700577 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700578 EXPECT_TRUE(response.update_exists);
579}
580
581
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700582TEST_F(OmahaRequestActionTest, ZeroUpdateCheckCountCausesNoScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700583 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700584 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700585 params.set_wall_clock_based_wait_enabled(true);
586 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700587
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700588 params.set_update_check_count_wait_enabled(true);
589 params.set_min_update_checks_needed(0);
590 params.set_max_update_checks_allowed(0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700591
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700592 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700593 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700594 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700595 -1,
596 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700597 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800598 metrics::CheckResult::kUpdateAvailable,
599 metrics::CheckReaction::kUpdating,
600 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700601 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700602 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700603
Ben Chan9abb7632014-08-07 00:10:53 -0700604 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700605 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700606 ASSERT_EQ(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700607 EXPECT_TRUE(response.update_exists);
608}
609
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700610TEST_F(OmahaRequestActionTest, NonZeroUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700611 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700612 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700613 params.set_wall_clock_based_wait_enabled(true);
614 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700615
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700616 params.set_update_check_count_wait_enabled(true);
617 params.set_min_update_checks_needed(1);
618 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700619
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700620 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700621 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700622 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700623 -1,
624 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700625 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800626 metrics::CheckResult::kUpdateAvailable,
627 metrics::CheckReaction::kDeferring,
628 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700629 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700630 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700631
Ben Chan9abb7632014-08-07 00:10:53 -0700632 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700633 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700634 ASSERT_GT(count, 0);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700635 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700636
637 // Verify if we are interactive check we don't defer.
638 params.set_interactive(true);
639 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700640 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700641 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700642 -1,
643 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700644 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800645 metrics::CheckResult::kUpdateAvailable,
646 metrics::CheckReaction::kUpdating,
647 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700648 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700649 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700650 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700651}
652
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700653TEST_F(OmahaRequestActionTest, ExistingUpdateCheckCountCausesScattering) {
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700654 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700655 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700656 params.set_wall_clock_based_wait_enabled(true);
657 params.set_waiting_period(TimeDelta());
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700658
Jay Srinivasanae4697c2013-03-18 17:08:08 -0700659 params.set_update_check_count_wait_enabled(true);
660 params.set_min_update_checks_needed(1);
661 params.set_max_update_checks_allowed(8);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700662
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700663 ASSERT_TRUE(fake_prefs_.SetInt64(kPrefsUpdateCheckCount, 5));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700664
665 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700666 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700667 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700668 -1,
669 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700670 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -0800671 metrics::CheckResult::kUpdateAvailable,
672 metrics::CheckReaction::kDeferring,
673 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700674 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700675 nullptr));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700676
Ben Chan9abb7632014-08-07 00:10:53 -0700677 int64_t count;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700678 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateCheckCount, &count));
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700679 // count remains the same, as the decrementing happens in update_attempter
680 // which this test doesn't exercise.
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700681 ASSERT_EQ(count, 5);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700682 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -0700683
684 // Verify if we are interactive check we don't defer.
685 params.set_interactive(true);
686 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700687 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -0700688 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -0700689 -1,
690 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700691 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800692 metrics::CheckResult::kUpdateAvailable,
693 metrics::CheckReaction::kUpdating,
694 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -0700695 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700696 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -0700697 EXPECT_TRUE(response.update_exists);
Jay Srinivasan480ddfa2012-06-01 19:15:26 -0700698}
Jay Srinivasan0a708742012-03-20 11:26:12 -0700699
Alex Deymo8e18f932015-03-27 16:16:59 -0700700TEST_F(OmahaRequestActionTest, CohortsArePersisted) {
701 OmahaResponse response;
702 OmahaRequestParams params = request_params_;
703 fake_update_response_.include_cohorts = true;
704 fake_update_response_.cohort = "s/154454/8479665";
705 fake_update_response_.cohorthint = "please-put-me-on-beta";
706 fake_update_response_.cohortname = "stable";
707
708 ASSERT_TRUE(TestUpdateCheck(&params,
709 fake_update_response_.GetUpdateResponse(),
710 -1,
711 false, // ping_only
712 ErrorCode::kSuccess,
713 metrics::CheckResult::kUpdateAvailable,
714 metrics::CheckReaction::kUpdating,
715 metrics::DownloadErrorCode::kUnset,
716 &response,
717 nullptr));
718
719 string value;
720 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
721 EXPECT_EQ(fake_update_response_.cohort, value);
722
723 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
724 EXPECT_EQ(fake_update_response_.cohorthint, value);
725
726 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
727 EXPECT_EQ(fake_update_response_.cohortname, value);
728}
729
730TEST_F(OmahaRequestActionTest, CohortsAreUpdated) {
731 OmahaResponse response;
732 OmahaRequestParams params = request_params_;
733 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
734 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortHint, "old_hint"));
735 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohortName, "old_name"));
736 fake_update_response_.include_cohorts = true;
737 fake_update_response_.cohort = "s/154454/8479665";
738 fake_update_response_.cohorthint = "please-put-me-on-beta";
739 fake_update_response_.cohortname = "";
740
741 ASSERT_TRUE(TestUpdateCheck(&params,
742 fake_update_response_.GetUpdateResponse(),
743 -1,
744 false, // ping_only
745 ErrorCode::kSuccess,
746 metrics::CheckResult::kUpdateAvailable,
747 metrics::CheckReaction::kUpdating,
748 metrics::DownloadErrorCode::kUnset,
749 &response,
750 nullptr));
751
752 string value;
753 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
754 EXPECT_EQ(fake_update_response_.cohort, value);
755
756 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
757 EXPECT_EQ(fake_update_response_.cohorthint, value);
758
759 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
760}
761
762TEST_F(OmahaRequestActionTest, CohortsAreNotModifiedWhenMissing) {
763 OmahaResponse response;
764 OmahaRequestParams params = request_params_;
765 EXPECT_TRUE(fake_prefs_.SetString(kPrefsOmahaCohort, "old_value"));
766
767 ASSERT_TRUE(TestUpdateCheck(&params,
768 fake_update_response_.GetUpdateResponse(),
769 -1,
770 false, // ping_only
771 ErrorCode::kSuccess,
772 metrics::CheckResult::kUpdateAvailable,
773 metrics::CheckReaction::kUpdating,
774 metrics::DownloadErrorCode::kUnset,
775 &response,
776 nullptr));
777
778 string value;
779 EXPECT_TRUE(fake_prefs_.GetString(kPrefsOmahaCohort, &value));
780 EXPECT_EQ("old_value", value);
781
782 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortHint, &value));
783 EXPECT_FALSE(fake_prefs_.GetString(kPrefsOmahaCohortName, &value));
784}
785
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700786TEST_F(OmahaRequestActionTest, NoOutputPipeTest) {
Alex Deymo8e18f932015-03-27 16:16:59 -0700787 const string http_response(fake_update_response_.GetNoUpdateResponse());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700788
Alex Deymo60ca1a72015-06-18 18:19:15 -0700789 chromeos::FakeMessageLoop loop(nullptr);
790 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700791
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700792 OmahaRequestParams params = request_params_;
793 fake_system_state_.set_request_params(&params);
794 OmahaRequestAction action(&fake_system_state_, nullptr,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700795 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800796 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700797 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -0700798 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700799 OmahaRequestActionTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700800 ActionProcessor processor;
801 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700802 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700803
Alex Deymo60ca1a72015-06-18 18:19:15 -0700804 loop.PostTask(base::Bind([&processor] { processor.StartProcessing(); }));
805 loop.Run();
806 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700807 EXPECT_FALSE(processor.IsRunning());
808}
809
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700810TEST_F(OmahaRequestActionTest, InvalidXmlTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700811 OmahaResponse response;
812 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700813 TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700814 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700815 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700816 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700817 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800818 metrics::CheckResult::kParsingError,
819 metrics::CheckReaction::kUnset,
820 metrics::DownloadErrorCode::kUnset,
Darin Petkovedc522e2010-11-05 09:35:17 -0700821 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700822 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -0700823 EXPECT_FALSE(response.update_exists);
824}
825
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700826TEST_F(OmahaRequestActionTest, EmptyResponseTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -0700827 OmahaResponse response;
828 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700829 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -0700830 "",
831 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700832 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700833 ErrorCode::kOmahaRequestEmptyResponseError,
David Zeuthen33bae492014-02-25 16:16:18 -0800834 metrics::CheckResult::kParsingError,
835 metrics::CheckReaction::kUnset,
836 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700837 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700838 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700839 EXPECT_FALSE(response.update_exists);
840}
841
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700842TEST_F(OmahaRequestActionTest, MissingStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700843 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700844 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700845 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700846 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
847 "<daystart elapsed_seconds=\"100\"/>"
848 "<app appid=\"foo\" status=\"ok\">"
849 "<ping status=\"ok\"/>"
850 "<updatecheck/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700851 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700852 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700853 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800854 metrics::CheckResult::kParsingError,
855 metrics::CheckReaction::kUnset,
856 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700857 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700858 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700859 EXPECT_FALSE(response.update_exists);
860}
861
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700862TEST_F(OmahaRequestActionTest, InvalidStatusTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700863 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700864 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700865 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700866 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
867 "<daystart elapsed_seconds=\"100\"/>"
868 "<app appid=\"foo\" status=\"ok\">"
869 "<ping status=\"ok\"/>"
870 "<updatecheck status=\"InvalidStatusTest\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700871 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700872 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700873 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800874 metrics::CheckResult::kParsingError,
875 metrics::CheckReaction::kUnset,
876 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700877 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700878 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700879 EXPECT_FALSE(response.update_exists);
880}
881
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700882TEST_F(OmahaRequestActionTest, MissingNodesetTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700883 OmahaResponse response;
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700884 ASSERT_FALSE(TestUpdateCheck(
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700885 nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700886 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
887 "<daystart elapsed_seconds=\"100\"/>"
888 "<app appid=\"foo\" status=\"ok\">"
889 "<ping status=\"ok\"/>"
890 "</app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -0700891 -1,
Darin Petkov265f2902011-05-09 15:17:40 -0700892 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700893 ErrorCode::kOmahaResponseInvalid,
David Zeuthen33bae492014-02-25 16:16:18 -0800894 metrics::CheckResult::kParsingError,
895 metrics::CheckReaction::kUnset,
896 metrics::DownloadErrorCode::kUnset,
Darin Petkov6a5b3222010-07-13 14:55:28 -0700897 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700898 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700899 EXPECT_FALSE(response.update_exists);
900}
901
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700902TEST_F(OmahaRequestActionTest, MissingFieldTest) {
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700903 string input_response =
904 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response protocol=\"3.0\">"
905 "<daystart elapsed_seconds=\"100\"/>"
906 "<app appid=\"xyz\" status=\"ok\">"
907 "<updatecheck status=\"ok\">"
908 "<urls><url codebase=\"http://missing/field/test/\"/></urls>"
Chris Sosa3b748432013-06-20 16:42:59 -0700909 "<manifest version=\"10.2.3.4\">"
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700910 "<packages><package hash=\"not-used\" name=\"f\" "
911 "size=\"587\"/></packages>"
912 "<actions><action event=\"postinstall\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700913 "ChromeOSVersion=\"10.2.3.4\" "
914 "Prompt=\"false\" "
915 "IsDelta=\"true\" "
Jay Srinivasand671e972013-01-11 17:17:19 -0800916 "IsDeltaPayload=\"false\" "
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700917 "sha256=\"lkq34j5345\" "
918 "needsadmin=\"true\" "
919 "/></actions></manifest></updatecheck></app></response>";
920 LOG(INFO) << "Input Response = " << input_response;
921
Darin Petkov6a5b3222010-07-13 14:55:28 -0700922 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700923 ASSERT_TRUE(TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700924 input_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::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -0800928 metrics::CheckResult::kUpdateAvailable,
929 metrics::CheckReaction::kUpdating,
930 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700931 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700932 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -0700933 EXPECT_TRUE(response.update_exists);
Chris Sosa3b748432013-06-20 16:42:59 -0700934 EXPECT_EQ("10.2.3.4", response.version);
Jay Srinivasan6f6ea002012-12-14 11:26:28 -0800935 EXPECT_EQ("http://missing/field/test/f", response.payload_urls[0]);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700936 EXPECT_EQ("", response.more_info_url);
Jay Srinivasan23b92a52012-10-27 02:00:21 -0700937 EXPECT_EQ("lkq34j5345", response.hash);
938 EXPECT_EQ(587, response.size);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700939 EXPECT_FALSE(response.prompt);
Darin Petkov6c118642010-10-21 12:06:30 -0700940 EXPECT_TRUE(response.deadline.empty());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700941}
942
943namespace {
944class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
945 public:
946 void ProcessingStopped(const ActionProcessor* processor) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700947 chromeos::MessageLoop::current()->BreakLoop();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700948 }
Darin Petkov6a5b3222010-07-13 14:55:28 -0700949};
950
Alex Deymo60ca1a72015-06-18 18:19:15 -0700951void TerminateTransferTestStarter(ActionProcessor* processor) {
Darin Petkov6a5b3222010-07-13 14:55:28 -0700952 processor->StartProcessing();
953 CHECK(processor->IsRunning());
954 processor->StopProcessing();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700955}
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700956} // namespace
Darin Petkov6a5b3222010-07-13 14:55:28 -0700957
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700958TEST_F(OmahaRequestActionTest, TerminateTransferTest) {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700959 chromeos::FakeMessageLoop loop(nullptr);
960 loop.SetAsCurrent();
Darin Petkov6a5b3222010-07-13 14:55:28 -0700961
Alex Deymo60ca1a72015-06-18 18:19:15 -0700962 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700963 OmahaRequestAction action(&fake_system_state_, nullptr,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -0700964 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800965 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700966 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -0700967 false);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700968 TerminateEarlyTestProcessorDelegate delegate;
Darin Petkov6a5b3222010-07-13 14:55:28 -0700969 ActionProcessor processor;
970 processor.set_delegate(&delegate);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700971 processor.EnqueueAction(&action);
Darin Petkov6a5b3222010-07-13 14:55:28 -0700972
Alex Deymo60ca1a72015-06-18 18:19:15 -0700973 loop.PostTask(base::Bind(&TerminateTransferTestStarter, &processor));
974 loop.Run();
975 EXPECT_FALSE(loop.PendingTasks());
Darin Petkov6a5b3222010-07-13 14:55:28 -0700976}
977
Alex Deymoe1e3afe2014-10-30 13:02:49 -0700978TEST_F(OmahaRequestActionTest, XmlEncodeTest) {
Alex Deymob0d74eb2015-03-30 17:59:17 -0700979 string output;
980 EXPECT_TRUE(XmlEncode("ab", &output));
981 EXPECT_EQ("ab", output);
982 EXPECT_TRUE(XmlEncode("a<b", &output));
983 EXPECT_EQ("a&lt;b", output);
Alex Deymocc457852015-06-18 18:35:50 -0700984 EXPECT_TRUE(XmlEncode("<&>\"\'\\", &output));
985 EXPECT_EQ("&lt;&amp;&gt;&quot;&apos;\\", output);
Alex Deymob0d74eb2015-03-30 17:59:17 -0700986 EXPECT_TRUE(XmlEncode("&lt;&amp;&gt;", &output));
987 EXPECT_EQ("&amp;lt;&amp;amp;&amp;gt;", output);
Alex Deymocc457852015-06-18 18:35:50 -0700988 // Check that unterminated UTF-8 strings are handled properly.
Alex Deymob0d74eb2015-03-30 17:59:17 -0700989 EXPECT_FALSE(XmlEncode("\xc2", &output));
990 // Fail with invalid ASCII-7 chars.
991 EXPECT_FALSE(XmlEncode("This is an 'n' with a tilde: \xc3\xb1", &output));
992}
Darin Petkov6a5b3222010-07-13 14:55:28 -0700993
Alex Deymob0d74eb2015-03-30 17:59:17 -0700994TEST_F(OmahaRequestActionTest, XmlEncodeWithDefaultTest) {
995 EXPECT_EQ("&lt;&amp;&gt;", XmlEncodeWithDefault("<&>", "something else"));
996 EXPECT_EQ("<not escaped>", XmlEncodeWithDefault("\xc2", "<not escaped>"));
997}
998
999TEST_F(OmahaRequestActionTest, XmlEncodeIsUsedForParams) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001000 chromeos::Blob post_data;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001001
1002 // Make sure XML Encode is being called on the params
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001003 OmahaRequestParams params(&fake_system_state_,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001004 OmahaRequestParams::kOsPlatform,
Darin Petkov6a5b3222010-07-13 14:55:28 -07001005 OmahaRequestParams::kOsVersion,
1006 "testtheservice_pack>",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001007 "x86 generic<id",
Darin Petkov6a5b3222010-07-13 14:55:28 -07001008 OmahaRequestParams::kAppId,
1009 "0.1.0.0",
1010 "en-US",
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001011 "unittest_track&lt;",
Darin Petkovfbb40092010-07-29 17:05:50 -07001012 "<OEM MODEL>",
Chris Sosac1972482013-04-30 22:31:10 -07001013 "ChromeOSFirmware.1.0",
1014 "EC100",
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001015 false, // delta okay
1016 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001017 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001018 ""); // target_version_prefix
Alex Deymo8e18f932015-03-27 16:16:59 -07001019 fake_prefs_.SetString(kPrefsOmahaCohort, "evil\nstring");
1020 fake_prefs_.SetString(kPrefsOmahaCohortHint, "evil&string\\");
1021 fake_prefs_.SetString(kPrefsOmahaCohortName,
1022 JoinString(vector<string>(100, "My spoon is too big."),
1023 ' '));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001024 OmahaResponse response;
1025 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001026 TestUpdateCheck(&params,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001027 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001028 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001029 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001030 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001031 metrics::CheckResult::kParsingError,
1032 metrics::CheckReaction::kUnset,
1033 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001034 &response,
1035 &post_data));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001036 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001037 string post_str(post_data.begin(), post_data.end());
Alex Deymo8e18f932015-03-27 16:16:59 -07001038 EXPECT_NE(string::npos, post_str.find("testtheservice_pack&gt;"));
1039 EXPECT_EQ(string::npos, post_str.find("testtheservice_pack>"));
1040 EXPECT_NE(string::npos, post_str.find("x86 generic&lt;id"));
1041 EXPECT_EQ(string::npos, post_str.find("x86 generic<id"));
1042 EXPECT_NE(string::npos, post_str.find("unittest_track&amp;lt;"));
1043 EXPECT_EQ(string::npos, post_str.find("unittest_track&lt;"));
1044 EXPECT_NE(string::npos, post_str.find("&lt;OEM MODEL&gt;"));
1045 EXPECT_EQ(string::npos, post_str.find("<OEM MODEL>"));
1046 EXPECT_NE(string::npos, post_str.find("cohort=\"evil\nstring\""));
1047 EXPECT_EQ(string::npos, post_str.find("cohorthint=\"evil&string\\\""));
1048 EXPECT_NE(string::npos, post_str.find("cohorthint=\"evil&amp;string\\\""));
1049 // Values from Prefs that are too big are removed from the XML instead of
1050 // encoded.
1051 EXPECT_EQ(string::npos, post_str.find("cohortname="));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001052}
1053
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001054TEST_F(OmahaRequestActionTest, XmlDecodeTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001055 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001056 fake_update_response_.deadline = "&lt;20110101";
1057 fake_update_response_.more_info_url = "testthe&lt;url";
1058 fake_update_response_.codebase = "testthe&amp;codebase/";
Darin Petkov6a5b3222010-07-13 14:55:28 -07001059 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001060 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001061 fake_update_response_.GetUpdateResponse(),
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::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001065 metrics::CheckResult::kUpdateAvailable,
1066 metrics::CheckReaction::kUpdating,
1067 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001068 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001069 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001070
1071 EXPECT_EQ(response.more_info_url, "testthe<url");
Jay Srinivasan6f6ea002012-12-14 11:26:28 -08001072 EXPECT_EQ(response.payload_urls[0], "testthe&codebase/file.signed");
Darin Petkov6c118642010-10-21 12:06:30 -07001073 EXPECT_EQ(response.deadline, "<20110101");
Darin Petkov6a5b3222010-07-13 14:55:28 -07001074}
1075
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001076TEST_F(OmahaRequestActionTest, ParseIntTest) {
Darin Petkov6a5b3222010-07-13 14:55:28 -07001077 OmahaResponse response;
Alex Deymo8e18f932015-03-27 16:16:59 -07001078 // overflows int32_t:
1079 fake_update_response_.size = 123123123123123ll;
Darin Petkov6a5b3222010-07-13 14:55:28 -07001080 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001081 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001082 fake_update_response_.GetUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001083 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001084 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001085 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001086 metrics::CheckResult::kUpdateAvailable,
1087 metrics::CheckReaction::kUpdating,
1088 metrics::DownloadErrorCode::kUnset,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001089 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001090 nullptr));
Darin Petkov6a5b3222010-07-13 14:55:28 -07001091
1092 EXPECT_EQ(response.size, 123123123123123ll);
1093}
1094
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001095TEST_F(OmahaRequestActionTest, FormatUpdateCheckOutputTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001096 chromeos::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001097 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001098 fake_system_state_.set_prefs(&prefs);
1099
Darin Petkov95508da2011-01-05 12:42:29 -08001100 EXPECT_CALL(prefs, GetString(kPrefsPreviousVersion, _))
1101 .WillOnce(DoAll(SetArgumentPointee<1>(string("")), Return(true)));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001102 EXPECT_CALL(prefs, SetString(kPrefsPreviousVersion, _)).Times(1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001103 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001104 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001105 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001106 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001107 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001108 metrics::CheckResult::kParsingError,
1109 metrics::CheckReaction::kUnset,
1110 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001111 nullptr, // response
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001112 &post_data));
1113 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001114 string post_str(post_data.begin(), post_data.end());
Thieu Le116fda32011-04-19 11:01:54 -07001115 EXPECT_NE(post_str.find(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001116 " <ping active=\"1\" a=\"-1\" r=\"-1\"></ping>\n"
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001117 " <updatecheck targetversionprefix=\"\"></updatecheck>\n"),
Jay Srinivasan0a708742012-03-20 11:26:12 -07001118 string::npos);
Darin Petkovfbb40092010-07-29 17:05:50 -07001119 EXPECT_NE(post_str.find("hardware_class=\"OEM MODEL 09235 7471\""),
1120 string::npos);
Chris Sosac1972482013-04-30 22:31:10 -07001121 EXPECT_NE(post_str.find("fw_version=\"ChromeOSFirmware.1.0\""),
1122 string::npos);
1123 EXPECT_NE(post_str.find("ec_version=\"0X0A1\""),
1124 string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001125}
1126
Jay Srinivasan0a708742012-03-20 11:26:12 -07001127
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001128TEST_F(OmahaRequestActionTest, FormatSuccessEventOutputTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001129 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001130 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001131 new OmahaEvent(OmahaEvent::kTypeUpdateDownloadStarted),
1132 "invalid xml>",
1133 &post_data);
1134 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001135 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001136 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001137 " <event eventtype=\"%d\" eventresult=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001138 OmahaEvent::kTypeUpdateDownloadStarted,
1139 OmahaEvent::kResultSuccess);
1140 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001141 EXPECT_EQ(post_str.find("ping"), string::npos);
1142 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkove17f86b2010-07-20 09:12:01 -07001143}
1144
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001145TEST_F(OmahaRequestActionTest, FormatErrorEventOutputTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001146 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001147 TestEvent(request_params_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001148 new OmahaEvent(OmahaEvent::kTypeDownloadComplete,
1149 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001150 ErrorCode::kError),
Darin Petkove17f86b2010-07-20 09:12:01 -07001151 "invalid xml>",
1152 &post_data);
1153 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001154 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001155 string expected_event = base::StringPrintf(
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001156 " <event eventtype=\"%d\" eventresult=\"%d\" "
1157 "errorcode=\"%d\"></event>\n",
Darin Petkove17f86b2010-07-20 09:12:01 -07001158 OmahaEvent::kTypeDownloadComplete,
1159 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001160 static_cast<int>(ErrorCode::kError));
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001161 EXPECT_NE(post_str.find(expected_event), string::npos);
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001162 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001163}
1164
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001165TEST_F(OmahaRequestActionTest, IsEventTest) {
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001166 string http_response("doesn't matter");
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001167 // Create a copy of the OmahaRequestParams to reuse it later.
1168 OmahaRequestParams params = request_params_;
1169 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001170 OmahaRequestAction update_check_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001171 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001172 nullptr,
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001173 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -08001174 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001175 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001176 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001177 EXPECT_FALSE(update_check_action.IsEvent());
1178
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001179 params = request_params_;
1180 fake_system_state_.set_request_params(&params);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001181 OmahaRequestAction event_action(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001182 &fake_system_state_,
Darin Petkove17f86b2010-07-20 09:12:01 -07001183 new OmahaEvent(OmahaEvent::kTypeUpdateComplete),
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001184 new MockHttpFetcher(http_response.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -08001185 http_response.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001186 nullptr),
Thieu Le116fda32011-04-19 11:01:54 -07001187 false);
Darin Petkov0dc8e9a2010-07-14 14:51:57 -07001188 EXPECT_TRUE(event_action.IsEvent());
1189}
1190
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001191TEST_F(OmahaRequestActionTest, FormatDeltaOkayOutputTest) {
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001192 for (int i = 0; i < 2; i++) {
1193 bool delta_okay = i == 1;
1194 const char* delta_okay_str = delta_okay ? "true" : "false";
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001195 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001196 OmahaRequestParams params(&fake_system_state_,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001197 OmahaRequestParams::kOsPlatform,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001198 OmahaRequestParams::kOsVersion,
1199 "service_pack",
1200 "x86-generic",
1201 OmahaRequestParams::kAppId,
1202 "0.1.0.0",
1203 "en-US",
1204 "unittest_track",
Darin Petkovfbb40092010-07-29 17:05:50 -07001205 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001206 "ChromeOSFirmware.1.0",
1207 "EC100",
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001208 delta_okay,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001209 false, // interactive
Jay Srinivasan0a708742012-03-20 11:26:12 -07001210 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001211 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001212 ASSERT_FALSE(TestUpdateCheck(&params,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001213 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001214 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001215 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001216 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001217 metrics::CheckResult::kParsingError,
1218 metrics::CheckReaction::kUnset,
1219 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001220 nullptr,
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001221 &post_data));
1222 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001223 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001224 EXPECT_NE(post_str.find(base::StringPrintf(" delta_okay=\"%s\"",
1225 delta_okay_str)),
Andrew de los Reyes3f0303a2010-07-15 22:35:35 -07001226 string::npos)
1227 << "i = " << i;
1228 }
1229}
1230
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001231TEST_F(OmahaRequestActionTest, FormatInteractiveOutputTest) {
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001232 for (int i = 0; i < 2; i++) {
1233 bool interactive = i == 1;
Gilad Arnold8a659d82013-01-24 11:26:00 -08001234 const char* interactive_str = interactive ? "ondemandupdate" : "scheduler";
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001235 chromeos::Blob post_data;
Gilad Arnold5bb4c902014-04-10 12:32:13 -07001236 FakeSystemState fake_system_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001237 OmahaRequestParams params(&fake_system_state_,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001238 OmahaRequestParams::kOsPlatform,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001239 OmahaRequestParams::kOsVersion,
1240 "service_pack",
1241 "x86-generic",
1242 OmahaRequestParams::kAppId,
1243 "0.1.0.0",
1244 "en-US",
1245 "unittest_track",
1246 "OEM MODEL REV 1234",
Chris Sosac1972482013-04-30 22:31:10 -07001247 "ChromeOSFirmware.1.0",
1248 "EC100",
David Zeuthen8f191b22013-08-06 12:27:50 -07001249 true, // delta_okay
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001250 interactive,
1251 "http://url",
Gilad Arnold74b5f552014-10-07 08:17:16 -07001252 ""); // target_version_prefix
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001253 ASSERT_FALSE(TestUpdateCheck(&params,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001254 "invalid xml>",
1255 -1,
1256 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001257 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001258 metrics::CheckResult::kParsingError,
1259 metrics::CheckReaction::kUnset,
1260 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001261 nullptr,
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001262 &post_data));
1263 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001264 string post_str(post_data.begin(), post_data.end());
Alex Vakulenko75039d72014-03-25 12:36:28 -07001265 EXPECT_NE(post_str.find(base::StringPrintf("installsource=\"%s\"",
1266 interactive_str)),
Gilad Arnoldbbdd4902013-01-10 16:06:30 -08001267 string::npos)
1268 << "i = " << i;
1269 }
1270}
1271
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001272TEST_F(OmahaRequestActionTest, OmahaEventTest) {
Darin Petkove17f86b2010-07-20 09:12:01 -07001273 OmahaEvent default_event;
1274 EXPECT_EQ(OmahaEvent::kTypeUnknown, default_event.type);
1275 EXPECT_EQ(OmahaEvent::kResultError, default_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001276 EXPECT_EQ(ErrorCode::kError, default_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001277
1278 OmahaEvent success_event(OmahaEvent::kTypeUpdateDownloadStarted);
1279 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadStarted, success_event.type);
1280 EXPECT_EQ(OmahaEvent::kResultSuccess, success_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001281 EXPECT_EQ(ErrorCode::kSuccess, success_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001282
1283 OmahaEvent error_event(OmahaEvent::kTypeUpdateDownloadFinished,
1284 OmahaEvent::kResultError,
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001285 ErrorCode::kError);
Darin Petkove17f86b2010-07-20 09:12:01 -07001286 EXPECT_EQ(OmahaEvent::kTypeUpdateDownloadFinished, error_event.type);
1287 EXPECT_EQ(OmahaEvent::kResultError, error_event.result);
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001288 EXPECT_EQ(ErrorCode::kError, error_event.error_code);
Darin Petkove17f86b2010-07-20 09:12:01 -07001289}
1290
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001291void OmahaRequestActionTest::PingTest(bool ping_only) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001292 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001293 fake_system_state_.set_prefs(&prefs);
1294 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1295 .Times(AnyNumber());
1296 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
1297 // Add a few hours to the day difference to test no rounding, etc.
1298 int64_t five_days_ago =
1299 (Time::Now() - TimeDelta::FromHours(5 * 24 + 13)).ToInternalValue();
1300 int64_t six_days_ago =
1301 (Time::Now() - TimeDelta::FromHours(6 * 24 + 11)).ToInternalValue();
1302 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1303 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
1304 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1305 .WillOnce(DoAll(SetArgumentPointee<1>(six_days_ago), Return(true)));
1306 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1307 .WillOnce(DoAll(SetArgumentPointee<1>(five_days_ago), Return(true)));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001308 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001309 ASSERT_TRUE(
1310 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001311 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001312 -1,
1313 ping_only,
1314 ErrorCode::kSuccess,
1315 metrics::CheckResult::kUnset,
1316 metrics::CheckReaction::kUnset,
1317 metrics::DownloadErrorCode::kUnset,
1318 nullptr,
1319 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001320 string post_str(post_data.begin(), post_data.end());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001321 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"6\" r=\"5\"></ping>"),
1322 string::npos);
1323 if (ping_only) {
1324 EXPECT_EQ(post_str.find("updatecheck"), string::npos);
1325 EXPECT_EQ(post_str.find("previousversion"), string::npos);
1326 } else {
1327 EXPECT_NE(post_str.find("updatecheck"), string::npos);
1328 EXPECT_NE(post_str.find("previousversion"), string::npos);
Darin Petkov265f2902011-05-09 15:17:40 -07001329 }
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001330}
1331
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001332TEST_F(OmahaRequestActionTest, PingTestSendOnlyAPing) {
1333 PingTest(true /* ping_only */);
1334}
1335
1336TEST_F(OmahaRequestActionTest, PingTestSendAlsoAnUpdateCheck) {
1337 PingTest(false /* ping_only */);
1338}
1339
1340TEST_F(OmahaRequestActionTest, ActivePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001341 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001342 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001343 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1344 .Times(AnyNumber());
1345 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001346 int64_t three_days_ago =
1347 (Time::Now() - TimeDelta::FromHours(3 * 24 + 12)).ToInternalValue();
1348 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001349 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1350 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001351 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1352 .WillOnce(DoAll(SetArgumentPointee<1>(three_days_ago), Return(true)));
1353 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1354 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001355 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001356 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001357 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001358 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001359 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001360 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001361 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001362 metrics::CheckResult::kNoUpdateAvailable,
1363 metrics::CheckReaction::kUnset,
1364 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001365 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001366 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001367 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001368 EXPECT_NE(post_str.find("<ping active=\"1\" a=\"3\"></ping>"),
Thieu Le116fda32011-04-19 11:01:54 -07001369 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001370}
1371
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001372TEST_F(OmahaRequestActionTest, RollCallPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001373 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001374 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001375 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1376 .Times(AnyNumber());
1377 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001378 int64_t four_days_ago =
1379 (Time::Now() - TimeDelta::FromHours(4 * 24)).ToInternalValue();
1380 int64_t now = Time::Now().ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001381 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1382 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001383 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1384 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1385 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1386 .WillOnce(DoAll(SetArgumentPointee<1>(four_days_ago), Return(true)));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001387 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001388 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001389 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001390 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001391 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001392 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001393 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001394 metrics::CheckResult::kNoUpdateAvailable,
1395 metrics::CheckReaction::kUnset,
1396 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001397 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001398 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001399 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001400 EXPECT_NE(post_str.find("<ping active=\"1\" r=\"4\"></ping>\n"),
Thieu Le116fda32011-04-19 11:01:54 -07001401 string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001402}
1403
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001404TEST_F(OmahaRequestActionTest, NoPingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001405 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001406 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001407 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1408 .Times(AnyNumber());
1409 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001410 int64_t one_hour_ago =
1411 (Time::Now() - TimeDelta::FromHours(1)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001412 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1413 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001414 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1415 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
1416 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1417 .WillOnce(DoAll(SetArgumentPointee<1>(one_hour_ago), Return(true)));
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001418 // LastActivePingDay and PrefsLastRollCallPingDay are set even if we didn't
1419 // send a ping.
1420 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1421 .WillOnce(Return(true));
1422 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1423 .WillOnce(Return(true));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001424 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001425 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001426 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001427 fake_update_response_.GetNoUpdateResponse(),
Darin Petkovedc522e2010-11-05 09:35:17 -07001428 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001429 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001430 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001431 metrics::CheckResult::kNoUpdateAvailable,
1432 metrics::CheckReaction::kUnset,
1433 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001434 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001435 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001436 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001437 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001438}
1439
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001440TEST_F(OmahaRequestActionTest, IgnoreEmptyPingTest) {
Thieu Leb44e9e82011-06-06 14:34:04 -07001441 // This test ensures that we ignore empty ping only requests.
Alex Deymo8427b4a2014-11-05 14:00:32 -08001442 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001443 fake_system_state_.set_prefs(&prefs);
Thieu Leb44e9e82011-06-06 14:34:04 -07001444 int64_t now = Time::Now().ToInternalValue();
1445 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1446 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1447 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1448 .WillOnce(DoAll(SetArgumentPointee<1>(now), Return(true)));
1449 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1450 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001451 chromeos::Blob post_data;
Thieu Leb44e9e82011-06-06 14:34:04 -07001452 EXPECT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001453 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001454 fake_update_response_.GetNoUpdateResponse(),
Thieu Leb44e9e82011-06-06 14:34:04 -07001455 -1,
1456 true, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001457 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001458 metrics::CheckResult::kUnset,
1459 metrics::CheckReaction::kUnset,
1460 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001461 nullptr,
Thieu Leb44e9e82011-06-06 14:34:04 -07001462 &post_data));
1463 EXPECT_EQ(post_data.size(), 0);
1464}
1465
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001466TEST_F(OmahaRequestActionTest, BackInTimePingTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001467 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001468 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001469 EXPECT_CALL(prefs, GetInt64(kPrefsMetricsCheckLastReportingTime, _))
1470 .Times(AnyNumber());
1471 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001472 int64_t future =
1473 (Time::Now() + TimeDelta::FromHours(3 * 24 + 4)).ToInternalValue();
David Zeuthen639aa362014-02-03 16:23:44 -08001474 EXPECT_CALL(prefs, GetInt64(kPrefsInstallDateDays, _))
1475 .WillOnce(DoAll(SetArgumentPointee<1>(0), Return(true)));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001476 EXPECT_CALL(prefs, GetInt64(kPrefsLastActivePingDay, _))
1477 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1478 EXPECT_CALL(prefs, GetInt64(kPrefsLastRollCallPingDay, _))
1479 .WillOnce(DoAll(SetArgumentPointee<1>(future), Return(true)));
1480 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _))
1481 .WillOnce(Return(true));
1482 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _))
1483 .WillOnce(Return(true));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001484 chromeos::Blob post_data;
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001485 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001486 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001487 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1488 "protocol=\"3.0\"><daystart elapsed_seconds=\"100\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001489 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001490 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001491 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001492 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001493 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001494 metrics::CheckResult::kNoUpdateAvailable,
1495 metrics::CheckReaction::kUnset,
1496 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001497 nullptr,
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001498 &post_data));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001499 string post_str(post_data.begin(), post_data.end());
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001500 EXPECT_EQ(post_str.find("ping"), string::npos);
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001501}
1502
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001503TEST_F(OmahaRequestActionTest, LastPingDayUpdateTest) {
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001504 // This test checks that the action updates the last ping day to now
Darin Petkov84c763c2010-07-29 16:27:58 -07001505 // minus 200 seconds with a slack of 5 seconds. Therefore, the test
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001506 // may fail if it runs for longer than 5 seconds. It shouldn't run
1507 // that long though.
1508 int64_t midnight =
1509 (Time::Now() - TimeDelta::FromSeconds(200)).ToInternalValue();
1510 int64_t midnight_slack =
1511 (Time::Now() - TimeDelta::FromSeconds(195)).ToInternalValue();
Alex Deymo8427b4a2014-11-05 14:00:32 -08001512 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001513 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001514 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1515 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001516 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay,
1517 AllOf(Ge(midnight), Le(midnight_slack))))
1518 .WillOnce(Return(true));
1519 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay,
1520 AllOf(Ge(midnight), Le(midnight_slack))))
1521 .WillOnce(Return(true));
1522 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001523 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001524 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1525 "protocol=\"3.0\"><daystart elapsed_seconds=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001526 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001527 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001528 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001529 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001530 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001531 metrics::CheckResult::kNoUpdateAvailable,
1532 metrics::CheckReaction::kUnset,
1533 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001534 nullptr,
1535 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001536}
1537
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001538TEST_F(OmahaRequestActionTest, NoElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001539 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001540 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001541 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1542 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001543 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1544 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1545 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001546 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001547 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1548 "protocol=\"3.0\"><daystart blah=\"200\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001549 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001550 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001551 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001552 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001553 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001554 metrics::CheckResult::kNoUpdateAvailable,
1555 metrics::CheckReaction::kUnset,
1556 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001557 nullptr,
1558 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001559}
1560
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001561TEST_F(OmahaRequestActionTest, BadElapsedSecondsTest) {
Alex Deymo8427b4a2014-11-05 14:00:32 -08001562 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001563 fake_system_state_.set_prefs(&prefs);
David Zeuthen33bae492014-02-25 16:16:18 -08001564 EXPECT_CALL(prefs, GetInt64(_, _)).Times(AnyNumber());
1565 EXPECT_CALL(prefs, SetInt64(_, _)).Times(AnyNumber());
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001566 EXPECT_CALL(prefs, SetInt64(kPrefsLastActivePingDay, _)).Times(0);
1567 EXPECT_CALL(prefs, SetInt64(kPrefsLastRollCallPingDay, _)).Times(0);
1568 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001569 TestUpdateCheck(nullptr, // request_params
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001570 "<?xml version=\"1.0\" encoding=\"UTF-8\"?><response "
1571 "protocol=\"3.0\"><daystart elapsed_seconds=\"x\"/>"
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001572 "<app appid=\"foo\" status=\"ok\"><ping status=\"ok\"/>"
Jay Srinivasan23b92a52012-10-27 02:00:21 -07001573 "<updatecheck status=\"noupdate\"/></app></response>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001574 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001575 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001576 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001577 metrics::CheckResult::kNoUpdateAvailable,
1578 metrics::CheckReaction::kUnset,
1579 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001580 nullptr,
1581 nullptr));
Darin Petkov1cbd78f2010-07-29 12:38:34 -07001582}
1583
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001584TEST_F(OmahaRequestActionTest, NoUniqueIDTest) {
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001585 chromeos::Blob post_data;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001586 ASSERT_FALSE(TestUpdateCheck(nullptr, // request_params
Darin Petkov84c763c2010-07-29 16:27:58 -07001587 "invalid xml>",
Darin Petkovedc522e2010-11-05 09:35:17 -07001588 -1,
Darin Petkov265f2902011-05-09 15:17:40 -07001589 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001590 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001591 metrics::CheckResult::kParsingError,
1592 metrics::CheckReaction::kUnset,
1593 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001594 nullptr, // response
Darin Petkov84c763c2010-07-29 16:27:58 -07001595 &post_data));
1596 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001597 string post_str(post_data.begin(), post_data.end());
Darin Petkov84c763c2010-07-29 16:27:58 -07001598 EXPECT_EQ(post_str.find("machineid="), string::npos);
1599 EXPECT_EQ(post_str.find("userid="), string::npos);
1600}
1601
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001602TEST_F(OmahaRequestActionTest, NetworkFailureTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001603 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001604 const int http_error_code =
1605 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 501;
Darin Petkovedc522e2010-11-05 09:35:17 -07001606 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001607 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001608 "",
1609 501,
Darin Petkov265f2902011-05-09 15:17:40 -07001610 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001611 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001612 metrics::CheckResult::kDownloadError,
1613 metrics::CheckReaction::kUnset,
1614 static_cast<metrics::DownloadErrorCode>(501),
Darin Petkovedc522e2010-11-05 09:35:17 -07001615 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001616 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001617 EXPECT_FALSE(response.update_exists);
1618}
1619
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001620TEST_F(OmahaRequestActionTest, NetworkFailureBadHTTPCodeTest) {
Darin Petkovedc522e2010-11-05 09:35:17 -07001621 OmahaResponse response;
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001622 const int http_error_code =
1623 static_cast<int>(ErrorCode::kOmahaRequestHTTPResponseBase) + 999;
Darin Petkovedc522e2010-11-05 09:35:17 -07001624 ASSERT_FALSE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001625 TestUpdateCheck(nullptr, // request_params
Darin Petkovedc522e2010-11-05 09:35:17 -07001626 "",
1627 1500,
Darin Petkov265f2902011-05-09 15:17:40 -07001628 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001629 static_cast<ErrorCode>(http_error_code),
David Zeuthen33bae492014-02-25 16:16:18 -08001630 metrics::CheckResult::kDownloadError,
1631 metrics::CheckReaction::kUnset,
1632 metrics::DownloadErrorCode::kHttpStatusOther,
Darin Petkovedc522e2010-11-05 09:35:17 -07001633 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001634 nullptr));
Darin Petkovedc522e2010-11-05 09:35:17 -07001635 EXPECT_FALSE(response.update_exists);
1636}
1637
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001638TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsPersistedFirstTime) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001639 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001640 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001641 params.set_wall_clock_based_wait_enabled(true);
1642 params.set_waiting_period(TimeDelta().FromDays(1));
1643 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001644
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001645 ASSERT_FALSE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001646 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001647 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001648 -1,
1649 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001650 ErrorCode::kOmahaUpdateDeferredPerPolicy,
David Zeuthen33bae492014-02-25 16:16:18 -08001651 metrics::CheckResult::kUpdateAvailable,
1652 metrics::CheckReaction::kDeferring,
1653 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001654 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001655 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001656
Ben Chan9abb7632014-08-07 00:10:53 -07001657 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001658 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001659 ASSERT_GT(timestamp, 0);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001660 EXPECT_FALSE(response.update_exists);
Chris Sosa968d0572013-08-23 14:46:02 -07001661
1662 // Verify if we are interactive check we don't defer.
1663 params.set_interactive(true);
1664 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001665 TestUpdateCheck(&params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001666 fake_update_response_.GetUpdateResponse(),
Chris Sosa968d0572013-08-23 14:46:02 -07001667 -1,
1668 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001669 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001670 metrics::CheckResult::kUpdateAvailable,
1671 metrics::CheckReaction::kUpdating,
1672 metrics::DownloadErrorCode::kUnset,
Chris Sosa968d0572013-08-23 14:46:02 -07001673 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001674 nullptr));
Chris Sosa968d0572013-08-23 14:46:02 -07001675 EXPECT_TRUE(response.update_exists);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001676}
1677
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001678TEST_F(OmahaRequestActionTest, TestUpdateFirstSeenAtGetsUsedIfAlreadyPresent) {
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001679 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001680 OmahaRequestParams params = request_params_;
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001681 params.set_wall_clock_based_wait_enabled(true);
1682 params.set_waiting_period(TimeDelta().FromDays(1));
1683 params.set_update_check_count_wait_enabled(false);
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001684
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001685 // Set the timestamp to a very old value such that it exceeds the
1686 // waiting period set above.
1687 Time t1;
1688 Time::FromString("1/1/2012", &t1);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001689 ASSERT_TRUE(fake_prefs_.SetInt64(
1690 kPrefsUpdateFirstSeenAt, t1.ToInternalValue()));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001691 ASSERT_TRUE(TestUpdateCheck(
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001692 &params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001693 fake_update_response_.GetUpdateResponse(),
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001694 -1,
1695 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001696 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001697 metrics::CheckResult::kUpdateAvailable,
1698 metrics::CheckReaction::kUpdating,
1699 metrics::DownloadErrorCode::kUnset,
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001700 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001701 nullptr));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001702
1703 EXPECT_TRUE(response.update_exists);
1704
1705 // Make sure the timestamp t1 is unchanged showing that it was reused.
Ben Chan9abb7632014-08-07 00:10:53 -07001706 int64_t timestamp = 0;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001707 ASSERT_TRUE(fake_prefs_.GetInt64(kPrefsUpdateFirstSeenAt, &timestamp));
Jay Srinivasan34b5d862012-07-23 11:43:22 -07001708 ASSERT_TRUE(timestamp == t1.ToInternalValue());
1709}
1710
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001711TEST_F(OmahaRequestActionTest, TestChangingToMoreStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001712 // Create a uniquely named test directory.
1713 string test_dir;
1714 ASSERT_TRUE(utils::MakeTempDirectory(
1715 "omaha_request_action-test-XXXXXX", &test_dir));
1716
1717 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1718 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001719 kStatefulPartition + "/etc"));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001720 chromeos::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001721 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001722 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001723 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001724 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001725 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1726 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1727 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
1728 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001729 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001730 "CHROMEOS_IS_POWERWASH_ALLOWED=true\n"
1731 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001732 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001733 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001734 params.SetLockDown(false);
1735 params.Init("1.2.3.4", "", 0);
1736 EXPECT_EQ("canary-channel", params.current_channel());
1737 EXPECT_EQ("stable-channel", params.target_channel());
1738 EXPECT_TRUE(params.to_more_stable_channel());
1739 EXPECT_TRUE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001740 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001741 "invalid xml>",
1742 -1,
1743 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001744 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001745 metrics::CheckResult::kParsingError,
1746 metrics::CheckReaction::kUnset,
1747 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001748 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001749 &post_data));
1750 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001751 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001752 EXPECT_NE(string::npos, post_str.find(
1753 "appid=\"{22222222-2222-2222-2222-222222222222}\" "
1754 "version=\"0.0.0.0\" from_version=\"1.2.3.4\" "
1755 "track=\"stable-channel\" from_track=\"canary-channel\" "));
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001756
Alex Deymo10875d92014-11-10 21:52:57 -08001757 ASSERT_TRUE(test_utils::RecursiveUnlinkDir(test_dir));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001758}
1759
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001760TEST_F(OmahaRequestActionTest, TestChangingToLessStableChannel) {
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001761 // Create a uniquely named test directory.
1762 string test_dir;
1763 ASSERT_TRUE(utils::MakeTempDirectory(
1764 "omaha_request_action-test-XXXXXX", &test_dir));
1765
1766 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir + "/etc"));
1767 ASSERT_EQ(0, System(string("mkdir -p ") + test_dir +
Chris Sosabe45bef2013-04-09 18:25:12 -07001768 kStatefulPartition + "/etc"));
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001769 chromeos::Blob post_data;
Alex Deymo8427b4a2014-11-05 14:00:32 -08001770 NiceMock<MockPrefs> prefs;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001771 fake_system_state_.set_prefs(&prefs);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001772 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001773 test_dir + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001774 "CHROMEOS_RELEASE_APPID={11111111-1111-1111-1111-111111111111}\n"
1775 "CHROMEOS_BOARD_APPID={22222222-2222-2222-2222-222222222222}\n"
1776 "CHROMEOS_RELEASE_TRACK=stable-channel\n"));
1777 ASSERT_TRUE(WriteFileString(
Gilad Arnoldeff87cc2013-07-22 18:32:09 -07001778 test_dir + kStatefulPartition + "/etc/lsb-release",
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001779 "CHROMEOS_RELEASE_TRACK=canary-channel\n"));
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001780 OmahaRequestParams params = request_params_;
Gilad Arnoldd04f8e22014-01-09 13:13:40 -08001781 params.set_root(test_dir);
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001782 params.SetLockDown(false);
1783 params.Init("5.6.7.8", "", 0);
1784 EXPECT_EQ("stable-channel", params.current_channel());
1785 EXPECT_EQ("canary-channel", params.target_channel());
1786 EXPECT_FALSE(params.to_more_stable_channel());
1787 EXPECT_FALSE(params.is_powerwash_allowed());
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001788 ASSERT_FALSE(TestUpdateCheck(&params,
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001789 "invalid xml>",
1790 -1,
1791 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001792 ErrorCode::kOmahaRequestXMLParseError,
David Zeuthen33bae492014-02-25 16:16:18 -08001793 metrics::CheckResult::kParsingError,
1794 metrics::CheckReaction::kUnset,
1795 metrics::DownloadErrorCode::kUnset,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001796 nullptr, // response
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001797 &post_data));
1798 // convert post_data to string
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001799 string post_str(post_data.begin(), post_data.end());
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001800 EXPECT_NE(string::npos, post_str.find(
1801 "appid=\"{11111111-1111-1111-1111-111111111111}\" "
1802 "version=\"5.6.7.8\" "
1803 "track=\"canary-channel\" from_track=\"stable-channel\""));
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001804 EXPECT_EQ(string::npos, post_str.find("from_version"));
Jay Srinivasanae4697c2013-03-18 17:08:08 -07001805}
1806
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001807// Checks that the initial ping with a=-1 r=-1 is not send when the device
1808// was powerwashed.
1809TEST_F(OmahaRequestActionTest, PingWhenPowerwashed) {
1810 fake_prefs_.SetString(kPrefsPreviousVersion, "");
1811
1812 // Flag that the device was powerwashed in the past.
1813 fake_system_state_.fake_hardware()->SetPowerwashCount(1);
1814
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001815 chromeos::Blob post_data;
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001816 ASSERT_TRUE(
1817 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001818 fake_update_response_.GetNoUpdateResponse(),
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001819 -1,
1820 false, // ping_only
1821 ErrorCode::kSuccess,
1822 metrics::CheckResult::kNoUpdateAvailable,
1823 metrics::CheckReaction::kUnset,
1824 metrics::DownloadErrorCode::kUnset,
1825 nullptr,
1826 &post_data));
1827 // We shouldn't send a ping in this case since powerwash > 0.
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -08001828 string post_str(post_data.begin(), post_data.end());
Alex Deymoebbe7ef2014-10-30 13:02:49 -07001829 EXPECT_EQ(string::npos, post_str.find("<ping"));
1830}
1831
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001832void OmahaRequestActionTest::P2PTest(
1833 bool initial_allow_p2p_for_downloading,
1834 bool initial_allow_p2p_for_sharing,
1835 bool omaha_disable_p2p_for_downloading,
1836 bool omaha_disable_p2p_for_sharing,
1837 bool payload_state_allow_p2p_attempt,
1838 bool expect_p2p_client_lookup,
1839 const string& p2p_client_result_url,
1840 bool expected_allow_p2p_for_downloading,
1841 bool expected_allow_p2p_for_sharing,
1842 const string& expected_p2p_url) {
David Zeuthen8f191b22013-08-06 12:27:50 -07001843 OmahaResponse response;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001844 OmahaRequestParams request_params = request_params_;
Gilad Arnold74b5f552014-10-07 08:17:16 -07001845 bool actual_allow_p2p_for_downloading = initial_allow_p2p_for_downloading;
1846 bool actual_allow_p2p_for_sharing = initial_allow_p2p_for_sharing;
1847 string actual_p2p_url;
David Zeuthen8f191b22013-08-06 12:27:50 -07001848
1849 MockPayloadState mock_payload_state;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001850 fake_system_state_.set_payload_state(&mock_payload_state);
David Zeuthen8f191b22013-08-06 12:27:50 -07001851 EXPECT_CALL(mock_payload_state, P2PAttemptAllowed())
1852 .WillRepeatedly(Return(payload_state_allow_p2p_attempt));
Gilad Arnold74b5f552014-10-07 08:17:16 -07001853 EXPECT_CALL(mock_payload_state, GetUsingP2PForDownloading())
1854 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_downloading));
1855 EXPECT_CALL(mock_payload_state, GetUsingP2PForSharing())
1856 .WillRepeatedly(ReturnPointee(&actual_allow_p2p_for_sharing));
1857 EXPECT_CALL(mock_payload_state, SetUsingP2PForDownloading(_))
1858 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_downloading));
1859 EXPECT_CALL(mock_payload_state, SetUsingP2PForSharing(_))
1860 .WillRepeatedly(SaveArg<0>(&actual_allow_p2p_for_sharing));
1861 EXPECT_CALL(mock_payload_state, SetP2PUrl(_))
1862 .WillRepeatedly(SaveArg<0>(&actual_p2p_url));
1863
David Zeuthen8f191b22013-08-06 12:27:50 -07001864 MockP2PManager mock_p2p_manager;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001865 fake_system_state_.set_p2p_manager(&mock_p2p_manager);
David Zeuthen8f191b22013-08-06 12:27:50 -07001866 mock_p2p_manager.fake().SetLookupUrlForFileResult(p2p_client_result_url);
1867
David Zeuthen4cc5ed22014-01-15 12:35:03 -08001868 TimeDelta timeout = TimeDelta::FromSeconds(kMaxP2PNetworkWaitTimeSeconds);
1869 EXPECT_CALL(mock_p2p_manager, LookupUrlForFile(_, _, timeout, _))
David Zeuthen8f191b22013-08-06 12:27:50 -07001870 .Times(expect_p2p_client_lookup ? 1 : 0);
1871
Alex Deymo8e18f932015-03-27 16:16:59 -07001872 fake_update_response_.disable_p2p_for_downloading =
1873 omaha_disable_p2p_for_downloading;
1874 fake_update_response_.disable_p2p_for_sharing = omaha_disable_p2p_for_sharing;
David Zeuthen8f191b22013-08-06 12:27:50 -07001875 ASSERT_TRUE(
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001876 TestUpdateCheck(&request_params,
Alex Deymo8e18f932015-03-27 16:16:59 -07001877 fake_update_response_.GetUpdateResponse(),
David Zeuthen8f191b22013-08-06 12:27:50 -07001878 -1,
1879 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001880 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001881 metrics::CheckResult::kUpdateAvailable,
1882 metrics::CheckReaction::kUpdating,
1883 metrics::DownloadErrorCode::kUnset,
David Zeuthen8f191b22013-08-06 12:27:50 -07001884 &response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001885 nullptr));
David Zeuthen8f191b22013-08-06 12:27:50 -07001886 EXPECT_TRUE(response.update_exists);
1887
Gilad Arnold74b5f552014-10-07 08:17:16 -07001888 EXPECT_EQ(omaha_disable_p2p_for_downloading,
1889 response.disable_p2p_for_downloading);
1890 EXPECT_EQ(omaha_disable_p2p_for_sharing,
1891 response.disable_p2p_for_sharing);
David Zeuthen8f191b22013-08-06 12:27:50 -07001892
Gilad Arnold74b5f552014-10-07 08:17:16 -07001893 EXPECT_EQ(expected_allow_p2p_for_downloading,
1894 actual_allow_p2p_for_downloading);
1895 EXPECT_EQ(expected_allow_p2p_for_sharing, actual_allow_p2p_for_sharing);
1896 EXPECT_EQ(expected_p2p_url, actual_p2p_url);
David Zeuthen8f191b22013-08-06 12:27:50 -07001897}
1898
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001899TEST_F(OmahaRequestActionTest, P2PWithPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001900 P2PTest(true, // initial_allow_p2p_for_downloading
1901 true, // initial_allow_p2p_for_sharing
1902 false, // omaha_disable_p2p_for_downloading
1903 false, // omaha_disable_p2p_for_sharing
1904 true, // payload_state_allow_p2p_attempt
1905 true, // expect_p2p_client_lookup
1906 "http://1.3.5.7/p2p", // p2p_client_result_url
1907 true, // expected_allow_p2p_for_downloading
1908 true, // expected_allow_p2p_for_sharing
1909 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001910}
1911
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001912TEST_F(OmahaRequestActionTest, P2PWithoutPeer) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001913 P2PTest(true, // initial_allow_p2p_for_downloading
1914 true, // initial_allow_p2p_for_sharing
1915 false, // omaha_disable_p2p_for_downloading
1916 false, // omaha_disable_p2p_for_sharing
1917 true, // payload_state_allow_p2p_attempt
1918 true, // expect_p2p_client_lookup
1919 "", // p2p_client_result_url
1920 false, // expected_allow_p2p_for_downloading
1921 true, // expected_allow_p2p_for_sharing
1922 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001923}
1924
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001925TEST_F(OmahaRequestActionTest, P2PDownloadNotAllowed) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001926 P2PTest(false, // initial_allow_p2p_for_downloading
1927 true, // initial_allow_p2p_for_sharing
1928 false, // omaha_disable_p2p_for_downloading
1929 false, // omaha_disable_p2p_for_sharing
1930 true, // payload_state_allow_p2p_attempt
1931 false, // expect_p2p_client_lookup
1932 "unset", // p2p_client_result_url
1933 false, // expected_allow_p2p_for_downloading
1934 true, // expected_allow_p2p_for_sharing
1935 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001936}
1937
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001938TEST_F(OmahaRequestActionTest, P2PWithPeerDownloadDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001939 P2PTest(true, // initial_allow_p2p_for_downloading
1940 true, // initial_allow_p2p_for_sharing
1941 true, // omaha_disable_p2p_for_downloading
1942 false, // omaha_disable_p2p_for_sharing
1943 true, // payload_state_allow_p2p_attempt
1944 false, // expect_p2p_client_lookup
1945 "unset", // p2p_client_result_url
1946 false, // expected_allow_p2p_for_downloading
1947 true, // expected_allow_p2p_for_sharing
1948 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001949}
1950
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001951TEST_F(OmahaRequestActionTest, P2PWithPeerSharingDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001952 P2PTest(true, // initial_allow_p2p_for_downloading
1953 true, // initial_allow_p2p_for_sharing
1954 false, // omaha_disable_p2p_for_downloading
1955 true, // omaha_disable_p2p_for_sharing
1956 true, // payload_state_allow_p2p_attempt
1957 true, // expect_p2p_client_lookup
1958 "http://1.3.5.7/p2p", // p2p_client_result_url
1959 true, // expected_allow_p2p_for_downloading
1960 false, // expected_allow_p2p_for_sharing
1961 "http://1.3.5.7/p2p"); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001962}
1963
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001964TEST_F(OmahaRequestActionTest, P2PWithPeerBothDisabledByOmaha) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -07001965 P2PTest(true, // initial_allow_p2p_for_downloading
1966 true, // initial_allow_p2p_for_sharing
1967 true, // omaha_disable_p2p_for_downloading
1968 true, // omaha_disable_p2p_for_sharing
1969 true, // payload_state_allow_p2p_attempt
1970 false, // expect_p2p_client_lookup
1971 "unset", // p2p_client_result_url
1972 false, // expected_allow_p2p_for_downloading
1973 false, // expected_allow_p2p_for_sharing
1974 ""); // expected_p2p_url
David Zeuthen8f191b22013-08-06 12:27:50 -07001975}
1976
Alex Deymof329b932014-10-30 01:37:48 -07001977bool OmahaRequestActionTest::InstallDateParseHelper(const string &elapsed_days,
1978 OmahaResponse *response) {
Alex Deymo8e18f932015-03-27 16:16:59 -07001979 fake_update_response_.elapsed_days = elapsed_days;
David Zeuthen639aa362014-02-03 16:23:44 -08001980 return
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001981 TestUpdateCheck(nullptr, // request_params
Alex Deymo8e18f932015-03-27 16:16:59 -07001982 fake_update_response_.GetUpdateResponse(),
David Zeuthen639aa362014-02-03 16:23:44 -08001983 -1,
1984 false, // ping_only
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -07001985 ErrorCode::kSuccess,
David Zeuthen33bae492014-02-25 16:16:18 -08001986 metrics::CheckResult::kUpdateAvailable,
1987 metrics::CheckReaction::kUpdating,
1988 metrics::DownloadErrorCode::kUnset,
David Zeuthen639aa362014-02-03 16:23:44 -08001989 response,
Alex Vakulenko88b591f2014-08-28 16:48:57 -07001990 nullptr);
David Zeuthen639aa362014-02-03 16:23:44 -08001991}
1992
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001993TEST_F(OmahaRequestActionTest, ParseInstallDateFromResponse) {
David Zeuthen639aa362014-02-03 16:23:44 -08001994 OmahaResponse response;
David Zeuthen639aa362014-02-03 16:23:44 -08001995
1996 // Check that we parse elapsed_days in the Omaha Response correctly.
1997 // and that the kPrefsInstallDateDays value is written to.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07001998 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
1999 EXPECT_TRUE(InstallDateParseHelper("42", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002000 EXPECT_TRUE(response.update_exists);
2001 EXPECT_EQ(42, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002002 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002003 int64_t prefs_days;
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002004 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002005 EXPECT_EQ(prefs_days, 42);
2006
2007 // If there already is a value set, we shouldn't do anything.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002008 EXPECT_TRUE(InstallDateParseHelper("7", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002009 EXPECT_TRUE(response.update_exists);
2010 EXPECT_EQ(7, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002011 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002012 EXPECT_EQ(prefs_days, 42);
2013
2014 // Note that elapsed_days is not necessarily divisible by 7 so check
2015 // that we round down correctly when populating kPrefsInstallDateDays.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002016 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2017 EXPECT_TRUE(InstallDateParseHelper("23", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002018 EXPECT_TRUE(response.update_exists);
2019 EXPECT_EQ(23, response.install_date_days);
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002020 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
David Zeuthen639aa362014-02-03 16:23:44 -08002021 EXPECT_EQ(prefs_days, 21);
2022
2023 // Check that we correctly handle elapsed_days not being included in
2024 // the Omaha Response.
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002025 EXPECT_TRUE(InstallDateParseHelper("", &response));
David Zeuthen639aa362014-02-03 16:23:44 -08002026 EXPECT_TRUE(response.update_exists);
2027 EXPECT_EQ(-1, response.install_date_days);
David Zeuthen639aa362014-02-03 16:23:44 -08002028}
2029
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002030// If there is no prefs and OOBE is not complete, we should not
2031// report anything to Omaha.
2032TEST_F(OmahaRequestActionTest, GetInstallDateWhenNoPrefsNorOOBE) {
2033 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2034 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2035}
David Zeuthen639aa362014-02-03 16:23:44 -08002036
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002037// If OOBE is complete and happened on a valid date (e.g. after Jan
2038// 1 2007 0:00 PST), that date should be used and written to
2039// prefs. However, first try with an invalid date and check we do
2040// nothing.
2041TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithInvalidDate) {
2042 Time oobe_date = Time::FromTimeT(42); // Dec 31, 1969 16:00:42 PST.
2043 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2044 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), -1);
2045 EXPECT_FALSE(fake_prefs_.Exists(kPrefsInstallDateDays));
2046}
David Zeuthen639aa362014-02-03 16:23:44 -08002047
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002048// Then check with a valid date. The date Jan 20, 2007 0:00 PST
2049// should yield an InstallDate of 14.
2050TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedWithValidDate) {
2051 Time oobe_date = Time::FromTimeT(1169280000); // Jan 20, 2007 0:00 PST.
2052 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2053 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
2054 EXPECT_TRUE(fake_prefs_.Exists(kPrefsInstallDateDays));
David Zeuthen639aa362014-02-03 16:23:44 -08002055
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002056 int64_t prefs_days;
2057 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2058 EXPECT_EQ(prefs_days, 14);
2059}
David Zeuthen639aa362014-02-03 16:23:44 -08002060
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002061// Now that we have a valid date in prefs, check that we keep using
2062// that even if OOBE date reports something else. The date Jan 30,
2063// 2007 0:00 PST should yield an InstallDate of 28... but since
2064// there's a prefs file, we should still get 14.
2065TEST_F(OmahaRequestActionTest, GetInstallDateWhenOOBECompletedDateChanges) {
2066 // Set a valid date in the prefs first.
2067 EXPECT_TRUE(fake_prefs_.SetInt64(kPrefsInstallDateDays, 14));
David Zeuthen639aa362014-02-03 16:23:44 -08002068
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002069 Time oobe_date = Time::FromTimeT(1170144000); // Jan 30, 2007 0:00 PST.
2070 fake_system_state_.fake_hardware()->SetIsOOBEComplete(oobe_date);
2071 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002072
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002073 int64_t prefs_days;
2074 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2075 EXPECT_EQ(prefs_days, 14);
David Zeuthen639aa362014-02-03 16:23:44 -08002076
Alex Deymoe1e3afe2014-10-30 13:02:49 -07002077 // If we delete the prefs file, we should get 28 days.
2078 EXPECT_TRUE(fake_prefs_.Delete(kPrefsInstallDateDays));
2079 EXPECT_EQ(OmahaRequestAction::GetInstallDate(&fake_system_state_), 28);
2080 EXPECT_TRUE(fake_prefs_.GetInt64(kPrefsInstallDateDays, &prefs_days));
2081 EXPECT_EQ(prefs_days, 28);
David Zeuthen639aa362014-02-03 16:23:44 -08002082}
2083
Darin Petkov6a5b3222010-07-13 14:55:28 -07002084} // namespace chromeos_update_engine