chore(profiling): implement native test install subdirs (PROF-14200)

[vlad-scherbich]

← Gab's #17849 | Next → #17624

Description

Adds an optional INSTALL_SUBDIR keyword argument to the dd_wrapper_add_test CMake
function in ddtrace/internal/datadog/profiling/stack/test/CMakeLists.txt.

When specified, the test binary is installed to test/<subdir>/ instead of the shared
test/ directory. Without the keyword, behavior is identical to before.

Motivation: build_base_venvs in GitLab CI runs as a parallel matrix across all Python
versions, and GitLab merges all job artifacts into a single directory for downstream jobs.
A version-specific test binary (e.g. one compiled against libpython3.15) placed in the
shared test/ directory will be picked up by the pytest gtest plugin in a py3.10 test
environment and crash immediately — reported as a silent "C++ failure" with no test output.

INSTALL_SUBDIR py315 (used in the next PR in the stack) isolates py3.15-only test
binaries so this collision cannot happen.

Also adds cmakeformat_fix script to hatch.toml (needed by the 07-run-cmake-format
pre-commit hook) and updates hooks/README.md to document the in-place formatting +
re-stage behavior.

Part of the Python 3.15 support stack. Relates to #17809.

Testing

• No behavior change for existing tests (test_thread_span_links, test_cpython_layout_contracts) — they call dd_wrapper_add_test without INSTALL_SUBDIR and install to the same test/ path as before.
• The INSTALL_SUBDIR keyword is exercised in the next PR (vlad/ddtracepy-315-profiling-only), where test_frame_state_315 uses INSTALL_SUBDIR py315.

Risks

None. The CMake function change is backward-compatible: INSTALL_SUBDIR is optional and the default path is unchanged.

Additional Notes

• cmake_parse_arguments separates the keyword from source file arguments via _ARG_UNPARSED_ARGUMENTS, so adding the keyword does not affect how source files are passed to add_executable.
• This pattern can be extended to dd_wrapper/test/CMakeLists.txt if that test directory ever needs similar isolation.

[cit-pr-commenter-54b7da]

Codeowners resolved as

ddtrace/internal/datadog/profiling/stack/test/CMakeLists.txt            @DataDog/profiling-python

[datadog-official]

 

✨ Fix all issues with BitsAI

⚠️ Warnings

🚦 4 Pipeline jobs failed

Changelog | Validate changelog     

DataDog/apm-reliability/dd-trace-py | build_base_venvs: [3.15]     

DataDog/apm-reliability/dd-trace-py | check_requirements_lockfiles     

View all 4 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

Useful? React with 👍 / 👎

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 3f47516 | Docs | Datadog PR Page | Give us feedback!}

[github-actions]

This pull request has been automatically closed after a period of inactivity.
After this much time, it will likely be easier to open a new pull request with the
same changes than to update this one from the base branch. Please comment or reopen
if you think this pull request was closed in error.