diff options
author | Ethan Furman <ethan@stoneleaf.us> | 2018-10-05 23:29:36 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-10-05 23:29:36 -0700 |
commit | cd45385ffad8910293e5659cfe7ab036e70613b7 (patch) | |
tree | 844790c5a632c9791f744110106765476516df71 | |
parent | 92878829c31ab2fc71c60555ce87a5f6cbc876f0 (diff) | |
download | cpython-git-cd45385ffad8910293e5659cfe7ab036e70613b7.tar.gz |
bpo-34909: keep searching mixins until base class is found (GH-9737)
-rw-r--r-- | Lib/enum.py | 4 | ||||
-rw-r--r-- | Lib/test/test_enum.py | 21 |
2 files changed, 22 insertions, 3 deletions
diff --git a/Lib/enum.py b/Lib/enum.py index 0ccb30d428..fec1aed9b2 100644 --- a/Lib/enum.py +++ b/Lib/enum.py @@ -486,7 +486,7 @@ class EnumMeta(type): if base is object: continue elif '__new__' in base.__dict__: - if issubclass(base, Enum) and not hasattr(base, '__new_member__'): + if issubclass(base, Enum): continue return base @@ -499,7 +499,6 @@ class EnumMeta(type): member_type = _find_data_type(bases) or object if first_enum._member_names_: raise TypeError("Cannot extend enumerations") - return member_type, first_enum @staticmethod @@ -545,7 +544,6 @@ class EnumMeta(type): use_args = False else: use_args = True - return __new__, save_new, use_args diff --git a/Lib/test/test_enum.py b/Lib/test/test_enum.py index aadc11fcc4..216f7d5c28 100644 --- a/Lib/test/test_enum.py +++ b/Lib/test/test_enum.py @@ -1842,6 +1842,27 @@ class TestEnum(unittest.TestCase): self.assertEqual(ConfusedColor.RED.social(), "what's up?") self.assertTrue(issubclass(ReformedColor, int)) + def test_multiple_inherited_mixin(self): + class StrEnum(str, Enum): + def __new__(cls, *args, **kwargs): + for a in args: + if not isinstance(a, str): + raise TypeError("Enumeration '%s' (%s) is not" + " a string" % (a, type(a).__name__)) + return str.__new__(cls, *args, **kwargs) + @unique + class Decision1(StrEnum): + REVERT = "REVERT" + REVERT_ALL = "REVERT_ALL" + RETRY = "RETRY" + class MyEnum(StrEnum): + pass + @unique + class Decision2(MyEnum): + REVERT = "REVERT" + REVERT_ALL = "REVERT_ALL" + RETRY = "RETRY" + class TestOrder(unittest.TestCase): |