longevity: revisione finale — il motore ora si calcola, il consenso è atomico, il codice cliente è tipato fuori dal corpo
Sei correzioni dalla revisione finale del 22/08, ognuna con test scritto prima: 1. salvaDalForm calcolava mai i punteggi (calcolaSessione/salvaScore) dopo il salvataggio: un cliente compilava venti domande e vedeva un referto vuoto. Ora avviene nella stessa transazione del salvataggio. 2. Il corpo della richiesta dell'endpoint questionario è tipato (CorpoQuestionario): client_code non esiste in quel tipo, non è più prendibile per errore. Il vecchio test a lista di nomi vietati era vacuo (cercava body./corpo., il file usa data.) — sostituito con un test comportamentale end-to-end sull'endpoint vero. 3. Il consenso: aggiunto il test sul ramo affermativo (non coperto, vacuo quanto il punto 2); la nota di consenso ora è nella stessa transazione dei dati sanitari; ha una versione propria (CONSENT_VERSION) invece di quella del questionario. 4. La prima apertura del referto ora spiega invece di sembrare rotta: Fitness Age con nota quando manca, spiegazione del radar prima del grafico, copertura sempre visibile, invito a fare i test in sala col trainer, numeri formattati all'italiana. 5. Aggiunto il link per ricompilare il questionario nella pagina del referto (prima spariva appena esisteva una compilazione). 6. Il test anti-query ora scandisce tutte le pagine sotto src/pages/longevity/, non due nomi a mano. Il test anti-giunzione identity/longevity resta con lo stesso limite di sempre (createIdentityDb è esportata per creaCliente) — corretto solo il commento perché lo dichiari onestamente. Report con comandi e output letterali: .superpowers/sdd/2026-08-22-longevity-interfaccia/review-finale-report.md
This commit is contained in:
@@ -1,6 +1,6 @@
|
||||
// tests/longevity/referto.test.ts
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { readFileSync } from 'node:fs';
|
||||
import { readFileSync, readdirSync } from 'node:fs';
|
||||
import { join } from 'node:path';
|
||||
import { createLongevityDb } from '../../src/lib/longevity/db';
|
||||
import { seedRegistro, seedPesi, MODEL_VERSION } from '../../src/lib/longevity/registro';
|
||||
@@ -59,6 +59,18 @@ describe('il referto del cliente', () => {
|
||||
expect(forza.mancano.length).toBeGreaterThan(0);
|
||||
});
|
||||
|
||||
it('espone la copertura della Fitness Age anche quando è insufficiente (serve alla nota in pagina)', () => {
|
||||
// conCompilazione() non porta nessuna misura del composito Fitness Age (cardio,
|
||||
// handgrip, hrv, forza_resto, composizione, stabilita: nessuno di questi sotto-domini
|
||||
// ha dati in questa sessione, solo il questionario), quindi fitnessAge è null — ma
|
||||
// PRIMA di questa correzione refertoDi scartava anche la copertura insieme al valore:
|
||||
// la pagina non aveva modo di dire "41%", solo di dire "manca".
|
||||
const { db, s } = conCompilazione();
|
||||
const referto = refertoDi(db, s);
|
||||
expect(referto.fitnessAge).toBeNull();
|
||||
expect(referto.fitnessAgeCopertura).toBe(0);
|
||||
});
|
||||
|
||||
it('un asse coperto porta il suo numero', () => {
|
||||
const { db, s } = conCompilazione();
|
||||
const stile = refertoDi(db, s).assi.find((a) => a.nome === 'Stile di Vita & Sonno')!;
|
||||
@@ -95,9 +107,18 @@ describe('il referto del cliente', () => {
|
||||
});
|
||||
|
||||
it('nessuna pagina interroga il database da sola', () => {
|
||||
for (const f of ['src/pages/longevity/io.astro', 'src/pages/longevity/questionario.astro']) {
|
||||
const src = readFileSync(join(process.cwd(), f), 'utf8');
|
||||
expect(src, `${f} contiene SQL`).not.toMatch(/\bSELECT\b|\.prepare\(/i);
|
||||
// Scandisce TUTTE le pagine sotto src/pages/longevity/, non un elenco scritto qui:
|
||||
// un elenco di due nomi non protegge una pagina nuova, che passerebbe indisturbata
|
||||
// anche con una query dentro (verificato: una prova-query.astro con un
|
||||
// db.prepare('SELECT 1') non veniva vista dalla versione precedente di questo test).
|
||||
const dir = join(process.cwd(), 'src/pages/longevity');
|
||||
const pagine = (readdirSync(dir, { recursive: true }) as string[])
|
||||
.filter((f) => f.endsWith('.astro'));
|
||||
expect(pagine.length).toBeGreaterThan(0); // se la cartella si svuota, il test non deve tacere per errore
|
||||
for (const f of pagine) {
|
||||
const percorso = `src/pages/longevity/${f}`;
|
||||
const src = readFileSync(join(dir, f), 'utf8');
|
||||
expect(src, `${percorso} contiene SQL`).not.toMatch(/\bSELECT\b|\.prepare\(/i);
|
||||
}
|
||||
});
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user