HTTP/3: Unskip ContentLength_Received_NoDataFrames_Reset (#31743)

JamesNK · web-flow · commit 9ab99b219c3a · 2021-04-13T08:52:28.000Z
diff --git a/src/Servers/Kestrel/test/InMemory.FunctionalTests/Http3/Http3StreamTests.cs b/src/Servers/Kestrel/test/InMemory.FunctionalTests/Http3/Http3StreamTests.cs
@@ -627,7 +627,7 @@ public async Task RemoveConnectionSpecificHeaders()
             Assert.Contains(LogMessages, m => m.Message.Equals("One or more of the following response headers have been removed because they are invalid for HTTP/2 and HTTP/3 responses: 'Connection', 'Transfer-Encoding', 'Keep-Alive', 'Upgrade' and 'Proxy-Connection'."));
         }
 
-        [Fact(Skip = "Http3OutputProducer.Complete is called before input recognizes there is an error. Why is this different than HTTP/2?")]
+        [Fact]
         public async Task ContentLength_Received_NoDataFrames_Reset()
         {
             var headers = new[]

Original file line number	Diff line number	Diff line change
`@@ -627,7 +627,7 @@ public async Task RemoveConnectionSpecificHeaders()`
`627`	`627`	`Assert.Contains(LogMessages, m => m.Message.Equals("One or more of the following response headers have been removed because they are invalid for HTTP/2 and HTTP/3 responses: 'Connection', 'Transfer-Encoding', 'Keep-Alive', 'Upgrade' and 'Proxy-Connection'."));`
`628`	`628`	`}`
`629`	`629`
`630`		`- [Fact(Skip = "Http3OutputProducer.Complete is called before input recognizes there is an error. Why is this different than HTTP/2?")]`
	`630`	`+ [Fact]`
`631`	`631`	`public async Task ContentLength_Received_NoDataFrames_Reset()`
`632`	`632`	`{`
`633`	`633`	`var headers = new[]`