Skip to content

refactor(tests): consolidate coverage and define test ownership - #192

Closed
chamsechan wants to merge 1 commit into
mainfrom
refactor/test-responsibilities
Closed

chamsechan wants to merge 1 commit into
mainfrom
refactor/test-responsibilities

Conversation

@chamsechan

@chamsechan chamsechan commented Oct 11, 2026 •

Copy link
Copy Markdown
Owner

变更

按测试职责合并重复覆盖,并删除被替代的历史测试,避免新旧两套长期并存。保留用例承接原有独立断言、批次边界、来源关系、失败回滚和生命周期顺序。

  • 完整业务请求/响应集中到八类 Operator 业务测试;删除重复 API 冒烟和跨业务混合集合,运行机制留在 runtime 测试。
  • ASR/OCR/Embedding/LLM 对实际注册 Node 共用类型化契约测试,业务算法、缓存、多端点及其他独有语义留在组件测试。
  • 合并 Converter、Blackboard、schema、批执行等重复场景,删除仅验证假实现自身分支的测试;强化 AdapterHarness 对成功返回超容量计数的检查。
  • 清理旧 CTest 入口及重复 CLI 冒烟矩阵,同步脚手架生成的行为测试入口与测试发现检查。
  • tests/README.md 明确全框架责任边界、何时复用/新增及删除旧测试的条件;组件开发指南链接到该规则。

相对 main 基线 bea7040,tests 下 C++、脚本与 CMake 源码从 54,382 行降至 52,298 行,净减少 2,084 行(3.83%;包含新增文件,不含文档及测试资产)。

验证

  • 本地统一门禁:92/92 CTest 通过;交付脚本在提交前再次运行该门禁。
  • 相关 ASAN/UBSAN 检查:35/35 CTest 通过。
  • 四个共享 runner 的 886 个 GoogleTest 用例均恰好匹配一个 CTest 入口,无空过滤器;加上独立 runner 共 904 个 GoogleTest 用例,无 DISABLED 用例。
  • 只读审查确认旧测试的独有断言已承接。

默认环境中确认有 9 个 GoogleTest 内部跳过,原因是 Kite SDK/模型配置或 Demo 未启用、未设置真实 GGUF 模型、whisper.cpp 未启用。CTest 通过不代表这些真实后端或目标硬件已验收。

@chamsechan chamsechan closed this Oct 14, 2026
@chamsechan
chamsechan deleted the refactor/test-responsibilities branch October 14, 2026 06:55
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.

1 participant