fix: node names mapping

Author: giancarloromeo

packages/models-library/src/models_library/projects.py

@@ -107,8 +107,9 @@ class BaseProjectModel(BaseModel):
     last_change_date: datetime
 
     # Pipeline of nodes (SEE projects_nodes.py)
-    # FIXME: pedro removes this one
-    workbench: Annotated[NodesDict, Field(description="Project's pipeline")]
+    # FIXME: pedro checks this one
+    # NOTE: GCG: a validation failed (See: services/storage/src/simcore_service_storage/modules/db/projects.py)
+    # workbench: Annotated[NodesDict, Field(description="Project's pipeline")]
 
 
 class ProjectAtDB(BaseProjectModel):

services/storage/src/simcore_service_storage/modules/db/projects.py

@@ -27,7 +27,7 @@ async def list_valid_projects_in(
         async with pass_or_acquire_connection(self.db_engine, connection) as conn:
             async for row in await conn.stream(
                 sa.select(projects).where(
-                    projects.c.uuid.in_([f"{pid}" for pid in project_uuids])
+                    projects.c.uuid.in_(f"{pid}" for pid in project_uuids)
                 )
             ):
                 with suppress(ValidationError):
@@ -43,7 +43,7 @@ async def get_project_id_and_node_id_to_names_map(
         async with pass_or_acquire_connection(self.db_engine, connection) as conn:
             async for row in await conn.stream(
                 sa.select(projects.c.uuid, projects.c.name).where(
-                    projects.c.uuid.in_([f"{pid}" for pid in project_uuids])
+                    projects.c.uuid.in_(f"{pid}" for pid in project_uuids)
                 )
             ):
                 names_map[ProjectID(row.uuid)] = {f"{row.uuid}": row.name}

services/storage/src/simcore_service_storage/simcore_s3_dsm.py

@@ -134,7 +134,7 @@ async def _add_frontend_needed_data(
         assert d.project_id  # nosec
         names_mapping = prj_names_mapping[d.project_id]
         d.project_name = names_mapping[f"{d.project_id}"]
-        if d.node_id in names_mapping:
+        if f"{d.node_id}" in names_mapping:
             assert d.node_id  # nosec
             d.node_name = names_mapping[f"{d.node_id}"]
         if d.node_name and d.project_name:

services/storage/tests/unit/test_handlers_files.py

@@ -63,7 +63,7 @@
 from types_aiobotocore_s3 import S3Client
 from yarl import URL
 
-pytest_simcore_core_services_selection = ["postgres"]
+pytest_simcore_core_services_selection = ["postgres", "rabbit"]
 pytest_simcore_ops_services_selection = ["adminer"]