packages/kit/test/build.test.ts (1)

177-203: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Add a timeout guard or keep the promise deterministic.

The test awaits a promise that only settles when the middleware calls res.end or when next runs. If createServer registers a middleware that returns early (for example a redirect that calls writeHead then end with no body), the assertion on body becomes brittle. The current mock returns undefined from writeHead, so a redirecting middleware would resolve the promise with an unexpected value. Consider asserting on mounted and the middleware output separately, or use server.middlewares.handle with a real http.ServerResponse.

This is optional. The test passes with the current plugin set.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@packages/kit/test/build.test.ts` around lines 177 - 203, Make the
middleware-response assertion in the real Vite server test deterministic by
avoiding reliance on the mock response’s incomplete redirect behavior. Update
the promise/mock around server.middlewares (or use a real ServerResponse) so
redirects and early middleware exits cannot resolve body with an unexpected
value, while preserving the existing mounted and '{"ok":true}' assertions.