Unquarantine (most) HTTP3 tests (#37756)

wtgodbe · web-flow · commit 25a869a39429 · 2021-10-23T12:14:26.000-07:00
diff --git a/src/Servers/Kestrel/Transport.Quic/test/QuicConnectionContextTests.cs b/src/Servers/Kestrel/Transport.Quic/test/QuicConnectionContextTests.cs
@@ -13,7 +13,6 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests
 {
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
     public class QuicConnectionContextTests : TestApplicationErrorLoggerLoggedTest
     {
         private static readonly byte[] TestData = Encoding.UTF8.GetBytes("Hello world");
diff --git a/src/Servers/Kestrel/Transport.Quic/test/QuicConnectionListenerTests.cs b/src/Servers/Kestrel/Transport.Quic/test/QuicConnectionListenerTests.cs
@@ -14,7 +14,6 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests
 {
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
     public class QuicConnectionListenerTests : TestApplicationErrorLoggerLoggedTest
     {
         private static readonly byte[] TestData = Encoding.UTF8.GetBytes("Hello world");
diff --git a/src/Servers/Kestrel/Transport.Quic/test/QuicStreamContextTests.cs b/src/Servers/Kestrel/Transport.Quic/test/QuicStreamContextTests.cs
@@ -19,7 +19,6 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests
 {
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
     public class QuicStreamContextTests : TestApplicationErrorLoggerLoggedTest
     {
         private static readonly byte[] TestData = Encoding.UTF8.GetBytes("Hello world");
diff --git a/src/Servers/Kestrel/Transport.Quic/test/QuicTransportFactoryTests.cs b/src/Servers/Kestrel/Transport.Quic/test/QuicTransportFactoryTests.cs
@@ -16,7 +16,6 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests
 {
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
     public class QuicTransportFactoryTests : TestApplicationErrorLoggerLoggedTest
     {
         [ConditionalFact]
diff --git a/src/Servers/Kestrel/Transport.Quic/test/WebHostTests.cs b/src/Servers/Kestrel/Transport.Quic/test/WebHostTests.cs
@@ -13,7 +13,6 @@
 
 namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests
 {
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
     public class WebHostTests : LoggedTest
     {
         [ConditionalFact]
diff --git a/src/Servers/Kestrel/test/Interop.FunctionalTests/Http3/Http3RequestTests.cs b/src/Servers/Kestrel/test/Interop.FunctionalTests/Http3/Http3RequestTests.cs
@@ -22,7 +22,6 @@
 
 namespace Interop.FunctionalTests.Http3
 {
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
     public class Http3RequestTests : LoggedTest
     {
         private class StreamingHttpContent : HttpContent
@@ -394,6 +393,7 @@ public async Task POST_ClientCancellationUpload_RequestAbortRaised(HttpProtocols
         [MsQuicSupported]
         [InlineData(HttpProtocols.Http3)]
         [InlineData(HttpProtocols.Http2)]
+        [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
         public async Task POST_ServerAbort_ClientReceivesAbort(HttpProtocols protocol)
         {
             // Arrange
@@ -1472,6 +1472,7 @@ public async Task GET_ConnectionInfo_PropertiesSet()
         [MsQuicSupported]
         [InlineData(HttpProtocols.Http3)]
         [InlineData(HttpProtocols.Http2)]
+        [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
         public async Task GET_GracefulServerShutdown_AbortRequestsAfterHostTimeout(HttpProtocols protocol)
         {
             // Arrange
diff --git a/src/Servers/Kestrel/test/Interop.FunctionalTests/Http3/Http3TlsTests.cs b/src/Servers/Kestrel/test/Interop.FunctionalTests/Http3/Http3TlsTests.cs
@@ -15,7 +15,6 @@
 
 namespace Interop.FunctionalTests.Http3
 {
-    [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
     public class Http3TlsTests : LoggedTest
     {
         [ConditionalFact]
@@ -156,6 +155,7 @@ public async Task ClientCertificate_NoOrDelayed_Available_Ignored(ClientCertific
         [InlineData(ClientCertificateMode.AllowCertificate, true)]
         [MsQuicSupported]
         [OSSkipCondition(OperatingSystems.MacOSX | OperatingSystems.Linux, SkipReason = "https://github.com/dotnet/aspnetcore/issues/35800")]
+        [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
         public async Task ClientCertificate_AllowOrRequire_Available_Invalid_Refused(ClientCertificateMode mode, bool serverAllowInvalid)
         {
             var builder = CreateHostBuilder(async context =>
@@ -212,6 +212,7 @@ public async Task ClientCertificate_AllowOrRequire_Available_Invalid_Refused(Cli
         [ConditionalFact]
         [MsQuicSupported]
         [OSSkipCondition(OperatingSystems.MacOSX | OperatingSystems.Linux, SkipReason = "https://github.com/dotnet/aspnetcore/issues/35800")]
+        [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]
         public async Task ClientCertificate_Allow_NotAvailable_Optional()
         {
             var builder = CreateHostBuilder(async context =>

Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,6 @@`
`13`	`13`
`14`	`14`	`namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests`
`15`	`15`	`{`
`16`		`- [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`17`	`16`	`public class QuicConnectionContextTests : TestApplicationErrorLoggerLoggedTest`
`18`	`17`	`{`
`19`	`18`	`private static readonly byte[] TestData = Encoding.UTF8.GetBytes("Hello world");`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,6 @@`
`14`	`14`
`15`	`15`	`namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests`
`16`	`16`	`{`
`17`		`- [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`18`	`17`	`public class QuicConnectionListenerTests : TestApplicationErrorLoggerLoggedTest`
`19`	`18`	`{`
`20`	`19`	`private static readonly byte[] TestData = Encoding.UTF8.GetBytes("Hello world");`
Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,6 @@`
`19`	`19`
`20`	`20`	`namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests`
`21`	`21`	`{`
`22`		`- [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`23`	`22`	`public class QuicStreamContextTests : TestApplicationErrorLoggerLoggedTest`
`24`	`23`	`{`
`25`	`24`	`private static readonly byte[] TestData = Encoding.UTF8.GetBytes("Hello world");`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,6 @@`
`16`	`16`
`17`	`17`	`namespace Microsoft.AspNetCore.Server.Kestrel.Transport.Quic.Tests`
`18`	`18`	`{`
`19`		`- [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`20`	`19`	`public class QuicTransportFactoryTests : TestApplicationErrorLoggerLoggedTest`
`21`	`20`	`{`
`22`	`21`	`[ConditionalFact]`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,6 @@`
`22`	`22`
`23`	`23`	`namespace Interop.FunctionalTests.Http3`
`24`	`24`	`{`
`25`		`- [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`26`	`25`	`public class Http3RequestTests : LoggedTest`
`27`	`26`	`{`
`28`	`27`	`private class StreamingHttpContent : HttpContent`
`@@ -394,6 +393,7 @@ public async Task POST_ClientCancellationUpload_RequestAbortRaised(HttpProtocols`
`394`	`393`	`[MsQuicSupported]`
`395`	`394`	`[InlineData(HttpProtocols.Http3)]`
`396`	`395`	`[InlineData(HttpProtocols.Http2)]`
	`396`	`+ [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`397`	`397`	`public async Task POST_ServerAbort_ClientReceivesAbort(HttpProtocols protocol)`
`398`	`398`	`{`
`399`	`399`	`// Arrange`
`@@ -1472,6 +1472,7 @@ public async Task GET_ConnectionInfo_PropertiesSet()`
`1472`	`1472`	`[MsQuicSupported]`
`1473`	`1473`	`[InlineData(HttpProtocols.Http3)]`
`1474`	`1474`	`[InlineData(HttpProtocols.Http2)]`
	`1475`	`+ [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`1475`	`1476`	`public async Task GET_GracefulServerShutdown_AbortRequestsAfterHostTimeout(HttpProtocols protocol)`
`1476`	`1477`	`{`
`1477`	`1478`	`// Arrange`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,6 @@`
`15`	`15`
`16`	`16`	`namespace Interop.FunctionalTests.Http3`
`17`	`17`	`{`
`18`		`- [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`19`	`18`	`public class Http3TlsTests : LoggedTest`
`20`	`19`	`{`
`21`	`20`	`[ConditionalFact]`
`@@ -156,6 +155,7 @@ public async Task ClientCertificate_NoOrDelayed_Available_Ignored(ClientCertific`
`156`	`155`	`[InlineData(ClientCertificateMode.AllowCertificate, true)]`
`157`	`156`	`[MsQuicSupported]`
`158`	`157`	`[OSSkipCondition(OperatingSystems.MacOSX \| OperatingSystems.Linux, SkipReason = "https://github.com/dotnet/aspnetcore/issues/35800")]`
	`158`	`+ [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`159`	`159`	`public async Task ClientCertificate_AllowOrRequire_Available_Invalid_Refused(ClientCertificateMode mode, bool serverAllowInvalid)`
`160`	`160`	`{`
`161`	`161`	`var builder = CreateHostBuilder(async context =>`
`@@ -212,6 +212,7 @@ public async Task ClientCertificate_AllowOrRequire_Available_Invalid_Refused(Cli`
`212`	`212`	`[ConditionalFact]`
`213`	`213`	`[MsQuicSupported]`
`214`	`214`	`[OSSkipCondition(OperatingSystems.MacOSX \| OperatingSystems.Linux, SkipReason = "https://github.com/dotnet/aspnetcore/issues/35800")]`
	`215`	`+ [QuarantinedTest("https://github.com/dotnet/aspnetcore/issues/35070")]`
`215`	`216`	`public async Task ClientCertificate_Allow_NotAvailable_Optional()`
`216`	`217`	`{`
`217`	`218`	`var builder = CreateHostBuilder(async context =>`