From c8720fef02e66b48c317246db9bf23e949ffd24f Mon Sep 17 00:00:00 2001 From: ZenGround0 Date: Thu, 12 Aug 2021 14:01:24 -0400 Subject: [PATCH] Fix tests --- itests/kit/client.go | 2 +- itests/kit/mockcli.go | 5 ++++- itests/multisig/suite.go | 3 +++ itests/paych_cli_test.go | 9 +++++---- 4 files changed, 13 insertions(+), 6 deletions(-) diff --git a/itests/kit/client.go b/itests/kit/client.go index bd81e0c04..c9f8946ec 100644 --- a/itests/kit/client.go +++ b/itests/kit/client.go @@ -26,7 +26,7 @@ func RunClientTest(t *testing.T, cmds []*lcli.Command, clientNode *TestFullNode) defer cancel() // Create mock CLI - mockCLI := NewMockCLI(ctx, t, cmds) + mockCLI := NewMockCLI(ctx, t, cmds, api.NodeFull) clientCLI := mockCLI.Client(clientNode.ListenAddr) // Get the Miner address diff --git a/itests/kit/mockcli.go b/itests/kit/mockcli.go index c0f218920..5a09a9af8 100644 --- a/itests/kit/mockcli.go +++ b/itests/kit/mockcli.go @@ -7,6 +7,7 @@ import ( "strings" "testing" + "github.com/filecoin-project/lotus/api" "github.com/multiformats/go-multiaddr" "github.com/stretchr/testify/require" lcli "github.com/urfave/cli/v2" @@ -19,7 +20,7 @@ type MockCLI struct { out *bytes.Buffer } -func NewMockCLI(ctx context.Context, t *testing.T, cmds []*lcli.Command) *MockCLI { +func NewMockCLI(ctx context.Context, t *testing.T, cmds []*lcli.Command, nodeType api.NodeType) *MockCLI { // Create a CLI App with an --api-url flag so that we can specify which node // the command should be executed against app := &lcli.App{ @@ -31,6 +32,8 @@ func NewMockCLI(ctx context.Context, t *testing.T, cmds []*lcli.Command) *MockCL }, Commands: cmds, } + // Set node type + api.RunningNodeType = nodeType var out bytes.Buffer app.Writer = &out diff --git a/itests/multisig/suite.go b/itests/multisig/suite.go index 86a8ab738..e24d9d5e7 100644 --- a/itests/multisig/suite.go +++ b/itests/multisig/suite.go @@ -8,6 +8,7 @@ import ( "testing" "github.com/filecoin-project/go-address" + "github.com/filecoin-project/lotus/api" "github.com/filecoin-project/lotus/chain/types" "github.com/filecoin-project/lotus/cli" "github.com/filecoin-project/lotus/itests/kit" @@ -19,6 +20,8 @@ func RunMultisigTests(t *testing.T, client *kit.TestFullNode) { ctx := context.Background() mockCLI := kit.NewMockCLI(ctx, t, cli.Commands) clientCLI := mockCLI.Client(client.ListenAddr) + // msig tests run against a full node + api.RunningNodeType = api.NodeFull // Create some wallets on the node to use for testing multisig var walletAddrs []address.Address diff --git a/itests/paych_cli_test.go b/itests/paych_cli_test.go index 82955e6c1..17e7bcbf6 100644 --- a/itests/paych_cli_test.go +++ b/itests/paych_cli_test.go @@ -10,6 +10,7 @@ import ( "testing" "time" + "github.com/filecoin-project/lotus/api" "github.com/filecoin-project/lotus/cli" "github.com/filecoin-project/lotus/itests/kit" @@ -42,7 +43,7 @@ func TestPaymentChannelsBasic(t *testing.T) { creatorAddr, receiverAddr := startPaychCreatorReceiverMiner(ctx, t, &paymentCreator, &paymentReceiver, blocktime) // Create mock CLI - mockCLI := kit.NewMockCLI(ctx, t, cli.Commands) + mockCLI := kit.NewMockCLI(ctx, t, cli.Commands, api.NodeFull) creatorCLI := mockCLI.Client(paymentCreator.ListenAddr) receiverCLI := mockCLI.Client(paymentReceiver.ListenAddr) @@ -98,7 +99,7 @@ func TestPaymentChannelStatus(t *testing.T) { creatorAddr, receiverAddr := startPaychCreatorReceiverMiner(ctx, t, &paymentCreator, &paymentReceiver, blocktime) // Create mock CLI - mockCLI := kit.NewMockCLI(ctx, t, cli.Commands) + mockCLI := kit.NewMockCLI(ctx, t, cli.Commands, api.NodeFull) creatorCLI := mockCLI.Client(paymentCreator.ListenAddr) // creator: paych status-by-from-to @@ -178,7 +179,7 @@ func TestPaymentChannelVouchers(t *testing.T) { creatorAddr, receiverAddr := startPaychCreatorReceiverMiner(ctx, t, &paymentCreator, &paymentReceiver, blocktime) // Create mock CLI - mockCLI := kit.NewMockCLI(ctx, t, cli.Commands) + mockCLI := kit.NewMockCLI(ctx, t, cli.Commands, api.NodeFull) creatorCLI := mockCLI.Client(paymentCreator.ListenAddr) receiverCLI := mockCLI.Client(paymentReceiver.ListenAddr) @@ -310,7 +311,7 @@ func TestPaymentChannelVoucherCreateShortfall(t *testing.T) { creatorAddr, receiverAddr := startPaychCreatorReceiverMiner(ctx, t, &paymentCreator, &paymentReceiver, blocktime) // Create mock CLI - mockCLI := kit.NewMockCLI(ctx, t, cli.Commands) + mockCLI := kit.NewMockCLI(ctx, t, cli.Commands, api.NodeFull) creatorCLI := mockCLI.Client(paymentCreator.ListenAddr) // creator: paych add-funds