Apply CodeRabbit review fixes

- serializers/base.py: fix "constitency" typo → "consistency"
- utils.py: remove redundant setdefault() on line 72; remove unnecessary
  list() wrapping in _diff_by_content(); add idempotency note to
  patch_transaction_test_case docstring
- fixture.py: wrap diff() body in try/finally so dump_path is always
  unlinked, even when a diff is found
- tox.ini: bump docs env Django from >=4.2,<5.0 to >=5.2,<6.0
- settings_postgresql.py: update ENGINE to django.db.backends.postgresql
- ci.yml: bump codecov/codecov-action@v4 → @v6
- CHANGELOG: split 0.9.7 entry into one item per line
- test_fixture.py: add inline comment noting expected fixture content
- docs/index.rst: add automodule directives for serializers.json and apps
- pyproject.toml: exclude test packages from wheel discovery

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: archTortugax

.github/workflows/ci.yml

@@ -71,6 +71,6 @@ jobs:
         DB_PASSWORD: dbdiff
         PGPASSWORD: dbdiff
     - name: Upload coverage
-      uses: codecov/codecov-action@v4
+      uses: codecov/codecov-action@v6
       with:
         fail_ci_if_error: false

CHANGELOG

@@ -1,4 +1,7 @@
-0.9.7 Fix always-False diff condition (temp file leak + assertNoDiff TypeError on exact match); add Python 3.10-3.14 and Django 4.2/5.2/6.0 support; drop Python 3.8/3.9 and Django EOL versions; migrate to pyproject.toml; modernize CI (checkout@v4, setup-python@v5, codecov-action); add automated PyPI release workflow
+0.9.7 Fix always-False diff condition (temp file leak + assertNoDiff TypeError on exact match)
+      Use ignore_pk in ContentTypeTestCase for Django 6.0 content-type ordering stability
+      Add Python 3.10-3.14 and Django 4.2/5.2/6.0; drop EOL Python 3.8/3.9 and Django 4.0/4.1
+      Migrate to pyproject.toml; modernize CI (checkout@v4, setup-python@v5, codecov-action@v6)
 
 0.9.6 Add support for Python 3.12
 

dbdiff/fixture.py

@@ -108,23 +108,25 @@ def diff(self, exclude=None, ignore_pk=None):
         exclude_final = copy.copy(self.exclude)
         exclude_final.update(exclude or {})
 
-        with os.fdopen(fh, 'w') as f:
-            self.dump(f)
+        try:
+            with os.fdopen(fh, 'w') as f:
+                self.dump(f)
 
-        with open(self.path, 'r') as e, open(dump_path, 'r') as r:
-            expected, result = json.load(e), json.load(r)
+            with open(self.path, 'r') as e, open(dump_path, 'r') as r:
+                expected, result = json.load(e), json.load(r)
 
-        if ignore_pk is None:
-            ignore_pk = self.ignore_pk
+            if ignore_pk is None:
+                ignore_pk = self.ignore_pk
 
-        unexpected, missing, different = diff(
-            get_tree(expected, exclude_final),
-            get_tree(result, exclude_final),
-            ignore_pk=ignore_pk,
-        )
+            unexpected, missing, different = diff(
+                get_tree(expected, exclude_final),
+                get_tree(result, exclude_final),
+                ignore_pk=ignore_pk,
+            )
+        finally:
+            os.unlink(dump_path)
 
         if not unexpected and not missing and not different:
-            os.unlink(dump_path)
             return None
 
         return unexpected, missing, different

dbdiff/serializers/base.py

@@ -49,7 +49,7 @@ def remove_microseconds(cls, data):
 
     @classmethod
     def normalize_decimals(cls, data):
-        """Strip trailing zeros for constitency.
+        """Strip trailing zeros for consistency.
 
         In addition, dbdiff serialization forces Decimal normalization, because
         trailing zeros could happen in inconsistent ways.

dbdiff/tests/project/settings_postgresql.py

@@ -4,7 +4,7 @@
 
 DATABASES = {
     'default': {
-        'ENGINE': 'django.db.backends.postgresql_psycopg2',
+        'ENGINE': 'django.db.backends.postgresql',
         'HOST': os.environ.get('DB_HOST', ''),
         'NAME': os.environ.get('DB_NAME', 'dbdiff_test'),
         'USER': os.environ.get('DB_USER', 'postgres'),

dbdiff/tests/test_fixture.py

@@ -28,5 +28,6 @@ def test_diff_exact_match_returns_none(self):
         # Regression: the always-False `not diff` (imported function) bug
         # caused diff() to never return None, leaking temp files and breaking
         # assertNoDiff() with a TypeError on exact matches.
+        # Fixture contains: [{"model": "auth.group", "pk": 1, "fields": {"name": "initial_name"}}]
         Group.objects.create(id=1, name='initial_name')
         assert self.fixture.diff() is None

dbdiff/utils.py

@@ -69,7 +69,6 @@ def diff(expected, result, ignore_pk=False):  # noqa: C901
             if expected_fields == result_fields:
                 continue
 
-            different.setdefault(model, {})
             different.setdefault(model, {}).setdefault(pk, {})
 
             for expected_field, expected_value in expected_fields.items():
@@ -94,7 +93,7 @@ def _diff_by_content(expected, result):  # noqa: C901
         result_list = list(result.get(model, {}).items())
         matched_result_indices = set()
 
-        for exp_pk, exp_fields in list(expected_list):
+        for exp_pk, exp_fields in expected_list:
             found = False
             for i, (res_pk, res_fields) in enumerate(result_list):
                 if i in matched_result_indices:
@@ -154,7 +153,14 @@ def get_models_tables(models):
 
 
 def patch_transaction_test_case():
-    """Monkeypatch TransactionTestCase._reset_sequences to support SQLite."""
+    """Monkeypatch TransactionTestCase._reset_sequences to support SQLite.
+
+    Safe to call once, from AppConfig.ready(). Calling it a second time would
+    wrap the already-patched classmethod again; the inner _needs_explicit_cls
+    check would then see a classmethod and call _original(db_name), passing
+    db_name correctly, so double-patching doesn't break anything — but callers
+    should avoid it anyway.
+    """
     import inspect
 
     from django.test.testcases import TransactionTestCase

docs/index.rst

@@ -14,3 +14,9 @@ API Reference
 
 .. automodule:: dbdiff.test
    :members:
+
+.. automodule:: dbdiff.serializers.json
+   :members:
+
+.. automodule:: dbdiff.apps
+   :members:

pyproject.toml

@@ -44,6 +44,7 @@ include-package-data = true
 
 [tool.setuptools.packages.find]
 where = ["."]
+exclude = ["dbdiff.tests*", "tests*", "tests.*"]
 
 [tool.ruff]
 line-length = 79

tox.ini

@@ -70,6 +70,6 @@ commands =
     sphinx-build -W -b html docs docs/_build/html
 deps =
     sphinx
-    Django>=4.2,<5.0
+    Django>=5.2,<6.0
 setenv =
     DJANGO_SETTINGS_MODULE=dbdiff.tests.project.settings_sqlite