Skip to content

Commit

Permalink
Fix and enable TestConsumerInterleavedClose
Browse files Browse the repository at this point in the history
  • Loading branch information
eapache committed Apr 28, 2015
1 parent 62c68a0 commit d5141e2
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions consumer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -336,8 +336,6 @@ func TestConsumerRebalancingMultiplePartitions(t *testing.T) {
}

func TestConsumerInterleavedClose(t *testing.T) {
t.Skip("Enable once bug #325 is fixed.")

seedBroker := newMockBroker(t, 1)
leader := newMockBroker(t, 2)

Expand Down Expand Up @@ -370,6 +368,7 @@ func TestConsumerInterleavedClose(t *testing.T) {
fetchResponse := new(FetchResponse)
fetchResponse.AddMessage("my_topic", 0, nil, ByteEncoder([]byte{0x00, 0x0E}), int64(0))
leader.Returns(fetchResponse)
time.Sleep(50 * time.Millisecond)

offsetResponseNewest1 := new(OffsetResponse)
offsetResponseNewest1.AddTopicPartition("my_topic", 1, 1234)
Expand All @@ -383,7 +382,9 @@ func TestConsumerInterleavedClose(t *testing.T) {
if err != nil {
t.Fatal(err)
}
<-c0.Messages()

fetchResponse.AddMessage("my_topic", 0, nil, ByteEncoder([]byte{0x00, 0x0E}), int64(1))
fetchResponse.AddMessage("my_topic", 1, nil, ByteEncoder([]byte{0x00, 0x0E}), int64(0))
leader.Returns(fetchResponse)

Expand Down

0 comments on commit d5141e2

Please sign in to comment.