jcarr
/
git2go
1
0
Fork 0
Code Issues 52 Pull Requests 31 Packages Projects Releases Wiki Activity

Make `TestApplyDiffAddFile()` explicitly `.Free()` stuff (#661) (#664) 

This change adds explicit `.Free()` calls in `TestApplyDiffAddFile()`.
It was discovered in #657 that some objects were not explicitly being
freed, so this fixes that!

(cherry picked from commit 3a4204bd93)

Co-authored-by: lhchavez <lhchavez@lhchavez.com>
This commit is contained in:
github-actions[bot] 2020-10-22 06:33:59 -07:00 committed by GitHub
parent 8b6f3d8056
commit 6badd3d00d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
diff_test.go
View File

@ -246,11 +246,16 @@ func TestApplyDiffAddfile(t *testing.T) {
seedTestRepo(t, repo) seedTestRepo(t, repo)
addFirstFileCommit, addFileTree := addAndGetTree(t, repo, "file1", `hello`) addFirstFileCommit, addFirstFileTree := addAndGetTree(t, repo, "file1", `hello`)
defer addFirstFileCommit.Free()
defer addFirstFileTree.Free()
addSecondFileCommit, addSecondFileTree := addAndGetTree(t, repo, "file2", `hello2`) addSecondFileCommit, addSecondFileTree := addAndGetTree(t, repo, "file2", `hello2`)
defer addSecondFileCommit.Free()
defer addSecondFileTree.Free()
diff, err := repo.DiffTreeToTree(addFileTree, addSecondFileTree, nil) diff, err := repo.DiffTreeToTree(addFirstFileTree, addSecondFileTree, nil)
checkFatal(t, err) checkFatal(t, err)
defer diff.Free()
t.Run("check does not apply to current tree because file exists", func(t *testing.T) { t.Run("check does not apply to current tree because file exists", func(t *testing.T) {
err = repo.ResetToCommit(addSecondFileCommit, ResetHard, &CheckoutOpts{}) err = repo.ResetToCommit(addSecondFileCommit, ResetHard, &CheckoutOpts{})
@ -293,12 +298,15 @@ func TestApplyDiffAddfile(t *testing.T) {
commit, err := repo.LookupCommit(head.Target()) commit, err := repo.LookupCommit(head.Target())
checkFatal(t, err) checkFatal(t, err)
defer commit.Free()
tree, err := commit.Tree() tree, err := commit.Tree()
checkFatal(t, err) checkFatal(t, err)
defer tree.Free()
newDiff, err := repo.DiffTreeToTree(addFileTree, tree, nil) newDiff, err := repo.DiffTreeToTree(addFirstFileTree, tree, nil)
checkFatal(t, err) checkFatal(t, err)
defer newDiff.Free()
raw1b, err := diff.ToBuf(DiffFormatPatch) raw1b, err := diff.ToBuf(DiffFormatPatch)
checkFatal(t, err) checkFatal(t, err)
@ -327,6 +335,7 @@ func TestApplyDiffAddfile(t *testing.T) {
diff2, err := DiffFromBuffer(raw, repo) diff2, err := DiffFromBuffer(raw, repo)
checkFatal(t, err) checkFatal(t, err)
defer diff2.Free()
err = repo.ApplyDiff(diff2, ApplyLocationBoth, nil) err = repo.ApplyDiff(diff2, ApplyLocationBoth, nil)
checkFatal(t, err) checkFatal(t, err)