Bt test fix

nav2_behavior_tree/test/plugins/action/test_back_up_cancel_node.cpp

@@ -83,6 +83,7 @@ class CancelBackUpActionTestFixture : public ::testing::Test
     config_ = nullptr;
     node_.reset();
     action_server_.reset();
+    client_.reset();
     factory_.reset();
   }
 

nav2_behavior_tree/test/plugins/action/test_controller_cancel_node.cpp

@@ -83,6 +83,7 @@ class CancelControllerActionTestFixture : public ::testing::Test
     config_ = nullptr;
     node_.reset();
     action_server_.reset();
+    client_.reset();
     factory_.reset();
   }
 

nav2_behavior_tree/test/plugins/action/test_drive_on_heading_cancel_node.cpp

@@ -86,6 +86,7 @@ class CancelDriveOnHeadingTestFixture : public ::testing::Test
     config_ = nullptr;
     node_.reset();
     action_server_.reset();
+    client_.reset();
     factory_.reset();
   }
 

nav2_behavior_tree/test/plugins/action/test_spin_cancel_node.cpp

@@ -83,6 +83,7 @@ class CancelSpinActionTestFixture : public ::testing::Test
     config_ = nullptr;
     node_.reset();
     action_server_.reset();
+    client_.reset();
     factory_.reset();
   }
 

nav2_behavior_tree/test/plugins/action/test_wait_cancel_node.cpp

@@ -83,6 +83,7 @@ class CancelWaitActionTestFixture : public ::testing::Test
     config_ = nullptr;
     node_.reset();
     action_server_.reset();
+    client_.reset();
     factory_.reset();
   }
 

nav2_behavior_tree/test/plugins/condition/test_is_path_valid.cpp

@@ -67,9 +67,10 @@ class IsPathValidTestFixture : public ::testing::Test
   {
     delete config_;
     config_ = nullptr;
+    server_.reset();
     node_.reset();
     factory_.reset();
-    server_.reset();
+    tree_.reset();
   }
 
   static std::shared_ptr<IsPathValidService> server_;

nav2_behavior_tree/test/plugins/condition/test_path_expiring_timer.cpp

@@ -43,6 +43,7 @@ class PathExpiringTimerConditionTestFixture : public nav2_behavior_tree::Behavio
   {
     delete config_;
     config_ = nullptr;
+    node_.reset();
     bt_node_.reset();
   }