refactor: convert keeper tests to testsuite (#10187)

<!--
The default pull request template is for types feat, fix, or refactor.
For other templates, add one of the following parameters to the url:
- template=docs.md
- template=other.md
-->

## Description

Closes: #10046 

<!-- Add a description of the changes that this PR introduces and the files that
are the most critical to review. -->

---

### Author Checklist

*All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.*

I have...

- [ ] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] added `!` to the type prefix if API or client breaking change
- [ ] targeted the correct branch (see [PR Targeting](https://github.com/cosmos/cosmos-sdk/blob/master/CONTRIBUTING.md#pr-targeting))
- [ ] provided a link to the relevant issue or specification
- [ ] followed the guidelines for [building modules](https://github.com/cosmos/cosmos-sdk/blob/master/docs/building-modules)
- [ ] included the necessary unit and integration [tests](https://github.com/cosmos/cosmos-sdk/blob/master/CONTRIBUTING.md#testing)
- [ ] added a changelog entry to `CHANGELOG.md`
- [ ] included comments for [documenting Go code](https://blog.golang.org/godoc)
- [ ] updated the relevant documentation or specification
- [ ] reviewed "Files changed" and left comments if necessary
- [ ] confirmed all CI checks have passed

### Reviewers Checklist

*All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.*

I have...

- [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title
- [ ] confirmed `!` in the type prefix if API or client breaking change
- [ ] confirmed all author checklist items have been addressed 
- [ ] reviewed state machine logic
- [ ] reviewed API design and naming
- [ ] reviewed documentation is accurate
- [ ] reviewed tests and test coverage
- [ ] manually tested (if applicable)
This commit is contained in:
NagaTulasi 2021-09-22 15:21:20 +05:30 committed by GitHub
parent 9ebd12bb93
commit 58673f85fa
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 38 additions and 40 deletions

View File

@ -375,4 +375,4 @@ func (suite *KeeperTestSuite) TestAddressStringToBytes() {
})
}
}
}

View File

@ -1,22 +0,0 @@
package keeper_test
import (
"testing"
tmproto "github.com/tendermint/tendermint/proto/tendermint/types"
"github.com/cosmos/cosmos-sdk/simapp"
sdk "github.com/cosmos/cosmos-sdk/types"
"github.com/cosmos/cosmos-sdk/x/mint/types"
)
// returns context and an app with updated mint keeper
func createTestApp(t *testing.T, isCheckTx bool) (*simapp.SimApp, sdk.Context) {
app := simapp.Setup(t, isCheckTx)
ctx := app.BaseApp.NewContext(isCheckTx, tmproto.Header{})
app.MintKeeper.SetParams(ctx, types.DefaultParams())
app.MintKeeper.SetMinter(ctx, types.DefaultInitialMinter())
return app, ctx
}

View File

@ -3,20 +3,42 @@ package keeper_test
import (
"testing"
"github.com/cosmos/cosmos-sdk/codec"
"github.com/stretchr/testify/require"
"github.com/stretchr/testify/suite"
abci "github.com/tendermint/tendermint/abci/types"
tmproto "github.com/tendermint/tendermint/proto/tendermint/types"
"github.com/cosmos/cosmos-sdk/codec"
"github.com/cosmos/cosmos-sdk/simapp"
sdk "github.com/cosmos/cosmos-sdk/types"
keep "github.com/cosmos/cosmos-sdk/x/mint/keeper"
"github.com/cosmos/cosmos-sdk/x/mint/types"
abci "github.com/tendermint/tendermint/abci/types"
)
func TestNewQuerier(t *testing.T) {
app, ctx := createTestApp(t, true)
type MintKeeperTestSuite struct {
suite.Suite
app *simapp.SimApp
ctx sdk.Context
legacyQuerierCdc *codec.AminoCodec
}
func (suite *MintKeeperTestSuite) SetupTest() {
app := simapp.Setup(suite.T(), true)
ctx := app.BaseApp.NewContext(true, tmproto.Header{})
app.MintKeeper.SetParams(ctx, types.DefaultParams())
app.MintKeeper.SetMinter(ctx, types.DefaultInitialMinter())
legacyQuerierCdc := codec.NewAminoCodec(app.LegacyAmino())
suite.app = app
suite.ctx = ctx
suite.legacyQuerierCdc = legacyQuerierCdc
}
func (suite *MintKeeperTestSuite) TestNewQuerier(t *testing.T) {
app, ctx, legacyQuerierCdc := suite.app, suite.ctx, suite.legacyQuerierCdc
querier := keep.NewQuerier(app.MintKeeper, legacyQuerierCdc.LegacyAmino)
query := abci.RequestQuery{
@ -37,9 +59,8 @@ func TestNewQuerier(t *testing.T) {
require.Error(t, err)
}
func TestQueryParams(t *testing.T) {
app, ctx := createTestApp(t, true)
legacyQuerierCdc := codec.NewAminoCodec(app.LegacyAmino())
func (suite *MintKeeperTestSuite) TestQueryParams(t *testing.T) {
app, ctx, legacyQuerierCdc := suite.app, suite.ctx, suite.legacyQuerierCdc
querier := keep.NewQuerier(app.MintKeeper, legacyQuerierCdc.LegacyAmino)
var params types.Params
@ -53,9 +74,8 @@ func TestQueryParams(t *testing.T) {
require.Equal(t, app.MintKeeper.GetParams(ctx), params)
}
func TestQueryInflation(t *testing.T) {
app, ctx := createTestApp(t, true)
legacyQuerierCdc := codec.NewAminoCodec(app.LegacyAmino())
func (suite *MintKeeperTestSuite) TestQueryInflation(t *testing.T) {
app, ctx, legacyQuerierCdc := suite.app, suite.ctx, suite.legacyQuerierCdc
querier := keep.NewQuerier(app.MintKeeper, legacyQuerierCdc.LegacyAmino)
var inflation sdk.Dec
@ -69,9 +89,8 @@ func TestQueryInflation(t *testing.T) {
require.Equal(t, app.MintKeeper.GetMinter(ctx).Inflation, inflation)
}
func TestQueryAnnualProvisions(t *testing.T) {
app, ctx := createTestApp(t, true)
legacyQuerierCdc := codec.NewAminoCodec(app.LegacyAmino())
func (suite *MintKeeperTestSuite) TestQueryAnnualProvisions(t *testing.T) {
app, ctx, legacyQuerierCdc := suite.app, suite.ctx, suite.legacyQuerierCdc
querier := keep.NewQuerier(app.MintKeeper, legacyQuerierCdc.LegacyAmino)
var annualProvisions sdk.Dec

View File

@ -26,9 +26,10 @@ type KeeperTestSuite struct {
}
func (suite *KeeperTestSuite) SetupTest() {
suite.app = simapp.Setup(suite.T(), false)
suite.ctx = suite.app.BaseApp.NewContext(false, tmproto.Header{})
app := simapp.Setup(suite.T(), true)
ctx := app.BaseApp.NewContext(true, tmproto.Header{})
suite.app, suite.ctx = app, ctx
queryHelper := baseapp.NewQueryServerTestHelper(suite.ctx, suite.app.InterfaceRegistry())
proposal.RegisterQueryServer(queryHelper, suite.app.ParamsKeeper)
suite.queryClient = proposal.NewQueryClient(queryHelper)