ros2 / rmw_implementation

CMake infrastructure and dependencies for rmw implementations
Apache License 2.0
21 stars 48 forks source link

Add RMW_DURATION_INFINITE basic compliance test #194

Closed emersonknapp closed 3 years ago

emersonknapp commented 3 years ago

Followup to https://github.com/ros2/rmw/pull/301 - adding a compliance test so that RMW implementations are guaranteed to handle these values properly.

Tests pass locally against all default checked-out RMW implementations:

For context, https://github.com/ros2/rosbag2/issues/756 was an uncaught case of this being a problem. It led to ros2/rmw_connextdds#43, where @jacobperron recommended I create this test.

For a failing case, I reverted ros2/rmw_connextdds#43 in a branch, and rant this test against that version to see a failure.

emersonknapp commented 3 years ago