-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: remove some deps from testutil/network #20843
Changes from all commits
819e55e
0621147
cb0fa95
ff51080
6beea70
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,74 @@ | ||
package codec_test | ||
|
||
import ( | ||
"testing" | ||
|
||
"github.com/stretchr/testify/require" | ||
|
||
authmodulev1 "cosmossdk.io/api/cosmos/auth/module/v1" | ||
"cosmossdk.io/core/address" | ||
"cosmossdk.io/depinject" | ||
"cosmossdk.io/log" | ||
|
||
"github.com/cosmos/cosmos-sdk/codec" | ||
) | ||
|
||
var _ address.Codec = (*customAddressCodec)(nil) | ||
|
||
type customAddressCodec struct{} | ||
|
||
func (c customAddressCodec) StringToBytes(text string) ([]byte, error) { | ||
return []byte(text), nil | ||
} | ||
|
||
func (c customAddressCodec) BytesToString(bz []byte) (string, error) { | ||
return string(bz), nil | ||
} | ||
|
||
func AuthConfig() *authmodulev1.Module { return &authmodulev1.Module{Bech32Prefix: "cosmos"} } | ||
|
||
func TestProvideAddressCodec(t *testing.T) { | ||
var addrCodec address.Codec | ||
var valAddressCodec address.ValidatorAddressCodec | ||
var consAddressCodec address.ConsensusAddressCodec | ||
|
||
err := depinject.Inject( | ||
depinject.Provide( | ||
AuthConfig, | ||
codec.ProvideAddressCodec, | ||
), | ||
&addrCodec, &valAddressCodec, &consAddressCodec) | ||
require.NoError(t, err) | ||
require.NotNil(t, addrCodec) | ||
_, ok := addrCodec.(customAddressCodec) | ||
require.False(t, ok) | ||
require.NotNil(t, valAddressCodec) | ||
_, ok = valAddressCodec.(customAddressCodec) | ||
require.False(t, ok) | ||
require.NotNil(t, consAddressCodec) | ||
_, ok = consAddressCodec.(customAddressCodec) | ||
require.False(t, ok) | ||
|
||
// Set the address codec to the custom one | ||
err = depinject.Inject( | ||
depinject.Configs( | ||
depinject.Provide(AuthConfig, codec.ProvideAddressCodec), | ||
depinject.Supply( | ||
log.NewNopLogger(), | ||
func() address.Codec { return customAddressCodec{} }, | ||
func() address.ValidatorAddressCodec { return customAddressCodec{} }, | ||
func() address.ConsensusAddressCodec { return customAddressCodec{} }, | ||
), | ||
), | ||
&addrCodec, &valAddressCodec, &consAddressCodec) | ||
require.NoError(t, err) | ||
require.NotNil(t, addrCodec) | ||
_, ok = addrCodec.(customAddressCodec) | ||
require.True(t, ok) | ||
require.NotNil(t, valAddressCodec) | ||
_, ok = valAddressCodec.(customAddressCodec) | ||
require.True(t, ok) | ||
require.NotNil(t, consAddressCodec) | ||
_, ok = consAddressCodec.(customAddressCodec) | ||
require.True(t, ok) | ||
} |
Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
@@ -12,15 +12,22 @@ import ( | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"google.golang.org/grpc" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"google.golang.org/grpc/metadata" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
_ "cosmossdk.io/x/accounts" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
_ "cosmossdk.io/x/auth" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
authclient "cosmossdk.io/x/auth/client" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
_ "cosmossdk.io/x/auth/tx/config" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
_ "cosmossdk.io/x/bank" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
banktypes "cosmossdk.io/x/bank/types" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
_ "cosmossdk.io/x/consensus" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
_ "cosmossdk.io/x/staking" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Comment on lines
+15
to
+22
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Sort the import statements. The import statements should be sorted according to the -import (
+import (
+ "context"
+ "fmt"
+ "testing"
+ "time"
+
+ "github.com/jhump/protoreflect/grpcreflect"
+ "github.com/stretchr/testify/require"
+ "github.com/stretchr/testify/suite"
+ "google.golang.org/grpc"
+ "google.golang.org/grpc/metadata"
+
+ _ "cosmossdk.io/x/accounts"
+ _ "cosmossdk.io/x/auth"
+ authclient "cosmossdk.io/x/auth/client"
+ _ "cosmossdk.io/x/auth/tx/config"
+ _ "cosmossdk.io/x/bank"
+ banktypes "cosmossdk.io/x/bank/types"
+ _ "cosmossdk.io/x/consensus"
+ _ "cosmossdk.io/x/staking"
+ stakingtypes "cosmossdk.io/x/staking/types"
+
+ "github.com/cosmos/cosmos-sdk/client"
+ reflectionv1 "github.com/cosmos/cosmos-sdk/client/grpc/reflection"
+ clienttx "github.com/cosmos/cosmos-sdk/client/tx"
+ "github.com/cosmos/cosmos-sdk/codec"
+ reflectionv2 "github.com/cosmos/cosmos-sdk/server/grpc/reflection/v2alpha1"
+ "github.com/cosmos/cosmos-sdk/testutil/configurator"
+ "github.com/cosmos/cosmos-sdk/testutil/network"
+ "github.com/cosmos/cosmos-sdk/testutil/testdata"
+ sdk "github.com/cosmos/cosmos-sdk/types"
+ grpctypes "github.com/cosmos/cosmos-sdk/types/grpc"
+ txtypes "github.com/cosmos/cosmos-sdk/types/tx"
+ "github.com/cosmos/cosmos-sdk/types/tx/signing"
+) Committable suggestion
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
stakingtypes "cosmossdk.io/x/staking/types" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"github.com/cosmos/cosmos-sdk/client" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
reflectionv1 "github.com/cosmos/cosmos-sdk/client/grpc/reflection" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
clienttx "github.com/cosmos/cosmos-sdk/client/tx" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"github.com/cosmos/cosmos-sdk/codec" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
reflectionv2 "github.com/cosmos/cosmos-sdk/server/grpc/reflection/v2alpha1" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"github.com/cosmos/cosmos-sdk/testutil/configurator" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"github.com/cosmos/cosmos-sdk/testutil/network" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
"github.com/cosmos/cosmos-sdk/testutil/testdata" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
sdk "github.com/cosmos/cosmos-sdk/types" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@@ -41,7 +48,15 @@ func (s *IntegrationTestSuite) SetupSuite() { | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
var err error | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
s.T().Log("setting up integration test suite") | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
s.cfg, err = network.DefaultConfigWithAppConfig(network.MinimumAppConfig()) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
s.cfg, err = network.DefaultConfigWithAppConfig(configurator.NewAppConfig( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
configurator.AccountsModule(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
configurator.AuthModule(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
configurator.BankModule(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
configurator.GenutilModule(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
configurator.StakingModule(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
configurator.ConsensusModule(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
configurator.TxModule(), | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
)) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
s.NoError(err) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
s.cfg.NumValidators = 1 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sort the import statements.
The import statements should be sorted according to the
gci
tool with the specified options.Committable suggestion