* 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>
* transition start and goal states can be null
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
* correct tests
Signed-off-by: Karsten Knese <karsten@openrobotics.org>
* Cleanup and address issues in parser.c
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
squash! Cleanup and address issues in parser.c
* Increase test coverage of parser
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* PR Fixup
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* PR Fixup
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
Co-authored-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* Increase test coverage and add more safety checks
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* More coverage
Signed-off-by: Stephen Brawner <brawner@gmail.com>
* Concatenating error messages
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* PR Fixup
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* Forgot a
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* PR Fixup
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* Moving var to top
Signed-off-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
Co-authored-by: Stephen Brawner <stephenbrawner@verbsurgical.com>
* warn about unused return value for set_logger_level
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
* revert change of RCL_UNUSED macro
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
Debugging #611 these timeouts were so long that the overall test timeout was being triggered before these internal timeouts were triggered.
1000 retries at 100ms -> 10 seconds each. I cut it down to 1 second for each to establish the connection.
Signed-off-by: Tully Foote <tfoote@osrfoundation.org>
* Add tests for time.c module
* Add test for function rcl_timer_clock
* Fix linter style issues
* Add test for rcl_timer_call
* Add callback test and improve call test
* Add fixture class to reuse timer init/fini code
* Change error code returned to match function being tested
* Add tests for timer_reset function
* Add tests for exchange_callback function
* Add null parameter test for get guard condition
* Add allocator testing tools auxiliar file
* Add tests for failing ini/fini conditions
* Add test for get_period function
* Add test to get_time_since_last_call
* Remove comment related to variable names
* Fix return value on test case for fini clock
* Address peer review comments
* Fix CLang warning, replacing NULL with false
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
* initial draft of the quality declaration
* Reformat to match REP-2004 template
* Fix README.md files accordingly for the rcl package
* Fix minor typos
* Fix minor issues according to peer review
* Address peer review comments
* Improve current state Feature documentation
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
Co-authored-by: William Woodall <william@osrfoundation.org>
* Add tests for the test_validate_enclave_name module
* Add test that is failing
* Add test that is failing
* Fix function used for publishing
* Fix function used for publishing
* Fix serialize subscription test
* Fix serialize subscription test
* Add tests for loan msgs and get_options
* Add tests for loan msgs and get_options
* Add support for taking a sequence of messages (#614)
* Add support for taking a sequence of messages
* Fix uncrustify
* Apply suggestions from code review
* Address reviewer feedback.
* Remove vertical whitespace
* Address reviewer feedback
* Remove blank line
* Remove extra character left when rebasing
* Remove test used for debugging
* Remove repeated tests
* Add test for get_options
* Fix uncrustify errors
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
* Save allocator when setting uninitialized clock
* Add tests for clock types init/fini
* Remove extra calls to set clock allocator
* Remove tests not related to this PR
* Replace test to make it clearer what is tested in the PR
Signed-off-by: Jorge Perez <jjperez@ekumenlabs.com>
* Add support for taking a sequence of messages
Signed-off-by: Michael Carroll <michael@openrobotics.org>
* Fix uncrustify
Signed-off-by: Michael Carroll <michael@openrobotics.org>
* Apply suggestions from code review
Co-Authored-By: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Michael Carroll <michael@openrobotics.org>
* Address reviewer feedback.
Signed-off-by: Michael Carroll <michael@openrobotics.org>
* Remove vertical whitespace
Signed-off-by: Michael Carroll <michael@openrobotics.org>
* Address reviewer feedback
Signed-off-by: Michael Carroll <michael@openrobotics.org>
* Remove blank line
Signed-off-by: Michael Carroll <michael@openrobotics.org>
Co-authored-by: Jacob Perron <jacob@openrobotics.org>
