mirror of
https://github.com/vee1e/gittuf.git
synced 2026-09-01 10:18:18 +00:00
8374150f3f
This PR adds focused tests for error paths and edge cases across pkg/gitinterface, raising coverage from 69.4% to 76.2%. Files changed: - references_test.go: TestAbsoluteReference with branch/tag/custom/ref lookups - tag_test.go: TestEnsureIsTag, error paths for GetTagTarget and verifyTagSignature - commit_test.go: TestEnsureIsCommit, error paths for commit inspection helpers - object_test.go: Error paths for GetObjectType and GetObjectSize - blob_test.go: Error path for ReadBlob with non-blob object - status_test.go: Tests for Status errors and StatusCode helpers - sync_test.go: Tests for WithFetchDepth, CreateRemote, push/fetch errors - config_test.go: Error path for SetGitConfig - utils_test.go: TestResetDueToError Closes #1244 Signed-off-by: dawgdevv <nraj02415@gmail.com>
119 lines
3.4 KiB
Go
119 lines
3.4 KiB
Go
// Copyright The gittuf Authors
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
package gitinterface
|
|
|
|
import (
|
|
"testing"
|
|
|
|
artifacts "github.com/gittuf/gittuf/internal/testartifacts"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestHasObject(t *testing.T) {
|
|
tempDir1 := t.TempDir()
|
|
repo := CreateTestGitRepository(t, tempDir1, true)
|
|
|
|
// Create a backup repo to compute Git IDs we test in repo
|
|
tempDir2 := t.TempDir()
|
|
backupRepo := CreateTestGitRepository(t, tempDir2, true)
|
|
|
|
blobID, err := backupRepo.WriteBlob([]byte("hello"))
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
assert.True(t, backupRepo.HasObject(blobID)) // backup has it
|
|
assert.False(t, repo.HasObject(blobID)) // repo does not
|
|
|
|
if _, err := repo.WriteBlob([]byte("hello")); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
assert.True(t, repo.HasObject(blobID)) // now repo has it too
|
|
|
|
backupRepoTreeBuilder := NewTreeBuilder(backupRepo)
|
|
treeID, err := backupRepoTreeBuilder.WriteTreeFromEntries([]TreeEntry{NewEntryBlob("file", blobID)})
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
assert.True(t, backupRepo.HasObject(treeID)) // backup has it
|
|
assert.False(t, repo.HasObject(treeID)) // repo does not
|
|
|
|
repoTreeBuilder := NewTreeBuilder(repo)
|
|
if _, err := repoTreeBuilder.WriteTreeFromEntries([]TreeEntry{NewEntryBlob("file", blobID)}); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
assert.True(t, repo.HasObject(treeID)) // now repo has it too
|
|
|
|
commitID, err := backupRepo.Commit(treeID, "refs/heads/main", "Initial commit\n", false)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
assert.True(t, backupRepo.HasObject(commitID)) // backup has it
|
|
assert.False(t, repo.HasObject(commitID)) // repo does not
|
|
|
|
if _, err := repo.Commit(treeID, "refs/heads/main", "Initial commit\n", false); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
// Note: This test passes because we control timestamps in
|
|
// CreateTestGitRepository. So, commit ID in both repos is the same.
|
|
assert.True(t, repo.HasObject(commitID)) // now repo has it too
|
|
}
|
|
|
|
func TestGetObjectType(t *testing.T) {
|
|
tmpDir := t.TempDir()
|
|
repo := CreateTestGitRepository(t, tmpDir, false)
|
|
|
|
blobID, err := repo.WriteBlob([]byte("gittuf"))
|
|
require.Nil(t, err)
|
|
|
|
objType, err := repo.GetObjectType(blobID)
|
|
assert.Nil(t, err)
|
|
assert.Equal(t, BlobObjectType, objType)
|
|
|
|
treeBuilder := NewTreeBuilder(repo)
|
|
treeID, err := treeBuilder.WriteTreeFromEntries([]TreeEntry{NewEntryBlob("foo", blobID)})
|
|
require.Nil(t, err)
|
|
|
|
objType, err = repo.GetObjectType(treeID)
|
|
assert.Nil(t, err)
|
|
assert.Equal(t, TreeObjectType, objType)
|
|
|
|
commitID, err := repo.Commit(treeID, "refs/heads/main", "Test commit\n", false)
|
|
require.Nil(t, err)
|
|
|
|
objType, err = repo.GetObjectType(commitID)
|
|
assert.Nil(t, err)
|
|
assert.Equal(t, CommitObjectType, objType)
|
|
|
|
tagID, err := repo.TagUsingSpecificKey(commitID, "test-tag", "Test tag\n", artifacts.GPGKey1Private)
|
|
require.Nil(t, err)
|
|
|
|
objType, err = repo.GetObjectType(tagID)
|
|
assert.Nil(t, err)
|
|
assert.Equal(t, TagObjectType, objType)
|
|
|
|
_, err = repo.GetObjectType(ZeroHash)
|
|
assert.ErrorContains(t, err, "unable to inspect object type")
|
|
}
|
|
|
|
func TestGetObjectSize(t *testing.T) {
|
|
tmpDir := t.TempDir()
|
|
repo := CreateTestGitRepository(t, tmpDir, false)
|
|
|
|
blobID, err := repo.WriteBlob([]byte("gittuf"))
|
|
require.Nil(t, err)
|
|
|
|
objSize, err := repo.GetObjectSize(blobID)
|
|
assert.Nil(t, err)
|
|
assert.Equal(t, uint64(6), objSize)
|
|
|
|
_, err = repo.GetObjectSize(ZeroHash)
|
|
assert.ErrorContains(t, err, "unable to inspect object size")
|
|
}
|