Skip to content

Commit 047f7fa

Browse files
authored
Disable Fast-RTPS security tests until they work on Focal. (#413)
eProsima/Fast-DDS#1087 is the issue that needs to be resolved. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
1 parent ce0a94d commit 047f7fa

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

test_security/CMakeLists.txt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -283,9 +283,12 @@ if(BUILD_TESTING)
283283
# TODO(mikaelarguedas) only Connext and FastRTPS support DDS-Security for now
284284
if(
285285
rmw_implementation STREQUAL "rmw_connext_cpp" OR
286-
rmw_implementation STREQUAL "rmw_connext_dynamic_cpp" OR
287-
rmw_implementation STREQUAL "rmw_fastrtps_cpp" OR
288-
rmw_implementation STREQUAL "rmw_fastrtps_dynamic_cpp"
286+
rmw_implementation STREQUAL "rmw_connext_dynamic_cpp"
287+
# TODO(clalancette): As of 2020-04-09, Fast-RTPS doesn't support OpenSSL 1.1.1{d,e,f}.
288+
# Since this is the version used in all of our target platforms, security tests can't work.
289+
# Disable the tests until https://github.com/eProsima/Fast-RTPS/issues/1087 is resolved.
290+
# rmw_implementation STREQUAL "rmw_fastrtps_cpp" OR
291+
# rmw_implementation STREQUAL "rmw_fastrtps_dynamic_cpp"
289292
)
290293
custom_security_test_c(test_security_nodes_c
291294
"test/test_invalid_secure_node_creation_c.cpp")

0 commit comments

Comments
 (0)