* Add action_server validity check, and remove dead code
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Increase test coverage of rcl_action
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Reorganizing impl structs into private header
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Addressing PR Feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Addressing memory error and PR feedback
Signed-off-by: Stephen Brawner <brawner@gmail.com>
Co-authored-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* add optional rmw payload to rcl options for pub and sub
Signed-off-by: William Woodall <william@osrfoundation.org>
* move ignore_local_publications into rmw options structure for subs
Signed-off-by: William Woodall <william@osrfoundation.org>
* Update action graph tests to account for type namespace
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Enable OpenSplice action graph test
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Exclude some action graph tests with OpenSplice
It appears that getting graph information from finalized nodes succeeds with OpenSplice, unlike the other RMW implementations.
Since we do not have tests covering this case in rcl, it's not clear if this is a bug or expected behaviour.
In the meantime, I've disabled testing this specific case for OpenSplice.
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* const bool
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* implement rcl_wait and rcl_take for rcl_event_t
Signed-off-by: Miaofei <miaofei@amazon.com>
* address feedback regarding formatting issues
Signed-off-by: Miaofei <miaofei@amazon.com>
* Remove dependency on sleep in test_events
Signed-off-by: Ross Desmond <44277324+ross-desmond@users.noreply.github.com>
* update total max wait time for setting up publisher and subscriber to 10 seconds
Signed-off-by: Miaofei <miaofei@amazon.com>
* Fix test_events for rmw_connext
Signed-off-by: Ross Desmond <44277324+ross-desmond@users.noreply.github.com>
* Refactor out timed loop from tests to function
Signed-off-by: Miaofei <miaofei@amazon.com>
* address additional feedback from pull request
Signed-off-by: Miaofei <miaofei@amazon.com>
* update comment regarding difference between connext and opensplice
Signed-off-by: Miaofei <miaofei@amazon.com>
* fix uncrustify issues
Signed-off-by: Miaofei <miaofei@amazon.com>
* update test_events for compatibility with API changes
Signed-off-by: Miaofei <miaofei@amazon.com>
* temporarily disable test_events for macOS
Signed-off-by: Miaofei <miaofei@amazon.com>
* Add return code to CancelGoal service response
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Add case for terminated goals
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Check for ERROR_NONE instead of 0
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
* Proposola of changes for RMW_Preallocate. Related /ros2/rmw#160
Signed-off-by: Gonzalo de Pedro <gonzalo@depedro.com.ar>
* Changed RCL interface
Signed-off-by: Gonzalo de Pedro <gonzalo@depedro.com.ar>
* Updates for allocation in serialize methods.
Signed-off-by: Michael Carroll <michael@openrobotics.org>
* Fix tests for new APIs.
Signed-off-by: Michael Carroll <michael@openrobotics.org>
