diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2017-09-27 04:38:03 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-09-27 04:38:03 (GMT) |
commit | 90fe25a051bd8cf64d52be533c9b60cad9bdd7fb (patch) | |
tree | 48e970eec5fd39ccd8f26e576fdeb3a1a0fae3f9 /Lib | |
parent | bdb215b18a42360b6a9c82876fa71f19ca1a416d (diff) | |
download | cpython-90fe25a051bd8cf64d52be533c9b60cad9bdd7fb.zip cpython-90fe25a051bd8cf64d52be533c9b60cad9bdd7fb.tar.gz cpython-90fe25a051bd8cf64d52be533c9b60cad9bdd7fb.tar.bz2 |
[3.6] bpo-31285: Fix an assertion failure and a SystemError in warnings.warn_explicit. (GH-3219) (#3775)
(cherry picked from commit 91fb0afe181986b48abfc6092dcca912b39de51d)
Diffstat (limited to 'Lib')
-rw-r--r-- | Lib/test/test_warnings/__init__.py | 36 |
1 files changed, 36 insertions, 0 deletions
diff --git a/Lib/test/test_warnings/__init__.py b/Lib/test/test_warnings/__init__.py index a07886c..ea81d11 100644 --- a/Lib/test/test_warnings/__init__.py +++ b/Lib/test/test_warnings/__init__.py @@ -794,6 +794,42 @@ class _WarningsTests(BaseTest, unittest.TestCase): self.assertNotIn(b'Warning!', stderr) self.assertNotIn(b'Error', stderr) + def test_issue31285(self): + # warn_explicit() should neither raise a SystemError nor cause an + # assertion failure, in case the return value of get_source() has a + # bad splitlines() method. + def get_bad_loader(splitlines_ret_val): + class BadLoader: + def get_source(self, fullname): + class BadSource(str): + def splitlines(self): + return splitlines_ret_val + return BadSource('spam') + return BadLoader() + + wmod = self.module + with original_warnings.catch_warnings(module=wmod): + wmod.filterwarnings('default', category=UserWarning) + + with support.captured_stderr() as stderr: + wmod.warn_explicit( + 'foo', UserWarning, 'bar', 1, + module_globals={'__loader__': get_bad_loader(42), + '__name__': 'foobar'}) + self.assertIn('UserWarning: foo', stderr.getvalue()) + + show = wmod._showwarnmsg + try: + del wmod._showwarnmsg + with support.captured_stderr() as stderr: + wmod.warn_explicit( + 'eggs', UserWarning, 'bar', 1, + module_globals={'__loader__': get_bad_loader([42]), + '__name__': 'foobar'}) + self.assertIn('UserWarning: eggs', stderr.getvalue()) + finally: + wmod._showwarnmsg = show + @support.cpython_only def test_issue31411(self): # warn_explicit() shouldn't raise a SystemError in case |