Skip to content

Quarantine flaky tests #20261

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

Merged
merged 2 commits into from
Mar 28, 2020
Merged

Quarantine flaky tests #20261

merged 2 commits into from
Mar 28, 2020

Conversation

BrennanConroy
Copy link
Member

@BrennanConroy BrennanConroy added this to the 5.0.0-preview3 milestone Mar 28, 2020
@ghost ghost added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Mar 28, 2020
@@ -24,7 +24,7 @@ public BlazorServerTemplateTest(ProjectFactoryFixture projectFactory, BrowserFix

public Project Project { get; private set; }

[ConditionalFact]
[ConditionalFact(Skip = "This test ran for over an hour")]
Copy link
Contributor

Choose a reason for hiding this comment

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

It's starting to feel like all the template tests can timeout.

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't think so, when you quarantined the grpc one you didn't do these 2 even though they showed up. And now they're showing up on their own. The rest of the template tests have yet to show these kinds of timeouts.

@Pilchie Pilchie merged commit a421a31 into master Mar 28, 2020
@Pilchie Pilchie deleted the brecon/quarantine branch March 28, 2020 18:03
@Pilchie
Copy link
Member

Pilchie commented Mar 28, 2020

Thanks @BrennanConroy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants