Skip to content

Commit 0a03af5

Browse files
rotuKarsten1987
authored andcommitted
Disable parameter event publishers on test nodes (#180)
Signed-off-by: Dan Rose <[email protected]>
1 parent 1e3ba4b commit 0a03af5

File tree

4 files changed

+13
-4
lines changed

4 files changed

+13
-4
lines changed

rosbag2_test_common/include/rosbag2_test_common/publisher_manager.hpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,9 @@ class PublisherManager
4444
const std::string & topic_name, std::shared_ptr<T> message, size_t expected_messages = 0)
4545
{
4646
auto node_name = std::string("publisher") + std::to_string(counter_++);
47-
auto publisher_node = std::make_shared<rclcpp::Node>(node_name);
47+
auto publisher_node = std::make_shared<rclcpp::Node>(
48+
node_name,
49+
rclcpp::NodeOptions().start_parameter_event_publisher(false));
4850
auto publisher = publisher_node->create_publisher<T>(topic_name, 10);
4951

5052
publisher_nodes_.push_back(publisher_node);

rosbag2_test_common/include/rosbag2_test_common/subscription_manager.hpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,10 @@ class SubscriptionManager
3333
public:
3434
SubscriptionManager()
3535
{
36-
subscriber_node_ = std::make_shared<rclcpp::Node>("subscriber_node");
36+
subscriber_node_ = std::make_shared<rclcpp::Node>(
37+
"subscriber_node",
38+
rclcpp::NodeOptions().start_parameter_event_publisher(false)
39+
);
3740
}
3841

3942
template<typename MessageT>

rosbag2_transport/test/rosbag2_transport/test_record_all_no_discovery.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ TEST_F(RecordIntegrationTestFixture, record_all_without_discovery_ignores_later_
2828
auto string_message = get_messages_strings()[0];
2929
string_message->string_value = "Hello World";
3030

31-
auto publisher_node = std::make_shared<rclcpp::Node>("publisher_for_test");
31+
auto publisher_node = std::make_shared<rclcpp::Node>(
32+
"publisher_for_test",
33+
rclcpp::NodeOptions().start_parameter_event_publisher(false));
3234

3335
start_recording({true, true, {}, "rmw_format", 1ms});
3436

rosbag2_transport/test/rosbag2_transport/test_rosbag2_node.cpp

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,9 @@ class RosBag2NodeFixture : public Test
3434
RosBag2NodeFixture()
3535
{
3636
node_ = std::make_shared<rosbag2_transport::Rosbag2Node>("rosbag2");
37-
publisher_node_ = std::make_shared<rclcpp::Node>("publisher_node");
37+
publisher_node_ = std::make_shared<rclcpp::Node>(
38+
"publisher_node",
39+
rclcpp::NodeOptions().start_parameter_event_publisher(false));
3840
}
3941

4042
static void SetUpTestCase()

0 commit comments

Comments
 (0)