Don't ignore an editorconfig if it happens to be in the gitignore or … (#1668)

…csharpierignore

closes #1582

Author: belav

Src/CSharpier.Cli/EditorConfig/EditorConfigLocator.cs

@@ -31,7 +31,7 @@ CancellationToken cancellationToken
                         var file = fileSystem.FileInfo.New(
                             fileSystem.Path.Combine(directoryInfo.FullName, ".editorconfig")
                         );
-                        if (file.Exists && !ignoreFile.IsIgnored(file.FullName))
+                        if (file.Exists)
                         {
                             var dirName = fileSystem.Path.GetDirectoryName(file.FullName);
                             ArgumentNullException.ThrowIfNull(dirName);

Src/CSharpier.Tests/CommandLineFormatterTests.cs

@@ -901,6 +901,44 @@ public void Should_Support_Config_Path_With_Editor_Config()
         context.GetFileContent(fileName).Should().Be("var myVariable =\n    someLongValue;\n");
     }
 
+    [Test]
+    public void Should_Not_Fail_With_Invalid_Editor_Config()
+    {
+        var context = new TestContext();
+        context.WhenAFileExists(
+            ".editorconfig",
+            """
+            [*
+            max_line_length 10
+            """
+        );
+        var fileName = context.WhenAFileExists("file1.cs", "var myVariable   = someLongValue;");
+
+        Format(context);
+
+        context.GetFileContent(fileName).Should().Be("var myVariable = someLongValue;\n");
+    }
+
+    [TestCase(".gitignore")]
+    [TestCase(".csharpierignore")]
+    public void Should_Respect_Ignored_Editor_Config(string ignoreFileName)
+    {
+        var context = new TestContext();
+        context.WhenAFileExists(
+            ".editorconfig",
+            """
+            [*]
+            max_line_length = 10
+            """
+        );
+        context.WhenAFileExists(ignoreFileName, ".editorconfig");
+        var fileName = context.WhenAFileExists("file1.cs", "var myVariable = someLongValue;");
+
+        Format(context);
+
+        context.GetFileContent(fileName).Should().Be("var myVariable =\n    someLongValue;\n");
+    }
+
     [TestCase("\r\n")]
     [TestCase("\n")]
     public void Format_XML_With_Multiline_Comment_Uses_Consistent_Line_Breaks(string lineBreak)

Src/CSharpier.Tests/OptionsProviderTests.cs

@@ -719,7 +719,7 @@ public async Task Should_Ignore_Invalid_EditorConfig_Lines()
     }
 
     [Test]
-    public async Task Should_Ignore_Ignored_EditorConfig()
+    public async Task Should_Not_Ignore_Ignored_EditorConfig()
     {
         var context = new TestContext();
         context.WhenAFileExists(
@@ -744,7 +744,7 @@ public async Task Should_Ignore_Ignored_EditorConfig()
             "c:/test",
             "c:/test/subfolder/test.cs"
         );
-        result.IndentSize.Should().Be(1);
+        result.IndentSize.Should().Be(2);
     }
 
     [Test]