Skip to content

Commit a51cde4

Browse files
committed
replace create_parameter with declare_parameter
Signed-off-by: William Woodall <william@osrfoundation.org>
1 parent c3558f1 commit a51cde4

File tree

2 files changed

+34
-34
lines changed

2 files changed

+34
-34
lines changed

test_rclcpp/test/parameter_fixtures.hpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -29,13 +29,13 @@ const double test_epsilon = 1e-6;
2929

3030
void declare_test_parameters(std::shared_ptr<rclcpp::Node> node)
3131
{
32-
node->create_parameter("foo");
33-
node->create_parameter("bar");
34-
node->create_parameter("barstr");
35-
node->create_parameter("baz");
36-
node->create_parameter("foo.first");
37-
node->create_parameter("foo.second");
38-
node->create_parameter("foobar");
32+
node->declare_parameter("foo");
33+
node->declare_parameter("bar");
34+
node->declare_parameter("barstr");
35+
node->declare_parameter("baz");
36+
node->declare_parameter("foo.first");
37+
node->declare_parameter("foo.second");
38+
node->declare_parameter("foobar");
3939
}
4040

4141
void set_test_parameters(

test_rclcpp/test/test_local_parameters.cpp

Lines changed: 27 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -110,12 +110,12 @@ class ParametersAsyncNode : public rclcpp::Node
110110
ParametersAsyncNode()
111111
: Node("test_local_parameters_async_with_callback")
112112
{
113-
this->create_parameter("foo");
114-
this->create_parameter("bar");
115-
this->create_parameter("barstr");
116-
this->create_parameter("baz");
117-
this->create_parameter("foobar");
118-
this->create_parameter("barfoo");
113+
this->declare_parameter("foo");
114+
this->declare_parameter("bar");
115+
this->declare_parameter("barstr");
116+
this->declare_parameter("baz");
117+
this->declare_parameter("foobar");
118+
this->declare_parameter("barfoo");
119119

120120
parameters_client_ =
121121
std::make_shared<rclcpp::AsyncParametersClient>(this);
@@ -167,12 +167,12 @@ TEST(CLASSNAME(test_local_parameters, RMW_IMPLEMENTATION), local_async_with_call
167167
TEST(CLASSNAME(test_local_parameters, RMW_IMPLEMENTATION), helpers) {
168168
if (!rclcpp::ok()) {rclcpp::init(0, nullptr);}
169169
auto node = rclcpp::Node::make_shared("test_parameters_local_helpers");
170-
node->create_parameter("foo");
171-
node->create_parameter("bar");
172-
node->create_parameter("barstr");
173-
node->create_parameter("baz");
174-
node->create_parameter("foobar");
175-
node->create_parameter("barfoo");
170+
node->declare_parameter("foo");
171+
node->declare_parameter("bar");
172+
node->declare_parameter("barstr");
173+
node->declare_parameter("baz");
174+
node->declare_parameter("foobar");
175+
node->declare_parameter("barfoo");
176176

177177
auto parameters_client = std::make_shared<rclcpp::SyncParametersClient>(node);
178178
if (!parameters_client->wait_for_service(20s)) {
@@ -284,12 +284,12 @@ TEST(CLASSNAME(test_local_parameters, RMW_IMPLEMENTATION), helpers) {
284284
TEST(CLASSNAME(test_local_parameters, RMW_IMPLEMENTATION), get_from_node_primitive_type) {
285285
if (!rclcpp::ok()) {rclcpp::init(0, nullptr);}
286286
auto node = rclcpp::Node::make_shared("test_parameters_local_helpers");
287-
node->create_parameter("foo");
288-
node->create_parameter("bar");
289-
node->create_parameter("barstr");
290-
node->create_parameter("baz");
291-
node->create_parameter("foobar");
292-
node->create_parameter("barfoo");
287+
node->declare_parameter("foo");
288+
node->declare_parameter("bar");
289+
node->declare_parameter("barstr");
290+
node->declare_parameter("baz");
291+
node->declare_parameter("foobar");
292+
node->declare_parameter("barfoo");
293293

294294
auto parameters_client = std::make_shared<rclcpp::SyncParametersClient>(node);
295295
if (!parameters_client->wait_for_service(20s)) {
@@ -357,12 +357,12 @@ TEST(CLASSNAME(test_local_parameters, RMW_IMPLEMENTATION), get_from_node_variant
357357
using rclcpp::Parameter;
358358

359359
auto node = rclcpp::Node::make_shared("test_parameters_local_helpers");
360-
node->create_parameter("foo");
361-
node->create_parameter("bar");
362-
node->create_parameter("barstr");
363-
node->create_parameter("baz");
364-
node->create_parameter("foobar");
365-
node->create_parameter("barfoo");
360+
node->declare_parameter("foo");
361+
node->declare_parameter("bar");
362+
node->declare_parameter("barstr");
363+
node->declare_parameter("baz");
364+
node->declare_parameter("foobar");
365+
node->declare_parameter("barfoo");
366366

367367
auto parameters_client = std::make_shared<rclcpp::SyncParametersClient>(node);
368368
if (!parameters_client->wait_for_service(20s)) {
@@ -422,7 +422,7 @@ TEST(CLASSNAME(test_local_parameters, RMW_IMPLEMENTATION), get_parameter_or) {
422422
using rclcpp::Parameter;
423423

424424
auto node = rclcpp::Node::make_shared("test_parameters_get_parameter_or");
425-
node->create_parameter("foo");
425+
node->declare_parameter("foo");
426426
auto set_parameters_results = node->set_parameters({
427427
Parameter("foo", 2),
428428
});
@@ -493,8 +493,8 @@ TEST(CLASSNAME(test_local_parameters, RMW_IMPLEMENTATION), set_parameter_if_not_
493493
using rclcpp::Parameter;
494494

495495
auto node = rclcpp::Node::make_shared("test_parameters_set_parameter_if_not_set");
496-
node->create_parameter("foo");
497-
node->create_parameter("bar");
496+
node->declare_parameter("foo");
497+
node->declare_parameter("bar");
498498

499499
auto set_parameters_results = node->set_parameters({
500500
Parameter("foo", 2),

0 commit comments

Comments
 (0)