code cleanup for E2E tests (#1710)

Author: terencefan

test/Microsoft.Azure.SignalR.E2ETests/SignalR/TestClientSet.cs

@@ -14,6 +14,7 @@ namespace Microsoft.Azure.SignalR.Tests
     internal class TestClientSet : ITestClientSet
     {
         private readonly IList<HubConnection> _connections;
+
         private ITestOutputHelper _output;
 
         public int Count => _connections?.Count ?? 0;

test/Microsoft.Azure.SignalR.E2ETests/SignalR/TestServer.cs

@@ -15,15 +15,22 @@ namespace Microsoft.Azure.SignalR.Tests
     internal class TestServer : TestServerBase
     {
         private IWebHost _host;
+
         private IServiceConnectionManager<TestHub> _scm;
 
         public override TestHubConnectionManager HubConnectionManager { get; }
 
-        public TestServer(ITestOutputHelper output): base(output)
+        public TestServer(ITestOutputHelper output) : base(output)
         {
             HubConnectionManager = new TestHubConnectionManager();
         }
 
+        public override async Task StopAsync()
+        {
+            await _host.StopAsync();
+            await _scm.StopAsync();
+        }
+
         protected override Task StartCoreAsync(string serverUrl, ITestOutputHelper output, Dictionary<string, string> configuration)
         {
             _host = new WebHostBuilder()
@@ -32,7 +39,7 @@ protected override Task StartCoreAsync(string serverUrl, ITestOutputHelper outpu
                     services.AddSingleton<TestHubConnectionManager>(HubConnectionManager);
                 })
                 .ConfigureLogging(logging => logging.AddXunit(output))
-                .ConfigureAppConfiguration(builder =>  builder.AddInMemoryCollection(configuration))
+                .ConfigureAppConfiguration(builder => builder.AddInMemoryCollection(configuration))
                 .UseStartup<TestStartup>()
                 .UseUrls(serverUrl)
                 .UseKestrel()
@@ -42,11 +49,5 @@ protected override Task StartCoreAsync(string serverUrl, ITestOutputHelper outpu
 
             return _host.StartAsync();
         }
-
-        public override async Task StopAsync()
-        {
-            await _host.StopAsync();
-            await _scm.StopAsync();
-        }
     }
 }

test/Microsoft.Azure.SignalR.E2ETests/SignalR/TestStartup.cs

@@ -25,10 +25,7 @@ public TestStartup(IConfiguration configuration)
         public void Configure(IApplicationBuilder app)
         {
             app.UseRouting();
-            app.UseEndpoints(configure =>
-            {
-                configure.MapHub<TestHub>($"/{nameof(TestHub)}");
-            });
+            app.UseEndpoints(configure => configure.MapHub<TestHub>($"/{nameof(TestHub)}"));
             app.UseMvc();
         }