Disable Connext security tests on Windows (#433)

* 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 <[email protected]> * Use parentheses Signed-off-by: Jacob Perron <[email protected]>
ros2 · Jun 1, 2020 · bcebc32 · bcebc32
1 parent b9bebaf
commit bcebc32
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/test_security/CMakeLists.txt b/test_security/CMakeLists.txt
@@ -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"
     )