cache: add basic unit testing

Michael Muré created 6 years ago

Change summary

cache/repo_cache.go      |  13 +++++
cache/repo_cache_test.go | 101 ++++++++++++++++++++++++++++++++++++++++++
2 files changed, 114 insertions(+)

Detailed changes

cache/repo_cache.go

@@ -153,6 +153,19 @@ func (c *RepoCache) lock() error {
 }
 
 func (c *RepoCache) Close() error {
+	for id := range c.identities {
+		delete(c.identities, id)
+	}
+	for id := range c.identitiesExcerpts {
+		delete(c.identitiesExcerpts, id)
+	}
+	for id := range c.bugs {
+		delete(c.bugs, id)
+	}
+	for id := range c.bugExcerpts {
+		delete(c.bugExcerpts, id)
+	}
+
 	lockPath := repoLockFilePath(c.repo)
 	return os.Remove(lockPath)
 }

cache/repo_cache_test.go

@@ -0,0 +1,101 @@
+package cache
+
+import (
+	"testing"
+
+	"github.com/MichaelMure/git-bug/util/test"
+	"github.com/stretchr/testify/require"
+)
+
+func TestCache(t *testing.T) {
+	repo := test.CreateRepo(false)
+
+	cache, err := NewRepoCache(repo)
+	require.NoError(t, err)
+
+	// Create, set and get user identity
+	iden1, err := cache.NewIdentity("René Descartes", "[e-mail]")
+	require.NoError(t, err)
+	err = cache.SetUserIdentity(iden1)
+	require.NoError(t, err)
+	userIden, err := cache.GetUserIdentity()
+	require.NoError(t, err)
+	require.Equal(t, iden1.Id(), userIden.Id())
+
+	// it's possible to create two identical identities
+	iden2, err := cache.NewIdentity("René Descartes", "[e-mail]")
+	require.NoError(t, err)
+
+	// Two identical identities yield a different id
+	require.NotEqual(t, iden1.Id(), iden2.Id())
+
+	// There is now two identities in the cache
+	require.Len(t, cache.AllIdentityIds(), 2)
+	require.Len(t, cache.identitiesExcerpts, 2)
+	require.Len(t, cache.identities, 2)
+
+	// Create a bug
+	bug1, err := cache.NewBug("title", "message")
+	require.NoError(t, err)
+
+	// It's possible to create two identical bugs
+	bug2, err := cache.NewBug("title", "message")
+	require.NoError(t, err)
+
+	// two identical bugs yield a different id
+	require.NotEqual(t, bug1.Id(), bug2.Id())
+
+	// There is now two bugs in the cache
+	require.Len(t, cache.AllBugsIds(), 2)
+	require.Len(t, cache.bugExcerpts, 2)
+	require.Len(t, cache.bugs, 2)
+
+	// Resolving
+	_, err = cache.ResolveIdentity(iden1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveIdentityExcerpt(iden1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveIdentityPrefix(iden1.Id()[:10])
+	require.NoError(t, err)
+
+	_, err = cache.ResolveBug(bug1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveBugExcerpt(bug1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveBugPrefix(bug1.Id()[:10])
+	require.NoError(t, err)
+
+	// Querying
+	query, err := ParseQuery("status:open author:descartes sort:edit-asc")
+	require.NoError(t, err)
+	require.Len(t, cache.QueryBugs(query), 2)
+
+	// Close
+	require.NoError(t, cache.Close())
+	require.Empty(t, cache.bugs)
+	require.Empty(t, cache.bugExcerpts)
+	require.Empty(t, cache.identities)
+	require.Empty(t, cache.identitiesExcerpts)
+
+	// Reload, only excerpt are loaded
+	require.NoError(t, cache.load())
+	require.Empty(t, cache.bugs)
+	require.Empty(t, cache.identities)
+	require.Len(t, cache.bugExcerpts, 2)
+	require.Len(t, cache.identitiesExcerpts, 2)
+
+	// Resolving load from the disk
+	_, err = cache.ResolveIdentity(iden1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveIdentityExcerpt(iden1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveIdentityPrefix(iden1.Id()[:10])
+	require.NoError(t, err)
+
+	_, err = cache.ResolveBug(bug1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveBugExcerpt(bug1.Id())
+	require.NoError(t, err)
+	_, err = cache.ResolveBugPrefix(bug1.Id()[:10])
+	require.NoError(t, err)
+}