Skip to content

Commit 2b18d5b

Browse files
authored
Fix and un-quarantine CanUseAntiforgeryAfterInitialRender test (#63640)
* Un-quarantine CanUseAntiforgeryAfterInitialRender test, try to fix its flakiness * Add interactive rendering check to test component, simplify test assertion
1 parent 1da465d commit 2b18d5b

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

src/Components/test/E2ETest/ServerRenderingTests/FormHandlingTests/AntiforgeryTests.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@ public override Task InitializeAsync()
3333
[Theory]
3434
[InlineData("server")]
3535
[InlineData("webassembly")]
36-
[QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/57766")]
3736
public void CanUseAntiforgeryAfterInitialRender(string target)
3837
{
3938
Navigate($"{ServerPathBase}/{target}-antiforgery-form");
@@ -47,7 +46,6 @@ public void CanUseAntiforgeryAfterInitialRender(string target)
4746
var submit = Browser.Exists(By.Id("submit"));
4847
submit.Click();
4948

50-
var result = Browser.Exists(By.Id("result"));
51-
Browser.Equal("Test", () => result.Text);
49+
Browser.Equal("Test", () => Browser.FindElement(By.Id("result")).Text);
5250
}
5351
}

src/Components/test/testassets/Components.TestServer/RazorComponents/Pages/Forms/ServerAntiforgeryForm.razor

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
{
77
<TestContentPackage.InteractiveAntiforgery RenderForm="bool.Parse(RenderForm)" />
88
}
9-
else
9+
else if (RendererInfo.IsInteractive)
1010
{
1111
<p id="result">@Name</p>
1212
}

0 commit comments

Comments
 (0)