Skip to content

Commit d944a84

Browse files
author
bpwilcox
committed
remove check for parameters after reset in obstacle_tests
1 parent 1a0b962 commit d944a84

File tree

1 file changed

+1
-11
lines changed

1 file changed

+1
-11
lines changed

nav2_costmap_2d/test/integration/obstacle_tests.cpp

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -357,20 +357,10 @@ TEST_F(TestNode, testRepeatedResets) {
357357
return plugin->hasParameter(layer_param);
358358
}));
359359

360-
// Reset all layers. This will un-declare all params and might re-declare internal ones
360+
// Reset all layers. Parameters should be declared if not declared, otherwise skipped.
361361
// Should run without throwing exceptions
362362
ASSERT_NO_THROW(
363363
for_each(begin(*plugins), end(*plugins), [](const auto & plugin) {
364364
plugin->reset();
365365
}));
366-
367-
// Check for node-level param
368-
ASSERT_TRUE(node_->has_parameter(node_dummy.first));
369-
370-
// Layer-level parameters shouldn't be found
371-
ASSERT_TRUE(
372-
none_of(begin(*plugins), end(*plugins), [&layer_dummy](const auto & plugin) {
373-
string layer_param = layer_dummy.first + "_" + plugin->getName();
374-
return plugin->hasParameter(layer_param);
375-
}));
376366
}

0 commit comments

Comments
 (0)