permissions: fix system librarian role only

A system librarian should have all the librarian rights. * Removes the librarian role to the system librarian of Martigny in the test fixtures to test this specific case. * Closes #1340. Co-Authored-by: Johnny Mariéthoz <[email protected]>
rero · Nov 5, 2020 · 8b52957 · 8b52957
1 parent f90cfbd
commit 8b52957
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/rero_ils/permissions.py b/rero_ils/permissions.py
@@ -28,7 +28,8 @@
 from .modules.patrons.api import Patron
 
 request_item_permission = Permission(RoleNeed('patron'))
-librarian_permission = Permission(RoleNeed('librarian'))
+librarian_permission = Permission(
+    RoleNeed('librarian'), RoleNeed('system_librarian'))
 admin_permission = Permission(RoleNeed('admin'))
 editor_permission = Permission(RoleNeed('editor'), RoleNeed('admin'))
 

diff --git a/tests/data/data.json b/tests/data/data.json
@@ -2767,8 +2767,7 @@
     "username": "simonetta",
     "phone": "+41324993585",
     "roles": [
-      "system_librarian",
-      "librarian"
+      "system_librarian"
     ],
     "patron": {
       "expiration_date": "2023-10-07",