[Libosinfo] [osinfo-db PATCH] tests, test_dates: Keep just one implementation of _parse_iso_date()
Pino Toscano
ptoscano at redhat.com
Wed May 22 13:22:56 UTC 2019
On Wednesday, 22 May 2019 15:16:27 CEST Fabiano Fidêncio wrote:
> As suggested by Cole, let's keep just one implementation of
> _parse_iso_date() (the most compatible one) and avoid diverging over
> time.
>
> Signed-off-by: Fabiano Fidêncio <fidencio at redhat.com>
> ---
> tests/test_dates.py | 12 ++++--------
> 1 file changed, 4 insertions(+), 8 deletions(-)
>
> diff --git a/tests/test_dates.py b/tests/test_dates.py
> index 49df613..aca7472 100644
> --- a/tests/test_dates.py
> +++ b/tests/test_dates.py
> @@ -8,14 +8,10 @@ import sys
> from . import util
>
>
> -if sys.version_info >= (3, 7):
> - def _parse_iso_date(date_string):
> - return datetime.date.fromisoformat(date_string)
> -else:
> - def _parse_iso_date(date_string):
> - m = re.match("([0-9]{4})-([0-9]{2})-([0-9]{2})", date_string)
> - assert m
> - return datetime.date(int(m.group(1)), int(m.group(2)), int(m.group(3)))
> +def _parse_iso_date(date_string):
> + m = re.match("([0-9]{4})-([0-9]{2})-([0-9]{2})", date_string)
> + assert m
> + return datetime.date(int(m.group(1)), int(m.group(2)), int(m.group(3)))
The purposes of having the two implementation are:
- datetime.date.fromisoformat() is available in newer versions, so it
makes sense to leverage it
- my implementation is not optimized as datetime.date.fromisoformat();
considering this is only a test, it is "good enough"
- having the version check means it is easier to drop the old
implementation when raising the minimum Python version required
So I'm on -1 on this.
--
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libosinfo/attachments/20190522/91ac5fe0/attachment.sig>
More information about the Libosinfo
mailing list