Skip to content

Commit 86019af

Browse files
author
Seppo Takalo
committed
Add unittest to verify removal of last event listener
1 parent e9f5ed4 commit 86019af

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

UNITTESTS/features/netsocket/NetworkInterface/test_NetworkInterface.cpp

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -168,6 +168,7 @@ TEST_F(TestNetworkInterface, add_event_listener)
168168

169169
TEST_F(TestNetworkInterface, remove_event_listener)
170170
{
171+
// Add two callback and check that both are called
171172
callback_is_called = false;
172173
second_callback_called = false;
173174
iface->add_event_listener(my_iface_callback);
@@ -176,15 +177,23 @@ TEST_F(TestNetworkInterface, remove_event_listener)
176177
EXPECT_EQ(callback_is_called, true);
177178
EXPECT_EQ(second_callback_called, true);
178179

180+
// Remove one of the callbacks
179181
iface->remove_event_listener(my_iface_callback2);
180182
callback_is_called = false;
181183
second_callback_called = false;
182184

185+
// expect only the one is called which remains in the list
183186
iface->event(NSAPI_EVENT_CONNECTION_STATUS_CHANGE, 0);
184187
EXPECT_EQ(callback_is_called, true);
185188
EXPECT_EQ(second_callback_called, false);
186189

190+
// Remove also the last callback, and expect nothing is called
187191
iface->remove_event_listener(my_iface_callback);
192+
callback_is_called = false;
193+
second_callback_called = false;
194+
iface->event(NSAPI_EVENT_CONNECTION_STATUS_CHANGE, 0);
195+
EXPECT_EQ(callback_is_called, false);
196+
EXPECT_EQ(second_callback_called, false);
188197
}
189198

190199
TEST_F(TestNetworkInterface, correct_event_listener_per_interface)

0 commit comments

Comments
 (0)