fix tox flake8 and unit tests & docker python path

Author: blankdots

Dockerfile

@@ -21,7 +21,7 @@ LABEL maintainer "CSC Developers"
 LABEL org.label-schema.schema-version="1.0"
 LABEL org.label-schema.vcs-url="https://github.com/CSCFI/swift-x-account-sharing"
 
-COPY --from=BACKEND /usr/local/lib/python3.7 /usr/local/lib/python3.7/
+COPY --from=BACKEND /usr/local/lib/python3.8 /usr/local/lib/python3.8/
 
 COPY --from=BACKEND /usr/local/bin/gunicorn /usr/local/bin/
 

swift_x_account_sharing/bindings/bind.py

@@ -16,7 +16,6 @@
 ssl_context.load_verify_locations(certifi.where())
 
 
-
 class SwiftXAccountSharing:
     """Swift X Account Sharing backend client."""
 
@@ -56,20 +55,23 @@ async def get_access(
 
         params = sign_api_request(path)
 
-        async with self.session.get(url, params=params, ssl=ssl_context) as resp:
+        async with self.session.get(url,
+                                    params=params,
+                                    ssl=ssl_context) as resp:
             if resp.status == 200:
                 try:
                     return json.loads(await resp.text())
                 except json.decoder.JSONDecodeError:
-                    logging.error("Decoding JSON error response was not possible.")
+                    logging.error("Decoding JSON error \
+                        response was not possible.")
                     raise
                 except Exception as e:
-                    logging.error(f"Unknown exception occured with content: {e}.")
+                    logging.error(f"Unknown exception \
+                        occured with content: {e}.")
                     raise
             else:
                 logging.error(f"response status: {resp.status}.")
                 raise Exception(f"response status: {resp.status}.")
-                
 
     async def get_access_details(
             self,
@@ -84,15 +86,19 @@ async def get_access_details(
         params = sign_api_request(path)
         params.update({"owner": owner})
 
-        async with self.session.get(url, params=params, ssl=ssl_context) as resp:
+        async with self.session.get(url,
+                                    params=params,
+                                    ssl=ssl_context) as resp:
             if resp.status == 200:
                 try:
                     return json.loads(await resp.text())
                 except json.decoder.JSONDecodeError:
-                    logging.error("Decoding JSON error response was not possible.")
+                    logging.error("Decoding JSON error \
+                        response was not possible.")
                     raise
                 except Exception as e:
-                    logging.error(f"Unknown exception occured with content: {e}.")
+                    logging.error(f"Unknown exception \
+                        occured with content: {e}.")
                     raise
             else:
                 logging.error(f"response status: {resp.status}.")
@@ -108,15 +114,19 @@ async def get_share(
 
         params = sign_api_request(path)
 
-        async with self.session.get(url, params=params, ssl=ssl_context) as resp:
+        async with self.session.get(url,
+                                    params=params,
+                                    ssl=ssl_context) as resp:
             if resp.status == 200:
                 try:
                     return json.loads(await resp.text())
                 except json.decoder.JSONDecodeError:
-                    logging.error("Decoding JSON error response was not possible.")
+                    logging.error("Decoding JSON error \
+                        response was not possible.")
                     raise
                 except Exception as e:
-                    logging.error(f"Unknown exception occured with content: {e}.")
+                    logging.error(f"Unknown exception \
+                        occured with content: {e}.")
                     raise
             else:
                 logging.error(f"response status: {resp.status}.")
@@ -133,15 +143,19 @@ async def get_share_details(
 
         params = sign_api_request(path)
 
-        async with self.session.get(url, params=params, ssl=ssl_context) as resp:
+        async with self.session.get(url,
+                                    params=params,
+                                    ssl=ssl_context) as resp:
             if resp.status == 200:
                 try:
                     return json.loads(await resp.text())
                 except json.decoder.JSONDecodeError:
-                    logging.error("Decoding JSON error response was not possible.")
+                    logging.error("Decoding JSON error \
+                        response was not possible.")
                     raise
                 except Exception as e:
-                    logging.error(f"Unknown exception occured with content: {e}.")
+                    logging.error(f"Unknown exception \
+                        occured with content: {e}.")
                     raise
             else:
                 logging.error(f"response status: {resp.status}.")
@@ -167,15 +181,19 @@ async def share_new_access(
             "address": address
         })
 
-        async with self.session.post(url, params=params, ssl=ssl_context) as resp:
+        async with self.session.post(url,
+                                     params=params,
+                                     ssl=ssl_context) as resp:
             if resp.status == 200:
                 try:
                     return json.loads(await resp.text())
                 except json.decoder.JSONDecodeError:
-                    logging.error("Decoding JSON error response was not possible.")
+                    logging.error("Decoding JSON error \
+                        response was not possible.")
                     raise
                 except Exception as e:
-                    logging.error(f"Unknown exception occured with content: {e}.")
+                    logging.error(f"Unknown exception \
+                        occured with content: {e}.")
                     raise
             else:
                 logging.error(f"response status: {resp.status}.")
@@ -198,15 +216,19 @@ async def share_edit_access(
             "access": self.parse_list_to_string(accesslist),
         })
 
-        async with self.session.patch(url, params=params, ssl=ssl_context) as resp:
+        async with self.session.patch(url,
+                                      params=params,
+                                      ssl=ssl_context) as resp:
             if resp.status == 200:
                 try:
                     return json.loads(await resp.text())
                 except json.decoder.JSONDecodeError:
-                    logging.error("Decoding JSON error response was not possible.")
+                    logging.error("Decoding JSON error \
+                        response was not possible.")
                     raise
                 except Exception as e:
-                    logging.error(f"Unknown exception occured with content: {e}.")
+                    logging.error(f"Unknown exception \
+                        occured with content: {e}.")
                     raise
             else:
                 logging.error(f"response status: {resp.status}.")
@@ -227,5 +249,7 @@ async def share_delete_access(
             "user": self.parse_list_to_string(userlist),
         })
 
-        async with self.session.delete(url, params=params, ssl=ssl_context) as resp:
+        async with self.session.delete(url,
+                                       params=params,
+                                       ssl=ssl_context) as resp:
             return bool(resp.status == 204)

tests/test_bindings_bind.py

@@ -20,7 +20,7 @@ def __init__(self, *args, **kwargs):
             "text": asynctest.CoroutineMock(
                 return_value="[]"
             ),
-            "status": 204
+            "status": 200
         })
 
     async def __aenter__(self, *args, **kwargs):