Adding tests to validate that this implementation of ignore is lacking

Author: belav

Directory.Packages.props

@@ -12,6 +12,7 @@
     <PackageVersion Include="GitHubActionsTestLogger" Version="2.4.1" />
     <PackageVersion Include="GitignoreParserNet" Version="0.2.0.14" />
     <PackageVersion Include="ini-parser-netstandard" Version="2.5.3" />
+    <PackageVersion Include="LibGit2Sharp" Version="0.31.0" />
     <PackageVersion Include="LovettSoftware.XmlDiff" Version="1.1.0" />
     <PackageVersion Include="Microsoft.AspNetCore.SpaServices.Extensions" Version="9.0.10" />
     <PackageVersion Include="Microsoft.CodeAnalysis.Common" Version="4.14.0" />
@@ -28,7 +29,7 @@
     <PackageVersion Include="Serilog.Extensions.Logging.File" Version="3.0.0" />
     <PackageVersion Include="StrongNamer" Version="0.2.5" />
     <PackageVersion Include="System.CommandLine" Version="2.0.0-beta4.22272.1" />
-    <PackageVersion Include="System.IO.Abstractions" Version="22.0.16" />
+    <PackageVersion Include="System.IO.Abstractions" Version="22.1.0" />
     <PackageVersion Include="System.IO.Abstractions.TestingHelpers" Version="22.0.16" />
     <PackageVersion Include="System.IO.Hashing" Version="9.0.10" />
     <PackageVersion Include="System.Text.Encoding.CodePages" Version="9.0.10" />

Src/CSharpier.Cli/IgnoreFile.cs

@@ -63,6 +63,9 @@ async Task<IgnoreWithBasePath> CreateIgnore(string ignoreFilePath, string? overr
 
             var content = await fileSystem.File.ReadAllTextAsync(ignoreFilePath, cancellationToken);
 
+            // TODO #1768 Gitignore Parser won't work as is, can maybe modify it. It is an apache license
+            // try out https://github.com/markashleybell/MAB.DotIgnore to see if that works and is fast
+            // if I remember correctly the modified version of https://github.com/goelhardik/ignore I originally used was too slow
             var (positives, negatives) = GitignoreParserNet.GitignoreParser.Parse(content, true);
 
             ignore.AddPositives(positives.Merged);

Src/CSharpier.Tests/CSharpier.Tests.csproj

@@ -12,6 +12,7 @@
     <PackageReference Include="DiffEngine" />
     <PackageReference Include="FluentAssertions" />
     <PackageReference Include="GitHubActionsTestLogger" PrivateAssets="all" />
+    <PackageReference Include="LibGit2Sharp" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" />
     <PackageReference Include="NUnit" />
     <PackageReference Include="NUnit3TestAdapter" />

Src/CSharpier.Tests/Cli/GitRepository.cs

@@ -0,0 +1,97 @@
+using System.IO.Abstractions;
+using LibGit2Sharp;
+
+namespace CSharpier.Tests.Cli;
+
+public class GitRepository
+{
+    private readonly IFileSystem fileSystem;
+    public string RepoPath { get; }
+
+    public GitRepository(IFileSystem fileSystem)
+    {
+        this.fileSystem = fileSystem;
+        var tempPath = this.fileSystem.Path.GetTempPath();
+        this.RepoPath = this.fileSystem.Path.Combine(tempPath, $"fakeRepo-{Guid.NewGuid()}");
+        if (this.fileSystem.Directory.Exists(this.RepoPath))
+        {
+            this.fileSystem.Directory.Delete(this.RepoPath, true);
+        }
+
+        this.fileSystem.Directory.CreateDirectory(this.RepoPath);
+
+        Repository.Init(this.RepoPath);
+    }
+
+    public void AddUntrackedFileToRepo(string relativePath, int numLines = 1)
+    {
+        string lineContent = $"Fake content line.{Environment.NewLine}";
+        this.AddUntrackedFileToRepo(
+            relativePath,
+            string.Concat(Enumerable.Repeat(lineContent, numLines))
+        );
+    }
+
+    public void AddTrackedFileToRepo(string relativePath, string content)
+    {
+        this.AddUntrackedFileToRepo(relativePath, content);
+        this.CommitPathToRepo(relativePath);
+    }
+
+    public void AddUntrackedDirToRepo(string relativePath)
+    {
+        var filePath = this.fileSystem.Path.Combine(this.RepoPath, relativePath);
+        var directoryInfo = Directory.GetParent(filePath);
+        Directory.CreateDirectory(directoryInfo.FullName);
+    }
+
+    public IEnumerable<StatusEntry> GetUntrackedFiles()
+    {
+        using var repo = new Repository(this.RepoPath);
+        var status = repo.RetrieveStatus();
+        return status.Untracked;
+    }
+
+    public void AddUntrackedFileToRepo(string relativePath, string content)
+    {
+        var filePath = this.fileSystem.Path.Combine(this.RepoPath, relativePath);
+        var directoryInfo = Directory.GetParent(filePath);
+        Directory.CreateDirectory(directoryInfo.FullName);
+        this.fileSystem.File.WriteAllText(
+            this.fileSystem.Path.Combine(this.RepoPath, relativePath),
+            content
+        );
+    }
+
+    public void DeleteRepoDirectory()
+    {
+        var dirInfo = this.fileSystem.DirectoryInfo.New(this.RepoPath);
+        if (dirInfo.Exists)
+        {
+            SetNormalAttribute(dirInfo);
+        }
+
+        dirInfo.Delete(true);
+    }
+
+    private static void SetNormalAttribute(IDirectoryInfo dirInfo)
+    {
+        foreach (var subDir in dirInfo.GetDirectories())
+        {
+            SetNormalAttribute(subDir);
+        }
+
+        foreach (var file in dirInfo.GetFiles())
+        {
+            file.Attributes = FileAttributes.Normal;
+        }
+    }
+
+    private void CommitPathToRepo(string relativePath)
+    {
+        var repo = new Repository(this.RepoPath);
+        Commands.Stage(repo, relativePath);
+        var author = new Signature("FakeUser", "[email protected]", DateTimeOffset.Now);
+        repo.Commit("Adding files", author, author);
+    }
+}