| Summary: | Add more test coverage for bug 30724 | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Nick Clemens (kidclamp) <nick> |
| Component: | Authentication | Assignee: | Tomás Cohen Arazi (tcohen) <tomascohen> |
| Status: | Pushed to main --- | QA Contact: | Jonathan Druart <jonathan.druart> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | dpavlin, jonathan.druart, kyle, lucas, paul.derscheid, tomascohen |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | Small patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
25.11.00
|
|
| Circulation function: | |||
| Bug Depends on: | 30724 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 41038: Add tests for superlibrarian managing other users' 2FA
Bug 41038: Add tests for superlibrarian managing other users' 2FA Bug 41038: Add tests for superlibrarian managing other users' 2FA |
||
|
Description
Nick Clemens (kidclamp)
2025-10-16 19:49:10 UTC
Created attachment 188995 [details] [review] Bug 41038: Add tests for superlibrarian managing other users' 2FA Adds comprehensive selenium tests for the new functionality where superlibrarians can manage 2FA settings for other patrons. To test: 1. Run the existing authentication_2fa.t selenium tests 2. Notice missing coverage for cross-user 2FA management 3. Apply the patch 4. Run the tests again 5. Verify new subtests pass: - Superlibrarian can view/disable other users' 2FA - Enable button is hidden with explanatory message - Non-superlibrarian gets 403 error for other users Created attachment 188999 [details] [review] Bug 41038: Add tests for superlibrarian managing other users' 2FA Adds comprehensive selenium tests for the new functionality where superlibrarians can manage 2FA settings for other patrons. To test: 1. Run the existing authentication_2fa.t selenium tests 2. Notice missing coverage for cross-user 2FA management 3. Apply the patch 4. Run the tests again 5. Verify new subtests pass: - Superlibrarian can view/disable other users' 2FA - Enable button is hidden with explanatory message - Non-superlibrarian gets 403 error for other users Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Created attachment 189602 [details] [review] Bug 41038: Add tests for superlibrarian managing other users' 2FA Adds comprehensive selenium tests for the new functionality where superlibrarians can manage 2FA settings for other patrons. To test: 1. Run the existing authentication_2fa.t selenium tests 2. Notice missing coverage for cross-user 2FA management 3. Apply the patch 4. Run the tests again 5. Verify new subtests pass: - Superlibrarian can view/disable other users' 2FA - Enable button is hidden with explanatory message - Non-superlibrarian gets 403 error for other users Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Nice work everyone! Pushed to main for 25.11 Nice work everyone! Pushed to 25.05.x |