Call fini to avoid memory leak

Signed-off-by: Chen Lihui <Lihui.Chen@sony.com>
ros2 · Sep 14, 2020 · 06712f7 · 06712f7
1 parent cbf3359
commit 06712f7
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/rclcpp/test/rclcpp/executors/test_static_executor_entities_collector.cpp b/rclcpp/test/rclcpp/executors/test_static_executor_entities_collector.cpp
@@ -179,6 +179,7 @@ TEST_F(TestStaticExecutorEntitiesCollector, add_remove_basic_node) {
 
   // Still one for the executor
   EXPECT_EQ(1u, entities_collector_->get_number_of_waitables());
+  entities_collector_->fini();
 }
 
 TEST_F(TestStaticExecutorEntitiesCollector, add_remove_node_out_of_scope) {
@@ -213,6 +214,7 @@ TEST_F(TestStaticExecutorEntitiesCollector, add_remove_node_out_of_scope) {
 
   // Still one for the executor
   EXPECT_EQ(1u, entities_collector_->get_number_of_waitables());
+  entities_collector_->fini();
 }
 
 class TestWaitable : public rclcpp::Waitable
@@ -292,4 +294,5 @@ TEST_F(TestStaticExecutorEntitiesCollector, add_remove_node_with_entities) {
   EXPECT_EQ(0u, entities_collector_->get_number_of_clients());
   // Still one for the executor
   EXPECT_EQ(1u, entities_collector_->get_number_of_waitables());
+  entities_collector_->fini();
 }