diff options
author | Ethan Furman <ethan@stoneleaf.us> | 2024-01-18 00:31:00 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-01-18 00:31:00 (GMT) |
commit | 4c7e09d0129dafddba58979ced9580f856f65efa (patch) | |
tree | 5c84b14e48f05019974558d35a8179a2432b9a35 | |
parent | 945540306c12116154d2e4cc6c17a8efd2290537 (diff) | |
download | cpython-4c7e09d0129dafddba58979ced9580f856f65efa.zip cpython-4c7e09d0129dafddba58979ced9580f856f65efa.tar.gz cpython-4c7e09d0129dafddba58979ced9580f856f65efa.tar.bz2 |
gh-114149: [Enum] revert #114160 and add more tuple-subclass tests (GH-114215)
This reverts commit 05e142b1543eb9662d6cc33722e7e16250c9219f.
-rw-r--r-- | Lib/enum.py | 6 | ||||
-rw-r--r-- | Lib/test/test_enum.py | 27 |
2 files changed, 24 insertions, 9 deletions
diff --git a/Lib/enum.py b/Lib/enum.py index e51850d..a8a50a5 100644 --- a/Lib/enum.py +++ b/Lib/enum.py @@ -250,7 +250,7 @@ class _proto_member: delattr(enum_class, member_name) # second step: create member based on enum_class value = self.value - if type(value) is not tuple: + if not isinstance(value, tuple): args = (value, ) else: args = value @@ -1777,7 +1777,7 @@ def _simple_enum(etype=Enum, *, boundary=None, use_args=None): else: # create the member if use_args: - if type(value) is not tuple: + if not isinstance(value, tuple): value = (value, ) member = new_member(enum_class, *value) value = value[0] @@ -1826,7 +1826,7 @@ def _simple_enum(etype=Enum, *, boundary=None, use_args=None): else: # create the member if use_args: - if type(value) is not tuple: + if not isinstance(value, tuple): value = (value, ) member = new_member(enum_class, *value) value = value[0] diff --git a/Lib/test/test_enum.py b/Lib/test/test_enum.py index 48e0b02..d045739 100644 --- a/Lib/test/test_enum.py +++ b/Lib/test/test_enum.py @@ -3201,19 +3201,34 @@ class TestSpecial(unittest.TestCase): [x.value for x in NTEnum], [TTuple(id=0, a=0, blist=[]), TTuple(id=1, a=2, blist=[4]), TTuple(id=2, a=4, blist=[0, 1, 2])], ) + + self.assertRaises(AttributeError, getattr, NTEnum.NONE, 'id') + # + class NTCEnum(TTuple, Enum): + NONE = 0, 0, [] + A = 1, 2, [4] + B = 2, 4, [0, 1, 2] + self.assertEqual(repr(NTCEnum.NONE), "<NTCEnum.NONE: TTuple(id=0, a=0, blist=[])>") + self.assertEqual(NTCEnum.NONE.value, TTuple(id=0, a=0, blist=[])) + self.assertEqual(NTCEnum.NONE.id, 0) + self.assertEqual(NTCEnum.A.a, 2) + self.assertEqual(NTCEnum.B.blist, [0, 1 ,2]) + self.assertEqual( + [x.value for x in NTCEnum], + [TTuple(id=0, a=0, blist=[]), TTuple(id=1, a=2, blist=[4]), TTuple(id=2, a=4, blist=[0, 1, 2])], + ) # class NTDEnum(Enum): - def __new__(cls, t_value): + def __new__(cls, id, a, blist): member = object.__new__(cls) - member._value_ = t_value[0] - member.id = t_value[0] - member.a = t_value[1] - member.blist = t_value[2] + member.id = id + member.a = a + member.blist = blist return member NONE = TTuple(0, 0, []) A = TTuple(1, 2, [4]) B = TTuple(2, 4, [0, 1, 2]) - self.assertEqual(repr(NTDEnum.NONE), "<NTDEnum.NONE: 0>") + self.assertEqual(repr(NTDEnum.NONE), "<NTDEnum.NONE: TTuple(id=0, a=0, blist=[])>") self.assertEqual(NTDEnum.NONE.id, 0) self.assertEqual(NTDEnum.A.a, 2) self.assertEqual(NTDEnum.B.blist, [0, 1 ,2]) |