pubsub: disable flaky test
This test flaked in
Probably the 50ms sleep: it seems that Travis's machines behave differently
with short sleeps. Since polling pull is going away soon, I just disable it.
@@ -123,9 +123,8 @@ func TestAfterAbortReturnsNoMoreThanOneMessage(t *testing.T) {
// Each test case is excercised by making two concurrent blocking calls on a
// messageIterator, and then aborting the iterator.
// The result should be one call to Next returning a message, and the other returning an error.
- if useStreamingPull {
- t.Skip("iterator tests are for polling pull only")
- }
+ t.Skip(`This test has subtle timing dependencies, making it flaky.
+It is not worth fixing because iterators will be removed shortly.`)
type testCase struct {
abort func(*messageIterator, context.CancelFunc)
// want is the error that should be returned from one Next invocation.
