Implementa il ruolo allenatore nell'app

- Modello: tipo sugli slot, inRosa() al posto del filtro su attivo in tutte le
  letture della rosa di gioco (presenze, convocati, voti, palloni, classifiche,
  badge, sollecito); gli allenatori restano solo nei compleanni e in Squadra.
- Permessi: user_roles letto una volta per tutti i ruoli; usePuoGestireEventi()
  apre /eventi e il sollecito presenze anche all'allenatore, lato server con
  richiediGestoreEventi(). Palloni, sondaggio e messaggi restano all'admin.
- Identità: useGiocatoreInCampo() è null per l'allenatore, così non risponde alle
  presenze e non vota; useSonoAllenatore() decide cosa si vede.
- Profilo: per l'allenatore solo Docs (nome, cognome, nascita, telefono, email) e
  Opzioni; card avatar e tab Opzioni estratte per riusarle.
- Home, Squadra e partita: niente statistiche personali, badge, cacche e
  obiettivi per l'allenatore; in Rosa compare con la dicitura «Allenatore».
- Admin: tipo in «Aggiungi», allenatori a parte in Profili, fuori da conteggi ed
  export CSI.

Test unit su rosa di gioco e completamento, route test sull'allenatore.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This commit is contained in:
davideandClaude Opus 5.5 committed 2026-09-24 14:54:35 +02:00
1 parent 15a010855e
commit 2ec72fd08f
35 files changed
+1215 -577

No files matched your search

