Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 42a2eec

Browse files
committedMar 20, 2025·
Use assert* comparison methods in test_timer_comparison
1 parent e3eba8c commit 42a2eec

File tree

1 file changed

+40
-36
lines changed

1 file changed

+40
-36
lines changed
 

‎Lib/test/test_asyncio/test_events.py

+40-36
Original file line numberDiff line numberDiff line change
@@ -2653,33 +2653,36 @@ def callback(*args):
26532653

26542654
h1 = asyncio.TimerHandle(when, callback, (), self.loop)
26552655
h2 = asyncio.TimerHandle(when, callback, (), self.loop)
2656-
# TODO: Use assertLess etc.
2657-
self.assertFalse(h1 < h2)
2658-
self.assertFalse(h2 < h1)
2659-
self.assertTrue(h1 <= h2)
2660-
self.assertTrue(h2 <= h1)
2661-
self.assertFalse(h1 > h2)
2662-
self.assertFalse(h2 > h1)
2663-
self.assertTrue(h1 >= h2)
2664-
self.assertTrue(h2 >= h1)
2665-
self.assertTrue(h1 == h2)
2666-
self.assertFalse(h1 != h2)
2656+
with self.assertRaises(AssertionError):
2657+
self.assertLess(h1, h2)
2658+
self.assertLess(h2, h1)
2659+
self.assertGreater(h1, h2)
2660+
self.assertGreater(h2, h1)
2661+
self.assertNotEqual(h1, h2)
2662+
2663+
self.assertLessEqual(h1, h2)
2664+
self.assertLessEqual(h2, h1)
2665+
self.assertGreaterEqual(h1, h2)
2666+
self.assertGreaterEqual(h2, h1)
2667+
self.assertEqual(h1, h2)
26672668

26682669
h2.cancel()
2669-
self.assertFalse(h1 == h2)
2670+
self.assertNotEqual(h1, h2)
26702671

26712672
h1 = asyncio.TimerHandle(when, callback, (), self.loop)
26722673
h2 = asyncio.TimerHandle(when + 10.0, callback, (), self.loop)
2673-
self.assertTrue(h1 < h2)
2674-
self.assertFalse(h2 < h1)
2675-
self.assertTrue(h1 <= h2)
2676-
self.assertFalse(h2 <= h1)
2677-
self.assertFalse(h1 > h2)
2678-
self.assertTrue(h2 > h1)
2679-
self.assertFalse(h1 >= h2)
2680-
self.assertTrue(h2 >= h1)
2681-
self.assertFalse(h1 == h2)
2682-
self.assertTrue(h1 != h2)
2674+
with self.assertRaises(AssertionError):
2675+
self.assertLess(h2, h1)
2676+
self.assertLessEqual(h1, h2)
2677+
self.assertGreater(h1, h2)
2678+
self.assertGreaterEqual(h1, h2)
2679+
self.assertEqual(h1, h2)
2680+
2681+
self.assertLess(h1, h2)
2682+
self.assertGreater(h2, h1)
2683+
self.assertLessEqual(h1, h2)
2684+
self.assertGreaterEqual(h2, h1)
2685+
self.assertNotEqual(h1, h2)
26832686

26842687
h3 = asyncio.Handle(callback, (), self.loop)
26852688
self.assertIs(NotImplemented, h1.__eq__(h3))
@@ -2692,20 +2695,21 @@ def callback(*args):
26922695
with self.assertRaises(TypeError):
26932696
h1 <= ()
26942697
with self.assertRaises(TypeError):
2695-
h1 >= ()
2696-
self.assertFalse(h1 == ())
2697-
self.assertTrue(h1 != ())
2698-
2699-
self.assertTrue(h1 == ALWAYS_EQ)
2700-
self.assertFalse(h1 != ALWAYS_EQ)
2701-
self.assertTrue(h1 < LARGEST)
2702-
self.assertFalse(h1 > LARGEST)
2703-
self.assertTrue(h1 <= LARGEST)
2704-
self.assertFalse(h1 >= LARGEST)
2705-
self.assertFalse(h1 < SMALLEST)
2706-
self.assertTrue(h1 > SMALLEST)
2707-
self.assertFalse(h1 <= SMALLEST)
2708-
self.assertTrue(h1 >= SMALLEST)
2698+
h1 >= ()
2699+
with self.assertRaises(AssertionError):
2700+
self.assertEqual(h1, ())
2701+
self.assertNotEqual(h1, ALWAYS_EQ)
2702+
self.assertGreater(h1, LARGEST)
2703+
self.assertGreaterEqual(h1, LARGEST)
2704+
self.assertLess(h1, SMALLEST)
2705+
self.assertLessEqual(h1, SMALLEST)
2706+
2707+
self.assertNotEqual(h1, ())
2708+
self.assertEqual(h1, ALWAYS_EQ)
2709+
self.assertLess(h1, LARGEST)
2710+
self.assertLessEqual(h1, LARGEST)
2711+
self.assertGreaterEqual(h1, SMALLEST)
2712+
self.assertGreater(h1, SMALLEST)
27092713

27102714

27112715
class AbstractEventLoopTests(unittest.TestCase):

0 commit comments

Comments
 (0)
Please sign in to comment.