Skip to content

Refactor: reorganize C++ UT into domain-based subdirectories#642

Merged
ChaoWao merged 1 commit intohw-native-sys:mainfrom
chenshengxin2026:ut-infra
Apr 23, 2026
Merged

Refactor: reorganize C++ UT into domain-based subdirectories#642
ChaoWao merged 1 commit intohw-native-sys:mainfrom
chenshengxin2026:ut-infra

Conversation

@chenshengxin2026
Copy link
Copy Markdown
Contributor

@chenshengxin2026 chenshengxin2026 commented Apr 22, 2026

Summary

  • Move existing C++ tests into domain-based subdirectories: hierarchical/, a2a3/, a5/, types/, hardware/
  • Add shared stubs/test_stubs.cpp for reusable test infrastructure
  • Introduce add_a2a3_runtime_test() CMake helper for runtime-linked tests
  • Remove pto2_ prefix from all CMake function and variable names for consistency
  • No new test coverage — structural reorganization only

New directory layout

tests/ut/cpp/
├── a2a3/              # A2A3 orchestration API tests
├── a5/                # A5 orchestration API tests
├── hardware/          # HCCL comm tests (requires_hardware label)
├── hierarchical/      # Orchestrator, ring, scheduler, scope, tensormap
├── stubs/             # Shared test stubs
├── types/             # Task interface type tests
└── CMakeLists.txt

Test plan

  • C++ UT build succeeds: cmake -B build -S tests/ut/cpp && cmake --build build
  • All 8 C++ tests pass: ctest --test-dir build --output-on-failure
  • Python UT: 208 passed, 2 skipped

@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

ChaoWao
ChaoWao previously approved these changes Apr 23, 2026
Move existing tests into hierarchical/, a2a3/, a5/, types/, hardware/
subdirectories. Add shared stubs/test_stubs.cpp for reusable test
infrastructure. Introduce add_a2a3_runtime_test() CMake helper for
runtime-linked tests. Remove unused test_helpers.h.

No new test coverage — structural reorganization only.
@ChaoWao ChaoWao merged commit 21a2359 into hw-native-sys:main Apr 23, 2026
14 checks passed
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