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 dc98b76 commit ab8f63e
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion cmake/test-targets.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -93,12 +93,14 @@ 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}
WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"
)
SET_TESTS_PROPERTIES(test-perl-testsuite
PROPERTIES ENVIRONMENT "TESTS=t")
add_test(
NAME test-local-author-perl
COMMAND "${PROVE_PATH}" xt
Expand Down

0 comments on commit ab8f63e

Please sign in to comment.