fix: issue with locking and getting environment variables

bindings/src/Capgemini.PowerApps.SpecFlowBindings/Configuration/ConfigHelper.cs

@@ -14,6 +14,11 @@ public static class ConfigHelper
         /// <returns>The environment variable value (if found) or the passed in value.</returns>
         public static string GetEnvironmentVariableIfExists(string value)
         {
+            if (string.IsNullOrEmpty(value))
+            {
+                return value;
+            }
+
             var environmentVariableValue = Environment.GetEnvironmentVariable(value);
 
             if (!string.IsNullOrEmpty(environmentVariableValue))

bindings/src/Capgemini.PowerApps.SpecFlowBindings/PowerAppsStepDefiner.cs

@@ -39,6 +39,7 @@ public abstract class PowerAppsStepDefiner
         private static XrmApp xrmApp;
 
         private static IDictionary<string, string> userProfilesDirectories;
+        private static object userProfilesDirectoriesLock = new object();
 
         /// <summary>
         /// Gets access token used to authenticate as the application user configured for testing.
@@ -176,14 +177,9 @@ protected static IDictionary<string, string> UserProfileDirectories
                     throw new NotSupportedException($"The {testConfig.BrowserOptions.BrowserType} does not support profiles.");
                 }
 
-                if (userProfilesDirectories == null)
+                lock (userProfilesDirectoriesLock)
                 {
-                    userProfilesDirectories = new Dictionary<string, string>();
-                }
-
-                lock (userProfilesDirectories)
-                {
-                    if (userProfilesDirectories.Any())
+                    if (userProfilesDirectories != null)
                     {
                         return userProfilesDirectories;
                     }