Fix Unraisable exception in PyTest (#43)

* Fixing test-tox and test-example

* Fix ResourceWarning: unclosed ssl.SSLSocket

* Fix ResourceWarning: unclosed ssl.SSLSocket for ApiClient

* Fix tests on Windows

Author: Denis-Averin

.github/workflows/python-package.yml

@@ -30,4 +30,4 @@ jobs:
     - name: Test
       env:
         TEST_CONFIGURATION_ACCESS_TOKEN: ${{ secrets.TEST_CONFIGURATION_ACCESS_TOKEN }}
-      run: make test
+      run: make test test-example

Makefile

@@ -1,7 +1,7 @@
 SRC=./aspose_barcode_cloud
 
 .PHONY: all
-all: format lint test-all
+all: format lint test-tox
 
 .PHONY: check_git
 check_git:
@@ -47,19 +47,23 @@ lint:
 	flake8 . --count --exit-zero --max-line-length=127 --statistics --extend-ignore=E501 --extend-exclude '.*'
 
 .PHONY: publish
-publish: check_git test-all clean dist
+publish: check_git test-tox clean dist
 	python3 -m twine upload dist/*
 
 .PHONY: publish-docker
-publish-docker: init-docker test-all dist
+publish-docker: init-docker test-tox dist
 	python3 -m twine upload dist/* --verbose
 
 .PHONY: test
 test:
 	python -Werror -m pytest --cov tests/
 
-.PHONY: test-all
-test-all:
+.PHONY: test-example
+test-example:
+	python -Werror example.py
+
+.PHONY: test-tox
+test-tox:
 	tox $(SRC)
 
 .PHONY: update

aspose_barcode_cloud/api_client.py

@@ -93,6 +93,7 @@ def __init__(self, configuration=None, header_name=None, header_value=None, cook
         self.user_agent = "Aspose-Barcode-SDK/22.8.0/python"
 
     def __del__(self):
+        self.rest_client.close()
         if self._pool is not None:
             self._pool.close()
             self._pool.join()

aspose_barcode_cloud/configuration.py

@@ -26,6 +26,7 @@
 
 from __future__ import absolute_import, division
 
+import contextlib
 import copy
 import json
 import logging
@@ -290,12 +291,20 @@ def to_debug_report(self):
 
     @staticmethod
     def fetch_token(client_id, client_secret, token_url):
-        client = RESTClientObject(Configuration(client_id=client_id, client_secret=client_secret, token_url=token_url))
-        response = client.POST(
-            token_url,
-            headers={"Content-Type": "application/x-www-form-urlencoded"},
-            post_params={"grant_type": "client_credentials", "client_id": client_id, "client_secret": client_secret},
-        )
-        js_data = json.loads(response.data)
+        with contextlib.closing(
+            RESTClientObject(Configuration(client_id=client_id, client_secret=client_secret, token_url=token_url))
+        ) as client:
+            response = client.POST(
+                token_url,
+                headers={"Content-Type": "application/x-www-form-urlencoded"},
+                post_params={
+                    "grant_type": "client_credentials",
+                    "client_id": client_id,
+                    "client_secret": client_secret,
+                },
+            )
+            js_data = json.loads(response.data)
+
         access_token = js_data["access_token"]
+
         return access_token

aspose_barcode_cloud/rest.py

@@ -370,6 +370,9 @@ def PATCH(
             body=body,
         )
 
+    def close(self):
+        self.pool_manager.clear()
+
 
 class ApiException(Exception):
     def __init__(self, status=None, reason=None, http_resp=None):

tests/test_generate_and_recognize.py

@@ -18,7 +18,8 @@ def test_generate_and_recognize(self):
         generated = self.api.get_barcode_generate(EncodeBarcodeType.QR, "Should recognize this")
         self.assertGreater(len(generated.data), 0)
 
-        _, temp_filename = tempfile.mkstemp()
+        fd, temp_filename = tempfile.mkstemp()
+        os.close(fd)
         try:
             with open(temp_filename, "wb") as f:
                 f.write(generated.data)

tox.ini

@@ -10,4 +10,3 @@ deps=-r{toxinidir}/requirements.txt
 
 commands=
     make test
-    python -Werror example.py