Skip to content

test(flake): 登记 FLK-003/004 并放宽渲染重用车道测试超时至 30s - #2180

Merged
DeliciousBuding merged 1 commit into
masterfrom
chore/cleanup-round-35
Sep 1, 2026
Merged

test(flake): 登记 FLK-003/004 并放宽渲染重用车道测试超时至 30s#2180
DeliciousBuding merged 1 commit into
masterfrom
chore/cleanup-round-35

Conversation

@DeliciousBuding

Copy link
Copy Markdown
Collaborator

背景

两个已知超时 flake(均「全量并行负载下偶发红、单文件重跑稳定绿」):

  • round-31:workbench sidebar.test.tsx › "round-trips task-052…"(Test timed out in 5000ms
  • round-34:desktop App.v4.test.tsx › "enters a clean Desktop demo workbench…"(全量 5054 中唯一失败)

根因同型:vitest 默认单测超时 5s,对渲染重型用例(完整 <App /> + 十余 mock hooks + 多段 waitFor)在全量并行(jsdom 变慢)下预算不足——非功能 flake。

变更

  • docs/governance/known-flaky.md SSOT 纪律登记 FLK-003 / FLK-004(含车道、复现命令、到期复审日、处置记录),根因结论与修复同 PR 落地,状态「已修复(归档)」。
  • app/workbench/vitest.config.tsapp/desktop/vitest.config.tsapp/desktop/vitest.desktop-ci.config.tsapp/desktop/vitest.desktop-tsx-ci.config.ts:显式 testTimeout: 30_000 / hookTimeout: 30_000(对齐 vitest.edge-real-ci.config.ts 先例)。重试预算保持 0 不变(超时不是重试;预算表未触碰)。
  • verify-doc-ssot.py:known-flaky.md 行数上限 120 → 170(登记两条后 161 行;该文件自身维护规则允许到 300 行再拆分归档)。

验证

  • workbench 全量并行(原 flake 车道):1659/1659 全绿(修复前该车道偶发 1 failed)。
  • sidebar.test.tsx 单文件 15/15、App.v4.test.tsx(tsx-ci config)14/14 全绿。
  • verifiers:doc-ssot / ci-gates / i18n-deadkeys / openapi-contract / migration-idempotency + git diff --check 全绿。
  • 纯测试基建改动:无生产代码 / Go / 契约变化;CI 前端车道复跑即最终验证。

备注

  • 到期复审日 2026-09-29:届时若超时放宽后仍偶发红,须查功能根因而非继续放宽。
  • 下一轮候选:ListContacts 在线状态 Redis N+1 → 批量 pipeline(探索 lane 已核实)。

@coderabbitai

coderabbitai Bot commented Sep 1, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 16d8b8f7-6248-40fa-b3bf-d1380c606b4c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@DeliciousBuding
DeliciousBuding merged commit 1a36bb2 into master Sep 1, 2026
45 checks passed
@DeliciousBuding
DeliciousBuding deleted the chore/cleanup-round-35 branch September 1, 2026 14:12
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