From cfd7d5369b5b442a9001f0faae111ddead4b63aa Mon Sep 17 00:00:00 2001 From: Brian Tiger Chow Date: Tue, 18 Nov 2014 21:52:58 -0800 Subject: [PATCH] test(bitswap) @whyrusleeping This appears to be a timing issue. The asynchronous nature of the new structure provides has the bitswap waiting on the context a bit more. This isn't a problem at all, but in this test, it makes the functions return in an inconveniently timely manner. TODO don't let the test depend on time. License: MIT Signed-off-by: Brian Tiger Chow --- exchange/bitswap/bitswap_test.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/exchange/bitswap/bitswap_test.go b/exchange/bitswap/bitswap_test.go index ee1e7644d..f69cb7629 100644 --- a/exchange/bitswap/bitswap_test.go +++ b/exchange/bitswap/bitswap_test.go @@ -152,6 +152,10 @@ func getOrFail(bitswap instance, b *blocks.Block, t *testing.T, wg *sync.WaitGro // TODO simplify this test. get to the _essence_! func TestSendToWantingPeer(t *testing.T) { + if testing.Short() { + t.SkipNow() + } + net := tn.VirtualNetwork() rs := mock.VirtualRoutingServer() sg := NewSessionGenerator(net, rs) @@ -167,7 +171,7 @@ func TestSendToWantingPeer(t *testing.T) { alpha := bg.Next() - const timeout = 1 * time.Millisecond // FIXME don't depend on time + const timeout = 100 * time.Millisecond // FIXME don't depend on time t.Logf("Peer %v attempts to get %v. NB: not available\n", w.peer, alpha.Key()) ctx, _ := context.WithTimeout(context.Background(), timeout)