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

Add missing assert in Fail2banRegexTest.testFrmtOutput 

There was no associated `assertLogged()` for the "multiple id combined
to a tuple" test so nothing was actually being tested.
pull/3216/head
John Kristensen 2022-02-11 17:33:07 +11:00
parent 8b11c89ed4
commit d17e61ed5b
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
fail2ban/tests/fail2banregextestcase.py
View File

@ -364,6 +364,7 @@ class Fail2banRegexTest(LogCaptureTestCase):
self.assertTrue(_test_exec('-o', 'id',
'1591983743.667 left 192.0.2.3 right',
r'^\s*<F-TUPLE_ID_1>\S+</F-TUPLE_ID_1> <F-ID/> <F-TUPLE_ID_2>\S+</F-TUPLE_ID_2>'))
self.assertLogged(str(('192.0.2.3', 'left', 'right')))
self.pruneLog()
# id had higher precedence as ip-address:
self.assertTrue(_test_exec('-o', 'id',