Skip to content

Commit 4e8277f

Browse files
committed
improve test coverage
1 parent e1bc272 commit 4e8277f

File tree

1 file changed

+24
-18
lines changed

1 file changed

+24
-18
lines changed

Lib/test/test_math.py

Lines changed: 24 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -629,19 +629,22 @@ def test_fmax(self):
629629
self.assertRaises(TypeError, math.fmax, 'x', 'y')
630630

631631
self.assertEqual(math.fmax(0., 0.), 0.)
632+
self.assertEqual(math.fmax(1., 2.), 2.)
633+
self.assertEqual(math.fmax(2., 1.), 2.)
634+
632635
# fmax() does not need to be sensitive to the sign of 0 (§F.10.9.2).
633-
self.assertIn(math.fmax(0., -0.), {-0., 0.})
634-
self.assertIn(math.fmax(-0., 0.), {-0., 0.})
636+
self.assertEqual(math.fmax(+0., -0.), 0.)
637+
self.assertEqual(math.fmax(-0., +0.), 0.)
635638

636-
self.assertEqual(math.fmax(1., 0.), 1.)
637-
self.assertEqual(math.fmax(0., 1.), 1.)
638-
self.assertEqual(math.fmax(1., -0.), 1.)
639-
self.assertEqual(math.fmax(-0., 1.), 1.)
639+
self.assertEqual(math.fmax(+1., +0.), 1.)
640+
self.assertEqual(math.fmax(+0., +1.), 1.)
641+
self.assertEqual(math.fmax(+1., -0.), 1.)
642+
self.assertEqual(math.fmax(-0., +1.), 1.)
640643

641-
self.assertEqual(math.fmax(-1., 0.), 0.)
642-
self.assertEqual(math.fmax(0., -1.), 0.)
643-
self.assertEqual(math.fmax(-1., -0.), -0.)
644-
self.assertEqual(math.fmax(-0., -1.), -0.)
644+
self.assertEqual(math.fmax(-1., +0.), 0.)
645+
self.assertEqual(math.fmax(+0., -1.), 0.)
646+
self.assertEqual(math.fmax(-1., -0.), 0.)
647+
self.assertEqual(math.fmax(-0., -1.), 0.)
645648

646649
for x in [NINF, -1., -0., 0., 1., INF]:
647650
self.assertFalse(math.isnan(x))
@@ -673,17 +676,20 @@ def test_fmin(self):
673676
self.assertRaises(TypeError, math.fmin, 'x', 'y')
674677

675678
self.assertEqual(math.fmin(0., 0.), 0.)
679+
self.assertEqual(math.fmin(1., 2.), 1.)
680+
self.assertEqual(math.fmin(2., 1.), 1.)
681+
676682
# fmin() does not need to be sensitive to the sign of 0 (§F.10.9.3).
677-
self.assertIn(math.fmin(0., -0.), {-0., 0.})
678-
self.assertIn(math.fmin(-0., 0.), {-0., 0.})
683+
self.assertEqual(math.fmin(+0., -0.), 0.)
684+
self.assertEqual(math.fmin(-0., +0.), 0.)
679685

680-
self.assertEqual(math.fmin(1., 0.), 0.)
681-
self.assertEqual(math.fmin(0., 1.), 0.)
682-
self.assertEqual(math.fmin(1., -0.), -0.)
683-
self.assertEqual(math.fmin(-0., 1.), -0.)
686+
self.assertEqual(math.fmin(+1., +0.), 0.)
687+
self.assertEqual(math.fmin(+0., +1.), 0.)
688+
self.assertEqual(math.fmin(+1., -0.), 0.)
689+
self.assertEqual(math.fmin(-0., +1.), 0.)
684690

685-
self.assertEqual(math.fmin(-1., 0.), -1.)
686-
self.assertEqual(math.fmin(0., -1.), -1.)
691+
self.assertEqual(math.fmin(-1., +0.), -1.)
692+
self.assertEqual(math.fmin(+0., -1.), -1.)
687693
self.assertEqual(math.fmin(-1., -0.), -1.)
688694
self.assertEqual(math.fmin(-0., -1.), -1.)
689695

0 commit comments

Comments
 (0)