Skip to content

Commit

Permalink
Merge pull request #94 from billpittman/dev/test/extra/NotificationTest
Browse files Browse the repository at this point in the history
NotificationTest: Fix bug in test wrt callback function
  • Loading branch information
gregmedd authored Aug 13, 2024
2 parents fff8796 + 7fb163e commit d0658a3
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/extra/NotificationTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ TEST_F(NotificationTest, BasicNotificationTestWithPayload) {
auto source_filter2 = getUUri(0x8001);
auto on_rx2 = [](const v1::UMessage& message) { FAIL(); };
auto maybe_sink2 = communication::NotificationSink::create(
transport, std::move(on_rx), std::move(source_filter2));
transport, std::move(on_rx2), std::move(source_filter2));
EXPECT_TRUE(maybe_sink2.has_value());

// Send the notification messages
Expand Down Expand Up @@ -130,7 +130,7 @@ TEST_F(NotificationTest, BasicNotificationTestWithoutPayload) {
auto source_filter2 = getUUri(0x8001);
auto on_rx2 = [](const v1::UMessage& message) { FAIL(); };
auto maybe_sink2 = communication::NotificationSink::create(
transport, std::move(on_rx), std::move(source_filter2));
transport, std::move(on_rx2), std::move(source_filter2));
EXPECT_TRUE(maybe_sink2.has_value());

// Send the notification messages
Expand Down

0 comments on commit d0658a3

Please sign in to comment.