| Summary: | Letters.t assumes an empty ReplyToDefault | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Marcel de Rooy <m.de.rooy> |
| Component: | Test Suite | Assignee: | Marcel de Rooy <m.de.rooy> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | minor | ||
| Priority: | P5 - low | CC: | fridolin.somers, lucas |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 |
| Patch complexity: | Trivial patch | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: |
24.05.00,23.11.06
|
Circulation function: | |
| Attachments: |
Bug 36268: Add a mock for ReplyToDefault
Bug 36268: Add a mock for ReplyToDefault |
||
|
Description
Marcel de Rooy
2024-03-07 10:08:56 UTC
Created attachment 162895 [details] [review] Bug 36268: Add a mock for ReplyToDefault Test plan: Without this patch, add email address in that pref. Run Letters.t. It fails. Now apply patch and run test again. Should pass. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Trivial test change: self SO Created attachment 164261 [details] [review] Bug 36268: Add a mock for ReplyToDefault Test plan: Without this patch, add email address in that pref. Run Letters.t. It fails. Now apply patch and run test again. Should pass. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed for 24.05! Well done everyone, thank you! Pushed to 23.11.x for 23.11.06 Doesn't apply cleanly to 23.05.x, no backport. |