[PR #13092] Fix various unit/feature test bugs #29383

Closed
opened 2026-01-31 09:34:33 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/13092

State: closed
Merged: Yes


This commit fixes various bugs in our unit/feature test suite:

  • 2 tests failed at 150% scale.
  • The "null key" (@ on a US keyboard) isn't necessarily Shift+2.
    The proper way to get it is with LOBYTE(VkKeyScanW(0))
  • InputEngineTest::C0Test never worked as it overwrote
    the loop variable, exiting the loop early
**Original Pull Request:** https://github.com/microsoft/terminal/pull/13092 **State:** closed **Merged:** Yes --- This commit fixes various bugs in our unit/feature test suite: * 2 tests failed at 150% scale. * The "null key" (@ on a US keyboard) isn't necessarily Shift+2. The proper way to get it is with `LOBYTE(VkKeyScanW(0))` * `InputEngineTest::C0Test` never worked as it overwrote the loop variable, exiting the loop early
claunia added the pull-request label 2026-01-31 09:34:34 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#29383