- Added config file envronment variable.

Author: paulmedynski

src/Microsoft.Data.SqlClient/netfx/src/Microsoft/Data/SqlClient/SqlInternalConnectionTds.cs

@@ -2545,15 +2545,6 @@ internal bool TryGetFedAuthTokenLocked(SqlFedAuthInfo fedAuthInfo, DbConnectionP
         /// <returns>SqlFedAuthToken</returns>
         internal SqlFedAuthToken GetFedAuthToken(SqlFedAuthInfo fedAuthInfo)
         {
-            // TODO: The netcore project doesn't consider fedAuthInfo to be
-            // possibly null here, so why does netfx?
-            // if (fedAuthInfo is null)
-            // {
-            //     // Throw what would have been thrown the first time we
-            //     // unconditionally use fedAuthInfo below.
-            //     throw new NullReferenceException("fedAuthInfo must not be null");
-            // }
-
             // Number of milliseconds to sleep for the initial back off, if a
             // retry period is not specified by the provider.
             const int defaultRetryPeriod = 100;

src/Microsoft.Data.SqlClient/tests/ManualTests/SQL/ConnectivityTests/AADConnectionTest.cs

@@ -8,7 +8,6 @@
 using System.Threading;
 using System.Threading.Tasks;
 using Azure.Core;
-using Azure.Identity;
 using Microsoft.Identity.Client;
 using Xunit;
 

src/Microsoft.Data.SqlClient/tests/tools/Microsoft.Data.SqlClient.TestUtilities/Config.cs

@@ -45,6 +45,9 @@ public class Config
 
         public static Config Load(string configPath = @"config.json")
         {
+            // Allow an override of the config path via an environment variable.
+            configPath = Environment.GetEnvironmentVariable("MDS_TEST_CONFIG") ?? configPath;
+
             try
             {
                 using (StreamReader r = new StreamReader(configPath))