Add docker-compose integration test harness#104
Conversation
Miracle656
left a comment
There was a problem hiding this comment.
Thanks for this. A few things to resolve before it can land:
-
The Integration tests job is failing —
Cannot find module '@rolldown/binding-linux-x64-gnu'(the npm optional-deps bug, npm/cli#4828). The committedpackage-lock.jsondoesn't include the Linux native binding Vitest needs in CI. Regenerating the lockfile so the platform-optional deps are captured (or pinning the binding) should fix it. -
Overlap with the existing harness.
mainalready has a docker-compose integration setup (from #65). Please clarify what this adds on top of it — if it's an improvement, ideally extend the existing harness rather than adding a parallelvitest.integration.config.tspath; if it supersedes #65's, say so explicitly. -
Unrelated change:
src/__tests__/opa.test.tsand thesrc/index.tsedit don't look related to an integration harness — if they belong to a different concern, please split them out.
Once the integration job is green and the relationship to the existing harness is clear, I'll re-review. Thanks!
Summary
Added a docker-compose-based integration test harness for Wraith that runs against a real Postgres instance, seeds representative fixtures, and exercises live API endpoints through Vitest. Also wired the suite into CI so it runs on every pull request.
Related issue
Type of change
Checklist
npx tsc --noEmitpassesnpm run buildpasses