Skip to content

test: align test names with llama.cpp conventions - #78

Merged
zhenghaoz merged 1 commit into
mainfrom
refactor/test-file-names
Nov 9, 2026
Merged

zhenghaoz merged 1 commit into
mainfrom
refactor/test-file-names

Conversation

@zhangzhenghao

Copy link
Copy Markdown
Collaborator

Summary

  • Align the C++ test naming convention with llama.cpp: rename all 24 test source files and corresponding executable/helper-library targets to test-*.
  • Rename all CTest cases to test-*, including parametrized cases, and update their dependency references.
  • Update Python callers, usage messages, and documentation for the renamed binaries. Keep Python module/script names, helper headers, fixtures, and test behavior unchanged.

Verification

  • Linux Release static build with system ICU: 35/35 CTest cases passed.
  • Linux Release shared build with system ICU: 34/34 CTest cases passed (the allocation test is static-only).
  • Verified that every registered case in both builds has a unique test-* name.
  • Parsed all 66 Python test/helper scripts, checked CMake source/script paths, and searched for stale source/target references.
  • Independent rename-focused review: no findings.
  • git diff --check passed.

Optional model-parity suites and GPU backends were not executed locally.

@zhenghaoz
zhenghaoz merged commit ada8fb1 into main Nov 9, 2026
9 checks passed
@zhenghaoz
zhenghaoz deleted the refactor/test-file-names branch November 9, 2026 00:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants