rename

Author: pcrespov

packages/postgres-database/src/simcore_postgres_database/utils_tags.py

@@ -239,7 +239,7 @@ async def has_access_rights(
         self,
         connection: AsyncConnection | None = None,
         *,
-        caller_id: int,
+        user_id: int,
         tag_id: int,
         read: bool = False,
         write: bool = False,
@@ -249,7 +249,7 @@ async def has_access_rights(
             group_id_or_none = await conn.scalar(
                 has_access_rights_stmt(
                     tag_id=tag_id,
-                    caller_user_id=caller_id,
+                    caller_user_id=user_id,
                     read=read,
                     write=write,
                     delete=delete,

packages/postgres-database/tests/test_utils_tags.py

@@ -653,7 +653,7 @@ async def test_tags_repo_create(
 
     # Checks defaults to full ownership
     assert await tags_repo.has_access_rights(
-        caller_id=user.id,
+        user_id=user.id,
         tag_id=tag_1["id"],
         read=True,
         write=True,
@@ -688,7 +688,7 @@ async def test_tags_repo_access_rights(
     }
 
     assert await tags_repo.has_access_rights(
-        caller_id=user.id,
+        user_id=user.id,
         tag_id=tag["id"],
         read=True,
         write=True,
@@ -705,15 +705,15 @@ async def test_tags_repo_access_rights(
     )
 
     assert not await tags_repo.has_access_rights(
-        caller_id=other_user.id,
+        user_id=other_user.id,
         tag_id=tag["id"],
         read=user_access["read"],
         write=user_access["write"],
         delete=user_access["delete"],
     )
 
     assert await tags_repo.has_access_rights(
-        caller_id=other_user.id,
+        user_id=other_user.id,
         tag_id=tag["id"],
         read=other_user_access["read"],
         write=other_user_access["write"],
@@ -735,13 +735,13 @@ async def test_tags_repo_access_rights(
 
     # checks partial
     assert await tags_repo.has_access_rights(
-        caller_id=other_user.id,
+        user_id=other_user.id,
         tag_id=tag["id"],
         read=False,
     )
 
     assert not await tags_repo.has_access_rights(
-        caller_id=other_user.id, tag_id=tag["id"], write=True
+        user_id=other_user.id, tag_id=tag["id"], write=True
     )
 
     # DELETE access to other-user