c95de9c417
* WIP: Msg authorization module added * fixing errors * fixed errors * fixed module.go * Add msg_tests * fixes compile issues * fix test * fix test * Add msg types tests * Fix Getmsgs * fixed codec issue * Fix syntax issues * Fix keeper * fixed proto issues * Fix keeper tests * fixed router in keeper * Fix query proto * Fix cli txs * Add grpc query client implementation * Add grpc-keeper test * Add grpc query tests Add revoke and exec authorization cli commands * Fix linting issues * Fix cli query * fix lint errors * Add Genesis state * Fix query authorization * Review changes * Fix grant authorization handler * Add cli tests * Add cli tests * Fix genesis test * Fix issues * update module to use proto msg services * Add simultion tests * Fix lint * fix lint * WIP simulations * WIP simulations * add msg tests * Fix simulation * Fix errors * fix genesis import export * fix sim tests * fix sim * fix test * Register RegisterMsgServer * WIP * WIP * Update keeper test * change msg_authorization module name to authz * changed type conversion for serviceMsg * serviceMsg change to any * Fix issues * fix msg tests * fix errors * proto format * remove LegacyQuerierHandler * Use MsgServiceRouter * fix keeper-test * fix query authorizations * fix NewCmdSendAs * fix simtests * fix error * fix lint * fix lint * add tests for generic authorization * fix imports * format * Update error message * remove println * add query all grants * Add pagination for queries * format * fix lint * review changes * fix grpc tests * add pagination to cli query * review changes * replace panic with error * lint * fix errors * fix tests * remove gogoproto extensions * update function doc * review changes * fix errors * fix query flags * fix grpc query test * init service-msg * remove unsed field * add proto-codec for simulations * fix codec issue * update authz simulations * change msgauth to authz * add check for invalid msg-type * change expiration flag to Unix * doc * update module.go * fix sims * fix grant-authorization sims * fix error * fix error * add build flag * fix codec issue * rename * review changes * format * review changes * go.mod * refactor * proto-gen * Update x/authz/keeper/grpc_query_test.go Co-authored-by: Amaury <amaury.martiny@protonmail.com> * Update x/authz/keeper/grpc_query_test.go Co-authored-by: Amaury <amaury.martiny@protonmail.com> * Update x/authz/keeper/grpc_query_test.go Co-authored-by: Amaury <amaury.martiny@protonmail.com> * Fix review comments * fix protogen * Follow Msg...Request style for msg requests * update comment * Fix error codes * fix review comment * improve msg validations * Handle error in casting msgs * rename actor => grantStoreKey * add godoc * add godoc * Fix simulations * Fix cli, cli_tests * Fix simulations * rename to GetOrRevokeAuthorization * Move events to keeper * Fix fmt * Update x/authz/client/cli/tx.go Co-authored-by: Amaury <amaury.martiny@protonmail.com> * rename actor * fix lint Co-authored-by: atheesh <atheesh@vitwit.com> Co-authored-by: atheeshp <59333759+atheeshp@users.noreply.github.com> Co-authored-by: Amaury Martiny <amaury.martiny@protonmail.com> Co-authored-by: MD Aleem <72057206+aleem1413@users.noreply.github.com> Co-authored-by: Anil Kumar Kammari <anil@vitwit.com>
728 lines
21 KiB
Go
728 lines
21 KiB
Go
// +build norace
|
|
|
|
package cli_test
|
|
|
|
import (
|
|
"fmt"
|
|
"strings"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/gogo/protobuf/proto"
|
|
|
|
"github.com/stretchr/testify/suite"
|
|
|
|
"github.com/cosmos/cosmos-sdk/crypto/hd"
|
|
"github.com/cosmos/cosmos-sdk/crypto/keyring"
|
|
tmcli "github.com/tendermint/tendermint/libs/cli"
|
|
|
|
"github.com/cosmos/cosmos-sdk/client/flags"
|
|
"github.com/cosmos/cosmos-sdk/testutil"
|
|
clitestutil "github.com/cosmos/cosmos-sdk/testutil/cli"
|
|
"github.com/cosmos/cosmos-sdk/testutil/network"
|
|
sdk "github.com/cosmos/cosmos-sdk/types"
|
|
"github.com/cosmos/cosmos-sdk/x/authz/client/cli"
|
|
govcli "github.com/cosmos/cosmos-sdk/x/gov/client/cli"
|
|
govtestutil "github.com/cosmos/cosmos-sdk/x/gov/client/testutil"
|
|
govtypes "github.com/cosmos/cosmos-sdk/x/gov/types"
|
|
|
|
"github.com/cosmos/cosmos-sdk/x/authz/types"
|
|
bankcli "github.com/cosmos/cosmos-sdk/x/bank/client/testutil"
|
|
banktestutil "github.com/cosmos/cosmos-sdk/x/bank/client/testutil"
|
|
)
|
|
|
|
type IntegrationTestSuite struct {
|
|
suite.Suite
|
|
|
|
cfg network.Config
|
|
network *network.Network
|
|
grantee sdk.AccAddress
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) SetupSuite() {
|
|
s.T().Log("setting up integration test suite")
|
|
|
|
cfg := network.DefaultConfig()
|
|
cfg.NumValidators = 1
|
|
|
|
s.cfg = cfg
|
|
s.network = network.New(s.T(), cfg)
|
|
|
|
val := s.network.Validators[0]
|
|
|
|
// Create new account in the keyring.
|
|
info, _, err := val.ClientCtx.Keyring.NewMnemonic("grantee", keyring.English, sdk.FullFundraiserPath, hd.Secp256k1)
|
|
s.Require().NoError(err)
|
|
newAddr := sdk.AccAddress(info.GetPubKey().Address())
|
|
|
|
// Send some funds to the new account.
|
|
_, err = banktestutil.MsgSendExec(
|
|
val.ClientCtx,
|
|
val.Address,
|
|
newAddr,
|
|
sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(200))), fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
)
|
|
s.Require().NoError(err)
|
|
s.grantee = newAddr
|
|
|
|
// create a proposal with deposit
|
|
_, err = govtestutil.MsgSubmitProposal(val.ClientCtx, val.Address.String(),
|
|
"Text Proposal 1", "Where is the title!?", govtypes.ProposalTypeText,
|
|
fmt.Sprintf("--%s=%s", govcli.FlagDeposit, sdk.NewCoin(s.cfg.BondDenom, govtypes.DefaultMinDepositTokens).String()))
|
|
s.Require().NoError(err)
|
|
|
|
_, err = s.network.WaitForHeight(1)
|
|
s.Require().NoError(err)
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) TearDownSuite() {
|
|
s.T().Log("tearing down integration test suite")
|
|
s.network.Cleanup()
|
|
}
|
|
|
|
var typeMsgSend = types.SendAuthorization{}.MethodName()
|
|
var typeMsgVote = "/cosmos.gov.v1beta1.Msg/Vote"
|
|
|
|
func (s *IntegrationTestSuite) TestQueryAuthorizations() {
|
|
val := s.network.Validators[0]
|
|
|
|
grantee := s.grantee
|
|
twoHours := time.Now().Add(time.Minute * time.Duration(120)).Unix()
|
|
|
|
_, err := execGrantAuthorization(
|
|
val,
|
|
[]string{
|
|
grantee.String(),
|
|
"send",
|
|
fmt.Sprintf("--%s=100steak", cli.FlagSpendLimit),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
)
|
|
s.Require().NoError(err)
|
|
|
|
testCases := []struct {
|
|
name string
|
|
args []string
|
|
expectErr bool
|
|
expErrMsg string
|
|
}{
|
|
{
|
|
"Error: Invalid grantee",
|
|
[]string{
|
|
val.Address.String(),
|
|
"invalid grantee",
|
|
fmt.Sprintf("--%s=json", tmcli.OutputFlag),
|
|
},
|
|
true,
|
|
"decoding bech32 failed: invalid character in string: ' '",
|
|
},
|
|
{
|
|
"Error: Invalid granter",
|
|
[]string{
|
|
"invalid granter",
|
|
grantee.String(),
|
|
fmt.Sprintf("--%s=json", tmcli.OutputFlag),
|
|
},
|
|
true,
|
|
"decoding bech32 failed: invalid character in string: ' '",
|
|
},
|
|
{
|
|
"Valid txn (json)",
|
|
[]string{
|
|
val.Address.String(),
|
|
grantee.String(),
|
|
fmt.Sprintf("--%s=json", tmcli.OutputFlag),
|
|
},
|
|
false,
|
|
``,
|
|
},
|
|
}
|
|
for _, tc := range testCases {
|
|
tc := tc
|
|
|
|
s.Run(tc.name, func() {
|
|
cmd := cli.GetCmdQueryAuthorizations()
|
|
clientCtx := val.ClientCtx
|
|
resp, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
|
|
if tc.expectErr {
|
|
s.Require().Error(err)
|
|
s.Require().Contains(string(resp.Bytes()), tc.expErrMsg)
|
|
} else {
|
|
s.Require().NoError(err)
|
|
var grants types.QueryAuthorizationsResponse
|
|
err = val.ClientCtx.JSONMarshaler.UnmarshalJSON(resp.Bytes(), &grants)
|
|
s.Require().NoError(err)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) TestQueryAuthorization() {
|
|
val := s.network.Validators[0]
|
|
|
|
grantee := s.grantee
|
|
twoHours := time.Now().Add(time.Minute * time.Duration(120)).Unix()
|
|
|
|
_, err := execGrantAuthorization(
|
|
val,
|
|
[]string{
|
|
grantee.String(),
|
|
"send",
|
|
fmt.Sprintf("--%s=100steak", cli.FlagSpendLimit),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
)
|
|
s.Require().NoError(err)
|
|
|
|
testCases := []struct {
|
|
name string
|
|
args []string
|
|
expectErr bool
|
|
expectedOutput string
|
|
}{
|
|
{
|
|
"Error: Invalid grantee",
|
|
[]string{
|
|
val.Address.String(),
|
|
"invalid grantee",
|
|
typeMsgSend,
|
|
fmt.Sprintf("--%s=json", tmcli.OutputFlag),
|
|
},
|
|
true,
|
|
"",
|
|
},
|
|
{
|
|
"Error: Invalid granter",
|
|
[]string{
|
|
"invalid granter",
|
|
grantee.String(),
|
|
typeMsgSend,
|
|
fmt.Sprintf("--%s=json", tmcli.OutputFlag),
|
|
},
|
|
true,
|
|
"",
|
|
},
|
|
{
|
|
"no authorization found",
|
|
[]string{
|
|
val.Address.String(),
|
|
grantee.String(),
|
|
"typeMsgSend",
|
|
fmt.Sprintf("--%s=json", tmcli.OutputFlag),
|
|
},
|
|
true,
|
|
"",
|
|
},
|
|
{
|
|
"Valid txn (json)",
|
|
[]string{
|
|
val.Address.String(),
|
|
grantee.String(),
|
|
typeMsgSend,
|
|
fmt.Sprintf("--%s=json", tmcli.OutputFlag),
|
|
},
|
|
false,
|
|
`{"@type":"/cosmos.authz.v1beta1.SendAuthorization","spend_limit":[{"denom":"steak","amount":"100"}]}`,
|
|
},
|
|
}
|
|
for _, tc := range testCases {
|
|
tc := tc
|
|
|
|
s.Run(tc.name, func() {
|
|
cmd := cli.GetCmdQueryAuthorization()
|
|
clientCtx := val.ClientCtx
|
|
out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
|
|
if tc.expectErr {
|
|
s.Require().Error(err)
|
|
} else {
|
|
s.Require().NoError(err)
|
|
s.Require().Contains(strings.TrimSpace(out.String()), tc.expectedOutput)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) TestCLITxGrantAuthorization() {
|
|
val := s.network.Validators[0]
|
|
grantee := s.grantee
|
|
|
|
twoHours := time.Now().Add(time.Minute * time.Duration(120)).Unix()
|
|
pastHour := time.Now().Add(time.Minute * time.Duration(-60)).Unix()
|
|
|
|
testCases := []struct {
|
|
name string
|
|
args []string
|
|
respType proto.Message
|
|
expectedCode uint32
|
|
expectErr bool
|
|
}{
|
|
{
|
|
"Invalid granter Address",
|
|
[]string{
|
|
"grantee_addr",
|
|
"send",
|
|
fmt.Sprintf("--%s=100steak", cli.FlagSpendLimit),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, "granter"),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
},
|
|
nil,
|
|
0,
|
|
true,
|
|
},
|
|
{
|
|
"Invalid grantee Address",
|
|
[]string{
|
|
"grantee_addr",
|
|
"send",
|
|
fmt.Sprintf("--%s=100steak", cli.FlagSpendLimit),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
},
|
|
nil, 0,
|
|
true,
|
|
},
|
|
{
|
|
"Invalid expiration time",
|
|
[]string{
|
|
grantee.String(),
|
|
"send",
|
|
fmt.Sprintf("--%s=100steak", cli.FlagSpendLimit),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, pastHour),
|
|
},
|
|
nil, 0,
|
|
true,
|
|
},
|
|
{
|
|
"fail with error invalid msg-type",
|
|
[]string{
|
|
grantee.String(),
|
|
"generic",
|
|
fmt.Sprintf("--%s=invalid-msg-type", cli.FlagMsgType),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
&sdk.TxResponse{}, 29,
|
|
false,
|
|
},
|
|
{
|
|
"Valid tx send authorization",
|
|
[]string{
|
|
grantee.String(),
|
|
"send",
|
|
fmt.Sprintf("--%s=100steak", cli.FlagSpendLimit),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
&sdk.TxResponse{}, 0,
|
|
false,
|
|
},
|
|
{
|
|
"Valid tx generic authorization",
|
|
[]string{
|
|
grantee.String(),
|
|
"generic",
|
|
fmt.Sprintf("--%s=%s", cli.FlagMsgType, typeMsgVote),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
&sdk.TxResponse{}, 0,
|
|
false,
|
|
},
|
|
}
|
|
|
|
for _, tc := range testCases {
|
|
tc := tc
|
|
s.Run(tc.name, func() {
|
|
clientCtx := val.ClientCtx
|
|
out, err := execGrantAuthorization(
|
|
val,
|
|
tc.args,
|
|
)
|
|
if tc.expectErr {
|
|
s.Require().Error(err)
|
|
} else {
|
|
s.Require().NoError(err)
|
|
s.Require().NoError(clientCtx.JSONMarshaler.UnmarshalJSON(out.Bytes(), tc.respType), out.String())
|
|
txResp := tc.respType.(*sdk.TxResponse)
|
|
s.Require().Equal(tc.expectedCode, txResp.Code, out.String())
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func execGrantAuthorization(val *network.Validator, args []string) (testutil.BufferWriter, error) {
|
|
cmd := cli.NewCmdGrantAuthorization()
|
|
clientCtx := val.ClientCtx
|
|
return clitestutil.ExecTestCLICmd(clientCtx, cmd, args)
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) TestCmdRevokeAuthorizations() {
|
|
val := s.network.Validators[0]
|
|
|
|
grantee := s.grantee
|
|
twoHours := time.Now().Add(time.Minute * time.Duration(120)).Unix()
|
|
|
|
// send-authorization
|
|
_, err := execGrantAuthorization(
|
|
val,
|
|
[]string{
|
|
grantee.String(),
|
|
"send",
|
|
fmt.Sprintf("--%s=100steak", cli.FlagSpendLimit),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
)
|
|
s.Require().NoError(err)
|
|
|
|
// generic-authorization
|
|
_, err = execGrantAuthorization(
|
|
val,
|
|
[]string{
|
|
grantee.String(),
|
|
"generic",
|
|
fmt.Sprintf("--%s=%s", cli.FlagMsgType, typeMsgVote),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
)
|
|
s.Require().NoError(err)
|
|
|
|
testCases := []struct {
|
|
name string
|
|
args []string
|
|
respType proto.Message
|
|
expectedCode uint32
|
|
expectErr bool
|
|
}{
|
|
{
|
|
"invalid grantee address",
|
|
[]string{
|
|
"invlid grantee",
|
|
typeMsgSend,
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
},
|
|
nil,
|
|
0,
|
|
true,
|
|
},
|
|
{
|
|
"invalid granter address",
|
|
[]string{
|
|
grantee.String(),
|
|
typeMsgSend,
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, "granter"),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
},
|
|
nil,
|
|
0,
|
|
true,
|
|
},
|
|
{
|
|
"Valid tx send authorization",
|
|
[]string{
|
|
grantee.String(),
|
|
typeMsgSend,
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
&sdk.TxResponse{}, 0,
|
|
false,
|
|
},
|
|
{
|
|
"Valid tx generic authorization",
|
|
[]string{
|
|
grantee.String(),
|
|
typeMsgVote,
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
&sdk.TxResponse{}, 0,
|
|
false,
|
|
},
|
|
}
|
|
for _, tc := range testCases {
|
|
tc := tc
|
|
s.Run(tc.name, func() {
|
|
cmd := cli.NewCmdRevokeAuthorization()
|
|
clientCtx := val.ClientCtx
|
|
|
|
out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
|
|
if tc.expectErr {
|
|
s.Require().Error(err)
|
|
} else {
|
|
s.Require().NoError(err)
|
|
s.Require().NoError(clientCtx.JSONMarshaler.UnmarshalJSON(out.Bytes(), tc.respType), out.String())
|
|
|
|
txResp := tc.respType.(*sdk.TxResponse)
|
|
s.Require().Equal(tc.expectedCode, txResp.Code, out.String())
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) TestExecAuthorizationWithExpiration() {
|
|
val := s.network.Validators[0]
|
|
grantee := s.grantee
|
|
tenSeconds := time.Now().Add(time.Second * time.Duration(10)).Unix()
|
|
|
|
_, err := execGrantAuthorization(
|
|
val,
|
|
[]string{
|
|
grantee.String(),
|
|
"generic",
|
|
fmt.Sprintf("--%s=%s", cli.FlagMsgType, typeMsgVote),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, tenSeconds),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
)
|
|
s.Require().NoError(err)
|
|
// msg vote
|
|
voteTx := fmt.Sprintf(`{"body":{"messages":[{"@type":"/cosmos.gov.v1beta1.Msg/Vote","proposal_id":"1","voter":"%s","option":"VOTE_OPTION_YES"}],"memo":"","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""}},"signatures":[]}`, val.Address.String())
|
|
execMsg := testutil.WriteToNewTempFile(s.T(), voteTx)
|
|
|
|
// waiting for authorization to expires
|
|
time.Sleep(12 * time.Second)
|
|
|
|
cmd := cli.NewCmdExecAuthorization()
|
|
clientCtx := val.ClientCtx
|
|
|
|
res, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, []string{
|
|
execMsg.Name(),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, grantee.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
})
|
|
s.Require().NoError(err)
|
|
s.Require().Contains(res.String(), "authorization not found")
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) TestNewExecGenericAuthorized() {
|
|
val := s.network.Validators[0]
|
|
grantee := s.grantee
|
|
twoHours := time.Now().Add(time.Minute * time.Duration(120)).Unix()
|
|
|
|
_, err := execGrantAuthorization(
|
|
val,
|
|
[]string{
|
|
grantee.String(),
|
|
"generic",
|
|
fmt.Sprintf("--%s=%s", cli.FlagMsgType, typeMsgVote),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
)
|
|
s.Require().NoError(err)
|
|
|
|
// msg vote
|
|
voteTx := fmt.Sprintf(`{"body":{"messages":[{"@type":"/cosmos.gov.v1beta1.Msg/Vote","proposal_id":"1","voter":"%s","option":"VOTE_OPTION_YES"}],"memo":"","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""}},"signatures":[]}`, val.Address.String())
|
|
execMsg := testutil.WriteToNewTempFile(s.T(), voteTx)
|
|
|
|
testCases := []struct {
|
|
name string
|
|
args []string
|
|
respType proto.Message
|
|
expectedCode uint32
|
|
expectErr bool
|
|
}{
|
|
{
|
|
"fail invalid grantee",
|
|
[]string{
|
|
execMsg.Name(),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, "grantee"),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
},
|
|
nil,
|
|
0,
|
|
true,
|
|
},
|
|
{
|
|
"fail invalid json path",
|
|
[]string{
|
|
"/invalid/file.txt",
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, grantee.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
},
|
|
nil,
|
|
0,
|
|
true,
|
|
},
|
|
{
|
|
"valid txn",
|
|
[]string{
|
|
execMsg.Name(),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, grantee.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
},
|
|
&sdk.TxResponse{},
|
|
0,
|
|
false,
|
|
},
|
|
}
|
|
|
|
for _, tc := range testCases {
|
|
tc := tc
|
|
s.Run(tc.name, func() {
|
|
|
|
cmd := cli.NewCmdExecAuthorization()
|
|
clientCtx := val.ClientCtx
|
|
|
|
out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
|
|
if tc.expectErr {
|
|
s.Require().Error(err)
|
|
} else {
|
|
s.Require().NoError(err)
|
|
s.Require().NoError(clientCtx.JSONMarshaler.UnmarshalJSON(out.Bytes(), tc.respType), out.String())
|
|
txResp := tc.respType.(*sdk.TxResponse)
|
|
s.Require().Equal(tc.expectedCode, txResp.Code, out.String())
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func (s *IntegrationTestSuite) TestNewExecGrantAuthorized() {
|
|
val := s.network.Validators[0]
|
|
grantee := s.grantee
|
|
twoHours := time.Now().Add(time.Minute * time.Duration(120)).Unix()
|
|
|
|
_, err := execGrantAuthorization(
|
|
val,
|
|
[]string{
|
|
grantee.String(),
|
|
"send",
|
|
fmt.Sprintf("--%s=12%stoken", cli.FlagSpendLimit, val.Moniker),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, val.Address.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%d", cli.FlagExpiration, twoHours),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
},
|
|
)
|
|
s.Require().NoError(err)
|
|
tokens := sdk.NewCoins(
|
|
sdk.NewCoin(fmt.Sprintf("%stoken", val.Moniker), sdk.NewInt(12)),
|
|
)
|
|
normalGeneratedTx, err := bankcli.ServiceMsgSendExec(
|
|
val.ClientCtx,
|
|
val.Address,
|
|
grantee,
|
|
tokens,
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
)
|
|
s.Require().NoError(err)
|
|
execMsg := testutil.WriteToNewTempFile(s.T(), normalGeneratedTx.String())
|
|
testCases := []struct {
|
|
name string
|
|
args []string
|
|
respType proto.Message
|
|
expectedCode uint32
|
|
expectErr bool
|
|
}{
|
|
{
|
|
"fail invalid grantee",
|
|
[]string{
|
|
execMsg.Name(),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, "grantee"),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=true", flags.FlagGenerateOnly),
|
|
},
|
|
nil,
|
|
0,
|
|
true,
|
|
},
|
|
{
|
|
"fail invalid json path",
|
|
[]string{
|
|
"/invalid/file.txt",
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, grantee.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
},
|
|
nil,
|
|
0,
|
|
true,
|
|
},
|
|
{
|
|
"valid txn",
|
|
[]string{
|
|
execMsg.Name(),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFrom, grantee.String()),
|
|
fmt.Sprintf("--%s=%s", flags.FlagBroadcastMode, flags.BroadcastBlock),
|
|
fmt.Sprintf("--%s=%s", flags.FlagFees, sdk.NewCoins(sdk.NewCoin(s.cfg.BondDenom, sdk.NewInt(10))).String()),
|
|
fmt.Sprintf("--%s=true", flags.FlagSkipConfirmation),
|
|
},
|
|
&sdk.TxResponse{},
|
|
0,
|
|
false,
|
|
},
|
|
}
|
|
|
|
for _, tc := range testCases {
|
|
tc := tc
|
|
s.Run(tc.name, func() {
|
|
cmd := cli.NewCmdExecAuthorization()
|
|
clientCtx := val.ClientCtx
|
|
|
|
out, err := clitestutil.ExecTestCLICmd(clientCtx, cmd, tc.args)
|
|
if tc.expectErr {
|
|
s.Require().Error(err)
|
|
} else {
|
|
s.Require().NoError(err)
|
|
s.Require().NoError(clientCtx.JSONMarshaler.UnmarshalJSON(out.Bytes(), tc.respType), out.String())
|
|
txResp := tc.respType.(*sdk.TxResponse)
|
|
s.Require().Equal(tc.expectedCode, txResp.Code, out.String())
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestIntegrationTestSuite(t *testing.T) {
|
|
suite.Run(t, new(IntegrationTestSuite))
|
|
}
|