Test adjustments, version bumps and associated adjustments - #1245
Open
peterthomassen wants to merge 6 commits into
Open
Test adjustments, version bumps and associated adjustments#1245peterthomassen wants to merge 6 commits into
peterthomassen wants to merge 6 commits into
Conversation
peterthomassen
force-pushed
the
20260819_bumps
branch
from
August 19, 2026 21:46
6953a4c to
7adbe21
Compare
DesecTestCase.setUp() creates a domain for each of settings.LOCAL_PUBLIC_SUFFIXES
("dedyn.$DESECSTACK_DOMAIN"). When DESECSTACK_DOMAIN is set to "io", the test
collides with that domain and fails with an IntegrityError. Use domain names
under the .example TLD instead, which cannot be a local public suffix.
peterthomassen
force-pushed
the
20260819_bumps
branch
2 times, most recently
from
August 20, 2026 00:00
ed57383 to
c6f1e83
Compare
Two incompatibilities had to be addressed: - Migration 0037: unsliced querysets are no longer accepted on the right-hand side of an exact lookup, so slice the correlated subqueries. The default policy is unique per token, so this does not change the semantics. - django-celery-email accesses the removed EmailMessage.mixed_subtype attribute, and has not seen a release since 2019. We only used its email_to_dict()/dict_to_email() helpers (the app was not installed), so inline them: all we need is a JSON-serializable representation of the message, which the message's attributes already are (except the connection, which the task recreates anyway). Test settings carry the connection over so that the mail outbox exposes which lane was used.
peterthomassen
force-pushed
the
20260819_bumps
branch
from
August 20, 2026 00:10
c6f1e83 to
82dd1ca
Compare
peterthomassen
changed the title
Cuts the API test suite runtime considerably by distributing tests over per-CPU worker processes with cloned test databases. Coverage needs a config file for this: the multiprocessing concurrency setting is what makes the worker processes record data, and it is passed on to them via the rcfile (command line options are not). The resulting per-process data files are merged by `coverage combine`.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upgrading only to Django 6.0, although Django 6.1. has been released two weeks ago. Two reasons to hold back: