[Koha-bugs] [Bug 31493] Circulation.t fails if "RenewalPeriodBase" is set as "the current date"

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Wed Aug 31 11:17:59 CEST 2022


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31493

Emmi Takkinen <emmi.takkinen at koha-suomi.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |Needs Signoff
           Assignee|chris at bigballofwax.co.nz    |emmi.takkinen at koha-suomi.fi

--- Comment #1 from Emmi Takkinen <emmi.takkinen at koha-suomi.fi> ---
Created attachment 139995
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=139995&action=edit
Bug 31493: Prevent Circulation.t failing on "AddRenewal left both fines"

If one has syspref "RenewalPeriodBase" set as "the current date"
Circulation.t tests fail on "AddRenewal left both fines". This causes
one of the test fines to be dropped/charged. Mocking syspref
"RenewalPeriodBase" as "the old due date of the checkout" solves the issue.

To test:
1. Have syspref "RenewalPeriodBase" set as "the current date".
2. Run prove t/db_dependent/Circulation.t.
=> Tests fail.
3. Apply patch.
4. Run tests again.
=> Tests pass.

Sponsored-by: Koha-Suomi Oy

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list