Use Close instead of Dispose, because Dispose swallows the exception

Author: MichelZ

src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/DataStreamTest/DataStreamTest.cs

@@ -31,20 +31,14 @@ public static void RunAllTestsForSingleServer_NP()
             }
         }
 
-        [ActiveIssue("5540")]
+        //[ActiveIssue("5540")]
         [ConditionalFact(typeof(DataTestUtility), nameof(DataTestUtility.AreConnStringsSetup), nameof(DataTestUtility.IsNotAzureServer), Skip = "One test fails.")]
         public static void RunAllTestsForSingleServer_TCP()
         {
-            RunAllTestsForSingleServer(DataTestUtility.TCPConnectionString, false, false);
+            RunAllTestsForSingleServer(DataTestUtility.TCPConnectionString, false);
         }
 
-        [ConditionalFact(typeof(DataTestUtility), nameof(DataTestUtility.AreConnStringsSetup), nameof(DataTestUtility.IsNotAzureServer), nameof(DataTestUtility.IsNotNamedInstance))]
-        public static void RunAllTestsForSingleServer_TCP_DisableFlaky()
-        {
-            RunAllTestsForSingleServer(DataTestUtility.TCPConnectionString, false, true);
-        }
-        
-        // Synapse: The statement failed. Column 'foo' has a data type that cannot participate in a columnstore index.
+      // Synapse: The statement failed. Column 'foo' has a data type that cannot participate in a columnstore index.
         [ConditionalFact(typeof(DataTestUtility), nameof(DataTestUtility.AreConnStringsSetup), nameof(DataTestUtility.IsNotAzureSynapse))]
         public static async Task AsyncMultiPacketStreamRead()
         {
@@ -158,7 +152,7 @@ IF OBJECT_ID('dbo.{tableName}', 'U') IS NOT NULL
             return data;
         }
 
-        private static void RunAllTestsForSingleServer(string connectionString, bool usingNamePipes = false, bool disableFlakyTest = true)
+        private static void RunAllTestsForSingleServer(string connectionString, bool usingNamePipes = false)
         {
             RowBuffer(connectionString);
             InvalidRead(connectionString);
@@ -194,10 +188,7 @@ private static void RunAllTestsForSingleServer(string connectionString, bool usi
                     TimeoutDuringReadAsyncWithClosedReaderTest(connectionString);
                 }
 
-                if (!disableFlakyTest)
-                {
-                    NonFatalTimeoutDuringRead(connectionString);
-                }
+                NonFatalTimeoutDuringRead(connectionString);
             }
         }
 
@@ -2084,7 +2075,7 @@ private static void NonFatalTimeoutDuringRead(string connectionString)
 
                             // Close will now observe the stored timeout error
                             string errorMessage = SystemDataResourceManager.Instance.SQL_Timeout_Execution;
-                            DataTestUtility.AssertThrowsWrapper<SqlException>(reader.Dispose, errorMessage);
+                            DataTestUtility.AssertThrowsWrapper<SqlException>(reader.Close, errorMessage);
                         }
                     }
                 }