Skip to content

Commit cbade4e

Browse files
amosttAygentic
andcommitted
fix(test-backend): set REDIS_PASSWORD for Settings validation
Fixed Settings validation error by setting REDIS_PASSWORD to dummy value. Root cause: Settings model requires REDIS_PASSWORD (required field, line 79 in config.py). Setting it to empty string "" caused Pydantic validation error: 'Field required'. Solution: Set REDIS_PASSWORD to 'test-redis-password' (matches test-e2e.yml). Note that Redis service container has no authentication - password is only needed to satisfy Settings validation, not actually used for connection. This aligns with test-e2e.yml pattern which also sets REDIS_PASSWORD to dummy value despite Redis service container having no auth. Related to CUR-46 🤖 Generated by Aygentic Co-Authored-By: Aygentic <[email protected]>
1 parent 3302263 commit cbade4e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

.github/workflows/test-backend.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ jobs:
9191
SUPABASE_ANON_KEY: test-anon-key
9292
SUPABASE_SERVICE_KEY: test-service-key
9393
REDIS_URL: redis://localhost:6379/0
94-
REDIS_PASSWORD: ""
94+
REDIS_PASSWORD: test-redis-password
9595
CELERY_BROKER_URL: redis://localhost:6379/0
9696
CELERY_RESULT_BACKEND: redis://localhost:6379/0
9797
SECRET_KEY: test-secret-key-for-ci

0 commit comments

Comments
 (0)