Skip to content

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Mar 23, 2024

Fixes #9342

+ @pytest.mark.xfail(pytz and DJANGO_VERSION >= (5, 0), reason="Django v5 drops support for pytz")

Dj5 drops support for pytz so skip this test on an invalid configuration.

https://docs.djangoproject.com/en/5.0/releases/5.0/#features-removed-in-5-0

  • The USE_DEPRECATED_PYTZ transitional setting is removed.
  • Support for pytz timezones is removed.

@cclauss cclauss changed the title GitHub Actions: Run tests_with_pytz_installed DRAFT: GitHub Actions: Run tests_with_pytz_installed Mar 23, 2024
@cclauss cclauss changed the title DRAFT: GitHub Actions: Run tests_with_pytz_installed xfail pytz test_invalid_inputs() on Django >= v5.0 Mar 24, 2024
@cclauss cclauss marked this pull request as ready for review March 24, 2024 15:24
assert self.field.run_validation(input_value) == expected_output, \
'input value: {}'.format(repr(input_value))

@pytest.mark.xfail(pytz and DJANGO_VERSION >= (5, 0), reason="Django v5 drops support for pytz")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like it'll skip a whole load of tests regardless of if pytz is relevant to that particular test case or not.

@cclauss cclauss closed this Apr 9, 2024
@cclauss cclauss deleted the patch-1 branch April 9, 2024 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants