Skip to content

Commit

Permalink
Pass validators to CMake file
Browse files Browse the repository at this point in the history
  • Loading branch information
christophfroehlich committed Dec 1, 2024
1 parent 9e702a4 commit e68482c
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
1 change: 1 addition & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ ament_target_dependencies(low_pass_filter PUBLIC ${CONTROL_FILTERS_INCLUDE_DEPEN
generate_parameter_library(
rate_limiter_parameters
src/control_filters/rate_limiter_parameters.yaml
include/control_filters/custom_validators.hpp
)

add_library(rate_limiter SHARED
Expand Down
1 change: 0 additions & 1 deletion include/control_filters/rate_limiter.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@
#include "filters/filter_base.hpp"

#include "control_toolbox/rate_limiter.hpp"
#include "custom_validators.hpp"
#include "rate_limiter_parameters.hpp"

namespace control_filters
Expand Down
4 changes: 2 additions & 2 deletions test/control_filters/test_rate_limiter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ TEST_F(RateLimiterTest, TestRateLimiterMissingParameter)
std::shared_ptr<filters::FilterBase<double>> filter_ =
std::make_shared<control_filters::RateLimiter<double>>();

// should deny configuration as sampling frequency is missing
// should deny configuration as sampling_interval is missing
ASSERT_FALSE(filter_->configure("", "TestRateLimiter",
node_->get_node_logging_interface(), node_->get_node_parameters_interface()));
}
Expand All @@ -46,7 +46,7 @@ TEST_F(RateLimiterTest, TestRateLimiterInvalidThenFixedParameter)
std::shared_ptr<filters::FilterBase<double>> filter_ =
std::make_shared<control_filters::RateLimiter<double>>();

// should deny configuration as sampling frequency is invalid
// should deny configuration as sampling_interval is invalid
ASSERT_FALSE(filter_->configure("", "TestRateLimiter",
node_->get_node_logging_interface(), node_->get_node_parameters_interface()));

Expand Down

0 comments on commit e68482c

Please sign in to comment.