Skip to content

Commit 2168fd2

Browse files
committed
the app now needs the worker in order to run
1 parent fa67a15 commit 2168fd2

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

services/storage/tests/conftest.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,11 @@ def app_settings(
221221

222222

223223
@pytest.fixture
224-
async def initialized_app(app_settings: ApplicationSettings) -> AsyncIterator[FastAPI]:
224+
async def initialized_app(
225+
mock_celery_app: None,
226+
app_settings: ApplicationSettings,
227+
celery_worker: CeleryTaskQueueWorker,
228+
) -> AsyncIterator[FastAPI]:
225229
app = create_app(app_settings)
226230
# NOTE: the timeout is sometime too small for CI machines, and even larger machines
227231
async with LifespanManager(
@@ -984,7 +988,6 @@ def mock_celery_app(mocker: MockFixture, celery_config: dict[str, Any]) -> None:
984988
@pytest.fixture
985989
async def celery_worker_controller(
986990
app_environment: EnvVarsDict,
987-
initialized_app: FastAPI,
988991
celery_app: Celery,
989992
monkeypatch: pytest.MonkeyPatch,
990993
) -> AsyncIterator[TestWorkController]:

services/storage/tests/unit/test_rpc_handlers_paths.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,10 +110,8 @@ async def _assert_compute_path_size(
110110
ids=str,
111111
)
112112
async def test_path_compute_size(
113-
mock_celery_app: None,
114113
initialized_app: FastAPI,
115114
storage_rabbitmq_rpc_client: RabbitMQRPCClient,
116-
celery_worker: CeleryTaskQueueWorker,
117115
user_id: UserID,
118116
location_id: LocationID,
119117
with_random_project_with_files: tuple[

0 commit comments

Comments
 (0)