[Koha-bugs] [Bug 16607] Remove CPL/MPL from two unit tests

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Mon Jun 13 09:44:02 CEST 2016


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

--- Comment #6 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
(In reply to Jonathan Druart from comment #5)
> (In reply to Marcel de Rooy from comment #4)
> > In this case the developer uses TestBuilder to INSERT IGNORE a CPL branch in
> > just one line. He knows that TestBuilder will not insert it if it is already
> > there and he does not want to see the warning.
> 
> This assumption is wrong, tests should create the data they need to pass and
> not rely on existing data.

It was a nice flag, do not add it in the future ;)
I adjusted the Accounts.t test now. Hilarious actually.
Removed CPL/MPL from AutomaticItemModificationByAge.t.
The Holds.t patch goes on another report.

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


More information about the Koha-bugs mailing list