From 5e4fdb60c8a415dcf0c702a8ee2896cd25d5550d Mon Sep 17 00:00:00 2001 From: sebres Date: Mon, 26 Sep 2016 10:50:02 +0200 Subject: [PATCH] extended test-cases (coverage) --- fail2ban/server/database.py | 2 +- fail2ban/tests/databasetestcase.py | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/fail2ban/server/database.py b/fail2ban/server/database.py index 120619f9..77786f57 100644 --- a/fail2ban/server/database.py +++ b/fail2ban/server/database.py @@ -211,7 +211,7 @@ class Fail2BanDb(object): if newversion == Fail2BanDb.__version__: logSys.warning( "Database updated from '%i' to '%i'", version, newversion) - else: + else: # pragma: no cover logSys.error( "Database update failed to achieve version '%i'" ": updated from '%i' to '%i'", Fail2BanDb.__version__, version, newversion) diff --git a/fail2ban/tests/databasetestcase.py b/fail2ban/tests/databasetestcase.py index 02d2c79b..b1c68a8f 100644 --- a/fail2ban/tests/databasetestcase.py +++ b/fail2ban/tests/databasetestcase.py @@ -353,6 +353,11 @@ class DatabaseTest(LogCaptureTestCase): # be returned tickets = self.db.getBansMerged(bantime=-1) self.assertEqual(len(tickets), 2) + # getCurrentBans: + tickets = self.db.getCurrentBans(jail=self.jail) + self.assertEqual(len(tickets), 2) + ticket = self.db.getCurrentBans(jail=None, ip="127.0.0.1"); + self.assertEqual(ticket.getIP(), "127.0.0.1") def testActionWithDB(self): # test action together with database functionality