http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Yohann Dufour <yohann.dufour@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Assignee|chris@bigballofwax.co.nz |yohann.dufour@biblibre.com -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 --- Comment #1 from Yohann Dufour <yohann.dufour@biblibre.com> --- Created attachment 29094 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29094&action=edit Bug 12457: Removal warning in t/db_dependent/Members.t When I execute prove t/db_dependent/Members.t, this warning appears twice : "Argument "Test" isn't numeric in modulus (%) at /home/yohann/koha/C4/Context.pm line 1256." The warning is due to a invalid use of the routine set_userenv Test plan: 1/ Execute : prove t/db_dependent/Members.t 2/ The warning appears twice : "Argument "Test" isn't numeric in modulus (%) at /home/yohann/koha/C4/Context.pm line 1256." 3/ Apply the patch 4/ Execute : prove t/db_dependent/Members.t 5/ The result has to be a success without error or warning : t/db_dependent/Members.t .. ok All tests successful. Files=1, Tests=26, 2 wallclock secs ( 0.04 usr 0.01 sys + 1.50 cusr 0.08 csys = 1.63 CPU) Result: PASS -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 --- Comment #2 from Yohann Dufour <yohann.dufour@biblibre.com> --- Created attachment 29173 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29173&action=edit Bug 12457: Adding unit tests for the routines AddMessage, GetMessages, GetMessagesCount, DeleteMessage, GetPendingIssues, GetAllIssues of the module C4/Members.pm and modification of the routine GetAllIssues Adding unit tests for the routines AddMessage, GetMessages, GetMessagesCount and DeleteMessage in t/db_dependent/Members.t Adding unit tests for the routines GetPendingIssues and GetAllIssues in separate files : t/db_dependent/Members/GetPendingIssues.t and t/db_dependent/Members/GetAllIssues.t The routine GetAllIssues has been modified because it does not test if the arguments was defined : - the borrowernumber argument is required - if the order argument is not given, it takes a value by default : 'date_due desc' - the limit argument is optional Test plan: 1/ Apply the patch 2/ Execute : prove t/db_dependent/Members.t t/db_dependent/Members/GetAllIssues.t t/db_dependent/Members/GetPendingIssues.t 3/ The result has to be a success without error or warning : t/db_dependent/Members.t ................... ok t/db_dependent/Members/GetAllIssues.t ...... ok t/db_dependent/Members/GetPendingIssues.t .. ok All tests successful. Files=3, Tests=83, 5 wallclock secs ( 0.06 usr 0.01 sys + 4.68 cusr 0.26 csys = 5.01 CPU) Result: PASS -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Yohann Dufour <yohann.dufour@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #5 from Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> --- First patch Ok, test pass On second patch, first test pass, but the other two fail :( t/db_dependent/Members.t ................... ok t/db_dependent/Members/GetAllIssues.t ...... DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`kohadev`.`items`, CONSTRAINT `items_ibfk_2` FOREIGN KEY (`homebranch`) REFERENCES `branches` (`branchcode`) ON UPDATE CASCADE) at t/db_dependent/Members/GetAllIssues.t line 22. DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`kohadev`.`items`, CONSTRAINT `items_ibfk_2` FOREIGN KEY (`homebranch`) REFERENCES `branches` (`branchcode`) ON UPDATE CASCADE) at t/db_dependent/Members/GetAllIssues.t line 22. # Looks like your test exited with 255 before it could output anything. t/db_dependent/Members/GetAllIssues.t ...... Dubious, test returned 255 (wstat 65280, 0xff00) Failed 16/16 subtests t/db_dependent/Members/GetPendingIssues.t .. DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`kohadev`.`items`, CONSTRAINT `items_ibfk_2` FOREIGN KEY (`homebranch`) REFERENCES `branches` (`branchcode`) ON UPDATE CASCADE) at t/db_dependent/Members/GetPendingIssues.t line 22. DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`kohadev`.`items`, CONSTRAINT `items_ibfk_2` FOREIGN KEY (`homebranch`) REFERENCES `branches` (`branchcode`) ON UPDATE CASCADE) at t/db_dependent/Members/GetPendingIssues.t line 22. # Looks like your test exited with 255 before it could output anything. t/db_dependent/Members/GetPendingIssues.t .. Dubious, test returned 255 (wstat 65280, 0xff00) Failed 20/20 subtest -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 --- Comment #6 from Yohann Dufour <yohann.dufour@biblibre.com> --- Created attachment 29908 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=29908&action=edit Bug 12457: Allow UT on a non empty database -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #29094|0 |1 is obsolete| | Attachment #29742|0 |1 is obsolete| | Attachment #29908|0 |1 is obsolete| | --- Comment #7 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 30116 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=30116&action=edit Bug 12457: Removal warning in t/db_dependent/Members.t When I execute prove t/db_dependent/Members.t, this warning appears twice : "Argument "Test" isn't numeric in modulus (%) at /home/yohann/koha/C4/Context.pm line 1256." The warning is due to a invalid use of the routine set_userenv Test plan: 1/ Execute : prove t/db_dependent/Members.t 2/ The warning appears twice : "Argument "Test" isn't numeric in modulus (%) at /home/yohann/koha/C4/Context.pm line 1256." 3/ Apply the patch 4/ Execute : prove t/db_dependent/Members.t 5/ The result has to be a success without error or warning : t/db_dependent/Members.t .. ok All tests successful. Files=1, Tests=26, 2 wallclock secs ( 0.04 usr 0.01 sys + 1.50 cusr 0.08 csys = 1.63 CPU) Result: PASS Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 --- Comment #9 from Jonathan Druart <jonathan.druart@biblibre.com> --- Created attachment 30118 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=30118&action=edit Bug 12457: Allow UT on a non empty database Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA CC| |katrin.fischer@bsz-bw.de --- Comment #11 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am sorry, but GetPendingIssues.t still fails for me: $ perl t/db_dependent/Members/GetPendingIssues.t 1..20 DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`koha`.`items`, CONSTRAINT `items_ibfk_3` FOREIGN KEY (`holdingbranch`) REFERENCES `branches` (`branchcode`) ON UPDATE CASCADE) at t/db_dependent/Members/GetPendingIssues.t line 21. DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`koha`.`items`, CONSTRAINT `items_ibfk_3` FOREIGN KEY (`holdingbranch`) REFERENCES `branches` (`branchcode`) ON UPDATE CASCADE) at t/db_dependent/Members/GetPendingIssues.t line 21. # Looks like your test exited with 255 before it could output anything. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Yohann Dufour <yohann.dufour@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #13 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- I am sorry Yohann, but this still fails for me: $ perl t/db_dependent/Members/GetAllIssues.t 1..16 DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`koha`.`issues`, CONSTRAINT `issues_ibfk_1` FOREIGN KEY (`borrowernumber`) REFERENCES `borrowers` (`borrowernumber`) ON UPDATE CASCADE) at t/db_dependent/Members/GetAllIssues.t line 20. DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`koha`.`issues`, CONSTRAINT `issues_ibfk_1` FOREIGN KEY (`borrowernumber`) REFERENCES `borrowers` (`borrowernumber`) ON UPDATE CASCADE) at t/db_dependent/Members/GetAllIssues.t line 20. # Looks like your test exited with 255 before it could output anything. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Yohann Dufour <yohann.dufour@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Signed Off -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #30116|0 |1 is obsolete| | Attachment #30117|0 |1 is obsolete| | Attachment #30266|0 |1 is obsolete| | --- Comment #15 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 30309 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=30309&action=edit [PASSED QA] Bug 12457: Removal warning in t/db_dependent/Members.t When I execute prove t/db_dependent/Members.t, this warning appears twice : "Argument "Test" isn't numeric in modulus (%) at /home/yohann/koha/C4/Context.pm line 1256." The warning is due to a invalid use of the routine set_userenv Test plan: 1/ Execute : prove t/db_dependent/Members.t 2/ The warning appears twice : "Argument "Test" isn't numeric in modulus (%) at /home/yohann/koha/C4/Context.pm line 1256." 3/ Apply the patch 4/ Execute : prove t/db_dependent/Members.t 5/ The result has to be a success without error or warning : t/db_dependent/Members.t .. ok All tests successful. Files=1, Tests=26, 2 wallclock secs ( 0.04 usr 0.01 sys + 1.50 cusr 0.08 csys = 1.63 CPU) Result: PASS Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> All new and old tests pass now. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12457 --- Comment #17 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 30311 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=30311&action=edit [PASSED QA] Bug 12457: Allow UT on a non empty database Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> -- You are receiving this mail because: You are watching all bug changes.