Run the CI auth pack against the production build
All checks were successful
CD / Build and push images (push) Successful in 40s
CI / Lint, typecheck, test (push) Successful in 1m14s
CI / Auth e2e pack (push) Successful in 1m39s
CI / Build container images (push) Has been skipped
CD / Deploy to Test (push) Successful in 8s
CD / Smoke tests against Test (push) Successful in 1m6s
CD / Promote to Int (push) Successful in 10s
All checks were successful
CD / Build and push images (push) Successful in 40s
CI / Lint, typecheck, test (push) Successful in 1m14s
CI / Auth e2e pack (push) Successful in 1m39s
CI / Build container images (push) Has been skipped
CD / Deploy to Test (push) Successful in 8s
CD / Smoke tests against Test (push) Successful in 1m6s
CD / Promote to Int (push) Successful in 10s
The Vite dev server died mid-run on the CI runner (memory pressure), failing every remaining test with connection refused. The auth-e2e job now serves apps/web/dist through a dependency-free static server with SPA fallback and /api proxy (scripts/e2e-static-server.mjs) — matching the production nginx/Caddy layout and testing the real build. Server logs are dumped when the job fails. Verified locally: six of six against the static server. Part of #20 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
parent
d29e95462c
commit
d05c36701b
@ -101,10 +101,10 @@ jobs:
|
|||||||
- name: Seed fixtures
|
- name: Seed fixtures
|
||||||
run: pnpm --filter @dorfteich/api db:seed
|
run: pnpm --filter @dorfteich/api db:seed
|
||||||
|
|
||||||
- name: Start api and web dev server
|
- name: Start api and static web server
|
||||||
run: |
|
run: |
|
||||||
(cd apps/api && PORT=3001 node dist/main.js &)
|
(cd apps/api && PORT=3001 node dist/main.js > /tmp/api.log 2>&1 &)
|
||||||
(pnpm --filter @dorfteich/web dev -- --host --port 5173 --strictPort &)
|
(PORT=5173 node scripts/e2e-static-server.mjs > /tmp/web.log 2>&1 &)
|
||||||
for i in $(seq 1 30); do
|
for i in $(seq 1 30); do
|
||||||
curl -sf http://localhost:3001/api/v1/readyz >/dev/null && break
|
curl -sf http://localhost:3001/api/v1/readyz >/dev/null && break
|
||||||
sleep 2
|
sleep 2
|
||||||
@ -119,6 +119,10 @@ jobs:
|
|||||||
E2E_BASE_URL=http://localhost:5173 E2E_MAILPIT_URL=http://mailpit:8025 \
|
E2E_BASE_URL=http://localhost:5173 E2E_MAILPIT_URL=http://mailpit:8025 \
|
||||||
pnpm --filter @dorfteich/web exec playwright test e2e/auth.spec.ts
|
pnpm --filter @dorfteich/web exec playwright test e2e/auth.spec.ts
|
||||||
|
|
||||||
|
- name: Dump server logs on failure
|
||||||
|
if: failure()
|
||||||
|
run: tail -50 /tmp/api.log /tmp/web.log || true
|
||||||
|
|
||||||
images:
|
images:
|
||||||
name: Build container images
|
name: Build container images
|
||||||
# PR-only: on main the CD workflow builds and pushes the same images —
|
# PR-only: on main the CD workflow builds and pushes the same images —
|
||||||
|
|||||||
@ -22,7 +22,7 @@ export default tseslint.config(
|
|||||||
// Plain-Node maintenance scripts (no TypeScript, no bundler).
|
// Plain-Node maintenance scripts (no TypeScript, no bundler).
|
||||||
files: ['scripts/**/*.mjs'],
|
files: ['scripts/**/*.mjs'],
|
||||||
languageOptions: {
|
languageOptions: {
|
||||||
globals: { console: 'readonly', process: 'readonly' },
|
globals: { console: 'readonly', process: 'readonly', URL: 'readonly' },
|
||||||
},
|
},
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
|
|||||||
70
scripts/e2e-static-server.mjs
Normal file
70
scripts/e2e-static-server.mjs
Normal file
@ -0,0 +1,70 @@
|
|||||||
|
#!/usr/bin/env node
|
||||||
|
/**
|
||||||
|
* Minimal server for e2e runs: serves the built SPA (apps/web/dist) with
|
||||||
|
* SPA fallback and proxies /api/* to the api process — mirroring what
|
||||||
|
* nginx + Caddy do in production, without a dev server that may die
|
||||||
|
* under CI memory pressure. Node builtins only.
|
||||||
|
*
|
||||||
|
* PORT=5173 API_TARGET=http://127.0.0.1:3001 node scripts/e2e-static-server.mjs
|
||||||
|
*/
|
||||||
|
import { readFile } from 'node:fs/promises';
|
||||||
|
import { createServer, request as httpRequest } from 'node:http';
|
||||||
|
import path from 'node:path';
|
||||||
|
import { fileURLToPath } from 'node:url';
|
||||||
|
|
||||||
|
const DIST = path.join(path.dirname(fileURLToPath(import.meta.url)), '../apps/web/dist');
|
||||||
|
const PORT = Number(process.env.PORT ?? 5173);
|
||||||
|
const API_TARGET = new URL(process.env.API_TARGET ?? 'http://127.0.0.1:3001');
|
||||||
|
|
||||||
|
const MIME = {
|
||||||
|
'.html': 'text/html; charset=utf-8',
|
||||||
|
'.js': 'application/javascript',
|
||||||
|
'.css': 'text/css',
|
||||||
|
'.json': 'application/json',
|
||||||
|
'.svg': 'image/svg+xml',
|
||||||
|
'.png': 'image/png',
|
||||||
|
'.ico': 'image/x-icon',
|
||||||
|
'.woff2': 'font/woff2',
|
||||||
|
};
|
||||||
|
|
||||||
|
function proxyApi(req, res) {
|
||||||
|
const upstream = httpRequest(
|
||||||
|
{
|
||||||
|
hostname: API_TARGET.hostname,
|
||||||
|
port: API_TARGET.port,
|
||||||
|
path: req.url,
|
||||||
|
method: req.method,
|
||||||
|
headers: { ...req.headers, host: `${API_TARGET.hostname}:${API_TARGET.port}` },
|
||||||
|
},
|
||||||
|
(upstreamRes) => {
|
||||||
|
res.writeHead(upstreamRes.statusCode ?? 502, upstreamRes.headers);
|
||||||
|
upstreamRes.pipe(res);
|
||||||
|
},
|
||||||
|
);
|
||||||
|
upstream.on('error', () => {
|
||||||
|
res.writeHead(502).end('api unreachable');
|
||||||
|
});
|
||||||
|
req.pipe(upstream);
|
||||||
|
}
|
||||||
|
|
||||||
|
async function serveStatic(req, res) {
|
||||||
|
const urlPath = new URL(req.url, 'http://x').pathname;
|
||||||
|
// Path traversal guard: resolve inside dist or fall back to the shell.
|
||||||
|
const filePath = path.normalize(path.join(DIST, urlPath));
|
||||||
|
const target =
|
||||||
|
filePath.startsWith(DIST) && path.extname(filePath) ? filePath : path.join(DIST, 'index.html');
|
||||||
|
try {
|
||||||
|
const body = await readFile(target);
|
||||||
|
res.writeHead(200, {
|
||||||
|
'Content-Type': MIME[path.extname(target)] ?? 'application/octet-stream',
|
||||||
|
});
|
||||||
|
res.end(body);
|
||||||
|
} catch {
|
||||||
|
res.writeHead(404).end('not found');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
createServer((req, res) => {
|
||||||
|
if (req.url.startsWith('/api/')) proxyApi(req, res);
|
||||||
|
else void serveStatic(req, res);
|
||||||
|
}).listen(PORT, () => console.log(`e2e static server on :${PORT} → ${API_TARGET.href}`));
|
||||||
Loading…
Reference in New Issue
Block a user