Add docs and tests for time.strptime

Author: StanFromIreland

Doc/library/datetime.rst

@@ -2432,7 +2432,8 @@ Class attributes:
 
 :class:`date`, :class:`.datetime`, and :class:`.time` objects all support a
 ``strftime(format)`` method, to create a string representing the time under the
-control of an explicit format string. The methods only accept ASCII digits.
+control of an explicit format string. A :exc:`ValueError` will be raised if digits
+are not ASCII.
 
 Conversely, the :meth:`date.strptime`, :meth:`datetime.strptime` and
 :meth:`time.strptime` class methods create an object from a string
@@ -2612,7 +2613,7 @@ differences between platforms in handling of unsupported format specifiers.
    ``%:z`` was added.
 
 .. versionchanged:: next
-   Digits must be ASCII
+   Non-ASCII digits are now rejected by ``strptime`` for numerical directives.
 
 Technical Detail
 ^^^^^^^^^^^^^^^^

Doc/library/time.rst

@@ -594,9 +594,10 @@ Functions
    :func:`strftime`; it defaults to ``"%a %b %d %H:%M:%S %Y"`` which matches the
    formatting returned by :func:`ctime`. If *string* cannot be parsed according
    to *format*, or if it has excess data after parsing, :exc:`ValueError` is
-   raised. The default values used to fill in any missing data when more
-   accurate values cannot be inferred are ``(1900, 1, 1, 0, 0, 0, 0, 1, -1)``.
-   Both *string* and *format* must be strings.
+   raised. :exc:`ValueError` is raised if digits are not ASCII. The default
+   values used to fill in any missing data when more accurate values cannot be
+   inferred are ``(1900, 1, 1, 0, 0, 0, 0, 1, -1)``. Both *string* and *format*
+   must be strings.
 
    For example:
 
@@ -616,6 +617,9 @@ Functions
    and thus does not necessarily support all directives available that are not
    documented as supported.
 
+.. versionchanged:: next
+   Non-ASCII digits are now rejected by ``strptime``.
+
 
 .. class:: struct_time
 

Doc/whatsnew/3.14.rst

@@ -504,10 +504,10 @@ datetime
 * Add :meth:`datetime.time.strptime` and :meth:`datetime.date.strptime`.
   (Contributed by Wannes Boeykens in :gh:`41431`.)
 
-* The ``strptime(format)`` methods: :meth:`datetime.date.strptime`,
-  :meth:`datetime.datetime.strptime` and :meth:`datetime.time.strptime`
-  only accept ASCII digits.
-  (Contributed by Stan Ulbrych in :gh:`131008`
+* The :meth:`datetime.date.strptime`, :meth:`datetime.datetime.strptime` and
+  :meth:`datetime.time.strptime` methods now only accept ASCII digits, will and
+  raise a :exc:`ValueError` if non-ASCII digits are specified.
+  (Contributed by Stan Ulbrych in :gh:`131008`.)
 
 decimal
 -------
@@ -883,6 +883,15 @@ sys.monitoring
 * Two new events are added: :monitoring-event:`BRANCH_LEFT` and
   :monitoring-event:`BRANCH_RIGHT`. The ``BRANCH`` event is deprecated.
 
+
+time
+----
+
+* The :meth:`time.strptime`, now only accept ASCII digits, and will raise a
+  :exc:`ValueError` if non-ASCII digits are specified.
+  (Contributed by Stan Ulbrych in :gh:`131008`.)
+
+
 threading
 ---------
 

Lib/_strptime.py

@@ -16,7 +16,7 @@
 import calendar
 from re import compile as re_compile
 from re import sub as re_sub
-from re import ASCII, IGNORECASE
+from re import IGNORECASE
 from re import escape as re_escape
 from datetime import (date as datetime_date,
                       timedelta as datetime_timedelta,
@@ -290,7 +290,7 @@ def __init__(self, locale_time=None):
             'f': r"(?P<f>[0-9]{1,6})",
             'H': r"(?P<H>2[0-3]|[0-1][0-9]|[0-9])",
             'I': r"(?P<I>1[0-2]|0[1-9]|[1-9]| [1-9])",
-            'G': r"(?P<G>[0-9][0-9][0-9][0-9])",
+            'G': r"(?P<G>[0-9]{4})",
             'j': r"(?P<j>36[0-6]|3[0-5][0-9]|[1-2][0-9][0-9]|0[1-9][0-9]|00[1-9]|[1-9][0-9]|0[1-9]|[1-9])",
             'm': r"(?P<m>1[0-2]|0[1-9]|[1-9])",
             'M': r"(?P<M>[0-5][0-9]|[0-9])",
@@ -300,8 +300,8 @@ def __init__(self, locale_time=None):
             'u': r"(?P<u>[1-7])",
             'V': r"(?P<V>5[0-3]|0[1-9]|[1-4][0-9]|[0-9])",
             # W is set below by using 'U'
-            'y': r"(?P<y>[0-9][0-9])",
-            'Y': r"(?P<Y>[0-9][0-9][0-9][0-9])",
+            'y': r"(?P<y>[0-9]{2})",
+            'Y': r"(?P<Y>[0-9]{4})",
             'z': r"(?P<z>[+-][0-9][0-9]:?[0-5][0-9](:?[0-5][0-9](\.[0-9]{1,6})?)?|(?-i:Z))",
             'A': self.__seqToRE(self.locale_time.f_weekday, 'A'),
             'a': self.__seqToRE(self.locale_time.a_weekday, 'a'),

Lib/test/test_time.py

@@ -352,6 +352,10 @@ def test_strptime_leap_year(self):
                                    r'.*day of month without a year.*'):
             time.strptime('02-07 18:28', '%m-%d %H:%M')
 
+    def test_strptime_non_ascii(self):
+        with self.assertRaises(ValueError):
+            time.strptime('٢025', '%Y')
+
     def test_asctime(self):
         time.asctime(time.gmtime(self.t))