Change the default capture of complete_func to capture return_status by value:

Now every TestNodeConfig instance has its own copy of return_status, and no dangling pointer ever occurs, eliminating the illegal IDLE return in onRunning().

Author: pepeRossRobotics

include/behaviortree_cpp/actions/test_node.h

@@ -39,7 +39,7 @@ struct TestNodeConfig
 
   /// Function invoked when the action is completed. By default just return [return_status]
   /// Override it to intorduce more comple cases
-  std::function<NodeStatus(void)> complete_func = [this]() { return return_status; };
+  std::function<NodeStatus(void)> complete_func = [ret = return_status]() { return ret; };
 };
 
 /**