+30
View File
@@ -66,8 +66,19 @@ if (!locale) {
body: JSON.stringify({ user_id: idAdmin, role: "admin" }),
});
// Il ruolo allenatore lo scrive il trigger di M21 al collegamento dello slot; qui basta
// la riga in `user_roles`, che è ciò che la route legge (DD-034).
const emailAllenatore = `test-route-allenatore-${Date.now()}@example.test`;
const idAllenatore = await creaUtente(emailAllenatore);
await fetch(`${SUPABASE}/rest/v1/user_roles`, {
method: "POST",
headers: { ...authAdmin, "content-type": "application/json" },
body: JSON.stringify({ user_id: idAllenatore, role: "allenatore" }),
});
const tokenGiocatore = await accedi(emailGiocatore);
const tokenAdmin = await accedi(emailAdmin);
const tokenAllenatore = await accedi(emailAllenatore);
const server = await avviaServer();
console.log(`permessi route su ${server.baseUrl} (database ${SUPABASE})`);
@@ -108,6 +119,20 @@ if (!locale) {
}
});
await prova(
"l'allenatore sollecita le presenze ma non manda palloni né sondaggio",
async () => {
const sollecito = await chiama("/api/public/sollecita-presenze", {
authorization: `Bearer ${tokenAllenatore}`,
});
assert.equal(sollecito.status, 404, "sollecito: superato l'accesso, evento inesistente");
for (const percorso of ["/api/public/apri-sondaggio", "/api/public/promemoria-palloni"]) {
const res = await chiama(percorso, { authorization: `Bearer ${tokenAllenatore}` });
assert.equal(res.status, 403, `${percorso}: restano all'admin`);
}
},
);
/** Messaggio libero: stesso controllo d'accesso, corpo diverso. */
const chiamaMessaggio = (intestazioni: Record<string, string> = {}) =>
fetch(`${server.baseUrl}/api/public/notifica-personalizzata`, {
@@ -125,6 +150,11 @@ if (!locale) {
assert.equal(res.status, 403);
});
await prova("l'allenatore non manda messaggi personalizzati", async () => {
const res = await chiamaMessaggio({ authorization: `Bearer ${tokenAllenatore}` });
assert.equal(res.status, 403);
});
await prova("un amministratore manda il messaggio (nessun dispositivo iscritto)", async () => {
const res = await chiamaMessaggio({ authorization: `Bearer ${tokenAdmin}` });
assert.equal(res.status, 200);
+64 -1
View File
@@ -1,10 +1,14 @@
/** Check dell'anagrafica squadra: `bun test/unit/giocatori-squadra.test.ts`. */
import assert from "node:assert/strict";
import {
daRigaSquadra,
inRosa,
isAllenatore,
nomeCompleto,
numeroGiaUsato,
prossimoIdGiocatore,
rosaFallback,
ruoloVisibile,
slotDi,
slotPerEmail,
validaDatiSquadra,
@@ -24,6 +28,7 @@ const riga = (parziale: Partial<GiocatoreSquadra> = {}): GiocatoreSquadra => ({
numeroTessera: null,
dataTessera: null,
nascita: null,
tipo: "giocatore",
...parziale,
});
@@ -92,7 +97,14 @@ assert.equal(
assert.equal(slotPerEmail(rosaEmail, ""), null, "stringa vuota trattata come nessuna email");
// --- validaDatiSquadra ----------------------------------------------------------------
const datiOk = { nome: "Mario", cognome: "Rossi", numero: 7, ruolo: "schiacciatore", email: null };
const datiOk = {
nome: "Mario",
cognome: "Rossi",
numero: 7,
ruolo: "schiacciatore",
email: null,
tipo: "giocatore" as const,
};
assert.equal(validaDatiSquadra(datiOk), null, "dati validi: nessun errore");
assert.match(validaDatiSquadra({ ...datiOk, nome: "" })!, /nome/i);
assert.match(
@@ -155,5 +167,56 @@ assert.equal(
"conta solo il 7 attivo di g1, non serve escludere g3",
);
assert.equal(numeroGiaUsato(rosaNumeri, "g4", 11), false, "un numero libero non risulta usato");
assert.equal(
numeroGiaUsato([riga({ id: "g5", numero: 0, tipo: "allenatore" })], "g4", 0),
false,
"l'allenatore non occupa numeri di maglia",
);
// --- allenatore (DD-034) ---------------------------------------------------------
const allenatore = riga({ id: "g20", numero: 0, ruolo: "", tipo: "allenatore" });
assert.equal(isAllenatore(allenatore), true);
assert.equal(isAllenatore(riga()), false);
assert.equal(isAllenatore(null), false, "senza slot non si è allenatori");
assert.equal(inRosa(riga()), true, "giocatore attivo: in rosa");
assert.equal(inRosa(riga({ attivo: false })), false, "giocatore disattivato: fuori");
assert.equal(inRosa(allenatore), false, "l'allenatore non è nella rosa di gioco");
assert.equal(ruoloVisibile(allenatore), "Allenatore", "dove c'è il ruolo in campo");
assert.equal(ruoloVisibile(riga({ ruolo: "Libero" })), "Libero");
assert.equal(
validaDatiSquadra({ ...datiOk, tipo: "allenatore", numero: 0, ruolo: "" }),
null,
"l'allenatore non ha numero né ruolo",
);
assert.match(
validaDatiSquadra({ ...datiOk, tipo: "allenatore", nome: " " })!,
/nome/i,
"il nome resta obbligatorio anche per l'allenatore",
);
assert.match(
validaDatiSquadra({ ...datiOk, tipo: "allenatore", email: "senza-chiocciola" })!,
/email/i,
);
const rigaDb = {
id: "g20",
nome: "Carla",
cognome: "Neri",
numero: null,
ruolo: "",
auth_user_id: null,
attivo: true,
email: "carla@gmail.com",
numero_tessera: null,
data_tessera: null,
nascita: null,
tipo: "allenatore" as const,
};
assert.equal(daRigaSquadra(rigaDb).numero, 0, "numero NULL a database diventa 0");
assert.equal(daRigaSquadra(rigaDb).tipo, "allenatore");
assert.equal(daRigaSquadra({ ...rigaDb, numero: 7, tipo: "giocatore" }).numero, 7);
assert.ok(
rosaFallback().every((g) => g.tipo === "giocatore"),
"il fallback contiene solo giocatori",
);
console.log("giocatori-squadra: ok");
+6 -5
View File
@@ -120,11 +120,12 @@ assert.equal(totaliEventiGiocatore("g1", conRistretti, OGGI), 5, "chi non è con
// --- destinatari del sollecito: chi non ha risposto, più i "forse" ------------
const squadra = [
{ id: "g1", attivo: true },
{ id: "g2", attivo: true },
{ id: "g3", attivo: true },
{ id: "g4", attivo: true },
{ id: "g5", attivo: false }, // uscito dalla squadra: non lo si disturba più
{ id: "g1", attivo: true, tipo: "giocatore" as const },
{ id: "g2", attivo: true, tipo: "giocatore" as const },
{ id: "g3", attivo: true, tipo: "giocatore" as const },
{ id: "g4", attivo: true, tipo: "giocatore" as const },
{ id: "g5", attivo: false, tipo: "giocatore" as const }, // uscito: non lo si disturba più
{ id: "g6", attivo: true, tipo: "allenatore" as const }, // non risponde: mai sollecitato (DD-034)
];
const risposte = [
+31 -1
View File
@@ -3,6 +3,7 @@ import assert from "node:assert/strict";
import {
aRigaProfilo,
completamento,
completamentoAllenatore,
csvTesseramento,
daRigaProfilo,
sezioniComplete,
@@ -115,6 +116,7 @@ const squadra: GiocatoreSquadra[] = [
numeroTessera: null,
dataTessera: null,
nascita: null,
tipo: "giocatore",
},
{
id: "g2",
@@ -128,6 +130,7 @@ const squadra: GiocatoreSquadra[] = [
numeroTessera: null,
dataTessera: null,
nascita: null,
tipo: "giocatore",
},
];
const csv = csvTesseramento(squadra, { g1: completo });
@@ -155,7 +158,14 @@ assert.equal(slotDi(squadra, "u2")?.id, "g2");
assert.equal(slotDi(squadra, "sconosciuto"), null);
// --- dati squadra modificabili dall'admin (DD-017) ---------------------------
const datiOk = { nome: "Ivan", cognome: "Cacciari", numero: 23, ruolo: "Banda", email: null };
const datiOk = {
nome: "Ivan",
cognome: "Cacciari",
numero: 23,
ruolo: "Banda",
email: null,
tipo: "giocatore" as const,
};
assert.equal(validaDatiSquadra(datiOk), null);
assert.match(validaDatiSquadra({ ...datiOk, nome: " " }) ?? "", /nome/i);
assert.match(validaDatiSquadra({ ...datiOk, cognome: "" }) ?? "", /cognome/i);
@@ -184,4 +194,24 @@ assert.ok(
"il fallback non può collegare account",
);
// --- completamento dell'allenatore (DD-034) --------------------------------------
assert.equal(completamentoAllenatore(null), 0, "senza profilo: 0%");
assert.equal(
completamentoAllenatore({ ...vuoto, dataNascita: "1980-01-01", telefono: "333" }),
50,
"due campi su quattro",
);
assert.equal(
completamentoAllenatore({
...vuoto,
dataNascita: "1980-01-01",
luogoNascita: "Bologna",
telefono: "333",
email: "a@b.it",
}),
100,
"bastano i quattro dati personali: niente indirizzo, documento, certificato, foto",
);
assert.equal(completamentoAllenatore({ ...vuoto, telefono: " " }), 0, "spazi soli non contano");
console.log("profili-core: ok");