[Koha-bugs] [Bug 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Status|RESOLVED|CLOSED -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 Jared Camins-Esakov changed: What|Removed |Added Version|rel_3_6 |rel_3_8 -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Attachment #8201|0 |1 is obsolete|| -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Status|Pushed to Master|ASSIGNED -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 Paul Poulain changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #12 from Paul Poulain --- follow-up pushed. The tests are now OK for me. -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Status|Needs Signoff |Signed Off -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Status|ASSIGNED|Needs Signoff -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Status|Pushed to Master|ASSIGNED --- Comment #11 from M. de Rooy --- Back to Passed QA for trivial followup resolving path and commenting out the two failing tests. -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Attachment #8067|0 |1 is obsolete|| --- Comment #10 from M. de Rooy --- Created attachment 8201 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8201&action=edit Followup patch -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 --- Comment #9 from M. de Rooy --- Will send a followup shortly. Tested it in its Record subdirectory. From the root it does not work. -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 Paul Poulain changed: What|Removed |Added Status|Passed QA |Pushed to Master Version|master |rel_3_6 --- Comment #8 from Paul Poulain --- trying to prove t/db_dependent/Record/Record.t after applying the patch, I get a nasty: t/db_dependent/Record/Record.t .. 1/23 Aucun fichier ou dossier de ce type at t/db_dependent/Record/Record.t line 46. # Looks like you planned 23 tests but ran 1. # Looks like your test exited with 2 just after 1. t/db_dependent/Record/Record.t .. Dubious, test returned 2 (wstat 512, 0x200) Failed 22/23 subtests Test Summary Report --- t/db_dependent/Record/Record.t (Wstat: 512 Tests: 1 Failed: 0) Non-zero exit status: 2 Parse errors: Bad plan. You planned 23 tests but ran 1. Files=1, Tests=1, 1 wallclock secs ( 0.04 usr 0.01 sys + 0.37 cusr 0.03 csys = 0.45 CPU) Result: FAIL What's wrong ? I also had a problem applying the patch: git bz apply 7661 Bug 7661 - Moving C4::tests to unit tests 7661 Moving C4::tests to unit tests Apply? [yn] y Applying: 7661 Moving C4::tests to unit tests Can't open perl script "C4/tests/Record_test.pl": Aucun fichier ou dossier de ce type Are you sure you want to commit ? You can commit with the --no-verify argument Patch left in /tmp/7661-Moving-C4tests-to-unit-tests-EEw6sJ.patch that was caused by the pre-apply hook I think, I had to force --no-verify I'll push it, we will see if things are OK on jenkins, but I have some doubts. -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 --- Comment #7 from M. de Rooy --- (In reply to comment #6) > mmm... isn't the 31.36KB patch > (http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8055) really > obsoleted ? I think it's a mistake, could you confirm ? Think it's okay. Obsoleted by signoff. The newer git version creates a smaller patch file. -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 Paul Poulain changed: What|Removed |Added CC||paul.poul...@biblibre.com --- Comment #6 from Paul Poulain --- mmm... isn't the 31.36KB patch (http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8055) really obsoleted ? I think it's a mistake, could you confirm ? -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 Ian Walls changed: What|Removed |Added Status|Signed Off |Passed QA --- Comment #5 from Ian Walls --- Fair enough. This patch just moves files, no regressions found. Cleanup for tests can be accomplished in a follow up. Marking as 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 --- Comment #4 from M. de Rooy --- (In reply to comment #3) > I believe t/db_dependent/Record/Record.t is redundant; there is a nearly > identical file in t/db_dependent with the same name. I just had a quick glance through that file and chose the careful path: move the work of someone else to a more appropriate location instead of deleting it rightaway. You are of course welcome to dig deeper and get the best of both in one test. Please send a followup ;) -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 --- Comment #3 from Ian Walls --- I believe t/db_dependent/Record/Record.t is redundant; there is a nearly identical file in t/db_dependent with the same name. -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 Liz Rea changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||wizzy...@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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 Liz Rea changed: What|Removed |Added Attachment #8055|0 |1 is obsolete|| --- Comment #2 from Liz Rea --- Created attachment 8067 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8067&action=edit 7661 Moving C4::tests to unit tests This is only a file moving patch. No behaviour changes. Signed-off-by: Liz Rea Ran tests, new tests were performed as expected. -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 M. de Rooy changed: What|Removed |Added Status|NEW |Needs Signoff Severity|normal |trivial -- 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 7661] Moving C4::tests to unit tests
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7661 --- Comment #1 from M. de Rooy --- Created attachment 8055 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=8055&action=edit Patch -- 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/