test: ceph_test_filejournal Add check of journalq in WriteTrim test

Commit all journal entries and check for empty journalq

Signed-off-by: David Zafman <dzafman@redhat.com>

diff --git a/src/os/FileJournal.h b/src/os/FileJournal.h
index 964dbe721bc76587fc02164199e9c014da6b5275..d565252f4efdeed2a2464017c3e2bd9ab446b510 100644 (file)
--- a/src/os/FileJournal.h
+++ b/src/os/FileJournal.h
@@ -214,6 +214,8 @@ public:
     }
   } __attribute__((__packed__, aligned(4)));
 
+  bool journalq_empty() { return journalq.empty(); }
+
 private:
   string fn;
 

diff --git a/src/test/test_filejournal.cc b/src/test/test_filejournal.cc
index befe761400db918aef4a1ee5696c31eaa77090a1..5bbaab1c99fb076fe999470a391c157654ff4ef0 100644 (file)
--- a/src/test/test_filejournal.cc
+++ b/src/test/test_filejournal.cc
@@ -360,9 +360,11 @@ TEST(TestFileJournal, WriteTrim) {
   while (ls.size()) {
     delete ls.front();
     ls.pop_front();
-    j.committed_thru(committed);
+    j.committed_thru(++committed);
   }
 
+  ASSERT_TRUE(j.journalq_empty());
+
   j.close();
 }