From 67c6f3c9705289f6e0ee08c000a085be84db6735 Mon Sep 17 00:00:00 2001 From: YueGuobin Date: Thu, 28 May 2026 01:24:22 +0800 Subject: [PATCH] test: fix privilege count assertions after adding LLMConfig privileges The LLMConfig.Audit and LLMConfig.Modify privileges added to the User role increased the default privilege count from 25 to 27. Update test assertions to match the new counts. --- tests/api/routes/controller/test_roles.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/api/routes/controller/test_roles.py b/tests/api/routes/controller/test_roles.py index b6fca22b6..a98be8bf0 100644 --- a/tests/api/routes/controller/test_roles.py +++ b/tests/api/routes/controller/test_roles.py @@ -125,7 +125,7 @@ class TestRolesPrivilegesRoutes: ) assert response.status_code == status.HTTP_204_NO_CONTENT privileges = await rbac_repo.get_role_privileges(role_in_db.role_id) - assert len(privileges) == 25 # 24 default privileges + 1 custom privilege + assert len(privileges) == 27 # 25 default privileges + 2 LLMConfig privileges async def test_get_role_privileges( self, @@ -143,7 +143,7 @@ class TestRolesPrivilegesRoutes: role_id=role_in_db.role_id) ) assert response.status_code == status.HTTP_200_OK - assert len(response.json()) == 25 # 24 default privileges + 1 custom privilege + assert len(response.json()) == 27 # 25 default privileges + 2 LLMConfig privileges async def test_remove_privilege_from_role( self, @@ -165,4 +165,4 @@ class TestRolesPrivilegesRoutes: ) assert response.status_code == status.HTTP_204_NO_CONTENT privileges = await rbac_repo.get_role_privileges(role_in_db.role_id) - assert len(privileges) == 24 # 24 default privileges + assert len(privileges) == 26 # 26 default privileges (24 + 2 LLMConfig)