mirror of
https://gerrit.googlesource.com/git-repo
synced 2025-06-26 20:17:52 +00:00
logging: Use log.formatter for coloring logs
Bug: b/292704435 Change-Id: Iebdf8fb7666592dc5df2b36aae3185d1fc71bd66 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/385514 Tested-by: Aravind Vasudevan <aravindvasudev@google.com> Commit-Queue: Aravind Vasudevan <aravindvasudev@google.com> Reviewed-by: Mike Frysinger <vapier@google.com>
This commit is contained in:
@ -20,42 +20,6 @@ from repo_logging import RepoLogger
|
||||
|
||||
|
||||
class TestRepoLogger(unittest.TestCase):
|
||||
def test_error_logs_error(self):
|
||||
"""Test if error fn outputs logs."""
|
||||
logger = RepoLogger(__name__)
|
||||
result = None
|
||||
|
||||
def mock_handler(log):
|
||||
nonlocal result
|
||||
result = log.getMessage()
|
||||
|
||||
mock_out = mock.MagicMock()
|
||||
mock_out.level = 0
|
||||
mock_out.handle = mock_handler
|
||||
logger.addHandler(mock_out)
|
||||
|
||||
logger.error("We're no strangers to love")
|
||||
|
||||
self.assertEqual(result, "We're no strangers to love")
|
||||
|
||||
def test_warning_logs_error(self):
|
||||
"""Test if warning fn outputs logs."""
|
||||
logger = RepoLogger(__name__)
|
||||
result = None
|
||||
|
||||
def mock_handler(log):
|
||||
nonlocal result
|
||||
result = log.getMessage()
|
||||
|
||||
mock_out = mock.MagicMock()
|
||||
mock_out.level = 0
|
||||
mock_out.handle = mock_handler
|
||||
logger.addHandler(mock_out)
|
||||
|
||||
logger.warning("You know the rules and so do I (do I)")
|
||||
|
||||
self.assertEqual(result, "You know the rules and so do I (do I)")
|
||||
|
||||
def test_log_aggregated_errors_logs_aggregated_errors(self):
|
||||
"""Test if log_aggregated_errors outputs aggregated errors."""
|
||||
logger = RepoLogger(__name__)
|
||||
|
Reference in New Issue
Block a user