Skip to content

Commit 0350b5a

Browse files
committed
refactor: Use setdefault for environment variables in test configuration
- Updated environment variable assignments in `conftest.py` to use `os.environ.setdefault`, allowing for overrides by existing environment variables. - This change enhances flexibility in test configurations while maintaining default values for local testing.
1 parent b4ccd7d commit 0350b5a

1 file changed

Lines changed: 8 additions & 7 deletions

File tree

tests/conftest.py

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,14 @@
1313

1414
# Set test environment before importing config
1515
# These match the docker-compose infrastructure settings
16-
os.environ["API_KEY"] = "test-api-key-for-testing-12345"
17-
os.environ["REDIS_HOST"] = "localhost"
18-
os.environ["REDIS_PORT"] = "6379"
19-
os.environ["MINIO_ENDPOINT"] = "localhost:9000"
20-
os.environ["MINIO_ACCESS_KEY"] = "minioadmin"
21-
os.environ["MINIO_SECRET_KEY"] = "minioadmin"
22-
os.environ["MINIO_SECURE"] = "false"
16+
# Use setdefault to allow environment variables to override defaults
17+
os.environ.setdefault("API_KEY", "test-api-key-for-testing-12345")
18+
os.environ.setdefault("REDIS_HOST", "localhost")
19+
os.environ.setdefault("REDIS_PORT", "6379")
20+
os.environ.setdefault("MINIO_ENDPOINT", "localhost:9000")
21+
os.environ.setdefault("MINIO_ACCESS_KEY", "minioadmin")
22+
os.environ.setdefault("MINIO_SECRET_KEY", "minioadmin")
23+
os.environ.setdefault("MINIO_SECURE", "false")
2324

2425
from src.config import settings
2526
from src.services.session import SessionService

0 commit comments

Comments
 (0)