[api] testing real progression event

This commit is contained in:
Devin Zuczek
2026-10-13 10:52:48 -04:00
parent 810e3b8f27
commit cf6868a753
2 changed files with 15 additions and 15 deletions
+11 -11
View File
@@ -565,29 +565,29 @@ export const progressionRoutes = new Hono<App>({ strict: false })
// The progression event running right now — the limited-time XP event the client shows // The progression event running right now — the limited-time XP event the client shows
// a banner and a progress track for. // a banner and a progress track for.
// //
// STUB: a bare `0` — the id of the active event, or 0 when there is none (this once // STUB: a bare `18` — the id of the active event, or 0 when there is none (this once
// served -1 for the same meaning). The client reads it as "no event on" and skips the // served -1 for the same meaning). 18 is hardcoded: it names an event the archive above
// event UI entirely. That is the honest answer (nothing here runs events) and the safe // holds, so the client can fetch its definition from `/event/18`, but nothing here
// one: a fabricated event would draw a track that never fills. No auth — whether an // tracks progress through it yet. No auth — whether an event is running is the same
// event is running is the same fact for everybody, and the client asks while loading. // fact for everybody, and the client asks while loading.
.get( .get(
'/api/progressionEvents/active', '/api/progressionEvents/active',
describeRoute({ describeRoute({
tags: ['Progression'], tags: ['Progression'],
summary: 'Progression event currently running (stub)', summary: 'Progression event currently running (stub)',
description: description:
'The limited-time XP event in progress, as a bare event id. Always `0` — the ' + 'The limited-time XP event in progress, as a bare event id (`0` would mean “no ' +
'“no active event” value; nothing on this server runs one — which the client reads ' + 'active event”). Always `18` for now — a hardcoded archived event; nothing on this ' +
'as “no event” and skips the event UI, where a 404 would stall the load. No auth: ' + 'server tracks progress through it yet. A 404 here would stall the load. No auth: ' +
'it is the same answer for every player.', 'it is the same answer for every player.',
responses: { 200: json(BareInteger, 'A bare `0` — no event is running') }, responses: { 200: json(BareInteger, 'A bare `18` — the hardcoded active event') },
}), }),
(c) => c.json(0) (c) => c.json(18)
) )
// One progression event's definition — its reward track, keepsake rooms, dates and boost // One progression event's definition — its reward track, keepsake rooms, dates and boost
// settings — served from the archive of official events above. Reading a definition says // settings — served from the archive of official events above. Reading a definition says
// nothing about whether the event is running (`/active` above still answers 0), and no // nothing about whether the event is running (`/active` above is a hardcoded 18), and no
// auth for the same reason as there: it is the same record for everybody. // auth for the same reason as there: it is the same record for everybody.
.get( .get(
'/api/progressionEvents/event/:id{[0-9]+}', '/api/progressionEvents/event/:id{[0-9]+}',
+4 -4
View File
@@ -2081,12 +2081,12 @@ describe('public endpoints', () => {
expect(res.status).toBe(400) expect(res.status).toBe(400)
}) })
test('GET /api/progressionEvents/active is a bare 0 (no auth)', async () => { test('GET /api/progressionEvents/active is a bare 18 (no auth)', async () => {
// 0 is the "no active event" value; the client reads it as "no event running" and // A hardcoded stub: the id of an archived event (0 would mean "no active event").
// skips the event UI. A 404 would stall its load instead. // A 404 would stall the client's load instead.
const res = await exports.default.fetch(`${ORIGIN}/api/progressionEvents/active`) const res = await exports.default.fetch(`${ORIGIN}/api/progressionEvents/active`)
expect(res.status).toBe(200) expect(res.status).toBe(200)
expect(await res.json()).toBe(0) expect(await res.json()).toBe(18)
}) })
test('GET /api/progressionEvents/event/:id serves the archived event verbatim', async () => { test('GET /api/progressionEvents/event/:id serves the archived event verbatim', async () => {