diff options
author | Tor Egge <Tor.Egge@broadpark.no> | 2020-10-28 17:32:09 +0100 |
---|---|---|
committer | Tor Egge <Tor.Egge@broadpark.no> | 2020-10-28 18:37:35 +0100 |
commit | b81f0401645d8cb1dd5c6373bf16c1fad50adb21 (patch) | |
tree | d524ca9e3845cd6fe273c836dd064bff56bcbda4 /storage/src/tests/persistence/has_mask_remapper_test.cpp | |
parent | 02785bc255415bb26aaf57be10b4261e2ce65bc8 (diff) |
Keep hasMask bits for nodes not involved in merge operation step.
Diffstat (limited to 'storage/src/tests/persistence/has_mask_remapper_test.cpp')
-rw-r--r-- | storage/src/tests/persistence/has_mask_remapper_test.cpp | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/storage/src/tests/persistence/has_mask_remapper_test.cpp b/storage/src/tests/persistence/has_mask_remapper_test.cpp index 58f6f5b3fe7..de87797fb26 100644 --- a/storage/src/tests/persistence/has_mask_remapper_test.cpp +++ b/storage/src/tests/persistence/has_mask_remapper_test.cpp @@ -15,6 +15,7 @@ TEST(HasMaskRemapperTest, test_remap_none) for (uint32_t i = 0; i < (1u << merge_operation_nodes.size()); ++i) { EXPECT_EQ(i, remap_mask(i)); } + EXPECT_EQ(31u, remap_mask(255u)); } TEST(HasMaskRemapperTest, test_remap_subset) @@ -39,4 +40,18 @@ TEST(HasMaskRemapperTest, test_remap_swapped_subset) EXPECT_EQ((std::vector<uint16_t>{0u, 2u, 1u, 3u}), remapped); } +TEST(HasMaskRemapperTest, test_keep_unremapped_bits) +{ + NodeList reply_nodes{{0, true}, {1, true}, {3, false}}; + HasMaskRemapper remap_mask(merge_operation_nodes, reply_nodes); + EXPECT_EQ(20u, remap_mask(0u, (1u << 5) - 1)); + EXPECT_EQ(11u, remap_mask((1u << 3) - 1, 0u)); + EXPECT_EQ(31u, remap_mask((1u << 3) - 1, (1u << 5) - 1)); + EXPECT_EQ(24u, remap_mask(4u, 16u)); + HasMaskRemapper same_nodes_remap_mask(merge_operation_nodes, merge_operation_nodes); + EXPECT_EQ(31u, same_nodes_remap_mask(255u, 0u)); + EXPECT_EQ(224u, same_nodes_remap_mask(0u, 255u)); + EXPECT_EQ(255u, same_nodes_remap_mask(255u, 255u)); +} + } |