disable flaky tests

Author: marten-seemann

dht_test.go

@@ -388,6 +388,7 @@ func TestValueSetInvalid(t *testing.T) {
 }
 
 func TestContextShutDown(t *testing.T) {
+	t.Skip("This test is flaky, see https://github.com/libp2p/go-libp2p-kad-dht/issues/724.")
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 
@@ -412,6 +413,8 @@ func TestContextShutDown(t *testing.T) {
 }
 
 func TestSearchValue(t *testing.T) {
+	t.Skip("This test is flaky, see https://github.com/libp2p/go-libp2p-kad-dht/issues/723.")
+
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 

providers/providers_manager_test.go

@@ -136,6 +136,8 @@ func TestProvidersSerialization(t *testing.T) {
 }
 
 func TestProvidesExpire(t *testing.T) {
+	t.Skip("This test is flaky, see https://github.com/libp2p/go-libp2p-kad-dht/issues/725.")
+
 	pval := ProvideValidity
 	cleanup := defaultCleanupInterval
 	ProvideValidity = time.Second / 2

rtrefresh/rt_refresh_manager_test.go

@@ -15,6 +15,8 @@ import (
 )
 
 func TestSkipRefreshOnGapCpls(t *testing.T) {
+	t.Skip("This test is flaky, see https://github.com/libp2p/go-libp2p-kad-dht/issues/722.")
+
 	ctx, cancel := context.WithCancel(context.Background())
 	defer cancel()
 	local := test.RandPeerIDFatal(t)
@@ -39,8 +41,8 @@ func TestSkipRefreshOnGapCpls(t *testing.T) {
 			p, err := rt.GenRandPeerID(uint(u))
 			require.NoError(t, err)
 			b, err := rt.TryAddPeer(p, true, false)
-			require.True(t, b)
 			require.NoError(t, err)
+			require.True(t, b)
 			return nil
 		}
 	}