✅ 清理高置信冗余测试 - #1653
Open
CodFrm wants to merge 9 commits into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Checklist / 检查清单
N/A — 本 PR 不关联待关闭的 issue;人工审查项保留未勾选,等待维护者审查。
Description / 描述
背景
这是全仓库测试价值审计的第一轮独立交付,仅清理已有精确替代覆盖、纯透传、同一等价类重复,或只验证语言/测试框架语义的高置信冗余测试。
本次改动
实现考虑
每项删除都要求先确认生产路径和替代覆盖。独立分支、variant、a11y、生命周期、持久化、协议、浏览器兼容、安全、错误和并发边界均保留;审查发现的误删项已通过后续提交恢复并重新验证。
已知限制
本轮明确不处理以下候选,留给后续独立 PR:
真实浏览器验证覆盖了本分支涉及的全部剩余 E2E 文件;未运行与本分支无关的永久 E2E spec。
建议审查重点
验证
pnpm test:310 个测试文件、3435 个测试通过;pnpm run typecheck:通过;pnpm run lint:通过;pnpm run build:通过(仅现有非阻塞 bundle/dependency warnings);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 通过;Screenshots / 截图
N/A — 本 PR 不修改 UI;真实浏览器验证截图保存在本地忽略的验证报告目录中。