Problem: no api to create cachemulti store from external cache store (#258)

Solution:
- add API NewFromParent to cache multistore.

Update store/CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

fix test

fix lint

Author: yihuang

store/cachemulti/store.go

@@ -25,7 +25,7 @@ type Store struct {
 
 	traceWriter  io.Writer
 	traceContext types.TraceContext
-	parentStore  func(types.StoreKey) types.CacheWrap
+	parentStore  func(types.StoreKey) types.CacheWrapper
 
 	branched bool
 }
@@ -61,12 +61,17 @@ func NewStore(
 	return NewFromKVStore(stores, traceWriter, traceContext)
 }
 
-func newCacheMultiStoreFromCMS(cms Store) Store {
+// NewFromParent constructs a cache multistore with a parent store lazily,
+// the parent is usually another cache multistore or the block-stm multiversion store.
+func NewFromParent(
+	parentStore func(types.StoreKey) types.CacheWrapper,
+	traceWriter io.Writer, traceContext types.TraceContext,
+) Store {
 	return Store{
 		stores:       make(map[types.StoreKey]types.CacheWrap),
-		traceWriter:  cms.traceWriter,
-		traceContext: cms.traceContext,
-		parentStore:  cms.getCacheWrap,
+		traceWriter:  traceWriter,
+		traceContext: traceContext,
+		parentStore:  parentStore,
 	}
 }
 
@@ -146,10 +151,10 @@ func (cms Store) CacheWrapWithTrace(_ io.Writer, _ types.TraceContext) types.Cac
 // CacheMultiStore implements MultiStore, returns a new CacheMultiStore from the
 // underlying CacheMultiStore.
 func (cms Store) CacheMultiStore() types.CacheMultiStore {
-	return newCacheMultiStoreFromCMS(cms)
+	return NewFromParent(cms.getCacheWrapper, cms.traceWriter, cms.traceContext)
 }
 
-func (cms Store) getCacheWrap(key types.StoreKey) types.CacheWrap {
+func (cms Store) getCacheWrapper(key types.StoreKey) types.CacheWrapper {
 	store, ok := cms.stores[key]
 	if !ok && cms.parentStore != nil {
 		// load on demand
@@ -163,7 +168,7 @@ func (cms Store) getCacheWrap(key types.StoreKey) types.CacheWrap {
 
 // GetStore returns an underlying Store by key.
 func (cms Store) GetStore(key types.StoreKey) types.Store {
-	store, ok := cms.getCacheWrap(key).(types.Store)
+	store, ok := cms.getCacheWrapper(key).(types.Store)
 	if !ok {
 		panic(fmt.Sprintf("store with key %v is not Store", key))
 	}
@@ -172,7 +177,7 @@ func (cms Store) GetStore(key types.StoreKey) types.Store {
 
 // GetKVStore returns an underlying KVStore by key.
 func (cms Store) GetKVStore(key types.StoreKey) types.KVStore {
-	store, ok := cms.getCacheWrap(key).(types.KVStore)
+	store, ok := cms.getCacheWrapper(key).(types.KVStore)
 	if !ok {
 		panic(fmt.Sprintf("store with key %v is not KVStore", key))
 	}
@@ -181,7 +186,7 @@ func (cms Store) GetKVStore(key types.StoreKey) types.KVStore {
 
 // GetObjKVStore returns an underlying KVStore by key.
 func (cms Store) GetObjKVStore(key types.StoreKey) types.ObjKVStore {
-	store, ok := cms.getCacheWrap(key).(types.ObjKVStore)
+	store, ok := cms.getCacheWrapper(key).(types.ObjKVStore)
 	if !ok {
 		panic(fmt.Sprintf("store with key %v is not ObjKVStore", key))
 	}

store/cachemulti/store_test.go

@@ -43,7 +43,7 @@ func TestRunAtomic(t *testing.T) {
 		keys["obj"]: objStore.CacheWrap(),
 	}}
 
-	s := Store{stores: map[types.StoreKey]types.CacheWrap{}, parentStore: parent.getCacheWrap}
+	s := Store{stores: map[types.StoreKey]types.CacheWrap{}, parentStore: parent.getCacheWrapper}
 	err := s.RunAtomic(func(ms types.CacheMultiStore) error {
 		ms.GetKVStore(keys["abc"]).Set([]byte("key"), []byte("value"))
 		ms.GetObjKVStore(keys["obj"]).Set([]byte("key"), "value")
@@ -78,7 +78,7 @@ func TestBranchStore(t *testing.T) {
 		keys["obj"]: objStore.CacheWrap(),
 	}}
 
-	s := Store{stores: map[types.StoreKey]types.CacheWrap{}, parentStore: parent.getCacheWrap}
+	s := Store{stores: map[types.StoreKey]types.CacheWrap{}, parentStore: parent.getCacheWrapper}
 	s.GetKVStore(keys["abc"]).Set([]byte("key"), []byte("value"))
 	snapshot := s.Clone()
 	s.GetKVStore(keys["abc"]).Set([]byte("key"), []byte("value2"))