Skip to content

test: smoke every admin changelist#433

Merged
alonisser merged 1 commit into
mainfrom
test/admin-changelist-smoke
May 26, 2026
Merged

test: smoke every admin changelist#433
alonisser merged 1 commit into
mainfrom
test/admin-changelist-smoke

Conversation

@alonisser
Copy link
Copy Markdown
Collaborator

Summary

  • Adds a parametrized smoke test in studies/tests/test_admin_smoke.py and uncontrast_studies/tests/test_admin_smoke.py that iterates admin.site._registry filtered by app label and GETs every registered model's changelist URL.
  • Closes the existing coverage gap (previously only ~9 of 44 registered admin models had a changelist smoke test) and stays in sync automatically as models are added.
  • Empty changelists still exercise list_display, list_filter, search_fields, and get_queryset overrides — useful as a safety net for Django / import-export / simple-history / two-factor upgrades.

Test plan

  • DJANGO_CONFIGURATION=Testing poetry run python manage.py test studies.tests.test_admin_smoke uncontrast_studies.tests.test_admin_smoke — both pass, covering 24 + 20 models.
  • CI green.

🤖 Generated with Claude Code

Iterates admin.site._registry filtered by app_label so every registered
ModelAdmin gets a GET on its changelist URL. Catches breakage from Django
or admin-related library upgrades (list_display, list_filter, search_fields,
get_queryset overrides) without needing per-model fixtures — empty
changelists still render the full admin pipeline.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@alonisser alonisser merged commit 4e20768 into main May 26, 2026
10 checks passed
@alonisser alonisser deleted the test/admin-changelist-smoke branch May 26, 2026 09:14
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.

1 participant