-
Notifications
You must be signed in to change notification settings - Fork 1k
Skip unstable ListView Tile tests and case ListView_Click_On_Second_Column_Does_Not_Alter_CheckBoxesAsync #13294
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Skip unstable ListView Tile tests and case ListView_Click_On_Second_Column_Does_Not_Alter_CheckBoxesAsync #13294
Conversation
…olumn_Does_Not_Alter_CheckBoxesAsync
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.
Comments suppressed due to low confidence (1)
src/test/integration/UIIntegrationTests/ListViewTests.cs:114
- [nitpick] Consider adding a TODO comment to track re-enabling these tests once the underlying issues are resolved.
[ActiveIssue("https://github.com/dotnet/winforms/issues/13291")]
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #13294 +/- ##
====================================================
- Coverage 62.19645% 40.32509% -21.87136%
====================================================
Files 1556 1018 -538
Lines 159366 96404 -62962
Branches 14854 13450 -1404
====================================================
- Hits 99120 38875 -60245
+ Misses 59495 57107 -2388
+ Partials 751 422 -329
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
@LeafShi1 - how many failures had you seen? Could you please investigate what the cause could be? |
ListView Tile related tests have failed 3 times in the past two weeks, the first failure was in this PR #13284 Everything has returned to normal in the past few days, I will follow up on these issues further |
@LeafShi1 - could you please share the link to look up test history? Interesting that it started to happen after an update to preview 4. Could you please investigate if these tests are reading any static variables (Cursor position?), is there any logging or additional validation to add to help understand the root cause? |
The test failure link https://dev.azure.com/dnceng-public/public/_test/analytics?definitionId=76&contextType=build |
Related #13288, #13291
Proposed changes