From 9799dec7cc2ffad1fe313d5a4905ee54d172d373 Mon Sep 17 00:00:00 2001 From: Kevin Pita Date: Mon, 4 May 2026 15:33:14 +0200 Subject: [PATCH 1/2] refactor: remove unused poa gov keeper --- x/poa/testutil/expected_keepers_mock.go | 39 ------------------------- x/poa/types/expected_keepers.go | 5 ---- 2 files changed, 44 deletions(-) diff --git a/x/poa/testutil/expected_keepers_mock.go b/x/poa/testutil/expected_keepers_mock.go index 203a15cd..1107d1aa 100644 --- a/x/poa/testutil/expected_keepers_mock.go +++ b/x/poa/testutil/expected_keepers_mock.go @@ -10,7 +10,6 @@ import ( math "cosmossdk.io/math" types "github.com/cosmos/cosmos-sdk/types" - v1 "github.com/cosmos/cosmos-sdk/x/gov/types/v1" types0 "github.com/cosmos/cosmos-sdk/x/slashing/types" types1 "github.com/cosmos/cosmos-sdk/x/staking/types" gomock "github.com/golang/mock/gomock" @@ -455,41 +454,3 @@ func (mr *MockSlashingKeeperMockRecorder) GetParams(ctx interface{}) *gomock.Cal mr.mock.ctrl.T.Helper() return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "GetParams", reflect.TypeOf((*MockSlashingKeeper)(nil).GetParams), ctx) } - -// MockGovKeeper is a mock of GovKeeper interface. -type MockGovKeeper struct { - ctrl *gomock.Controller - recorder *MockGovKeeperMockRecorder -} - -// MockGovKeeperMockRecorder is the mock recorder for MockGovKeeper. -type MockGovKeeperMockRecorder struct { - mock *MockGovKeeper -} - -// NewMockGovKeeper creates a new mock instance. -func NewMockGovKeeper(ctrl *gomock.Controller) *MockGovKeeper { - mock := &MockGovKeeper{ctrl: ctrl} - mock.recorder = &MockGovKeeperMockRecorder{mock} - return mock -} - -// EXPECT returns an object that allows the caller to indicate expected use. -func (m *MockGovKeeper) EXPECT() *MockGovKeeperMockRecorder { - return m.recorder -} - -// SubmitProposal mocks base method. -func (m *MockGovKeeper) SubmitProposal(ctx context.Context, messages []types.Msg, metadata string) (v1.Proposal, error) { - m.ctrl.T.Helper() - ret := m.ctrl.Call(m, "SubmitProposal", ctx, messages, metadata) - ret0, _ := ret[0].(v1.Proposal) - ret1, _ := ret[1].(error) - return ret0, ret1 -} - -// SubmitProposal indicates an expected call of SubmitProposal. -func (mr *MockGovKeeperMockRecorder) SubmitProposal(ctx, messages, metadata interface{}) *gomock.Call { - mr.mock.ctrl.T.Helper() - return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "SubmitProposal", reflect.TypeOf((*MockGovKeeper)(nil).SubmitProposal), ctx, messages, metadata) -} diff --git a/x/poa/types/expected_keepers.go b/x/poa/types/expected_keepers.go index 6578a32a..11141b32 100644 --- a/x/poa/types/expected_keepers.go +++ b/x/poa/types/expected_keepers.go @@ -5,7 +5,6 @@ import ( "cosmossdk.io/math" sdk "github.com/cosmos/cosmos-sdk/types" - v1 "github.com/cosmos/cosmos-sdk/x/gov/types/v1" slashingtypes "github.com/cosmos/cosmos-sdk/x/slashing/types" stakingtypes "github.com/cosmos/cosmos-sdk/x/staking/types" ) @@ -49,7 +48,3 @@ type StakingKeeper interface { type SlashingKeeper interface { GetParams(ctx context.Context) (params slashingtypes.Params, err error) } - -type GovKeeper interface { - SubmitProposal(ctx context.Context, messages []sdk.Msg, metadata string) (v1.Proposal, error) -} From d565c96b9be8d4393b1257028ebd759ab62db8af Mon Sep 17 00:00:00 2001 From: AdriaCarrera Date: Wed, 13 May 2026 13:24:24 +0200 Subject: [PATCH 2/2] refactor: remove unused MockGovKeeper implementation from POA test utilities --- x/poa/testutil/expected_keepers_mock.go | 40 ------------------------- 1 file changed, 40 deletions(-) diff --git a/x/poa/testutil/expected_keepers_mock.go b/x/poa/testutil/expected_keepers_mock.go index 8ebf6abd..3e83d294 100644 --- a/x/poa/testutil/expected_keepers_mock.go +++ b/x/poa/testutil/expected_keepers_mock.go @@ -15,7 +15,6 @@ import ( math "cosmossdk.io/math" types "github.com/cosmos/cosmos-sdk/types" - v1 "github.com/cosmos/cosmos-sdk/x/gov/types/v1" types0 "github.com/cosmos/cosmos-sdk/x/slashing/types" types1 "github.com/cosmos/cosmos-sdk/x/staking/types" gomock "go.uber.org/mock/gomock" @@ -464,42 +463,3 @@ func (mr *MockSlashingKeeperMockRecorder) GetParams(ctx any) *gomock.Call { mr.mock.ctrl.T.Helper() return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "GetParams", reflect.TypeOf((*MockSlashingKeeper)(nil).GetParams), ctx) } - -// MockGovKeeper is a mock of GovKeeper interface. -type MockGovKeeper struct { - ctrl *gomock.Controller - recorder *MockGovKeeperMockRecorder - isgomock struct{} -} - -// MockGovKeeperMockRecorder is the mock recorder for MockGovKeeper. -type MockGovKeeperMockRecorder struct { - mock *MockGovKeeper -} - -// NewMockGovKeeper creates a new mock instance. -func NewMockGovKeeper(ctrl *gomock.Controller) *MockGovKeeper { - mock := &MockGovKeeper{ctrl: ctrl} - mock.recorder = &MockGovKeeperMockRecorder{mock} - return mock -} - -// EXPECT returns an object that allows the caller to indicate expected use. -func (m *MockGovKeeper) EXPECT() *MockGovKeeperMockRecorder { - return m.recorder -} - -// SubmitProposal mocks base method. -func (m *MockGovKeeper) SubmitProposal(ctx context.Context, messages []types.Msg, metadata string) (v1.Proposal, error) { - m.ctrl.T.Helper() - ret := m.ctrl.Call(m, "SubmitProposal", ctx, messages, metadata) - ret0, _ := ret[0].(v1.Proposal) - ret1, _ := ret[1].(error) - return ret0, ret1 -} - -// SubmitProposal indicates an expected call of SubmitProposal. -func (mr *MockGovKeeperMockRecorder) SubmitProposal(ctx, messages, metadata any) *gomock.Call { - mr.mock.ctrl.T.Helper() - return mr.mock.ctrl.RecordCallWithMethodType(mr.mock, "SubmitProposal", reflect.TypeOf((*MockGovKeeper)(nil).SubmitProposal), ctx, messages, metadata) -}