Matt
d3960af340
Four low-risk adds before the Zod 4 / drizzle-zod headliner: - @total-typescript/ts-reset: tightens TS stdlib types globally (JSON.parse → unknown, fetch().json() → unknown, .filter(Boolean) narrows, Set literals respect typed Set targets). Caught 179 latent type errors; fixed all production sites (8 files) and added `any` cast escape hatch in test files (ESLint exemption scoped to tests/). - web-vitals + /api/v1/internal/vitals endpoint + WebVitalsReporter client component: establishes Core Web Vitals baseline (LCP/INP/CLS/ FCP/TTFB) via navigator.sendBeacon. Required before optimisation work. - @hookform/devtools + FormDevtool wrapper: dev-only RHF state inspector, lazy-loaded via next/dynamic so the chunk is excluded from prod bundles entirely. - @tanstack/query-broadcast-client-experimental: cross-tab cache sync via BroadcastChannel — wired in query-provider.tsx, 1-liner. Audit doc updated with sections 35 + 36 (PDF stack overhaul + comprehensive second-pass package sweep) covering ~20 package adoption candidates and 4-5 deprecation candidates. Verified: tsc clean, vitest 1293/1293 pass. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
37 lines
1.4 KiB
TypeScript
37 lines
1.4 KiB
TypeScript
import { describe, it, expect } from 'vitest';
|
|
|
|
import { GET as healthGet } from '@/app/api/health/route';
|
|
import { GET as readyGet } from '@/app/api/ready/route';
|
|
|
|
describe('GET /api/health (liveness)', () => {
|
|
it('returns 200 + status=ok regardless of downstream dependency state', async () => {
|
|
const res = await healthGet();
|
|
expect(res.status).toBe(200);
|
|
const body = (await res.json()) as any;
|
|
expect(body.status).toBe('ok');
|
|
expect(body.timestamp).toMatch(/^\d{4}-\d{2}-\d{2}T/);
|
|
});
|
|
});
|
|
|
|
describe('GET /api/ready (readiness)', () => {
|
|
it('returns ready=200 when postgres + redis + storage all answer', async () => {
|
|
// The dev/test environment has all three reachable; this covers the
|
|
// happy path. The degraded path is not exercised here because
|
|
// simulating a down dep without leaking into other tests is awkward;
|
|
// the route's logic is intentionally trivial (Promise.allSettled +
|
|
// every-ok check) and worth covering at the unit level only.
|
|
//
|
|
// `storage` (renamed from `minio`) probes whichever backend the
|
|
// active port is configured for, via getStorageBackend().head().
|
|
const res = await readyGet();
|
|
expect(res.status).toBe(200);
|
|
const body = (await res.json()) as any;
|
|
expect(body.status).toBe('ready');
|
|
expect(body.checks).toEqual({
|
|
postgres: 'ok',
|
|
redis: 'ok',
|
|
storage: 'ok',
|
|
});
|
|
});
|
|
});
|