Skip to content
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

Adding more Skip attributes for flaky tests #6655

Conversation

vladimir-krestov
Copy link
Contributor

@vladimir-krestov vladimir-krestov commented Feb 9, 2022

Related Issue #6610 and Issue #6654
These tests fail sometimes in x86 CI build, that blocks deployment of fixes into sdk.

Proposed changes

  • Skipped unit tests that fail sometimes in arm64 and x86 environments, that blocks fixes deployment.

Customer Impact

  • No

Regression?

  • No

Risk

  • Minimal

Test methodology

  • CI build
Microsoft Reviewers: Open in CodeFlow

Related Issue dotnet#6610 and Issue dotnet#6654
These tests fail sometimes in x86 CI build, that blocks deployment of fixes into sdk.
@vladimir-krestov vladimir-krestov requested a review from a team as a code owner February 9, 2022 09:00
@ghost ghost assigned vladimir-krestov Feb 9, 2022
@vladimir-krestov vladimir-krestov added the waiting-review This item is waiting on review by one or more members of team label Feb 9, 2022
@@ -51,7 +51,8 @@ public async Task TabControl_TabPage_IsHoveredWithMouse_IsFalse_WhenMouseIs_Outs
});
}

[WinFormsFact]
[ActiveIssue("https://github.com/dotnet/winforms/issues/6654")]
[WinFormsFact(Skip = "Flaky tests, see: https://github.com/dotnet/winforms/issues/6654")]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Didn't add conditional Fact due to #6653

@dreddy-work dreddy-work merged commit 0007003 into dotnet:main Feb 9, 2022
@ghost ghost added this to the 7.0 Preview2 milestone Feb 9, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Mar 11, 2022
@RussKie RussKie removed the waiting-review This item is waiting on review by one or more members of team label Oct 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants