test: the Changes button's keycap is its tooltip's shortcut hint (no bead)

Closed
TE
25d ago
+3-1

Summary

webviews/test/chat-header-changes.test.tsx:66 fails on feat-cmux-next (bf3cac7392bb): Expected "Changes (⌃⇧⌘G)", Received "Changes". It has been red in web / react-apps-check on every PR since 08:10Z (#19780, #19782, #19779).

#19770 intentionally moved header keycaps out of title into the tooltip's shortcut hint (data-tooltip-shortcut, drawn in secondary ink like the native tooltip). It updated chat-header-tooltips.test.tsx but not this older test. The test now asserts the new contract: title Changes, shortcut hint ⌃⇧⌘G.

Testing

heavy --mem 4G bun test test/chat-header-changes.test.tsx test/chat-header-tooltips.test.tsx:

  • Red on feat-cmux-next bf3cac7392bb: 0 pass, 1 fail, at the line above.
  • Green on this branch: 2 pass.

bun run check passes.

Changelog

none

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.


Summary by cubic

Updates chat-header-changes.test.tsx to match the tooltip behavior change from #19770, where the Changes button's keycap moved from the title attribute into data-tooltip-shortcut. This fixes the failing test that had been red in react-apps-check on feat-cmux-next for every PR.

The test now asserts title is "Changes" and data-tooltip-shortcut is "⌃⇧⌘G".