Freegle / Iznik / 5857

72%

master: 70%

Build Type

Pull #252

circleci

Committed by edwh

Commit Message

test(coverage): TDD coverage for POST /api/group (CreateGroup)

Adds 14 tests for the CreateGroup handler covering: unauthorized gate,
missing-name validation, regular-user 403, moderator/owner/admin/support
can create, default and custom group types, creator auto-Owner membership,
admin lat/lng storage, non-admin lat/lng ignored, /apiv2 path, and
non-zero ID smoke test.

Also fixes a real handler bug discovered via TDD: the original INSERT used
the non-existent column `namedisplay` and an invalid `region='UK'` enum
value. The fix uses `namefull` (the correct column), removes the invalid
region value, and supplies a valid initial `polyindex` geometry value
(`ST_GeomFromText('POINT(0 0)', 3857)`) required by the spatial index.

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

Pull Request Pull Request #252: test(coverage): TDD coverage for POST /api/group (CreateGroup)

Coverage Stats

13399 of 20329 branches covered (65.91%)

Branch coverage included in aggregate %.

24 of 30 new or added lines in 3 files covered. (80.0%)

93 existing lines in 4 files now uncovered.

96390 of 132987 relevant lines covered (72.48%)

21.58 hits per line