Skip to content

Quarantine CanRenderLargeComponentsWithServerRenderMode #49976

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
Aug 10, 2023

Conversation

captainsafia
Copy link
Member

No description provided.

@captainsafia captainsafia requested a review from a team as a code owner August 9, 2023 23:20
@ghost ghost added the area-blazor Includes: Blazor, Razor Components label Aug 9, 2023
Copy link
Member

@MackinnonBuck MackinnonBuck left a comment

Choose a reason for hiding this comment

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

🥲

@captainsafia captainsafia enabled auto-merge (squash) August 9, 2023 23:28
@captainsafia captainsafia merged commit 7834350 into main Aug 10, 2023
@captainsafia captainsafia deleted the captainsafia-patch-1 branch August 10, 2023 01:02
@ghost ghost added this to the 8.0-rc1 milestone Aug 10, 2023
@SteveSandersonMS
Copy link
Member

@captainsafia @MackinnonBuck Does this bring the new product code's correctness into question, or is there a known reason for the test to fail that is separate from the product implementation?

@MackinnonBuck
Copy link
Member

@SteveSandersonMS I haven't been able to reproduce the issue locally, but my guess is that the prerendered output is getting replaced with non-prerendered output in between the call to Browser.FindElement() and the subsequent access of the returned element's Text property, causing a StaleElementReferenceException. So I think this is more likely to be a test bug than a product bug.

@MackinnonBuck
Copy link
Member

I wonder if it would be satisfactory to have the test render non-interactive components since we're specifically testing SSR functionality anyway.

@dotnet dotnet deleted a comment Aug 14, 2023
@dotnet dotnet deleted a comment Aug 14, 2023
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.

3 participants