Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

gh-71339: Use new assertion methods in test_enum #129128

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.

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

Closed
wants to merge 2 commits into from
Closed
Changes from 1 commit
Commits
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
Next Next commit
gh-71339: Use new assertion methods in test_enum
  • Loading branch information
srinivasreddy committed Jan 21, 2025
commit a4fe0256c01e0b83c67219d022d7488d69eeb460
44 changes: 22 additions & 22 deletions 44 Lib/test/test_enum.py
Original file line number Diff line number Diff line change
Expand Up @@ -433,9 +433,9 @@ class Season(self.enum_type):
def spam(cls):
pass
#
self.assertTrue(hasattr(Season, 'spam'))
self.assertHasAttr(Season, 'spam')
del Season.spam
self.assertFalse(hasattr(Season, 'spam'))
self.assertNotHasAttr(Season, 'spam')
#
with self.assertRaises(AttributeError):
del Season.SPRING
Expand Down Expand Up @@ -1415,7 +1415,7 @@ class Outer(Enum):
class Inner(Enum):
foo = 10
bar = 11
self.assertTrue(isinstance(Outer.Inner, Outer))
self.assertIsInstance(Outer.Inner, Outer)
self.assertEqual(Outer.a.value, 1)
self.assertEqual(Outer.Inner.value.foo.value, 10)
self.assertEqual(
Expand All @@ -1439,7 +1439,7 @@ class Outer(Enum):
class Inner(Enum):
foo = 10
bar = 11
self.assertTrue(isinstance(Outer.Inner, type))
self.assertIsInstance(Outer.Inner, type)
self.assertEqual(Outer.a.value, 1)
self.assertEqual(Outer.Inner.foo.value, 10)
self.assertEqual(
Expand All @@ -1459,7 +1459,7 @@ class Outer(Enum):
class Inner(Enum):
foo = 10
bar = 11
self.assertTrue(isinstance(Outer.Inner, type))
self.assertIsInstance(Outer.Inner, type)
self.assertEqual(Outer.a.value, 1)
self.assertEqual(Outer.Inner.foo.value, 10)
self.assertEqual(
Expand Down Expand Up @@ -1525,7 +1525,7 @@ class Outer(Enum):
class Inner(Enum):
foo = 10
bar = 11
self.assertTrue(isinstance(Outer.Inner, Outer))
self.assertIsInstance(Outer.Inner, Outer)
self.assertEqual(Outer.a.value, 1)
self.assertEqual(Outer.Inner.value.foo.value, 10)
self.assertEqual(
Expand Down Expand Up @@ -2639,10 +2639,10 @@ def __new__(cls, value, period):
OneDay = day_1
OneWeek = week_1
OneMonth = month_1
self.assertFalse(hasattr(Period, '_ignore_'))
self.assertFalse(hasattr(Period, 'Period'))
self.assertFalse(hasattr(Period, 'i'))
self.assertTrue(isinstance(Period.day_1, timedelta))
self.assertNotHasAttr(Period, '_ignore_')
self.assertNotHasAttr(Period, 'Period')
self.assertNotHasAttr(Period, 'i')
self.assertIsInstance(Period.day_1, timedelta)
self.assertTrue(Period.month_1 is Period.day_30)
self.assertTrue(Period.week_4 is Period.day_28)

Expand Down Expand Up @@ -2719,13 +2719,13 @@ def _missing_(cls, item):
try:
Color('bad return')
except TypeError as exc:
self.assertTrue(isinstance(exc.__context__, ValueError))
self.assertIsInstance(exc.__context__, ValueError)
else:
raise Exception('Exception not raised.')
try:
Color('error out')
except ZeroDivisionError as exc:
self.assertTrue(isinstance(exc.__context__, ValueError))
self.assertIsInstance(exc.__context__, ValueError)
else:
raise Exception('Exception not raised.')

Expand Down Expand Up @@ -2864,7 +2864,7 @@ class ReformedColor(StrMixin, IntEnum, SomeEnum, AnotherEnum):
self.assertEqual(str(ReformedColor.BLUE), 'blue')
self.assertEqual(ReformedColor.RED.behavior(), 'booyah')
self.assertEqual(ConfusedColor.RED.social(), "what's up?")
self.assertTrue(issubclass(ReformedColor, int))
self.assertIsSubclass(ReformedColor, int)

def test_multiple_inherited_mixin(self):
@unique
Expand Down Expand Up @@ -2894,7 +2894,7 @@ class MyIntEnum(HexMixin, MyInt, enum.Enum):

class Foo(MyIntEnum):
TEST = 1
self.assertTrue(isinstance(Foo.TEST, MyInt))
self.assertIsInstance(Foo.TEST, MyInt)
self.assertEqual(Foo._member_type_, MyInt)
self.assertEqual(repr(Foo.TEST), "0x1")

Expand Down Expand Up @@ -2948,7 +2948,7 @@ class JobStatus(CaseInsensitiveStrEnum, LenientStrEnum):
self.assertEqual(JS.ACTIVE.value, 'active')
self.assertIs(JS('Active'), JS.ACTIVE)
self.assertTrue(JS.ACTIVE.valid)
self.assertTrue(isinstance(missing, JS))
self.assertIsInstance(missing, JS)
self.assertFalse(missing.valid)

def test_empty_globals(self):
Expand Down Expand Up @@ -3169,7 +3169,7 @@ class Entries(Foo, Enum):
ENTRY1 = 1
self.assertEqual(repr(Entries.ENTRY1), '<Entries.ENTRY1: ha hah!>')
self.assertTrue(Entries.ENTRY1.value == Foo(1), Entries.ENTRY1.value)
self.assertTrue(isinstance(Entries.ENTRY1, Foo))
self.assertIsInstance(Entries.ENTRY1, Foo)
self.assertTrue(Entries._member_type_ is Foo, Entries._member_type_)
#
# check auto-generated dataclass __repr__ is not used
Expand Down Expand Up @@ -3256,8 +3256,8 @@ class MyIntFlag(int, Flag):
ONE = 1
TWO = 2
FOUR = 4
self.assertTrue(isinstance(MyIntFlag.ONE | MyIntFlag.TWO, MyIntFlag), MyIntFlag.ONE | MyIntFlag.TWO)
self.assertTrue(isinstance(MyIntFlag.ONE | 2, MyIntFlag))
self.assertIsInstance(MyIntFlag.ONE | MyIntFlag.TWO, MyIntFlag, MyIntFlag.ONE | MyIntFlag.TWO)
self.assertIsInstance(MyIntFlag.ONE | 2, MyIntFlag)

def test_int_flags_copy(self):
class MyIntFlag(IntFlag):
Expand Down Expand Up @@ -4068,14 +4068,14 @@ def test_type(self):
self.assertTrue(Perm._member_type_ is int)
Open = self.Open
for f in Perm:
self.assertTrue(isinstance(f, Perm))
self.assertIsInstance(f, Perm)
self.assertEqual(f, f.value)
self.assertTrue(isinstance(Perm.W | Perm.X, Perm))
self.assertIsInstance(Perm.W | Perm.X, Perm)
self.assertEqual(Perm.W | Perm.X, 3)
for f in Open:
self.assertTrue(isinstance(f, Open))
self.assertIsInstance(f, Open)
self.assertEqual(f, f.value)
self.assertTrue(isinstance(Open.WO | Open.RW, Open))
self.assertIsInstance(Open.WO | Open.RW, Open)
self.assertEqual(Open.WO | Open.RW, 3)

@reraise_if_not_enum(HeadlightsK)
Expand Down
Loading
Morty Proxy This is a proxified and sanitized view of the page, visit original site.