fix: bank store migration (#13821)
This commit is contained in:
parent
a5e8e2db4b
commit
d314a12e68
@ -81,7 +81,7 @@ func migrateDenomMetadata(store sdk.KVStore, logger log.Logger) error {
|
||||
|
||||
for ; oldDenomMetaDataIter.Valid(); oldDenomMetaDataIter.Next() {
|
||||
oldKey := oldDenomMetaDataIter.Key()
|
||||
l := len(oldKey)/2 + 1
|
||||
l := len(oldKey) / 2
|
||||
|
||||
newKey := make([]byte, len(types.DenomMetadataPrefix)+l)
|
||||
// old key: prefix_bytes | denom_bytes | denom_bytes
|
||||
|
||||
@ -89,8 +89,8 @@ func TestMigrateDenomMetaData(t *testing.T) {
|
||||
denomMetadataStore := prefix.NewStore(store, v2.DenomMetadataPrefix)
|
||||
|
||||
for i := range []int{0, 1} {
|
||||
key := append(v2.DenomMetadataPrefix, []byte(metaData[i].Base)...)
|
||||
// keys before 0.45 had denom two times in the key
|
||||
key := append([]byte{}, []byte(metaData[i].Base)...)
|
||||
key = append(key, []byte(metaData[i].Base)...)
|
||||
bz, err := encCfg.Codec.Marshal(&metaData[i])
|
||||
require.NoError(t, err)
|
||||
@ -107,11 +107,11 @@ func TestMigrateDenomMetaData(t *testing.T) {
|
||||
newKey := denomMetadataIter.Key()
|
||||
|
||||
// make sure old entry is deleted
|
||||
oldKey := append(newKey, newKey[1:]...)
|
||||
oldKey := append(newKey, newKey[0:]...)
|
||||
bz := denomMetadataStore.Get(oldKey)
|
||||
require.Nil(t, bz)
|
||||
|
||||
require.Equal(t, string(newKey)[1:], metaData[i].Base, "idx: %d", i)
|
||||
require.Equal(t, string(newKey), metaData[i].Base, "idx: %d", i)
|
||||
bz = denomMetadataStore.Get(denomMetadataIter.Key())
|
||||
require.NotNil(t, bz)
|
||||
err := encCfg.Codec.Unmarshal(bz, &result)
|
||||
|
||||
Loading…
Reference in New Issue
Block a user