Skip to content

Commit

Permalink
Merge pull request #958 from Aiven-Open/jjaakola-aiven-remove-test-co…
Browse files Browse the repository at this point in the history
…mmitted-flaky-test

test: remove flaky TestCommitted test
  • Loading branch information
nosahama authored Sep 24, 2024
2 parents 57dfafc + 6897daa commit 6be6b46
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions tests/integration/kafka/test_consumer.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"""
from __future__ import annotations

from aiokafka.errors import IllegalStateError, KafkaTimeoutError, UnknownTopicOrPartitionError
from aiokafka.errors import IllegalStateError, UnknownTopicOrPartitionError
from confluent_kafka import OFFSET_BEGINNING, OFFSET_END, TopicPartition
from confluent_kafka.admin import NewTopic
from confluent_kafka.error import KafkaError
Expand Down Expand Up @@ -180,12 +180,6 @@ def test_commit_raises_for_unknown_partition(
consumer.commit(offsets=[TopicPartition(new_topic.topic, partition=99, offset=0)])


class TestCommitted:
def test_committed_raises_on_timeout(self, consumer: KafkaConsumer) -> None:
with pytest.raises(KafkaTimeoutError):
consumer.committed([], timeout=0.00001)


class TestSubscribe:
def _poll_until_no_message(self, consumer: KafkaConsumer) -> None:
"""Polls until there is no message returned.
Expand Down

0 comments on commit 6be6b46

Please sign in to comment.