From c1cf011746457c2dace5f5236b486af83030813b Mon Sep 17 00:00:00 2001 From: Scott K Logan Date: Fri, 1 Mar 2024 13:41:57 -0600 Subject: [PATCH] Subtle fix for ament_libraries_deduplicate tests (#516) Evidently 'release' isn't a config keyword, but 'general' is. I also modified the test to assert that the keyword is actually treated like a keyword. Signed-off-by: Scott K Logan --- ament_cmake_libraries/test/test_deduplicate.cmake | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/ament_cmake_libraries/test/test_deduplicate.cmake b/ament_cmake_libraries/test/test_deduplicate.cmake index 23cd4cd1..2e9bb584 100644 --- a/ament_cmake_libraries/test/test_deduplicate.cmake +++ b/ament_cmake_libraries/test/test_deduplicate.cmake @@ -26,6 +26,6 @@ ament_libraries_deduplicate(ACTUAL ${TEST_IN}) assert_equal("debug;foo;debug;bar;debug;baz;bar" "${ACTUAL}") # With mismatched build configs -set(TEST_IN "debug;foo;debug;bar;debug;baz;release;bar") +set(TEST_IN "optimized;foo;optimized;bar;general;baz;general;bar") ament_libraries_deduplicate(ACTUAL ${TEST_IN}) -assert_equal("debug;foo;debug;bar;debug;baz;release;bar" "${ACTUAL}") +assert_equal("optimized;foo;optimized;bar;general;baz;general;bar" "${ACTUAL}")