Skip to content

Commit a7a9b78

Browse files
authored
Implement get_node_type_descriptions_interface for lifecyclenode and add smoke test for it (#2237)
Signed-off-by: Emerson Knapp <[email protected]>
1 parent 945d254 commit a7a9b78

File tree

4 files changed

+12
-0
lines changed

4 files changed

+12
-0
lines changed

rclcpp/test/rclcpp/test_node.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,7 @@ TEST_F(TestNode, construction_and_destruction) {
7878
EXPECT_NE(nullptr, node->get_node_options().get_rcl_node_options());
7979
EXPECT_NE(nullptr, node->get_graph_event());
8080
EXPECT_NE(nullptr, node->get_clock());
81+
EXPECT_NE(nullptr, node->get_node_type_descriptions_interface());
8182
}
8283

8384
{

rclcpp_lifecycle/include/rclcpp_lifecycle/lifecycle_node.hpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -824,6 +824,10 @@ class LifecycleNode : public node_interfaces::LifecycleNodeInterface,
824824
rclcpp::node_interfaces::NodeTimeSourceInterface::SharedPtr
825825
get_node_time_source_interface();
826826

827+
/// Return the Node's internal NodeTypeDescriptionsInterface implementation.
828+
/**
829+
* \sa rclcpp::Node::get_node_type_descriptions_interface
830+
*/
827831
RCLCPP_LIFECYCLE_PUBLIC
828832
rclcpp::node_interfaces::NodeTypeDescriptionsInterface::SharedPtr
829833
get_node_type_descriptions_interface();

rclcpp_lifecycle/src/lifecycle_node.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -475,6 +475,12 @@ LifecycleNode::get_node_topics_interface()
475475
return node_topics_;
476476
}
477477

478+
rclcpp::node_interfaces::NodeTypeDescriptionsInterface::SharedPtr
479+
LifecycleNode::get_node_type_descriptions_interface()
480+
{
481+
return node_type_descriptions_;
482+
}
483+
478484
rclcpp::node_interfaces::NodeServicesInterface::SharedPtr
479485
LifecycleNode::get_node_services_interface()
480486
{

rclcpp_lifecycle/test/test_lifecycle_node.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -641,6 +641,7 @@ TEST_F(TestDefaultStateMachine, test_getters) {
641641
EXPECT_LT(0u, test_node->now().nanoseconds());
642642
EXPECT_STREQ("testnode", test_node->get_logger().get_name());
643643
EXPECT_NE(nullptr, const_cast<const EmptyLifecycleNode *>(test_node.get())->get_clock());
644+
EXPECT_NE(nullptr, test_node->get_node_type_descriptions_interface());
644645
}
645646

646647
TEST_F(TestDefaultStateMachine, test_graph_topics) {

0 commit comments

Comments
 (0)