diff options
| author | Yifan Hong <elsk@google.com> | 2020-08-13 13:59:54 -0700 |
|---|---|---|
| committer | Treehugger Robot <treehugger-gerrit@google.com> | 2020-08-19 18:42:31 +0000 |
| commit | 8d6df9ac7a70f4b07ebb86f50fb3548b693acad5 (patch) | |
| tree | 0173db33bc1bd984878ca8bd254a8f0b4e3b051d /dynamic_partition_test_utils.h | |
| parent | b753e0e9258170dce7c62733eed78e616b4a37b2 (diff) | |
Check allocatable space correctly when sideloading on VAB
On a device with Virtual A/B, when sideloading and there's
not enough space in super partition to hold CoW, update_engine
falls back to overwriting all source partitions. In that case,
the allocatable space should be the whole super partition, not
a half of it.
Also update doc comments.
Test: unit test. RecoveryErrorShouldDeleteSource fails without the patch
but succeeds with the patch.
Bug: 163613538
Change-Id: I6bd6895a7eabeb4e8436e57b0ac6830c11d1e98f
Diffstat (limited to 'dynamic_partition_test_utils.h')
| -rw-r--r-- | dynamic_partition_test_utils.h | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/dynamic_partition_test_utils.h b/dynamic_partition_test_utils.h index 70a176b5..d701dce8 100644 --- a/dynamic_partition_test_utils.h +++ b/dynamic_partition_test_utils.h @@ -175,9 +175,11 @@ inline DeltaArchiveManifest PartitionSizesToManifest( } inline std::unique_ptr<MetadataBuilder> NewFakeMetadata( - const DeltaArchiveManifest& manifest, uint32_t partition_attr = 0) { + const DeltaArchiveManifest& manifest, + uint32_t partition_attr = 0, + uint64_t super_size = kDefaultSuperSize) { auto builder = - MetadataBuilder::New(kDefaultSuperSize, kFakeMetadataSize, kMaxNumSlots); + MetadataBuilder::New(super_size, kFakeMetadataSize, kMaxNumSlots); for (const auto& group : manifest.dynamic_partition_metadata().groups()) { EXPECT_TRUE(builder->AddGroup(group.name(), group.size())); for (const auto& partition_name : group.partition_names()) { |