From 482fa3041abd08038a4f8b0d8e720dfd566e6fcf Mon Sep 17 00:00:00 2001 From: Ronen Friedman Date: Sat, 4 Apr 2026 13:50:49 +0300 Subject: [PATCH] crimson/test: rm 'assert's in test_object_data_handlers.cc As these 'assert's triggered 'unused variable' warnings. Replaced with equivalent ASSERT_EQ() macros. Signed-off-by: Ronen Friedman --- src/test/crimson/seastore/test_object_data_handler.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/crimson/seastore/test_object_data_handler.cc b/src/test/crimson/seastore/test_object_data_handler.cc index 248fb739d623..b9b354f4417b 100644 --- a/src/test/crimson/seastore/test_object_data_handler.cc +++ b/src/test/crimson/seastore/test_object_data_handler.cc @@ -816,7 +816,7 @@ TEST_P(object_data_handler_test_t, multiple_remap) { auto base = pins.front().get_key(); int i = 0; for (auto &pin : pins) { - assert(pin.get_key().get_byte_distance(base) == res[i]); + ASSERT_EQ(pin.get_key().get_byte_distance(base), res[i]); i++; } read(0, 128<<10); @@ -882,7 +882,7 @@ TEST_P(object_data_handler_test_t, overwrite_then_read_within_transaction) { t = create_mutate_transaction(); { auto pins = get_mappings(*t, base, len); - assert(pins.size() == 1); + ASSERT_EQ(pins.size(), 1); auto pin1 = remap_pin(*t, std::move(pins.front()), 4096, 8192); auto ext = get_extent(*t, base + 4096, 4096 * 2); ASSERT_TRUE(ext->is_exist_clean()); -- 2.47.3