diff options
author | Dana Powers <dana.powers@rd.io> | 2015-06-10 09:46:33 -0700 |
---|---|---|
committer | Dana Powers <dana.powers@rd.io> | 2015-06-10 10:34:44 -0700 |
commit | 680a8dc3376badccccf0aab27a2307adc0b4cb0d (patch) | |
tree | 4313f419ee77eb3df2645f148d3365f404b879a4 /test | |
parent | f021609911d25dbb7ef20410890483c1439edd58 (diff) | |
download | kafka-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.py | 17 |
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 |