Add test coverage for domains/platforms/etsy #31

Open
opened 2026-08-20 23:30:25 -06:00 by angel · 0 comments
Owner

domains/platforms/etsy has no *_test.go files at all - it's the only platform package without integration test coverage (see AGENTS.md's Testing section for the pattern every other domain follows: real Postgres-backed tests via internal/testdb). This gap is exactly what let the broken saveNewEtsyUser query (#24) go unnoticed. Add coverage for the OAuth connection flow first, then extend to the webhook receiver (#25) and sync logic (#8, #27, #28) as those land.

`domains/platforms/etsy` has no `*_test.go` files at all - it's the only platform package without integration test coverage (see AGENTS.md's Testing section for the pattern every other domain follows: real Postgres-backed tests via `internal/testdb`). This gap is exactly what let the broken `saveNewEtsyUser` query (#24) go unnoticed. Add coverage for the OAuth connection flow first, then extend to the webhook receiver (#25) and sync logic (#8, #27, #28) as those land.
angel added this to the Etsy Integration milestone 2026-08-20 23:30:25 -06:00
angel added this to the MVP project 2026-08-20 23:49:47 -06:00
Sign in to join this conversation.