HttpClientMiniStressTest: Disable parallelism (#34941)
authorTalAloni <tal.aloni.il@gmail.com>
Tue, 14 Apr 2020 22:51:14 +0000 (01:51 +0300)
committerGitHub <noreply@github.com>
Tue, 14 Apr 2020 22:51:14 +0000 (18:51 -0400)
* Update HttpClientMiniStressTest.cs

Currently, it's possible for 4 different instances of SingleClient_ManyGets_Sync to run in parallel, each using up to 2 x 5000 sockets, totaling in 40,000 sockets. way beyond the original intent of testing 5000 client instances. since this additional stress is random in nature, I believe it is not intentional.
This change make sure that those tests will not run in parallel, this avoids some issues I've encountered in some systems.

* HttpClientMiniStressTest: Disable Parallelism: Used CollectionAttribute on the base class

src/libraries/System.Net.Http/tests/FunctionalTests/HttpClientMiniStressTest.cs

index 2f28f98..f0c388c 100644 (file)
@@ -86,6 +86,7 @@ namespace System.Net.Http.Functional.Tests
         }
     }
 
+    [Collection(nameof(HttpClientMiniStress))]
     public abstract class HttpClientMiniStress : HttpClientHandlerTestBase
     {
         public HttpClientMiniStress(ITestOutputHelper output) : base(output) { }