Add param migration handler (#756)

* Add param migration handle

* Add param migration handle

* test

* test

* test

* test

* test

* test

* test

Author: philipsu522

x/dex/ante_test.go

@@ -81,7 +81,7 @@ func TestCheckDexGasDecorator(t *testing.T) {
 			types.NewMsgPlaceOrders("someone", []*types.Order{{}, {}}, keepertest.TestContract, sdk.NewCoins()),
 			types.NewMsgCancelOrders("someone", []*types.Cancellation{{}, {}, {}}, keepertest.TestContract),
 		},
-		fee: sdk.NewCoins(sdk.NewCoin("usei", sdk.NewInt(27499))),
+		fee: sdk.NewCoins(sdk.NewCoin("usei", sdk.NewInt(25499))),
 	}
 	_, err = decorator.AnteHandle(ctx, tx, false, terminator)
 	require.NotNil(t, err)

x/dex/migrations/v12_to_v13.go

@@ -0,0 +1,15 @@
+package migrations
+
+import (
+	sdk "github.com/cosmos/cosmos-sdk/types"
+	"github.com/sei-protocol/sei-chain/x/dex/keeper"
+	"github.com/sei-protocol/sei-chain/x/dex/types"
+)
+
+func V12ToV13(ctx sdk.Context, dexkeeper keeper.Keeper) error {
+	// This isn't the cleanest migration since it could potentially revert any dex params we have changed
+	// but we haven't, so we'll just do this.
+	defaultParams := types.DefaultParams()
+	dexkeeper.SetParams(ctx, defaultParams)
+	return nil
+}

x/dex/migrations/v12_to_v13_test.go

@@ -0,0 +1,32 @@
+package migrations_test
+
+import (
+	"testing"
+
+	sdk "github.com/cosmos/cosmos-sdk/types"
+	keepertest "github.com/sei-protocol/sei-chain/testutil/keeper"
+	"github.com/sei-protocol/sei-chain/x/dex/migrations"
+	"github.com/sei-protocol/sei-chain/x/dex/types"
+	"github.com/stretchr/testify/require"
+)
+
+func TestMigrate12to13(t *testing.T) {
+	dexkeeper, ctx := keepertest.DexKeeper(t)
+	// write old params
+	prevParams := types.Params{
+		PriceSnapshotRetention: 1,
+		SudoCallGasPrice:       sdk.OneDec(),
+		BeginBlockGasLimit:     1,
+		EndBlockGasLimit:       1,
+		DefaultGasPerOrder:     1,
+		DefaultGasPerCancel:    1,
+	}
+	dexkeeper.SetParams(ctx, prevParams)
+
+	// migrate to default params
+	err := migrations.V12ToV13(ctx, *dexkeeper)
+	require.NoError(t, err)
+	params := dexkeeper.GetParams(ctx)
+	require.Equal(t, params.GasAllowancePerSettlement, uint64(types.DefaultGasAllowancePerSettlement))
+	require.Equal(t, params.MinProcessableRent, uint64(types.DefaultMinProcessableRent))
+}

x/dex/module.go

@@ -197,6 +197,9 @@ func (am AppModule) RegisterServices(cfg module.Configurator) {
 	_ = cfg.RegisterMigration(types.ModuleName, 11, func(ctx sdk.Context) error {
 		return migrations.V11ToV12(ctx, am.keeper)
 	})
+	_ = cfg.RegisterMigration(types.ModuleName, 12, func(ctx sdk.Context) error {
+		return migrations.V12ToV13(ctx, am.keeper)
+	})
 }
 
 // RegisterInvariants registers the capability module's invariants.
@@ -221,7 +224,7 @@ func (am AppModule) ExportGenesis(ctx sdk.Context, cdc codec.JSONCodec) json.Raw
 }
 
 // ConsensusVersion implements ConsensusVersion.
-func (AppModule) ConsensusVersion() uint64 { return 12 }
+func (AppModule) ConsensusVersion() uint64 { return 13 }
 
 func (am AppModule) getAllContractInfo(ctx sdk.Context) []types.ContractInfoV2 {
 	// Do not process any contract that has zero rent balance

x/dex/types/params.go

@@ -25,7 +25,7 @@ const (
 	DefaultBeginBlockGasLimit        = 200000000  // 200M
 	DefaultEndBlockGasLimit          = 1000000000 // 1B
 	DefaultDefaultGasPerOrder        = 55000
-	DefaultDefaultGasPerCancel       = 55000
+	DefaultDefaultGasPerCancel       = 53000
 	DefaultMinRentDeposit            = 10000000 // 10 sei
 	DefaultGasAllowancePerSettlement = 10000
 	DefaultMinProcessableRent        = 100000