diff options
author | Jeff Widman <jeff@jeffwidman.com> | 2017-09-12 13:56:06 -0700 |
---|---|---|
committer | Jeff Widman <jeff@jeffwidman.com> | 2017-09-12 13:56:06 -0700 |
commit | 84aec293f7074b4fe46741cd2be694d1efd91386 (patch) | |
tree | 3f3d17b526653c45f062aed8b8ae2d4984e0d86a /test/test_sender.py | |
parent | 370169029a9104f1963227005349f1dc6420a924 (diff) | |
download | kafka-python-84aec293f7074b4fe46741cd2be694d1efd91386.tar.gz |
Stop using mutable types for default arg values1212-stop-using-mutable-types-for-default-arg-values
Using mutable types for default args is typically a no-no unless their
surprising behavior is being explicitly abused, for an explanation see:
http://effbot.org/zone/default-values.htm
Fix #1212
Diffstat (limited to 'test/test_sender.py')
-rw-r--r-- | test/test_sender.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/test_sender.py b/test/test_sender.py index cf911e1..f37e194 100644 --- a/test/test_sender.py +++ b/test/test_sender.py @@ -19,7 +19,7 @@ from kafka.structs import TopicPartition, OffsetAndMetadata @pytest.fixture def client(mocker): - _cli = mocker.Mock(spec=KafkaClient(bootstrap_servers=[], api_version=(0, 9))) + _cli = mocker.Mock(spec=KafkaClient(bootstrap_servers=(), api_version=(0, 9))) _cli.cluster = mocker.Mock(spec=ClusterMetadata()) return _cli |