[release/10.0] Fix and un-quarantine CanUseAntiforgeryAfterInitialRender test #63644
+2
−4
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of #63640 to release/10.0
/cc @oroztocil
Fix and un-quarantine CanUseAntiforgeryAfterInitialRender test
Fix and un-quarantine a flaky E2E test for Blazor.
Description
The
ServerRenderingTests.FormHandlingTests.CanUseAntiforgeryAfterInitialRender
test was flaky due to a race condition between re-rendering an element and checking its text value in the test assertion.As the final step of the test, we check the text value of a specific DOM element. There was a race condition due to which the element can get re-rerendered between the moment when we acquire a WebElement reference to the DOM element (using the
Browser.Exists
method) and when we read the element's text value. When that happens,StaleElementReferenceException
is thrown.The PR tries to remove the flakiness by:
Fixes #63640
Customer Impact
The change increases the test coverage of our code.
Regression?
Risk
Does not impact shipped code.
Verification
Packaging changes reviewed?
When servicing release/2.3