review

yijiem · yijiem · commit 168967e9de15 · 2024-12-05T16:35:47.000-08:00
diff --git a/test/cpp/common/alarm_test.cc b/test/cpp/common/alarm_test.cc
@@ -459,9 +459,7 @@ TEST(AlarmTest, CallbackSetInCallback) {
               });
   }
   std::unique_lock<std::mutex> l(c->mu);
-  EXPECT_TRUE(c->cv.wait_until(
-      l, std::chrono::system_clock::now() + std::chrono::seconds(20),
-      [c] { return c->completed; }));
+  c->cv.wait(l, [c] { return c->completed; });
 }
 
 }  // namespace

Original file line number	Diff line number	Diff line change
`@@ -459,9 +459,7 @@ TEST(AlarmTest, CallbackSetInCallback) {`
`459`	`459`	`});`
`460`	`460`	`}`
`461`	`461`	`std::unique_lock<std::mutex> l(c->mu);`
`462`		`- EXPECT_TRUE(c->cv.wait_until(`
`463`		`- l, std::chrono::system_clock::now() + std::chrono::seconds(20),`
`464`		`- [c] { return c->completed; }));`
	`462`	`+ c->cv.wait(l, [c] { return c->completed; });`
`465`	`463`	`}`
`466`	`464`
`467`	`465`	`} // namespace`