use constants for cache type

pkg/infra/remotecache/database_storage.go

@@ -10,6 +10,8 @@ import (
 
 
 var getTime = time.Now
 var getTime = time.Now
 
 
+const databaseCacheType = "database"
+
 type databaseCache struct {
 type databaseCache struct {
 	SQLStore *sqlstore.SqlStore
 	SQLStore *sqlstore.SqlStore
 	log      log.Logger
 	log      log.Logger

pkg/infra/remotecache/memcached_storage.go

@@ -7,6 +7,8 @@ import (
 	"github.com/grafana/grafana/pkg/setting"
 	"github.com/grafana/grafana/pkg/setting"
 )
 )
 
 
+const memcachedCacheType = "memcached"
+
 type memcachedStorage struct {
 type memcachedStorage struct {
 	c *memcache.Client
 	c *memcache.Client
 }
 }

pkg/infra/remotecache/memcached_storage_integration_test.go

@@ -9,7 +9,7 @@ import (
 )
 )
 
 
 func TestMemcachedCacheStorage(t *testing.T) {
 func TestMemcachedCacheStorage(t *testing.T) {
-	opts := &setting.RemoteCacheOptions{Name: "memcached", ConnStr: "localhost:11211"}
+	opts := &setting.RemoteCacheOptions{Name: memcachedCacheType, ConnStr: "localhost:11211"}
 	client := createTestClient(t, opts, nil)
 	client := createTestClient(t, opts, nil)
 	runTestsForClient(t, client)
 	runTestsForClient(t, client)
 }
 }

pkg/infra/remotecache/redis_storage.go

@@ -7,6 +7,8 @@ import (
 	redis "gopkg.in/redis.v2"
 	redis "gopkg.in/redis.v2"
 )
 )
 
 
+const redisCacheType = "redis"
+
 type redisStorage struct {
 type redisStorage struct {
 	c *redis.Client
 	c *redis.Client
 }
 }

pkg/infra/remotecache/redis_storage_integration_test.go

@@ -10,7 +10,7 @@ import (
 
 
 func TestRedisCacheStorage(t *testing.T) {
 func TestRedisCacheStorage(t *testing.T) {
 
 
-	opts := &setting.RemoteCacheOptions{Name: "redis", ConnStr: "localhost:6379"}
+	opts := &setting.RemoteCacheOptions{Name: redisCacheType, ConnStr: "localhost:6379"}
 	client := createTestClient(t, opts, nil)
 	client := createTestClient(t, opts, nil)
 	runTestsForClient(t, client)
 	runTestsForClient(t, client)
 }
 }

pkg/infra/remotecache/remotecache.go

@@ -92,15 +92,15 @@ func (ds *RemoteCache) Run(ctx context.Context) error {
 }
 }
 
 
 func createClient(opts *setting.RemoteCacheOptions, sqlstore *sqlstore.SqlStore) (CacheStorage, error) {
 func createClient(opts *setting.RemoteCacheOptions, sqlstore *sqlstore.SqlStore) (CacheStorage, error) {
-	if opts.Name == "redis" {
+	if opts.Name == redisCacheType {
 		return newRedisStorage(opts), nil
 		return newRedisStorage(opts), nil
 	}
 	}
 
 
-	if opts.Name == "memcached" {
+	if opts.Name == memcachedCacheType {
 		return newMemcachedStorage(opts), nil
 		return newMemcachedStorage(opts), nil
 	}
 	}
 
 
-	if opts.Name == "database" {
+	if opts.Name == databaseCacheType {
 		return newDatabaseCache(sqlstore), nil
 		return newDatabaseCache(sqlstore), nil
 	}
 	}