refactor(pathfinder): replace info_summary_append fixture with Python logging

Replace the custom `info_summary_append` pytest fixture and terminal summary
with standard Python `logging`. Test diagnostic info now writes to a per-
strictness log file (`test-info-summary-{strictness}.log`) via a
`logging.FileHandler`, uploaded as a GHA run artifact.

- Drop `custom_info` list, `pytest_terminal_summary`, and the
  `info_summary_append` fixture from conftest
- Add session-scoped `_info_summary_handler` yield fixture (FileHandler
  lifecycle) and function-scoped `info_log` fixture (LoggerAdapter with
  test node name)
- Update test call sites to use `info_log.info(...)` instead of
  `info_summary_append(...)`
- Remove `tee` + `grep` verification from `ci/tools/run-tests`
- Add `upload-artifact` step to both test-wheel workflows with
  `if-no-files-found: error`

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: cpcloud

.github/workflows/test-wheel-linux.yml

@@ -298,3 +298,11 @@ jobs:
           CUDA_PATHFINDER_TEST_FIND_NVIDIA_HEADERS_STRICTNESS: all_must_work
           CUDA_PATHFINDER_TEST_FIND_NVIDIA_BITCODE_LIB_STRICTNESS: all_must_work
         run: run-tests pathfinder
+
+      - name: Upload cuda.pathfinder test info summary
+        if: ${{ !cancelled() }}
+        uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f  # v6.0.0
+        with:
+          name: cuda-pathfinder-test-info-${{ inputs.host-platform }}-py${{ matrix.PY_VER }}-cuda${{ matrix.CUDA_VER }}-${{ matrix.LOCAL_CTK == '1' && 'local' || 'wheels' }}-${{ matrix.GPU }}
+          path: cuda_pathfinder/test-info-summary-*.log
+          if-no-files-found: error

.github/workflows/test-wheel-windows.yml

@@ -275,3 +275,11 @@ jobs:
           CUDA_PATHFINDER_TEST_FIND_NVIDIA_BITCODE_LIB_STRICTNESS: all_must_work
         shell: bash --noprofile --norc -xeuo pipefail {0}
         run: run-tests pathfinder
+
+      - name: Upload cuda.pathfinder test info summary
+        if: ${{ !cancelled() }}
+        uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f  # v6.0.0
+        with:
+          name: cuda-pathfinder-test-info-${{ inputs.host-platform }}-py${{ matrix.PY_VER }}-cuda${{ matrix.CUDA_VER }}-${{ matrix.LOCAL_CTK == '1' && 'local' || 'wheels' }}-${{ matrix.GPU }}
+          path: cuda_pathfinder/test-info-summary-*.log
+          if-no-files-found: error

ci/tools/run-tests

@@ -33,11 +33,7 @@ if [[ "${test_module}" == "pathfinder" ]]; then
     "LD:${CUDA_PATHFINDER_TEST_LOAD_NVIDIA_DYNAMIC_LIB_STRICTNESS} " \
     "FH:${CUDA_PATHFINDER_TEST_FIND_NVIDIA_HEADERS_STRICTNESS} " \
     "BC:${CUDA_PATHFINDER_TEST_FIND_NVIDIA_BITCODE_LIB_STRICTNESS}"
-  pytest -ra -s -v --durations=0 tests/ |& tee /tmp/pathfinder_test_log.txt
-  # Report the number of "INFO test_" lines (including zero)
-  # to support quick validations based on GHA log archives.
-  line_count=$(awk '/^INFO test_/ {count++} END {print count+0}' /tmp/pathfinder_test_log.txt)
-  echo "Number of \"INFO test_\" lines: $line_count"
+  pytest -ra -s -v --durations=0 tests/
   popd
 elif [[ "${test_module}" == "bindings" ]]; then
   echo "Installing bindings wheel"

cuda_pathfinder/tests/conftest.py

@@ -2,33 +2,42 @@
 # SPDX-License-Identifier: Apache-2.0
 
 
+import logging
+import os
+
 import pytest
 
+_LOGGER_NAME = "cuda_pathfinder.test_info"
 
-def pytest_configure(config):
-    config.custom_info = []
 
+def _info_summary_log_filename():
+    strictness = os.environ.get("CUDA_PATHFINDER_TEST_LOAD_NVIDIA_DYNAMIC_LIB_STRICTNESS", "default")
+    return f"test-info-summary-{strictness}.log"
 
-def pytest_terminal_summary(terminalreporter, exitstatus, config):
-    if not config.getoption("verbose"):
-        return
-    if hasattr(config.option, "iterations"):  # pytest-freethreaded runs all tests at least twice
-        return
-    if getattr(config.option, "count", 1) > 1:  # pytest-repeat
-        return
 
