Changing default timeout to None

Author: sanders41

meilisearch/client.py

@@ -14,7 +14,7 @@ class Client():
     config = None
     http = None
 
-    def __init__(self, url, apiKey=None, timeout=10):
+    def __init__(self, url, apiKey=None, timeout=None):
         """
         Parameters
         ----------

meilisearch/config.py

@@ -23,7 +23,7 @@ class Paths():
         attributes_for_faceting = 'attributes-for-faceting'
         dumps = 'dumps'
 
-    def __init__(self, url, api_key=None, timeout=10):
+    def __init__(self, url, api_key=None, timeout=None):
         """
         Parameters
         ----------

meilisearch/tests/client/test_client.py

@@ -21,7 +21,7 @@ def test_client_timeout_set():
 
 
 def test_client_timeout_not_set():
-    default_timeout = 10
+    default_timeout = None
     client = meilisearch.Client(BASE_URL, MASTER_KEY)
     response = client.health()
     assert client.config.timeout == default_timeout

meilisearch/tests/errors/test_communication_error_meilisearch.py

@@ -11,6 +11,6 @@
 @patch("requests.post")
 def test_meilisearch_communication_error_host(mock_post):
     mock_post.side_effect = requests.exceptions.ConnectionError()
-    client = meilisearch.Client("http://wrongurl:1234", MASTER_KEY, timeout=None)
+    client = meilisearch.Client("http://wrongurl:1234", MASTER_KEY)
     with pytest.raises(MeiliSearchCommunicationError):
         client.create_index("some_index")

meilisearch/tests/errors/test_timeout_error_meilisearch.py

@@ -21,13 +21,3 @@ def test_client_timeout_set():
         client.health()
 
     assert client.config.timeout == timeout
-
-
-def test_client_timeout_not_set():
-    timeout = 10
-    client = meilisearch.Client("http://wrongurl:1234", MASTER_KEY)
-
-    with pytest.raises(Exception):
-        client.health()
-
-    assert client.config.timeout == timeout