From d17e61ed5bc95735b06f30855873fc15b7a98ac9 Mon Sep 17 00:00:00 2001 From: John Kristensen Date: Fri, 11 Feb 2022 17:33:07 +1100 Subject: [PATCH] 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. --- fail2ban/tests/fail2banregextestcase.py | 1 + 1 file changed, 1 insertion(+) diff --git a/fail2ban/tests/fail2banregextestcase.py b/fail2ban/tests/fail2banregextestcase.py index 26b40abb..b526dbf5 100644 --- a/fail2ban/tests/fail2banregextestcase.py +++ b/fail2ban/tests/fail2banregextestcase.py @@ -364,6 +364,7 @@ class Fail2banRegexTest(LogCaptureTestCase): self.assertTrue(_test_exec('-o', 'id', '1591983743.667 left 192.0.2.3 right', r'^\s*\S+ \S+')) + self.assertLogged(str(('192.0.2.3', 'left', 'right'))) self.pruneLog() # id had higher precedence as ip-address: self.assertTrue(_test_exec('-o', 'id',