aboutsummaryrefslogtreecommitdiffstats
path: root/cache
diff options
context:
space:
mode:
authorMichael Muré <michael.mure@consensys.net>2019-03-25 21:58:27 +0100
committerMichael Muré <michael.mure@consensys.net>2019-03-25 22:06:01 +0100
commitd27e3849b826309bdf2dbf6f30f2a8ac9db71649 (patch)
treead4a751d9e13d2d8b4d163a4b61d1e32fd1ab855 /cache
parenta40dcc8a6a5acf01d823e1ca0f00e0d24c653655 (diff)
downloadgit-bug-d27e3849b826309bdf2dbf6f30f2a8ac9db71649.tar.gz
cache: add basic unit testing
Diffstat (limited to 'cache')
-rw-r--r--cache/repo_cache.go13
-rw-r--r--cache/repo_cache_test.go101
2 files changed, 114 insertions, 0 deletions
diff --git a/cache/repo_cache.go b/cache/repo_cache.go
index 2b0fa360..bec733e3 100644
--- a/cache/repo_cache.go
+++ b/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)
}
diff --git a/cache/repo_cache_test.go b/cache/repo_cache_test.go
new file mode 100644
index 00000000..c90ad28e
--- /dev/null
+++ b/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", "rene@descartes.fr")
+ 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", "rene@descartes.fr")
+ 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)
+}