From fa6e6794928762b0158f5b999968f6644be82a56 Mon Sep 17 00:00:00 2001 From: Marko Date: Tue, 6 Dec 2022 16:24:23 +0000 Subject: [PATCH] refactor: remove suite from chaindecorator handler test #14173 --- types/handler_test.go | 24 ++++++------------------ 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/types/handler_test.go b/types/handler_test.go index d656360a4a..14c38d2824 100644 --- a/types/handler_test.go +++ b/types/handler_test.go @@ -4,34 +4,22 @@ import ( "testing" "github.com/golang/mock/gomock" - "github.com/stretchr/testify/suite" + "github.com/stretchr/testify/require" "github.com/cosmos/cosmos-sdk/testutil/mock" sdk "github.com/cosmos/cosmos-sdk/types" ) -type handlerTestSuite struct { - suite.Suite -} - -func TestHandlerTestSuite(t *testing.T) { - suite.Run(t, new(handlerTestSuite)) -} - -func (s *handlerTestSuite) SetupSuite() { - s.T().Parallel() -} - -func (s *handlerTestSuite) TestChainAnteDecorators() { +func TestChainAnteDecorators(t *testing.T) { // test panic - s.Require().Nil(sdk.ChainAnteDecorators([]sdk.AnteDecorator{}...)) + require.Nil(t, sdk.ChainAnteDecorators([]sdk.AnteDecorator{}...)) ctx, tx := sdk.Context{}, sdk.Tx(nil) - mockCtrl := gomock.NewController(s.T()) + mockCtrl := gomock.NewController(t) mockAnteDecorator1 := mock.NewMockAnteDecorator(mockCtrl) mockAnteDecorator1.EXPECT().AnteHandle(gomock.Eq(ctx), gomock.Eq(tx), true, gomock.Any()).Times(1) _, err := sdk.ChainAnteDecorators(mockAnteDecorator1)(ctx, tx, true) - s.Require().NoError(err) + require.NoError(t, err) mockAnteDecorator2 := mock.NewMockAnteDecorator(mockCtrl) // NOTE: we can't check that mockAnteDecorator2 is passed as the last argument because @@ -43,5 +31,5 @@ func (s *handlerTestSuite) TestChainAnteDecorators() { _, err = sdk.ChainAnteDecorators( mockAnteDecorator1, mockAnteDecorator2)(ctx, tx, true) - s.Require().NoError(err) + require.NoError(t, err) }