Skip to content

Commit

Permalink
Don't pass arguments in cmake target test-perl-testsuite
Browse files Browse the repository at this point in the history
Passing arguments to tools/invoke-tests overrides TESTS
and makes it impossible to run individual tests as we
document in the README.md, for example:

    make test-perl-testsuite TESTS="t/23-baseclass.t"

Without this change all tests from t are always executed.
  • Loading branch information
kalikiana committed Jun 26, 2024
1 parent b55275c commit eb390c7
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cmake/test-targets.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -93,10 +93,10 @@ endif ()
find_program(PROVE_PATH prove)
find_program(UNBUFFER_PATH unbuffer)
if (PROVE_PATH)
set(INVOKE_TEST_ARGS --prove-tool "${PROVE_PATH}" --make-tool "${CMAKE_MAKE_PROGRAM}" --unbuffer-tool "${UNBUFFER_PATH}" --build-directory "${CMAKE_CURRENT_BINARY_DIR}" t)
set(INVOKE_TEST_ARGS --prove-tool "${PROVE_PATH}" --make-tool "${CMAKE_MAKE_PROGRAM}" --unbuffer-tool "${UNBUFFER_PATH}" --build-directory "${CMAKE_CURRENT_BINARY_DIR}")
add_test(
NAME test-perl-testsuite
COMMAND "${CMAKE_CURRENT_SOURCE_DIR}/tools/invoke-tests" ${INVOKE_TEST_ARGS}
COMMAND "${CMAKE_CURRENT_SOURCE_DIR}/tools/invoke-tests" ${INVOKE_TEST_ARGS} TESTS=t
WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
)
add_test(
Expand Down

0 comments on commit eb390c7

Please sign in to comment.