From a71f438e28e832c8b8fa8767627d92a7e37b9e7e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Florian=20Omn=C3=A8s?= <florian.omnes@rte-france.com>
Date: Mon, 6 Jan 2025 10:31:39 +0100
Subject: [PATCH] Improve `add_boost_test` CMake function (#2552)

- Start parsing argument from the 2nd one, ignore the 1st argument
`${ARGV0}` since it's reserved to the test name
- Remove the obsolete linking to `stdc++fs` (no longer necessary with
modern compilers/linkers)
---
 src/tests/macros.cmake | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/src/tests/macros.cmake b/src/tests/macros.cmake
index 8d57ddf55b..d68c7b4f2e 100644
--- a/src/tests/macros.cmake
+++ b/src/tests/macros.cmake
@@ -9,7 +9,7 @@ function(add_boost_test)
     set(options "")
     set(oneValueArgs)
     set(multiValueArgs SRC LIBS INCLUDE)
-    cmake_parse_arguments(PARSE_ARGV 0 arg
+    cmake_parse_arguments(PARSE_ARGV 1 arg
         "${options}" "${oneValueArgs}" "${multiValueArgs}")
     # Bypass cmake_parse_arguments for the 1st argument
     set(TEST_NAME ${ARGV0})
@@ -29,10 +29,4 @@ function(add_boost_test)
 
     # Give the IDE some directions to display tests in a "Unit-tests" folder
     set_target_properties(${TEST_NAME} PROPERTIES FOLDER Unit-tests)
-
-    # Linux only. TODO remove ?
-    if(UNIX AND NOT APPLE)
-      target_link_libraries(${TEST_NAME} PRIVATE stdc++fs)
-    endif()
-
 endfunction()