Bug 11537: Increasing test coverage for C4::Log
authoralex_h <dhorsman@xtra.co.nz>
Mon, 13 Jan 2014 22:52:04 +0000 (11:52 +1300)
committerGalen Charlton <gmc@esilibrary.com>
Tue, 14 Jan 2014 16:30:18 +0000 (16:30 +0000)
To test:
1. run prove t/Log.t
   Notice 1 test is run
2. apply patch
3. run prove t/Log.t
   Notice 3 tests are run

Signed-off-by: David Cook <dcook@prosentient.com.au>
Works as described.

I think there might be a typo in "Testing git log status". Should
this read "Testing get log status"? (Noticed while looking at the
code/running perl t/Log.t.)

In any case, thanks for the patch. Looks good!

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Fixed typo, thanks David

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described:
1..3
ok 1 - use C4::Log;
ok 2 - Testing GetLogStatus
ok 3 - Testing hash is non empty

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
t/Log.t

diff --git a/t/Log.t b/t/Log.t
index eca6b9c..7974758 100755 (executable)
--- a/t/Log.t
+++ b/t/Log.t
@@ -6,9 +6,12 @@
 use strict;
 use warnings;
 
-use Test::More tests => 1;
+use Test::More tests => 3;
 
 BEGIN {
         use_ok('C4::Log');
 }
 
+ok( my $hash=GetLogStatus(),"Testing GetLogStatus");
+
+ok( $hash->{BorrowersLog}, 'Testing hash is non empty');