diff options
| author | Aravind Vasudevan <aravindvasudev@google.com> | 2023-09-06 17:25:58 +0000 |
|---|---|---|
| committer | LUCI <gerrit-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-09-06 18:07:55 +0000 |
| commit | 712e62b9b07f690abbb40e089a17f4ddec6ba952 (patch) | |
| tree | cac20eaeb631df384031f42797d33189de70465d /tests/test_repo_logging.py | |
| parent | daf2ad38eb62fb990fadc3fc872f303113ac4768 (diff) | |
| download | git-repo-712e62b9b07f690abbb40e089a17f4ddec6ba952.tar.gz | |
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>
Diffstat (limited to 'tests/test_repo_logging.py')
| -rw-r--r-- | tests/test_repo_logging.py | 36 |
1 files changed, 0 insertions, 36 deletions
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 | |||
| 20 | 20 | ||
| 21 | 21 | ||
| 22 | class TestRepoLogger(unittest.TestCase): | 22 | class TestRepoLogger(unittest.TestCase): |
| 23 | def test_error_logs_error(self): | ||
| 24 | """Test if error fn outputs logs.""" | ||
| 25 | logger = RepoLogger(__name__) | ||
| 26 | result = None | ||
| 27 | |||
| 28 | def mock_handler(log): | ||
| 29 | nonlocal result | ||
| 30 | result = log.getMessage() | ||
| 31 | |||
| 32 | mock_out = mock.MagicMock() | ||
| 33 | mock_out.level = 0 | ||
| 34 | mock_out.handle = mock_handler | ||
| 35 | logger.addHandler(mock_out) | ||
| 36 | |||
| 37 | logger.error("We're no strangers to love") | ||
| 38 | |||
| 39 | self.assertEqual(result, "We're no strangers to love") | ||
| 40 | |||
| 41 | def test_warning_logs_error(self): | ||
| 42 | """Test if warning fn outputs logs.""" | ||
| 43 | logger = RepoLogger(__name__) | ||
| 44 | result = None | ||
| 45 | |||
| 46 | def mock_handler(log): | ||
| 47 | nonlocal result | ||
| 48 | result = log.getMessage() | ||
| 49 | |||
| 50 | mock_out = mock.MagicMock() | ||
| 51 | mock_out.level = 0 | ||
| 52 | mock_out.handle = mock_handler | ||
| 53 | logger.addHandler(mock_out) | ||
| 54 | |||
| 55 | logger.warning("You know the rules and so do I (do I)") | ||
| 56 | |||
| 57 | self.assertEqual(result, "You know the rules and so do I (do I)") | ||
| 58 | |||
| 59 | def test_log_aggregated_errors_logs_aggregated_errors(self): | 23 | def test_log_aggregated_errors_logs_aggregated_errors(self): |
| 60 | """Test if log_aggregated_errors outputs aggregated errors.""" | 24 | """Test if log_aggregated_errors outputs aggregated errors.""" |
| 61 | logger = RepoLogger(__name__) | 25 | logger = RepoLogger(__name__) |
