SNOW-3560694 Post-CI-stabilization cleanup - fix xUnit2002;xUnit2008;xUnit1026;xUnit3003;xUnit1026;xUnit1025 (#1390)

sfc-gh-mcepiga · web-flow · commit f26e1ff75bd2 · 2026-06-18T09:13:06.000+02:00
diff --git a/Snowflake.Data.Tests/IntegrationTests/ConnectionSinglePoolCacheAsyncIT.cs b/Snowflake.Data.Tests/IntegrationTests/ConnectionSinglePoolCacheAsyncIT.cs
@@ -393,7 +393,7 @@ public async Task TestCloseSessionAfterTimeout()
             await conn2.CloseAsync(CancellationToken.None);
         }
 
-        public static Task ConcurrentPoolingAsyncHelper(string connectionString, bool closeConnection, int tasksCount, int connectionsInTask, int abandonedConnectionsCount)
+        internal static Task ConcurrentPoolingAsyncHelper(string connectionString, bool closeConnection, int tasksCount, int connectionsInTask, int abandonedConnectionsCount)
         {
             var tasks = new Task[tasksCount + 1];
             for (int i = 0; i < tasksCount; i++)
diff --git a/Snowflake.Data.Tests/IntegrationTests/SFPutGetTest.cs b/Snowflake.Data.Tests/IntegrationTests/SFPutGetTest.cs
@@ -830,7 +830,7 @@ private static void VerifyFilesAreUploaded(DbConnection conn, ICollection<string
             }
         }
 
-        public async TaskOrValueTask DisposeAsync()
+        private async TaskOrValueTask DisposeAsync()
         {
             using (var conn = new SnowflakeDbConnection(_fixture.ConnectionString))
             {
@@ -857,7 +857,7 @@ public async TaskOrValueTask DisposeAsync()
             }
         }
 
-        public async TaskOrValueTask InitializeAsync()
+        private async TaskOrValueTask InitializeAsync()
         {
             // Base object's names on on worker thread id
             var suffix = Guid.NewGuid().ToString("N");
diff --git a/Snowflake.Data.Tests/PackageTests/MiniCorePackageIT.cs b/Snowflake.Data.Tests/PackageTests/MiniCorePackageIT.cs
@@ -32,7 +32,8 @@ public MiniCorePackageIT(ITestOutputHelper helper)
         private string _tempDir;
         private string _artifactsDir;
         private string _repoRoot;
-        public void Setup()
+
+        private void Setup()
         {
             _repoRoot = Path.GetFullPath(Path.Combine(AppContext.BaseDirectory, "../../../../"));
             _artifactsDir = Path.Combine(_repoRoot, "artifacts");
@@ -41,7 +42,8 @@ public void Setup()
             Directory.CreateDirectory(_artifactsDir);
             Directory.CreateDirectory(_tempDir);
         }
-        public void TearDown()
+
+        private void TearDown()
         {
             try { Directory.Delete(_tempDir, true); } catch { }
         }
diff --git a/Snowflake.Data.Tests/Snowflake.Data.Tests.csproj b/Snowflake.Data.Tests/Snowflake.Data.Tests.csproj
@@ -15,8 +15,6 @@
         <LangVersion>13</LangVersion>
         <TestingPlatformDotnetTestSupport>true</TestingPlatformDotnetTestSupport>
         <OutputType Condition="'$(OldXunit)' != 'True'">Exe</OutputType>
-        <!-- SNOW-3560694 !-->
-        <NoWarn>$(NoWarn);xUnit1013;xUnit2003;xUnit2002;xUnit2008;xUnit1026;xUnit3003;xUnit1026;xUnit1025</NoWarn>
     </PropertyGroup>
     <ItemGroup>
         <Content Include="wiremock\**\*.*">
diff --git a/Snowflake.Data.Tests/UnitTests/Authenticator/ChallengeProviderTest.cs b/Snowflake.Data.Tests/UnitTests/Authenticator/ChallengeProviderTest.cs
@@ -18,7 +18,7 @@ public void TestGenerateState()
 
             // assert
             Assert.Equal(32, state.Length);
-            Assert.True(_onlyDigitsOrNumbers.IsMatch(state));
+            Assert.Matches(_onlyDigitsOrNumbers, state);
         }
 
         [SFFact]
