An extra setup Escape can close Home in the tagged TUI test
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 新手友好度
- 76/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- go
- 领域
- cli, testing-qa
调研方向
Start in internal/e2e/tmux_test.go at skipSetup around line 247, then read the setupIsUp predicate around line 273 and the tagged TestHomeRestoredNavigationNoModel/180 subtest. Run the stated e2e test and trace the helper's scripted transitions. Done means the delayed transition repeatedly leaves the test on Home, the unit test proves no Escape is sent after Home appears, and the timing assumption is recorded in invalidates.
由索引模型根据 Issue 内容生成。
描述
Seen on dev@debffabf6. Surfaced while verifying #1389; not introduced by it.
What happened
TestHomeRestoredNavigationNoModel/180 failed once under load when skipSetup sent another Escape after setup had already disappeared. That Escape landed on Home and closed it. The helper checks the screen after each press but waits 900 ms before checking; a transition within that interval can put the next key on a different page. This is one observed flake, not a measured failure rate.
Replication
Deterministic (no model). In the tagged e2e rig, script setup to disappear immediately after an Escape while the next captured frame still contains setup text briefly. Call skipSetup; its loop can issue another Escape, which Home interprets as close. Assert the final page is Home. The real subtest is go test -tags e2e -run 'TestHomeRestoredNavigationNoModel/180' ./internal/e2e/.
Where
internal/e2e/tmux_test.go:247 at dev@debffabf6, skipSetup, sends up to six Escapes separated by a fixed sleep; its screen predicate is setupIsUp at :273.
The fix
Wait for a confirmed transition to the target page after each setup action, and stop sending Escape once that transition is observed.
Acceptance
- e2e: the 180-column tagged subtest repeatedly ends on Home under a delayed setup transition.
- Unit: a scripted transition test proves the helper sends no Escape after Home appears.
- Record the former timing assumption in the test change entry's
invalidates.
- 主要语言
- Go
- 星标
- 115
- 派生
- 14
- 平均合并
- 9 小时 37 分钟
- 30 天内合并 PR
- 755
环境准备
我们还没有检查这个项目的环境配置文件。先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Agent-Field/CodeAF 的其他 Issue
-
area:chat bug sev:papercut
难度 2/5 1-3 小时 新手友好度 86/100
Agent-Field/CodeAF#1592 ·
维护者通常 1 天内回复
-
area:headless bug sev:critical
难度 2/5 1-3 小时 新手友好度 88/100
Agent-Field/CodeAF#1566 · 已指派 1 人 ·
维护者通常 1 天内回复
-
area:chat feature
难度 2/5 1-3 小时 新手友好度 88/100
Agent-Field/CodeAF#1510 ·
维护者通常 1 天内回复
-
area:tests bug
难度 2/5 1-3 小时 新手友好度 82/100
Agent-Field/CodeAF#1489 ·
维护者通常 1 天内回复
-
area:chat bug good first issue sev:papercut
难度 2/5 1-3 小时 新手友好度 78/100
Agent-Field/CodeAF#1470 ·
维护者通常 1 天内回复
查看 Agent-Field/CodeAF 的全部 Issue
相似的 Issue
-
agentic-workflows
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
priority/4/normal status/needs-triage type/bug/unconfirmed
难度 2/5 1-3 小时 新手友好度 78/100
authelia/authelia#13292 · 1 条评论 ·
维护者通常 1 天内回复
-
bug
难度 2/5 1-3 小时 新手友好度 68/100
blinklabs-io/actions#138 ·
维护者通常 1 天内回复
-
[UI] AlbumDetails collapses multi-genre list to single primary genre on viewports < lg breakpoint未关闭
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复