[Koha-bugs] [Bug 10508] UT: C4::Branch.pm needs unit tests

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

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

--- Comment #22 from Galen Charlton <gmcharlt at gmail.com> ---
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.