[Blazor] Reliability improvements for the E2E tests

Author: javiercn

src/Components/Ignitor/src/BlazorClient.cs

@@ -345,7 +345,6 @@ public async Task<bool> ConnectAsync(Uri uri, bool connectAutomatically = true)
             });
 
             HubConnection = builder.Build();
-            await HubConnection.StartAsync(CancellationToken);
 
             HubConnection.On<int, string>("JS.AttachComponent", OnAttachComponent);
             HubConnection.On<int, string, string>("JS.BeginInvokeJS", OnBeginInvokeJS);
@@ -354,6 +353,8 @@ public async Task<bool> ConnectAsync(Uri uri, bool connectAutomatically = true)
             HubConnection.On<string>("JS.Error", OnError);
             HubConnection.Closed += OnClosedAsync;
 
+            await HubConnection.StartAsync(CancellationToken);
+
             if (CaptureOperations)
             {
                 Operations = new Operations();

src/Components/test/E2ETest/Microsoft.AspNetCore.Components.E2ETests.csproj

@@ -57,4 +57,10 @@
     <Compile Include="$(RepoRoot)src\Shared\Components\ServerComponentMarker.cs" />
   </ItemGroup>
 
+  <ItemGroup>
+    <None Update="xunit.runner.json">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
+  </ItemGroup>
+
 </Project>

src/Components/test/E2ETest/ServerExecutionTests/ComponentHubReliabilityTest.cs

@@ -11,6 +11,7 @@
 using Microsoft.AspNetCore.Components.RenderTree;
 using Microsoft.AspNetCore.Components.Web;
 using Microsoft.AspNetCore.SignalR.Client;
+using Microsoft.AspNetCore.Testing;
 using Microsoft.Extensions.Logging;
 using TestServer;
 using Xunit;
@@ -119,6 +120,7 @@ await Client.ExpectCircuitErrorAndDisconnect(() => Client.HubConnection.SendAsyn
         }
 
         [Fact]
+        [Repeat(1000)]
         public async Task CannotInvokeJSInteropCallbackCompletionsBeforeInitialization()
         {
             // Arrange

src/Components/test/E2ETest/ServerExecutionTests/IgnitorTest.cs

@@ -83,14 +83,14 @@ protected virtual Task InitializeAsync()
             return Task.CompletedTask;
         }
 
-        protected virtual Task DisposeAsync()
+        protected async virtual Task DisposeAsync()
         {
             if (TestSink != null)
             {
                 TestSink.MessageLogged -= TestSink_MessageLogged;
             }
 
-            return Task.CompletedTask;
+            await Client.DisposeAsync();
         }
 
         private void TestSink_MessageLogged(WriteContext context)

src/Components/test/E2ETest/xunit.runner.json

@@ -0,0 +1,7 @@
+{
+  // This is set to -1 to allow the usage of an
+  // unlimited ammount of threads.
+  "maxParallelThreads": -1,
+  "diagnosticMessages": true,
+  "longRunningTestSeconds": 30
+}