Skip to content

✅ 清理高置信冗余测试 - #1653

Open
CodFrm wants to merge 9 commits into
mainfrom
test/repository-test-cleanup-direct
Open

✅ 清理高置信冗余测试#1653
CodFrm wants to merge 9 commits into
mainfrom
test/repository-test-cleanup-direct

Conversation

@CodFrm

@CodFrm CodFrm commented Aug 4, 2026

Copy link
Copy Markdown
Member

Checklist / 检查清单

  • Fixes mentioned issues / 修复已提及的问题
  • Code reviewed by human / 代码通过人工检查
  • Changes tested / 已完成测试

N/A — 本 PR 不关联待关闭的 issue;人工审查项保留未勾选,等待维护者审查。

Description / 描述

背景

这是全仓库测试价值审计的第一轮独立交付,仅清理已有精确替代覆盖、纯透传、同一等价类重复,或只验证语言/测试框架语义的高置信冗余测试。

本次改动

  • 净删除 61 条 Vitest、3 条 E2E,以及 1 个重复 E2E spec 文件;
  • 共涉及 41 个测试文件,净删除 750 行(6 行新增、756 行删除);
  • 不修改生产代码或生产行为;
  • 在逐任务审查和两轴收尾审查中恢复被识别为唯一回归信号的测试,包括 Service Worker 响应分派、Popup/Agent/Provider/VSCode 浏览器边界、Logger 刷新映射、空内容块、Radix 外部事件和 toast 转发合同。

实现考虑

每项删除都要求先确认生产路径和替代覆盖。独立分支、variant、a11y、生命周期、持久化、协议、浏览器兼容、安全、错误和并发边界均保留;审查发现的误删项已通过后续提交恢复并重新验证。

已知限制

本轮明确不处理以下候选,留给后续独立 PR:

  • 需要改写有效断言的 tautology / mislabeled / 等价类矩阵;
  • 需要迁移测试层级的案例;
  • 需要先增加 lint、AST 或结构化 checker 的源码内容断言。

真实浏览器验证覆盖了本分支涉及的全部剩余 E2E 文件;未运行与本分支无关的永久 E2E spec。

建议审查重点

  • 被删除用例是否确有相同公共边界的替代覆盖;
  • 协议、浏览器和生命周期类唯一回归信号是否均已保留;
  • 本轮是否严格保持“只删除测试、不改生产行为”的范围。

验证

  • pnpm test:310 个测试文件、3435 个测试通过;
  • pnpm run typecheck:通过;
  • pnpm run lint:通过;
  • pnpm run build:通过(仅现有非阻塞 bundle/dependency warnings);
  • 真实构建扩展 scratch Playwright:1/1 通过,观察到 Popup 7 个菜单项及 Agent welcome/model selector;
  • e2e/popup.spec.ts e2e/script-management.spec.ts e2e/agent-chat.spec.ts e2e/agent-provider.spec.ts e2e/vscode-connect.spec.ts:15/15 通过;
  • 最终 spec verification 与 code review:均无发现。

Screenshots / 截图

N/A — 本 PR 不修改 UI;真实浏览器验证截图保存在本地忽略的验证报告目录中。

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