@@ -29,7 +29,7 @@ public void TestGenerateCodeVerifier()
 
             // assert
             Assert.InRange(codeVerifier.Length, 43, 128);
-            Assert.True(_onlyDigitsOrNumbers.IsMatch(codeVerifier));
+            Assert.Matches(_onlyDigitsOrNumbers, codeVerifier);
         }
     }
 }
diff --git a/Snowflake.Data.Tests/UnitTests/Configuration/EasyLoggingConfigParserTest.cs b/Snowflake.Data.Tests/UnitTests/Configuration/EasyLoggingConfigParserTest.cs
@@ -31,7 +31,7 @@ public EasyLoggingConfigParserTest(EasyLoggingConfigParserTestFixture fixture)
         {
         }
 
-        public static void BeforeAll()
+        internal static void BeforeAll()
         {
             if (!Directory.Exists(s_workingDirectory))
             {
diff --git a/Snowflake.Data.Tests/UnitTests/CredentialManager/SFCredentialManagerFileImplTest.cs b/Snowflake.Data.Tests/UnitTests/CredentialManager/SFCredentialManagerFileImplTest.cs
@@ -36,7 +36,8 @@ public SFCredentialManagerFileImplTest()
         private static readonly string s_customLockPath = Path.Combine(CustomJsonDir, SFCredentialManagerFileStorage.CredentialCacheLockName);
 
         private const int UserId = 1;
-        public void SetUp()
+
+        private void SetUp()
         {
             t_fileOperations = new Mock<FileOperations>();
             t_directoryOperations = new Mock<DirectoryOperations>();
diff --git a/Snowflake.Data.Tests/UnitTests/CredentialManager/SnowflakeCredentialManagerFactoryTest.cs b/Snowflake.Data.Tests/UnitTests/CredentialManager/SnowflakeCredentialManagerFactoryTest.cs
@@ -9,7 +9,7 @@ namespace Snowflake.Data.Tests.UnitTests.CredentialManager
 {
     public class SnowflakeCredentialManagerFactoryTest : IDisposable
     {
-        public void TearDown()
+        private void TearDown()
         {
             SnowflakeCredentialManagerFactory.UseDefaultCredentialManager();
         }
diff --git a/Snowflake.Data.Tests/UnitTests/HttpUtilTest.cs b/Snowflake.Data.Tests/UnitTests/HttpUtilTest.cs
@@ -209,6 +209,9 @@ public void TestDefaultConnectionLimitIsNotChangedWhenOver50()
         public void TestDefaultConnectionLimitIsChangedToDefaultWhenUnder50()
         {
             // arrange
+#if NET9_0_OR_GREATER
+            Skip.When(true, "TODO SNOW-3662960");
+#endif
             var originalLimit = ServicePointManager.DefaultConnectionLimit;
             ServicePointManager.DefaultConnectionLimit = 49;
 
diff --git a/Snowflake.Data.Tests/UnitTests/Logger/EasyLoggingStarterTest.cs b/Snowflake.Data.Tests/UnitTests/Logger/EasyLoggingStarterTest.cs
@@ -142,18 +142,19 @@ public void TestThatThrowsErrorWhenLogPathIsNotSetAndHomeDirectoryThrowsAnExcept
 
 
         [SFTheory(SkipCondition.SkipOnWindows)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead | FileAccessPermissions.GroupExecute)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead | FileAccessPermissions.GroupWrite | FileAccessPermissions.GroupExecute)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupReadWriteExecute | FileAccessPermissions.OtherRead | FileAccessPermissions.OtherExecute)]
-        [InlineData(FileAccessPermissions.AllPermissions)]
-        [InlineData(FileAccessPermissions.GroupReadWriteExecute)]
-        [InlineData(FileAccessPermissions.OtherReadWriteExecute)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.OtherRead)]
-        [InlineData(FileAccessPermissions.UserRead | FileAccessPermissions.UserWrite)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead)]
-        public void TestThatThrowsErrorWhenLogDirectoryHasInvalidPermissions(FileAccessPermissions invalidPermissions)
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead | FileAccessPermissions.GroupExecute, "User RWX + Group RX")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead | FileAccessPermissions.GroupWrite | FileAccessPermissions.GroupExecute, "User RWX + Group RWX")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupReadWriteExecute | FileAccessPermissions.OtherRead | FileAccessPermissions.OtherExecute, "User RWX + Group RWX + Other RX")]
+        [InlineData(FileAccessPermissions.AllPermissions, "All")]
+        [InlineData(FileAccessPermissions.GroupReadWriteExecute, "Group RWX")]
+        [InlineData(FileAccessPermissions.OtherReadWriteExecute, "Other RWX")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.OtherRead, "User RWX + Other R")]
+        [InlineData(FileAccessPermissions.UserRead | FileAccessPermissions.UserWrite, "User RW")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead, "User RWX + Group R")]
+        public void TestThatThrowsErrorWhenLogDirectoryHasInvalidPermissions(FileAccessPermissions invalidPermissions, string log)
         {
             // arrange
+            Console.WriteLine($@"Executing {nameof(TestThatThrowsErrorWhenLogDirectoryHasInvalidPermissions)} with {log}..");
             t_easyLoggingProvider
                 .Setup(provider => provider.ProvideConfig(ConfigPath))
                 .Returns(s_configWithInfoLevel);
diff --git a/Snowflake.Data.Tests/UnitTests/Revocation/CrlCacheManagerTest.cs b/Snowflake.Data.Tests/UnitTests/Revocation/CrlCacheManagerTest.cs
@@ -24,7 +24,8 @@ public CrlCacheManagerTest()
 
         const string CrlUrl1 = "http://snowflakecomputing.com/crl1.crl";
         const string CrlUrl2 = "http://snowflakecomputing.com/crl2.crl";
-        public void SetUp()
+
+        private void SetUp()
         {
             var cacheDir = GetCrlCacheDirectory();
             if (Directory.Exists(cacheDir))
diff --git a/Snowflake.Data.Tests/UnitTests/Revocation/CrlRepositoryTest.cs b/Snowflake.Data.Tests/UnitTests/Revocation/CrlRepositoryTest.cs
@@ -12,7 +12,7 @@ public CrlRepositoryTest()
             SetUp();
         }
 
-        public void SetUp()
+        private void SetUp()
         {
             Environment.SetEnvironmentVariable("SF_CRL_CACHE_REMOVAL_DELAY", null);
         }
diff --git a/Snowflake.Data.Tests/UnitTests/Revocation/CrlTest.cs b/Snowflake.Data.Tests/UnitTests/Revocation/CrlTest.cs
@@ -158,6 +158,7 @@ public void TestIsRevokedIsCaseInsensitive(string serialNumber)
             Assert.True(isRevoked);
         }
 
+#pragma warning disable xUnit1026
         [SFTheory]
         [InlineData(true)]
         [InlineData(false)]
@@ -215,6 +216,7 @@ public void TestIsRevokedHandlesLeadingZeroBytePadding(bool serialHasHighBitSet)
             Assert.True(isRevoked, "Certificate should be found in the revocation list");
 #endif
         }
+#pragma warning restore xUnit1026
 
 #if NET8_0_OR_GREATER
         private static X509Certificate2 BuildSelfSignedCertificate(int runNo)
diff --git a/Snowflake.Data.Tests/UnitTests/Revocation/FileCrlCacheTest.cs b/Snowflake.Data.Tests/UnitTests/Revocation/FileCrlCacheTest.cs
@@ -464,11 +464,10 @@ public void TestWithMocksUnixSaveCrlShouldFixIncorrectPermissions()
         }
 
         [SFTheory(SkipCondition.SkipOnWindows)]
-        [InlineData(1, 0, 0, 0)]
-        [InlineData(0, 1, 0, 0)]
-        [InlineData(0, 0, 1, 0)]
-        [InlineData(0, 0, 1, 1)]
-        public void TestWithMocksUnixCrlShouldNotBeSavedIfFailedToSetSecurePermissions(long changeDirOwnerResult, long changeDirPermissionsResult, long changeFileOwnerResult, long changeFilePermissionsResult)
+        [InlineData(1, 0, 0)]
+        [InlineData(0, 1, 0)]
+        [InlineData(0, 0, 1)]
+        public void TestWithMocksUnixCrlShouldNotBeSavedIfFailedToSetSecurePermissions(long changeDirOwnerResult, long changeDirPermissionsResult, long changeFileOwnerResult)
         {
             // arrange
             var crl = CreateCrl();
diff --git a/Snowflake.Data.Tests/UnitTests/SFDbParameterCollectionTest.cs b/Snowflake.Data.Tests/UnitTests/SFDbParameterCollectionTest.cs
@@ -164,9 +164,8 @@ public void TestDbParameterCollectionIndexOfName(SFDataType SFDataType)
             Assert.Equal(PARAM_COUNT - 1, _parameterCollection.IndexOf(paramName));
         }
 
-        [SFTheory]
-        [MemberData(nameof(AllSFDataTypes))]
-        public void TestDbParameterCollectionIndexOfNameNotExists(SFDataType SFDataType)
+        [SFFact]
+        public void TestDbParameterCollectionIndexOfNameNotExists()
         {
             int expectedParameterIndex = -1;
             string paramName = "1";
diff --git a/Snowflake.Data.Tests/UnitTests/SFFileTransferAgentTests.cs b/Snowflake.Data.Tests/UnitTests/SFFileTransferAgentTests.cs
@@ -78,7 +78,7 @@ public void Dispose()
             AfterEachTest();
         }
 
-        public void BeforeEachTest()
+        private void BeforeEachTest()
         {
             // Base object's names on worker thread id
             var threadSuffix = Thread.CurrentThread.ManagedThreadId.ToString()?.Replace('#', '_');
@@ -127,7 +127,8 @@ public void BeforeEachTest()
 
             _session = new SFSession(ConnectionStringMock, new SessionPropertiesContext());
         }
-        public void AfterEachTest()
+
+        private void AfterEachTest()
         {
             // Delete stage directory recursively
             if (Directory.Exists(t_locationStage))
@@ -665,7 +666,7 @@ public void TestGetFilePathWithSpacesAndWithSingleQuotesFromPutCommand()
             TestGetFilePathFromPutCommand("PUT 'file://" + s_filePathWithSpaces + "'  @TestStage", s_filePathWithSpaces);
         }
 
-        public void TestGetFilePathFromPutCommand(string query, string expectedFilePath)
+        private void TestGetFilePathFromPutCommand(string query, string expectedFilePath)
         {
             var actualFilePath = SFFileTransferAgent.getFilePathFromPutCommand(query);
             Assert.Equal(expectedFilePath, actualFilePath);
diff --git a/Snowflake.Data.Tests/UnitTests/SecretDetectorTest.cs b/Snowflake.Data.Tests/UnitTests/SecretDetectorTest.cs
@@ -16,7 +16,7 @@ public SecretDetectorTest()
             mask = SecretDetector.MaskSecrets(null);
         }
 
-        public void BasicMasking(string text)
+        private void BasicMasking(string text)
         {
             mask = SecretDetector.MaskSecrets(text);
             Assert.False(mask.isMasked);
@@ -51,7 +51,7 @@ public void TestExceptionInMasking()
             Assert.Equal("Test exception", mask.errStr);
         }
 
-        public void BasicMasking(string text, string expectedText)
+        private void BasicMasking(string text, string expectedText)
         {
             mask = SecretDetector.MaskSecrets(text);
             Assert.True(mask.isMasked);
diff --git a/Snowflake.Data.Tests/UnitTests/Session/SessionPoolTest.cs b/Snowflake.Data.Tests/UnitTests/Session/SessionPoolTest.cs
@@ -122,7 +122,7 @@ public void TestPoolIdentificationBasedOnInternalId()
             var poolIdentification = pool.PoolIdentificationBasedOnInternalId;
 
             // assert
-            Assert.True(poolIdRegex.IsMatch(poolIdentification));
+            Assert.Matches(poolIdRegex, poolIdentification);
         }
 
         [SFFact]
diff --git a/Snowflake.Data.Tests/UnitTests/Session/SessionPropertiesWithDefaultValuesExtractorTest.cs b/Snowflake.Data.Tests/UnitTests/Session/SessionPropertiesWithDefaultValuesExtractorTest.cs
@@ -27,11 +27,8 @@ public void TestReturnExtractedValue()
             Assert.Equal(15, value);
         }
 
-        [SFTheory]
-        [InlineData(false)]
-        [InlineData(true)]
-        public void TestReturnDefaultValueWhenValueIsMissing(
-            bool failOnWrongValue)
+        [SFFact]
+        public void TestReturnDefaultValueWhenValueIsMissing()
         {
             // arrange
             var properties = SFSessionProperties.ParseConnectionString($"account=test;user=test;password=test", new SessionPropertiesContext());
diff --git a/Snowflake.Data.Tests/UnitTests/Telemetry/ActivityStarterTest.cs b/Snowflake.Data.Tests/UnitTests/Telemetry/ActivityStarterTest.cs
@@ -102,7 +102,6 @@ public void TestSetExceptionSetsErrorStatusAndAddsEvent()
         Assert.Equal("ERROR", _sessionIdCapturedActivitiesMap[session.sessionId].GetTagItem(TelemetryTags.StatusCode));
 
         var exceptionEvent = _sessionIdCapturedActivitiesMap[session.sessionId].Events.FirstOrDefault(e => e.Name == "exception");
-        Assert.NotNull(exceptionEvent);
         Assert.Equal(typeof(AbandonedMutexException).FullName,
             exceptionEvent.Tags.First(t => t.Key == TelemetryTags.Exception).Value);
     }
@@ -268,7 +267,7 @@ public void TestAddTelemetryEventAddsNamedEvent()
 
         Assert.True(_sessionIdCapturedActivitiesMap.ContainsKey(session.sessionId));
         var evt = _sessionIdCapturedActivitiesMap[session.sessionId].Events.FirstOrDefault(e => e.Name == "TestEvent");
-        Assert.NotNull(evt);
+        Assert.NotEqual(default, evt);
     }
 
     [SFFact]
@@ -285,7 +284,6 @@ public void TestAddTelemetryEventWithTagsStoresTagsOnEvent()
 
         Assert.True(_sessionIdCapturedActivitiesMap.ContainsKey(session.sessionId));
         var evt = _sessionIdCapturedActivitiesMap[session.sessionId].Events.FirstOrDefault(e => e.Name == "StepComplete");
-        Assert.NotNull(evt);
         Assert.Equal("validation", evt.Tags.First(t => t.Key == "step.name").Value);
         Assert.Equal(42, evt.Tags.First(t => t.Key == "step.duration_ms").Value);
     }
@@ -302,7 +300,6 @@ public void TestAddTelemetryEventWithNoTagsCreatesEventWithEmptyTags()
 
         Assert.True(_sessionIdCapturedActivitiesMap.ContainsKey(session.sessionId));
         var evt = _sessionIdCapturedActivitiesMap[session.sessionId].Events.FirstOrDefault(e => e.Name == "SimpleEvent");
-        Assert.NotNull(evt);
         Assert.False(evt.Tags.Any());
     }
 
diff --git a/Snowflake.Data.Tests/UnitTests/Tools/DirectoryUnixInformationTest.cs b/Snowflake.Data.Tests/UnitTests/Tools/DirectoryUnixInformationTest.cs
@@ -1,3 +1,4 @@
+using System;
 using System.IO;
 using Mono.Unix;
 using Xunit;
@@ -13,13 +14,14 @@ public class DirectoryUnixInformationTest
         static readonly string s_directoryFullName = Path.Combine(Path.GetTempPath(), Path.GetRandomFileName());
 
         [SFTheory(SkipCondition.SkipOnWindows)]
-        [InlineData(FileAccessPermissions.UserWrite)]
-        [InlineData(FileAccessPermissions.UserRead)]
-        [InlineData(FileAccessPermissions.UserExecute)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute)]
-        public void TestSafeDirectory(FileAccessPermissions securePermissions)
+        [InlineData(FileAccessPermissions.UserWrite, "User W")]
+        [InlineData(FileAccessPermissions.UserRead, "User R")]
+        [InlineData(FileAccessPermissions.UserExecute, "User X")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute, "User RWX")]
+        public void TestSafeDirectory(FileAccessPermissions securePermissions, string log)
         {
             // arrange
+            Console.WriteLine($@"Executing {nameof(TestSafeDirectory)} with {log}..");
             var dirInfo = new DirectoryUnixInformation(s_directoryFullName, true, securePermissions, UserId);
 
             // act
@@ -30,11 +32,12 @@ public void TestSafeDirectory(FileAccessPermissions securePermissions)
         }
 
         [SFTheory(SkipCondition.SkipOnWindows)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.OtherRead)]
-        public void TestUnsafePermissions(FileAccessPermissions unsecurePermissions)
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead, "User RW + Group R")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.OtherRead, "User RWX + Other R")]
+        public void TestUnsafePermissions(FileAccessPermissions unsecurePermissions, string log)
         {
             // arrange
+            Console.WriteLine($@"Executing {nameof(TestUnsafePermissions)} with {log}..");
             var dirInfo = new DirectoryUnixInformation(s_directoryFullName, true, unsecurePermissions, UserId);
 
             // act
@@ -58,12 +61,13 @@ public void TestSafeExactlyDirectory()
         }
 
         [SFTheory(SkipCondition.SkipOnWindows)]
-        [InlineData(FileAccessPermissions.UserRead)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead)]
-        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.OtherRead)]
-        public void TestUnsafeExactlyPermissions(FileAccessPermissions unsecurePermissions)
+        [InlineData(FileAccessPermissions.UserRead, "User R")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.GroupRead, "User RWX + Group R")]
+        [InlineData(FileAccessPermissions.UserReadWriteExecute | FileAccessPermissions.OtherRead, "User RWX + Other R")]
+        public void TestUnsafeExactlyPermissions(FileAccessPermissions unsecurePermissions, string log)
         {
             // arrange
+            Console.WriteLine($@"Executing {nameof(TestUnsafePermissions)} with {log}..");
             var dirInfo = new DirectoryUnixInformation(s_directoryFullName, true, unsecurePermissions, UserId);
 
             // act
diff --git a/Snowflake.Data.Tests/UnitTests/Tools/FileOperationsUnixTest.cs b/Snowflake.Data.Tests/UnitTests/Tools/FileOperationsUnixTest.cs
@@ -26,7 +26,8 @@ public FileOperationsUnixTest()
         private static readonly string s_workingDirectory = Path.Combine(TempUtil.GetTempPath(), s_relativeWorkingDirectory);
         private static readonly string s_content = "random text";
         private static readonly string s_fileName = "testfile";
-        public static void Before()
+
+        private static void Before()
         {
             if (!Directory.Exists(s_workingDirectory))
             {
@@ -35,7 +36,8 @@ public static void Before()
 
             s_fileOperations = new FileOperations();
         }
-        public static void After()
+
+        private static void After()
         {
             Directory.Delete(s_workingDirectory, true);
         }
diff --git a/Snowflake.Data.Tests/UnitTests/Tools/FileOperationsWindowsTest.cs b/Snowflake.Data.Tests/UnitTests/Tools/FileOperationsWindowsTest.cs
diff --git a/Snowflake.Data.Tests/UnitTests/Tools/PlatformDetectionTest.cs b/Snowflake.Data.Tests/UnitTests/Tools/PlatformDetectionTest.cs
diff --git a/Snowflake.Data.Tests/UnitTests/Tools/SpcsTokenProviderTest.cs b/Snowflake.Data.Tests/UnitTests/Tools/SpcsTokenProviderTest.cs
diff --git a/Snowflake.Data.Tests/Util/SFFactAttribute.cs b/Snowflake.Data.Tests/Util/SFFactAttribute.cs
diff --git a/Snowflake.Data/Client/SnowflakeDbException.cs b/Snowflake.Data/Client/SnowflakeDbException.cs
diff --git a/Snowflake.Data/Core/Authenticator/WorkflowIdentity/AwsSdkWrapper.cs b/Snowflake.Data/Core/Authenticator/WorkflowIdentity/AwsSdkWrapper.cs
diff --git a/Snowflake.Data/Core/Authenticator/WorkflowIdentity/WorkflowIdentityAwsAttestationRetriever.cs b/Snowflake.Data/Core/Authenticator/WorkflowIdentity/WorkflowIdentityAwsAttestationRetriever.cs
diff --git a/Snowflake.Data/Core/Authenticator/WorkflowIdentity/WorkflowIdentityGcpAttestationRetriever.cs b/Snowflake.Data/Core/Authenticator/WorkflowIdentity/WorkflowIdentityGcpAttestationRetriever.cs
diff --git a/Snowflake.Data/Core/FileTransfer/SFFileTransferAgent.cs b/Snowflake.Data/Core/FileTransfer/SFFileTransferAgent.cs
diff --git a/Snowflake.Data/Core/FileTransfer/StorageClient/SFGCSClient.cs b/Snowflake.Data/Core/FileTransfer/StorageClient/SFGCSClient.cs
diff --git a/Snowflake.Data/Core/HttpUtil.cs b/Snowflake.Data/Core/HttpUtil.cs

Original file line number	Diff line number	Diff line change
`@@ -393,7 +393,7 @@ public async Task TestCloseSessionAfterTimeout()`
`393`	`393`	`await conn2.CloseAsync(CancellationToken.None);`
`394`	`394`	`}`
`395`	`395`
`396`		`- public static Task ConcurrentPoolingAsyncHelper(string connectionString, bool closeConnection, int tasksCount, int connectionsInTask, int abandonedConnectionsCount)`
	`396`	`+ internal static Task ConcurrentPoolingAsyncHelper(string connectionString, bool closeConnection, int tasksCount, int connectionsInTask, int abandonedConnectionsCount)`
`397`	`397`	`{`
`398`	`398`	`var tasks = new Task[tasksCount + 1];`
`399`	`399`	`for (int i = 0; i < tasksCount; i++)`
Original file line number	Diff line number	Diff line change
`@@ -830,7 +830,7 @@ private static void VerifyFilesAreUploaded(DbConnection conn, ICollection<string`
`830`	`830`	`}`
`831`	`831`	`}`
`832`	`832`
`833`		`- public async TaskOrValueTask DisposeAsync()`
	`833`	`+ private async TaskOrValueTask DisposeAsync()`
`834`	`834`	`{`
`835`	`835`	`using (var conn = new SnowflakeDbConnection(_fixture.ConnectionString))`
`836`	`836`	`{`
`@@ -857,7 +857,7 @@ public async TaskOrValueTask DisposeAsync()`
`857`	`857`	`}`
`858`	`858`	`}`
`859`	`859`
`860`		`- public async TaskOrValueTask InitializeAsync()`
	`860`	`+ private async TaskOrValueTask InitializeAsync()`
`861`	`861`	`{`
`862`	`862`	`// Base object's names on on worker thread id`
`863`	`863`	`var suffix = Guid.NewGuid().ToString("N");`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,8 @@ public MiniCorePackageIT(ITestOutputHelper helper)`
`32`	`32`	`private string _tempDir;`
`33`	`33`	`private string _artifactsDir;`
`34`	`34`	`private string _repoRoot;`
`35`		`- public void Setup()`
	`35`	`+`
	`36`	`+ private void Setup()`
`36`	`37`	`{`
`37`	`38`	`_repoRoot = Path.GetFullPath(Path.Combine(AppContext.BaseDirectory, "../../../../"));`
`38`	`39`	`_artifactsDir = Path.Combine(_repoRoot, "artifacts");`
`@@ -41,7 +42,8 @@ public void Setup()`
`41`	`42`	`Directory.CreateDirectory(_artifactsDir);`
`42`	`43`	`Directory.CreateDirectory(_tempDir);`
`43`	`44`	`}`
`44`		`- public void TearDown()`
	`45`	`+`
	`46`	`+ private void TearDown()`
`45`	`47`	`{`
`46`	`48`	`try { Directory.Delete(_tempDir, true); } catch { }`
`47`	`49`	`}`
Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ public void TestGenerateState()`
`18`	`18`
`19`	`19`	`// assert`
`20`	`20`	`Assert.Equal(32, state.Length);`
`21`		`- Assert.True(_onlyDigitsOrNumbers.IsMatch(state));`
	`21`	`+ Assert.Matches(_onlyDigitsOrNumbers, state);`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`[SFFact]`
`@@ -29,7 +29,7 @@ public void TestGenerateCodeVerifier()`
`29`	`29`
`30`	`30`	`// assert`
`31`	`31`	`Assert.InRange(codeVerifier.Length, 43, 128);`
`32`		`- Assert.True(_onlyDigitsOrNumbers.IsMatch(codeVerifier));`
	`32`	`+ Assert.Matches(_onlyDigitsOrNumbers, codeVerifier);`
`33`	`33`	`}`
`34`	`34`	`}`
`35`	`35`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public EasyLoggingConfigParserTest(EasyLoggingConfigParserTestFixture fixture)`
`31`	`31`	`{`
`32`	`32`	`}`
`33`	`33`
`34`		`- public static void BeforeAll()`
	`34`	`+ internal static void BeforeAll()`
`35`	`35`	`{`
`36`	`36`	`if (!Directory.Exists(s_workingDirectory))`
`37`	`37`	`{`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,8 @@ public SFCredentialManagerFileImplTest()`
`36`	`36`	`private static readonly string s_customLockPath = Path.Combine(CustomJsonDir, SFCredentialManagerFileStorage.CredentialCacheLockName);`
`37`	`37`
`38`	`38`	`private const int UserId = 1;`
`39`		`- public void SetUp()`
	`39`	`+`
	`40`	`+ private void SetUp()`
`40`	`41`	`{`
`41`	`42`	`t_fileOperations = new Mock<FileOperations>();`
`42`	`43`	`t_directoryOperations = new Mock<DirectoryOperations>();`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ namespace Snowflake.Data.Tests.UnitTests.CredentialManager`
`9`	`9`	`{`
`10`	`10`	`public class SnowflakeCredentialManagerFactoryTest : IDisposable`
`11`	`11`	`{`
`12`		`- public void TearDown()`
	`12`	`+ private void TearDown()`
`13`	`13`	`{`
`14`	`14`	`SnowflakeCredentialManagerFactory.UseDefaultCredentialManager();`
`15`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -209,6 +209,9 @@ public void TestDefaultConnectionLimitIsNotChangedWhenOver50()`
`209`	`209`	`public void TestDefaultConnectionLimitIsChangedToDefaultWhenUnder50()`
`210`	`210`	`{`
`211`	`211`	`// arrange`
	`212`	`+#if NET9_0_OR_GREATER`
	`213`	`+ Skip.When(true, "TODO SNOW-3662960");`
	`214`	`+#endif`
`212`	`215`	`var originalLimit = ServicePointManager.DefaultConnectionLimit;`
`213`	`216`	`ServicePointManager.DefaultConnectionLimit = 49;`
`214`	`217`