The test creates two users, and says list should return both, however, the user performing the call doesn't have 'view_borrower_infos_from_any_libraries' permission and only lists 1, which is what the test checks
Created attachment 173873 [details] [review] Bug 38322: Adust tests
Created attachment 174297 [details] [review] Bug 38322: Adjust tests Signed-off-by: David Nind <david@davidnind.com>
Tests pass before and after the patch: prove t/db_dependent/api/v1/erm_users.t
Created attachment 174450 [details] [review] Bug 38322: Adjust tests Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Created attachment 174451 [details] [review] Bug 38322: (QA follow-up) Fix spelling Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Pushed for 24.11! Well done everyone, thank you!
Backported to 24.05.x for upcoming 24.05.06
I've added a release note, but I'm not sure that I have described what the problem was, and what was fixed.
Pushed to 23.11.x for 23.11.11
Not pushed to LTS. Marked Resolved. If you feel this should be in LTS please reply with your reason.