Remove Parallel stuff

MichelZ · MichelZ · commit 0a788b9bb87d · 2024-11-17T18:23:27.000+01:00
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/AlwaysEncrypted/EnclaveAzureDatabaseTests.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/AlwaysEncrypted/EnclaveAzureDatabaseTests.cs
@@ -11,7 +11,6 @@
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests.AlwaysEncrypted
 {
     // This test class is for internal use only
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public sealed class EnclaveAzureDatabaseTests : IDisposable
     {
         private ColumnMasterKey akvColumnMasterKey;
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/DoNotParallelizeTestCollection.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/DoNotParallelizeTestCollection.cs
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/Microsoft.Data.SqlClient.ManualTesting.Tests.csproj b/src/Microsoft.Data.SqlClient/tests/ManualTests/Microsoft.Data.SqlClient.ManualTesting.Tests.csproj
@@ -290,7 +290,6 @@
     <Compile Include="DataCommon\ProxyServer.cs" />
     <Compile Include="DataCommon\SqlClientCustomTokenCredential.cs" />
     <Compile Include="DataCommon\SystemDataResourceManager.cs" />
-    <Compile Include="DoNotParallelizeTestCollection.cs" />
     <Compile Include="SQL\Common\AsyncDebugScope.cs" />
     <Compile Include="SQL\Common\ConnectionPoolWrapper.cs" />
     <Compile Include="SQL\Common\InternalConnectionWrapper.cs" />
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/ProviderAgnostic/MultipleResultsTest/MultipleResultsTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/ProviderAgnostic/MultipleResultsTest/MultipleResultsTest.cs
@@ -12,7 +12,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class MultipleResultsTest
     {
         private StringBuilder _globalBuilder = new StringBuilder();
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionPoolTest/ConnectionPoolTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionPoolTest/ConnectionPoolTest.cs
@@ -29,7 +29,6 @@ public IEnumerator<object[]> GetEnumerator()
     }
 
     // TODO Synapse: Fix these tests for Azure Synapse.
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public static class ConnectionPoolTest
     {
         /// <summary>
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionPoolTest/PoolBlockPeriodTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionPoolTest/PoolBlockPeriodTest.cs
@@ -7,7 +7,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class PoolBlockPeriodTest
     {
         private const string AzureEndpointSample = "nonexistent.database.windows.net";
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionPoolTest/TransactionPoolTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionPoolTest/TransactionPoolTest.cs
@@ -4,7 +4,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public static class TransactionPoolTest
     {
         /// <summary>
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionTestWithSSLCert/CertificateTestWithTdsServer.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectionTestWithSSLCert/CertificateTestWithTdsServer.cs
@@ -17,7 +17,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class CertificateTestWithTdsServer : IDisposable
     {
         private static readonly string s_fullPathToPowershellScript = Path.Combine(Directory.GetCurrentDirectory(), "makepfxcert.ps1");
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ExceptionTest/ConnectionExceptionTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ExceptionTest/ConnectionExceptionTest.cs
@@ -7,7 +7,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class ConnectionExceptionTest
     {
         // test connection string
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/MARSSessionPoolingTest/MARSSessionPoolingTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/MARSSessionPoolingTest/MARSSessionPoolingTest.cs
@@ -9,7 +9,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public static class MARSSessionPoolingTest
     {
         private const string COMMAND_STATUS = "select count(*) as ConnectionCount, @@spid as spid from sys.dm_exec_connections where session_id=@@spid and net_transport='Session'; " +
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/MARSTest/MARSTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/MARSTest/MARSTest.cs
@@ -688,7 +688,7 @@ FROM [{table}] AS [l]
                 }
                 catch (Exception e)
                 {
-                    Assert.Fail("CRITIAL: Test should not fail randomly. Exception occurred: " + e.Message);
+                    Assert.Fail("CRITICAL: Test should not fail randomly. Exception occurred: " + e.Message);
                 }
                 finally
                 {
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ParameterTest/TvpTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ParameterTest/TvpTest.cs
@@ -21,7 +21,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class TvpTest
     {
         private const string TvpName = "@tvp";
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/RetryLogic/SqlCommandReliabilityTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/RetryLogic/SqlCommandReliabilityTest.cs
@@ -12,7 +12,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class SqlCommandReliabilityTest : IDisposable
     {
         private const int ConcurrentParallelExecutions = 3;
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/SqlNotificationTest/SqlNotificationTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/SqlNotificationTest/SqlNotificationTest.cs
@@ -9,7 +9,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public sealed class SqlNotificationTest : IDisposable
     {
         // Misc constants
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/TransactionTest/TransactionEnlistmentTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/TransactionTest/TransactionEnlistmentTest.cs
@@ -10,7 +10,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class TransactionEnlistmentTest
     {
         // TODO Synapse: Cannot find data type 'text'.
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/TracingTests/EventCounterTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/TracingTests/EventCounterTest.cs
@@ -12,7 +12,6 @@ namespace Microsoft.Data.SqlClient.ManualTesting.Tests
     /// <summary>
     /// This unit test is just valid for .NetCore 3.0 and above
     /// </summary>
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class EventCounterTest
     {
         public EventCounterTest()
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/TracingTests/EventSourceTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/TracingTests/EventSourceTest.cs
@@ -7,7 +7,6 @@
 
 namespace Microsoft.Data.SqlClient.ManualTesting.Tests
 {
-    [Collection(nameof(DoNotParallelizeTestCollection))]
     public class EventSourceTest
     {
         [ConditionalFact(typeof(DataTestUtility), nameof(DataTestUtility.AreConnStringsSetup))]

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,6 @@`
`11`	`11`	`namespace Microsoft.Data.SqlClient.ManualTesting.Tests.AlwaysEncrypted`
`12`	`12`	`{`
`13`	`13`	`// This test class is for internal use only`
`14`		`- [Collection(nameof(DoNotParallelizeTestCollection))]`
`15`	`14`	`public sealed class EnclaveAzureDatabaseTests : IDisposable`
`16`	`15`	`{`
`17`	`16`	`private ColumnMasterKey akvColumnMasterKey;`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,6 @@`
`12`	`12`
`13`	`13`	`namespace Microsoft.Data.SqlClient.ManualTesting.Tests`
`14`	`14`	`{`
`15`		`- [Collection(nameof(DoNotParallelizeTestCollection))]`
`16`	`15`	`public class MultipleResultsTest`
`17`	`16`	`{`
`18`	`17`	`private StringBuilder _globalBuilder = new StringBuilder();`
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,6 @@ public IEnumerator<object[]> GetEnumerator()`
`29`	`29`	`}`
`30`	`30`
`31`	`31`	`// TODO Synapse: Fix these tests for Azure Synapse.`
`32`		`- [Collection(nameof(DoNotParallelizeTestCollection))]`
`33`	`32`	`public static class ConnectionPoolTest`
`34`	`33`	`{`
`35`	`34`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,6 @@`
`7`	`7`
`8`	`8`	`namespace Microsoft.Data.SqlClient.ManualTesting.Tests`
`9`	`9`	`{`
`10`		`- [Collection(nameof(DoNotParallelizeTestCollection))]`
`11`	`10`	`public class PoolBlockPeriodTest`
`12`	`11`	`{`
`13`	`12`	`private const string AzureEndpointSample = "nonexistent.database.windows.net";`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,6 @@`
`4`	`4`
`5`	`5`	`namespace Microsoft.Data.SqlClient.ManualTesting.Tests`
`6`	`6`	`{`
`7`		`- [Collection(nameof(DoNotParallelizeTestCollection))]`
`8`	`7`	`public static class TransactionPoolTest`
`9`	`8`	`{`
`10`	`9`	`/// <summary>`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,6 @@`
`17`	`17`
`18`	`18`	`namespace Microsoft.Data.SqlClient.ManualTesting.Tests`
`19`	`19`	`{`
`20`		`- [Collection(nameof(DoNotParallelizeTestCollection))]`
`21`	`20`	`public class CertificateTestWithTdsServer : IDisposable`
`22`	`21`	`{`
`23`	`22`	`private static readonly string s_fullPathToPowershellScript = Path.Combine(Directory.GetCurrentDirectory(), "makepfxcert.ps1");`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,6 @@`
`9`	`9`
`10`	`10`	`namespace Microsoft.Data.SqlClient.ManualTesting.Tests`
`11`	`11`	`{`
`12`		`- [Collection(nameof(DoNotParallelizeTestCollection))]`
`13`	`12`	`public static class MARSSessionPoolingTest`
`14`	`13`	`{`
`15`	`14`	`private const string COMMAND_STATUS = "select count(*) as ConnectionCount, @@spid as spid from sys.dm_exec_connections where session_id=@@spid and net_transport='Session'; " +`