mirror/erigon-pulse
1
0
Fork 0
mirror of https://gitlab.com/pulsechaincom/erigon-pulse.git synced 2024-12-22 03:30:37 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

devnet: integration tests port clash fix (#9194)

Browse Source 
Integration tests CI is failing due to a port clash in devnet tests. I
believe this is because there are 2 packages of devnet integration tests
and go can run tests from separate packages in parallel (by default it
does package level parallelism). A simple fix would be to just have all
devnet integration tests in 1 package and run all these tests
sequentially within the package (ie not use t.Parallel). This PR moves
all devnet integration tests in 1 package.

`"ContextStart devnet start failed: private api: could not create
listener: listen top 127.0.0.1:10090: bind: address already in use,
addr=localhost:10090"`

![Screenshot 2024-01-10 at 13 38
37](https://github.com/ledgerwatch/erigon/assets/94537774/06bda987-45e5-46ef-9e0b-3876b3f85c01)
This commit is contained in:
milen 2024-01-10 19:04:27 +00:00 committed by GitHub
parent b05ffc909d
commit f690301c03
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
cmd/devnet/tests/bor/devnet_test.go → cmd/devnet/tests/bor_devnet_test.go
View File

@ -1,6 +1,6 @@
//go:build integration
package bor
package tests
import (
"context"
@ -13,11 +13,10 @@ import (
contracts_steps "github.com/ledgerwatch/erigon/cmd/devnet/contracts/steps"
"github.com/ledgerwatch/erigon/cmd/devnet/requests"
"github.com/ledgerwatch/erigon/cmd/devnet/services"
"github.com/ledgerwatch/erigon/cmd/devnet/tests"
)
func TestStateSync(t *testing.T) {
runCtx, err := tests.ContextStart(t, networkname.BorDevnetChainName)
runCtx, err := ContextStart(t, networkname.BorDevnetChainName)
require.Nil(t, err)
var ctx context.Context = runCtx
@ -56,7 +55,7 @@ func TestStateSync(t *testing.T) {
func TestChildChainExit(t *testing.T) {
t.Skip("FIXME: step CreateAccountWithFunds fails: Failed to get transfer tx: failed to search reserves for hashes: no block heads subscription")
runCtx, err := tests.ContextStart(t, networkname.BorDevnetChainName)
runCtx, err := ContextStart(t, networkname.BorDevnetChainName)
require.Nil(t, err)
var ctx context.Context = runCtx

6
cmd/devnet/tests/context.go
View File

@ -7,18 +7,19 @@ import (
"strconv"
"testing"
"github.com/ledgerwatch/log/v3"
"github.com/ledgerwatch/erigon-lib/chain/networkname"
"github.com/ledgerwatch/erigon/cmd/devnet/devnet"
"github.com/ledgerwatch/erigon/cmd/devnet/networks"
"github.com/ledgerwatch/erigon/cmd/devnet/services"
"github.com/ledgerwatch/erigon/cmd/devnet/services/polygon"
"github.com/ledgerwatch/erigon/turbo/debug"
"github.com/ledgerwatch/log/v3"
)
func initDevnet(chainName string, dataDir string, producerCount int, gasLimit uint64, logger log.Logger, consoleLogLevel log.Lvl, dirLogLevel log.Lvl) (devnet.Devnet, error) {
const baseRpcHost = "localhost"
const baseRpcPort = 8545
const baseRpcPort = 9545
switch chainName {
case networkname.BorDevnetChainName:
@ -42,6 +43,7 @@ func initDevnet(chainName string, dataDir string, producerCount int, gasLimit ui
}
func ContextStart(t *testing.T, chainName string) (devnet.Context, error) {
//goland:noinspection GoBoolExpressions
if runtime.GOOS == "windows" {
t.Skip("FIXME: TempDir RemoveAll cleanup error: remove dev-0\\clique\\db\\clique\\mdbx.dat: The process cannot access the file because it is being used by another process")
}

9
cmd/devnet/tests/generic/devnet_test.go → cmd/devnet/tests/generic_devnet_test.go
View File

@ -1,6 +1,6 @@
//go:build integration
package generic
package tests
import (
"context"
@ -14,7 +14,6 @@ import (
"github.com/ledgerwatch/erigon/cmd/devnet/contracts/steps"
"github.com/ledgerwatch/erigon/cmd/devnet/requests"
"github.com/ledgerwatch/erigon/cmd/devnet/services"
"github.com/ledgerwatch/erigon/cmd/devnet/tests"
"github.com/ledgerwatch/erigon/cmd/devnet/transactions"
)
@ -40,19 +39,19 @@ func testDynamicTx(t *testing.T, ctx context.Context) {
}
func TestDynamicTxNode0(t *testing.T) {
runCtx, err := tests.ContextStart(t, "")
runCtx, err := ContextStart(t, "")
require.Nil(t, err)
testDynamicTx(t, runCtx.WithCurrentNetwork(0).WithCurrentNode(0))
}
func TestDynamicTxAnyNode(t *testing.T) {
runCtx, err := tests.ContextStart(t, "")
runCtx, err := ContextStart(t, "")
require.Nil(t, err)
testDynamicTx(t, runCtx.WithCurrentNetwork(0))
}
func TestCallContract(t *testing.T) {
runCtx, err := tests.ContextStart(t, "")
runCtx, err := ContextStart(t, "")
require.Nil(t, err)
ctx := runCtx.WithCurrentNetwork(0)