Skip to content

Commit

Permalink
Disable Connext security tests on Windows (#433)
Browse files Browse the repository at this point in the history
* Disable Connext security tests on Windows

Connext needs a different version of OpenSSL (1.0.2n) than the system
version.
Disabling these tests until we can figure out how to run them on CI.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>

* Use parentheses

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
  • Loading branch information
jacobperron authored Jun 1, 2020
1 parent b9bebaf commit bcebc32
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions test_security/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -269,9 +269,10 @@ if(BUILD_TESTING)
endif()

# TODO(mikaelarguedas) only Connext and FastRTPS support DDS-Security for now
# TODO(jacobperron) Disable Connext on Windows until we fix issue with CI
if(
rmw_implementation STREQUAL "rmw_connext_cpp" OR
rmw_implementation STREQUAL "rmw_connext_dynamic_cpp" OR
(rmw_implementation STREQUAL "rmw_connext_cpp" AND NOT WIN32) OR
(rmw_implementation STREQUAL "rmw_connext_dynamic_cpp" AND NOT WIN32) OR
rmw_implementation STREQUAL "rmw_fastrtps_cpp" OR
rmw_implementation STREQUAL "rmw_fastrtps_dynamic_cpp"
)
Expand Down

0 comments on commit bcebc32

Please sign in to comment.