From b6617fc356f8c7a037e1db3a242355b820fb2572 Mon Sep 17 00:00:00 2001 From: Asger Geel Weirsoee Date: Sun, 1 Mar 2026 12:21:18 +0000 Subject: [PATCH] feat(spa): add Angular API client for health and session read --- frontend/src/api/angular-client.ts | 62 +++++++++++++++ frontend/tests/angular-api-client.test.ts | 92 +++++++++++++++++++++++ 2 files changed, 154 insertions(+) create mode 100644 frontend/src/api/angular-client.ts create mode 100644 frontend/tests/angular-api-client.test.ts diff --git a/frontend/src/api/angular-client.ts b/frontend/src/api/angular-client.ts new file mode 100644 index 0000000..add41c1 --- /dev/null +++ b/frontend/src/api/angular-client.ts @@ -0,0 +1,62 @@ +import type { ApiFailure, ApiResult, HealthResponse, SessionDetailResponse } from './types'; + +export interface AngularHttpError { + status?: number; + message?: string; + error?: unknown; +} + +export interface AngularHttpClientLike { + get(url: string, options?: { withCredentials?: boolean }): Promise; +} + +export interface AngularApiClient { + health(): Promise>; + getSession(code: string): Promise>; +} + +function toFailure(error: unknown): ApiFailure { + const candidate = (error ?? {}) as AngularHttpError; + const status = typeof candidate.status === 'number' ? candidate.status : 0; + const payload = candidate.error; + + if (status === 0) { + return { + kind: 'network', + status: 0, + message: candidate.message ?? 'Network error while contacting API' + }; + } + + return { + kind: 'http', + status, + message: candidate.message ?? `HTTP ${status}`, + ...(payload === undefined ? {} : { payload }) + }; +} + +function normalizeCode(code: string): string { + return code.trim().toUpperCase(); +} + +async function wrapGet(call: () => Promise): Promise> { + try { + const data = await call(); + return { ok: true, status: 200, data }; + } catch (error: unknown) { + return { ok: false, status: typeof (error as AngularHttpError)?.status === 'number' ? (error as AngularHttpError).status! : 0, error: toFailure(error) }; + } +} + +export function createAngularApiClient(http: AngularHttpClientLike, baseUrl = ''): AngularApiClient { + return { + health: () => wrapGet(() => http.get(`${baseUrl}/healthz`, { withCredentials: true })), + getSession: (code: string) => + wrapGet(() => + http.get(`${baseUrl}/lobby/sessions/${encodeURIComponent(normalizeCode(code))}`, { + withCredentials: true + }) + ) + }; +} diff --git a/frontend/tests/angular-api-client.test.ts b/frontend/tests/angular-api-client.test.ts new file mode 100644 index 0000000..1020362 --- /dev/null +++ b/frontend/tests/angular-api-client.test.ts @@ -0,0 +1,92 @@ +import { describe, expect, it, vi } from 'vitest'; + +import { createAngularApiClient, type AngularHttpClientLike } from '../src/api/angular-client'; + +describe('createAngularApiClient', () => { + it('reads health and session detail using Django-compatible endpoints', async () => { + const get = vi.fn(async (url: string) => { + if (url === '/healthz') { + return { ok: true, service: 'partyhub' } as T; + } + + if (url === '/lobby/sessions/ABCD12') { + return { + session: { code: 'ABCD12', status: 'lobby', host_id: 1, current_round: 1, players_count: 2 }, + players: [ + { id: 2, nickname: 'Maja', score: 0, is_connected: true }, + { id: 3, nickname: 'Bo', score: 0, is_connected: false } + ], + round_question: null, + phase_view_model: { + status: 'lobby', + round_number: 1, + players_count: 2, + constraints: { + min_players_to_start: 2, + max_players_mvp: 8, + min_players_reached: true, + max_players_allowed: true + }, + host: { + can_start_round: true, + can_show_question: false, + can_mix_answers: false, + can_calculate_scores: false, + can_reveal_scoreboard: false, + can_start_next_round: false, + can_finish_game: false + }, + player: { + can_join: true, + can_submit_lie: false, + can_submit_guess: false, + can_view_final_result: false + } + } + } as T; + } + + throw { status: 404, error: { error: 'Not found' } }; + }); + + const http = { get }; + const client = createAngularApiClient(http as AngularHttpClientLike); + + const health = await client.health(); + expect(health.ok).toBe(true); + if (health.ok) { + expect(health.data.ok).toBe(true); + expect(health.data.service).toBe('partyhub'); + } + + const session = await client.getSession(' abcd12 '); + expect(session.ok).toBe(true); + if (session.ok) { + expect(session.data.session.code).toBe('ABCD12'); + expect(session.data.session.host_id).toBe(1); + expect(session.data.phase_view_model.host.can_start_round).toBe(true); + } + + expect(get).toHaveBeenNthCalledWith(1, '/healthz', { withCredentials: true }); + expect(get).toHaveBeenNthCalledWith(2, '/lobby/sessions/ABCD12', { withCredentials: true }); + }); + + it('maps HttpErrorResponse-style failures to ApiResult errors', async () => { + const http = { + get: vi.fn(async () => { + throw { status: 503, message: 'Service unavailable', error: { error: 'maintenance' } }; + }) + }; + + const client = createAngularApiClient(http as AngularHttpClientLike); + const result = await client.health(); + + expect(result.ok).toBe(false); + if (!result.ok) { + expect(result.status).toBe(503); + expect(result.error.kind).toBe('http'); + expect(result.error.payload).toEqual({ error: 'maintenance' }); + expect(result.error.message).toContain('Service unavailable'); + } + }); +}); -- 2.39.5