summaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorDana Powers <dana.powers@rd.io>2015-06-10 09:46:33 -0700
committerDana Powers <dana.powers@rd.io>2015-06-10 10:34:44 -0700
commit680a8dc3376badccccf0aab27a2307adc0b4cb0d (patch)
tree4313f419ee77eb3df2645f148d3365f404b879a4 /test
parentf021609911d25dbb7ef20410890483c1439edd58 (diff)
downloadkafka-python-680a8dc3376badccccf0aab27a2307adc0b4cb0d.tar.gz
(test) SimpleConsumer.reset_partition_offset should not raise exception on failure
Diffstat (limited to 'test')
-rw-r--r--test/test_consumer.py17
1 files changed, 16 insertions, 1 deletions
diff --git a/test/test_consumer.py b/test/test_consumer.py
index 57c69a2..df15115 100644
--- a/test/test_consumer.py
+++ b/test/test_consumer.py
@@ -4,7 +4,7 @@ from . import unittest
from kafka import SimpleConsumer, KafkaConsumer, MultiProcessConsumer
from kafka.common import (
- KafkaConfigurationError, FetchResponse,
+ KafkaConfigurationError, FetchResponse, OffsetFetchResponse,
FailedPayloadsError, OffsetAndMessage,
NotLeaderForPartitionError, UnknownTopicOrPartitionError
)
@@ -105,6 +105,21 @@ class TestSimpleConsumer(unittest.TestCase):
# This should not raise an exception
self.assertFalse(consumer.commit(partitions=[0, 1]))
+ def test_simple_consumer_reset_partition_offset(self):
+ client = MagicMock()
+
+ def mock_offset_request(payloads, **kwargs):
+ raise FailedPayloadsError(payloads[0])
+
+ client.send_offset_request.side_effect = mock_offset_request
+
+ consumer = SimpleConsumer(client, group='foobar',
+ topic='topic', partitions=[0, 1],
+ auto_commit=False)
+
+ # This should not raise an exception
+ self.assertEqual(consumer.reset_partition_offset(0), None)
+
@staticmethod
def fail_requests_factory(error_factory):
# Mock so that only the first request gets a valid response