Integration test refactor

flarelabs-net / vite-plugin-cloudflare

13 stars 1 forks source link

Integration test refactor #27

Closed petebacondarwin closed 1 month ago

petebacondarwin commented 1 month ago

There are two commits here:

wire up workerd/miniflare logs to Vite
refactor: make integration tests their own workspace packages

petebacondarwin commented 1 month ago

This is now ready for a review. The excess SIGTERM handlers I was seeing were because I was creating many vite dev servers in beforeEach() handlers but not then cleaning them up.

petebacondarwin commented 1 month ago

Looks great! I think we'll need to cleanup the server in the Durable Objects tests too but it doesn't have to be done in this PR.

The node-compat PR does some further clean up of tests I think...