Skip to content

Commit 231753f

Browse files
authored
fix: converts redis dsn to string (#9)
1 parent 03af0ff commit 231753f

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

aind-tars-service-server/src/aind_tars_service_server/main.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ async def lifespan(_: FastAPI) -> AsyncIterator[None]:
3636
"""Init cache and add to lifespan of app"""
3737
settings = get_settings()
3838
if settings.redis_url is not None:
39-
redis = from_url(settings.redis_url)
39+
redis = from_url(settings.redis_url.unicode_string())
4040
FastAPICache.init(RedisBackend(redis), prefix="fastapi-cache")
4141
else:
4242
FastAPICache.init(InMemoryBackend(), prefix="fastapi-cache")

aind-tars-service-server/tests/conftest.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
import pytest
1010
from fastapi.testclient import TestClient
11+
from pydantic import RedisDsn
1112

1213
from aind_tars_service_server.configs import Settings
1314

@@ -85,7 +86,7 @@ def client_with_redis() -> Generator[TestClient, Any, None]:
8586

8687
settings = Settings()
8788
settings_with_redis = settings.model_copy(
88-
update={"redis_url": "redis"}, deep=True
89+
update={"redis_url": RedisDsn("redis://example.com:1234")}, deep=True
8990
)
9091
with (
9192
patch(

0 commit comments

Comments
 (0)