go-libp2p-quic-transport icon indicating copy to clipboard operation
go-libp2p-quic-transport copied to clipboard

test: libp2p transport test suite

Open marten-seemann opened this issue 5 years ago • 8 comments

Taking over #13. Can't figure out how to commit to the original branch and continue that PR.

marten-seemann avatar Dec 07 '19 06:12 marten-seemann

The tests are failing because we're running into the StressTestTimeout (60s). It's just that without a decent test framework, we don't get a helpful error message.

marten-seemann avatar Dec 07 '19 08:12 marten-seemann

@marten-seemann it would be really nice to get this passing.

Stebalien avatar May 22 '20 02:05 Stebalien

I've updated to the latest test-suite and made the build pass again: https://github.com/liamsi/go-libp2p-quic-transport/tree/transport-test-updated

The tests fail locally but with other fails than StressTestTimeout (60s) (or at least I don't see it if this happens under the hood): https://github.com/liamsi/go-libp2p-quic-transport/runs/3039813604

Edit: Although it seems the SubtestStress1Conn1Stream100Msg is the only one that panics.

liamsi avatar Jul 11 '21 10:07 liamsi

@liamsi pulled.

Stebalien avatar Jul 22 '21 23:07 Stebalien

Ok, after https://github.com/libp2p/go-libp2p-testing/pull/34, everything but windows is passing, and windows is failing consistently. This looks like it might be a QUIC bug.

Stebalien avatar Jul 23 '21 01:07 Stebalien

Hm. I take that back? I'm getting:

  stream_suite.go:241: timeout: no recent network activity

On ubuntu as well, but not as reliably.

Also, after a quite a few runs, I'm getting "open sockets", even though I'm cleaning them up. Clearly we're something is failing to garbage collect (could be the test?).

Stebalien avatar Jul 23 '21 01:07 Stebalien

Also, after a quite a few runs, I'm getting "open sockets", even though I'm cleaning them up. Clearly we're something is failing to garbage collect (could be the test?).

Ok, that's just us hanging on to old sessions.

Stebalien avatar Jul 23 '21 02:07 Stebalien

"retired connections"

Stebalien avatar Jul 23 '21 02:07 Stebalien