Skip to content

Commit 27d1443

Browse files
Use assert* comparison methods in test_timer_comparison (#129513)
1 parent a6cf827 commit 27d1443

File tree

1 file changed

+54
-35
lines changed

1 file changed

+54
-35
lines changed

Lib/test/test_asyncio/test_events.py

Lines changed: 54 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -2652,33 +2652,46 @@ def callback(*args):
26522652

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

26672672
h2.cancel()
2668-
self.assertFalse(h1 == h2)
2673+
with self.assertRaises(AssertionError):
2674+
self.assertEqual(h1, h2)
2675+
self.assertNotEqual(h1, h2)
26692676

26702677
h1 = asyncio.TimerHandle(when, callback, (), self.loop)
26712678
h2 = asyncio.TimerHandle(when + 10.0, callback, (), self.loop)
2672-
self.assertTrue(h1 < h2)
2673-
self.assertFalse(h2 < h1)
2674-
self.assertTrue(h1 <= h2)
2675-
self.assertFalse(h2 <= h1)
2676-
self.assertFalse(h1 > h2)
2677-
self.assertTrue(h2 > h1)
2678-
self.assertFalse(h1 >= h2)
2679-
self.assertTrue(h2 >= h1)
2680-
self.assertFalse(h1 == h2)
2681-
self.assertTrue(h1 != h2)
2679+
with self.assertRaises(AssertionError):
2680+
self.assertLess(h2, h1)
2681+
with self.assertRaises(AssertionError):
2682+
self.assertLessEqual(h2, h1)
2683+
with self.assertRaises(AssertionError):
2684+
self.assertGreater(h1, h2)
2685+
with self.assertRaises(AssertionError):
2686+
self.assertGreaterEqual(h1, h2)
2687+
with self.assertRaises(AssertionError):
2688+
self.assertEqual(h1, h2)
2689+
2690+
self.assertLess(h1, h2)
2691+
self.assertGreater(h2, h1)
2692+
self.assertLessEqual(h1, h2)
2693+
self.assertGreaterEqual(h2, h1)
2694+
self.assertNotEqual(h1, h2)
26822695

26832696
h3 = asyncio.Handle(callback, (), self.loop)
26842697
self.assertIs(NotImplemented, h1.__eq__(h3))
@@ -2692,19 +2705,25 @@ def callback(*args):
26922705
h1 <= ()
26932706
with self.assertRaises(TypeError):
26942707
h1 >= ()
2695-
self.assertFalse(h1 == ())
2696-
self.assertTrue(h1 != ())
2697-
2698-
self.assertTrue(h1 == ALWAYS_EQ)
2699-
self.assertFalse(h1 != ALWAYS_EQ)
2700-
self.assertTrue(h1 < LARGEST)
2701-
self.assertFalse(h1 > LARGEST)
2702-
self.assertTrue(h1 <= LARGEST)
2703-
self.assertFalse(h1 >= LARGEST)
2704-
self.assertFalse(h1 < SMALLEST)
2705-
self.assertTrue(h1 > SMALLEST)
2706-
self.assertFalse(h1 <= SMALLEST)
2707-
self.assertTrue(h1 >= SMALLEST)
2708+
with self.assertRaises(AssertionError):
2709+
self.assertEqual(h1, ())
2710+
with self.assertRaises(AssertionError):
2711+
self.assertNotEqual(h1, ALWAYS_EQ)
2712+
with self.assertRaises(AssertionError):
2713+
self.assertGreater(h1, LARGEST)
2714+
with self.assertRaises(AssertionError):
2715+
self.assertGreaterEqual(h1, LARGEST)
2716+
with self.assertRaises(AssertionError):
2717+
self.assertLess(h1, SMALLEST)
2718+
with self.assertRaises(AssertionError):
2719+
self.assertLessEqual(h1, SMALLEST)
2720+
2721+
self.assertNotEqual(h1, ())
2722+
self.assertEqual(h1, ALWAYS_EQ)
2723+
self.assertLess(h1, LARGEST)
2724+
self.assertLessEqual(h1, LARGEST)
2725+
self.assertGreaterEqual(h1, SMALLEST)
2726+
self.assertGreater(h1, SMALLEST)
27082727

27092728

27102729
class AbstractEventLoopTests(unittest.TestCase):

0 commit comments

Comments
 (0)