-
Notifications
You must be signed in to change notification settings - Fork 0
Follow-up: a7 test / skills / docs review findings #1
Copy link
Copy link
Open
Description
背景
在准备实际投入使用前,对 a7 当前测试体系、skills、文档做了一轮完整审查与本地复跑。
飞书详细报告:
当前结论
- CI 基础测试可通过
- 严格按当前 CI 的 E2E workflow 口径,本地复跑可以通过
- 但该“通过”包含大量
skip - 当补充
A7_GATEWAY_URL=http://192.168.10.100:19080与HTTPBIN_URL=http://192.168.10.100:3030后,原先很多被跳过的关键流量 case 暴露出真实问题 - 当前
skills没有 dedicated e2e,也没有validate-skills - 当前总代码覆盖率约为
41.4%
主要问题
1. route / debug / stream-route / secret 测试模型和当前 EE API 不匹配
当前已确认的问题包括:
route create相关测试仍按旧模型构造请求,缺少当前 EE 所要求的service_iddebug trace相关测试仍使用旧 route 模型,导致真实失败stream-route相关测试与当前 schema 要求不一致secret的文档和测试使用的参数形式与当前 CLI 不匹配
2. skills 自动化缺失
- 没有
scripts/validate-skills.sh - 没有
make validate-skills - 没有
test/e2e/skills
3. skills 文档和当前实现/环境不一致
已确认的问题包括:
docs/skills.md中声明了仓库里并不存在的 skill 名称- persona 文档中引用了当前 CLI 不存在的命令,如
health、portal - 多个 recipe 文档仍围绕旧的 route/upstream 模型编写
upstream create在当前 EE 环境中直接返回resource not found
需要继续跟进的事项
P0
-
重写 route 相关 e2e
- 采用
service create + route create(service_id)的当前可行模型
- 采用
-
重写 debug trace e2e
- 基于当前真实可用的 route/service 模型
-
重写 stream-route e2e
- 与当前 schema 对齐
-
重写 secret e2e
- 与当前 CLI 参数形式对齐
-
重写 auth 类场景验证
- 统一改用
credential create的当前可用模型
- 统一改用
-
为
skills增加自动化validate-skillstest/e2e/skills
P1
-
系统修正文档中的旧模型示例
- route
- upstream
- debug
- auth
- recipe
-
修正
docs/skills.md中不存在的 skill 名称和 roadmap 项 -
对现有 skills 做一轮“当前环境可落地性”清洗
- 区分:可用、需改文档、依赖外部系统、当前不可用
P2
- 继续补 unit test,逐步拉升覆盖率
- 将关键行为 case 从“可人工验证”固化为自动化测试
建议产出
-
先完成一轮“模型纠偏”
- 先把 route / credential / debug / stream-route / secret 测试模型和文档统一到当前 EE API
-
再补 skills 自动化
- 不要在旧模型基础上直接堆自动化
-
最终形成:
- 可维护的 e2e
- 可执行的 skill 验证
- 与当前实现一致的文档
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels