Convert static helper into a extension method

Author: davidwengier

….Razor.LanguageServer/DirectoryHelper.cs ….LanguageServer/IFileSystemExtensions.cssrc/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/DirectoryHelper.cs renamed to src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/IFileSystemExtensions.cs src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/DirectoryHelper.cs renamed to src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/IFileSystemExtensions.cs

@@ -11,7 +11,7 @@
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer;
 
-internal static class DirectoryHelper
+internal static class IFileSystemExtensions
 {
     /// <summary>
     /// Finds all the files in a directory which meet the given criteria.
@@ -24,10 +24,10 @@ internal static class DirectoryHelper
     /// <returns>A list of files within the given directory that meet the search criteria.</returns>
     /// <remarks>This method is needed to avoid problematic folders such as "node_modules" which are known not to yield the desired results or may cause performance issues.</remarks>
     internal static IEnumerable<string> GetFilteredFiles(
+        this IFileSystem fileSystem,
         string workspaceDirectory,
         string searchPattern,
         IReadOnlyCollection<string> ignoredDirectories,
-        IFileSystem fileSystem,
         ILogger logger)
     {
         IEnumerable<string> files;
@@ -94,7 +94,7 @@ internal static IEnumerable<string> GetFilteredFiles(
             var directory = Path.GetFileName(path);
             if (!ignoredDirectories.Contains(directory, FilePathComparer.Instance))
             {
-                foreach (var result in GetFilteredFiles(path, searchPattern, ignoredDirectories, fileSystem, logger))
+                foreach (var result in GetFilteredFiles(fileSystem, path, searchPattern, ignoredDirectories, logger))
                 {
                     yield return result;
                 }

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/RazorFileChangeDetector.cs

@@ -224,7 +224,7 @@ protected virtual ImmutableArray<string> GetExistingRazorFiles(string workspaceD
 
         foreach (var extension in s_razorFileExtensions)
         {
-            var existingFiles = DirectoryHelper.GetFilteredFiles(workspaceDirectory, "*" + extension, s_ignoredDirectories, _fileSystem, _logger);
+            var existingFiles = _fileSystem.GetFilteredFiles(workspaceDirectory, "*" + extension, s_ignoredDirectories, _logger);
             result.AddRange(existingFiles);
         }
 

…nguageServer.Test/DirectoryHelperTest.cs …Server.Test/IFileSystemExtensionsTest.cssrc/Razor/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/DirectoryHelperTest.cs renamed to src/Razor/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/IFileSystemExtensionsTest.cs src/Razor/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/DirectoryHelperTest.cs renamed to src/Razor/test/Microsoft.AspNetCore.Razor.LanguageServer.Test/IFileSystemExtensionsTest.cs

@@ -11,7 +11,7 @@
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.Test;
 
-public class DirectoryHelperTest(ITestOutputHelper testOutput) : TagHelperServiceTestBase(testOutput)
+public class IFileSystemExtensionsTest(ITestOutputHelper testOutput) : TagHelperServiceTestBase(testOutput)
 {
     [Fact]
     public void GetFilteredFiles_FindsFiles()
@@ -38,7 +38,7 @@ public void GetFilteredFiles_FindsFiles()
         var fileSystem = new TestFileSystem(fileResults, directoryResults);
 
         // Act
-        var files = DirectoryHelper.GetFilteredFiles(workspaceDirectory, searchPattern, ignoredDirectories, fileSystem, Logger);
+        var files = fileSystem.GetFilteredFiles(workspaceDirectory, searchPattern, ignoredDirectories, Logger);
 
         // Assert
         Assert.Collection(files,