Skip to content

Commit 8da9af1

Browse files
committed
refactor fixtures in licensing
1 parent 3be5bbd commit 8da9af1

File tree

2 files changed

+130
-85
lines changed

2 files changed

+130
-85
lines changed

services/api-server/tests/unit/api_functions/conftest.py

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,9 +92,7 @@ def _new(app: FastAPI):
9292

9393

9494
@pytest.fixture
95-
async def mock_get_wb_api_rpc_client(
96-
app: FastAPI, mocker: MockerFixture
97-
) -> MockerFixture:
95+
async def mock_get_wb_api_rpc_client(app: FastAPI, mocker: MockerFixture) -> None:
9896
def _new():
9997
from simcore_service_api_server.services_rpc import wb_api_server
10098

@@ -104,7 +102,6 @@ def _new():
104102
) # noqa: SLF001
105103

106104
app.dependency_overrides[get_wb_api_rpc_client] = _new
107-
return mocker
108105

109106

110107
@pytest.fixture
@@ -274,7 +271,7 @@ def fake_registered_function_job_collection(
274271

275272
@pytest.fixture()
276273
def mock_handler_in_functions_rpc_interface(
277-
mock_get_wb_api_rpc_client: MockerFixture,
274+
mock_get_wb_api_rpc_client: None,
278275
mocker: MockerFixture,
279276
) -> Callable[[str, Any, Exception | None, Callable | None], MockType]:
280277
def _create(
@@ -301,7 +298,7 @@ def _create(
301298

302299
@pytest.fixture()
303300
def mock_method_in_jobs_service(
304-
mock_get_wb_api_rpc_client: MockerFixture,
301+
mock_get_wb_api_rpc_client: None,
305302
mocker: MockerFixture,
306303
) -> Callable[[str, Any, Exception | None], MockType]:
307304
def _create(

0 commit comments

Comments
 (0)