Merge pull request ceph#63292 from tchaikov/test_not_before_queue-fix-leaks

test/test_not_before_queue: Fix memory leak by freeing ineligible items in destructor

Reviewed-by: Ronen Friedman <[email protected]>

Author: tchaikov

src/test/test_not_before_queue.cc

@@ -62,6 +62,16 @@ class NotBeforeTest : public testing::Test {
  public:
   using queue_t = not_before_queue_t<tv_t, test_time_t>;
 
+  ~NotBeforeTest() {
+    // Advance time until all queued items become eligible for processing.
+    // This ensures complete dequeuing during test teardown, preventing memory leaks.
+    // We assume test cutoff timepoints remain within reasonable bounds. If a specified
+    // timepoint precedes current time, the process continues advancing normally.
+    for (unsigned when = 1; queue.eligible_count() < queue.total_count(); when += 1) {
+      queue.advance_time(when);
+    }
+    while (queue.dequeue()) {}
+  }
   void load_test_data(const std::vector<tv_t> &dt) {
     for (const auto &d : dt) {
       queue.enqueue(d);