caches git objects

Renames `GetOrCreate` to `GetOrWrap` to better reflect its caching behavior.

Author: arturcic

src/GitVersion.LibGit2Sharp/Git/Branch.cs

@@ -18,7 +18,7 @@ internal Branch(LibGit2Sharp.Branch branch, LibGit2Sharp.Diff diff, GitRepositor
         Name = new(branch.CanonicalName);
 
         var commit = this.innerBranch.Tip;
-        Tip = commit is null ? null : repo.GetOrCreate(commit, diff);
+        Tip = commit is null ? null : repo.GetOrWrap(commit, diff);
 
         var commits = this.innerBranch.Commits;
         Commits = new CommitCollection(commits, diff, repo);

src/GitVersion.LibGit2Sharp/Git/BranchCollection.cs

@@ -13,7 +13,7 @@ internal sealed class BranchCollection : IBranchCollection
     internal BranchCollection(LibGit2Sharp.BranchCollection collection, Diff diff, GitRepository repo)
     {
         this.innerCollection = collection.NotNull();
-        this.branches = new Lazy<IReadOnlyCollection<IBranch>>(() => [.. this.innerCollection.Select(branch => repo.GetOrCreate(branch, diff))]);
+        this.branches = new Lazy<IReadOnlyCollection<IBranch>>(() => [.. this.innerCollection.Select(branch => repo.GetOrWrap(branch, diff))]);
         this.diff = diff.NotNull();
         this.repo = repo.NotNull();
     }
@@ -28,7 +28,7 @@ public IBranch? this[string name]
         {
             name = name.NotNull();
             var branch = this.innerCollection[name];
-            return branch is null ? null : this.repo.GetOrCreate(branch, this.diff);
+            return branch is null ? null : this.repo.GetOrWrap(branch, this.diff);
         }
     }
 

src/GitVersion.LibGit2Sharp/Git/Commit.cs

@@ -19,7 +19,7 @@ internal Commit(LibGit2Sharp.Commit innerCommit, LibGit2Sharp.Diff repoDiff, Git
         repoDiff.NotNull();
         repo.NotNull();
         this.innerCommit = innerCommit.NotNull();
-        this.parentsLazy = new(() => innerCommit.Parents.Select(parent => repo.GetOrCreate(parent, repoDiff)).ToList());
+        this.parentsLazy = new(() => innerCommit.Parents.Select(parent => repo.GetOrWrap(parent, repoDiff)).ToList());
         When = innerCommit.Committer.When;
         this.repoDiff = repoDiff;
     }

src/GitVersion.LibGit2Sharp/Git/CommitCollection.cs

@@ -13,7 +13,7 @@ internal sealed class CommitCollection : ICommitCollection
     internal CommitCollection(ICommitLog collection, Diff diff, GitRepository repo)
     {
         this.innerCollection = collection.NotNull();
-        this.commits = new Lazy<IReadOnlyCollection<ICommit>>(() => [.. this.innerCollection.Select(commit => repo.GetOrCreate(commit, diff))]);
+        this.commits = new Lazy<IReadOnlyCollection<ICommit>>(() => [.. this.innerCollection.Select(commit => repo.GetOrWrap(commit, diff))]);
         this.diff = diff.NotNull();
         this.repo = repo.NotNull();
     }

src/GitVersion.LibGit2Sharp/Git/GitRepository.cs

@@ -25,7 +25,7 @@ private IRepository RepositoryInstance
     public string WorkingDirectory => RepositoryInstance.Info.WorkingDirectory;
     public bool IsHeadDetached => RepositoryInstance.Info.IsHeadDetached;
     public bool IsShallow => RepositoryInstance.Info.IsShallow;
-    public IBranch Head => GetOrCreate(RepositoryInstance.Head, RepositoryInstance.Diff);
+    public IBranch Head => GetOrWrap(RepositoryInstance.Head, RepositoryInstance.Diff);
 
     public ITagCollection Tags => new TagCollection(RepositoryInstance.Tags, RepositoryInstance.Diff, this);
     public IReferenceCollection Refs => new ReferenceCollection(RepositoryInstance.Refs);
@@ -53,7 +53,7 @@ public void DiscoverRepository(string? gitDirectory)
             var first = (Commit)commit;
             var second = (Commit)otherCommit;
             var mergeBase = RepositoryInstance.ObjectDatabase.FindMergeBase(first, second);
-            return mergeBase == null ? null : GetOrCreate(mergeBase, RepositoryInstance.Diff);
+            return mergeBase == null ? null : GetOrWrap(mergeBase, RepositoryInstance.Diff);
         });
     }
 
@@ -63,7 +63,7 @@ public int UncommittedChangesCount()
         return retryAction.Execute(GetUncommittedChangesCountInternal);
     }
 
-    public Branch GetOrCreate(LibGit2Sharp.Branch innerBranch, Diff repoDiff)
+    public Branch GetOrWrap(LibGit2Sharp.Branch innerBranch, Diff repoDiff)
     {
         if (innerBranch.Tip is null)
         {
@@ -74,10 +74,10 @@ public Branch GetOrCreate(LibGit2Sharp.Branch innerBranch, Diff repoDiff)
         return cachedBranches.GetOrAdd(cacheKey, _ => new Branch(innerBranch, repoDiff, this));
     }
 
-    public Commit GetOrCreate(LibGit2Sharp.Commit innerCommit, Diff repoDiff) =>
+    public Commit GetOrWrap(LibGit2Sharp.Commit innerCommit, Diff repoDiff) =>
         cachedCommits.GetOrAdd(innerCommit.Sha, _ => new Commit(innerCommit, repoDiff, this));
 
-    public Tag GetOrCreate(LibGit2Sharp.Tag innerTag, Diff repoDiff)
+    public Tag GetOrWrap(LibGit2Sharp.Tag innerTag, Diff repoDiff)
     {
         var cacheKey = $"{innerTag.CanonicalName}@{innerTag.Target.Sha}";
         return cachedTags.GetOrAdd(cacheKey, _ => new Tag(innerTag, repoDiff, this));

src/GitVersion.LibGit2Sharp/Git/Tag.cs

@@ -37,7 +37,7 @@ internal Tag(LibGit2Sharp.Tag tag, Diff diff, GitRepository repo)
             target = annotation.Target;
         }
 
-        return target is LibGit2Sharp.Commit commit ? this.repo.GetOrCreate(commit, this.diff) : null;
+        return target is LibGit2Sharp.Commit commit ? this.repo.GetOrWrap(commit, this.diff) : null;
     }
 
     public override bool Equals(object? obj) => Equals(obj as ITag);

src/GitVersion.LibGit2Sharp/Git/TagCollection.cs

@@ -11,7 +11,7 @@ internal TagCollection(LibGit2Sharp.TagCollection collection, LibGit2Sharp.Diff
         collection.NotNull();
         diff.NotNull();
         repo.NotNull();
-        this.tags = new Lazy<IReadOnlyCollection<ITag>>(() => [.. collection.Select(tag => repo.GetOrCreate(tag, diff))]);
+        this.tags = new Lazy<IReadOnlyCollection<ITag>>(() => [.. collection.Select(tag => repo.GetOrWrap(tag, diff))]);
     }
 
     public IEnumerator<ITag> GetEnumerator() => this.tags.Value.GetEnumerator();