Skip to content

test(completion): cover top and sidebar workspace Tab routing - #496

Merged
Kuddev merged 1 commit into
mainfrom
test/completion-workspace-layouts-20261006
Oct 6, 2026
Merged

Kuddev merged 1 commit into
mainfrom
test/completion-workspace-layouts-20261006

Conversation

@Kuddev

@Kuddev Kuddev commented Oct 6, 2026

Copy link
Copy Markdown
Owner

验证目标

Refs #358. 用户提出“顶部标签模式不补全,是否走了另一条路径”。现有原生验收直接挂载 TerminalView 并调用输入处理函数,没有覆盖完整工作区的标签激活、焦点恢复和窗口级 Tab 分发。

本 PR 做了什么

  • 在同一工作区、同一终端实体上走 Sidebar → Top → Sidebar。
  • 使用真实工作区和标签控件;顶部模式从设置页通过点击终端标签返回,侧边模式走已有返回动作后点击标签。
  • 在每种布局中保存行内/混合模式,并由 SettingsPane 的 Changed 事件经真实订阅链热应用到已存在的终端,而不是直接改终端模式。
  • 经窗口分发 Tab/ESC,核对终端焦点、输入快照、候选数量、弹窗几何,以及没有把 Tab 当作普通输入写给模拟 PTY。
  • 复用既有终端夹具,新增的访问方法只在测试中编译;不新增生产补全路径。

隔离

本用例会触发真实设置持久化,因此同时持有已有设置锁和字节恢复守卫,并将它的完整测试名称加入已有 nextest 设置串行组。对应配置合同同步更新;没有放宽超时、断言、预算或并行规则的范围。

已执行验证

  • 工作区布局/焦点/按键回归通过,补充设置热应用后仍通过。
  • 5 项编辑器状态回归和 2 项既有历史/混合模式回归通过。
  • 18 项 CI 调度/配置合同测试通过。
  • 格式、架构、平台条件计数与差异检查通过。
  • 本地所有执行均显式使用临时配置目录。

结论边界

这是一项 GPUI 工作区级回归,PTY 采用受控夹具,不冒充 Windows 原生鼠标命中或报告者配置的现场复现。真实 PowerShell/PSReadLine 的 19 个独立原生场景已在 #495 记录。当前矩阵没有复现顶部独有失败,#358 保持打开;#444 仍需按当前主线证明和修正剩余场景。本 PR 不声称新修复了产品补全行为。

@Kuddev Kuddev left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

复核精确 head e888d17:本 PR 仅增加测试覆盖及现有测试串行组的精确条目,不改变生产补全逻辑、依赖、超时或门禁预算。完整工作区中覆盖侧栏→顶部→侧栏、从设置页返回终端、已有终端热应用补全模式,以及窗口派发 Tab/ESC;复用既有终端夹具并校验焦点、候选和弹窗几何。五个平台实际原生测试与十项必需检查均通过。

验收边界:工作区用模拟 PTY,真实 PowerShell 验收则在独立终端界面进行;两者不等于报告者环境的完整复现。#358 与 #444 保持打开。作者与审阅账号相同,记录 COMMENT,不作自我 APPROVE。

@Kuddev
Kuddev merged commit c1d5e56 into main Oct 6, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M 101–500 行

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant