* increase test timeout necessary for Connext
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
* revert changes overlapping with another PR
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
* Add service and client benchmarks
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Style
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Uncrustify
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Set CMakeLists to only use default rmw for benchmarks
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Add comment
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Refactor test CMakeLists in prep for benchmarks (#1422)
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Address #1423 by moving rosidl_generate_interfaces call (#1424)
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Remove rolling-only tests
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Increase test coverage of rclcpp_lifecycle to 96%
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* PR Fixup
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* test_depend
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* rcutils test_depend
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* More windows warnings
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Increase coverage rclcpp_action to 95%
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* PR fixup
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Address PR Feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Rebase onto #1311
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* rcutils test depend
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Cleaning up
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Clear members for StaticExecutorEntitiesCollector to avoid shared_ptr dependency
Signed-off-by: Chen Lihui <Lihui.Chen@sony.com>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Increase test timeouts of slow running tests with rmw_connext_cpp
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Fix other issues with connext
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Add unit tests for qos and qos_event files
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* PR Feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Address PR Feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Fix windows CI
Signed-off-by: Stephen Brawner <brawner@gmail.com>
In particular, add API coverage for spin_node_until_future_complete,
spin_until_future_complete, and spin_node_once.
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Add test for ParameterService
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Address PR feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Change value used as max representation
* Add coverage tests time
* Add call to detach clock
* Add tests time
* Add duration construction tests
* Add const qualifier to constants
* Add check clock stays the same
* Make operator RCLCPP_PUBLIC
* Add tests exceptions duration
* Fix division by 0 on windows
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
* Test the remaining node public API
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Address PR feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Add comment
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Complete coverage of Parameter and ParameterValue API
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Adding comments
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Add tests for node_options API
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Remove c-style casts
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Improve the error messages in the Executor class.
In particular, make sure to use 'throw_from_rcl_error'
as much as possible.
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Allow mimick patching of methods with up to 9 arguments.
This will be needed by the executor tests.
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Add in unit tests for the Executor class.
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Adjust test_executor for foxy
Signed-off-by: Stephen Brawner <brawner@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
* Add coverage for client API
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* PR feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* PR Feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Remove deprecated executor::FutureReturnCode APIs. (#1327)
While we are here, add in another test for the stream operator for future_return_code.cpp
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Revert removing deprecated API
Signed-off-by: Stephen Brawner <brawner@gmail.com>
Co-authored-by: Chris Lalancette <clalancette@openrobotics.org>
* Increase coverage of publisher/subscription API
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* PR Feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Add coverage for missing API (except executors
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* PR Fixup
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Do not check state
Signed-off-by: Stephen Brawner <brawner@gmail.com>
