Revert "Disable fomatting checks"

This reverts commit ccbfd86.

Author: nohwnd

.editorconfig

@@ -329,7 +329,7 @@ csharp_style_prefer_parameter_null_checking = false # not default, disabled as n
 #### .NET Formatting Rules ####
 
 # IDE0055: Fix formatting - Set the severity of all .NET and C# formatting rules (https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/formatting-rules)
-dotnet_diagnostic.IDE0055.severity = none # Nice formatting is great, but when copilot changes code we more than not end up with some extra whitespace.
+dotnet_diagnostic.IDE0055.severity = warning # ensure all formatting rules are enforced on build
 
 # IDE0057: Use range operator
 dotnet_diagnostic.IDE0057.severity = none # Range operator is not supported in some TFMs.

test/vstest.console.UnitTests/Internal/FilePatternParserTests.cs

@@ -119,16 +119,16 @@ public void FilePatternParserShouldCorrectlySplitPatternAndDirectoryWithForwardS
     {
         var patternMatchingResult = new PatternMatchingResult(new List<FilePatternMatch>());
         _mockMatcherHelper.Setup(x => x.Execute(It.IsAny<DirectoryInfoWrapper>())).Returns(patternMatchingResult);
-
+        
         // Test with forward slashes - this should work on all platforms
         // This specifically tests the fix for issue #14993
         _filePatternParser.GetMatchingFiles("C:/Users/someUser/Desktop/a/c/*bc.dll");
 
         // Assert that the pattern is parsed correctly
         _mockMatcherHelper.Verify(x => x.AddInclude("*bc.dll"));
         // On Windows, the path may be normalized, so we verify the key components are present
-        _mockMatcherHelper.Verify(x => x.Execute(It.Is<DirectoryInfoWrapper>(y =>
-            y.FullName.Contains("someUser") && y.FullName.Contains("Desktop") &&
+        _mockMatcherHelper.Verify(x => x.Execute(It.Is<DirectoryInfoWrapper>(y => 
+            y.FullName.Contains("someUser") && y.FullName.Contains("Desktop") && 
             y.FullName.Contains("a") && y.FullName.EndsWith("c"))));
     }
 
@@ -137,13 +137,13 @@ public void FilePatternParserShouldCorrectlySplitWithArbitraryDirectoryDepthWith
     {
         var patternMatchingResult = new PatternMatchingResult(new List<FilePatternMatch>());
         _mockMatcherHelper.Setup(x => x.Execute(It.IsAny<DirectoryInfoWrapper>())).Returns(patternMatchingResult);
-
+        
         // Test with forward slashes and recursive patterns
         _filePatternParser.GetMatchingFiles("C:/Users/vanidhi/**/c/*bc.txt");
 
         // Assert
         _mockMatcherHelper.Verify(x => x.AddInclude("**/c/*bc.txt"));
-        _mockMatcherHelper.Verify(x => x.Execute(It.Is<DirectoryInfoWrapper>(y =>
+        _mockMatcherHelper.Verify(x => x.Execute(It.Is<DirectoryInfoWrapper>(y => 
             y.FullName.Contains("vanidhi"))));
     }
 
@@ -152,7 +152,7 @@ public void FilePatternParserShouldHandleForwardSlashesWithoutThrowingException(
     {
         var patternMatchingResult = new PatternMatchingResult(new List<FilePatternMatch>());
         _mockMatcherHelper.Setup(x => x.Execute(It.IsAny<DirectoryInfoWrapper>())).Returns(patternMatchingResult);
-
+        
         // This is the specific case from the original bug report that was throwing ArgumentOutOfRangeException
         // Before the fix: System.ArgumentOutOfRangeException: length ('-1') must be a non-negative value
         _filePatternParser.GetMatchingFiles("C:/path/to/my/tests/*_Tests.dll");