increase coverage, better test and output of errors

fail2ban/client/fail2banregex.py

@@ -40,7 +40,7 @@ from optparse import OptionParser, Option
 
 from ConfigParser import NoOptionError, NoSectionError, MissingSectionHeaderError
 
-try:
+try: # pragma: no cover
 	from systemd import journal
 	from ..server.filtersystemd import FilterSystemd
 except ImportError:
@@ -80,7 +80,7 @@ def pprint_list(l, header=None):
 		s = ''
 	output( s + "|  " + "\n|  ".join(l) + '\n`-' )
 
-def journal_lines_gen(myjournal):
+def journal_lines_gen(myjournal): # pragma: no cover
 	while True:
 		try:
 			entry = myjournal.get_next()
@@ -342,7 +342,7 @@ class Fail2banRegex(object):
 				found = True
 				regex = self._ignoreregex[ret].inc()
 		except RegexException as e:
-			output( e )
+			output( 'ERROR: %s' % e )
 			return False
 		return found
 
@@ -359,10 +359,7 @@ class Fail2banRegex(object):
 				regex.inc()
 				regex.appendIP(match)
 		except RegexException as e:
-			output( e )
+			output( 'ERROR: %s' % e )
-			return False
-		except IndexError:
-			output( "Sorry, but no <HOST> found in regex" )
 			return False
 		for bufLine in orgLineBuffer[int(fullBuffer):]:
 			if bufLine not in self._filter._Filter__lineBuffer:
@@ -508,11 +505,14 @@ class Fail2banRegex(object):
 
 		cmd_log, cmd_regex = args[:2]
 
+		try:
 			if not self.readRegex(cmd_regex, 'fail'):
 				return False
-
 			if len(args) == 3 and not self.readRegex(args[2], 'ignore'):
 				return False
+		except RegexException as e:
+			output( 'ERROR: %s' % e )
+			return False
 
 		if os.path.isfile(cmd_log):
 			try:
@@ -558,18 +558,17 @@ class Fail2banRegex(object):
 def exec_command_line(*args):
 	parser = get_opt_parser()
 	(opts, args) = parser.parse_args(*args)
-	if opts.print_no_missed and opts.print_all_missed:
+	errors = []
-		sys.stderr.write("ERROR: --print-no-missed and --print-all-missed are mutually exclusive.\n\n")
+	if opts.print_no_missed and opts.print_all_missed: # pragma: no cover
-		parser.print_help()
+		errors.append("ERROR: --print-no-missed and --print-all-missed are mutually exclusive.")
-		sys.exit(-1)
+	if opts.print_no_ignored and opts.print_all_ignored: # pragma: no cover
-	if opts.print_no_ignored and opts.print_all_ignored:
+		errors.append("ERROR: --print-no-ignored and --print-all-ignored are mutually exclusive.")
-		sys.stderr.write("ERROR: --print-no-ignored and --print-all-ignored are mutually exclusive.\n\n")
-		parser.print_help()
-		sys.exit(-1)
 
 	# We need 2 or 3 parameters
 	if not len(args) in (2, 3):
-		sys.stderr.write("ERROR: provide both <LOG> and <REGEX>.\n\n")
+		errors.append("ERROR: provide both <LOG> and <REGEX>.")
+	if errors:
+		sys.stderr.write("\n".join(errors) + "\n\n")
 		parser.print_help()
 		sys.exit(-1)
 

fail2ban/tests/fail2banregextestcase.py

@@ -96,14 +96,14 @@ class Fail2banRegexTest(LogCaptureTestCase):
 		(opts, args, fail2banRegex) = _Fail2banRegex(
 			"test", r".** from <HOST>$"
 		)
-		self.assertRaises(Exception, lambda: fail2banRegex.start(opts, args))
+		self.assertFalse(fail2banRegex.start(opts, args))
 		self.assertLogged("Unable to compile regular expression")
 
 	def testWrongIngnoreRE(self):
 		(opts, args, fail2banRegex) = _Fail2banRegex(
 			"test", r".*? from <HOST>$", r".**"
 		)
-		self.assertRaises(Exception, lambda: fail2banRegex.start(opts, args))
+		self.assertFalse(fail2banRegex.start(opts, args))
 		self.assertLogged("Unable to compile regular expression")
 
 	def testDirectFound(self):
@@ -221,3 +221,11 @@ class Fail2banRegexTest(LogCaptureTestCase):
 			r"Authentication failure for .*? from <HOST>$"
 		), 0)
 		self.assertLogged('Lines: 1 lines, 0 ignored, 1 matched, 0 missed')
+		
+	def testExecCmdLine_MissFailID(self):
+		self.assertNotEqual(_test_exec_command_line(
+			'-l', 'info',
+			"Dec 31 11:59:59 [sshd] error: PAM: Authentication failure for kevin from 192.0.2.0",
+			r"Authentication failure"
+		), 0)
+		self.assertLogged('No failure-id group in ')