cleanup

Author: pcrespov

services/web/server/src/simcore_service_webserver/groups/_groups_api.py

@@ -178,7 +178,10 @@ async def update_user_in_group(
 
 
 async def delete_user_in_group(
-    app: web.Application, user_id: UserID, gid: GroupID, the_user_id_in_group: int
+    app: web.Application,
+    user_id: UserID,
+    gid: GroupID,
+    the_user_id_in_group: int,
 ) -> None:
     return await _groups_db.delete_user_in_group(
         app, user_id=user_id, gid=gid, the_user_id_in_group=the_user_id_in_group

services/web/server/tests/unit/with_dbs/01/test_groups.py

@@ -12,6 +12,7 @@
 import pytest
 from aiohttp.test_utils import TestClient
 from faker import Faker
+from models_library.groups import AccessRightsDict
 from pytest_simcore.helpers.assert_checks import assert_status
 from pytest_simcore.helpers.webserver_login import LoggedUser, NewUser, UserInfoDict
 from pytest_simcore.helpers.webserver_parametrizations import (
@@ -25,7 +26,6 @@
 from simcore_service_webserver._meta import API_VTAG
 from simcore_service_webserver.application_settings import setup_settings
 from simcore_service_webserver.db.plugin import setup_db
-from simcore_service_webserver.groups._common.types import AccessRightsDict
 from simcore_service_webserver.groups._groups_db import (
     _DEFAULT_GROUP_OWNER_ACCESS_RIGHTS,
     _DEFAULT_GROUP_READ_ACCESS_RIGHTS,
@@ -62,9 +62,7 @@ def client(
     # fake config
     app = create_safe_application(cfg)
 
-    settings = setup_settings(app)
-    print(settings.model_dump_json(indent=1))
-
+    setup_settings(app)
     setup_db(app)
     setup_session(app)
     setup_security(app)
@@ -92,9 +90,9 @@ def _assert__group_user(
     actual_user: dict,
 ):
     assert "first_name" in actual_user
-    assert actual_user["first_name"] == expected_user["first_name"]
+    assert actual_user["first_name"] == expected_user.get("first_name")
     assert "last_name" in actual_user
-    assert actual_user["last_name"] == expected_user["last_name"]
+    assert actual_user["last_name"] == expected_user.get("last_name")
     assert "login" in actual_user
     assert actual_user["login"] == expected_user["email"]
     assert "gravatar_id" in actual_user