Dan Helfman před 4 měsíci
rodič
revize
803361b850
2 změnil soubory, kde provedl 11 přidání a 6 odebrání
  1. 6 4
      borgmatic/logger.py
  2. 5 2
      tests/unit/test_logger.py

+ 6 - 4
borgmatic/logger.py

@@ -298,8 +298,10 @@ def configure_logging(
             syslog_handler = logging.handlers.SysLogHandler(address=syslog_path)
             syslog_handler.setFormatter(
                 logging.Formatter(
-                    'borgmatic: {levelname} {prefix}{message}', style='{', defaults={'prefix': ''}
-                )  # noqa: FS003
+                    'borgmatic: {levelname} {prefix}{message}',  # noqa: FS003
+                    style='{',
+                    defaults={'prefix': ''},
+                )
             )
             syslog_handler.setLevel(syslog_log_level)
             handlers.append(syslog_handler)
@@ -308,9 +310,9 @@ def configure_logging(
         file_handler = logging.handlers.WatchedFileHandler(log_file)
         file_handler.setFormatter(
             logging.Formatter(
-                log_file_format or '[{asctime}] {levelname}: {prefix}{message}',
+                log_file_format or '[{asctime}] {levelname}: {prefix}{message}',  # noqa: FS003
                 style='{',
-                defaults={'prefix': ''},  # noqa: FS003
+                defaults={'prefix': ''},
             )
         )
         file_handler.setLevel(log_file_log_level)

+ 5 - 2
tests/unit/test_logger.py

@@ -177,8 +177,9 @@ def test_multi_stream_handler_logs_to_handler_for_log_level():
 def test_console_color_formatter_format_includes_log_message():
     flexmock(module).should_receive('add_custom_log_levels')
     flexmock(module.logging).ANSWER = module.ANSWER
+    flexmock(module.logging.Formatter).should_receive('format')
     plain_message = 'uh oh'
-    record = flexmock(levelno=logging.CRITICAL, msg=plain_message)
+    record = flexmock(levelno=logging.CRITICAL)
 
     colored_message = module.Console_color_formatter().format(record)
 
@@ -409,7 +410,9 @@ def test_configure_logging_to_log_file_formats_with_custom_log_format():
     flexmock(module).should_receive('add_custom_log_levels')
     flexmock(module.logging).ANSWER = module.ANSWER
     flexmock(module.logging).should_receive('Formatter').with_args(
-        '{message}', style='{'  # noqa: FS003
+        '{message}',  # noqa: FS003
+        style='{',
+        defaults={'prefix': ''},
     ).once()
     fake_formatter = flexmock()
     flexmock(module).should_receive('Console_color_formatter').and_return(fake_formatter)