Nick SzaunigandClaude Opus 5.5
c01a38835a
Platform backbone (plan 2026-10-13-plattform-rueckgrat), reviewed tasks: - run roles api/worker/job/all with shared startup checks (Task 1) - schema and migration for DomainEvent, deliveries, JobRun, ThrottlerHit (Task 2) - audit, notifications and domain events written in the same transaction (Task 3) - dispatcher with SKIP LOCKED claims, retries and stale release (Task 4) - worker building blocks: tick, LISTEN/NOTIFY wake-up, health (Task 5) - worker process and entry points (Task 6) - throttler counters in Postgres (Task 19) - separate keys per purpose with key ring (Task 20) - public origin page without exact location (Task 22) - shop keys split into read and write (Task 23) - Kustomize base, components, overlay encry, CI validation (Task 26) Field capture T3 (plan 2026-10-13-t3-erfassung-am-stand), reviewed tasks: - brood frames, read endpoints, apiary overview and offline pack - network basics, offline queue v3, navigation sheets, form building blocks - new inspection with last-time hints and weather, varroa per day, treatment prefill, apiary page with round - inspection photo (Task 10) is implemented, review pending Not rolled out. Remaining tasks follow in later commits. Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
62 lines
1.8 KiB
TypeScript
62 lines
1.8 KiB
TypeScript
import { INestApplication } from '@nestjs/common';
|
|
import request from 'supertest';
|
|
import { App } from 'supertest/types';
|
|
import { PrismaThrottlerStorage } from '../src/common/throttler/prisma-throttler.storage';
|
|
import { createTestApp } from './utils/create-test-app';
|
|
|
|
describe('Health (e2e)', () => {
|
|
let app: INestApplication<App>;
|
|
|
|
beforeAll(async () => {
|
|
app = await createTestApp();
|
|
});
|
|
|
|
afterAll(async () => {
|
|
await app.close();
|
|
});
|
|
|
|
it('GET /health returns ok without auth', () => {
|
|
return request(app.getHttpServer())
|
|
.get('/health')
|
|
.expect(200)
|
|
.expect({ status: 'ok' });
|
|
});
|
|
|
|
it('GET /health/ready returns ok when the database is reachable', () => {
|
|
return request(app.getHttpServer())
|
|
.get('/health/ready')
|
|
.expect(200)
|
|
.expect({ status: 'ok' });
|
|
});
|
|
|
|
describe('when the throttler storage is down', () => {
|
|
afterEach(() => jest.restoreAllMocks());
|
|
|
|
it('GET /health stays 200 (liveness must not depend on the database)', async () => {
|
|
jest
|
|
.spyOn(app.get(PrismaThrottlerStorage), 'increment')
|
|
.mockRejectedValue(new Error('db down'));
|
|
|
|
await request(app.getHttpServer()).get('/health').expect(200);
|
|
});
|
|
|
|
it('GET /health/ready stays throttled (fails with the storage)', async () => {
|
|
jest
|
|
.spyOn(app.get(PrismaThrottlerStorage), 'increment')
|
|
.mockRejectedValue(new Error('db down'));
|
|
|
|
await request(app.getHttpServer()).get('/health/ready').expect(500);
|
|
});
|
|
|
|
it('other routes fail closed instead of being allowed', async () => {
|
|
jest
|
|
.spyOn(app.get(PrismaThrottlerStorage), 'increment')
|
|
.mockRejectedValue(new Error('db down'));
|
|
|
|
await request(app.getHttpServer())
|
|
.get('/api/v1/public/batches/NONE-x')
|
|
.expect(500);
|
|
});
|
|
});
|
|
});
|