Make MsgSubmitProposal a nondeterministic message

integration-tests/chain.go

@@ -119,9 +119,10 @@ func (c ChainContext) GasLimitByMultiSendMsgs(msgs ...sdk.Msg) uint64 {
 
 // BalancesOptions is the input type for the ComputeNeededBalanceFromOptions.
 type BalancesOptions struct {
-	Messages []sdk.Msg
-	GasPrice sdk.Dec
-	Amount   sdk.Int
+	Messages                    []sdk.Msg
+	NondeterministicMessagesGas uint64
+	GasPrice                    sdk.Dec
+	Amount                      sdk.Int
 }
 
 // ComputeNeededBalanceFromOptions computes the required balance based on the input options.
@@ -146,7 +147,7 @@ func (c ChainContext) ComputeNeededBalanceFromOptions(options BalancesOptions) s
 		totalAmount = totalAmount.Add(amt)
 	}
 
-	return totalAmount.Add(options.Amount)
+	return totalAmount.Add(options.GasPrice.Mul(sdk.NewDec(int64(options.NondeterministicMessagesGas))).Ceil().RoundInt()).Add(options.Amount)
 }
 
 // ChainConfig defines the config arguments required for the test chain initialisation.

integration-tests/gov.go

@@ -17,6 +17,8 @@ import (
 	"github.com/CoreumFoundation/coreum/testutil/event"
 )
 
+const submitProposalGas = 200_000
+
 // Governance keep the test chain predefined account for the governance operations.
 type Governance struct {
 	chainCtx       ChainContext
@@ -52,8 +54,8 @@ func (g Governance) ComputeProposerBalance(ctx context.Context) (sdk.Coin, error
 
 	minDeposit := govParams.DepositParams.MinDeposit[0]
 	proposerInitialBalance := g.chainCtx.ComputeNeededBalanceFromOptions(BalancesOptions{
-		Messages: []sdk.Msg{&govtypes.MsgSubmitProposal{}},
-		Amount:   minDeposit.Amount,
+		NondeterministicMessagesGas: submitProposalGas, // to cover MsgSubmitProposal
+		Amount:                      minDeposit.Amount,
 	})
 
 	return g.chainCtx.NewCoin(proposerInitialBalance), nil
@@ -124,8 +126,7 @@ func (g Governance) ProposeAndVote(ctx context.Context, proposer sdk.AccAddress,
 
 // Propose creates a new proposal.
 func (g Governance) Propose(ctx context.Context, msg *govtypes.MsgSubmitProposal) (uint64, error) {
-	txf := g.chainCtx.TxFactory().
-		WithGas(g.chainCtx.GasLimitByMsgs(&govtypes.MsgSubmitProposal{}))
+	txf := g.chainCtx.TxFactory().WithGas(submitProposalGas)
 	result, err := client.BroadcastTx(
 		ctx,
 		g.chainCtx.ClientContext.WithFromAddress(msg.GetProposer()),

integration-tests/modules/bank_test.go

@@ -80,8 +80,9 @@ func TestBankMultiSendBatchOutputs(t *testing.T) {
 	}
 
 	requireT.NoError(chain.Faucet.FundAccountsWithOptions(ctx, issuer, integrationtests.BalancesOptions{
-		Messages: append([]sdk.Msg{issueMsg}, multiSendMsgs...),
-		Amount:   chain.NetworkConfig.AssetFTConfig.IssueFee.Add(sdk.NewInt(10_000_000)), // add more coins to cover extra bytes because of the message size
+		Messages:                    append([]sdk.Msg{issueMsg}, multiSendMsgs...),
+		NondeterministicMessagesGas: 10_000_000, // to cover extra bytes because of the message size
+		Amount:                      chain.NetworkConfig.AssetFTConfig.IssueFee,
 	}))
 
 	// issue fungible tokens

x/deterministicgas/config.go

@@ -82,10 +82,9 @@ func DefaultConfig() Config {
 		MsgType(&feegranttypes.MsgRevokeAllowance{}): constantGasFunc(2500),
 
 		// gov
-		MsgType(&govtypes.MsgSubmitProposal{}): constantGasFunc(65000),
-		MsgType(&govtypes.MsgVote{}):           constantGasFunc(7000),
-		MsgType(&govtypes.MsgVoteWeighted{}):   constantGasFunc(9000),
-		MsgType(&govtypes.MsgDeposit{}):        constantGasFunc(52000),
+		MsgType(&govtypes.MsgVote{}):         constantGasFunc(7000),
+		MsgType(&govtypes.MsgVoteWeighted{}): constantGasFunc(9000),
+		MsgType(&govtypes.MsgDeposit{}):      constantGasFunc(52000),
 
 		// nft
 		MsgType(&nfttypes.MsgSend{}): constantGasFunc(16000),
@@ -111,6 +110,11 @@ func DefaultConfig() Config {
 	registerUndeterministicGasFuncs(
 		&cfg,
 		[]sdk.Msg{
+			// gov
+			// MsgSubmitProposal is defined as undeterministic because it runs a proposal handler function
+			// specific for each proposal and those functions consume unknown amount of gas.
+			&govtypes.MsgSubmitProposal{},
+
 			// crisis
 			// MsgVerifyInvariant is defined as undeterministic since fee
 			// charged by this tx type is defined as param inside module.

x/deterministicgas/config_test.go

@@ -94,6 +94,9 @@ func TestDeterministicGas_DeterministicMessages(t *testing.T) {
 
 	// WASM messages will be added here
 	undetermMsgTypes := []string{
+		// gov
+		"/cosmos.gov.v1beta1.MsgSubmitProposal",
+
 		// crisis
 		"/cosmos.crisis.v1beta1.MsgVerifyInvariant",
 
@@ -142,8 +145,8 @@ func TestDeterministicGas_DeterministicMessages(t *testing.T) {
 	// To make sure we do not increase/decrease deterministic types accidentally
 	// we assert length to be equal to exact number, so each change requires
 	// explicit adjustment of tests.
-	assert.Equal(t, 9, len(undetermMsgs))
-	assert.Equal(t, 38, len(determMsgs))
+	assert.Equal(t, 10, len(undetermMsgs))
+	assert.Equal(t, 37, len(determMsgs))
 
 	for _, sdkMsg := range determMsgs {
 		sdkMsg := sdkMsg