chore(test): Added tests for `WindowsFormsSynchronizationContextProvi…

…der`
dailydevops · Mar 25, 2024 · e202d5e · e202d5e
1 parent 95a7c44
commit e202d5e
Show file tree

Hide file tree

Showing 2 changed files with 381 additions and 2 deletions.
diff --git a/...volve.Extensions.Hosting.WinForms/Internals/WindowsFormsSynchronizationContextProvider.cs b/...volve.Extensions.Hosting.WinForms/Internals/WindowsFormsSynchronizationContextProvider.cs
@@ -4,18 +4,23 @@
 using System.Diagnostics.CodeAnalysis;
 using System.Threading;
 using System.Threading.Tasks;
-using System.Windows.Forms;
 
 /// <inheritdoc />
+[SuppressMessage(
+    "Usage",
+    "VSTHRD001:Avoid legacy thread switching APIs",
+    Justification = "As designed."
+)]
 internal sealed class WindowsFormsSynchronizationContextProvider
     : IWindowsFormsSynchronizationContextProvider
 {
-    internal WindowsFormsSynchronizationContext Context { get; set; } = default!;
+    internal SynchronizationContext Context { get; set; } = default!;
 
     /// <inheritdoc/>
     public void Invoke([NotNull] Action action)
     {
         ArgumentNullException.ThrowIfNull(action);
+        ArgumentNullException.ThrowIfNull(Context);
 
         Context.Send(
             delegate
@@ -31,6 +36,7 @@ public void Invoke([NotNull] Action action)
     public TResult Invoke<TResult>([NotNull] Func<TResult> action)
     {
         ArgumentNullException.ThrowIfNull(action);
+        ArgumentNullException.ThrowIfNull(Context);
 
         TResult result = default!;
         Context.Send(
@@ -40,6 +46,7 @@ public TResult Invoke<TResult>([NotNull] Func<TResult> action)
             },
             null
         );
+
         return result;
     }
 
@@ -48,6 +55,7 @@ public TResult Invoke<TResult>([NotNull] Func<TResult> action)
     public TResult Invoke<TResult, TInput>([NotNull] Func<TInput, TResult> action, TInput input)
     {
         ArgumentNullException.ThrowIfNull(action);
+        ArgumentNullException.ThrowIfNull(Context);
 
         TResult result = default!;
         Context.Send(
@@ -57,6 +65,7 @@ public TResult Invoke<TResult, TInput>([NotNull] Func<TInput, TResult> action, T
             },
             null
         );
+
         return result;
     }
 
@@ -67,6 +76,7 @@ public async ValueTask InvokeAsync(
     )
     {
         ArgumentNullException.ThrowIfNull(action);
+        ArgumentNullException.ThrowIfNull(Context);
 
         var tcs = new TaskCompletionSource();
         Context.Post(
@@ -95,6 +105,7 @@ public async ValueTask<TResult> InvokeAsync<TResult>(
     )
     {
         ArgumentNullException.ThrowIfNull(action);
+        ArgumentNullException.ThrowIfNull(Context);
 
         var tcs = new TaskCompletionSource<TResult>();
         Context.Post(
@@ -112,6 +123,7 @@ public async ValueTask<TResult> InvokeAsync<TResult>(
             },
             tcs
         );
+
         return await tcs.Task.WaitAsync(cancellationToken).ConfigureAwait(true);
     }
 
@@ -123,6 +135,7 @@ public async ValueTask<TResult> InvokeAsync<TResult, TInput>(
     )
     {
         ArgumentNullException.ThrowIfNull(action);
+        ArgumentNullException.ThrowIfNull(Context);
 
         var tcs = new TaskCompletionSource<TResult>();
         Context.Post(
@@ -140,6 +153,7 @@ public async ValueTask<TResult> InvokeAsync<TResult, TInput>(
             },
             tcs
         );
+
         return await tcs.Task.WaitAsync(cancellationToken).ConfigureAwait(true);
     }
 }