Fix failing UIA tests from #10886 #14844

Closed
opened 2026-01-31 04:21:03 +00:00 by claunia · 3 comments
Owner

Originally created by @carlos-zamora on GitHub (Aug 12, 2021).

Originally assigned to: @carlos-zamora on GitHub.

#10886 follow-up

There's some failing tests in the CSV that we're skipping right now. They shouldn't be failing, but the tests were verified to be correct when comparing our behavior to MS Word's.

We should update our UIA implementation to make these tests not fail.

IIRC I believe the failing tests are mainly:

  • "move by line" backwards on degenerate ranges are off by one
  • "move by document" do not report that they moved by the correct amount
  • a non-degenerate range that moves 0 should still expand by whatever unit you're moving by (even if you tried to move by 5/1/0 but ended up moving 0)
Originally created by @carlos-zamora on GitHub (Aug 12, 2021). Originally assigned to: @carlos-zamora on GitHub. #10886 follow-up There's some failing tests in the CSV that we're skipping right now. They shouldn't be failing, but the tests were verified to be correct when comparing our behavior to MS Word's. We should update our UIA implementation to make these tests not fail. IIRC I believe the failing tests are mainly: - "move by line" backwards on degenerate ranges are off by one - "move by document" do not report that they moved by the correct amount - a non-degenerate range that moves 0 should still expand by whatever unit you're moving by (even if you tried to move by 5/1/0 but ended up moving 0)
Author
Owner

@carlos-zamora commented on GitHub (Aug 12, 2021):

Added to the v1.11 milestone, but feel free to move it elsewhere.

Also, this is in the shared UIA implementation, so the bug affects Windows Terminal and Conhost

@carlos-zamora commented on GitHub (Aug 12, 2021): Added to the v1.11 milestone, but feel free to move it elsewhere. Also, this is in the shared UIA implementation, so the bug affects Windows Terminal and Conhost
Author
Owner

@ghost commented on GitHub (Aug 31, 2021):

:tada:This issue was addressed in #10991, which has now been successfully released as Windows Terminal Preview v1.10.2383.0.🎉

Handy links:

@ghost commented on GitHub (Aug 31, 2021): :tada:This issue was addressed in #10991, which has now been successfully released as `Windows Terminal Preview v1.10.2383.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.10.2383.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@ghost commented on GitHub (Aug 31, 2021):

:tada:This issue was addressed in #10991, which has now been successfully released as Windows Terminal Preview v1.11.2421.0.🎉

Handy links:

@ghost commented on GitHub (Aug 31, 2021): :tada:This issue was addressed in #10991, which has now been successfully released as `Windows Terminal Preview v1.11.2421.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.11.2421.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#14844