#29754: Trunc() doesn't handle NonExistingTimeError/AmbiguousTimeError
-------------------------------------+-------------------------------------
Reporter: Alexander | Owner: nobody
Holmbäck |
Type: Bug | Status: new
Component: Database | Version: master
layer (models, ORM) |
Severity: Normal | Keywords: pytz, Trunc()
Triage Stage: | Has patch: 0
Unreviewed |
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-------------------------------------+-------------------------------------
When `Trunc()` truncates to a nonexisting or ambiguous datetime, the
exception raised by pytz remains unhandled. The expected behavior would,
IMO, be to use naive datetimes for truncated dates, or adjust to closest
existing non-ambiguous datetime.
This test for example:
{{{#!python
import datetime
import pytz
from django.db.models.functions import Trunc
from django.test import TestCase
from django.utils import timezone
from .models import Log
class TestTruncateToNonExistingTime(TestCase):
def test_truncate_to_dst_ends_stockholm(self):
tzinfo = pytz.timezone('Europe/Stockholm')
timestamp = datetime.datetime(2018, 10, 28, 2, tzinfo=tzinfo)
Log.objects.create(timestamp=timestamp)
logs = Log.objects.annotate(day=Trunc('timestamp', 'hour')).all()
timezone.activate(tzinfo)
self.assertEqual(logs[0].day.day, 28)
}}}
Results in the following error:
{{{
======================================================================
ERROR: test_truncate_to_dst_ends_stockholm
(trunc.tests.TestTruncateToNonExistingTime)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/alex/tickets/trunc/tests.py", line 47, in
test_truncate_to_dst_ends_stockholm
self.assertEqual(logs[0].day.day, 28)
File "/home/alex/django/django/db/models/query.py", line 303, in
__getitem__
qs._fetch_all()
File "/home/alex/django/django/db/models/query.py", line 1190, in
_fetch_all
self._result_cache = list(self._iterable_class(self))
File "/home/alex/django/django/db/models/query.py", line 64, in __iter__
for row in compiler.results_iter(results):
File "/home/alex/django/django/db/models/sql/compiler.py", line 1013, in
apply_converters
value = converter(value, expression, connection)
File "/home/alex/django/django/db/models/functions/datetime.py", line
225, in convert_value
value = timezone.make_aware(value, self.tzinfo)
File "/home/alex/django/django/utils/timezone.py", line 270, in
make_aware
return timezone.localize(value, is_dst=is_dst)
File "/home/alex/.virtualenvs/djangodev/lib/python3.6/site-
packages/pytz/tzinfo.py", line 363, in localize
raise AmbiguousTimeError(dt)
pytz.exceptions.AmbiguousTimeError: 2018-10-28 02:00:00
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/29754>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/051.e0a2081e6da704587b06d7512bf6225b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
