diff options
| author |   Łukasz Langa <lukasz@langa.pl> | 2017-10-18 17:28:51 -0700 |
|---|---|---|
| committer |   GitHub <noreply@github.com> | 2017-10-18 17:28:51 -0700 |
| commit | 0b6a118a45ac2eded1348fea6ed300d5651f7471 (patch) | |
| tree | c4669de7f6be919053a564411ff6b8f559eb1fa9 /Lib | |
| parent | Update TestCase.assertAlmostEqual and assertNotAlmostEqual docstrings. (GH-3998) (diff) | |
| download | cpython-0b6a118a45ac2eded1348fea6ed300d5651f7471.tar.gz cpython-0b6a118a45ac2eded1348fea6ed300d5651f7471.tar.bz2 cpython-0b6a118a45ac2eded1348fea6ed300d5651f7471.zip | |
bpo-31457: Make the `LoggerAdapter.manager` property settable (#4042)
Due to a bug in the initial fix, the setter was in fact creating a different
property. This is now fixed.
Diffstat (limited to 'Lib')
| -rw-r--r-- | Lib/logging/__init__.py | 2 | ||||
| -rw-r--r-- | Lib/test/test_logging.py | 15 |
2 files changed, 15 insertions, 2 deletions
diff --git a/Lib/logging/__init__.py b/Lib/logging/__init__.py index b941eab97d6..75441906608 100644 --- a/Lib/logging/__init__.py +++ b/Lib/logging/__init__.py @@ -1757,7 +1757,7 @@ class LoggerAdapter(object): return self.logger.manager @manager.setter - def set_manager(self, value): + def manager(self, value): self.logger.manager = value def __repr__(self): diff --git a/Lib/test/test_logging.py b/Lib/test/test_logging.py index fb0de9de046..583394cc73f 100644 --- a/Lib/test/test_logging.py +++ b/Lib/test/test_logging.py @@ -3982,12 +3982,25 @@ class LoggerAdapterTest(unittest.TestCase): msg = 'Adapters can be nested, yo.' adapter_adapter = logging.LoggerAdapter(logger=self.adapter, extra=None) adapter_adapter.log(logging.CRITICAL, msg, self.recording) - self.assertEqual(len(self.recording.records), 1) record = self.recording.records[0] self.assertEqual(record.levelno, logging.CRITICAL) self.assertEqual(record.msg, msg) self.assertEqual(record.args, (self.recording,)) + orig_manager = adapter_adapter.manager + self.assertIs(self.adapter.manager, orig_manager) + self.assertIs(self.logger.manager, orig_manager) + temp_manager = object() + try: + adapter_adapter.manager = temp_manager + self.assertIs(adapter_adapter.manager, temp_manager) + self.assertIs(self.adapter.manager, temp_manager) + self.assertIs(self.logger.manager, temp_manager) + finally: + adapter_adapter.manager = orig_manager + self.assertIs(adapter_adapter.manager, orig_manager) + self.assertIs(self.adapter.manager, orig_manager) + self.assertIs(self.logger.manager, orig_manager) class LoggerTest(BaseTest): |