Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: Review server app setup #4432

Merged
merged 8 commits into from
Dec 22, 2023
Prev Previous commit
Next Next commit
chore: fix old module refs
  • Loading branch information
frascuchon committed Dec 19, 2023
commit 5d776fca122668b1aeb1efb3400fc25a50347d62
4 changes: 2 additions & 2 deletions tests/integration/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@
from argilla.datasets.__init__ import configure_dataset
from argilla.server.database import get_async_db
from argilla.server.models import User, UserRole, Workspace
from argilla.server.server import app
from argilla.server.app import app
from argilla.server.settings import settings
from argilla.utils import telemetry
from argilla.utils.telemetry import TelemetryClient
Expand Down Expand Up @@ -132,7 +132,7 @@ async def override_get_async_db():
session = TestSession()
yield session

mocker.patch("argilla.server.server._get_db_wrapper", wraps=contextlib.asynccontextmanager(override_get_async_db))
mocker.patch("argilla.server.app._get_db_wrapper", wraps=contextlib.asynccontextmanager(override_get_async_db))

app.dependency_overrides[get_async_db] = override_get_async_db

Expand Down
2 changes: 1 addition & 1 deletion tests/unit/server/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ async def override_get_async_db():
async def override_get_search_engine():
yield mock_search_engine

mocker.patch("argilla.server.server._get_db_wrapper", wraps=contextlib.asynccontextmanager(override_get_async_db))
mocker.patch("argilla.server.app._get_db_wrapper", wraps=contextlib.asynccontextmanager(override_get_async_db))

app.dependency_overrides[get_async_db] = override_get_async_db
app.dependency_overrides[get_search_engine] = override_get_search_engine
Expand Down
27 changes: 0 additions & 27 deletions tests/unit/server/test_server_app.py

This file was deleted.