Cleanup TestApplication

src/Platform/Microsoft.Testing.Platform/Builder/TestApplication.cs

@@ -15,11 +15,6 @@ namespace Microsoft.Testing.Platform.Builder;
 /// Represents a test application.
 /// </summary>
 public sealed class TestApplication : ITestApplication
-#if NETCOREAPP
-#pragma warning disable SA1001 // Commas should be spaced correctly
-    , IAsyncDisposable
-#pragma warning restore SA1001 // Commas should be spaced correctly
-#endif
 {
     private readonly IHost _host;
     private static UnhandledExceptionHandler? s_unhandledExceptionHandler;
@@ -41,6 +36,7 @@ static TestApplication() =>
     /// <param name="args">The command line arguments.</param>
     /// <param name="testApplicationOptions">The test application options.</param>
     /// <returns>The task representing the asynchronous operation.</returns>
+    [Obsolete("This method is obsolete. Use CreateBuilderAsync instead.")]
     public static Task<ITestApplicationBuilder> CreateServerModeBuilderAsync(string[] args, TestApplicationOptions? testApplicationOptions = null)
     {
         if (args.Contains($"--{PlatformCommandLineProvider.ServerOptionKey}") || args.Contains($"-{PlatformCommandLineProvider.ServerOptionKey}"))
@@ -214,14 +210,6 @@ async Task LogVariableAsync(string key)
     public void Dispose()
         => (_host as IDisposable)?.Dispose();
 
-#if NETCOREAPP
-    /// <inheritdoc />
-    public ValueTask DisposeAsync()
-        => _host is IAsyncDisposable asyncDisposable
-            ? asyncDisposable.DisposeAsync()
-            : ValueTask.CompletedTask;
-#endif
-
     /// <inheritdoc />
     public async Task<int> RunAsync()
         => await _host.RunAsync().ConfigureAwait(false);

src/Platform/Microsoft.Testing.Platform/PublicAPI/net/PublicAPI.Unshipped.txt

@@ -1 +1,2 @@
 #nullable enable
+*REMOVED*Microsoft.Testing.Platform.Builder.TestApplication.DisposeAsync() -> System.Threading.Tasks.ValueTask