| Summary: | Missing Cypress tests for patron display | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
| Component: | Test Suite | Assignee: | Jonathan Druart <jonathan.druart> |
| Status: | Pushed to main --- | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | lucas, nick |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | --- |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
25.11.00
|
|
| Circulation function: | |||
| Bug Depends on: | |||
| Bug Blocks: | 40958 | ||
| Attachments: |
Bug 41003: Remove extra spaces
Bug 41003: Add Cypress tests to cover patron formatting Bug 41003: Add Cypress tests to cover patron formatting Bug 41003: Add Cypress tests to cover patron formatting |
||
|
Description
Jonathan Druart
2025-10-13 13:16:03 UTC
Created attachment 187837 [details] [review] Bug 41003: Remove extra spaces Created attachment 187838 [details] [review] Bug 41003: Add Cypress tests to cover patron formatting Created attachment 187840 [details] [review] Bug 41003: Add Cypress tests to cover patron formatting No one tested this? (In reply to Lucas Gass (lukeg) from comment #4) > No one tested this? I didn't think it was needed for several reasons: * This patch adds tests for "main", nothing else (the "Remove extra spaces" change is not something visible on the interface, only to make the tests pass) * It has been added as blocker of other bugs and I didn't want to block them because of tests (especially bug 40245 that is not from me) * Follow-up bugs will extend those tests, and so testers and QAers will confirm they work anyway But feel free to ask for a QA stamp if you are not comfortable with the patches! Created attachment 188802 [details] [review] Bug 41003: Add Cypress tests to cover patron formatting Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Conflicts from bug 30568 - I don't believe first patch was needed any longer - dropped Nice work everyone! Pushed to main for 25.11 |