change types

Author: PrathyushaLakkireddy

x/resource/genesis.go

@@ -41,7 +41,11 @@ func ExportGenesis(ctx sdk.Context, k keeper.Keeper) *types.GenesisState {
 	}
 
 	// get fee params
-	feeParams := k.GetParams(ctx)
+	feeParams, err := k.GetParams(ctx)
+	if err != nil {
+		panic(fmt.Sprintln("Cannot get fee params: %s", err.Error()))
+
+	}
 
 	return &types.GenesisState{
 		Resources: resourceList,

x/resource/ibc_module.go

@@ -78,7 +78,10 @@ func (im IBCModule) OnChanOpenTry(
 	counterpartyVersion string,
 ) (string, error) {
 	// Require portID is the portID module is bound to
-	boundPort := im.keeper.GetPort(ctx)
+	boundPort, err := im.keeper.GetPort(ctx)
+	if err != nil {
+		return "", err
+	}
 	if boundPort != portID {
 		return "", errorsmod.Wrapf(porttypes.ErrInvalidPort, "invalid port: %s, expected %s", portID, boundPort)
 	}

x/resource/keeper/keeper_param_change_proposal_test.go

@@ -1,10 +1,9 @@
 package keeper_test
 
 import (
+	sdkmath "cosmossdk.io/math"
 	"github.com/stretchr/testify/suite"
 
-	tmproto "github.com/cometbft/cometbft/proto/tendermint/types"
-
 	cheqdapp "github.com/cheqd/cheqd-node/app"
 	resourcetypes "github.com/cheqd/cheqd-node/x/resource/types"
 	sdk "github.com/cosmos/cosmos-sdk/types"
@@ -30,7 +29,7 @@ func (suite *HandlerTestSuite) SetupTest() error {
 	if err != nil {
 		return err
 	}
-	suite.ctx = suite.app.BaseApp.NewContext(false, tmproto.Header{})
+	suite.ctx = suite.app.BaseApp.NewContext(false)
 	suite.govHandler = params.NewParamChangeProposalHandler(suite.app.ParamsKeeper)
 	return nil
 }
@@ -68,13 +67,14 @@ var _ = DescribeTable("Proposal Handler", func(testcase TestCaseKeeperProposal)
 			}),
 			func(handlerSuite *HandlerTestSuite) {
 				expectedFeeParams := resourcetypes.FeeParams{
-					Image:      sdk.Coin{Denom: resourcetypes.BaseMinimalDenom, Amount: sdk.NewInt(10000000000)},
-					Json:       sdk.Coin{Denom: resourcetypes.BaseMinimalDenom, Amount: sdk.NewInt(4000000000)},
-					Default:    sdk.Coin{Denom: resourcetypes.BaseMinimalDenom, Amount: sdk.NewInt(2000000000)},
-					BurnFactor: sdk.MustNewDecFromStr("0.600000000000000000"),
+					Image:      sdk.Coin{Denom: resourcetypes.BaseMinimalDenom, Amount: sdkmath.NewInt(10000000000)},
+					Json:       sdk.Coin{Denom: resourcetypes.BaseMinimalDenom, Amount: sdkmath.NewInt(4000000000)},
+					Default:    sdk.Coin{Denom: resourcetypes.BaseMinimalDenom, Amount: sdkmath.NewInt(2000000000)},
+					BurnFactor: sdkmath.LegacyMustNewDecFromStr("0.600000000000000000"),
 				}
 
-				feeParams := handlerSuite.app.ResourceKeeper.GetParams(handlerSuite.ctx)
+				feeParams, err := handlerSuite.app.ResourceKeeper.GetParams(handlerSuite.ctx)
+				Expect(err).To(BeNil())
 
 				Expect(expectedFeeParams).To(Equal(feeParams))
 			},

x/resource/types/params.go

@@ -27,7 +27,7 @@ func ParamKeyTable() paramstypes.KeyTable {
 // )
 
 // NewParams creates a new FeeParams object with specified parameters
-func NewParams(image, json, defaultFee sdk.Coin, burnFactor sdkmath.Dec) FeeParams {
+func NewParams(image, json, defaultFee sdk.Coin, burnFactor sdkmath.LegacyDec) FeeParams {
 	return FeeParams{
 		Image:      image,
 		Json:       json,
@@ -61,7 +61,7 @@ func (tfp *FeeParams) ValidateBasic() error {
 		return fmt.Errorf("invalid create resource default tx fee: %s", tfp.Json)
 	}
 
-	if !tfp.BurnFactor.IsPositive() || tfp.BurnFactor.GTE(sdk.OneDec()) {
+	if !tfp.BurnFactor.IsPositive() || tfp.BurnFactor.GTE(sdkmath.LegacyOneDec()) {
 		return fmt.Errorf("invalid burn factor: %s", tfp.BurnFactor)
 	}
 
@@ -120,7 +120,7 @@ func validateDefault(i interface{}) error {
 }
 
 func validateBurnFactor(i interface{}) error {
-	v, ok := i.(sdk.Dec)
+	v, ok := i.(sdkmath.LegacyDec)
 	if !ok {
 		return fmt.Errorf("invalid parameter type: %T", i)
 	}
@@ -133,7 +133,7 @@ func validateBurnFactor(i interface{}) error {
 		return fmt.Errorf("burn factor must not be negative: %s", v)
 	}
 
-	if v.GTE(sdk.OneDec()) {
+	if v.GTE(sdkmath.LegacyOneDec()) {
 		return fmt.Errorf("burn factor must be less than 1: %s", v)
 	}