diff options
author | Barry Warsaw <barry@python.org> | 2016-06-08 21:47:26 (GMT) |
---|---|---|
committer | Barry Warsaw <barry@python.org> | 2016-06-08 21:47:26 (GMT) |
commit | 480e28538d42b9cf8e8b2163bfa9e23e27b80793 (patch) | |
tree | bd028ce2b5a97755fc3aec27208fc0b2067ebe1a /Lib | |
parent | fda4d67842a406d2fa285f929ddf890c2393e1c2 (diff) | |
download | cpython-480e28538d42b9cf8e8b2163bfa9e23e27b80793.zip cpython-480e28538d42b9cf8e8b2163bfa9e23e27b80793.tar.gz cpython-480e28538d42b9cf8e8b2163bfa9e23e27b80793.tar.bz2 |
Issue #27066: Fixed SystemError if a custom opener (for open()) returns
a negative number without setting an exception.
Diffstat (limited to 'Lib')
-rw-r--r-- | Lib/test/test_io.py | 16 |
1 files changed, 16 insertions, 0 deletions
diff --git a/Lib/test/test_io.py b/Lib/test/test_io.py index 7323198..000b525 100644 --- a/Lib/test/test_io.py +++ b/Lib/test/test_io.py @@ -805,6 +805,22 @@ class IOTest(unittest.TestCase): with self.open("non-existent", "r", opener=opener) as f: self.assertEqual(f.read(), "egg\n") + def test_bad_opener_negative_1(self): + # Issue #27066. + def badopener(fname, flags): + return -1 + with self.assertRaises(ValueError) as cm: + open('non-existent', 'r', opener=badopener) + self.assertEqual(str(cm.exception), 'opener returned -1') + + def test_bad_opener_other_negative(self): + # Issue #27066. + def badopener(fname, flags): + return -2 + with self.assertRaises(ValueError) as cm: + open('non-existent', 'r', opener=badopener) + self.assertEqual(str(cm.exception), 'opener returned -2') + def test_fileio_closefd(self): # Issue #4841 with self.open(__file__, 'rb') as f1, \ |