Skip to content

[CI verification] test(backend): endpoint tests for feedback, stars, notifications#1

Closed
ShawK91 wants to merge 1 commit into
developfrom
test/229-endpoint-tests-feedback-stars-notifications
Closed

[CI verification] test(backend): endpoint tests for feedback, stars, notifications#1
ShawK91 wants to merge 1 commit into
developfrom
test/229-endpoint-tests-feedback-stars-notifications

Conversation

@ShawK91

@ShawK91 ShawK91 commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Draft verification PR on my own fork to run the backend test workflow before submitting upstream (hotosm#229). Not for merge.

…ons apps

Adds API test coverage for three previously-untested apps (hotosm#229):

- feedback: auth required, create sets request user, GeoJSON response
  shape, invalid geometry/action rejected, stac_id/action filters,
  owner-only update/delete (IsOwnerOrAdminOrReadOnly)
- stars: target_id validation, anonymous star dedupe via IP+UA hash,
  distinct anon clients counted separately, authed star/unstar state,
  anon unstar removes only own star
- notifications: public banner list hides unstarted banners, expired
  banners flagged not displayable, per-user notification scoping,
  mark-read / mark-all-read behavior incl. cross-user isolation

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@ShawK91 ShawK91 force-pushed the test/229-endpoint-tests-feedback-stars-notifications branch from f6b9a72 to a0b6ab8 Compare June 11, 2026 13:56
@ShawK91

ShawK91 commented Jun 11, 2026

Copy link
Copy Markdown
Owner Author

CI verification complete (154 passed); submitted upstream as hotosm#506.

@ShawK91 ShawK91 closed this Jun 11, 2026
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