From d6dfd6e596eb2debde8e0bbb6850c8462e5dd77e Mon Sep 17 00:00:00 2001
From: Dimitris Staratzis <33267511+DimitrisStaratzis@users.noreply.github.com>
Date: Wed, 4 Sep 2024 13:42:09 +0300
Subject: [PATCH] add more tests

---
 test/src/unit-cppapi-max-fragment-size.cc | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/test/src/unit-cppapi-max-fragment-size.cc b/test/src/unit-cppapi-max-fragment-size.cc
index d1fc02f77ea..a4dfc5f95c5 100644
--- a/test/src/unit-cppapi-max-fragment-size.cc
+++ b/test/src/unit-cppapi-max-fragment-size.cc
@@ -518,14 +518,18 @@ TEST_CASE(
   };
 
   cleanup();
+  std::string cons_buffer_size;
 
-  bool more_than_one_loop = false;
-  SECTION("More than one loop write") {
-    more_than_one_loop = true;
+  SECTION("More than one loop write, one row or more at the time") {
+    cons_buffer_size = "30";
+  }
+
+  SECTION("More than one loop write, one or two tiles at the time") {
+    cons_buffer_size = "10";
   }
 
   SECTION("One loop write") {
-    more_than_one_loop = false;
+    cons_buffer_size = "10000";  // needed only to speed up consolidation
   }
 
   // Remove the array at the end of this test.
@@ -582,10 +586,7 @@ TEST_CASE(
   // the creation of two new fragments.
   tiledb::Config cfg;
   cfg.set("sm.consolidation.max_fragment_size", "150");
-  cfg.set("sm.consolidation.buffer_size", "10000");  // speed up consolidation
-  if (more_than_one_loop) {
-    cfg.set("sm.consolidation.buffer_size", "10");
-  }
+  cfg.set("sm.consolidation.buffer_size", cons_buffer_size);
 
   ctx = Context(cfg);
   Array::consolidate(ctx, array_name);