From: Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> Date: Sun, 23 Mar 2025 16:51:26 +0000 (+0100) Subject: [3.13] gh-131045: [Enum] fix flag containment checks when using values (GH-131053... X-Git-Tag: v3.13.3~88 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b3dccc0ce2f53936fdae50498388679f2a408b3f;p=thirdparty%2FPython%2Fcpython.git [3.13] gh-131045: [Enum] fix flag containment checks when using values (GH-131053) (#131167) gh-131045: [Enum] fix flag containment checks when using values (GH-131053) Check would fail if value would create a pseudo-member, but that member had not yet been created. We now attempt to create a pseudo-member for a passed-in value first. (cherry picked from commit 17d06aeb5476099bc1acd89cd6f69e239e0f9350) Co-authored-by: Ethan Furman Co-authored-by: Tomas R. --- diff --git a/Lib/enum.py b/Lib/enum.py index 37f16976bbac..9cab804115e4 100644 --- a/Lib/enum.py +++ b/Lib/enum.py @@ -746,12 +746,14 @@ class EnumType(type): `value` is in `cls` if: 1) `value` is a member of `cls`, or 2) `value` is the value of one of the `cls`'s members. + 3) `value` is a pseudo-member (flags) """ if isinstance(value, cls): return True try: - return value in cls._value2member_map_ - except TypeError: + cls(value) + return True + except ValueError: return ( value in cls._unhashable_values_ # both structures are lists or value in cls._hashable_values_ diff --git a/Lib/test/test_enum.py b/Lib/test/test_enum.py index 11e95d5b88b8..a2eb81c1da55 100644 --- a/Lib/test/test_enum.py +++ b/Lib/test/test_enum.py @@ -463,6 +463,7 @@ class _EnumTests: self.assertEqual(str(TE), "") self.assertEqual(format(TE), "") self.assertTrue(TE(5) is self.dupe2) + self.assertTrue(7 in TE) else: self.assertEqual(repr(TE), "") self.assertEqual(str(TE), "") @@ -4968,6 +4969,7 @@ class Color(enum.Enum) | `value` is in `cls` if: | 1) `value` is a member of `cls`, or | 2) `value` is the value of one of the `cls`'s members. + | 3) `value` is a pseudo-member (flags) | | __getitem__(name) | Return the member matching `name`. diff --git a/Misc/NEWS.d/next/Library/2025-03-10-12-26-56.gh-issue-131045.s1TssJ.rst b/Misc/NEWS.d/next/Library/2025-03-10-12-26-56.gh-issue-131045.s1TssJ.rst new file mode 100644 index 000000000000..b6aa07276bb5 --- /dev/null +++ b/Misc/NEWS.d/next/Library/2025-03-10-12-26-56.gh-issue-131045.s1TssJ.rst @@ -0,0 +1 @@ +Fix issue with ``__contains__``, values, and pseudo-members for :class:`enum.Flag`.