aboutsummaryrefslogtreecommitdiff
path: root/migrations/auth
diff options
context:
space:
mode:
authorFrederick Muriuki Muriithi2023-03-06 15:19:35 +0300
committerFrederick Muriuki Muriithi2023-03-06 15:19:35 +0300
commit361a934d1c0d75da01bd19d8ac78bf15aa7e2cad (patch)
treeb8b1e401d5e6fada3ec0200bcf76c213d3cfc658 /migrations/auth
parent98e93be1b8e5353656e18f1452026db6f2902e6c (diff)
downloadgenenetwork3-361a934d1c0d75da01bd19d8ac78bf15aa7e2cad.tar.gz
auth: privileges/roles: fix issues caught by tests.
Diffstat (limited to 'migrations/auth')
-rw-r--r--migrations/auth/20230306_01_pRfxl-add-system-user-list-privilege.py2
-rw-r--r--migrations/auth/20230306_02_7GnRY-add-system-user-list-privilege-to-system-administrator-and-group-leader-roles.py2
2 files changed, 2 insertions, 2 deletions
diff --git a/migrations/auth/20230306_01_pRfxl-add-system-user-list-privilege.py b/migrations/auth/20230306_01_pRfxl-add-system-user-list-privilege.py
index 0393cd3..17db17c 100644
--- a/migrations/auth/20230306_01_pRfxl-add-system-user-list-privilege.py
+++ b/migrations/auth/20230306_01_pRfxl-add-system-user-list-privilege.py
@@ -12,7 +12,7 @@ def insert_users_list_priv(conn):
with contextlib.closing(conn.cursor()) as cursor:
cursor.execute(
"INSERT INTO privileges(privilege_id, privilege_description) "
- "VALUES('system:user:list', 'List users in the system.') "
+ "VALUES('system:user:list', 'List users in the system') "
"ON CONFLICT (privilege_id) DO NOTHING")
def delete_users_list_priv(conn):
diff --git a/migrations/auth/20230306_02_7GnRY-add-system-user-list-privilege-to-system-administrator-and-group-leader-roles.py b/migrations/auth/20230306_02_7GnRY-add-system-user-list-privilege-to-system-administrator-and-group-leader-roles.py
index 4cfd068..3caad55 100644
--- a/migrations/auth/20230306_02_7GnRY-add-system-user-list-privilege-to-system-administrator-and-group-leader-roles.py
+++ b/migrations/auth/20230306_02_7GnRY-add-system-user-list-privilege-to-system-administrator-and-group-leader-roles.py
@@ -35,7 +35,7 @@ def del_privilege_from_roles(conn):
cursor.execute(
"DELETE FROM role_privileges WHERE "
"role_id IN (?, ?) AND privilege_id='system:user:list'",
- tuple(role_ids(cursor)))
+ tuple(str(role_id) for role_id in role_ids(cursor)))
steps = [
step(add_privilege_to_roles, del_privilege_from_roles)