test: remove some deps from testutil/network (#20843)
This commit is contained in:
@@ -6,6 +6,8 @@ import (
|
||||
|
||||
"github.com/stretchr/testify/require"
|
||||
|
||||
"cosmossdk.io/depinject"
|
||||
|
||||
"github.com/cosmos/cosmos-sdk/server"
|
||||
clitestutil "github.com/cosmos/cosmos-sdk/testutil/cli"
|
||||
"github.com/cosmos/cosmos-sdk/testutil/network"
|
||||
@@ -14,7 +16,7 @@ import (
|
||||
func TestStatusCommand(t *testing.T) {
|
||||
t.Skip() // https://github.com/cosmos/cosmos-sdk/issues/17446
|
||||
|
||||
cfg, err := network.DefaultConfigWithAppConfig(network.MinimumAppConfig())
|
||||
cfg, err := network.DefaultConfigWithAppConfig(depinject.Configs() /* TODO, test skipped anyway */)
|
||||
require.NoError(t, err)
|
||||
|
||||
network, err := network.New(t, t.TempDir(), cfg)
|
||||
|
||||
+20
-53
@@ -6,59 +6,34 @@ import (
|
||||
"testing"
|
||||
|
||||
abci "github.com/cometbft/cometbft/api/cometbft/abci/v1"
|
||||
"github.com/stretchr/testify/suite"
|
||||
"github.com/stretchr/testify/require"
|
||||
"google.golang.org/grpc"
|
||||
"google.golang.org/grpc/metadata"
|
||||
|
||||
_ "cosmossdk.io/x/accounts"
|
||||
|
||||
"github.com/cosmos/cosmos-sdk/testutil/network"
|
||||
"github.com/cosmos/cosmos-sdk/client"
|
||||
clitestutil "github.com/cosmos/cosmos-sdk/testutil/cli"
|
||||
"github.com/cosmos/cosmos-sdk/testutil/testdata"
|
||||
"github.com/cosmos/cosmos-sdk/types/address"
|
||||
grpctypes "github.com/cosmos/cosmos-sdk/types/grpc"
|
||||
)
|
||||
|
||||
type IntegrationTestSuite struct {
|
||||
suite.Suite
|
||||
func TestCLIQueryConn(t *testing.T) {
|
||||
t.Skip("data race in comet is causing this to fail")
|
||||
|
||||
network network.NetworkI
|
||||
}
|
||||
|
||||
func (s *IntegrationTestSuite) SetupSuite() {
|
||||
s.T().Log("setting up integration test suite")
|
||||
|
||||
cfg, err := network.DefaultConfigWithAppConfig(network.MinimumAppConfig())
|
||||
|
||||
s.NoError(err)
|
||||
|
||||
s.network, err = network.New(s.T(), s.T().TempDir(), cfg)
|
||||
s.Require().NoError(err)
|
||||
|
||||
s.Require().NoError(s.network.WaitForNextBlock())
|
||||
}
|
||||
|
||||
func (s *IntegrationTestSuite) TearDownSuite() {
|
||||
s.T().Log("tearing down integration test suite")
|
||||
s.network.Cleanup()
|
||||
}
|
||||
|
||||
func (s *IntegrationTestSuite) TestCLIQueryConn() {
|
||||
s.T().Skip("data race in comet is causing this to fail")
|
||||
var header metadata.MD
|
||||
|
||||
testClient := testdata.NewQueryClient(s.network.GetValidators()[0].GetClientCtx())
|
||||
testClient := testdata.NewQueryClient(client.Context{})
|
||||
res, err := testClient.Echo(context.Background(), &testdata.EchoRequest{Message: "hello"}, grpc.Header(&header))
|
||||
s.NoError(err)
|
||||
require.NoError(t, err)
|
||||
|
||||
blockHeight := header.Get(grpctypes.GRPCBlockHeightHeader)
|
||||
height, err := strconv.Atoi(blockHeight[0])
|
||||
s.Require().NoError(err)
|
||||
s.Require().GreaterOrEqual(height, 1) // at least the 1st block
|
||||
|
||||
s.Equal("hello", res.Message)
|
||||
require.NoError(t, err)
|
||||
require.GreaterOrEqual(t, height, 1) // at least the 1st block
|
||||
require.Equal(t, "hello", res.Message)
|
||||
}
|
||||
|
||||
func (s *IntegrationTestSuite) TestQueryABCIHeight() {
|
||||
func TestQueryABCIHeight(t *testing.T) {
|
||||
testCases := []struct {
|
||||
name string
|
||||
reqHeight int64
|
||||
@@ -86,30 +61,22 @@ func (s *IntegrationTestSuite) TestQueryABCIHeight() {
|
||||
}
|
||||
|
||||
for _, tc := range testCases {
|
||||
s.Run(tc.name, func() {
|
||||
_, err := s.network.WaitForHeight(tc.expHeight)
|
||||
s.Require().NoError(err)
|
||||
|
||||
val := s.network.GetValidators()[0]
|
||||
|
||||
clientCtx := val.GetClientCtx()
|
||||
clientCtx = clientCtx.WithHeight(tc.ctxHeight)
|
||||
|
||||
t.Run(tc.name, func(t *testing.T) {
|
||||
req := abci.QueryRequest{
|
||||
Path: "store/bank/key",
|
||||
Height: tc.reqHeight,
|
||||
Data: address.MustLengthPrefix(val.GetAddress()),
|
||||
Data: address.MustLengthPrefix([]byte{}),
|
||||
Prove: true,
|
||||
}
|
||||
|
||||
res, err := clientCtx.QueryABCI(req)
|
||||
s.Require().NoError(err)
|
||||
clientCtx := client.Context{}.WithHeight(tc.ctxHeight).
|
||||
WithClient(clitestutil.NewMockCometRPC(abci.QueryResponse{
|
||||
Height: tc.expHeight,
|
||||
}))
|
||||
|
||||
s.Require().Equal(tc.expHeight, res.Height)
|
||||
res, err := clientCtx.QueryABCI(req)
|
||||
require.NoError(t, err)
|
||||
require.Equal(t, tc.expHeight, res.Height)
|
||||
})
|
||||
}
|
||||
}
|
||||
|
||||
func TestIntegrationTestSuite(t *testing.T) {
|
||||
suite.Run(t, new(IntegrationTestSuite))
|
||||
}
|
||||
|
||||
@@ -10,8 +10,6 @@ cosmossdk.io/errors v1.0.1 h1:bzu+Kcr0kS/1DuPBtUFdWjzLqyUuCiyHjyJB6srBV/0=
|
||||
cosmossdk.io/errors v1.0.1/go.mod h1:MeelVSZThMi4bEakzhhhE/CKqVv3nOJDA25bIqRDu/U=
|
||||
cosmossdk.io/math v1.3.0 h1:RC+jryuKeytIiictDslBP9i1fhkVm6ZDmZEoNP316zE=
|
||||
cosmossdk.io/math v1.3.0/go.mod h1:vnRTxewy+M7BtXBNFybkuhSH4WfedVAAnERHgVFhp3k=
|
||||
cosmossdk.io/x/accounts/defaults/lockup v0.0.0-20240417181816-5e7aae0db1f5 h1:eb0kcGyaYHSS0do7+MIWg7UKlskSH01biRNENbm/zDA=
|
||||
cosmossdk.io/x/accounts/defaults/lockup v0.0.0-20240417181816-5e7aae0db1f5/go.mod h1:drzY4oVisyWvSgpsM7ccQ7IX3efMuVIvd9Eij1Gm/6o=
|
||||
filippo.io/edwards25519 v1.1.0 h1:FNf4tywRC1HmFuKW5xopWpigGjJKiJSV0Cqo0cJWDaA=
|
||||
filippo.io/edwards25519 v1.1.0/go.mod h1:BxyFTGdWcka3PhytdK4V28tE5sGfRvvvRV7EaN4VDT4=
|
||||
github.com/99designs/go-keychain v0.0.0-20191008050251-8e49817e8af4 h1:/vQbFIOMbk2FiG/kXiLl8BRyzTWDw7gX/Hz7Dd5eDMs=
|
||||
|
||||
Reference in New Issue
Block a user