From e33bd24350673ef2dd2b721f9c7a7a8cdb6c8931 Mon Sep 17 00:00:00 2001 From: TxCorpi0x <6095314+TxCorpi0x@users.noreply.github.com> Date: Wed, 11 Mar 2026 19:37:04 +0300 Subject: [PATCH] Move MsgSubmitProposal of group module to non-determinstic gas --- x/deterministicgas/config.go | 2 +- x/deterministicgas/config_test.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/x/deterministicgas/config.go b/x/deterministicgas/config.go index d2f882cc..ab1ab87b 100644 --- a/x/deterministicgas/config.go +++ b/x/deterministicgas/config.go @@ -156,7 +156,6 @@ func DefaultConfig() Config { MsgToMsgURL(&group.MsgUpdateGroupPolicyAdmin{}): constantGasFunc(20_000), MsgToMsgURL(&group.MsgUpdateGroupPolicyDecisionPolicy{}): constantGasFunc(17_000), MsgToMsgURL(&group.MsgUpdateGroupPolicyMetadata{}): constantGasFunc(15_000), - MsgToMsgURL(&group.MsgWithdrawProposal{}): constantGasFunc(22_000), MsgToMsgURL(&group.MsgLeaveGroup{}): constantGasFunc(17_500), // nft @@ -266,6 +265,7 @@ func DefaultConfig() Config { // For MsgSubmitProposal we have access to the list of messages but estimation depends on Exec attribute // value that is why we decided to make it non-deterministic to simple logic and consistent with other 2. &group.MsgSubmitProposal{}, + &group.MsgWithdrawProposal{}, &group.MsgVote{}, &group.MsgExec{}, diff --git a/x/deterministicgas/config_test.go b/x/deterministicgas/config_test.go index 80083ea9..d7ce5488 100644 --- a/x/deterministicgas/config_test.go +++ b/x/deterministicgas/config_test.go @@ -97,8 +97,8 @@ func TestDeterministicGas_DeterministicMessages(t *testing.T) { // To make sure we do not increase/decrease deterministic and extension types accidentally, // we assert length to be equal to exact number, so each change requires // explicit adjustment of tests. - assert.Equal(t, 95, nondeterministicMsgCount) - assert.Equal(t, 68, deterministicMsgCount) + assert.Equal(t, 96, nondeterministicMsgCount) + assert.Equal(t, 67, deterministicMsgCount) assert.Equal(t, 12, extensionMsgCount) assert.Equal(t, 151, nonExtensionMsgCount) }