From b64a5245f12d277abc3866a1924999c3c34e3014 Mon Sep 17 00:00:00 2001
From: Mark Tompsett <mtompset@hotmail.com>
Date: Mon, 20 Apr 2015 04:32:06 -0400
Subject: [PATCH] Bug 13889: Added Test Coverage

Previous commit was lacking test coverage of new function.

TEST PLAN
---------
1) Apply all the patches
2) prove -v t/db_dependent/Log.t
   -- should be success.
3) koha qa test tools
---
 t/db_dependent/Log.t | 27 ++++++++++++++++++++++++++-
 1 file changed, 26 insertions(+), 1 deletion(-)

diff --git a/t/db_dependent/Log.t b/t/db_dependent/Log.t
index 8a25c18..cbd8937 100644
--- a/t/db_dependent/Log.t
+++ b/t/db_dependent/Log.t
@@ -6,10 +6,15 @@
 
 use strict;
 use warnings;
-use Test::More tests => 5;
+use Test::More tests => 7;
 
 # We need C4::Dates to handle the dates
 use C4::Dates;
+use C4::Context;
+use t::lib::Mocks qw/mock_preference/; # to mock CronjobLog
+use Data::Dumper;
+
+$| = 1;
 
 BEGIN {
 	use_ok('C4::Log');
@@ -53,3 +58,23 @@ eval {
     $success = 0;
 };
 ok($success, "GetLogs seemed to find ".$success." like our test record in a tighter search");
+
+# Make sure we can rollback.
+my $dbh = C4::Context->dbh;
+$dbh->{AutoCommit} = 0;
+$dbh->{RaiseError} = 1;
+
+# We want numbers to be the same between runs.
+$dbh->do("DELETE FROM action_logs;");
+
+t::lib::Mocks::mock_preference('CronjobLog',0);
+cronlogaction();
+my $cronJobCount = $dbh->selectrow_array("SELECT COUNT(*) FROM action_logs WHERE module='CRONJOBS';",{});
+is($cronJobCount,0,"Cronjob not logged as expected.");
+
+t::lib::Mocks::mock_preference('CronjobLog',1);
+cronlogaction();
+$cronJobCount = $dbh->selectrow_array("SELECT COUNT(*) FROM action_logs WHERE module='CRONJOBS';",{});
+is($cronJobCount,1,"Cronjob logged as expected.");
+
+$dbh->rollback();
-- 
1.9.1