Upgrade + add logging (#17)

* Upgrade + add logging

* bump pre-commit

Author: inverse

.github/workflows/main.yml

@@ -21,11 +21,11 @@ jobs:
           ${{ runner.os }}-poetry-
     - uses: actions/setup-python@v4
       with:
-        python-version: '3.10'
+        python-version: '3.11'
     - name: Install poetry
       uses: abatilo/actions-poetry@v2.3.0
       with:
-        poetry-version: '1.3.1'
+        poetry-version: '1.5.0'
     - name: Install pip dependencies
       run: poetry install
     - uses: pre-commit/action@v3.0.0
@@ -41,11 +41,11 @@ jobs:
           ${{ runner.os }}-poetry-
     - uses: actions/setup-python@v4
       with:
-        python-version: '3.10'
+        python-version: '3.11'
     - name: Install poetry
       uses: abatilo/actions-poetry@v2.3.0
       with:
-        poetry-version: '1.3.1'
+        poetry-version: '1.5.0'
     - name: Install pip dependencies
       run: poetry install
     - name: Run tests

.github/workflows/publish.yml

@@ -11,11 +11,11 @@ jobs:
     - uses: actions/checkout@master
     - uses: actions/setup-python@v4
       with:
-        python-version: '3.10'
+        python-version: '3.11'
     - name: Install poetry
       uses: abatilo/actions-poetry@v2.3.0
       with:
-        poetry-version: '1.3.1'
+        poetry-version: '1.5.0'
     - name: Install pip dependencies
       run: poetry install
     - name: Package

.pre-commit-config.yaml

@@ -8,7 +8,7 @@ repos:
     - id: check-toml
     - id: check-added-large-files
 -   repo: https://github.com/psf/black
-    rev: 22.10.0
+    rev: 23.3.0
     hooks:
     - id: black
 -   repo: https://github.com/pre-commit/mirrors-isort
@@ -17,7 +17,7 @@ repos:
     - id: isort
       additional_dependencies: [toml]
 -   repo: https://github.com/pre-commit/mirrors-mypy
-    rev: v0.991
+    rev: v1.3.0
     hooks:
     -   id: mypy
         additional_dependencies: [types-requests]

edge_addons_api/client.py

@@ -25,7 +25,6 @@ class Options:
 
 
 class Client:
-
     BASE_URL = "https://api.addons.microsoftedge.microsoft.com"
 
     def __init__(self, options: Options):
@@ -41,6 +40,7 @@ def submit(self, file_path: str, notes: str) -> str:
         return self._publish(notes, access_token)
 
     def fetch_publish_status(self, operation_id: str) -> dict:
+        logger.debug(f"Fetching publish status for {operation_id}")
         access_token = self._get_access_token()
         response = requests.get(
             self._publish_status_endpoint(operation_id),
@@ -50,9 +50,12 @@ def fetch_publish_status(self, operation_id: str) -> dict:
         )
 
         response.raise_for_status()
+
+        logger.debug(f"Publish status response: {response.content.decode()}")
         return response.json()
 
     def _publish(self, notes: str, access_token: str) -> str:
+        logger.debug("Publishing")
         response = requests.post(
             self._publish_endpoint(),
             data={"notes": notes},
@@ -68,6 +71,7 @@ def _publish(self, notes: str, access_token: str) -> str:
         return response.headers["Location"]
 
     def _upload(self, file_path: str, access_token: str) -> str:
+        logger.debug(f"Uploading {file_path}")
         with open(file_path, "rb") as f:
             response = requests.post(
                 self._upload_endpoint(),
@@ -80,6 +84,8 @@ def _upload(self, file_path: str, access_token: str) -> str:
 
         response.raise_for_status()
 
+        logger.debug("Finished upload")
+
         return response.headers["Location"]
 
     def _check_upload(
@@ -89,6 +95,8 @@ def _check_upload(
         retry_count: int = 5,
         sleep_seconds: int = 3,
     ) -> str:
+        logger.debug("Checking upload")
+
         upload_status = ""
         attempts = 0