[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 --- Comment #11 from Tomás Cohen Arazi--- (In reply to Frédéric Demians from comment #10) > Is this patch needed on 3.20.x branch for its test suite? I don't think so. It is part of a bigger tests rewrite. If it applies, it shouldn't harm and could help make jenkins stable for 3.20. But maybe it is not worth the trouble. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Frédéric Demianschanged: What|Removed |Added CC||frede...@tamil.fr --- Comment #10 from Frédéric Demians --- Is this patch needed on 3.20.x branch for its test suite? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Jonathan Druartchanged: What|Removed |Added Blocks||14878 Referenced Bugs: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14878 [Bug 14878] Tests - Create the branchcodes if needed -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 --- Comment #7 from Kyle M Hall--- Created attachment 44388 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44388=edit [PASSED QA] Bug 15081: (followup) Make test files using TestBuilder handle their transactions This trivial patch introduces the code needed on the test files so they handle the DB transaction instead of relying on the (removed) transaction started/rolled back by TestBuilder. Tested all of the files before and after applying the patch, resultes are the same. (Pass exept of t/db_dependent/Barcodes_ValueBuilder.t, this has the same error). Signed-off-by: Marc Véron Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 --- Comment #8 from Kyle M Hall--- Created attachment 44389 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44389=edit Bug 15081 [QA Followup] - Fix issue with Barcodes_ValueBuilder.t -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Kyle M Hallchanged: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Kyle M Hallchanged: What|Removed |Added Attachment #44284|0 |1 is obsolete|| Attachment #44286|0 |1 is obsolete|| --- Comment #6 from Kyle M Hall --- Created attachment 44387 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44387=edit [PASSED QA] Bug 15081: Take transaction handling out from TestBuilder This patch removes transaction handling code from TestBuilder. It fixes the TestBuilder.t to handle the transaction on its own. Verify that t/db_dependent/TestBuilder.t passes. Followed test plan, TestBuilder.t passes Signed-off-by: Marc Véron Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Tomás Cohen Arazichanged: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #9 from Tomás Cohen Arazi --- Patches pushed to master. Thanks for the followup, Kyle! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Tomás Cohen Arazichanged: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org Assignee|ch...@bigballofwax.co.nz|tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Tomás Cohen Arazichanged: What|Removed |Added Patch complexity|--- |Trivial patch Status|NEW |Needs Signoff CC||tomasco...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 --- Comment #2 from Tomás Cohen Arazi--- Created attachment 44282 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44282=edit Bug 15081: (followup) Make test files using TestBuilder handle their transactions This trivial patch introduces the code needed on the test files so they handle the DB transaction instead of relying on the (removed) transaction started/rolled back by TestBuilder. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 --- Comment #1 from Tomás Cohen Arazi--- Created attachment 44281 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44281=edit Bug 15081: Take transaction handling out from TestBuilder This patch removes transaction handling code from TestBuilder. It fixes the TestBuilder.t to handle the transaction on its own. Verify that t/db_dependent/TestBuilder.t passes. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Marc Véronchanged: What|Removed |Added Attachment #44282|0 |1 is obsolete|| --- Comment #4 from Marc Véron --- Created attachment 44286 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44286=edit Bug 15081: (followup) Make test files using TestBuilder handle their transactions This trivial patch introduces the code needed on the test files so they handle the DB transaction instead of relying on the (removed) transaction started/rolled back by TestBuilder. Tested all of the files before and after applying the patch, resultes are the same. (Pass exept of t/db_dependent/Barcodes_ValueBuilder.t, this has the same error). Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Marc Véronchanged: What|Removed |Added Status|Needs Signoff |Signed Off CC||ve...@veron.ch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 Marc Véronchanged: What|Removed |Added Attachment #44281|0 |1 is obsolete|| --- Comment #3 from Marc Véron --- Created attachment 44284 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=44284=edit Bug 15081: Take transaction handling out from TestBuilder This patch removes transaction handling code from TestBuilder. It fixes the TestBuilder.t to handle the transaction on its own. Verify that t/db_dependent/TestBuilder.t passes. Followed test plan, TestBuilder.t passes Signed-off-by: Marc Véron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 15081] TestBuilder should not handle the transaction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15081 --- Comment #5 from Marc Véron--- Note: With and without patch, t/db_dependent/Barcodes_ValueBuilder.t fails with: DBD::mysql::db do failed: Cannot delete or update a parent row: a foreign key constraint fails (`koha_kohadev`.`issues`, CONSTRAINT `issues_ibfk_2` FOREIGN KEY (`itemnumber`) REFERENCES `items` (`itemnumber`) ON UPDATE CASCADE) [for Statement "DELETE FROM items"] at t/db_dependent/Barcodes_ValueBuilder.t line 35. not ok 2 - incremental barcode -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/