Skip to content

Add a simplified version of ServerComponentRenderingTest.CanDispatchAsyncWorkToSyncContext #21633

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 1 commit into from
May 11, 2020

Conversation

pranavkm
Copy link
Contributor

@pranavkm pranavkm commented May 8, 2020

ServerComponentRenderingTest.CanDispatchAsyncWorkToSyncContext is quarantined but since it's not running on Helix there isn't any history available for it. With all the moving
parts in the server test, it's unclear if it's a product vs test setup issue.

Authoring a more simplified test so we can track test history. Note that it's starting off as quarantined because there's no evidence that the product code isn't broken

Fixes #19413

…syncWorkToSyncContext

ServerComponentRenderingTest.CanDispatchAsyncWorkToSyncContext is quarantined but since it's not running on Helix there isn't any history available for it. With all the moving
parts in the server test, it's unclear if it's a product vs test setup issue.

Authoring a more simplified test so we can track test history. Note that it's starting off as quarantined because there's no evidence that the product code isn't broken

Fixes #19413
@pranavkm pranavkm requested a review from SteveSandersonMS as a code owner May 8, 2020 20:47
@ghost ghost added the area-blazor Includes: Blazor, Razor Components label May 8, 2020
@pranavkm pranavkm requested a review from a team May 8, 2020 20:48
Copy link
Member

@captainsafia captainsafia left a comment

Choose a reason for hiding this comment

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

Getting rid of the overhead of the full end-to-end tests seems like a sensible approach here.

@javiercn
Copy link
Member

Do we understand why that test was being flaky? It has value to have tests like these that are E2E as they exercise the pipeline much closer to reality and with these type of things (ambient stuff) it's more important to try and be faithful to the implementation.

@pranavkm
Copy link
Contributor Author

Per the description, we don't know if the test is still flaky. We have no quarantine history. This is a replica of the functional test that should tell us if it is.

@pranavkm pranavkm merged commit 4b88074 into master May 11, 2020
@pranavkm pranavkm deleted the prkrishn/19413 branch May 11, 2020 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-blazor Includes: Blazor, Razor Components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Flaky test] Microsoft.AspNetCore.Components.E2ETest.Tests.ServerComponentRenderingTest.CanDispatchAsyncWorkToSyncContext
3 participants