Skip to content

Commit a641238

Browse files
patrick-ogradypatrick-ogrady
committed
fix tests
1 parent 7f63b0d commit a641238

File tree

1 file changed

+23
-11
lines changed

1 file changed

+23
-11
lines changed

‎local/network_test.go

+23-11
Original file line numberDiff line numberDiff line change
@@ -150,6 +150,8 @@ func TestNewNetworkEmpty(t *testing.T) {
150150
"",
151151
"",
152152
false,
153+
false,
154+
false,
153155
)
154156
require.NoError(err)
155157
err = net.loadConfig(context.Background(), networkConfig)
@@ -213,6 +215,8 @@ func TestNewNetworkOneNode(t *testing.T) {
213215
"",
214216
"",
215217
false,
218+
false,
219+
false,
216220
)
217221
require.NoError(err)
218222
err = net.loadConfig(context.Background(), networkConfig)
@@ -241,6 +245,8 @@ func TestNewNetworkFailToStartNode(t *testing.T) {
241245
"",
242246
"",
243247
false,
248+
false,
249+
false,
244250
)
245251
require.NoError(err)
246252
err = net.loadConfig(context.Background(), networkConfig)
@@ -477,7 +483,7 @@ func TestWrongNetworkConfigs(t *testing.T) {
477483
require := require.New(t)
478484
for name, tt := range tests {
479485
t.Run(name, func(t *testing.T) {
480-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
486+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
481487
require.NoError(err)
482488
err = net.loadConfig(context.Background(), tt.config)
483489
require.Error(err)
@@ -491,7 +497,7 @@ func TestUnhealthyNetwork(t *testing.T) {
491497
t.Parallel()
492498
require := require.New(t)
493499
networkConfig := testNetworkConfig(t)
494-
net, err := newNetwork(logging.NoLog{}, newMockAPIUnhealthy, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
500+
net, err := newNetwork(logging.NoLog{}, newMockAPIUnhealthy, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
495501
require.NoError(err)
496502
err = net.loadConfig(context.Background(), networkConfig)
497503
require.NoError(err)
@@ -506,7 +512,7 @@ func TestGeneratedNodesNames(t *testing.T) {
506512
for i := range networkConfig.NodeConfigs {
507513
networkConfig.NodeConfigs[i].Name = ""
508514
}
509-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
515+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
510516
require.NoError(err)
511517
err = net.loadConfig(context.Background(), networkConfig)
512518
require.NoError(err)
@@ -526,7 +532,7 @@ func TestGenerateDefaultNetwork(t *testing.T) {
526532
require := require.New(t)
527533
binaryPath := "pepito"
528534
networkConfig := NewDefaultConfig(binaryPath)
529-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
535+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
530536
require.NoError(err)
531537
err = net.loadConfig(context.Background(), networkConfig)
532538
require.NoError(err)
@@ -576,7 +582,7 @@ func TestNetworkFromConfig(t *testing.T) {
576582
t.Parallel()
577583
require := require.New(t)
578584
networkConfig := testNetworkConfig(t)
579-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
585+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
580586
require.NoError(err)
581587
err = net.loadConfig(context.Background(), networkConfig)
582588
require.NoError(err)
@@ -600,7 +606,7 @@ func TestNetworkNodeOps(t *testing.T) {
600606
// Start a new, empty network
601607
emptyNetworkConfig, err := emptyNetworkConfig()
602608
require.NoError(err)
603-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
609+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
604610
require.NoError(err)
605611
err = net.loadConfig(context.Background(), emptyNetworkConfig)
606612
require.NoError(err)
@@ -638,7 +644,7 @@ func TestNodeNotFound(t *testing.T) {
638644
emptyNetworkConfig, err := emptyNetworkConfig()
639645
require.NoError(err)
640646
networkConfig := testNetworkConfig(t)
641-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
647+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
642648
require.NoError(err)
643649
err = net.loadConfig(context.Background(), emptyNetworkConfig)
644650
require.NoError(err)
@@ -671,7 +677,7 @@ func TestStoppedNetwork(t *testing.T) {
671677
emptyNetworkConfig, err := emptyNetworkConfig()
672678
require.NoError(err)
673679
networkConfig := testNetworkConfig(t)
674-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
680+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
675681
require.NoError(err)
676682
err = net.loadConfig(context.Background(), emptyNetworkConfig)
677683
require.NoError(err)
@@ -704,7 +710,7 @@ func TestStoppedNetwork(t *testing.T) {
704710
func TestGetAllNodes(t *testing.T) {
705711
require := require.New(t)
706712
networkConfig := testNetworkConfig(t)
707-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
713+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
708714
require.NoError(err)
709715
err = net.loadConfig(context.Background(), networkConfig)
710716
require.NoError(err)
@@ -756,6 +762,8 @@ func TestFlags(t *testing.T) {
756762
"",
757763
"",
758764
false,
765+
false,
766+
false,
759767
)
760768
require.NoError(err)
761769
err = nw.loadConfig(context.Background(), networkConfig)
@@ -785,6 +793,8 @@ func TestFlags(t *testing.T) {
785793
"",
786794
"",
787795
false,
796+
false,
797+
false,
788798
)
789799
require.NoError(err)
790800
err = nw.loadConfig(context.Background(), networkConfig)
@@ -813,6 +823,8 @@ func TestFlags(t *testing.T) {
813823
"",
814824
"",
815825
false,
826+
false,
827+
false,
816828
)
817829
require.NoError(err)
818830
err = nw.loadConfig(context.Background(), networkConfig)
@@ -1269,7 +1281,7 @@ func TestRemoveBeacon(t *testing.T) {
12691281
// create a network with no nodes in it
12701282
emptyNetworkConfig, err := emptyNetworkConfig()
12711283
require.NoError(err)
1272-
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
1284+
net, err := newNetwork(logging.NoLog{}, newMockAPISuccessful, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
12731285
require.NoError(err)
12741286
err = net.loadConfig(context.Background(), emptyNetworkConfig)
12751287
require.NoError(err)
@@ -1320,7 +1332,7 @@ func TestHealthyDuringNetworkStop(t *testing.T) {
13201332
require := require.New(t)
13211333
networkConfig := testNetworkConfig(t)
13221334
// Calls to a node's Healthy() function blocks until context cancelled
1323-
net, err := newNetwork(logging.NoLog{}, newMockAPIHealthyBlocks, &localTestSuccessfulNodeProcessCreator{}, "", "", false)
1335+
net, err := newNetwork(logging.NoLog{}, newMockAPIHealthyBlocks, &localTestSuccessfulNodeProcessCreator{}, "", "", false, false, false)
13241336
require.NoError(err)
13251337
err = net.loadConfig(context.Background(), networkConfig)
13261338
require.NoError(err)

0 commit comments

Comments
 (0)