Use ctest to run t_algorithms_test in CI workflow

[Copilot]

The t-algorithms-test workflow job invoked the algorithms_test binary directly, bypassing the test definition set in CMake (including LLVM_PROFILE_FILE and any other future test properties).

Change

Instead of downloading just the test binary and running it directly, the build job now produces a slim build archive (containing the CTestTestfile.cmake hierarchy and the test binary) for the clang++ Release ASAN configuration. The t-algorithms-test job downloads this archive and runs the test via ctest, exactly as defined in src/tests/algorithms_test/CMakeLists.txt:

add_test(NAME t_algorithms_test COMMAND env LLVM_PROFILE_FILE=algorithms_test.profraw $<TARGET_FILE:algorithms_test>)

The workflow step now runs:

ctest --test-dir build -V -R "^t_algorithms_test$"

This ensures the job automatically stays in sync with any future changes to the CMake test definition (new environment variables, working directory, test properties, etc.) without requiring manual updates to the workflow.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[wdconinc]

@copilot Are there benefits in providing the full build artifact and use the same ctest calls as in the original build test to ensure we keep in sync with any other changes in the build environment?

[Copilot]

@copilot Are there benefits in providing the full build artifact and use the same ctest calls as in the original build test to ensure we keep in sync with any other changes in the bu...

Yes — using ctest ensures the job automatically picks up any future changes to the test definition (e.g., new env vars, working directory, test properties) without needing manual workflow updates.

Implemented in commit 3106091: replaced the binary-only artifact upload with a slim archive containing just the CTestTestfile.cmake hierarchy + binary, conditioned specifically on clang++ Release ASAN. The t-algorithms-test job now downloads that archive and runs ctest --test-dir build -V -R "^t_algorithms_test$", mirroring the main build job's ctest invocation exactly.

[Copilot]

Pull request overview

Updates the CI workflow so the t-algorithms-test job runs t_algorithms_test via ctest (as defined in CMake), rather than invoking the test binary directly—ensuring test properties like LLVM_PROFILE_FILE are honored and future test-definition changes don’t require workflow edits.

Changes:

• Build job now creates and uploads a slim artifact containing the CTestTestfile.cmake hierarchy plus the algorithms_test binary for the clang++ Release ASAN configuration.
• t-algorithms-test job downloads that artifact and runs the test via ctest --test-dir build -V -R "^t_algorithms_test$".

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Capybara summary for PR 2558

• rec_dis_10x100_minQ2=0_craterlake_10x100_MT
• rec_dis_10x100_minQ2=1000_craterlake_tracking_only
• rec_dis_18x275_minQ2=0_craterlake_18x275
• rec_dis_18x275_minQ2=0_craterlake_18x275_MT
• rec_dis_18x275_minQ2=0_craterlake_18x275_ST_MT
• rec_dis_18x275_minQ2=1000_craterlake_18x275
• rec_dis_18x275_minQ2=1000_craterlake_18x275_MT
• rec_dis_18x275_minQ2=1000_craterlake_18x275_ST_MT
• rec_dis_5x41_minQ2=0_craterlake_5x41
• rec_e_1GeV_20GeV_craterlake
• rec_pi_1GeV_20GeV_craterlake
• two_stage_raw_e_1GeV_20GeV_craterlake
• two_stage_rec_e_1GeV_20GeV_craterlake

Last updated 2026-03-11T18:14-04:00 3106091