diff options
author | Sen Jiang <senj@google.com> | 2015-08-28 13:23:27 -0700 |
---|---|---|
committer | Sen Jiang <senj@google.com> | 2015-09-18 12:30:46 -0700 |
commit | 70a6ab091beb261a501012bc09955de452dd4736 (patch) | |
tree | 252b2c67e0535aec5a0f1439c1b911528475f1fe /payload_generator/payload_file_unittest.cc | |
parent | 5fc5e10aaaa476bce655efa86a3bf3ec25210620 (diff) |
Move install operations to partitions field in major version 2.
install_operations and kernel_install_operations will be empty in major
version 2, they are now present in partitions field.
partition_info are still in old location, more refactoring are needed before
moving them.
This patch also moves the kLegancyPartitionName constants from install_plan.h
to payload_constants.h.
TEST=cros_generate_update_payload generate same payload version 1 as before.
Call delta_generator manually with --major_version=2, then cros payload show.
cros_workon_make update_engine --test
Bug: 23694580
Change-Id: If9e9531656480b1174e13b39af7ecb6a81060aac
Diffstat (limited to 'payload_generator/payload_file_unittest.cc')
-rw-r--r-- | payload_generator/payload_file_unittest.cc | 27 |
1 files changed, 12 insertions, 15 deletions
diff --git a/payload_generator/payload_file_unittest.cc b/payload_generator/payload_file_unittest.cc index 2206d273..ff8c8a44 100644 --- a/payload_generator/payload_file_unittest.cc +++ b/payload_generator/payload_file_unittest.cc @@ -22,7 +22,6 @@ #include <gtest/gtest.h> -#include "update_engine/payload_constants.h" #include "update_engine/payload_generator/extent_ranges.h" #include "update_engine/test_utils.h" @@ -64,33 +63,31 @@ TEST_F(PayloadFileTest, ReorderBlobsTest) { aop.op.set_data_offset(7); aop.op.set_data_length(1); aops.push_back(aop); - payload_.AddPartitionOperations(PartitionName::kRootfs, aops); + payload_.AddPartitionOperations("part0", aops); aop.op.set_data_offset(0); aop.op.set_data_length(6); aops = {aop}; - payload_.AddPartitionOperations(PartitionName::kKernel, aops); + payload_.AddPartitionOperations("part1", aops); EXPECT_TRUE(payload_.ReorderDataBlobs(orig_blobs, new_blobs)); - const vector<AnnotatedOperation>& rootfs_aops = - payload_.aops_map_[PartitionName::kRootfs]; - const vector<AnnotatedOperation>& kernel_aops = - payload_.aops_map_[PartitionName::kKernel]; + const vector<AnnotatedOperation>& part0_aops = payload_.aops_vec_[0].second; + const vector<AnnotatedOperation>& part1_aops = payload_.aops_vec_[1].second; string new_data; EXPECT_TRUE(utils::ReadFile(new_blobs, &new_data)); // Kernel blobs should appear at the end. EXPECT_EQ("bcdakernel", new_data); - EXPECT_EQ(2, rootfs_aops.size()); - EXPECT_EQ(0, rootfs_aops[0].op.data_offset()); - EXPECT_EQ(3, rootfs_aops[0].op.data_length()); - EXPECT_EQ(3, rootfs_aops[1].op.data_offset()); - EXPECT_EQ(1, rootfs_aops[1].op.data_length()); + EXPECT_EQ(2, part0_aops.size()); + EXPECT_EQ(0, part0_aops[0].op.data_offset()); + EXPECT_EQ(3, part0_aops[0].op.data_length()); + EXPECT_EQ(3, part0_aops[1].op.data_offset()); + EXPECT_EQ(1, part0_aops[1].op.data_length()); - EXPECT_EQ(1, kernel_aops.size()); - EXPECT_EQ(4, kernel_aops[0].op.data_offset()); - EXPECT_EQ(6, kernel_aops[0].op.data_length()); + EXPECT_EQ(1, part1_aops.size()); + EXPECT_EQ(4, part1_aops[0].op.data_offset()); + EXPECT_EQ(6, part1_aops[0].op.data_length()); } } // namespace chromeos_update_engine |