test/mds: don't drop await_idle_v under NDEBUG

await_idle_v() is wrapped in assert(), so with NDEBUG it is never
called and the test reads a default-constructed ack, racing the agent
thread. Use EXPECT_TRUE() like the rest of the file.

Signed-off-by: Sun Yuechi <sunyuechi@iscas.ac.cn>

diff --git a/src/test/mds/TestQuiesceAgent.cc b/src/test/mds/TestQuiesceAgent.cc
index aa108075faa59fec54bde8a62f4cb83caf9788b6..7940d8a500406189539cf53d2dabae90410c6f6b 100644 (file)
--- a/src/test/mds/TestQuiesceAgent.cc
+++ b/src/test/mds/TestQuiesceAgent.cc
@@ -192,7 +192,7 @@ class QuiesceAgentTest : public testing::Test {
       if (WaitForAgent::No == wait) {
         return std::nullopt;
       } else {
-        assert(await_idle_v(v.set_version));
+        EXPECT_TRUE(await_idle_v(v.set_version));
         return async_ack;
       }
     }