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 <rfriedma@redhat.com>

diff --git a/src/test/crimson/seastore/test_object_data_handler.cc b/src/test/crimson/seastore/test_object_data_handler.cc
index 248fb739d6237e2180168760e64955e76f6338d1..b9b354f4417bfff3781481c2eed41a9cd9ae3ca9 100644 (file)
--- 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<size_t>(base) == res[i]);
+      ASSERT_EQ(pin.get_key().get_byte_distance<size_t>(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());