Skip to content

Commit

Permalink
fix: no len check on Validators
Browse files Browse the repository at this point in the history
  • Loading branch information
Reecepbcups committed Sep 20, 2024
1 parent 78418f6 commit 087473a
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions examples/ibc/ics_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -68,15 +68,17 @@ func icsTest(t *testing.T, version string, rly ibc.RelayerImplementation) {
consumerBechPrefix = "consumer"
}

validators := 2

cf := interchaintest.NewBuiltinChainFactory(zaptest.NewLogger(t), []*interchaintest.ChainSpec{
{
Name: "ics-provider", Version: version,
NumValidators: &numVals, NumFullNodes: &numFullNodes,
NumValidators: &validators, NumFullNodes: &numFullNodes,
ChainConfig: ibc.ChainConfig{GasAdjustment: 1.5, ChainID: providerChainID, TrustingPeriod: "336h"},
},
{
Name: "ics-consumer", Version: version,
NumValidators: &numVals, NumFullNodes: &numFullNodes,
NumValidators: &validators, NumFullNodes: &numFullNodes,
ChainConfig: ibc.ChainConfig{GasAdjustment: 1.5, ChainID: "consumer-1", Bech32Prefix: consumerBechPrefix, InterchainSecurityConfig: ibc.ICSConfig{
ConsumerCopyProviderKey: func(i int) bool {
return i == 0
Expand Down Expand Up @@ -125,6 +127,8 @@ func icsTest(t *testing.T, version string, rly ibc.RelayerImplementation) {

// ------------------ ICS Setup ------------------

require.Greater(t, len(provider.Validators), 2)

// Finish the ICS provider chain initialization.
// - Restarts the relayer to connect ics20-1 transfer channel
// - Delegates tokens to the provider to update consensus value
Expand Down

0 comments on commit 087473a

Please sign in to comment.