blob: e6ca219b0c7d4c4f923b82785ea2c204483d260e [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>
Sen Jiangba2213a2018-02-27 16:28:20 -080025#include <vector>
David Zeuthen8f191b22013-08-06 12:27:50 -070026
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/message_loops/fake_message_loop.h>
33#include <brillo/message_loops/message_loop.h>
David Zeuthen8f191b22013-08-06 12:27:50 -070034
Alex Deymo39910dc2015-11-09 17:04:30 -080035#include "update_engine/common/action_pipe.h"
36#include "update_engine/common/hash_calculator.h"
37#include "update_engine/common/mock_http_fetcher.h"
38#include "update_engine/common/mock_prefs.h"
39#include "update_engine/common/test_utils.h"
40#include "update_engine/common/utils.h"
David Zeuthen8f191b22013-08-06 12:27:50 -070041#include "update_engine/fake_p2p_manager_configuration.h"
Gilad Arnold5bb4c902014-04-10 12:32:13 -070042#include "update_engine/fake_system_state.h"
Aaron Wood9321f502017-09-07 11:18:54 -070043#include "update_engine/mock_file_writer.h"
Alex Deymo542c19b2015-12-03 07:43:31 -030044#include "update_engine/payload_consumer/mock_download_action.h"
Gilad Arnold4a0321b2014-10-28 15:57:30 -070045#include "update_engine/update_manager/fake_update_manager.h"
rspangler@google.com49fdf182009-10-10 00:57:34 +000046
47namespace chromeos_update_engine {
48
Alex Deymof329b932014-10-30 01:37:48 -070049using base::FilePath;
50using base::ReadFileToString;
51using base::WriteFile;
rspangler@google.com49fdf182009-10-10 00:57:34 +000052using std::string;
Ben Chan02f7c1d2014-10-18 15:18:02 -070053using std::unique_ptr;
Alex Deymoe5e5fe92015-10-05 09:28:19 -070054using test_utils::ScopedTempFile;
Amin Hassani008c4582019-01-13 16:22:47 -080055using testing::_;
Darin Petkov9d911fa2010-08-19 09:36:08 -070056using testing::AtLeast;
57using testing::InSequence;
Gilad Arnold74b5f552014-10-07 08:17:16 -070058using testing::Return;
Aaron Wood9321f502017-09-07 11:18:54 -070059using testing::SetArgPointee;
rspangler@google.com49fdf182009-10-10 00:57:34 +000060
Amin Hassani008c4582019-01-13 16:22:47 -080061class DownloadActionTest : public ::testing::Test {};
rspangler@google.com49fdf182009-10-10 00:57:34 +000062
63namespace {
Darin Petkov9d911fa2010-08-19 09:36:08 -070064
rspangler@google.com49fdf182009-10-10 00:57:34 +000065class DownloadActionTestProcessorDelegate : public ActionProcessorDelegate {
66 public:
Amin Hassanid3f4bea2018-04-30 14:52:40 -070067 DownloadActionTestProcessorDelegate()
68 : processing_done_called_(false), expected_code_(ErrorCode::kSuccess) {}
Alex Deymo610277e2014-11-11 21:18:11 -080069 ~DownloadActionTestProcessorDelegate() override {
rspangler@google.com49fdf182009-10-10 00:57:34 +000070 EXPECT_TRUE(processing_done_called_);
71 }
Yunlian Jiang35866ed2015-01-29 13:09:20 -080072 void ProcessingDone(const ActionProcessor* processor,
73 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070074 brillo::MessageLoop::current()->BreakLoop();
75 brillo::Blob found_data;
adlr@google.comc98a7ed2009-12-04 18:54:03 +000076 ASSERT_TRUE(utils::ReadFile(path_, &found_data));
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -070077 if (expected_code_ != ErrorCode::kDownloadWriteError) {
Darin Petkov9ce452b2010-11-17 14:33:28 -080078 ASSERT_EQ(expected_data_.size(), found_data.size());
79 for (unsigned i = 0; i < expected_data_.size(); i++) {
80 EXPECT_EQ(expected_data_[i], found_data[i]);
81 }
rspangler@google.com49fdf182009-10-10 00:57:34 +000082 }
83 processing_done_called_ = true;
84 }
85
Yunlian Jiang35866ed2015-01-29 13:09:20 -080086 void ActionCompleted(ActionProcessor* processor,
87 AbstractAction* action,
88 ErrorCode code) override {
Darin Petkovc97435c2010-07-20 12:37:43 -070089 const string type = action->Type();
90 if (type == DownloadAction::StaticType()) {
91 EXPECT_EQ(expected_code_, code);
Amin Hassanid3f4bea2018-04-30 14:52:40 -070092 p2p_file_id_ = static_cast<DownloadAction*>(action)->p2p_file_id();
Darin Petkovc97435c2010-07-20 12:37:43 -070093 } else {
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -070094 EXPECT_EQ(ErrorCode::kSuccess, code);
Darin Petkovc97435c2010-07-20 12:37:43 -070095 }
rspangler@google.com49fdf182009-10-10 00:57:34 +000096 }
97
rspangler@google.com49fdf182009-10-10 00:57:34 +000098 string path_;
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -070099 brillo::Blob expected_data_;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000100 bool processing_done_called_;
David Zeuthena99981f2013-04-29 13:42:47 -0700101 ErrorCode expected_code_;
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700102 string p2p_file_id_;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000103};
104
Darin Petkov9ce452b2010-11-17 14:33:28 -0800105class TestDirectFileWriter : public DirectFileWriter {
106 public:
107 TestDirectFileWriter() : fail_write_(0), current_write_(0) {}
108 void set_fail_write(int fail_write) { fail_write_ = fail_write; }
109
Don Garrette410e0f2011-11-10 15:39:01 -0800110 virtual bool Write(const void* bytes, size_t count) {
Darin Petkov9ce452b2010-11-17 14:33:28 -0800111 if (++current_write_ == fail_write_) {
Don Garrette410e0f2011-11-10 15:39:01 -0800112 return false;
Darin Petkov9ce452b2010-11-17 14:33:28 -0800113 }
114 return DirectFileWriter::Write(bytes, count);
115 }
116
117 private:
118 // If positive, fail on the |fail_write_| call to Write.
119 int fail_write_;
120 int current_write_;
121};
122
Alex Deymo60ca1a72015-06-18 18:19:15 -0700123void StartProcessorInRunLoop(ActionProcessor* processor,
124 MockHttpFetcher* http_fetcher) {
rspangler@google.com49fdf182009-10-10 00:57:34 +0000125 processor->StartProcessing();
Andrew de los Reyes34e41a12010-10-26 20:07:58 -0700126 http_fetcher->SetOffset(1);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000127}
128
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700129void TestWithData(const brillo::Blob& data,
Darin Petkov9ce452b2010-11-17 14:33:28 -0800130 int fail_write,
Darin Petkov9d911fa2010-08-19 09:36:08 -0700131 bool use_download_delegate) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700132 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700133 loop.SetAsCurrent();
Alex Deymo5ed695e2015-10-05 16:59:23 -0700134 FakeSystemState fake_system_state;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000135
136 // TODO(adlr): see if we need a different file for build bots
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700137 ScopedTempFile output_temp_file;
Darin Petkov9ce452b2010-11-17 14:33:28 -0800138 TestDirectFileWriter writer;
Alex Deymobffa0602016-02-12 17:16:29 -0800139 EXPECT_EQ(
140 0, writer.Open(output_temp_file.path().c_str(), O_WRONLY | O_CREAT, 0));
Darin Petkov9ce452b2010-11-17 14:33:28 -0800141 writer.set_fail_write(fail_write);
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700142
Sen Jiang5ae865b2017-04-18 14:24:40 -0700143 uint64_t size = data.size() - 1;
Alex Deymo64d98782016-02-05 18:03:48 -0800144 InstallPlan install_plan;
Sen Jiangcdd52062017-05-18 15:33:10 -0700145 install_plan.payloads.push_back(
146 {.size = size, .type = InstallPayloadType::kDelta});
Sen Jiang2703ef42017-03-16 13:36:21 -0700147 // We pull off the first byte from data and seek past it.
148 EXPECT_TRUE(HashCalculator::RawHashOfBytes(
Sen Jiang0affc2c2017-02-10 15:55:05 -0800149 &data[1], data.size() - 1, &install_plan.payloads[0].hash));
Alex Deymo5ed695e2015-10-05 16:59:23 -0700150 install_plan.source_slot = 0;
151 install_plan.target_slot = 1;
152 // We mark both slots as bootable. Only the target slot should be unbootable
153 // after the download starts.
154 fake_system_state.fake_boot_control()->SetSlotBootable(
155 install_plan.source_slot, true);
156 fake_system_state.fake_boot_control()->SetSlotBootable(
157 install_plan.target_slot, true);
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700158 auto feeder_action = std::make_unique<ObjectFeederAction<InstallPlan>>();
159 feeder_action->set_obj(install_plan);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800160 MockPrefs prefs;
Amin Hassani008c4582019-01-13 16:22:47 -0800161 MockHttpFetcher* http_fetcher =
162 new MockHttpFetcher(data.data(), data.size(), nullptr);
Andrew de los Reyes34e41a12010-10-26 20:07:58 -0700163 // takes ownership of passed in HttpFetcher
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700164 auto download_action =
165 std::make_unique<DownloadAction>(&prefs,
166 fake_system_state.boot_control(),
167 fake_system_state.hardware(),
168 &fake_system_state,
169 http_fetcher,
170 false /* interactive */);
171 download_action->SetTestFileWriter(&writer);
172 BondActions(feeder_action.get(), download_action.get());
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;
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700176 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));
Sen Jiang7ebfccf2018-03-15 13:55:55 -0700181 EXPECT_CALL(download_delegate, DownloadComplete())
182 .Times(fail_write == 0 ? 1 : 0);
Darin Petkov9d911fa2010-08-19 09:36:08 -0700183 }
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700184 DownloadActionTestProcessorDelegate delegate;
185 delegate.expected_code_ =
186 (fail_write > 0) ? ErrorCode::kDownloadWriteError : ErrorCode::kSuccess;
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700187 delegate.expected_data_ = brillo::Blob(data.begin() + 1, data.end());
Alex Deymobffa0602016-02-12 17:16:29 -0800188 delegate.path_ = output_temp_file.path();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000189 ActionProcessor processor;
190 processor.set_delegate(&delegate);
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700191 processor.EnqueueAction(std::move(feeder_action));
192 processor.EnqueueAction(std::move(download_action));
rspangler@google.com49fdf182009-10-10 00:57:34 +0000193
Alex Deymo60ca1a72015-06-18 18:19:15 -0700194 loop.PostTask(FROM_HERE,
195 base::Bind(&StartProcessorInRunLoop, &processor, http_fetcher));
196 loop.Run();
197 EXPECT_FALSE(loop.PendingTasks());
Alex Deymo5ed695e2015-10-05 16:59:23 -0700198
199 EXPECT_TRUE(fake_system_state.fake_boot_control()->IsSlotBootable(
200 install_plan.source_slot));
201 EXPECT_FALSE(fake_system_state.fake_boot_control()->IsSlotBootable(
202 install_plan.target_slot));
rspangler@google.com49fdf182009-10-10 00:57:34 +0000203}
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700204} // namespace
rspangler@google.com49fdf182009-10-10 00:57:34 +0000205
206TEST(DownloadActionTest, SimpleTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700207 brillo::Blob small;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000208 const char* foo = "foo";
209 small.insert(small.end(), foo, foo + strlen(foo));
Darin Petkov50332f12010-09-24 11:44:47 -0700210 TestWithData(small,
Amin Hassani008c4582019-01-13 16:22:47 -0800211 0, // fail_write
Darin Petkov50332f12010-09-24 11:44:47 -0700212 true); // use_download_delegate
rspangler@google.com49fdf182009-10-10 00:57:34 +0000213}
214
215TEST(DownloadActionTest, LargeTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700216 brillo::Blob big(5 * kMockHttpFetcherChunkSize);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000217 char c = '0';
218 for (unsigned int i = 0; i < big.size(); i++) {
219 big[i] = c;
Darin Petkov9ce452b2010-11-17 14:33:28 -0800220 c = ('9' == c) ? '0' : c + 1;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000221 }
Darin Petkov50332f12010-09-24 11:44:47 -0700222 TestWithData(big,
Amin Hassani008c4582019-01-13 16:22:47 -0800223 0, // fail_write
Darin Petkov9ce452b2010-11-17 14:33:28 -0800224 true); // use_download_delegate
225}
226
227TEST(DownloadActionTest, FailWriteTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700228 brillo::Blob big(5 * kMockHttpFetcherChunkSize);
Darin Petkov9ce452b2010-11-17 14:33:28 -0800229 char c = '0';
230 for (unsigned int i = 0; i < big.size(); i++) {
231 big[i] = c;
232 c = ('9' == c) ? '0' : c + 1;
233 }
234 TestWithData(big,
Amin Hassani008c4582019-01-13 16:22:47 -0800235 2, // fail_write
Darin Petkov50332f12010-09-24 11:44:47 -0700236 true); // use_download_delegate
Darin Petkovc97435c2010-07-20 12:37:43 -0700237}
238
Darin Petkov9d911fa2010-08-19 09:36:08 -0700239TEST(DownloadActionTest, NoDownloadDelegateTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700240 brillo::Blob small;
Darin Petkov9d911fa2010-08-19 09:36:08 -0700241 const char* foo = "foofoo";
242 small.insert(small.end(), foo, foo + strlen(foo));
Darin Petkov50332f12010-09-24 11:44:47 -0700243 TestWithData(small,
Amin Hassani008c4582019-01-13 16:22:47 -0800244 0, // fail_write
Darin Petkov50332f12010-09-24 11:44:47 -0700245 false); // use_download_delegate
rspangler@google.com49fdf182009-10-10 00:57:34 +0000246}
247
Aaron Wood9321f502017-09-07 11:18:54 -0700248TEST(DownloadActionTest, MultiPayloadProgressTest) {
249 std::vector<brillo::Blob> payload_datas;
250 // the first payload must be the largest, as it's the actual payload used by
251 // the MockHttpFetcher for all downloaded data.
252 payload_datas.emplace_back(4 * kMockHttpFetcherChunkSize + 256);
253 payload_datas.emplace_back(2 * kMockHttpFetcherChunkSize);
254 brillo::FakeMessageLoop loop(nullptr);
255 loop.SetAsCurrent();
256 FakeSystemState fake_system_state;
257 EXPECT_CALL(*fake_system_state.mock_payload_state(), NextPayload())
258 .WillOnce(Return(true));
259
260 MockFileWriter mock_file_writer;
261 EXPECT_CALL(mock_file_writer, Close()).WillRepeatedly(Return(0));
262 EXPECT_CALL(mock_file_writer, Write(_, _, _))
263 .WillRepeatedly(
264 DoAll(SetArgPointee<2>(ErrorCode::kSuccess), Return(true)));
265
266 InstallPlan install_plan;
267 uint64_t total_expected_download_size{0};
268 for (const auto& data : payload_datas) {
269 uint64_t size = data.size();
270 install_plan.payloads.push_back(
271 {.size = size, .type = InstallPayloadType::kFull});
272 total_expected_download_size += size;
273 }
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700274 auto feeder_action = std::make_unique<ObjectFeederAction<InstallPlan>>();
275 feeder_action->set_obj(install_plan);
Aaron Wood9321f502017-09-07 11:18:54 -0700276 MockPrefs prefs;
277 MockHttpFetcher* http_fetcher = new MockHttpFetcher(
278 payload_datas[0].data(), payload_datas[0].size(), nullptr);
279 // takes ownership of passed in HttpFetcher
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700280 auto download_action =
281 std::make_unique<DownloadAction>(&prefs,
282 fake_system_state.boot_control(),
283 fake_system_state.hardware(),
284 &fake_system_state,
285 http_fetcher,
286 false /* interactive */);
287 download_action->SetTestFileWriter(&mock_file_writer);
288 BondActions(feeder_action.get(), download_action.get());
Aaron Wood9321f502017-09-07 11:18:54 -0700289 MockDownloadActionDelegate download_delegate;
290 {
291 InSequence s;
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700292 download_action->set_delegate(&download_delegate);
Aaron Wood9321f502017-09-07 11:18:54 -0700293 // these are hand-computed based on the payloads specified above
294 EXPECT_CALL(download_delegate,
295 BytesReceived(kMockHttpFetcherChunkSize,
296 kMockHttpFetcherChunkSize,
297 total_expected_download_size));
298 EXPECT_CALL(download_delegate,
299 BytesReceived(kMockHttpFetcherChunkSize,
300 kMockHttpFetcherChunkSize * 2,
301 total_expected_download_size));
302 EXPECT_CALL(download_delegate,
303 BytesReceived(kMockHttpFetcherChunkSize,
304 kMockHttpFetcherChunkSize * 3,
305 total_expected_download_size));
306 EXPECT_CALL(download_delegate,
307 BytesReceived(kMockHttpFetcherChunkSize,
308 kMockHttpFetcherChunkSize * 4,
309 total_expected_download_size));
310 EXPECT_CALL(download_delegate,
311 BytesReceived(256,
312 kMockHttpFetcherChunkSize * 4 + 256,
313 total_expected_download_size));
314 EXPECT_CALL(download_delegate,
315 BytesReceived(kMockHttpFetcherChunkSize,
316 kMockHttpFetcherChunkSize * 5 + 256,
317 total_expected_download_size));
318 EXPECT_CALL(download_delegate,
319 BytesReceived(kMockHttpFetcherChunkSize,
320 total_expected_download_size,
321 total_expected_download_size));
322 }
323 ActionProcessor processor;
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700324 processor.EnqueueAction(std::move(feeder_action));
325 processor.EnqueueAction(std::move(download_action));
Aaron Wood9321f502017-09-07 11:18:54 -0700326
327 loop.PostTask(
328 FROM_HERE,
329 base::Bind(
330 [](ActionProcessor* processor) { processor->StartProcessing(); },
331 base::Unretained(&processor)));
332 loop.Run();
333 EXPECT_FALSE(loop.PendingTasks());
334}
335
rspangler@google.com49fdf182009-10-10 00:57:34 +0000336namespace {
337class TerminateEarlyTestProcessorDelegate : public ActionProcessorDelegate {
338 public:
339 void ProcessingStopped(const ActionProcessor* processor) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700340 brillo::MessageLoop::current()->BreakLoop();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000341 }
rspangler@google.com49fdf182009-10-10 00:57:34 +0000342};
343
Alex Deymo60ca1a72015-06-18 18:19:15 -0700344void TerminateEarlyTestStarter(ActionProcessor* processor) {
rspangler@google.com49fdf182009-10-10 00:57:34 +0000345 processor->StartProcessing();
346 CHECK(processor->IsRunning());
347 processor->StopProcessing();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000348}
349
Darin Petkov9d911fa2010-08-19 09:36:08 -0700350void TestTerminateEarly(bool use_download_delegate) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700351 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700352 loop.SetAsCurrent();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000353
Amin Hassani008c4582019-01-13 16:22:47 -0800354 brillo::Blob data(kMockHttpFetcherChunkSize + kMockHttpFetcherChunkSize / 2);
Alex Vakulenkof68bbbc2015-02-09 12:53:18 -0800355 memset(data.data(), 0, data.size());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000356
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700357 ScopedTempFile temp_file;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000358 {
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700359 DirectFileWriter writer;
Alex Deymobffa0602016-02-12 17:16:29 -0800360 EXPECT_EQ(0, writer.Open(temp_file.path().c_str(), O_WRONLY | O_CREAT, 0));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700361
rspangler@google.com49fdf182009-10-10 00:57:34 +0000362 // takes ownership of passed in HttpFetcher
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700363 auto feeder_action = std::make_unique<ObjectFeederAction<InstallPlan>>();
Alex Deymo64d98782016-02-05 18:03:48 -0800364 InstallPlan install_plan;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800365 install_plan.payloads.resize(1);
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700366 feeder_action->set_obj(install_plan);
Alex Deymo5ed695e2015-10-05 16:59:23 -0700367 FakeSystemState fake_system_state_;
Alex Deymo8427b4a2014-11-05 14:00:32 -0800368 MockPrefs prefs;
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700369 auto download_action = std::make_unique<DownloadAction>(
Alex Deymo1b3556c2016-02-03 09:54:02 -0800370 &prefs,
371 fake_system_state_.boot_control(),
372 fake_system_state_.hardware(),
373 &fake_system_state_,
Amin Hassani7ecda262017-07-11 17:10:50 -0700374 new MockHttpFetcher(data.data(), data.size(), nullptr),
Amin Hassanied37d682018-04-06 13:22:00 -0700375 false /* interactive */);
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700376 download_action->SetTestFileWriter(&writer);
Alex Deymo542c19b2015-12-03 07:43:31 -0300377 MockDownloadActionDelegate download_delegate;
Darin Petkov9d911fa2010-08-19 09:36:08 -0700378 if (use_download_delegate) {
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700379 download_action->set_delegate(&download_delegate);
Alex Deymo542c19b2015-12-03 07:43:31 -0300380 EXPECT_CALL(download_delegate, BytesReceived(_, _, _)).Times(0);
Darin Petkov9d911fa2010-08-19 09:36:08 -0700381 }
rspangler@google.com49fdf182009-10-10 00:57:34 +0000382 TerminateEarlyTestProcessorDelegate delegate;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000383 ActionProcessor processor;
384 processor.set_delegate(&delegate);
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700385 BondActions(feeder_action.get(), download_action.get());
386 processor.EnqueueAction(std::move(feeder_action));
387 processor.EnqueueAction(std::move(download_action));
rspangler@google.com49fdf182009-10-10 00:57:34 +0000388
Alex Deymo60ca1a72015-06-18 18:19:15 -0700389 loop.PostTask(FROM_HERE,
390 base::Bind(&TerminateEarlyTestStarter, &processor));
391 loop.Run();
392 EXPECT_FALSE(loop.PendingTasks());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000393 }
394
395 // 1 or 0 chunks should have come through
Alex Deymobffa0602016-02-12 17:16:29 -0800396 const off_t resulting_file_size(utils::FileSize(temp_file.path()));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700397 EXPECT_GE(resulting_file_size, 0);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000398 if (resulting_file_size != 0)
Alex Deymo5fe0c4e2016-02-16 18:46:24 -0800399 EXPECT_EQ(kMockHttpFetcherChunkSize,
400 static_cast<size_t>(resulting_file_size));
rspangler@google.com49fdf182009-10-10 00:57:34 +0000401}
402
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700403} // namespace
Darin Petkov9d911fa2010-08-19 09:36:08 -0700404
405TEST(DownloadActionTest, TerminateEarlyTest) {
406 TestTerminateEarly(true);
407}
408
409TEST(DownloadActionTest, TerminateEarlyNoDownloadDelegateTest) {
410 TestTerminateEarly(false);
411}
412
rspangler@google.com49fdf182009-10-10 00:57:34 +0000413class DownloadActionTestAction;
414
Amin Hassani008c4582019-01-13 16:22:47 -0800415template <>
rspangler@google.com49fdf182009-10-10 00:57:34 +0000416class ActionTraits<DownloadActionTestAction> {
417 public:
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000418 typedef InstallPlan OutputObjectType;
419 typedef InstallPlan InputObjectType;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000420};
421
422// This is a simple Action class for testing.
Yunlian Jiang2dac5762013-04-12 09:53:09 -0700423class DownloadActionTestAction : public Action<DownloadActionTestAction> {
424 public:
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700425 DownloadActionTestAction() = default;
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000426 typedef InstallPlan InputObjectType;
427 typedef InstallPlan OutputObjectType;
428 ActionPipe<InstallPlan>* in_pipe() { return in_pipe_.get(); }
429 ActionPipe<InstallPlan>* out_pipe() { return out_pipe_.get(); }
rspangler@google.com49fdf182009-10-10 00:57:34 +0000430 ActionProcessor* processor() { return processor_; }
431 void PerformAction() {
rspangler@google.com49fdf182009-10-10 00:57:34 +0000432 ASSERT_TRUE(HasInputObject());
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000433 EXPECT_TRUE(expected_input_object_ == GetInputObject());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000434 ASSERT_TRUE(processor());
Gilad Arnoldd1c4d2d2014-06-05 14:07:53 -0700435 processor()->ActionComplete(this, ErrorCode::kSuccess);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000436 }
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700437 static std::string StaticType() { return "DownloadActionTestAction"; }
438 string Type() const { return StaticType(); }
adlr@google.comc98a7ed2009-12-04 18:54:03 +0000439 InstallPlan expected_input_object_;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000440};
441
442namespace {
443// This class is an ActionProcessorDelegate that simply terminates the
444// run loop when the ActionProcessor has completed processing. It's used
445// only by the test PassObjectOutTest.
446class PassObjectOutTestProcessorDelegate : public ActionProcessorDelegate {
447 public:
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700448 void ProcessingDone(const ActionProcessor* processor,
449 ErrorCode code) override {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700450 brillo::MessageLoop::current()->BreakLoop();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000451 }
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700452 void ActionCompleted(ActionProcessor* processor,
453 AbstractAction* action,
454 ErrorCode code) override {
455 if (action->Type() == DownloadActionTestAction::StaticType()) {
456 did_test_action_run_ = true;
457 }
458 }
459
460 bool did_test_action_run_ = false;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000461};
462
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700463} // namespace
rspangler@google.com49fdf182009-10-10 00:57:34 +0000464
465TEST(DownloadActionTest, PassObjectOutTest) {
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700466 brillo::FakeMessageLoop loop(nullptr);
Alex Deymo60ca1a72015-06-18 18:19:15 -0700467 loop.SetAsCurrent();
rspangler@google.com49fdf182009-10-10 00:57:34 +0000468
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700469 DirectFileWriter writer;
Alex Deymoe5e5fe92015-10-05 09:28:19 -0700470 EXPECT_EQ(0, writer.Open("/dev/null", O_WRONLY | O_CREAT, 0));
Andrew de los Reyesf9185172010-05-03 11:07:05 -0700471
rspangler@google.com49fdf182009-10-10 00:57:34 +0000472 // takes ownership of passed in HttpFetcher
Alex Deymo64d98782016-02-05 18:03:48 -0800473 InstallPlan install_plan;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800474 install_plan.payloads.push_back({.size = 1});
475 EXPECT_TRUE(
476 HashCalculator::RawHashOfData({'x'}, &install_plan.payloads[0].hash));
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700477 auto feeder_action = std::make_unique<ObjectFeederAction<InstallPlan>>();
478 feeder_action->set_obj(install_plan);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800479 MockPrefs prefs;
Alex Deymo5ed695e2015-10-05 16:59:23 -0700480 FakeSystemState fake_system_state_;
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700481 auto download_action =
482 std::make_unique<DownloadAction>(&prefs,
483 fake_system_state_.boot_control(),
484 fake_system_state_.hardware(),
485 &fake_system_state_,
486 new MockHttpFetcher("x", 1, nullptr),
487 false /* interactive */);
488 download_action->SetTestFileWriter(&writer);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000489
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700490 auto test_action = std::make_unique<DownloadActionTestAction>();
491 test_action->expected_input_object_ = install_plan;
492 BondActions(feeder_action.get(), download_action.get());
493 BondActions(download_action.get(), test_action.get());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000494
495 ActionProcessor processor;
496 PassObjectOutTestProcessorDelegate delegate;
rspangler@google.com49fdf182009-10-10 00:57:34 +0000497 processor.set_delegate(&delegate);
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700498 processor.EnqueueAction(std::move(feeder_action));
499 processor.EnqueueAction(std::move(download_action));
500 processor.EnqueueAction(std::move(test_action));
rspangler@google.com49fdf182009-10-10 00:57:34 +0000501
Luis Hector Chavezf1cf3482016-07-19 14:29:19 -0700502 loop.PostTask(
503 FROM_HERE,
504 base::Bind(
505 [](ActionProcessor* processor) { processor->StartProcessing(); },
506 base::Unretained(&processor)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700507 loop.Run();
508 EXPECT_FALSE(loop.PendingTasks());
rspangler@google.com49fdf182009-10-10 00:57:34 +0000509
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700510 EXPECT_EQ(true, delegate.did_test_action_run_);
rspangler@google.com49fdf182009-10-10 00:57:34 +0000511}
512
David Zeuthen8f191b22013-08-06 12:27:50 -0700513// Test fixture for P2P tests.
514class P2PDownloadActionTest : public testing::Test {
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700515 protected:
David Zeuthen8f191b22013-08-06 12:27:50 -0700516 P2PDownloadActionTest()
Amin Hassani008c4582019-01-13 16:22:47 -0800517 : start_at_offset_(0), fake_um_(fake_system_state_.fake_clock()) {}
David Zeuthen8f191b22013-08-06 12:27:50 -0700518
Alex Deymo610277e2014-11-11 21:18:11 -0800519 ~P2PDownloadActionTest() override {}
David Zeuthen8f191b22013-08-06 12:27:50 -0700520
521 // Derived from testing::Test.
Amin Hassani008c4582019-01-13 16:22:47 -0800522 void SetUp() override { loop_.SetAsCurrent(); }
David Zeuthen8f191b22013-08-06 12:27:50 -0700523
524 // Derived from testing::Test.
Amin Hassani008c4582019-01-13 16:22:47 -0800525 void TearDown() override { EXPECT_FALSE(loop_.PendingTasks()); }
David Zeuthen8f191b22013-08-06 12:27:50 -0700526
527 // To be called by tests to setup the download. The
528 // |starting_offset| parameter is for where to resume.
529 void SetupDownload(off_t starting_offset) {
530 start_at_offset_ = starting_offset;
531 // Prepare data 10 kB of data.
532 data_.clear();
533 for (unsigned int i = 0; i < 10 * 1000; i++)
534 data_ += 'a' + (i % 25);
535
536 // Setup p2p.
Amin Hassani008c4582019-01-13 16:22:47 -0800537 FakeP2PManagerConfiguration* test_conf = new FakeP2PManagerConfiguration();
538 p2p_manager_.reset(P2PManager::Construct(test_conf,
539 nullptr,
540 &fake_um_,
541 "cros_au",
542 3,
543 base::TimeDelta::FromDays(5)));
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700544 fake_system_state_.set_p2p_manager(p2p_manager_.get());
David Zeuthen8f191b22013-08-06 12:27:50 -0700545 }
546
547 // To be called by tests to perform the download. The
548 // |use_p2p_to_share| parameter is used to indicate whether the
549 // payload should be shared via p2p.
550 void StartDownload(bool use_p2p_to_share) {
Gilad Arnold74b5f552014-10-07 08:17:16 -0700551 EXPECT_CALL(*fake_system_state_.mock_payload_state(),
552 GetUsingP2PForSharing())
553 .WillRepeatedly(Return(use_p2p_to_share));
David Zeuthen8f191b22013-08-06 12:27:50 -0700554
555 ScopedTempFile output_temp_file;
556 TestDirectFileWriter writer;
Alex Deymobffa0602016-02-12 17:16:29 -0800557 EXPECT_EQ(
558 0, writer.Open(output_temp_file.path().c_str(), O_WRONLY | O_CREAT, 0));
Alex Deymo64d98782016-02-05 18:03:48 -0800559 InstallPlan install_plan;
Sen Jiang0affc2c2017-02-10 15:55:05 -0800560 install_plan.payloads.push_back(
561 {.size = data_.length(),
562 .hash = {'1', '2', '3', '4', 'h', 'a', 's', 'h'}});
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700563 auto feeder_action = std::make_unique<ObjectFeederAction<InstallPlan>>();
564 feeder_action->set_obj(install_plan);
Alex Deymo8427b4a2014-11-05 14:00:32 -0800565 MockPrefs prefs;
David Zeuthen8f191b22013-08-06 12:27:50 -0700566 // Note that DownloadAction takes ownership of the passed in HttpFetcher.
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700567 auto download_action = std::make_unique<DownloadAction>(
568 &prefs,
569 fake_system_state_.boot_control(),
570 fake_system_state_.hardware(),
571 &fake_system_state_,
572 new MockHttpFetcher(data_.c_str(), data_.length(), nullptr),
573 false /* interactive */);
574 auto http_fetcher = download_action->http_fetcher();
575 download_action->SetTestFileWriter(&writer);
576 BondActions(feeder_action.get(), download_action.get());
577 delegate_.expected_data_ =
578 brillo::Blob(data_.begin() + start_at_offset_, data_.end());
579 delegate_.path_ = output_temp_file.path();
580 processor_.set_delegate(&delegate_);
581 processor_.EnqueueAction(std::move(feeder_action));
582 processor_.EnqueueAction(std::move(download_action));
David Zeuthen8f191b22013-08-06 12:27:50 -0700583
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700584 loop_.PostTask(
585 FROM_HERE,
586 base::Bind(
587 [](P2PDownloadActionTest* action_test, HttpFetcher* http_fetcher) {
588 action_test->processor_.StartProcessing();
589 http_fetcher->SetOffset(action_test->start_at_offset_);
590 },
591 base::Unretained(this),
592 base::Unretained(http_fetcher)));
Alex Deymo60ca1a72015-06-18 18:19:15 -0700593 loop_.Run();
David Zeuthen8f191b22013-08-06 12:27:50 -0700594 }
595
Alex Deymo60ca1a72015-06-18 18:19:15 -0700596 // Mainloop used to make StartDownload() synchronous.
Alex Vakulenko3f39d5c2015-10-13 09:27:13 -0700597 brillo::FakeMessageLoop loop_{nullptr};
Alex Deymo60ca1a72015-06-18 18:19:15 -0700598
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700599 // Delegate that is passed to the ActionProcessor.
600 DownloadActionTestProcessorDelegate delegate_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700601
602 // The P2PManager used in the test.
Ben Chan02f7c1d2014-10-18 15:18:02 -0700603 unique_ptr<P2PManager> p2p_manager_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700604
605 // The ActionProcessor used for running the actions.
606 ActionProcessor processor_;
607
608 // A fake system state.
Gilad Arnold5bb4c902014-04-10 12:32:13 -0700609 FakeSystemState fake_system_state_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700610
611 // The data being downloaded.
612 string data_;
613
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700614 private:
David Zeuthen8f191b22013-08-06 12:27:50 -0700615 // The requested starting offset passed to SetupDownload().
616 off_t start_at_offset_;
Gilad Arnold4a0321b2014-10-28 15:57:30 -0700617
618 chromeos_update_manager::FakeUpdateManager fake_um_;
David Zeuthen8f191b22013-08-06 12:27:50 -0700619};
620
621TEST_F(P2PDownloadActionTest, IsWrittenTo) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700622 SetupDownload(0); // starting_offset
623 StartDownload(true); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700624
625 // Check the p2p file and its content matches what was sent.
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700626 string file_id = delegate_.p2p_file_id_;
Gilad Arnold74b5f552014-10-07 08:17:16 -0700627 EXPECT_NE("", file_id);
Alex Deymo80f70ff2016-02-10 16:08:11 -0800628 EXPECT_EQ(static_cast<int>(data_.length()),
629 p2p_manager_->FileGetSize(file_id));
630 EXPECT_EQ(static_cast<int>(data_.length()),
631 p2p_manager_->FileGetExpectedSize(file_id));
David Zeuthen8f191b22013-08-06 12:27:50 -0700632 string p2p_file_contents;
Amin Hassani008c4582019-01-13 16:22:47 -0800633 EXPECT_TRUE(
634 ReadFileToString(p2p_manager_->FileGetPath(file_id), &p2p_file_contents));
David Zeuthen8f191b22013-08-06 12:27:50 -0700635 EXPECT_EQ(data_, p2p_file_contents);
636}
637
638TEST_F(P2PDownloadActionTest, DeleteIfHoleExists) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700639 SetupDownload(1000); // starting_offset
640 StartDownload(true); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700641
642 // DownloadAction should convey that the file is not being shared.
643 // and that we don't have any p2p files.
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700644 EXPECT_EQ(delegate_.p2p_file_id_, "");
David Zeuthen8f191b22013-08-06 12:27:50 -0700645 EXPECT_EQ(p2p_manager_->CountSharedFiles(), 0);
646}
647
648TEST_F(P2PDownloadActionTest, CanAppend) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700649 SetupDownload(1000); // starting_offset
David Zeuthen8f191b22013-08-06 12:27:50 -0700650
651 // Prepare the file with existing data before starting to write to
652 // it via DownloadAction.
Sen Jiang2703ef42017-03-16 13:36:21 -0700653 string file_id = utils::CalculateP2PFileId(
654 {'1', '2', '3', '4', 'h', 'a', 's', 'h'}, data_.length());
David Zeuthen8f191b22013-08-06 12:27:50 -0700655 ASSERT_TRUE(p2p_manager_->FileShare(file_id, data_.length()));
656 string existing_data;
657 for (unsigned int i = 0; i < 1000; i++)
658 existing_data += '0' + (i % 10);
Amin Hassani008c4582019-01-13 16:22:47 -0800659 ASSERT_EQ(
660 WriteFile(
661 p2p_manager_->FileGetPath(file_id), existing_data.c_str(), 1000),
662 1000);
David Zeuthen8f191b22013-08-06 12:27:50 -0700663
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700664 StartDownload(true); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700665
666 // DownloadAction should convey the same file_id and the file should
667 // have the expected size.
Amin Hassanid3f4bea2018-04-30 14:52:40 -0700668 EXPECT_EQ(delegate_.p2p_file_id_, file_id);
Alex Deymo80f70ff2016-02-10 16:08:11 -0800669 EXPECT_EQ(static_cast<ssize_t>(data_.length()),
670 p2p_manager_->FileGetSize(file_id));
671 EXPECT_EQ(static_cast<ssize_t>(data_.length()),
672 p2p_manager_->FileGetExpectedSize(file_id));
David Zeuthen8f191b22013-08-06 12:27:50 -0700673 string p2p_file_contents;
674 // Check that the first 1000 bytes wasn't touched and that we
675 // appended the remaining as appropriate.
Amin Hassani008c4582019-01-13 16:22:47 -0800676 EXPECT_TRUE(
677 ReadFileToString(p2p_manager_->FileGetPath(file_id), &p2p_file_contents));
David Zeuthen8f191b22013-08-06 12:27:50 -0700678 EXPECT_EQ(existing_data, p2p_file_contents.substr(0, 1000));
679 EXPECT_EQ(data_.substr(1000), p2p_file_contents.substr(1000));
680}
681
David Zeuthen8f191b22013-08-06 12:27:50 -0700682TEST_F(P2PDownloadActionTest, DeletePartialP2PFileIfResumingWithoutP2P) {
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700683 SetupDownload(1000); // starting_offset
David Zeuthen8f191b22013-08-06 12:27:50 -0700684
685 // Prepare the file with all existing data before starting to write
686 // to it via DownloadAction.
Sen Jiang2703ef42017-03-16 13:36:21 -0700687 string file_id = utils::CalculateP2PFileId(
688 {'1', '2', '3', '4', 'h', 'a', 's', 'h'}, data_.length());
David Zeuthen8f191b22013-08-06 12:27:50 -0700689 ASSERT_TRUE(p2p_manager_->FileShare(file_id, data_.length()));
690 string existing_data;
691 for (unsigned int i = 0; i < 1000; i++)
692 existing_data += '0' + (i % 10);
Amin Hassani008c4582019-01-13 16:22:47 -0800693 ASSERT_EQ(
694 WriteFile(
695 p2p_manager_->FileGetPath(file_id), existing_data.c_str(), 1000),
696 1000);
David Zeuthen8f191b22013-08-06 12:27:50 -0700697
698 // Check that the file is there.
Alex Deymo80f70ff2016-02-10 16:08:11 -0800699 EXPECT_EQ(1000, p2p_manager_->FileGetSize(file_id));
700 EXPECT_EQ(1, p2p_manager_->CountSharedFiles());
David Zeuthen8f191b22013-08-06 12:27:50 -0700701
Alex Vakulenkod2779df2014-06-16 13:19:00 -0700702 StartDownload(false); // use_p2p_to_share
David Zeuthen8f191b22013-08-06 12:27:50 -0700703
704 // DownloadAction should have deleted the p2p file. Check that it's gone.
Alex Deymo80f70ff2016-02-10 16:08:11 -0800705 EXPECT_EQ(-1, p2p_manager_->FileGetSize(file_id));
706 EXPECT_EQ(0, p2p_manager_->CountSharedFiles());
David Zeuthen8f191b22013-08-06 12:27:50 -0700707}
708
rspangler@google.com49fdf182009-10-10 00:57:34 +0000709} // namespace chromeos_update_engine