diff options
author | Michael Muré <michael.mure@consensys.net> | 2019-03-25 21:58:27 +0100 |
---|---|---|
committer | Michael Muré <michael.mure@consensys.net> | 2019-03-25 22:06:01 +0100 |
commit | d27e3849b826309bdf2dbf6f30f2a8ac9db71649 (patch) | |
tree | ad4a751d9e13d2d8b4d163a4b61d1e32fd1ab855 | |
parent | a40dcc8a6a5acf01d823e1ca0f00e0d24c653655 (diff) | |
download | git-bug-d27e3849b826309bdf2dbf6f30f2a8ac9db71649.tar.gz |
cache: add basic unit testing
-rw-r--r-- | cache/repo_cache.go | 13 | ||||
-rw-r--r-- | cache/repo_cache_test.go | 101 |
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) +} |