fix: update test workflow to set environment variables for test URLs …

…and refactor conftest.py to use them
taco-xyz · Jan 7, 2025 · f85cbb0 · f85cbb0
1 parent 45545c2
commit f85cbb0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -122,9 +122,11 @@ jobs:
           docker compose -f docker-compose.test.yml up -d --wait
 
       - name: Run end-to-end tests
+        env:
+          MANAGER_TEST_URL: http://localhost:3000
+          BROKER_TEST_URL: amqp://user:password@localhost:5672
         run: |
           cd client_sdks/python
-          source .venv/bin/activate
           uv run pytest
 
       - name: Cleanup test environment

diff --git a/client_sdks/python/tests/conftest.py b/client_sdks/python/tests/conftest.py
@@ -3,8 +3,12 @@
 from broker import BrokerConfig
 import pytest
 
-MANAGER_TEST_URL = "http://localhost:3000"
-BROKER_TEST_URL = "amqp://user:password@localhost:5672/"
+import os
+
+MANAGER_TEST_URL = os.environ.get("MANAGER_TEST_URL", "http://localhost:3000")
+BROKER_TEST_URL = os.environ.get(
+    "BROKER_TEST_URL", "amqp://user:password@localhost:5672/"
+)
 
 WORKER_NAME = "test_worker"