Unquarantine ServerReset_BeforeRequestBody_ClientBodyThrows (#57470)

amcasey · web-flow · commit 8a9be3df3cbd · 2024-08-22T17:03:57.000-07:00
Fixes #46717 There isn't an obvious race in the test and the component that timed out appears to be from runtime (`HttpClient` and/or `HttpContent`). Since the test hasn't failed in 30 days and runtime has likely changed in the 18 months since the issue was filed, I think we should unquarantine the test.
diff --git a/src/Servers/Kestrel/test/Interop.FunctionalTests/HttpClientHttp2InteropTests.cs b/src/Servers/Kestrel/test/Interop.FunctionalTests/HttpClientHttp2InteropTests.cs
@@ -689,7 +689,6 @@ public async Task ServerReset_AfterTrailers_NoError(string scheme)
         await host.StopAsync().DefaultTimeout();
     }
 
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/46717")]
     [Theory]
     [MemberData(nameof(SupportedSchemes))]
     public async Task ServerReset_BeforeRequestBody_ClientBodyThrows(string scheme)

Original file line number	Diff line number	Diff line change
`@@ -689,7 +689,6 @@ public async Task ServerReset_AfterTrailers_NoError(string scheme)`
`689`	`689`	`await host.StopAsync().DefaultTimeout();`
`690`	`690`	`}`
`691`	`691`
`692`		`- [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/46717")]`
`693`	`692`	`[Theory]`
`694`	`693`	`[MemberData(nameof(SupportedSchemes))]`
`695`	`694`	`public async Task ServerReset_BeforeRequestBody_ClientBodyThrows(string scheme)`