From 2a67ef519c3a28b64b5b1096173b6bf8090d066d Mon Sep 17 00:00:00 2001 From: Daniel Black Date: Sun, 22 Dec 2013 08:43:57 +0000 Subject: [PATCH] TST: missing logpath raises IOError --- fail2ban-testcases | 1 + testcases/filtertestcase.py | 21 +++++++++++++++++---- 2 files changed, 18 insertions(+), 4 deletions(-) diff --git a/fail2ban-testcases b/fail2ban-testcases index 21b8fda4..c1191d0e 100755 --- a/fail2ban-testcases +++ b/fail2ban-testcases @@ -177,6 +177,7 @@ if not opts.no_network: tests.addTest(unittest.makeSuite(filtertestcase.IgnoreIP)) tests.addTest(unittest.makeSuite(filtertestcase.BasicFilter)) tests.addTest(unittest.makeSuite(filtertestcase.LogFile)) +tests.addTest(unittest.makeSuite(filtertestcase.LogFileFilterPoll)) tests.addTest(unittest.makeSuite(filtertestcase.LogFileMonitor)) if not opts.no_network: tests.addTest(unittest.makeSuite(filtertestcase.GetFailures)) diff --git a/testcases/filtertestcase.py b/testcases/filtertestcase.py index 8c706d58..8111bcd4 100644 --- a/testcases/filtertestcase.py +++ b/testcases/filtertestcase.py @@ -215,15 +215,28 @@ class IgnoreIPDNS(IgnoreIP): self.assertFalse(self.filter.inIgnoreIPList("128.178.50.11")) self.assertFalse(self.filter.inIgnoreIPList("128.178.50.13")) +class LogFile(LogCaptureTestCase): -class LogFile(unittest.TestCase): + MISSING = 'testcases/missingLogFile' + + def setUp(self): + LogCaptureTestCase.setUp(self) + + def tearDown(self): + LogCaptureTestCase.tearDown(self) + + def testMissingLogFiles(self): + self.filter = FilterPoll(None) + self.assertRaises(IOError, self.filter.addLogPath, LogFile.MISSING) + +class LogFileFilterPoll(unittest.TestCase): FILENAME = "testcases/files/testcase01.log" def setUp(self): """Call before every test case.""" self.filter = FilterPoll(None) - self.filter.addLogPath(LogFile.FILENAME) + self.filter.addLogPath(LogFileFilterPoll.FILENAME) def tearDown(self): """Call after every test case.""" @@ -233,8 +246,8 @@ class LogFile(unittest.TestCase): # self.filter.openLogFile(LogFile.FILENAME) def testIsModified(self): - self.assertTrue(self.filter.isModified(LogFile.FILENAME)) - self.assertFalse(self.filter.isModified(LogFile.FILENAME)) + self.assertTrue(self.filter.isModified(LogFileFilterPoll.FILENAME)) + self.assertFalse(self.filter.isModified(LogFileFilterPoll.FILENAME)) class LogFileMonitor(LogCaptureTestCase):