TestMouseWheelReadConsoleInputHelper tests are flaky #6324

Open
opened 2026-01-31 00:35:47 +00:00 by claunia · 3 comments
Owner

Originally created by @miniksa on GitHub (Feb 6, 2020).

In the InputTests, there are a handful of them that inject mouse events into the window handle.

They center around the method TestMouseWheelReadConsoleInputHelper.

They tend to fail when there are two input events in the queue instead of just the one they expect.

My theory is that the extra event in the queue is a focus or unfocus event.

We should make them all more robust to the point where they don't care/skip excess focus/unfocus events.

Originally created by @miniksa on GitHub (Feb 6, 2020). In the InputTests, there are a handful of them that inject mouse events into the window handle. They center around the method `TestMouseWheelReadConsoleInputHelper`. They tend to fail when there are two input events in the queue instead of just the one they expect. My theory is that the extra event in the queue is a focus or unfocus event. We should make them all more robust to the point where they don't care/skip excess focus/unfocus events.
claunia added the Help WantedProduct-ConhostIssue-TaskArea-InputArea-CodeHealth labels 2026-01-31 00:35:48 +00:00
Author
Owner

@DHowett-MSFT commented on GitHub (Feb 7, 2020):

Carlos has been looking into mouse stuff for 1.x. @carlos-zamora, would you mind taking this one on (post-v1?)

@DHowett-MSFT commented on GitHub (Feb 7, 2020): Carlos has been looking into mouse stuff for 1.x. @carlos-zamora, would you mind taking this one on (post-v1?)
Author
Owner

@carlos-zamora commented on GitHub (Feb 7, 2020):

Carlos has been looking into mouse stuff for 1.x. @carlos-zamora, would you mind taking this one on (post-v1?)

Will do! Should I assign it to myself then?

@carlos-zamora commented on GitHub (Feb 7, 2020): > Carlos has been looking into mouse stuff for 1.x. @carlos-zamora, would you mind taking this one on (post-v1?) Will do! Should I assign it to myself then?
Author
Owner

@DHowett-MSFT commented on GitHub (Feb 8, 2020):

Sure!

@DHowett-MSFT commented on GitHub (Feb 8, 2020): Sure!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#6324