[Koha-bugs] [Bug 9016] Multi transport types for notices

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

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9016

Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Needs Signoff               |Failed QA

--- Comment #30 from Olli-Antti Kivilahti <olli-antti.kivilahti at jns.fi> ---
Testing using the latest master

---------------------
+ updatedatabase.pl +
---------------------
Having a problem updating these changes to my DB from updatedatabase.pl 
+$DBversion = "3.13.00.XXX";
+if ( C4::Context->preference("Version") < TransformToNum($DBversion) ) {
Disabling this if-clause enables the DB modifications which work ok.

-------------------
+ perl Overdues.t +
-------------------
ok 1 - use C4::Overdues;
ok 2 - C4::Overdues->can('GetOverdueMessageTransportTypes')
DBD::mysql::db do failed: Duplicate entry 'CPL-PT' for key 'PRIMARY' at
Overdues.t line 23.
DBD::mysql::db do failed: Duplicate entry 'CPL-PT' for key 'PRIMARY' at
Overdues.t line 23.
# Tests were run but no plan was declared and done_testing() was not seen.
# Looks like your test exited with 255 just after 2.

Caused by me having the records in my db already, problematic since test plan
encourages to have existing data in db. A check should be in place to make sure
no duplicate entries are INSERTed or maybe use a internal function to update
the db?
Otherwise tests run ok.



Sorry for taking so long to get back at this.
So close Joubu! Lemme sign this off!

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