zostera / django-bootstrap5 / 33147117278

96%

push

github

Add tests for django.contrib.gis form field rendering (#861)

`django.contrib.gis` has been in INSTALLED_APPS for tests since the
bootstrap4 days, and CI installs GDAL, but nothing has exercised a GIS
form field here since the Bootstrap 5 rewrite. Commit d88bfe2 ("Render
form fields in Bootstrap 5") dropped the import and the field, carrying
over django-bootstrap3's commented-out state; the comment itself was
removed later, leaving the INSTALLED_APPS entry and the GDAL install as
vestige.

The coverage is worth having. A geometry widget builds its own template
context from `attrs`, and Django ticket #28105 was a crash in exactly
that path, triggered by a template pack passing an attr whose name the
widget already used. That seam is bootstrap-specific, and it is live
code here: recent work forwards per-option attrs and stops attrs leaking
onto wrapper elements.

Written as a dedicated test file with explicit assertions, matching this
repo's one-file-per-field-type layout, rather than django-bootstrap4's
approach of dropping a PointField into a shared form and relying on bulk
rendering. That also pins the current behavior: the widget's serialized
textarea keeps its own classes and does not get form-control.

Verified: 158 tests pass (2 new), lint passes, on Django 5.2.16.

193 of 206 branches covered (93.69%)

Branch coverage included in aggregate %.

664 of 686 relevant lines covered (96.79%)

13.55 hits per line