From 712e62b9b07f690abbb40e089a17f4ddec6ba952 Mon Sep 17 00:00:00 2001 From: Aravind Vasudevan Date: Wed, 6 Sep 2023 17:25:58 +0000 Subject: 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 Commit-Queue: Aravind Vasudevan Reviewed-by: Mike Frysinger --- tests/test_repo_logging.py | 36 ------------------------------------ 1 file changed, 36 deletions(-) (limited to 'tests/test_repo_logging.py') diff --git a/tests/test_repo_logging.py b/tests/test_repo_logging.py index b51e6270..52f251a7 100644 --- a/tests/test_repo_logging.py +++ b/tests/test_repo_logging.py @@ -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__) -- cgit v1.2.3-54-g00ecf