Skip to content

Commit

Permalink
Make async VSTestConsoleWrapper apis obsolete (#4648)
Browse files Browse the repository at this point in the history
  • Loading branch information
@nohwnd
nohwnd committed Aug 10, 2023
1 parent d21ac0a commit c8447e7
Show file tree
Hide file tree
Showing 2 changed files with 29 additions and 713 deletions.
33 changes: 29 additions & 4 deletions ...soft.TestPlatform.VsTestConsole.TranslationLayer/Interfaces/IVsTestConsoleWrapperAsync.cs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ public interface IVsTestConsoleWrapperAsync
/// <summary>
/// Asynchronous equivalent of <see cref="IVsTestConsoleWrapper.StartSession"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task StartSessionAsync();

/// <summary>
Expand All @@ -30,7 +31,7 @@ public interface IVsTestConsoleWrapperAsync
/// string,
/// ITestSessionEventsHandler)"/>.
/// </summary>
[Obsolete("This API is not final yet and is subject to changes.", false)]
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task<ITestSession?> StartTestSessionAsync(
IList<string> sources,
string? runSettings,
Expand All @@ -44,7 +45,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestPlatformOptions,
/// ITestSessionEventsHandler)"/>.
/// </summary>
[Obsolete("This API is not final yet and is subject to changes.", false)]
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task<ITestSession?> StartTestSessionAsync(
IList<string> sources,
string? runSettings,
Expand All @@ -60,7 +61,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestSessionEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("This API is not final yet and is subject to changes.", false)]
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task<ITestSession?> StartTestSessionAsync(
IList<string> sources,
string? runSettings,
Expand All @@ -74,7 +75,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestSessionInfo,
/// ITestSessionEventsHandler)"/>.
/// </summary>
[Obsolete("This API is not final yet and is subject to changes.", false)]
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task<bool> StopTestSessionAsync(
TestSessionInfo? testSessionInfo,
ITestSessionEventsHandler eventsHandler);
Expand All @@ -97,6 +98,7 @@ public interface IVsTestConsoleWrapperAsync
/// IVsTestConsoleWrapper.InitializeExtensions(
/// IEnumerable{string})"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task InitializeExtensionsAsync(IEnumerable<string> pathToAdditionalExtensions);

/// <summary>
Expand All @@ -106,6 +108,7 @@ public interface IVsTestConsoleWrapperAsync
/// string,
/// ITestDiscoveryEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task DiscoverTestsAsync(
IEnumerable<string> sources,
string? discoverySettings,
Expand All @@ -119,6 +122,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestPlatformOptions,
/// ITestDiscoveryEventsHandler2)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task DiscoverTestsAsync(
IEnumerable<string> sources,
string? discoverySettings,
Expand All @@ -134,6 +138,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestSessionInfo,
/// ITestDiscoveryEventsHandler2)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task DiscoverTestsAsync(
IEnumerable<string> sources,
string? discoverySettings,
Expand All @@ -144,6 +149,7 @@ public interface IVsTestConsoleWrapperAsync
/// <summary>
/// See <see cref="IVsTestConsoleWrapper.CancelDiscovery"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
void CancelDiscovery();

/// <summary>
Expand All @@ -153,6 +159,7 @@ public interface IVsTestConsoleWrapperAsync
/// string,
/// ITestRunEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -166,6 +173,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestPlatformOptions,
/// ITestRunEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -181,6 +189,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestSessionInfo,
/// ITestRunEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -198,6 +207,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITelemetryEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -213,6 +223,7 @@ public interface IVsTestConsoleWrapperAsync
/// string,
/// ITestRunEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -226,6 +237,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestPlatformOptions,
/// ITestRunEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -241,6 +253,7 @@ public interface IVsTestConsoleWrapperAsync
/// TestSessionInfo,
/// ITestRunEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -258,6 +271,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITelemetryEventsHandler)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -274,6 +288,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -289,6 +304,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -306,6 +322,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -325,6 +342,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITelemetryEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<string> sources,
string? runSettings,
Expand All @@ -342,6 +360,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -357,6 +376,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -374,6 +394,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITestRunEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -393,6 +414,7 @@ public interface IVsTestConsoleWrapperAsync
/// ITelemetryEventsHandler,
/// ITestHostLauncher)"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
Task RunTestsWithCustomTestHostAsync(
IEnumerable<TestCase> testCases,
string? runSettings,
Expand All @@ -405,11 +427,13 @@ public interface IVsTestConsoleWrapperAsync
/// <summary>
/// See <see cref="IVsTestConsoleWrapper.CancelTestRun"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
void CancelTestRun();

/// <summary>
/// See <see cref="IVsTestConsoleWrapper.AbortTestRun"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
void AbortTestRun();

/// <summary>
Expand Down Expand Up @@ -457,5 +481,6 @@ public interface IVsTestConsoleWrapperAsync
/// <summary>
/// See <see cref="IVsTestConsoleWrapper.EndSession"/>.
/// </summary>
[Obsolete("The async APIs don't work, use the sync API instead.")]
void EndSession();
}

0 comments on commit c8447e7

Please sign in to comment.