refactor: rename mock function for clarity and consistency across tests

Author: pcrespov

services/web/server/tests/unit/with_dbs/02/test_projects_crud_handlers__patch.py

@@ -32,9 +32,9 @@ def mock_catalog_api_get_services_for_user_in_product(mocker: MockerFixture):
 
 
 @pytest.fixture
-def mock_project_uses_available_services(mocker: MockerFixture):
+def mock_are_project_services_available(mocker: MockerFixture):
     mocker.patch(
-        "simcore_service_webserver.projects._controller.projects_rest.project_uses_available_services",
+        "simcore_service_webserver.projects._controller.projects_rest.are_project_services_available",
         spec=True,
         return_value=True,
     )
@@ -82,7 +82,7 @@ async def test_patch_project(
     user_project: ProjectDict,
     expected: HTTPStatus,
     mock_catalog_api_get_services_for_user_in_product,
-    mock_project_uses_available_services,
+    mock_are_project_services_available,
 ):
     assert client.app
     base_url = client.app.router["patch_project"].url_for(

services/web/server/tests/unit/with_dbs/02/test_projects_groups_handlers.py

@@ -33,9 +33,9 @@ def mock_catalog_api_get_services_for_user_in_product(
 
 
 @pytest.fixture
-def mock_project_uses_available_services(mocker: MockerFixture) -> MockType:
+def mock_are_project_services_available(mocker: MockerFixture) -> MockType:
     return mocker.patch(
-        "simcore_service_webserver.projects._controller.projects_rest.project_uses_available_services",
+        "simcore_service_webserver.projects._controller.projects_rest.are_project_services_available",
         spec=True,
         return_value=True,
     )
@@ -51,7 +51,7 @@ async def test_projects_groups_full_workflow(  # noqa: PLR0915
     user_project: ProjectDict,
     expected: HTTPStatus,
     mock_catalog_api_get_services_for_user_in_product: MockType,
-    mock_project_uses_available_services: MockType,
+    mock_are_project_services_available: MockType,
 ):
     assert client.app
     # check the default project permissions
@@ -265,7 +265,7 @@ async def test_share_project(
     logged_user: UserInfoDict,
     user_project: ProjectDict,
     mock_catalog_api_get_services_for_user_in_product: MockType,
-    mock_project_uses_available_services: MockType,
+    mock_are_project_services_available: MockType,
 ):
     assert client.app
 
@@ -333,7 +333,7 @@ async def test_share_project_with_roles(
     logged_user: UserInfoDict,
     user_project: ProjectDict,
     mock_catalog_api_get_services_for_user_in_product: MockType,
-    mock_project_uses_available_services: MockType,
+    mock_are_project_services_available: MockType,
     user_role: UserRole,
     expected_status: HTTPStatus,
 ):

services/web/server/tests/unit/with_dbs/02/test_projects_nodes_handlers__patch.py

@@ -36,9 +36,9 @@ def mock_catalog_api_get_services_for_user_in_product(mocker: MockerFixture):
 
 
 @pytest.fixture
-def mock_project_uses_available_services(mocker: MockerFixture):
+def mock_are_project_services_available(mocker: MockerFixture):
     mocker.patch(
-        "simcore_service_webserver.projects._controller.projects_rest.project_uses_available_services",
+        "simcore_service_webserver.projects._controller.projects_rest.are_project_services_available",
         spec=True,
         return_value=True,
     )
@@ -100,7 +100,7 @@ async def test_patch_project_node(
     user_project: ProjectDict,
     expected: HTTPStatus,
     mock_catalog_api_get_services_for_user_in_product: None,
-    mock_project_uses_available_services: None,
+    mock_are_project_services_available: None,
     mock_catalog_rpc_check_for_service: None,
 ):
     node_id = next(iter(user_project["workbench"]))
@@ -222,7 +222,7 @@ async def test_patch_project_node_notifies(
     user_project: ProjectDict,
     expected: HTTPStatus,
     mock_catalog_api_get_services_for_user_in_product,
-    mock_project_uses_available_services,
+    mock_are_project_services_available,
     mock_catalog_rpc_check_for_service,
     mocked_notify_project_node_update,
 ):
@@ -258,7 +258,7 @@ async def test_patch_project_node_inputs_notifies(
     user_project: ProjectDict,
     expected: HTTPStatus,
     mock_catalog_api_get_services_for_user_in_product,
-    mock_project_uses_available_services,
+    mock_are_project_services_available,
     mocked_notify_project_node_update,
 ):
     node_id = next(iter(user_project["workbench"]))
@@ -298,7 +298,7 @@ async def test_patch_project_node_inputs_with_data_type_change(
     user_project: ProjectDict,
     expected: HTTPStatus,
     mock_catalog_api_get_services_for_user_in_product,
-    mock_project_uses_available_services,
+    mock_are_project_services_available,
 ):
     node_id = next(iter(user_project["workbench"]))
     assert client.app
@@ -351,7 +351,7 @@ async def test_patch_project_node_service_key_with_error(
     user_project: ProjectDict,
     expected: HTTPStatus,
     mock_catalog_api_get_services_for_user_in_product,
-    mock_project_uses_available_services,
+    mock_are_project_services_available,
     mocker: MockerFixture,
 ):
     node_id = next(iter(user_project["workbench"]))

services/web/server/tests/unit/with_dbs/04/workspaces/conftest.py

@@ -26,7 +26,7 @@ def mock_catalog_api_get_services_for_user_in_product(mocker: MockerFixture):
         return_value=[],
     )
     mocker.patch(
-        "simcore_service_webserver.projects._controller.projects_rest.project_uses_available_services",
+        "simcore_service_webserver.projects._controller.projects_rest.are_project_services_available",
         spec=True,
         return_value=True,
     )