[Koha-bugs] [Bug 7595] Add branchcode to accountlines

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #37 from Kyle M Hall <kyle at bywatersolutions.com> ---
(In reply to M. Tompsett from comment #36)
> (In reply to Jonathan Druart from comment #35)
> > Please provide tests for changes in modules.
> 
> True, most is covered:
> t/db_dependent/Accounts.t
> t/db_dependent/Circulation.t
> t/db_dependent/Circulation_issue.t
> t/db_dependent/Reserves.t
> 
> However, the makepayment, chargelostitem, manualinvoice,
> recordpayment_selectaccts, and makepartialpayment need test coverage.
> I'm also not certain how to trigger _FixAccountForLostAndReturned, except
> that AddIssue does call it.

I'm going to argue that we should for the time being, pass on tests for the
three payment handling subroutines, as they will be unified by bug 15894. If
this is agreeable, I will write tests for chargelostitem, manualinvoice, and
_FixAccountForLostAndReturned. Bug 15894 will include unit testing for
payments.

How does that sound?

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