italia/developers-italia-api

Webhooks and catalogs lack DB persistence checks for write endpoints

Open

#384 aperta il 10 mag 2026

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Go (8 fork)auto 404
good first issue

Metriche repository

Star
 (16 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Most resources have endpoint tests that only verify the HTTP response, not what actually lands in the database. POST and PATCH can return 200 with the right body and still write the wrong row, and DELETE can return 204 without removing anything.

logs, publishers and software already have TestXxxPostDBChecks, TestXxxPatchDBChecks and TestXxxDeleteDBChecks (in #294 and follow-ups). Two resources are still missing equivalent coverage:

  • webhooks: no DB persistence checks for POST, PATCH or DELETE
  • catalogs: existing checks cover only catalog_sources and DELETE, nothing asserts that POST or PATCH actually persist name, alternativeId or publishersNamespace on the catalog row itself

Guida contributor

Direzione di ricerca
Esamina gli esistenti `TestXxxPostDBChecks`, `TestXxxPatchDBChecks` e `TestXxxDeleteDBChecks` per logs, publishers e software (dalle issue #294 e successive). Replica la loro struttura per le risorse webhooks e catalogs.
Tech stack
go
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Gotesting basicsdatabase
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75