-    if config.custom_info:
-        terminalreporter.write_sep("=", "INFO summary")
-        for msg in config.custom_info:
-            terminalreporter.line(f"INFO {msg}")
+@pytest.fixture(scope="session")
+def _info_summary_handler(request):
+    log_path = request.config.rootpath / _info_summary_log_filename()
+    handler = logging.FileHandler(log_path, mode="w")
+    handler.setFormatter(logging.Formatter("%(test_node)s: %(message)s"))
 
+    logger = logging.getLogger(_LOGGER_NAME)
+    logger.addHandler(handler)
+    logger.setLevel(logging.DEBUG)
+    logger.propagate = False
 
-@pytest.fixture
-def info_summary_append(request):
-    def _append(message):
-        request.config.custom_info.append(f"{request.node.name}: {message}")
+    yield handler
+
+    logger.removeHandler(handler)
+    handler.close()
 
-    return _append
+
+@pytest.fixture
+def info_log(request, _info_summary_handler):  # noqa: ARG001
+    return logging.LoggerAdapter(
+        logging.getLogger(_LOGGER_NAME),
+        extra={"test_node": request.node.name},
+    )
 
 
 def skip_if_missing_libnvcudla_so(libname: str, *, timeout: float) -> None:

cuda_pathfinder/tests/test_find_nvidia_binaries.py

@@ -21,9 +21,9 @@ def test_unknown_utility_name():
 
 
 @pytest.mark.parametrize("utility_name", SUPPORTED_BINARIES)
-def test_find_binary_utilities(info_summary_append, utility_name):
+def test_find_binary_utilities(info_log, utility_name):
     bin_path = find_nvidia_binary_utility(utility_name)
-    info_summary_append(f"{bin_path=!r}")
+    info_log.info(f"{bin_path=!r}")
 
     assert bin_path is None or os.path.isfile(bin_path)
 

cuda_pathfinder/tests/test_find_nvidia_headers.py

@@ -113,12 +113,12 @@ def _fake_cudart_canary_abs_path(ctk_root: Path) -> str:
 
 
 @pytest.mark.parametrize("libname", SUPPORTED_HEADERS_NON_CTK.keys())
-def test_locate_non_ctk_headers(info_summary_append, libname):
+def test_locate_non_ctk_headers(info_log, libname):
     hdr_dir = find_nvidia_header_directory(libname)
     located_hdr_dir = locate_nvidia_header_directory(libname)
     assert hdr_dir is None if not located_hdr_dir else hdr_dir == located_hdr_dir.abs_path
 
-    info_summary_append(f"{hdr_dir=!r}")
+    info_log.info(f"{hdr_dir=!r}")
     if hdr_dir:
         _located_hdr_dir_asserts(located_hdr_dir)
         assert os.path.isdir(hdr_dir)
@@ -147,12 +147,12 @@ def test_supported_headers_site_packages_ctk_consistency():
 
 
 @pytest.mark.parametrize("libname", SUPPORTED_HEADERS_CTK.keys())
-def test_locate_ctk_headers(info_summary_append, libname):
+def test_locate_ctk_headers(info_log, libname):
     hdr_dir = find_nvidia_header_directory(libname)
     located_hdr_dir = locate_nvidia_header_directory(libname)
     assert hdr_dir is None if not located_hdr_dir else hdr_dir == located_hdr_dir.abs_path
 
-    info_summary_append(f"{hdr_dir=!r}")
+    info_log.info(f"{hdr_dir=!r}")
     if hdr_dir:
         _located_hdr_dir_asserts(located_hdr_dir)
         assert os.path.isdir(hdr_dir)

cuda_pathfinder/tests/test_load_nvidia_dynamic_lib.py

@@ -111,7 +111,7 @@ def _is_expected_load_nvidia_dynamic_lib_failure(libname):
     "libname",
     supported_nvidia_libs.SUPPORTED_WINDOWS_DLLS if IS_WINDOWS else supported_nvidia_libs.SUPPORTED_LINUX_SONAMES,
 )
-def test_load_nvidia_dynamic_lib(info_summary_append, libname):
+def test_load_nvidia_dynamic_lib(info_log, libname):
     # Use a fresh Python subprocess for each load to isolate global dynamic
     # loader state and keep the tests aligned with the canary probe model.
     timeout = 120 if IS_WINDOWS else 30
@@ -133,9 +133,9 @@ def raise_child_process_failed():
         skip_if_missing_libnvcudla_so(libname, timeout=timeout)
         if STRICTNESS == "all_must_work" and not _is_expected_load_nvidia_dynamic_lib_failure(libname):
             raise_child_process_failed()
-        info_summary_append(f"Not found: {libname=!r}")
+        info_log.info(f"Not found: {libname=!r}")
     else:
         abs_path = payload.abs_path
         assert abs_path is not None
-        info_summary_append(f"abs_path={quote_for_shell(abs_path)}")
+        info_log.info(f"abs_path={quote_for_shell(abs_path)}")
         assert os.path.isfile(abs_path)  # double-check the abs_path