Skip to content

Commit

Permalink
fix: blobstream tests by using v1
Browse files Browse the repository at this point in the history
  • Loading branch information
rootulp committed Apr 24, 2024
1 parent fbe7354 commit 1061c24
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
4 changes: 3 additions & 1 deletion app/test/qgb_rpc_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,9 @@ func TestBlobstreamRPCQueries(t *testing.T) {
t.Skip("skipping blobstream integration test in short mode.")
}
ecfg := encoding.MakeConfig(app.ModuleEncodingRegisters...)
cfg := testnode.DefaultConfig().WithModifiers(genesis.SetDataCommitmentWindow(ecfg.Codec, 100))
cfg := testnode.DefaultConfig().
WithModifiers(genesis.SetDataCommitmentWindow(ecfg.Codec, 100)).
WithConsensusParams(app.DefaultInitialConsensusParams())

cctx, _, _ := testnode.NewNetwork(t, cfg)

Expand Down
6 changes: 2 additions & 4 deletions x/blobstream/client/suite_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package client_test
import (
"testing"

"github.com/celestiaorg/celestia-app/v2/app"
"github.com/celestiaorg/celestia-app/v2/test/util/testnode"
"github.com/stretchr/testify/suite"
tmrand "github.com/tendermint/tendermint/libs/rand"
Expand All @@ -19,17 +20,14 @@ func (s *CLITestSuite) SetupSuite() {
s.T().Skip("skipping Blobstream CLI tests in short mode.")
}

cfg := testnode.DefaultConfig()
cfg := testnode.DefaultConfig().WithConsensusParams(app.DefaultInitialConsensusParams())

numAccounts := 120
accounts := make([]string, numAccounts)
for i := 0; i < numAccounts; i++ {
accounts[i] = tmrand.Str(20)
}
cfg.WithFundedAccounts(accounts...)
consensusParams := testnode.DefaultConsensusParams()
consensusParams.Version.AppVersion = 1
cfg.WithConsensusParams(consensusParams)

s.cfg = cfg
s.cctx, _, _ = testnode.NewNetwork(s.T(), cfg)
Expand Down
6 changes: 3 additions & 3 deletions x/blobstream/integration_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,9 @@ func (s *BlobstreamIntegrationSuite) SetupSuite() {

s.accounts = []string{"jimmy"}

consensusParams := app.DefaultConsensusParams()
consensusParams.Version.AppVersion = 1
cfg := testnode.DefaultConfig().WithFundedAccounts(s.accounts...).WithConsensusParams(consensusParams)
cfg := testnode.DefaultConfig().
WithFundedAccounts(s.accounts...).
WithConsensusParams(app.DefaultInitialConsensusParams())
cctx, _, _ := testnode.NewNetwork(t, cfg)
s.ecfg = encoding.MakeConfig(app.ModuleEncodingRegisters...)
s.cctx = cctx
Expand Down

0 comments on commit 1061c24

Please sign in to comment.