test: add CLI, gRPC, MsgServer tests (#1405)

* Add global test codec

* Add TransferTxCmd test

* Add the remaining tx cli tests

* Add cli query test

* Add fbridge auth test case

* Add grpc-query test

* Add msg server tests

* Add type tests

* Add CHANGELOG

CHANGELOG.md

@@ -67,6 +67,7 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * (x/fbridge) [\#1395](https://github.com/Finschia/finschia-sdk/pull/1395) Return error instead of panic for behaviors triggered by client
 * (x/fswap) [\#1396](https://github.com/Finschia/finschia-sdk/pull/1396) refactor to use snake_case in proto
 * (x/fswap) [\#1391](https://github.com/Finschia/finschia-sdk/pull/1391) add cli_test for fswap module
+* (x/fbridge) [\#1405](https://github.com/Finschia/finschia-sdk/pull/1405) Add CLI, gRPC, MsgServer tests
 * (x/fswap) [\#1415](https://github.com/Finschia/finschia-sdk/pull/1415) add more testcases for fswap module
 
 ### Bug Fixes

client/tendermint.go

@@ -0,0 +1,28 @@
+package client
+
+import (
+	"context"
+
+	rpcclient "github.com/tendermint/tendermint/rpc/client"
+	coretypes "github.com/tendermint/tendermint/rpc/core/types"
+)
+
+// TendermintRPC defines the interface of a Tendermint RPC client needed for
+// queries and transaction handling.
+type TendermintRPC interface {
+	rpcclient.ABCIClient
+
+	Validators(ctx context.Context, height *int64, page, perPage *int) (*coretypes.ResultValidators, error)
+	Status(context.Context) (*coretypes.ResultStatus, error)
+	Block(ctx context.Context, height *int64) (*coretypes.ResultBlock, error)
+	BlockchainInfo(ctx context.Context, minHeight, maxHeight int64) (*coretypes.ResultBlockchainInfo, error)
+	Commit(ctx context.Context, height *int64) (*coretypes.ResultCommit, error)
+	Tx(ctx context.Context, hash []byte, prove bool) (*coretypes.ResultTx, error)
+	TxSearch(
+		ctx context.Context,
+		query string,
+		prove bool,
+		page, perPage *int,
+		orderBy string,
+	) (*coretypes.ResultTxSearch, error)
+}

testutil/cli/tm_mocks.go

@@ -0,0 +1,41 @@
+package cli
+
+import (
+	"context"
+
+	abci "github.com/tendermint/tendermint/abci/types"
+	tmbytes "github.com/tendermint/tendermint/libs/bytes"
+	rpcclient "github.com/tendermint/tendermint/rpc/client"
+	rpcclientmock "github.com/tendermint/tendermint/rpc/client/mock"
+	coretypes "github.com/tendermint/tendermint/rpc/core/types"
+	tmtypes "github.com/tendermint/tendermint/types"
+
+	"github.com/Finschia/finschia-sdk/client"
+)
+
+var _ client.TendermintRPC = (*MockTendermintRPC)(nil)
+
+type MockTendermintRPC struct {
+	rpcclientmock.Client
+
+	responseQuery abci.ResponseQuery
+}
+
+// NewMockTendermintRPC returns a mock TendermintRPC implementation.
+// It is used for CLI testing.
+func NewMockTendermintRPC(respQuery abci.ResponseQuery) MockTendermintRPC {
+	return MockTendermintRPC{responseQuery: respQuery}
+}
+
+func (MockTendermintRPC) BroadcastTxSync(context.Context, tmtypes.Tx) (*coretypes.ResultBroadcastTx, error) {
+	return &coretypes.ResultBroadcastTx{Code: 0}, nil
+}
+
+func (m MockTendermintRPC) ABCIQueryWithOptions(
+	_ context.Context,
+	_ string,
+	_ tmbytes.HexBytes,
+	_ rpcclient.ABCIQueryOptions,
+) (*coretypes.ResultABCIQuery, error) {
+	return &coretypes.ResultABCIQuery{Response: m.responseQuery}, nil
+}

types/module/testutil/codec.go

@@ -0,0 +1,78 @@
+package testutil
+
+import (
+	"github.com/Finschia/finschia-sdk/client"
+	"github.com/Finschia/finschia-sdk/codec"
+	"github.com/Finschia/finschia-sdk/codec/types"
+	"github.com/Finschia/finschia-sdk/std"
+	"github.com/Finschia/finschia-sdk/types/module"
+	"github.com/Finschia/finschia-sdk/x/auth/tx"
+)
+
+// TestEncodingConfig defines an encoding configuration that is used for testing
+// purposes. Note, MakeTestEncodingConfig takes a series of AppModuleBasic types
+// which should only contain the relevant module being tested and any potential
+// dependencies.
+type TestEncodingConfig struct {
+	InterfaceRegistry types.InterfaceRegistry
+	Codec             codec.Codec
+	TxConfig          client.TxConfig
+	Amino             *codec.LegacyAmino
+}
+
+func MakeTestEncodingConfig(modules ...module.AppModuleBasic) TestEncodingConfig {
+	aminoCdc := codec.NewLegacyAmino()
+	interfaceRegistry := types.NewInterfaceRegistry()
+	cdc := codec.NewProtoCodec(interfaceRegistry)
+
+	encCfg := TestEncodingConfig{
+		InterfaceRegistry: interfaceRegistry,
+		Codec:             cdc,
+		TxConfig:          tx.NewTxConfig(cdc, tx.DefaultSignModes),
+		Amino:             aminoCdc,
+	}
+
+	mb := module.NewBasicManager(modules...)
+
+	std.RegisterLegacyAminoCodec(encCfg.Amino)
+	std.RegisterInterfaces(encCfg.InterfaceRegistry)
+	mb.RegisterLegacyAminoCodec(encCfg.Amino)
+	mb.RegisterInterfaces(encCfg.InterfaceRegistry)
+
+	return encCfg
+}
+
+func MakeTestTxConfig() client.TxConfig {
+	interfaceRegistry := types.NewInterfaceRegistry()
+	cdc := codec.NewProtoCodec(interfaceRegistry)
+	return tx.NewTxConfig(cdc, tx.DefaultSignModes)
+}
+
+type TestBuilderTxConfig struct {
+	client.TxConfig
+	TxBuilder *TestTxBuilder
+}
+
+func MakeBuilderTestTxConfig() TestBuilderTxConfig {
+	return TestBuilderTxConfig{
+		TxConfig: MakeTestTxConfig(),
+	}
+}
+
+func (cfg TestBuilderTxConfig) NewTxBuilder() client.TxBuilder {
+	if cfg.TxBuilder == nil {
+		cfg.TxBuilder = &TestTxBuilder{
+			TxBuilder: cfg.TxConfig.NewTxBuilder(),
+		}
+	}
+	return cfg.TxBuilder
+}
+
+type TestTxBuilder struct {
+	client.TxBuilder
+	ExtOptions []*types.Any
+}
+
+func (b *TestTxBuilder) SetExtensionOptions(extOpts ...*types.Any) {
+	b.ExtOptions = extOpts
+}

x/fbridge/client/cli/query.go

@@ -13,7 +13,7 @@ import (
 )
 
 const (
-	flagSequences = "sequences"
+	FlagSequences = "sequences"
 )
 
 // NewQueryCmd returns the query commands for fbridge module
@@ -100,7 +100,7 @@ func NewQuerySeqToBlocknumsCmd() *cobra.Command {
 			}
 			qc := types.NewQueryClient(clientCtx)
 
-			seqSlice, err := cmd.Flags().GetInt64Slice(flagSequences)
+			seqSlice, err := cmd.Flags().GetInt64Slice(FlagSequences)
 			if err != nil {
 				return err
 			}
@@ -119,7 +119,7 @@ func NewQuerySeqToBlocknumsCmd() *cobra.Command {
 		},
 	}
 
-	cmd.Flags().Int64Slice(flagSequences, []int64{}, "comma separated list of bridge sequnece numbers")
+	cmd.Flags().Int64Slice(FlagSequences, []int64{}, "comma separated list of bridge sequnece numbers")
 	flags.AddQueryFlagsToCmd(cmd)
 	return cmd
 }