Clean up unused helper methods in LabUserHelper

- Remove GetLabUserDataAsync (only used by obsolete GetAdfsUserAsync)
- Remove GetAdfsUserAsync (no external callers, replaced by GetDefaultAdfsUserAsync)
- Remove GetHybridSpaAccontAsync (no external callers, test now uses GetDefaultUserWithMultiTenantAppAsync)
- Remove s_userCache field and System.Collections.Concurrent import (no longer needed)
- Update TODO comments to remove references to deleted methods
- All active functionality preserved, ~45 lines of obsolete code removed

Author: RyAuld

tests/Microsoft.Identity.Test.LabInfrastructure/LabUserHelper.cs

@@ -2,7 +2,6 @@
 // Licensed under the MIT License.
 
 using System;
-using System.Collections.Concurrent;
 using System.Diagnostics;
 using System.Threading.Tasks;
 using Newtonsoft.Json;
@@ -13,8 +12,6 @@ namespace Microsoft.Identity.Test.LabInfrastructure
     public static class LabUserHelper
     {
         private static readonly LabServiceApi s_labService;
-        private static readonly ConcurrentDictionary<UserQuery, LabResponse> s_userCache =
-            new ConcurrentDictionary<UserQuery, LabResponse>();
 
         public static KeyVaultSecretsProvider KeyVaultSecretsProviderMsal { get; }
         public static KeyVaultSecretsProvider KeyVaultSecretsProviderMsid { get; }
@@ -26,31 +23,11 @@ static LabUserHelper()
             s_labService = new LabServiceApi();
         }
 
-        public static async Task<LabResponse> GetLabUserDataAsync(UserQuery query)
-        {
-            if (s_userCache.ContainsKey(query))
-            {
-                var cachedResponse = s_userCache[query];
-                Debug.WriteLine($"Lab cache hit: {cachedResponse.User?.Upn ?? "N/A"} | {cachedResponse.App?.AppId ?? "N/A"} | {cachedResponse.Lab?.TenantId ?? "N/A"}");
-                return cachedResponse;
-            }
-
-            var response = await s_labService.GetLabResponseFromApiAsync(query).ConfigureAwait(false);
-            if (response == null)
-            {
-                Debug.WriteLine($"Lab API returned null for query: {query}");
-                throw new LabUserNotFoundException(query, "Found no users for the given query.");
-            }
 
-            Debug.WriteLine($"Lab API: {response.User?.Upn ?? "N/A"} | {response.App?.AppId ?? "N/A"} | {response.Lab?.TenantId ?? "N/A"} | {response.User?.AzureEnvironment.ToString() ?? "N/A"}");
-
-            s_userCache.TryAdd(query, response);
-            return response;
-        }
 
         private static async Task<LabResponse> GetKVLabDataAsync(string secret)
         {
-            // TODO: Implement caching similar to GetLabUserDataAsync to avoid repeated Key Vault calls
+            // TODO: Implement caching to avoid repeated Key Vault calls
             try
             {
                 var keyVaultSecret = await KeyVaultSecretsProviderMsal.GetSecretByNameAsync(secret).ConfigureAwait(false);
@@ -166,10 +143,7 @@ public static Task<LabResponse> GetDefaultAdfsUserAsync()
             return MergeKVLabDataAsync("MSAL-USER-FedDefault-JSON", "ID4SLAB1", "MSAL-App-Default-JSON");
         }
 
-        public static Task<LabResponse> GetHybridSpaAccontAsync()
-        {
-            return MergeKVLabDataAsync("MSAL-User-Default-JSON", "ID4SLAB1", "MSAL-App-Default-JSON");
-        }
+
 
         public static Task<LabResponse> GetB2CLocalAccountAsync()
         {
@@ -194,28 +168,11 @@ public static Task<LabResponse> GetCIAMUserAsync()
             return MergeKVLabDataAsync("MSAL-User-CIAM-JSON", "MSIDLABCIAM6", "MSAL-App-CIAM-JSON");
         }
 
-        public static Task<LabResponse> GetAdfsUserAsync(FederationProvider federationProvider, bool federated = true)
-        {
-            var query = new UserQuery()
-            {
-                AzureEnvironment = LabInfrastructure.AzureEnvironment.azurecloud,
-                FederationProvider = federationProvider,
-                UserType = federated ? UserType.Federated : UserType.Cloud
-            };
 
-            if (!federated && federationProvider != FederationProvider.ADFSv2019)
-            {
-                Debug.WriteLine($"Invalid ADFS config: {federationProvider} non-federated not supported");
-                throw new InvalidOperationException("Test Setup Error: MSAL only supports ADFS2019 direct (non-federated) access. " +
-                    "Support for older versions of ADFS is exclusively via federation");
-            }
-
-            return GetLabUserDataAsync(query);
-        }
 
         public static string FetchUserPassword(string userLabName)
         {
-            // TODO: Implement caching similar to GetLabUserDataAsync to avoid repeated Key Vault calls
+            // TODO: Implement caching to avoid repeated Key Vault calls
             if (string.IsNullOrWhiteSpace(userLabName))
             {
                 Debug.WriteLine("Password fetch failed: empty lab name");