From cf6774a53b40243d35183b4300a9385b68fd8c75 Mon Sep 17 00:00:00 2001 From: abhiabhi94 <13880786+abhiabhi94@users.noreply.github.com> Date: Wed, 7 Jul 2021 12:51:12 +0530 Subject: Fixed #32904 -- Made parse_time() more strict. Thanks Keryn Knight for the report. --- tests/utils_tests/test_dateparse.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'tests/utils_tests/test_dateparse.py') diff --git a/tests/utils_tests/test_dateparse.py b/tests/utils_tests/test_dateparse.py index 9974b85f07..0880de2034 100644 --- a/tests/utils_tests/test_dateparse.py +++ b/tests/utils_tests/test_dateparse.py @@ -27,10 +27,12 @@ class DateParseTests(unittest.TestCase): self.assertEqual(parse_time('4:8:16'), time(4, 8, 16)) # Time zone offset is ignored. self.assertEqual(parse_time('00:05:23+04:00'), time(0, 5, 23)) - # These should be invalid, see #32904. - self.assertEqual(parse_time('00:05:'), time(0, 5)) - self.assertEqual(parse_time('4:18:101'), time(4, 18, 10)) # Invalid inputs + self.assertIsNone(parse_time('00:05:')) + self.assertIsNone(parse_time('00:05:23,')) + self.assertIsNone(parse_time('00:05:23+')) + self.assertIsNone(parse_time('00:05:23+25:00')) + self.assertIsNone(parse_time('4:18:101')) self.assertIsNone(parse_time('091500')) with self.assertRaises(ValueError): parse_time('09:15:90') -- cgit v1.3