Undo prefix 0 changes

dbader · Feb 14, 2021 · c0465f1 · c0465f1
1 parent bee0bef
commit c0465f1
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 10 deletions.
diff --git a/schedule/__init__.py b/schedule/__init__.py
@@ -449,7 +449,7 @@ def at(self, time_str):
         if not isinstance(time_str, str):
             raise TypeError("at() should be passed a string")
         if self.unit == "days" or self.start_day:
-            if not re.match(r"^[0-2]?\d:[0-5]\d(:[0-5]\d)?$", time_str):
+            if not re.match(r"^([0-2]\d:)?[0-5]\d:[0-5]\d$", time_str):
                 raise ScheduleValueError(
                     "Invalid time format for an daily job(valid format is H?H:MM(:SS)?)"
                 )

diff --git a/test_schedule.py b/test_schedule.py
@@ -170,25 +170,16 @@ def test_time_range_repr(self):
 
     def test_at_time(self):
         mock_job = make_mock_job()
-
         assert every().day.at("10:30").do(mock_job).next_run.hour == 10
         assert every().day.at("10:30").do(mock_job).next_run.minute == 30
         assert every().day.at("10:30:50").do(mock_job).next_run.second == 50
 
-        assert every().day.at("8:30").do(mock_job).next_run.hour == 8
-        assert every().day.at("8:30:50").do(mock_job).next_run.hour == 8
-        assert every().day.at("08:30").do(mock_job).next_run.hour == 8
-        assert every().day.at("08:30:50").do(mock_job).next_run.hour == 8
-
         self.assertRaises(ScheduleValueError, every().day.at, "2:30:000001")
         self.assertRaises(ScheduleValueError, every().day.at, "::2")
         self.assertRaises(ScheduleValueError, every().day.at, ".2")
         self.assertRaises(ScheduleValueError, every().day.at, "2")
         self.assertRaises(ScheduleValueError, every().day.at, ":2")
         self.assertRaises(ScheduleValueError, every().day.at, " 2:30:00")
-        self.assertRaises(ScheduleValueError, every().day.at, "25:30:00")
-        self.assertRaises(ScheduleValueError, every().day.at, "25:30")
-        self.assertRaises(ScheduleValueError, every().day.at, "25")
         self.assertRaises(ScheduleValueError, every().do, lambda: 0)
         self.assertRaises(TypeError, every().day.at, 2)