Disable flaky idling timeouts test
@@ -146,17 +146,18 @@ func TestEndToEnd(t *testing.T) {
cf()
require.Equal(t, connectivity.Ready, sink.grpcConn.GetState())
- time.Sleep(time.Millisecond)
+ time.Sleep(5 * time.Millisecond)
err = sink.Ingest(testSpan)
assert.NoError(t, err)
- time.Sleep(time.Millisecond)
+ time.Sleep(5 * time.Millisecond)
require.Equal(t, mock.spanCount(), 2)
}
// It should be nearly unreachable for an idle state to be reached by the
// channel, but this test ensures we handle it properly in the event that it
-// does.
-func TestClientIdleRecovery(t *testing.T) {
+// does. It can be flaky, though (it's too dependent on sleep timings), so
+// it's disabled, but preserved for future debugging purposes.
+func testClientIdleRecovery(t *testing.T) {
testaddr := "127.0.0.1:15112"
lis, err := net.Listen("tcp", testaddr)
if err != nil {
@@ -226,7 +227,7 @@ func TestClientIdleRecovery(t *testing.T) {
err = sink.Ingest(testSpan)
// This should be enough to make it through loopback TCP. Bump up if flaky.
- time.Sleep(time.Millisecond)
+ time.Sleep(5 * time.Millisecond)
testSpan.Tags = map[string]string{
"foo": "bar",
"baz": "qux",
