Comments from copilot round 1

Author: benrr101

src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/Utilities/AsyncHelper.cs

@@ -768,9 +768,11 @@ private static void ObserveContinuationException(Task continuationTask)
                 static task =>
                 {
                     SqlClientEventSource.Log.TryTraceEvent($"Unobserved task exception: {task.Exception}");
-                    return _ = task.Exception;
+                    _ = task.Exception;
                 },
-                TaskContinuationOptions.OnlyOnFaulted | TaskContinuationOptions.ExecuteSynchronously);
+                CancellationToken.None,
+                TaskContinuationOptions.OnlyOnFaulted | TaskContinuationOptions.ExecuteSynchronously,
+                TaskScheduler.Default);
         }
 
         private record ContinuationState(

src/Microsoft.Data.SqlClient/tests/UnitTests/Microsoft/Data/SqlClient/Utilities/AsyncHelperTest.cs

@@ -1304,9 +1304,13 @@ public void WaitForCompletion_DoesNotCreateUnobservedException()
                     onTimeout: null,
                     rethrowExceptions: true);
 
+                // - Task has timed out, simulate faulting task completion source
+                tcs.SetException(new Exception("late failure"));
+
                 // - Force collection of unobserved task
                 GC.Collect();
                 GC.WaitForPendingFinalizers();
+                GC.Collect();
 
                 // Assert
                 // - Make sure no unobserved tasks happened