From 84aec293f7074b4fe46741cd2be694d1efd91386 Mon Sep 17 00:00:00 2001 From: Jeff Widman Date: Tue, 12 Sep 2017 13:56:06 -0700 Subject: 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 --- test/test_client.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'test/test_client.py') diff --git a/test/test_client.py b/test/test_client.py index 79ac8be..42a1623 100644 --- a/test/test_client.py +++ b/test/test_client.py @@ -398,7 +398,7 @@ class TestSimpleClient(unittest.TestCase): def test_correlation_rollover(self): with patch.object(SimpleClient, 'load_metadata_for_topics'): big_num = 2**31 - 3 - client = SimpleClient(hosts=[], correlation_id=big_num) + client = SimpleClient(hosts=(), correlation_id=big_num) self.assertEqual(big_num + 1, client._next_id()) self.assertEqual(big_num + 2, client._next_id()) self.assertEqual(0, client._next_id()) -- cgit v1.2.1