Stop spamming System.Net.Http.Tests output with HTTP/2 skipped tests (#35913)
authorStephen Toub <stoub@microsoft.com>
Thu, 7 May 2020 01:02:49 +0000 (21:02 -0400)
committerGitHub <noreply@github.com>
Thu, 7 May 2020 01:02:49 +0000 (21:02 -0400)
commitea70890b7d8e67e748db5b2f7627d98cb8c79319
tree86269e10759f376ac1bc468a32be7fb18c246200
parentddade10a265c8539a281328cea7659e17194bdd1
Stop spamming System.Net.Http.Tests output with HTTP/2 skipped tests (#35913)

There are a bunch of tests that don't make sense on HTTP/2, but end up executing because of how our tests are structured.  As a result, the test itself checks whether HTTP/2 is applicable and bails if it isn't.  Currently some of these are bailing by throwing a SkipTestException, which results in a message being output on the console.  Such messages are meant for tests that can be run in some circumstances, e.g. a stress test that needs an environment variable set.  They aren't needed when a test is never going to run.

I also marked the stress tests as outerloop so they don't spam the console when just running innerloop.
src/libraries/Common/tests/System/Net/Http/HttpClientHandlerTest.Cancellation.cs
src/libraries/Common/tests/System/Net/Http/HttpClientHandlerTest.cs
src/libraries/System.Net.Http/tests/FunctionalTests/HttpClientMiniStressTest.cs