https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16607 --- Comment #6 from Marcel de Rooy <m.de.rooy@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.