610b2eec6a
## Description Reverts the usage of a singleton `legacy.Cdc` codec while (de)serializing `x/authz` messages. Closes: #11643 --- ### 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)
206 lines
7.3 KiB
Go
206 lines
7.3 KiB
Go
package simulation_test
|
|
|
|
import (
|
|
"math/rand"
|
|
"testing"
|
|
|
|
"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/simapp"
|
|
simappparams "github.com/cosmos/cosmos-sdk/simapp/params"
|
|
sdk "github.com/cosmos/cosmos-sdk/types"
|
|
simtypes "github.com/cosmos/cosmos-sdk/types/simulation"
|
|
"github.com/cosmos/cosmos-sdk/x/bank/simulation"
|
|
"github.com/cosmos/cosmos-sdk/x/bank/testutil"
|
|
"github.com/cosmos/cosmos-sdk/x/bank/types"
|
|
)
|
|
|
|
type SimTestSuite struct {
|
|
suite.Suite
|
|
|
|
ctx sdk.Context
|
|
app *simapp.SimApp
|
|
}
|
|
|
|
func (suite *SimTestSuite) SetupTest() {
|
|
checkTx := false
|
|
app := simapp.Setup(suite.T(), checkTx)
|
|
suite.app = app
|
|
suite.ctx = app.BaseApp.NewContext(checkTx, tmproto.Header{})
|
|
}
|
|
|
|
// TestWeightedOperations tests the weights of the operations.
|
|
func (suite *SimTestSuite) TestWeightedOperations() {
|
|
cdc := suite.app.AppCodec()
|
|
appParams := make(simtypes.AppParams)
|
|
|
|
weightesOps := simulation.WeightedOperations(appParams, cdc, suite.app.AccountKeeper, suite.app.BankKeeper)
|
|
|
|
// setup 3 accounts
|
|
s := rand.NewSource(1)
|
|
r := rand.New(s)
|
|
accs := suite.getTestingAccounts(r, 3)
|
|
|
|
expected := []struct {
|
|
weight int
|
|
opMsgRoute string
|
|
opMsgName string
|
|
}{
|
|
{simappparams.DefaultWeightMsgSend, types.ModuleName, types.TypeMsgSend},
|
|
{simappparams.DefaultWeightMsgMultiSend, types.ModuleName, types.TypeMsgMultiSend},
|
|
}
|
|
|
|
for i, w := range weightesOps {
|
|
operationMsg, _, _ := w.Op()(r, suite.app.BaseApp, suite.ctx, accs, "")
|
|
// the following checks are very much dependent from the ordering of the output given
|
|
// by WeightedOperations. if the ordering in WeightedOperations changes some tests
|
|
// will fail
|
|
suite.Require().Equal(expected[i].weight, w.Weight(), "weight should be the same")
|
|
suite.Require().Equal(expected[i].opMsgRoute, operationMsg.Route, "route should be the same")
|
|
suite.Require().Equal(expected[i].opMsgName, operationMsg.Name, "operation Msg name should be the same")
|
|
}
|
|
}
|
|
|
|
// TestSimulateMsgSend tests the normal scenario of a valid message of type TypeMsgSend.
|
|
// Abonormal scenarios, where the message is created by an errors, are not tested here.
|
|
func (suite *SimTestSuite) TestSimulateMsgSend() {
|
|
// setup 3 accounts
|
|
s := rand.NewSource(1)
|
|
r := rand.New(s)
|
|
accounts := suite.getTestingAccounts(r, 3)
|
|
|
|
// begin a new block
|
|
suite.app.BeginBlock(abci.RequestBeginBlock{Header: tmproto.Header{Height: suite.app.LastBlockHeight() + 1, AppHash: suite.app.LastCommitID().Hash}})
|
|
|
|
// execute operation
|
|
op := simulation.SimulateMsgSend(suite.app.AccountKeeper, suite.app.BankKeeper)
|
|
operationMsg, futureOperations, err := op(r, suite.app.BaseApp, suite.ctx, accounts, "")
|
|
suite.Require().NoError(err)
|
|
|
|
var msg types.MsgSend
|
|
types.ModuleCdc.UnmarshalJSON(operationMsg.Msg, &msg)
|
|
|
|
suite.Require().True(operationMsg.OK)
|
|
suite.Require().Equal("65337742stake", msg.Amount.String())
|
|
suite.Require().Equal("cosmos1ghekyjucln7y67ntx7cf27m9dpuxxemn4c8g4r", msg.FromAddress)
|
|
suite.Require().Equal("cosmos1p8wcgrjr4pjju90xg6u9cgq55dxwq8j7u4x9a0", msg.ToAddress)
|
|
suite.Require().Equal(types.TypeMsgSend, msg.Type())
|
|
suite.Require().Equal(types.ModuleName, msg.Route())
|
|
suite.Require().Len(futureOperations, 0)
|
|
}
|
|
|
|
// TestSimulateMsgSend tests the normal scenario of a valid message of type TypeMsgMultiSend.
|
|
// Abonormal scenarios, where the message is created by an errors, are not tested here.
|
|
func (suite *SimTestSuite) TestSimulateMsgMultiSend() {
|
|
// setup 3 accounts
|
|
s := rand.NewSource(1)
|
|
r := rand.New(s)
|
|
accounts := suite.getTestingAccounts(r, 3)
|
|
|
|
// begin a new block
|
|
suite.app.BeginBlock(abci.RequestBeginBlock{Header: tmproto.Header{Height: suite.app.LastBlockHeight() + 1, AppHash: suite.app.LastCommitID().Hash}})
|
|
|
|
// execute operation
|
|
op := simulation.SimulateMsgMultiSend(suite.app.AccountKeeper, suite.app.BankKeeper)
|
|
operationMsg, futureOperations, err := op(r, suite.app.BaseApp, suite.ctx, accounts, "")
|
|
require := suite.Require()
|
|
require.NoError(err)
|
|
|
|
var msg types.MsgMultiSend
|
|
types.ModuleCdc.UnmarshalJSON(operationMsg.Msg, &msg)
|
|
|
|
require.True(operationMsg.OK)
|
|
require.Len(msg.Inputs, 3)
|
|
require.Equal("cosmos1p8wcgrjr4pjju90xg6u9cgq55dxwq8j7u4x9a0", msg.Inputs[1].Address)
|
|
require.Equal("185121068stake", msg.Inputs[1].Coins.String())
|
|
require.Len(msg.Outputs, 2)
|
|
require.Equal("cosmos1ghekyjucln7y67ntx7cf27m9dpuxxemn4c8g4r", msg.Outputs[1].Address)
|
|
require.Equal("260469617stake", msg.Outputs[1].Coins.String())
|
|
require.Equal(types.TypeMsgMultiSend, msg.Type())
|
|
require.Equal(types.ModuleName, msg.Route())
|
|
require.Len(futureOperations, 0)
|
|
}
|
|
|
|
func (suite *SimTestSuite) TestSimulateModuleAccountMsgSend() {
|
|
const (
|
|
accCount = 1
|
|
moduleAccCount = 1
|
|
)
|
|
|
|
s := rand.NewSource(1)
|
|
r := rand.New(s)
|
|
accounts := suite.getTestingAccounts(r, accCount)
|
|
|
|
// begin a new block
|
|
suite.app.BeginBlock(abci.RequestBeginBlock{Header: tmproto.Header{Height: suite.app.LastBlockHeight() + 1, AppHash: suite.app.LastCommitID().Hash}})
|
|
|
|
// execute operation
|
|
op := simulation.SimulateMsgSendToModuleAccount(suite.app.AccountKeeper, suite.app.BankKeeper, moduleAccCount)
|
|
|
|
s = rand.NewSource(1)
|
|
r = rand.New(s)
|
|
|
|
operationMsg, futureOperations, err := op(r, suite.app.BaseApp, suite.ctx, accounts, "")
|
|
suite.Require().Error(err)
|
|
|
|
var msg types.MsgSend
|
|
types.ModuleCdc.UnmarshalJSON(operationMsg.Msg, &msg)
|
|
|
|
suite.Require().False(operationMsg.OK)
|
|
suite.Require().Equal(operationMsg.Comment, "invalid transfers")
|
|
suite.Require().Equal(types.TypeMsgSend, msg.Type())
|
|
suite.Require().Equal(types.ModuleName, msg.Route())
|
|
suite.Require().Len(futureOperations, 0)
|
|
}
|
|
|
|
func (suite *SimTestSuite) TestSimulateMsgMultiSendToModuleAccount() {
|
|
const (
|
|
accCount = 2
|
|
mAccCount = 2
|
|
)
|
|
|
|
s := rand.NewSource(1)
|
|
r := rand.New(s)
|
|
accounts := suite.getTestingAccounts(r, accCount)
|
|
|
|
// begin a new block
|
|
suite.app.BeginBlock(abci.RequestBeginBlock{Header: tmproto.Header{Height: suite.app.LastBlockHeight() + 1, AppHash: suite.app.LastCommitID().Hash}})
|
|
|
|
// execute operation
|
|
op := simulation.SimulateMsgMultiSendToModuleAccount(suite.app.AccountKeeper, suite.app.BankKeeper, mAccCount)
|
|
|
|
operationMsg, futureOperations, err := op(r, suite.app.BaseApp, suite.ctx, accounts, "")
|
|
suite.Require().Error(err)
|
|
|
|
var msg types.MsgMultiSend
|
|
types.ModuleCdc.UnmarshalJSON(operationMsg.Msg, &msg)
|
|
|
|
suite.Require().False(operationMsg.OK) // sending tokens to a module account should fail
|
|
suite.Require().Equal(operationMsg.Comment, "invalid transfers")
|
|
suite.Require().Equal(types.TypeMsgMultiSend, msg.Type())
|
|
suite.Require().Equal(types.ModuleName, msg.Route())
|
|
suite.Require().Len(futureOperations, 0)
|
|
}
|
|
|
|
func (suite *SimTestSuite) getTestingAccounts(r *rand.Rand, n int) []simtypes.Account {
|
|
accounts := simtypes.RandomAccounts(r, n)
|
|
|
|
initAmt := suite.app.StakingKeeper.TokensFromConsensusPower(suite.ctx, 200)
|
|
initCoins := sdk.NewCoins(sdk.NewCoin(sdk.DefaultBondDenom, initAmt))
|
|
|
|
// add coins to the accounts
|
|
for _, account := range accounts {
|
|
acc := suite.app.AccountKeeper.NewAccountWithAddress(suite.ctx, account.Address)
|
|
suite.app.AccountKeeper.SetAccount(suite.ctx, acc)
|
|
suite.Require().NoError(testutil.FundAccount(suite.app.BankKeeper, suite.ctx, account.Address, initCoins))
|
|
}
|
|
|
|
return accounts
|
|
}
|
|
|
|
func TestSimTestSuite(t *testing.T) {
|
|
suite.Run(t, new(SimTestSuite))
|
|
}
|