cache: rename RootCache into MultiRepoCache

Michael Muré created 7 years ago

Underline the fact that it's fine to use RepoCache alone

Change summary

cache/multi_repo_cache.go     | 16 ++++++++--------
graphql/models/models.go      |  4 ++--
graphql/resolvers/mutation.go |  2 +-
graphql/resolvers/query.go    |  2 +-
graphql/resolvers/root.go     |  8 ++++----
5 files changed, 16 insertions(+), 16 deletions(-)

Detailed changes

cache/cache.go → cache/multi_repo_cache.go

@@ -9,18 +9,18 @@ import (
 const lockfile = "lock"
 const excerptsFile = "excerpts"
 
-type RootCache struct {
+type MultiRepoCache struct {
 	repos map[string]*RepoCache
 }
 
-func NewCache() RootCache {
-	return RootCache{
+func NewMultiRepoCache() MultiRepoCache {
+	return MultiRepoCache{
 		repos: make(map[string]*RepoCache),
 	}
 }
 
 // RegisterRepository register a named repository. Use this for multi-repo setup
-func (c *RootCache) RegisterRepository(ref string, repo repository.Repo) error {
+func (c *MultiRepoCache) RegisterRepository(ref string, repo repository.Repo) error {
 	r, err := NewRepoCache(repo)
 	if err != nil {
 		return err
@@ -31,7 +31,7 @@ func (c *RootCache) RegisterRepository(ref string, repo repository.Repo) error {
 }
 
 // RegisterDefaultRepository register a unnamed repository. Use this for mono-repo setup
-func (c *RootCache) RegisterDefaultRepository(repo repository.Repo) error {
+func (c *MultiRepoCache) RegisterDefaultRepository(repo repository.Repo) error {
 	r, err := NewRepoCache(repo)
 	if err != nil {
 		return err
@@ -42,7 +42,7 @@ func (c *RootCache) RegisterDefaultRepository(repo repository.Repo) error {
 }
 
 // ResolveRepo retrieve a repository by name
-func (c *RootCache) DefaultRepo() (*RepoCache, error) {
+func (c *MultiRepoCache) DefaultRepo() (*RepoCache, error) {
 	if len(c.repos) != 1 {
 		return nil, fmt.Errorf("repository is not unique")
 	}
@@ -55,7 +55,7 @@ func (c *RootCache) DefaultRepo() (*RepoCache, error) {
 }
 
 // DefaultRepo retrieve the default repository
-func (c *RootCache) ResolveRepo(ref string) (*RepoCache, error) {
+func (c *MultiRepoCache) ResolveRepo(ref string) (*RepoCache, error) {
 	r, ok := c.repos[ref]
 	if !ok {
 		return nil, fmt.Errorf("unknown repo")
@@ -64,7 +64,7 @@ func (c *RootCache) ResolveRepo(ref string) (*RepoCache, error) {
 }
 
 // Close will do anything that is needed to close the cache properly
-func (c *RootCache) Close() error {
+func (c *MultiRepoCache) Close() error {
 	for _, cachedRepo := range c.repos {
 		err := cachedRepo.Close()
 		if err != nil {

graphql/models/models.go

@@ -12,11 +12,11 @@ type ConnectionInput struct {
 }
 
 type Repository struct {
-	Cache *cache.RootCache
+	Cache *cache.MultiRepoCache
 	Repo  *cache.RepoCache
 }
 
 type RepositoryMutation struct {
-	Cache *cache.RootCache
+	Cache *cache.MultiRepoCache
 	Repo  *cache.RepoCache
 }

graphql/resolvers/mutation.go

@@ -9,7 +9,7 @@ import (
 )
 
 type mutationResolver struct {
-	cache *cache.RootCache
+	cache *cache.MultiRepoCache
 }
 
 func (r mutationResolver) getRepo(repoRef *string) (*cache.RepoCache, error) {

graphql/resolvers/query.go

@@ -8,7 +8,7 @@ import (
 )
 
 type rootQueryResolver struct {
-	cache *cache.RootCache
+	cache *cache.MultiRepoCache
 }
 
 func (r rootQueryResolver) DefaultRepository(ctx context.Context) (*models.Repository, error) {

graphql/resolvers/root.go

@@ -6,24 +6,24 @@ import (
 )
 
 type RootResolver struct {
-	cache.RootCache
+	cache.MultiRepoCache
 }
 
 func NewRootResolver() *RootResolver {
 	return &RootResolver{
-		RootCache: cache.NewCache(),
+		MultiRepoCache: cache.NewMultiRepoCache(),
 	}
 }
 
 func (r RootResolver) Query() graph.QueryResolver {
 	return &rootQueryResolver{
-		cache: &r.RootCache,
+		cache: &r.MultiRepoCache,
 	}
 }
 
 func (r RootResolver) Mutation() graph.MutationResolver {
 	return &mutationResolver{
-		cache: &r.RootCache,
+		cache: &r.MultiRepoCache,
 	}
 }