search

libp2p / go-libp2p-circuit

Circuit Switching for libp2p
MIT License
48 stars 16 forks source link

Random Test Failure #12

Closed Stebalien closed 5 years ago

Stebalien commented 7 years ago

I occasionally get the following error when running the tests:

--- FAIL: TestFullAddressTransportDial (0.12s)
    transport_test.go:77: dial attempt failed: failed to dial <peer.ID NMJzVN> (default failure)
vyzo commented 7 years ago

Looks like it didn't make it to the dialer, this error comes from the default dial failure. Perhaps there is a race with registering the dialer?

vyzo commented 7 years ago

yes, it looks like that Test is missing the sleep delay other tests have.

vyzo commented 7 years ago

oh nm, there is a delay in testSetupRelay for all tests.

Stebalien commented 7 years ago

Personally, I like running tests in parallel to repro issues like this (messes with the scheduler).

> gx-go rw
> parallel -j7 -i go test -run TestFullAddressTransportDial -- {1..100}
raulk commented 5 years ago

Closing as a result of spring cleaning. Reopen if this is still a problem.

Stebalien commented 5 years ago

I can't reproduce. This may have been fixed with all the connection tracking fixes.