Skip to content

G
googletest
Commit 6a5eb807 by Abseil Team Committed by Dino Radaković
Options

Googletest export

Update the example for Notify to use a lambda. It is much less boilerplate and easier to remember. PiperOrigin-RevId: 367284222
parent 8a65bc03
Showing with 1 addition and 9 deletions
docs/gmock_cook_book.md
...@@ -2705,18 +2705,10 @@ behavior nondeterministic. A better way is to use gMock actions and ...@@ -2705,18 +2705,10 @@ behavior nondeterministic. A better way is to use gMock actions and
`Notification` objects to force your asynchronous test to behave synchronously. `Notification` objects to force your asynchronous test to behave synchronously.
```cpp ```cpp
using ::testing::DoAll;
using ::testing::InvokeWithoutArgs;
using ::testing::Return;
class MockEventDispatcher : public EventDispatcher { class MockEventDispatcher : public EventDispatcher {
MOCK_METHOD(bool, DispatchEvent, (int32), (override)); MOCK_METHOD(bool, DispatchEvent, (int32), (override));
}; };
ACTION_P(Notify, notification) {
notification->Notify();
}
TEST(EventQueueTest, EnqueueEventTest) { TEST(EventQueueTest, EnqueueEventTest) {
MockEventDispatcher mock_event_dispatcher; MockEventDispatcher mock_event_dispatcher;
EventQueue event_queue(&mock_event_dispatcher); EventQueue event_queue(&mock_event_dispatcher);
...@@ -2724,7 +2716,7 @@ TEST(EventQueueTest, EnqueueEventTest) { ...@@ -2724,7 +2716,7 @@ TEST(EventQueueTest, EnqueueEventTest) {
const int32 kEventId = 321; const int32 kEventId = 321;
absl::Notification done; absl::Notification done;
EXPECT_CALL(mock_event_dispatcher, DispatchEvent(kEventId)) EXPECT_CALL(mock_event_dispatcher, DispatchEvent(kEventId))
.WillOnce(Notify(&done)); .WillOnce([&done] { done.Notify(); });
event_queue.EnqueueEvent(kEventId); event_queue.EnqueueEvent(kEventId);
done.WaitForNotification(); done.WaitForNotification();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment