Skip to content

Commit 030d8f7

Browse files
committed
Applied suggestions by Davoud "Even though this PR suggests an update on ManagedSNI, we expect the same behavior on native SNI.".
1 parent 6c6f6dc commit 030d8f7

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ public static void ConnectManagedWithInstanceNameTest(bool useMultiSubnetFailove
8686

8787
// Note: This Unit test was tested in a domain-joined VM connecting to a remote
8888
// SQL Server using Kerberos in the same domain.
89-
[ConditionalFact(nameof(IsKerberosManagedSNI))]
89+
[ConditionalFact(nameof(IsKerberos))]
9090
public static void PortNumberInSPNTest()
9191
{
9292
string connStr = DataTestUtility.TCPConnectionString;
@@ -204,9 +204,9 @@ private static string GetSPNInfo(string datasource, out int out_port)
204204
return spnInfo;
205205
}
206206

207-
private static bool IsKerberosManagedSNI()
207+
private static bool IsKerberos()
208208
{
209-
return (DataTestUtility.AreConnStringsSetup() && DataTestUtility.IsUsingManagedSNI() && DataTestUtility.IsNotLocalhost() && DataTestUtility.IsKerberosTest && DataTestUtility.IsNotAzureServer() && DataTestUtility.IsNotAzureSynapse());
209+
return (DataTestUtility.AreConnStringsSetup() && DataTestUtility.IsNotLocalhost() && DataTestUtility.IsKerberosTest && DataTestUtility.IsNotAzureServer() && DataTestUtility.IsNotAzureSynapse());
210210
}
211211

212212
private static bool IsBrowserAlive(string browserHostname)

0 commit comments

Comments
 (0)