Skip to content

Commit

Permalink
Revert "Disable Fast-RTPS security tests until they work on Focal. (#413
Browse files Browse the repository at this point in the history
)"

This reverts commit 047f7fa.
  • Loading branch information
nuclearsandwich authored and mikaelarguedas committed Apr 22, 2020
1 parent 94a1274 commit 1b5958d
Showing 1 changed file with 3 additions and 6 deletions.
9 changes: 3 additions & 6 deletions test_security/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -283,12 +283,9 @@ if(BUILD_TESTING)
# TODO(mikaelarguedas) only Connext and FastRTPS support DDS-Security for now
if(
rmw_implementation STREQUAL "rmw_connext_cpp" OR
rmw_implementation STREQUAL "rmw_connext_dynamic_cpp"
# TODO(clalancette): As of 2020-04-09, Fast-RTPS doesn't support OpenSSL 1.1.1{d,e,f}.
# Since this is the version used in all of our target platforms, security tests can't work.
# Disable the tests until https://github.com/eProsima/Fast-RTPS/issues/1087 is resolved.
# rmw_implementation STREQUAL "rmw_fastrtps_cpp" OR
# rmw_implementation STREQUAL "rmw_fastrtps_dynamic_cpp"
rmw_implementation STREQUAL "rmw_connext_dynamic_cpp" OR
rmw_implementation STREQUAL "rmw_fastrtps_cpp" OR
rmw_implementation STREQUAL "rmw_fastrtps_dynamic_cpp"
)
custom_security_test_c(test_security_nodes_c
"test/test_invalid_secure_node_creation_c.cpp")
Expand Down

0 comments on commit 1b5958d

Please sign in to comment.