Skip to content

Commit

Permalink
Skip keyed messages for RMWs other than fastrtps ones.
Browse files Browse the repository at this point in the history
Signed-off-by: Miguel Company <[email protected]>
  • Loading branch information
MiguelCompany committed Apr 3, 2024
1 parent 486acce commit 2eaefd2
Showing 1 changed file with 19 additions and 0 deletions.
19 changes: 19 additions & 0 deletions test_communication/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,25 @@ if(BUILD_TESTING)
set(TEST_MESSAGE_TYPES "")
foreach(message_file ${message_files})
get_filename_component(message_type "${message_file}" NAME_WE)
set(message_has_keys FALSE)
if(
"${message_type}" STREQUAL "KeyedString" OR
"${message_type}" STREQUAL "ComplexNestedKey"
)
set(message_has_keys TRUE)
endif()

# TODO(MiguelCompany): Only fastrtps RMWs have support for keyed messages
if(
message_has_keys AND
(
(NOT rmw_implementation1_is_fastrtps) OR
(NOT rmw_implementation2_is_fastrtps)
)
)
continue()
endif()

# TODO(dirk-thomas) WStrings published by FastRTPS can't be received
# correctly by Connext on macOS
if(
Expand Down

0 comments on commit 2eaefd2

Please sign in to comment.