Run E2E upload test in CI and improve diagnostics

Author: kvz

.github/workflows/ci.yml

@@ -89,3 +89,54 @@ jobs:
           path: |
             coverage.json
             htmlcov/
+
+  python-e2e:
+    runs-on: ubuntu-latest
+    needs: python
+    env:
+      PYTHON_SDK_E2E: "1"
+      TRANSLOADIT_KEY: ${{ secrets.TRANSLOADIT_E2E_KEY }}
+      TRANSLOADIT_SECRET: ${{ secrets.TRANSLOADIT_E2E_SECRET }}
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '20'
+      - name: Install Transloadit CLI
+        run: npm install -g transloadit
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.12'
+          architecture: x64
+          cache: 'pip'
+
+      - name: Install Poetry
+        run: pip install --upgrade poetry
+
+      - name: Export optional Transloadit overrides
+        run: |
+          if [ -n "${{ secrets.TRANSLOADIT_E2E_HOST }}" ]; then
+            echo "TRANSLOADIT_HOST=${{ secrets.TRANSLOADIT_E2E_HOST }}" >> "$GITHUB_ENV"
+          fi
+          if [ -n "${{ secrets.TRANSLOADIT_E2E_REGION }}" ]; then
+            echo "TRANSLOADIT_REGION=${{ secrets.TRANSLOADIT_E2E_REGION }}" >> "$GITHUB_ENV"
+          fi
+
+      - name: Install dependencies
+        run: poetry install
+
+      - name: Ensure credentials present
+        run: |
+          if [ -z "$TRANSLOADIT_KEY" ] || [ -z "$TRANSLOADIT_SECRET" ]; then
+            echo "TRANSLOADIT_E2E_KEY/TRANSLOADIT_E2E_SECRET secrets must be configured for the E2E job" >&2
+            exit 1
+          fi
+
+      - name: Run E2E upload test
+        env:
+          TEST_NODE_PARITY: 0
+        run: |
+          poetry run pytest tests/test_e2e_upload.py -q --maxfail=1

README.md

@@ -60,7 +60,7 @@ This script will:
 - install Poetry, Node.js 20, and the Transloadit CLI
 - pass credentials from `.env` (if present) so end-to-end tests can run against real Transloadit accounts
 
-Signature parity tests use `npx transloadit smart_sig` under the hood, matching the reference implementation used by our other SDKs.
+Signature parity tests use `npx transloadit smart_sig` under the hood, matching the reference implementation used by our other SDKs. Our GitHub Actions workflow also runs the E2E upload against Python 3.12 on every push/PR using a dedicated Transloadit test account.
 
 Pass `--python 3.12` (or set `PYTHON_VERSIONS`) to restrict the matrix, or append a custom command after `--`, for example `scripts/test-in-docker.sh -- pytest -k smartcdn`.
 

tests/test_e2e_upload.py

@@ -29,7 +29,7 @@ def _build_service():
     return None
 
 
-def test_e2e_image_resize(tmp_path):
+def test_e2e_image_resize():
     key = os.getenv("TRANSLOADIT_KEY")
     secret = os.getenv("TRANSLOADIT_SECRET")
 
@@ -63,6 +63,9 @@ def test_e2e_image_resize(tmp_path):
         response = assembly.create(wait=True, resumable=False)
 
     data = response.data
+    assembly_ssl_url = data.get("assembly_ssl_url") or data.get("assembly_url")
+    assembly_id = data.get("assembly_id")
+    print(f"[python-sdk][e2e] Assembly URL: {assembly_ssl_url} (id={assembly_id})")
     assert data.get("ok") == "ASSEMBLY_COMPLETED", data
 
     uploads = data.get("uploads") or []
@@ -91,3 +94,8 @@ def test_e2e_image_resize(tmp_path):
         height = int(height)
     assert width and height, f"Missing dimensions in result metadata: {meta}"
     assert 0 < width <= 128 and 0 < height <= 128
+    print(
+        "[python-sdk][e2e] Result dimensions: "
+        f"{width}x{height}, ssl_url={ssl_url}, basename={upload_info.get('basename')}, "
+        f"filename={upload_info.get('name')}"
+    )