Tests: MeiliSearchCommunicationError and MeiliSearchApiError

Author: eskombro

meilisearch/_httprequests.py

@@ -13,7 +13,6 @@ def __init__(self, config):
             'Content-Type': 'application/json'
         }
 
-
     def get(self, path):
         try:
             request = requests.get(
@@ -22,7 +21,7 @@ def get(self, path):
             )
             return self.__validate(request)
         except requests.exceptions.ConnectionError as err:
-            raise MeiliSearchCommunicationError(err) from None
+            raise MeiliSearchCommunicationError(err)
 
     def post(self, path, body=None):
         if body is None:
@@ -35,7 +34,7 @@ def post(self, path, body=None):
             )
             return self.__validate(request)
         except requests.exceptions.ConnectionError as err:
-            raise MeiliSearchCommunicationError(err) from None
+            raise MeiliSearchCommunicationError(err)
     def put(self, path, body=None):
         if body is None:
             body = {}
@@ -47,7 +46,7 @@ def put(self, path, body=None):
             )
             return self.__validate(request)
         except requests.exceptions.ConnectionError as err:
-            raise MeiliSearchCommunicationError(err) from None
+            raise MeiliSearchCommunicationError(err)
 
     def delete(self, path, body=None):
         if body is None:
@@ -60,7 +59,7 @@ def delete(self, path, body=None):
             )
             return self.__validate(request)
         except requests.exceptions.ConnectionError as err:
-            raise MeiliSearchCommunicationError(err) from None
+            raise MeiliSearchCommunicationError(err)
 
     @staticmethod
     def __to_json(request):
@@ -74,4 +73,4 @@ def __validate(request):
             request.raise_for_status()
             return HttpRequests.__to_json(request)
         except requests.exceptions.HTTPError as err:
-            raise MeiliSearchApiError(err, request) from None
+            raise MeiliSearchApiError(err, request)

meilisearch/errors.py

@@ -25,7 +25,7 @@ def __str__(self):
         return f'MeiliSearchApiError, HTTP status: {self.status_code} -> {self.message}'
 
 class MeiliSearchCommunicationError(MeiliSearchError):
-    """Error stablishing a connection to MeiliSearch"""
+    """Error connecting to MeiliSearch"""
 
     def __str__(self):
         return f'MeiliSearchCommunicationError, {self.message}'

meilisearch/tests/errors/__init__.py

@@ -0,0 +1,18 @@
+import pytest
+import meilisearch
+from meilisearch.errors import MeiliSearchApiError
+from meilisearch.tests import BASE_URL, MASTER_KEY
+
+class TestMeiliSearchApiError:
+
+    """ TESTS: MeiliSearchApiError class """
+
+    def test_meilisearch_api_error_no_master_key(self):
+        client = meilisearch.Client(BASE_URL)
+        with pytest.raises(MeiliSearchApiError):
+            client.create_index("Some index")
+
+    def test_meilisearch_api_error_WRONG_master_key(self):
+        client = meilisearch.Client(BASE_URL, MASTER_KEY + '123')
+        with pytest.raises(MeiliSearchApiError):
+            client.create_index("Some index")

meilisearch/tests/errors/test_meilisearch_api_error_meilisearch .py

@@ -0,0 +1,13 @@
+import pytest
+import meilisearch
+from meilisearch.errors import MeiliSearchCommunicationError
+from meilisearch.tests import MASTER_KEY
+
+class TestMeiliSearchCommunicationError:
+
+    """ TESTS: MeiliSearchCommunicationError class """
+
+    def test_meilisearch_communication_error_host(self):
+        client = meilisearch.Client("http://localhost:1234", MASTER_KEY)
+        with pytest.raises(MeiliSearchCommunicationError):
+            client.create_index("Some index")