summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMario Šaško <mariosasko777@gmail.com>2024-10-22 20:42:22 (GMT)
committerGitHub <noreply@github.com>2024-10-22 20:42:22 (GMT)
commit34653bba644aa5481613f398153757d7357e39ea (patch)
tree837d85c45083e45145b2fc885a954856f7e8bda5
parentaaed91cabcedc16c089c4b1c9abb1114659a83d3 (diff)
downloadcpython-34653bba644aa5481613f398153757d7357e39ea.zip
cpython-34653bba644aa5481613f398153757d7357e39ea.tar.gz
cpython-34653bba644aa5481613f398153757d7357e39ea.tar.bz2
gh-125259: Fix error notes removal in enum initialization (GH-125647)
-rw-r--r--Lib/enum.py16
-rw-r--r--Lib/test/test_enum.py19
-rw-r--r--Misc/NEWS.d/next/Library/2024-10-17-16-10-29.gh-issue-125259.oMew0c.rst1
3 files changed, 25 insertions, 11 deletions
diff --git a/Lib/enum.py b/Lib/enum.py
index 4f99122..27be3fb 100644
--- a/Lib/enum.py
+++ b/Lib/enum.py
@@ -557,22 +557,16 @@ class EnumType(type):
classdict['_all_bits_'] = 0
classdict['_inverted_'] = None
try:
- exc = None
classdict['_%s__in_progress' % cls] = True
enum_class = super().__new__(metacls, cls, bases, classdict, **kwds)
classdict['_%s__in_progress' % cls] = False
delattr(enum_class, '_%s__in_progress' % cls)
except Exception as e:
- # since 3.12 the line "Error calling __set_name__ on '_proto_member' instance ..."
- # is tacked on to the error instead of raising a RuntimeError
- # recreate the exception to discard
- exc = type(e)(str(e))
- exc.__cause__ = e.__cause__
- exc.__context__ = e.__context__
- tb = e.__traceback__
- if exc is not None:
- raise exc.with_traceback(tb)
- #
+ # since 3.12 the note "Error calling __set_name__ on '_proto_member' instance ..."
+ # is tacked on to the error instead of raising a RuntimeError, so discard it
+ if hasattr(e, '__notes__'):
+ del e.__notes__
+ raise
# update classdict with any changes made by __init_subclass__
classdict.update(enum_class.__dict__)
#
diff --git a/Lib/test/test_enum.py b/Lib/test/test_enum.py
index 7184769..b9e13fb 100644
--- a/Lib/test/test_enum.py
+++ b/Lib/test/test_enum.py
@@ -1888,6 +1888,25 @@ class TestSpecial(unittest.TestCase):
class Wrong(Enum, str):
NotHere = 'error before this point'
+ def test_raise_custom_error_on_creation(self):
+ class InvalidRgbColorError(ValueError):
+ def __init__(self, r, g, b):
+ self.r = r
+ self.g = g
+ self.b = b
+ super().__init__(f'({r}, {g}, {b}) is not a valid RGB color')
+
+ with self.assertRaises(InvalidRgbColorError):
+ class RgbColor(Enum):
+ RED = (255, 0, 0)
+ GREEN = (0, 255, 0)
+ BLUE = (0, 0, 255)
+ INVALID = (256, 0, 0)
+
+ def __init__(self, r, g, b):
+ if not all(0 <= val <= 255 for val in (r, g, b)):
+ raise InvalidRgbColorError(r, g, b)
+
def test_intenum_transitivity(self):
class number(IntEnum):
one = 1
diff --git a/Misc/NEWS.d/next/Library/2024-10-17-16-10-29.gh-issue-125259.oMew0c.rst b/Misc/NEWS.d/next/Library/2024-10-17-16-10-29.gh-issue-125259.oMew0c.rst
new file mode 100644
index 0000000..4fa6330
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2024-10-17-16-10-29.gh-issue-125259.oMew0c.rst
@@ -0,0 +1 @@
+Fix the notes removal logic for errors thrown in enum initialization.