diff options
author | Jeff Widman <jeff@jeffwidman.com> | 2019-03-14 17:34:59 -0700 |
---|---|---|
committer | Dana Powers <dana.powers@gmail.com> | 2019-03-14 17:34:59 -0700 |
commit | 965d21b21387e69c53f583fd76cdcec2a4d0f944 (patch) | |
tree | 81b157cc73458aec84f5fc4687763f6a869c0d97 /test/test_consumer.py | |
parent | 812de351f75beefe73bd9bef55847ab61ccc951d (diff) | |
download | kafka-python-965d21b21387e69c53f583fd76cdcec2a4d0f944.tar.gz |
Error if connections_max_idle_ms not larger than request_timeout_ms (#1688)
Diffstat (limited to 'test/test_consumer.py')
-rw-r--r-- | test/test_consumer.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/test/test_consumer.py b/test/test_consumer.py index 4ea01c8..edcc2d8 100644 --- a/test/test_consumer.py +++ b/test/test_consumer.py @@ -15,11 +15,15 @@ from kafka.structs import ( class TestKafkaConsumer: def test_session_timeout_larger_than_request_timeout_raises(self): with pytest.raises(KafkaConfigurationError): - KafkaConsumer(bootstrap_servers='localhost:9092', api_version=(0,9), group_id='foo', session_timeout_ms=60000, request_timeout_ms=40000) + KafkaConsumer(bootstrap_servers='localhost:9092', api_version=(0, 9), group_id='foo', session_timeout_ms=50000, request_timeout_ms=40000) def test_fetch_max_wait_larger_than_request_timeout_raises(self): with pytest.raises(KafkaConfigurationError): - KafkaConsumer(bootstrap_servers='localhost:9092', fetch_max_wait_ms=41000, request_timeout_ms=40000) + KafkaConsumer(bootstrap_servers='localhost:9092', fetch_max_wait_ms=50000, request_timeout_ms=40000) + + def test_request_timeout_larger_than_connections_max_idle_ms_raises(self): + with pytest.raises(KafkaConfigurationError): + KafkaConsumer(bootstrap_servers='localhost:9092', api_version=(0, 9), request_timeout_ms=50000, connections_max_idle_ms=40000) def test_subscription_copy(self): consumer = KafkaConsumer('foo', api_version=(0, 10)) |