github
/
fail2ban
mirror of https://github.com/fail2ban/fail2ban
1
0
Fork 0
Code Issues Releases Wiki Activity

ENH/BF(TST): making permissions restrictive is not sufficient -- really remove file to test

pull/784/head
Yaroslav Halchenko 2014-08-12 11:31:42 -04:00
parent 3576c509f5
commit b2a1032f57
1 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
fail2ban/tests/filtertestcase.py
View File

@ -24,6 +24,7 @@ __license__ = "GPL"
from __builtin__ import open as fopen from __builtin__ import open as fopen
import unittest import unittest
import getpass
import os import os
import sys import sys
import time import time
@ -349,10 +350,20 @@ class LogFileMonitor(LogCaptureTestCase):
# shorter wait time for not modified status # shorter wait time for not modified status
return not self.isModified(0.4) return not self.isModified(0.4)
def testNoLogFile(self): def testUnaccessibleLogFile(self):
os.chmod(self.name, 0) os.chmod(self.name, 0)
self.filter.getFailures(self.name) self.filter.getFailures(self.name)
self.assertTrue(self._is_logged('Unable to open %s' % self.name)) failure_was_logged = self._is_logged('Unable to open %s' % self.name)
is_root = getpass.getuser() == 'root'
# If ran as root, those restrictive permissions would not
# forbid log to be read.
self.assertTrue(failure_was_logged != is_root)
def testNoLogFile(self):
_killfile(self.file, self.name)
self.filter.getFailures(self.name)
failure_was_logged = self._is_logged('Unable to open %s' % self.name)
self.assertTrue(failure_was_logged)
def testRemovingFailRegex(self): def testRemovingFailRegex(self):
self.filter.delFailRegex(0) self.filter.delFailRegex(0)