From 0245f2c55aea20f14e5385e6efe8315e9d20d1ad Mon Sep 17 00:00:00 2001 From: Cosimo Lupo Date: Fri, 28 Sep 2018 19:08:04 +0100 Subject: [PATCH] loggingTools: add msg=None argument to CapturingLogHandler.assertRegex like all the rest of unittest.TestCase assertSomething --- Lib/fontTools/misc/loggingTools.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/Lib/fontTools/misc/loggingTools.py b/Lib/fontTools/misc/loggingTools.py index 6b5996fcd..e267df310 100644 --- a/Lib/fontTools/misc/loggingTools.py +++ b/Lib/fontTools/misc/loggingTools.py @@ -461,13 +461,15 @@ class CapturingLogHandler(logging.Handler): def emit(self, record): self.records.append(record) - def assertRegex(self, regexp): + def assertRegex(self, regexp, msg=None): import re pattern = re.compile(regexp) for r in self.records: if pattern.search(r.getMessage()): return True - assert 0, "Pattern '%s' not found in logger records" % regexp + if msg is None: + msg = "Pattern '%s' not found in logger records" % regexp + assert 0, msg class LogMixin(object):