blob: f42b1d8eb3a85c49d76b14bdb45ce407a0abe120 [file] [log] [blame]
Alex Deymoaea4c1c2015-08-19 20:24:43 -07001//
2// Copyright (C) 2011 The Android Open Source Project
3//
4// Licensed under the Apache License, Version 2.0 (the "License");
5// you may not use this file except in compliance with the License.
6// You may obtain a copy of the License at
7//
8// http://www.apache.org/licenses/LICENSE-2.0
9//
10// Unless required by applicable law or agreed to in writing, software
11// distributed under the License is distributed on an "AS IS" BASIS,
12// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13// See the License for the specific language governing permissions and
14// limitations under the License.
15//
rspangler@google.com49fdf182009-10-10 00:57:34 +000016
Alex Deymo39910dc2015-11-09 17:04:30 -080017#include "update_engine/payload_consumer/download_action.h"
Alex Deymo8427b4a2014-11-05 14:00:32 -080018
Darin Petkov9d911fa2010-08-19 09:36:08 -070019#include <gmock/gmock.h>
rspangler@google.com49fdf182009-10-10 00:57:34 +000020#include <gtest/gtest.h>
Darin Petkov73058b42010-10-06 16:32:19 -070021
Ben Chan02f7c1d2014-10-18 15:18:02 -070022#include <memory>
David Zeuthen8f191b22013-08-06 12:27:50 -070023#include <string>
24#include <utility>
25#include <vector>
26
Alex Deymo60ca1a72015-06-18 18:19:15 -070027#include <base/bind.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070028#include <base/files/file_path.h>
Ben Chan06c76a42014-09-05 08:21:06 -070029#include <base/files/file_util.h>
Alex Deymo60ca1a72015-06-18 18:19:15 -070030#include <base/location.h>
Alex Vakulenko75039d72014-03-25 12:36:28 -070031#include <base/strings/stringprintf.h>
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070032#include <brillo/bind_lambda.h>
33#include <brillo/message_loops/fake_message_loop.h>
34#include <brillo/message_loops/message_loop.h>
David Zeuthen8f191b22013-08-06 12:27:50 -070035
Alex Deymo39910dc2015-11-09 17:04:30 -080036#include "update_engine/common/action_pipe.h"
37#include "update_engine/common/hash_calculator.h"
38#include "update_engine/common/mock_http_fetcher.h"
39#include "update_engine/common/mock_prefs.h"
40#include "update_engine/common/test_utils.h"
41#include "update_engine/common/utils.h"
David Zeuthen8f191b22013-08-06 12:27:50 -070042#include "update_engine/fake_p2p_manager_configuration.h"
Gilad Arnold5bb4c902014-04-10 12:32:13 -070043#include "update_engine/fake_system_state.h"
Aaron Wood9321f502017-09-07 11:18:54 -070044#include "update_engine/mock_file_writer.h"
Alex Deymo542c19b2015-12-03 07:43:31 -030045#include "update_engine/payload_consumer/mock_download_action.h"
Gilad Arnold4a0321b2014-10-28 15:57:30 -070046#include "update_engine/update_manager/fake_update_manager.h"
rspangler@google.com49fdf182009-10-10 00:57:34 +000047
48namespace chromeos_update_engine {
49
Alex Deymof329b932014-10-30 01:37:48 -070050using base::FilePath;
51using base::ReadFileToString;
52using base::WriteFile;
rspangler@google.com49fdf182009-10-10 00:57:34 +000053using std::string;
Ben Chan02f7c1d2014-10-18 15:18:02 -070054using std::unique_ptr;
rspangler@google.com49fdf182009-10-10 00:57:34 +000055using std::vector;
Alex Deymoe5e5fe92015-10-05 09:28:19 -070056using test_utils::ScopedTempFile;
Darin Petkov9d911fa2010-08-19 09:36:08 -070057using testing::AtLeast;
58using testing::InSequence;
Gilad Arnold74b5f552014-10-07 08:17:16 -070059using testing::Return;
Aaron Wood9321f502017-09-07 11:18:54 -070060using testing::SetArgPointee;
Alex Deymof329b932014-10-30 01:37:48 -070061using testing::_;
rspangler@google.com49fdf182009-10-10 00:57:34 +000062
63class DownloadActionTest : public ::testing::Test { };
64
65namespace {
Darin Petkov9d911fa2010-08-19 09:36:08 -070066
rspangler@google.com49fdf182009-10-10 00:57:34 +000067class DownloadActionTestProcessorDelegate : public ActionProcessorDelegate {
68 public:
David Zeuthena99981f2013-04-29 13:42:47 -070069 explicit DownloadActionTestProcessorDelegate(ErrorCode expected_code)
Alex Deymo60ca1a72015-06-18 18:19:15 -070070 : processing_done_called_(false),
Darin Petkovc97435c2010-07-20 12:37:43 -070071 expected_code_(expected_code) {}
Alex Deymo610277e2014-11-11 21:18:11 -080072 ~DownloadActionTestProcessorDelegate() override {
rspangler@google.com49fdf182009-10-10 00:57:34 +000073 EXPECT_TRUE(processing_done_called_);
74 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -080075 void ProcessingDone(const ActionProcessor* processor,
76 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070077 brillo::MessageLoop::current()->BreakLoop();
78 brillo::Blob found_data;
adlr@google.comc98a7ed2009-12-04 18:54:03 +000079 ASSERT_TRUE(utils::ReadFile(path_, &found_data));
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -070080 if (expected_code_ != ErrorCode::kDownloadWriteError) {
Darin Petkov9ce452b2010-11-17 14:33:28 -080081 ASSERT_EQ(expected_data_.size(), found_data.size());
82 for (unsigned i = 0; i < expected_data_.size(); i++) {
83 EXPECT_EQ(expected_data_[i], found_data[i]);
84 }
rspangler@google.com49fdf182009-10-10 00:57:34 +000085 }
86 processing_done_called_ = true;
87 }
88
Yunlian Jiang35866ed2015-01-29 13:09:20 -080089 void ActionCompleted(ActionProcessor* processor,
90 AbstractAction* action,
91 ErrorCode code) override {
Darin Petkovc97435c2010-07-20 12:37:43 -070092 const string type = action->Type();
93 if (type == DownloadAction::StaticType()) {
94 EXPECT_EQ(expected_code_, code);
95 } else {
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -070096 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkovc97435c2010-07-20 12:37:43 -070097 }
rspangler@google.com49fdf182009-10-10 00:57:34 +000098 }
99
rspangler@google.com49fdf182009-10-10 00:57:34 +0000100 string path_;
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700101 brillo::Blob expected_data_;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000102 bool processing_done_called_;
David Zeuthena99981f2013-04-29 13:42:47 -0700103 ErrorCode expected_code_;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000104};
105
Darin Petkov9ce452b2010-11-17 14:33:28 -0800106class TestDirectFileWriter : public DirectFileWriter {
107 public:
108 TestDirectFileWriter() : fail_write_(0), current_write_(0) {}
109 void set_fail_write(int fail_write) { fail_write_ = fail_write; }
110
Don Garrette410e0f2011-11-10 15:39:01 -0800111 virtual bool Write(const void* bytes, size_t count) {
Darin Petkov9ce452b2010-11-17 14:33:28 -0800112 if (++current_write_ == fail_write_) {
Don Garrette410e0f2011-11-10 15:39:01 -0800113 return false;
Darin Petkov9ce452b2010-11-17 14:33:28 -0800114 }
115 return DirectFileWriter::Write(bytes, count);
116 }
117
118 private:
119 // If positive, fail on the |fail_write_| call to Write.
120 int fail_write_;
121 int current_write_;
122};
123
Alex Deymo60ca1a72015-06-18 18:19:15 -0700124void StartProcessorInRunLoop(ActionProcessor* processor,
125 MockHttpFetcher* http_fetcher) {
rspangler@google.com49fdf182009-10-10 00:57:34 +0000126 processor->StartProcessing();
Andrew de los Reyes34e41a12010-10-26 20:07:58 -0700127 http_fetcher->SetOffset(1);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000128}
129
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700130void TestWithData(const brillo::Blob& data,
Darin Petkov9ce452b2010-11-17 14:33:28 -0800131 int fail_write,
Darin Petkov9d911fa2010-08-19 09:36:08 -0700132 bool use_download_delegate) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700133 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700134 loop.SetAsCurrent();
Alex Deymo5ed695e2015-10-05 16:59:23 -0700135 FakeSystemState fake_system_state;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000136
137 // TODO(adlr): see if we need a different file for build bots
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700138 ScopedTempFile output_temp_file;
Darin Petkov9ce452b2010-11-17 14:33:28 -0800139 TestDirectFileWriter writer;
Alex Deymobffa0602016-02-12 17:16:29 -0800140 EXPECT_EQ(
141 0, writer.Open(output_temp_file.path().c_str(), O_WRONLY | O_CREAT, 0));
Darin Petkov9ce452b2010-11-17 14:33:28 -0800142 writer.set_fail_write(fail_write);
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700143
Sen Jiang5ae865b2017-04-18 14:24:40 -0700144 uint64_t size = data.size() - 1;
Alex Deymo64d98782016-02-05 18:03:48 -0800145 InstallPlan install_plan;
Sen Jiangcdd52062017-05-18 15:33:10 -0700146 install_plan.payloads.push_back(
147 {.size = size, .type = InstallPayloadType::kDelta});
Sen Jiang2703ef42017-03-16 13:36:21 -0700148 // We pull off the first byte from data and seek past it.
149 EXPECT_TRUE(HashCalculator::RawHashOfBytes(
Sen Jiang0affc2c2017-02-10 15:55:05 -0800150 &data[1], data.size() - 1, &install_plan.payloads[0].hash));
Alex Deymo5ed695e2015-10-05 16:59:23 -0700151 install_plan.source_slot = 0;
152 install_plan.target_slot = 1;
153 // We mark both slots as bootable. Only the target slot should be unbootable
154 // after the download starts.
155 fake_system_state.fake_boot_control()->SetSlotBootable(
156 install_plan.source_slot, true);
157 fake_system_state.fake_boot_control()->SetSlotBootable(
158 install_plan.target_slot, true);
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000159 ObjectFeederAction<InstallPlan> feeder_action;
160 feeder_action.set_obj(install_plan);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800161 MockPrefs prefs;
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800162 MockHttpFetcher* http_fetcher = new MockHttpFetcher(data.data(),
Andrew de los Reyes45168102010-11-22 11:13:50 -0800163 data.size(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700164 nullptr);
Andrew de los Reyes34e41a12010-10-26 20:07:58 -0700165 // takes ownership of passed in HttpFetcher
Alex Deymo1b3556c2016-02-03 09:54:02 -0800166 DownloadAction download_action(&prefs,
167 fake_system_state.boot_control(),
168 fake_system_state.hardware(),
169 &fake_system_state,
170 http_fetcher);
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700171 download_action.SetTestFileWriter(&writer);
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000172 BondActions(&feeder_action, &download_action);
Alex Deymo542c19b2015-12-03 07:43:31 -0300173 MockDownloadActionDelegate download_delegate;
Darin Petkov9d911fa2010-08-19 09:36:08 -0700174 if (use_download_delegate) {
175 InSequence s;
176 download_action.set_delegate(&download_delegate);
Andrew de los Reyes34e41a12010-10-26 20:07:58 -0700177 if (data.size() > kMockHttpFetcherChunkSize)
178 EXPECT_CALL(download_delegate,
Sen Jiang5ae865b2017-04-18 14:24:40 -0700179 BytesReceived(_, kMockHttpFetcherChunkSize, _));
Alex Deymo542c19b2015-12-03 07:43:31 -0300180 EXPECT_CALL(download_delegate, BytesReceived(_, _, _)).Times(AtLeast(1));
Darin Petkov9d911fa2010-08-19 09:36:08 -0700181 }
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700182 ErrorCode expected_code = ErrorCode::kSuccess;
Darin Petkov7ed561b2011-10-04 02:59:03 -0700183 if (fail_write > 0)
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700184 expected_code = ErrorCode::kDownloadWriteError;
Darin Petkov50332f12010-09-24 11:44:47 -0700185 DownloadActionTestProcessorDelegate delegate(expected_code);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700186 delegate.expected_data_ = brillo::Blob(data.begin() + 1, data.end());
Alex Deymobffa0602016-02-12 17:16:29 -0800187 delegate.path_ = output_temp_file.path();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000188 ActionProcessor processor;
189 processor.set_delegate(&delegate);
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000190 processor.EnqueueAction(&feeder_action);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000191 processor.EnqueueAction(&download_action);
192
Alex Deymo60ca1a72015-06-18 18:19:15 -0700193 loop.PostTask(FROM_HERE,
194 base::Bind(&StartProcessorInRunLoop, &processor, http_fetcher));
195 loop.Run();
196 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo5ed695e2015-10-05 16:59:23 -0700197
198 EXPECT_TRUE(fake_system_state.fake_boot_control()->IsSlotBootable(
199 install_plan.source_slot));
200 EXPECT_FALSE(fake_system_state.fake_boot_control()->IsSlotBootable(
201 install_plan.target_slot));
rspangler@google.com49fdf182009-10-10 00:57:34 +0000202}
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700203} // namespace
rspangler@google.com49fdf182009-10-10 00:57:34 +0000204
205TEST(DownloadActionTest, SimpleTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700206 brillo::Blob small;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000207 const char* foo = "foo";
208 small.insert(small.end(), foo, foo + strlen(foo));
Darin Petkov50332f12010-09-24 11:44:47 -0700209 TestWithData(small,
Darin Petkov9ce452b2010-11-17 14:33:28 -0800210 0, // fail_write
Darin Petkov50332f12010-09-24 11:44:47 -0700211 true); // use_download_delegate
rspangler@google.com49fdf182009-10-10 00:57:34 +0000212}
213
214TEST(DownloadActionTest, LargeTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700215 brillo::Blob big(5 * kMockHttpFetcherChunkSize);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000216 char c = '0';
217 for (unsigned int i = 0; i < big.size(); i++) {
218 big[i] = c;
Darin Petkov9ce452b2010-11-17 14:33:28 -0800219 c = ('9' == c) ? '0' : c + 1;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000220 }
Darin Petkov50332f12010-09-24 11:44:47 -0700221 TestWithData(big,
Darin Petkov9ce452b2010-11-17 14:33:28 -0800222 0, // fail_write
223 true); // use_download_delegate
224}
225
226TEST(DownloadActionTest, FailWriteTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700227 brillo::Blob big(5 * kMockHttpFetcherChunkSize);
Darin Petkov9ce452b2010-11-17 14:33:28 -0800228 char c = '0';
229 for (unsigned int i = 0; i < big.size(); i++) {
230 big[i] = c;
231 c = ('9' == c) ? '0' : c + 1;
232 }
233 TestWithData(big,
Darin Petkov9ce452b2010-11-17 14:33:28 -0800234 2, // fail_write
Darin Petkov50332f12010-09-24 11:44:47 -0700235 true); // use_download_delegate
Darin Petkovc97435c2010-07-20 12:37:43 -0700236}
237
Darin Petkov9d911fa2010-08-19 09:36:08 -0700238TEST(DownloadActionTest, NoDownloadDelegateTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700239 brillo::Blob small;
Darin Petkov9d911fa2010-08-19 09:36:08 -0700240 const char* foo = "foofoo";
241 small.insert(small.end(), foo, foo + strlen(foo));
Darin Petkov50332f12010-09-24 11:44:47 -0700242 TestWithData(small,
Darin Petkov9ce452b2010-11-17 14:33:28 -0800243 0, // fail_write
Darin Petkov50332f12010-09-24 11:44:47 -0700244 false); // use_download_delegate
rspangler@google.com49fdf182009-10-10 00:57:34 +0000245}
246
Aaron Wood9321f502017-09-07 11:18:54 -0700247TEST(DownloadActionTest, MultiPayloadProgressTest) {
248 std::vector<brillo::Blob> payload_datas;
249 // the first payload must be the largest, as it's the actual payload used by
250 // the MockHttpFetcher for all downloaded data.
251 payload_datas.emplace_back(4 * kMockHttpFetcherChunkSize + 256);
252 payload_datas.emplace_back(2 * kMockHttpFetcherChunkSize);
253 brillo::FakeMessageLoop loop(nullptr);
254 loop.SetAsCurrent();
255 FakeSystemState fake_system_state;
256 EXPECT_CALL(*fake_system_state.mock_payload_state(), NextPayload())
257 .WillOnce(Return(true));
258
259 MockFileWriter mock_file_writer;
260 EXPECT_CALL(mock_file_writer, Close()).WillRepeatedly(Return(0));
261 EXPECT_CALL(mock_file_writer, Write(_, _, _))
262 .WillRepeatedly(
263 DoAll(SetArgPointee<2>(ErrorCode::kSuccess), Return(true)));
264
265 InstallPlan install_plan;
266 uint64_t total_expected_download_size{0};
267 for (const auto& data : payload_datas) {
268 uint64_t size = data.size();
269 install_plan.payloads.push_back(
270 {.size = size, .type = InstallPayloadType::kFull});
271 total_expected_download_size += size;
272 }
273 ObjectFeederAction<InstallPlan> feeder_action;
274 feeder_action.set_obj(install_plan);
275 MockPrefs prefs;
276 MockHttpFetcher* http_fetcher = new MockHttpFetcher(
277 payload_datas[0].data(), payload_datas[0].size(), nullptr);
278 // takes ownership of passed in HttpFetcher
279 DownloadAction download_action(&prefs,
280 fake_system_state.boot_control(),
281 fake_system_state.hardware(),
282 &fake_system_state,
283 http_fetcher);
284 download_action.SetTestFileWriter(&mock_file_writer);
285 BondActions(&feeder_action, &download_action);
286 MockDownloadActionDelegate download_delegate;
287 {
288 InSequence s;
289 download_action.set_delegate(&download_delegate);
290 // these are hand-computed based on the payloads specified above
291 EXPECT_CALL(download_delegate,
292 BytesReceived(kMockHttpFetcherChunkSize,
293 kMockHttpFetcherChunkSize,
294 total_expected_download_size));
295 EXPECT_CALL(download_delegate,
296 BytesReceived(kMockHttpFetcherChunkSize,
297 kMockHttpFetcherChunkSize * 2,
298 total_expected_download_size));
299 EXPECT_CALL(download_delegate,
300 BytesReceived(kMockHttpFetcherChunkSize,
301 kMockHttpFetcherChunkSize * 3,
302 total_expected_download_size));
303 EXPECT_CALL(download_delegate,
304 BytesReceived(kMockHttpFetcherChunkSize,
305 kMockHttpFetcherChunkSize * 4,
306 total_expected_download_size));
307 EXPECT_CALL(download_delegate,
308 BytesReceived(256,
309 kMockHttpFetcherChunkSize * 4 + 256,
310 total_expected_download_size));
311 EXPECT_CALL(download_delegate,
312 BytesReceived(kMockHttpFetcherChunkSize,
313 kMockHttpFetcherChunkSize * 5 + 256,
314 total_expected_download_size));
315 EXPECT_CALL(download_delegate,
316 BytesReceived(kMockHttpFetcherChunkSize,
317 total_expected_download_size,
318 total_expected_download_size));
319 }
320 ActionProcessor processor;
321 processor.EnqueueAction(&feeder_action);
322 processor.EnqueueAction(&download_action);
323
324 loop.PostTask(
325 FROM_HERE,
326 base::Bind(
327 [](ActionProcessor* processor) { processor->StartProcessing(); },
328 base::Unretained(&processor)));
329 loop.Run();
330 EXPECT_FALSE(loop.PendingTasks());
331}
332
rspangler@google.com49fdf182009-10-10 00:57:34 +0000333namespace {
334class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
335 public:
336 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700337 brillo::MessageLoop::current()->BreakLoop();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000338 }
rspangler@google.com49fdf182009-10-10 00:57:34 +0000339};
340
Alex Deymo60ca1a72015-06-18 18:19:15 -0700341void TerminateEarlyTestStarter(ActionProcessor* processor) {
rspangler@google.com49fdf182009-10-10 00:57:34 +0000342 processor->StartProcessing();
343 CHECK(processor->IsRunning());
344 processor->StopProcessing();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000345}
346
Darin Petkov9d911fa2010-08-19 09:36:08 -0700347void TestTerminateEarly(bool use_download_delegate) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700348 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700349 loop.SetAsCurrent();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000350
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700351 brillo::Blob data(kMockHttpFetcherChunkSize +
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800352 kMockHttpFetcherChunkSize / 2);
353 memset(data.data(), 0, data.size());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000354
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700355 ScopedTempFile temp_file;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000356 {
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700357 DirectFileWriter writer;
Alex Deymobffa0602016-02-12 17:16:29 -0800358 EXPECT_EQ(0, writer.Open(temp_file.path().c_str(), O_WRONLY | O_CREAT, 0));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700359
rspangler@google.com49fdf182009-10-10 00:57:34 +0000360 // takes ownership of passed in HttpFetcher
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000361 ObjectFeederAction<InstallPlan> feeder_action;
Alex Deymo64d98782016-02-05 18:03:48 -0800362 InstallPlan install_plan;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800363 install_plan.payloads.resize(1);
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000364 feeder_action.set_obj(install_plan);
Alex Deymo5ed695e2015-10-05 16:59:23 -0700365 FakeSystemState fake_system_state_;
Alex Deymo8427b4a2014-11-05 14:00:32 -0800366 MockPrefs prefs;
Alex Deymo1b3556c2016-02-03 09:54:02 -0800367 DownloadAction download_action(
368 &prefs,
369 fake_system_state_.boot_control(),
370 fake_system_state_.hardware(),
371 &fake_system_state_,
372 new MockHttpFetcher(data.data(), data.size(), nullptr));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700373 download_action.SetTestFileWriter(&writer);
Alex Deymo542c19b2015-12-03 07:43:31 -0300374 MockDownloadActionDelegate download_delegate;
Darin Petkov9d911fa2010-08-19 09:36:08 -0700375 if (use_download_delegate) {
Darin Petkov9d911fa2010-08-19 09:36:08 -0700376 download_action.set_delegate(&download_delegate);
Alex Deymo542c19b2015-12-03 07:43:31 -0300377 EXPECT_CALL(download_delegate, BytesReceived(_, _, _)).Times(0);
Darin Petkov9d911fa2010-08-19 09:36:08 -0700378 }
rspangler@google.com49fdf182009-10-10 00:57:34 +0000379 TerminateEarlyTestProcessorDelegate delegate;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000380 ActionProcessor processor;
381 processor.set_delegate(&delegate);
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000382 processor.EnqueueAction(&feeder_action);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000383 processor.EnqueueAction(&download_action);
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000384 BondActions(&feeder_action, &download_action);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000385
Alex Deymo60ca1a72015-06-18 18:19:15 -0700386 loop.PostTask(FROM_HERE,
387 base::Bind(&TerminateEarlyTestStarter, &processor));
388 loop.Run();
389 EXPECT_FALSE(loop.PendingTasks());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000390 }
391
392 // 1 or 0 chunks should have come through
Alex Deymobffa0602016-02-12 17:16:29 -0800393 const off_t resulting_file_size(utils::FileSize(temp_file.path()));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700394 EXPECT_GE(resulting_file_size, 0);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000395 if (resulting_file_size != 0)
Alex Deymo5fe0c4e2016-02-16 18:46:24 -0800396 EXPECT_EQ(kMockHttpFetcherChunkSize,
397 static_cast<size_t>(resulting_file_size));
rspangler@google.com49fdf182009-10-10 00:57:34 +0000398}
399
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700400} // namespace
Darin Petkov9d911fa2010-08-19 09:36:08 -0700401
402TEST(DownloadActionTest, TerminateEarlyTest) {
403 TestTerminateEarly(true);
404}
405
406TEST(DownloadActionTest, TerminateEarlyNoDownloadDelegateTest) {
407 TestTerminateEarly(false);
408}
409
rspangler@google.com49fdf182009-10-10 00:57:34 +0000410class DownloadActionTestAction;
411
412template<>
413class ActionTraits<DownloadActionTestAction> {
414 public:
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000415 typedef InstallPlan OutputObjectType;
416 typedef InstallPlan InputObjectType;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000417};
418
419// This is a simple Action class for testing.
Yunlian Jiang2dac5762013-04-12 09:53:09 -0700420class DownloadActionTestAction : public Action<DownloadActionTestAction> {
421 public:
rspangler@google.com49fdf182009-10-10 00:57:34 +0000422 DownloadActionTestAction() : did_run_(false) {}
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000423 typedef InstallPlan InputObjectType;
424 typedef InstallPlan OutputObjectType;
425 ActionPipe<InstallPlan>* in_pipe() { return in_pipe_.get(); }
426 ActionPipe<InstallPlan>* out_pipe() { return out_pipe_.get(); }
rspangler@google.com49fdf182009-10-10 00:57:34 +0000427 ActionProcessor* processor() { return processor_; }
428 void PerformAction() {
429 did_run_ = true;
430 ASSERT_TRUE(HasInputObject());
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000431 EXPECT_TRUE(expected_input_object_ == GetInputObject());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000432 ASSERT_TRUE(processor());
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700433 processor()->ActionComplete(this, ErrorCode::kSuccess);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000434 }
435 string Type() const { return "DownloadActionTestAction"; }
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000436 InstallPlan expected_input_object_;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000437 bool did_run_;
438};
439
440namespace {
441// This class is an ActionProcessorDelegate that simply terminates the
442// run loop when the ActionProcessor has completed processing. It's used
443// only by the test PassObjectOutTest.
444class PassObjectOutTestProcessorDelegate : public ActionProcessorDelegate {
445 public:
David Zeuthena99981f2013-04-29 13:42:47 -0700446 void ProcessingDone(const ActionProcessor* processor, ErrorCode code) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700447 brillo::MessageLoop::current()->BreakLoop();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000448 }
rspangler@google.com49fdf182009-10-10 00:57:34 +0000449};
450
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700451} // namespace
rspangler@google.com49fdf182009-10-10 00:57:34 +0000452
453TEST(DownloadActionTest, PassObjectOutTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700454 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700455 loop.SetAsCurrent();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000456
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700457 DirectFileWriter writer;
Alex Deymoe5e5fe92015-10-05 09:28:19 -0700458 EXPECT_EQ(0, writer.Open("/dev/null", O_WRONLY | O_CREAT, 0));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700459
rspangler@google.com49fdf182009-10-10 00:57:34 +0000460 // takes ownership of passed in HttpFetcher
Alex Deymo64d98782016-02-05 18:03:48 -0800461 InstallPlan install_plan;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800462 install_plan.payloads.push_back({.size = 1});
463 EXPECT_TRUE(
464 HashCalculator::RawHashOfData({'x'}, &install_plan.payloads[0].hash));
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000465 ObjectFeederAction<InstallPlan> feeder_action;
466 feeder_action.set_obj(install_plan);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800467 MockPrefs prefs;
Alex Deymo5ed695e2015-10-05 16:59:23 -0700468 FakeSystemState fake_system_state_;
Alex Deymo1b3556c2016-02-03 09:54:02 -0800469 DownloadAction download_action(&prefs,
470 fake_system_state_.boot_control(),
471 fake_system_state_.hardware(),
472 &fake_system_state_,
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700473 new MockHttpFetcher("x", 1, nullptr));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700474 download_action.SetTestFileWriter(&writer);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000475
476 DownloadActionTestAction test_action;
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000477 test_action.expected_input_object_ = install_plan;
478 BondActions(&feeder_action, &download_action);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000479 BondActions(&download_action, &test_action);
480
481 ActionProcessor processor;
482 PassObjectOutTestProcessorDelegate delegate;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000483 processor.set_delegate(&delegate);
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000484 processor.EnqueueAction(&feeder_action);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000485 processor.EnqueueAction(&download_action);
486 processor.EnqueueAction(&test_action);
487
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700488 loop.PostTask(
489 FROM_HERE,
490 base::Bind(
491 [](ActionProcessor* processor) { processor->StartProcessing(); },
492 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700493 loop.Run();
494 EXPECT_FALSE(loop.PendingTasks());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000495
496 EXPECT_EQ(true, test_action.did_run_);
497}
498
David Zeuthen8f191b22013-08-06 12:27:50 -0700499// Test fixture for P2P tests.
500class P2PDownloadActionTest : public testing::Test {
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700501 protected:
David Zeuthen8f191b22013-08-06 12:27:50 -0700502 P2PDownloadActionTest()
Alex Deymo60ca1a72015-06-18 18:19:15 -0700503 : start_at_offset_(0),
Gilad Arnold4a0321b2014-10-28 15:57:30 -0700504 fake_um_(fake_system_state_.fake_clock()) {}
David Zeuthen8f191b22013-08-06 12:27:50 -0700505
Alex Deymo610277e2014-11-11 21:18:11 -0800506 ~P2PDownloadActionTest() override {}
David Zeuthen8f191b22013-08-06 12:27:50 -0700507
508 // Derived from testing::Test.
Alex Deymo610277e2014-11-11 21:18:11 -0800509 void SetUp() override {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700510 loop_.SetAsCurrent();
David Zeuthen8f191b22013-08-06 12:27:50 -0700511 }
512
513 // Derived from testing::Test.
Alex Deymo610277e2014-11-11 21:18:11 -0800514 void TearDown() override {
Alex Deymo60ca1a72015-06-18 18:19:15 -0700515 EXPECT_FALSE(loop_.PendingTasks());
David Zeuthen8f191b22013-08-06 12:27:50 -0700516 }
517
518 // To be called by tests to setup the download. The
519 // |starting_offset| parameter is for where to resume.
520 void SetupDownload(off_t starting_offset) {
521 start_at_offset_ = starting_offset;
522 // Prepare data 10 kB of data.
523 data_.clear();
524 for (unsigned int i = 0; i < 10 * 1000; i++)
525 data_ += 'a' + (i % 25);
526
527 // Setup p2p.
528 FakeP2PManagerConfiguration *test_conf = new FakeP2PManagerConfiguration();
David Zeuthen41f2cf52014-11-05 12:29:45 -0500529 p2p_manager_.reset(P2PManager::Construct(
Gilad Arnold4a0321b2014-10-28 15:57:30 -0700530 test_conf, nullptr, &fake_um_, "cros_au", 3,
David Zeuthen41f2cf52014-11-05 12:29:45 -0500531 base::TimeDelta::FromDays(5)));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700532 fake_system_state_.set_p2p_manager(p2p_manager_.get());
David Zeuthen8f191b22013-08-06 12:27:50 -0700533 }
534
535 // To be called by tests to perform the download. The
536 // |use_p2p_to_share| parameter is used to indicate whether the
537 // payload should be shared via p2p.
538 void StartDownload(bool use_p2p_to_share) {
Gilad Arnold74b5f552014-10-07 08:17:16 -0700539 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
540 GetUsingP2PForSharing())
541 .WillRepeatedly(Return(use_p2p_to_share));
David Zeuthen8f191b22013-08-06 12:27:50 -0700542
543 ScopedTempFile output_temp_file;
544 TestDirectFileWriter writer;
Alex Deymobffa0602016-02-12 17:16:29 -0800545 EXPECT_EQ(
546 0, writer.Open(output_temp_file.path().c_str(), O_WRONLY | O_CREAT, 0));
Alex Deymo64d98782016-02-05 18:03:48 -0800547 InstallPlan install_plan;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800548 install_plan.payloads.push_back(
549 {.size = data_.length(),
550 .hash = {'1', '2', '3', '4', 'h', 'a', 's', 'h'}});
David Zeuthen8f191b22013-08-06 12:27:50 -0700551 ObjectFeederAction<InstallPlan> feeder_action;
552 feeder_action.set_obj(install_plan);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800553 MockPrefs prefs;
David Zeuthen8f191b22013-08-06 12:27:50 -0700554 http_fetcher_ = new MockHttpFetcher(data_.c_str(),
555 data_.length(),
Alex Vakulenko88b591f2014-08-28 16:48:57 -0700556 nullptr);
David Zeuthen8f191b22013-08-06 12:27:50 -0700557 // Note that DownloadAction takes ownership of the passed in HttpFetcher.
Alex Deymo1b3556c2016-02-03 09:54:02 -0800558 download_action_.reset(new DownloadAction(&prefs,
559 fake_system_state_.boot_control(),
560 fake_system_state_.hardware(),
561 &fake_system_state_,
David Zeuthen8f191b22013-08-06 12:27:50 -0700562 http_fetcher_));
563 download_action_->SetTestFileWriter(&writer);
564 BondActions(&feeder_action, download_action_.get());
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700565 DownloadActionTestProcessorDelegate delegate(ErrorCode::kSuccess);
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700566 delegate.expected_data_ = brillo::Blob(data_.begin() + start_at_offset_,
567 data_.end());
Alex Deymobffa0602016-02-12 17:16:29 -0800568 delegate.path_ = output_temp_file.path();
David Zeuthen8f191b22013-08-06 12:27:50 -0700569 processor_.set_delegate(&delegate);
570 processor_.EnqueueAction(&feeder_action);
571 processor_.EnqueueAction(download_action_.get());
572
Alex Deymo60ca1a72015-06-18 18:19:15 -0700573 loop_.PostTask(FROM_HERE, base::Bind(
574 &P2PDownloadActionTest::StartProcessorInRunLoopForP2P,
575 base::Unretained(this)));
576 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700577 }
578
Alex Deymo60ca1a72015-06-18 18:19:15 -0700579 // Mainloop used to make StartDownload() synchronous.
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700580 brillo::FakeMessageLoop loop_{nullptr};
Alex Deymo60ca1a72015-06-18 18:19:15 -0700581
David Zeuthen8f191b22013-08-06 12:27:50 -0700582 // The DownloadAction instance under test.
Ben Chan02f7c1d2014-10-18 15:18:02 -0700583 unique_ptr<DownloadAction> download_action_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700584
585 // The HttpFetcher used in the test.
586 MockHttpFetcher* http_fetcher_;
587
588 // The P2PManager used in the test.
Ben Chan02f7c1d2014-10-18 15:18:02 -0700589 unique_ptr<P2PManager> p2p_manager_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700590
591 // The ActionProcessor used for running the actions.
592 ActionProcessor processor_;
593
594 // A fake system state.
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700595 FakeSystemState fake_system_state_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700596
597 // The data being downloaded.
598 string data_;
599
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700600 private:
David Zeuthen8f191b22013-08-06 12:27:50 -0700601 // Callback used in StartDownload() method.
Alex Deymo60ca1a72015-06-18 18:19:15 -0700602 void StartProcessorInRunLoopForP2P() {
603 processor_.StartProcessing();
604 http_fetcher_->SetOffset(start_at_offset_);
David Zeuthen8f191b22013-08-06 12:27:50 -0700605 }
606
David Zeuthen8f191b22013-08-06 12:27:50 -0700607 // The requested starting offset passed to SetupDownload().
608 off_t start_at_offset_;
Gilad Arnold4a0321b2014-10-28 15:57:30 -0700609
610 chromeos_update_manager::FakeUpdateManager fake_um_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700611};
612
613TEST_F(P2PDownloadActionTest, IsWrittenTo) {
Alex Deymo10875d92014-11-10 21:52:57 -0800614 if (!test_utils::IsXAttrSupported(FilePath("/tmp"))) {
David Zeuthen910ec5b2013-09-26 12:10:58 -0700615 LOG(WARNING) << "Skipping test because /tmp does not support xattr. "
616 << "Please update your system to support this feature.";
617 return;
618 }
619
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700620 SetupDownload(0); // starting_offset
621 StartDownload(true); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700622
623 // Check the p2p file and its content matches what was sent.
624 string file_id = download_action_->p2p_file_id();
Gilad Arnold74b5f552014-10-07 08:17:16 -0700625 EXPECT_NE("", file_id);
Alex Deymo80f70ff2016-02-10 16:08:11 -0800626 EXPECT_EQ(static_cast<int>(data_.length()),
627 p2p_manager_->FileGetSize(file_id));
628 EXPECT_EQ(static_cast<int>(data_.length()),
629 p2p_manager_->FileGetExpectedSize(file_id));
David Zeuthen8f191b22013-08-06 12:27:50 -0700630 string p2p_file_contents;
631 EXPECT_TRUE(ReadFileToString(p2p_manager_->FileGetPath(file_id),
632 &p2p_file_contents));
633 EXPECT_EQ(data_, p2p_file_contents);
634}
635
636TEST_F(P2PDownloadActionTest, DeleteIfHoleExists) {
Alex Deymo10875d92014-11-10 21:52:57 -0800637 if (!test_utils::IsXAttrSupported(FilePath("/tmp"))) {
David Zeuthen910ec5b2013-09-26 12:10:58 -0700638 LOG(WARNING) << "Skipping test because /tmp does not support xattr. "
639 << "Please update your system to support this feature.";
640 return;
641 }
642
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700643 SetupDownload(1000); // starting_offset
644 StartDownload(true); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700645
646 // DownloadAction should convey that the file is not being shared.
647 // and that we don't have any p2p files.
648 EXPECT_EQ(download_action_->p2p_file_id(), "");
649 EXPECT_EQ(p2p_manager_->CountSharedFiles(), 0);
650}
651
652TEST_F(P2PDownloadActionTest, CanAppend) {
Alex Deymo10875d92014-11-10 21:52:57 -0800653 if (!test_utils::IsXAttrSupported(FilePath("/tmp"))) {
David Zeuthen910ec5b2013-09-26 12:10:58 -0700654 LOG(WARNING) << "Skipping test because /tmp does not support xattr. "
655 << "Please update your system to support this feature.";
656 return;
657 }
658
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700659 SetupDownload(1000); // starting_offset
David Zeuthen8f191b22013-08-06 12:27:50 -0700660
661 // Prepare the file with existing data before starting to write to
662 // it via DownloadAction.
Sen Jiang2703ef42017-03-16 13:36:21 -0700663 string file_id = utils::CalculateP2PFileId(
664 {'1', '2', '3', '4', 'h', 'a', 's', 'h'}, data_.length());
David Zeuthen8f191b22013-08-06 12:27:50 -0700665 ASSERT_TRUE(p2p_manager_->FileShare(file_id, data_.length()));
666 string existing_data;
667 for (unsigned int i = 0; i < 1000; i++)
668 existing_data += '0' + (i % 10);
669 ASSERT_EQ(WriteFile(p2p_manager_->FileGetPath(file_id), existing_data.c_str(),
670 1000), 1000);
671
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700672 StartDownload(true); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700673
674 // DownloadAction should convey the same file_id and the file should
675 // have the expected size.
676 EXPECT_EQ(download_action_->p2p_file_id(), file_id);
Alex Deymo80f70ff2016-02-10 16:08:11 -0800677 EXPECT_EQ(static_cast<ssize_t>(data_.length()),
678 p2p_manager_->FileGetSize(file_id));
679 EXPECT_EQ(static_cast<ssize_t>(data_.length()),
680 p2p_manager_->FileGetExpectedSize(file_id));
David Zeuthen8f191b22013-08-06 12:27:50 -0700681 string p2p_file_contents;
682 // Check that the first 1000 bytes wasn't touched and that we
683 // appended the remaining as appropriate.
684 EXPECT_TRUE(ReadFileToString(p2p_manager_->FileGetPath(file_id),
685 &p2p_file_contents));
686 EXPECT_EQ(existing_data, p2p_file_contents.substr(0, 1000));
687 EXPECT_EQ(data_.substr(1000), p2p_file_contents.substr(1000));
688}
689
David Zeuthen8f191b22013-08-06 12:27:50 -0700690TEST_F(P2PDownloadActionTest, DeletePartialP2PFileIfResumingWithoutP2P) {
Alex Deymo10875d92014-11-10 21:52:57 -0800691 if (!test_utils::IsXAttrSupported(FilePath("/tmp"))) {
David Zeuthen910ec5b2013-09-26 12:10:58 -0700692 LOG(WARNING) << "Skipping test because /tmp does not support xattr. "
693 << "Please update your system to support this feature.";
694 return;
695 }
696
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700697 SetupDownload(1000); // starting_offset
David Zeuthen8f191b22013-08-06 12:27:50 -0700698
699 // Prepare the file with all existing data before starting to write
700 // to it via DownloadAction.
Sen Jiang2703ef42017-03-16 13:36:21 -0700701 string file_id = utils::CalculateP2PFileId(
702 {'1', '2', '3', '4', 'h', 'a', 's', 'h'}, data_.length());
David Zeuthen8f191b22013-08-06 12:27:50 -0700703 ASSERT_TRUE(p2p_manager_->FileShare(file_id, data_.length()));
704 string existing_data;
705 for (unsigned int i = 0; i < 1000; i++)
706 existing_data += '0' + (i % 10);
707 ASSERT_EQ(WriteFile(p2p_manager_->FileGetPath(file_id), existing_data.c_str(),
708 1000), 1000);
709
710 // Check that the file is there.
Alex Deymo80f70ff2016-02-10 16:08:11 -0800711 EXPECT_EQ(1000, p2p_manager_->FileGetSize(file_id));
712 EXPECT_EQ(1, p2p_manager_->CountSharedFiles());
David Zeuthen8f191b22013-08-06 12:27:50 -0700713
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700714 StartDownload(false); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700715
716 // DownloadAction should have deleted the p2p file. Check that it's gone.
Alex Deymo80f70ff2016-02-10 16:08:11 -0800717 EXPECT_EQ(-1, p2p_manager_->FileGetSize(file_id));
718 EXPECT_EQ(0, p2p_manager_->CountSharedFiles());
David Zeuthen8f191b22013-08-06 12:27:50 -0700719}
720
rspangler@google.com49fdf182009-10-10 00:57:34 +0000721} // namespace chromeos_update_engine