Skip to content

Fix test with Django 5 when pytz is available #9715

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
browniebroke merged 9 commits into from
Jul 24, 2025
+14 −15
Merged

Fix test with Django 5 when pytz is available #9715

Changes from 2 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
656d835
Fix test with Django 5 when pytz is available
kulikjak Jun 4, 2025
4e42975
fix formatting
kulikjak Jun 4, 2025
5b25346
remove original condition
kulikjak Jun 5, 2025
3817fb9
remove trailing whitespace
kulikjak Jun 5, 2025
d3ff351
further improvements
kulikjak Jun 5, 2025
a0fbdbd
let's not skip the pytz test - it should always be executed when test…
kulikjak Jun 5, 2025
a415b60
add comment to test requirements
kulikjak Jul 21, 2025
ecccc15
simplify the pytz import as it should always be available
kulikjak Jul 21, 2025
89e1ba3
make isort happy
kulikjak Jul 21, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion tests/test_fields.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
except ImportError:
pytz = None
Comment thread
browniebroke marked this conversation as resolved.
Outdated

import django
from django.core.exceptions import ValidationError as DjangoValidationError
from django.db.models import IntegerChoices, TextChoices
from django.http import QueryDict
Expand Down Expand Up @@ -1624,7 +1625,8 @@ def test_should_render_date_time_in_default_timezone(self):
assert rendered_date == rendered_date_in_timezone


@pytest.mark.skipif(pytz is None, reason="Django 5.0 has removed pytz; this test should eventually be able to get removed.")
@pytest.mark.skipif(pytz is None or django.VERSION >= (5,),
reason="Django 5.0 has removed pytz; this test should eventually be able to get removed.")
Comment thread
kulikjak marked this conversation as resolved.
Outdated
class TestPytzNaiveDayLightSavingTimeTimeZoneDateTimeField(FieldValues):
"""
Invalid values for `DateTimeField` with datetime in DST shift (non-existing or ambiguous) and timezone with DST.
Expand Down
Loading
⚡