Merge pull request #30 from trustpilot/SP-148-gha

chore: migrate to GHA

Author: tpfrne

.github/workflows/pushes.yaml

@@ -0,0 +1,54 @@
+name: Build and release
+
+'on':
+  push:
+    branches:
+      - master
+  pull_request:
+    types:
+      - opened
+      - synchronize
+
+permissions:
+  contents: read
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ['3.8', '3.9', '3.10', '3.11', '3.12', '3.13']
+    steps:
+      - name: Checkout the repo
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.head_ref }}
+
+      - name: Setup Python ${{ matrix.python-version }}
+        uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+          cache: 'pip'
+
+      - name: Install poetry
+        run: |
+          pip install --upgrade pip
+          pip install poetry
+          python -V
+          pip -V
+          poetry -V
+          poetry config virtualenvs.create false
+        shell: bash
+
+      - name: poetry install
+        run: poetry install --verbose
+        shell: bash
+
+      - name: Lint
+        run: poetry run black --check .
+        shell: bash
+
+      - name: Test
+        run: poetry run pytest -vvv tests
+        shell: bash

.travis.yml

@@ -15,9 +15,12 @@ classifiers = [
     "Development Status :: 5 - Production/Stable",
     "Intended Audience :: Developers",
     "Natural Language :: English",
-    "Programming Language :: Python :: 3.7",
     "Programming Language :: Python :: 3.8",
-    "Programming Language :: Python :: 3.9"
+    "Programming Language :: Python :: 3.9",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13"
 ]
 
 [tool.poetry.dependencies]

poetry.lock

@@ -49,7 +49,7 @@ def setUp(self):
             }
         }
         self.response_mock = response_mock
-        self.expected_output = u"""{
+        self.expected_output = """{
           "url": "https://api.trustpilot.com/v1/business-units/5400267300006400057a0113/reviews",
           "status": 401,
           "content": {
@@ -62,7 +62,7 @@ def setUp(self):
           }
         }
         """
-        self.expected_verbose_output = u"""{
+        self.expected_verbose_output = """{
           "url": "https://api.trustpilot.com/v1/business-units/5400267300006400057a0113/reviews",
           "status": 401,
           "headers": {
@@ -106,7 +106,6 @@ def test_create_access_token(self, client_mock):
     @mock.patch("trustpilot.cli.client", autospec=True)
     @mock.patch("trustpilot.cli.auth")
     def test_no_verbosity_with_get(self, auth_mock, client_mock):
-
         client_mock.get.return_value = self.response_mock
 
         result = self.runner.invoke(
@@ -278,7 +277,6 @@ def assert_output_equal(self, output, expected_output):
     @mock.patch("trustpilot.cli.client", autospec=True)
     @mock.patch("trustpilot.cli.auth")
     def test_no_verbosity_with_get(self, auth_mock, client_mock):
-
         client_mock.get.return_value = self.response_mock
 
         result = self.runner.invoke(

pyproject.toml

@@ -30,7 +30,6 @@ def setup(
         user_agent=None,
         **kwargs
     ):
-
         self.api_host = api_host or environ.get(
             "TRUSTPILOT_API_HOST", "https://api.trustpilot.com"
         )
@@ -113,7 +112,6 @@ async def post(self, url, *args, **kwargs):
         return await self.authenticated_request("post", url, *args, **kwargs)
 
     async def get(self, url, *args, **kwargs):
-
         return await self.authenticated_request("get", url, *args, **kwargs)
 
     async def put(self, url, *args, **kwargs):

tests/test_cli.py

@@ -50,7 +50,6 @@ def setup(
         user_agent=None,
         **kwargs
     ):
-
         self.api_host = api_host or environ.get(
             "TRUSTPILOT_API_HOST", "https://api.trustpilot.com"
         )
@@ -112,7 +111,10 @@ def _post_request_callback(self, response, *args, **kwargs):
         req = response.request
         retry = getattr(req, "authentication_retry", True)
 
-        if retry and response.status_code in (requests.codes.unauthorized, requests.codes.forbidden):
+        if retry and response.status_code in (
+            requests.codes.unauthorized,
+            requests.codes.forbidden,
+        ):
             logger.debug(
                 {"message": "reauthenticating and retrying once", "url": req.url}
             )