test/store_test: Fix BluestorePerPoolOmapFixOnMount

We called inject_legacy_omap() before waiting to commit
initial transaction that created the object.
The results is that sometimes the initial clean state
removed legacy omap signalling used by unittest.

Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>
(cherry picked from commit 3c9defab46e3ea7f9b9fbd05bdae9d6597687f0a)

diff --git a/src/test/objectstore/store_test.cc b/src/test/objectstore/store_test.cc
index db4b509b3e7e441896207120ae4cf50421b267f3..38ce90a95b73cb82e223f70de18fb352d2d61f29 100644 (file)
--- a/src/test/objectstore/store_test.cc
+++ b/src/test/objectstore/store_test.cc
@@ -10703,8 +10703,11 @@ TEST_P(StoreTest, BluestorePerPoolOmapFixOnMount)
     t.omap_setheader(cid, oid, h);
     t.touch(cid, oid2);
     t.omap_setheader(cid, oid2, h);
+    C_SaferCond c;
+    t.register_on_commit(&c);
     int r = queue_transaction(store, ch, std::move(t));
     ASSERT_EQ(r, 0);
+    c.wait();
   }
 
   // inject legacy omaps