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

--- Comment #22 from Galen Charlton <[email protected]> ---
Comment on attachment 19853
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=19853
[PASSED QA] Bug 10508 : C4::Branch.pm needs unit tests.

Review of attachment 19853:
 --> 
(http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=10508&attachment=19853)
-----------------------------------------------------------------

::: t/db_dependent/Branch.t
@@ +21,4 @@
>  use C4::Context;
>  use Data::Dumper;
>  
> +use Test::More ;#tests => 6;

There is nothing in this test script that makes the number of tests
unpredictable.  Consequently, I strongly prefer that the number of tests be
declared explicitly, and that done_testing() be used only in cases where it is
legitimately uncertain how many tests will be run.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
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/

Reply via email to