Skip to content

Add BaseException.__reduce__ #3665

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
youknowone merged 6 commits into from
Apr 29, 2022
Merged

Add BaseException.__reduce__ #3665

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
d516afb
Add BaseException.__reduce__
fanninpm Apr 28, 2022
2815f3d
Unmark passing test
fanninpm Apr 28, 2022
9d898de
Fix Clippy warning
fanninpm Apr 28, 2022
d51593d
Run rustfmt
fanninpm Apr 28, 2022
e2af440
Add case where exception dict is empty or missing
fanninpm Apr 28, 2022
dfbd8dc
Unmark passing tests
fanninpm Apr 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Unmark passing tests
  • Loading branch information
@fanninpm
fanninpm committed Apr 28, 2022
commit dfbd8dc8db50d7a31aec10c1472a2cff14a3d30b
22 changes: 0 additions & 22 deletions Lib/test/test_configparser.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1703,8 +1703,6 @@ def test_inconsistent_converters_state(self):
class ExceptionPicklingTestCase(unittest.TestCase):
"""Tests for issue #13760: ConfigParser exceptions are not picklable."""

# TODO: RUSTPYTHON Exception.__reduce__ missing.
@unittest.expectedFailure
def test_error(self):
import pickle
e1 = configparser.Error('value')
Expand All @@ -1714,8 +1712,6 @@ def test_error(self):
self.assertEqual(e1.message, e2.message)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_nosectionerror(self):
import pickle
e1 = configparser.NoSectionError('section')
Expand All @@ -1727,8 +1723,6 @@ def test_nosectionerror(self):
self.assertEqual(e1.section, e2.section)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_nooptionerror(self):
import pickle
e1 = configparser.NoOptionError('option', 'section')
Expand All @@ -1741,8 +1735,6 @@ def test_nooptionerror(self):
self.assertEqual(e1.option, e2.option)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_duplicatesectionerror(self):
import pickle
e1 = configparser.DuplicateSectionError('section', 'source', 123)
Expand All @@ -1756,8 +1748,6 @@ def test_duplicatesectionerror(self):
self.assertEqual(e1.lineno, e2.lineno)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_duplicateoptionerror(self):
import pickle
e1 = configparser.DuplicateOptionError('section', 'option', 'source',
Expand All @@ -1773,8 +1763,6 @@ def test_duplicateoptionerror(self):
self.assertEqual(e1.lineno, e2.lineno)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_interpolationerror(self):
import pickle
e1 = configparser.InterpolationError('option', 'section', 'msg')
Expand All @@ -1787,8 +1775,6 @@ def test_interpolationerror(self):
self.assertEqual(e1.option, e2.option)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_interpolationmissingoptionerror(self):
import pickle
e1 = configparser.InterpolationMissingOptionError('option', 'section',
Expand All @@ -1803,8 +1789,6 @@ def test_interpolationmissingoptionerror(self):
self.assertEqual(e1.reference, e2.reference)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_interpolationsyntaxerror(self):
import pickle
e1 = configparser.InterpolationSyntaxError('option', 'section', 'msg')
Expand All @@ -1817,8 +1801,6 @@ def test_interpolationsyntaxerror(self):
self.assertEqual(e1.option, e2.option)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_interpolationdeptherror(self):
import pickle
e1 = configparser.InterpolationDepthError('option', 'section',
Expand All @@ -1832,8 +1814,6 @@ def test_interpolationdeptherror(self):
self.assertEqual(e1.option, e2.option)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_parsingerror(self):
import pickle
e1 = configparser.ParsingError('source')
Expand Down Expand Up @@ -1861,8 +1841,6 @@ def test_parsingerror(self):
self.assertEqual(e1.errors, e2.errors)
self.assertEqual(repr(e1), repr(e2))

# TODO: RUSTPYTHON
@unittest.expectedFailure
def test_missingsectionheadererror(self):
import pickle
e1 = configparser.MissingSectionHeaderError('filename', 123, 'line')
Expand Down
2 changes: 0 additions & 2 deletions Lib/unittest/test/test_discovery.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -499,8 +499,6 @@ def test_discover_with_modules_that_fail_to_import(self):
with self.assertRaises(ImportError):
test.test_this_does_not_exist()

# TODO: RUSTPYTHON ImportError.__reduce__ missing
@unittest.expectedFailure
def test_discover_with_init_modules_that_fail_to_import(self):
vfs = {abspath('/foo'): ['my_package'],
abspath('/foo/my_package'): ['__init__.py', 'test_module.py']}
Expand Down