Bug 40373 - Reserves.t generates a warning
Summary: Reserves.t generates a warning
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: Test Suite (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low minor
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 25515
  Show dependency treegraph
 
Reported: 2025-07-15 09:20 UTC by Jonathan Druart
Modified: 2025-07-17 22:00 UTC (History)
0 users

See Also:
GIT URL:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 40373: Remove warning from Reserves.t (1.57 KB, patch)
2025-07-15 09:21 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 40373: Remove warning from Reserves.t (1.62 KB, patch)
2025-07-17 22:00 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2025-07-15 09:20:27 UTC
t/db_dependent/Reserves.t .. 10/70 No reserves HOLD_CANCELLATION letter transported by email at /kohadevbox/koha/C4/Letters.pm line 602.
Comment 1 Jonathan Druart 2025-07-15 09:21:45 UTC
Created attachment 184076 [details] [review]
Bug 40373: Remove warning from Reserves.t

t/db_dependent/Reserves.t .. 10/70 No reserves HOLD_CANCELLATION letter transported by email at /kohadevbox/koha/C4/Letters.pm line 602.

We do not test the notification here, we can simply ignore the letters code.
However we do test it later so we need to limit the scope of mock
Comment 2 David Nind 2025-07-17 22:00:08 UTC
Created attachment 184241 [details] [review]
Bug 40373: Remove warning from Reserves.t

t/db_dependent/Reserves.t .. 10/70 No reserves HOLD_CANCELLATION letter transported by email at /kohadevbox/koha/C4/Letters.pm line 602.

We do not test the notification here, we can simply ignore the letters code.
However we do test it later so we need to limit the scope of mock

Signed-off-by: David Nind <david@davidnind.com>