Test | Do not check IsUsingManagedSNI in NonAzureNoProtocolConnectionTestWindows for netfx #1410 (#1533)

Kaur-Parminder · web-flow · commit be9731cb3184 · 2022-05-05T10:25:03.000-07:00
diff --git a/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectivityTests/TcpDefaultForAzureTest.cs b/src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectivityTests/TcpDefaultForAzureTest.cs
@@ -42,7 +42,11 @@ static TcpDefaultForAzureTest()
         public static void NonAzureNoProtocolConnectionTestWindows()
         {
             builder.DataSource = InvalidHostname;
+#if NETFRAMEWORK
+            CheckConnectionFailure(builder.ConnectionString, NP);
+#else
             CheckConnectionFailure(builder.ConnectionString, DataTestUtility.IsUsingManagedSNI() ? TCP : NP);
+#endif
         }
 
         [ConditionalFact(typeof(DataTestUtility), nameof(DataTestUtility.AreConnStringsSetup), nameof(DataTestUtility.IsNotAzureSynapse))]

Original file line number	Diff line number	Diff line change
`@@ -42,7 +42,11 @@ static TcpDefaultForAzureTest()`
`42`	`42`	`public static void NonAzureNoProtocolConnectionTestWindows()`
`43`	`43`	`{`
`44`	`44`	`builder.DataSource = InvalidHostname;`
	`45`	`+#if NETFRAMEWORK`
	`46`	`+ CheckConnectionFailure(builder.ConnectionString, NP);`
	`47`	`+#else`
`45`	`48`	`CheckConnectionFailure(builder.ConnectionString, DataTestUtility.IsUsingManagedSNI() ? TCP : NP);`
	`49`	`+#endif`
`46`	`50`	`}`
`47`	`51`
`48`	`52`	`[ConditionalFact(typeof(DataTestUtility), nameof(DataTestUtility.AreConnStringsSetup), nameof(DataTestUtility.IsNotAzureSynapse))]`