Skip to content

Commit

Permalink
GitHub Actions: Remove usage of CTest
Browse files Browse the repository at this point in the history
It seems that new versions of CTest now require case-sensitive arguments,
resulting in an "unknown argument" error for "-vv", while documentation
shows the argument for the full verbosity option as "-VV".

In CMakeLists.txt, there is currently only 1 test registered to CMake
with add_test(), and that is to run the ninja_test binary.

Since using CTest is not reducing the number of commands used for testing,
fix the testing invocation by replacing CTest with a direct run of the test.

Signed-off-by: Michael Pratt <[email protected]>
  • Loading branch information
mcprat committed Aug 25, 2024
1 parent 27698d1 commit a2df75c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ jobs:
cmake --build build/Release
- name: Test ninja
run: ctest -vv
run: ./ninja_test
working-directory: build/Release

- name: Create ninja archive
Expand Down

0 comments on commit a2df75c

Please sign in to comment.