diff options
Diffstat (limited to 'tests/test_repo_logging.py')
| -rw-r--r-- | tests/test_repo_logging.py | 31 |
1 files changed, 7 insertions, 24 deletions
diff --git a/tests/test_repo_logging.py b/tests/test_repo_logging.py index 5402a90b..b51e6270 100644 --- a/tests/test_repo_logging.py +++ b/tests/test_repo_logging.py | |||
| @@ -23,7 +23,6 @@ class TestRepoLogger(unittest.TestCase): | |||
| 23 | def test_error_logs_error(self): | 23 | def test_error_logs_error(self): |
| 24 | """Test if error fn outputs logs.""" | 24 | """Test if error fn outputs logs.""" |
| 25 | logger = RepoLogger(__name__) | 25 | logger = RepoLogger(__name__) |
| 26 | RepoLogger.errors[:] = [] | ||
| 27 | result = None | 26 | result = None |
| 28 | 27 | ||
| 29 | def mock_handler(log): | 28 | def mock_handler(log): |
| @@ -42,7 +41,6 @@ class TestRepoLogger(unittest.TestCase): | |||
| 42 | def test_warning_logs_error(self): | 41 | def test_warning_logs_error(self): |
| 43 | """Test if warning fn outputs logs.""" | 42 | """Test if warning fn outputs logs.""" |
| 44 | logger = RepoLogger(__name__) | 43 | logger = RepoLogger(__name__) |
| 45 | RepoLogger.errors[:] = [] | ||
| 46 | result = None | 44 | result = None |
| 47 | 45 | ||
| 48 | def mock_handler(log): | 46 | def mock_handler(log): |
| @@ -58,30 +56,9 @@ class TestRepoLogger(unittest.TestCase): | |||
| 58 | 56 | ||
| 59 | self.assertEqual(result, "You know the rules and so do I (do I)") | 57 | self.assertEqual(result, "You know the rules and so do I (do I)") |
| 60 | 58 | ||
| 61 | def test_error_aggregates_error_msg(self): | ||
| 62 | """Test if error fn aggregates error logs.""" | ||
| 63 | logger = RepoLogger(__name__) | ||
| 64 | RepoLogger.errors[:] = [] | ||
| 65 | |||
| 66 | logger.error("A full commitment's what I'm thinking of") | ||
| 67 | logger.error("You wouldn't get this from any other guy") | ||
| 68 | logger.error("I just wanna tell you how I'm feeling") | ||
| 69 | logger.error("Gotta make you understand") | ||
| 70 | |||
| 71 | self.assertEqual( | ||
| 72 | RepoLogger.errors[:], | ||
| 73 | [ | ||
| 74 | "A full commitment's what I'm thinking of", | ||
| 75 | "You wouldn't get this from any other guy", | ||
| 76 | "I just wanna tell you how I'm feeling", | ||
| 77 | "Gotta make you understand", | ||
| 78 | ], | ||
| 79 | ) | ||
| 80 | |||
| 81 | def test_log_aggregated_errors_logs_aggregated_errors(self): | 59 | def test_log_aggregated_errors_logs_aggregated_errors(self): |
| 82 | """Test if log_aggregated_errors outputs aggregated errors.""" | 60 | """Test if log_aggregated_errors outputs aggregated errors.""" |
| 83 | logger = RepoLogger(__name__) | 61 | logger = RepoLogger(__name__) |
| 84 | RepoLogger.errors[:] = [] | ||
| 85 | result = [] | 62 | result = [] |
| 86 | 63 | ||
| 87 | def mock_handler(log): | 64 | def mock_handler(log): |
| @@ -96,7 +73,13 @@ class TestRepoLogger(unittest.TestCase): | |||
| 96 | logger.error("Never gonna give you up") | 73 | logger.error("Never gonna give you up") |
| 97 | logger.error("Never gonna let you down") | 74 | logger.error("Never gonna let you down") |
| 98 | logger.error("Never gonna run around and desert you") | 75 | logger.error("Never gonna run around and desert you") |
| 99 | logger.log_aggregated_errors() | 76 | logger.log_aggregated_errors( |
| 77 | [ | ||
| 78 | "Never gonna give you up", | ||
| 79 | "Never gonna let you down", | ||
| 80 | "Never gonna run around and desert you", | ||
| 81 | ] | ||
| 82 | ) | ||
| 100 | 83 | ||
| 101 | self.assertEqual( | 84 | self.assertEqual( |
| 102 | result, | 85 | result, |
