tests cleanup

Author: pcrespov

services/web/server/tests/unit/with_dbs/04/notifications/test_notifications__db_comp_tasks_listening_task.py

@@ -232,8 +232,11 @@ async def mock_dynamic_service_rpc(
     """
     Mocks the dynamic service RPC calls to avoid actual service calls during tests.
     """
-    return mocker.patch(
-        "servicelib.rabbitmq.rpc_interfaces.dynamic_scheduler.services.retrieve_inputs",
+    import servicelib.rabbitmq.rpc_interfaces.dynamic_scheduler.services
+
+    return mocker.patch.object(
+        servicelib.rabbitmq.rpc_interfaces.dynamic_scheduler.services,
+        "retrieve_inputs",
         autospec=True,
     )
 

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

@@ -78,7 +78,7 @@ async def test_workspaces_full_workflow_with_folders_and_projects(  # noqa: PLR0
     data, _ = await assert_status(resp, status.HTTP_200_OK)
     assert data["uuid"] == project["uuid"]
     assert data["workspaceId"] == added_workspace.workspace_id
-    assert data["folderId"] is None
+    assert data.get("folderId") is None
 
     # Create folder in workspace
     url = client.app.router["create_folder"].url_for()

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

@@ -93,7 +93,7 @@ async def test_moving_between_private_and_shared_workspaces(
     base_url = client.app.router["get_project"].url_for(project_id=project["uuid"])
     resp = await client.get(f"{base_url}")
     data, _ = await assert_status(resp, status.HTTP_200_OK)
-    assert data["workspaceId"] is None  # <-- Workspace ID is None
+    assert data.get("workspaceId") is None  # <-- Workspace ID is None
 
     # Move project from your private workspace to shared workspace
     base_url = client.app.router["move_project_to_workspace"].url_for(
@@ -252,7 +252,7 @@ async def test_moving_between_workspaces_check_removed_from_folder(
     base_url = client.app.router["get_project"].url_for(project_id=project["uuid"])
     resp = await client.get(f"{base_url}")
     data, _ = await assert_status(resp, status.HTTP_200_OK)
-    assert data["workspaceId"] is None  # <-- Workspace ID is None
+    assert data.get("workspaceId") is None  # <-- Workspace ID is None
 
     # Check project_to_folders DB is empty
     with postgres_db.connect() as con: