e2e: Add bootstrap checks to migrated kurtosis tests (#1935)

Author: maru-ava

tests/e2e/README.md

@@ -76,3 +76,15 @@ ginkgo -v ./tests/e2e -- \
 ```
 
 See the testnet fixture [README](../fixture/testnet/README.md) for more details.
+
+## Skipping bootstrap checks
+
+By default many tests will attempt to bootstrap a new node with the
+post-test network state. While this is a valuable activity to perform
+in CI, it can add considerable latency to test development. To disable
+these bootstrap checks during development, set the
+`E2E_SKIP_BOOTSTRAP_CHECKS` env var to a non-empty value:
+
+```bash
+E2E_SKIP_BOOTSTRAP_CHECKS=1 ginkgo -v ./tests/e2e ...
+```

tests/e2e/c/interchain_workflow.go

@@ -163,5 +163,7 @@ var _ = e2e.DescribeCChain("[Interchain Workflow]", func() {
 			require.NoError(err)
 			require.Greater(balances[avaxAssetID], uint64(0))
 		})
+
+		e2e.CheckBootstrapIsPossible(e2e.Env.GetNetwork())
 	})
 })

tests/e2e/e2e.go

@@ -11,6 +11,7 @@ import (
 	"fmt"
 	"math/big"
 	"math/rand"
+	"os"
 	"strings"
 	"time"
 
@@ -53,6 +54,11 @@ const (
 	// Interval appropriate for network operations that should be
 	// retried periodically but not too often.
 	DefaultPollingInterval = 500 * time.Millisecond
+
+	// Setting this env will disable post-test bootstrap
+	// checks. Useful for speeding up iteration during test
+	// development.
+	SkipBootstrapChecksEnvName = "E2E_SKIP_BOOTSTRAP_CHECKS"
 )
 
 // Env is used to access shared test fixture. Intended to be
@@ -247,3 +253,14 @@ func WithSuggestedGasPrice(ethClient ethclient.Client) common.Option {
 	baseFee := SuggestGasPrice(ethClient)
 	return common.WithBaseFee(baseFee)
 }
+
+// Verify that a new node can bootstrap into the network.
+func CheckBootstrapIsPossible(network testnet.Network) {
+	if len(os.Getenv(SkipBootstrapChecksEnvName)) > 0 {
+		tests.Outf("{{yellow}}Skipping bootstrap check due to the %s env var being set", SkipBootstrapChecksEnvName)
+		return
+	}
+	ginkgo.By("checking if bootstrap is possible with the current network state")
+	node := AddEphemeralNode(network, testnet.FlagsMap{})
+	WaitForHealthy(node)
+}

tests/e2e/faultinjection/duplicate_node_id.go

@@ -57,6 +57,8 @@ var _ = ginkgo.Describe("Duplicate node handling", func() {
 
 		ginkgo.By("checking that the second new node is connected to its peers")
 		checkConnectedPeers(nodes, node2)
+
+		// A bootstrap check was already performed by the second node.
 	})
 })
 

tests/e2e/x/interchain_workflow.go

@@ -148,5 +148,7 @@ var _ = e2e.DescribeXChain("[Interchain Workflow]", func() {
 			require.NoError(err)
 			require.Greater(balances[avaxAssetID], uint64(0))
 		})
+
+		e2e.CheckBootstrapIsPossible(e2e.Env.GetNetwork())
 	})
 })