Compare commits

11 Commits

Author SHA1 Message Date
davide ff6a305554 Merge branch 'update-striscia-18' (#18)
Registra cambi e time out per set nella striscia, ristilizza il
referto in stile FIPAV e porta la copertura test dal 76% al 99%.
2026-07-03 12:09:06 +02:00
davide 82c2286ca6 docs: aggiorna README, tests/README e CLAUDE.md con la nuova copertura test
Riflette i test aggiunti nei commit precedenti: rilevamento
piattaforma di server-utils, avvio effettivo di startServer, rami di
errore del WebSocket handler, cleanup/riconnessione di wsMixin,
side-effect di generaReferto e layout esteso del controller.
Aggiorna anche i conteggi di esempio (15 file/362 test Vitest, 81
E2E) e documenta come verificare la copertura (>99% statement).
2026-07-03 12:04:17 +02:00
davide 0b7cbd00f6 test(component): porta ControllerPage e DisplayPage al 100% di statement coverage
Aggiunge 65 test per ControllerPage.vue (layout esteso/landscape,
popup automatico di set vinto con soglia decisiva, modal di
configurazione, validazione completa dei cambi giocatore, modal
time out con selezione video/ticker/countdown, cleanup dei listener)
e 35 per DisplayPage.vue (countdown overlay, riallineamento video su
riconnessione, fallback muto per autoplay bloccato, selezione voce
per la sintesi vocale). Nessun bug riscontrato nei componenti.
2026-07-03 12:00:45 +02:00
davide 6638d6fe44 test(server): copri l'avvio effettivo di startServer (HTTP + WebSocket)
startServer() non era mai invocato nei test: solo createApp() (routing)
era coperto. Mocka src/persist.js per non toccare .segnapunti/state.json
reale, e verifica l'ascolto HTTP, l'accettazione dell'upgrade su /ws,
il rifiuto su altri percorsi e la persistenza dello stato dopo
un'azione del controller.
2026-07-03 12:00:30 +02:00
davide cbf927255a test(wsMixin): copri cleanup, riconnessione e guardie residue
Aggiunge test per beforeUnmount con connessione OPEN e con ws.close
che lancia, l'annullamento del reconnectTimeout allo smontaggio, la
guardia isConnecting, la chiusura della WebSocket precedente prima
di riconnettersi (incluso il caso in cui close lancia), gli errori
di registrazione/parsing nei gestori onopen/onmessage e la guardia
di scheduleReconnect contro timer doppi.
2026-07-03 12:00:16 +02:00
davide 3d54d162e6 test(websocket-handler): copri i rami di errore su send/terminate/applyAction
Aggiunge scenari in cui ws.send o ws.terminate lanciano eccezioni e
in cui applyAction fallisce a metà azione: verifica che l'errore
venga loggato e lo stato precedente ripristinato senza propagare
l'eccezione al chiamante.
2026-07-03 12:00:02 +02:00
davide eb91b71d7e test(server-utils): copri il rilevamento piattaforma (WSL/PowerShell)
getNetworkIPs() senza parametri usa isWSL()/execSync/os.networkInterfaces,
codice mai esercitato perché tutti gli altri test iniettano `nets`
per bypassarlo. Mocka fs/child_process/os per coprire in modo
deterministico i rami WSL, non-WSL, fallimento PowerShell e lettura
/proc fallita.
2026-07-03 11:59:49 +02:00
davide 2625284e99 test(referto): copri generaReferto e il branch guest di vincitoreSet
buildRefertoHtml mancava il caso vinc nullo con guest in vantaggio;
generaReferto (window.open/print) non era testato affatto perché
richiede un DOM — aggiunto un test component in happy-dom che mocka
window.open.
2026-07-03 11:59:33 +02:00
davide fdd1d0c990 test(gameState): copri i branch residui di nuovoSet/confermaCambi/resetta/startTimeout
Aggiunge casi per team 'guest' in nuovoSet, cambi senza array
`cambi`, resetta con striscia vuota e startTimeout senza startedAt:
gameState.js passa dal 100% al 100% statement ma sale al 99% branch.
2026-07-03 11:59:19 +02:00
davide 5bdd8ad986 style(referto): riprendi il layout del modulo ufficiale FIPAV
Cornice nera, barre teal "SET N" per set, badge A/B per squadre e
box risultato finale in stile tabella, per avvicinare il referto
stampabile all'aspetto del referto cartaceo FIPAV mantenendo solo
i dati che l'app traccia realmente.
2026-07-03 11:31:57 +02:00
davide f6f3dd11d4 feat(striscia): registra cambi e time out per set (#18)
La striscia salva ora storico cambi formazione e time out per ogni
set, con riferimento al punteggio al momento dell'evento anziché a
un timestamp: startTimeout accumula in set.timeouts, confermaCambi
in set.cambi. Il referto elenca entrambi ordinati per punteggio.
2026-07-03 11:16:47 +02:00
15 changed files with 1630 additions and 53 deletions
+7 -1
View File
@@ -40,11 +40,15 @@ Display (Vue) ──WebSocket──┤── websocket-handler.js ── gam
Punteggio, set e servizio si ricavano sempre dalla `striscia`: al primo punto di ogni set `applyAction('incPunt')` salva in `formInizio` uno snapshot della formazione di partenza (usato dal referto); `decPunt` lo rimuove se il set torna a 0-0. Punteggio, set e servizio si ricavano sempre dalla `striscia`: al primo punto di ogni set `applyAction('incPunt')` salva in `formInizio` uno snapshot della formazione di partenza (usato dal referto); `decPunt` lo rimuove se il set torna a 0-0.
Ogni voce della `striscia` accumula anche lo storico di cambi e time out del set, popolato lazy (nessun campo se il set non ha eventi), senza timestamp — il "quando" è il punteggio al momento dell'evento, non l'orologio:
- `set.timeouts`: array di `{ team, punteggio: { home, guest } }`, appeso da `startTimeout` (solo se il time out non è già attivo, per non duplicare l'evento su un restart).
- `set.cambi`: array di `{ team, in, out, punteggio }`, uno per ogni sostituzione effettivamente applicata da `confermaCambi` (nulla viene registrato se la validazione del cambio fallisce).
`src/websocket-handler.js` riceve i messaggi WebSocket, valida che il mittente sia un `controller` (non un `display`), chiama `applyAction`, fa il broadcast del nuovo stato a tutti i client, poi invoca `onStateChange` per persistere su disco. `src/websocket-handler.js` riceve i messaggi WebSocket, valida che il mittente sia un `controller` (non un `display`), chiama `applyAction`, fa il broadcast del nuovo stato a tutti i client, poi invoca `onStateChange` per persistere su disco.
`src/wsMixin.js` è il mixin Vue lato client: gestisce connessione/riconnessione WebSocket (backoff esponenziale) ed espone i computed `punt`/`servHome`/`set`. `src/wsMixin.js` è il mixin Vue lato client: gestisce connessione/riconnessione WebSocket (backoff esponenziale) ed espone i computed `punt`/`servHome`/`set`.
`src/referto.js` genera il referto di gara: `buildRefertoHtml(state, now)` è una funzione pura che restituisce l'HTML (testabile), mentre `generaReferto(state)` è il wrapper che apre la finestra con `window.open` e avvia la stampa. `src/referto.js` genera il referto di gara: `buildRefertoHtml(state, now)` è una funzione pura che restituisce l'HTML (testabile), mentre `generaReferto(state)` è il wrapper che apre la finestra con `window.open` e avvia la stampa. Per ogni set, oltre alla formazione di partenza e alla griglia punti, mostra (se presenti) l'elenco di `set.timeouts` e `set.cambi` ordinato per punteggio crescente.
`src/persist.js` carica/salva lo stato su `.segnapunti/state.json`. `src/server-utils.js` ricava gli IP di rete (`getNetworkIPs`, con sorgente iniettabile) e stampa gli URL all'avvio. `src/spot-utils.js` espone `listSpotVideos(spotDir)`, che elenca i `.mp4` (ordinati) della cartella `spot/`; ritorna `[]` se la cartella manca. `src/persist.js` carica/salva lo stato su `.segnapunti/state.json`. `src/server-utils.js` ricava gli IP di rete (`getNetworkIPs`, con sorgente iniettabile) e stampa gli URL all'avvio. `src/spot-utils.js` espone `listSpotVideos(spotDir)`, che elenca i `.mp4` (ordinati) della cartella `spot/`; ritorna `[]` se la cartella manca.
@@ -74,6 +78,8 @@ In sviluppo, `vite-plugin-websocket.js` incorpora il server WebSocket dentro il
I test E2E girano in serie (`workers: 1`) per evitare race condition sullo stato WebSocket. Eseguire `npm run serve` prima di `npm run test:e2e`. I test E2E girano in serie (`workers: 1`) per evitare race condition sullo stato WebSocket. Eseguire `npm run serve` prima di `npm run test:e2e`.
La copertura statement/linee della suite Vitest è oltre il 99%. Per testare `startServer()` (`server.js`) o i rami platform-dependent di `getNetworkIPs()` (`server-utils.js`) senza effetti collaterali reali (scrittura su `.segnapunti/state.json`, dipendenza dalla piattaforma), si mocka il modulo con `vi.mock`/`vi.doMock` (vedi `tests/integration/startServer.test.js` e `tests/unit/server-utils-platform.test.js`) invece di eseguire il codice reale.
## Deploy ## Deploy
Il progetto si distribuisce tramite Docker. L'immagine è pubblicata sul registry Gitea self-hosted (`santantonio.sytes.net`): Il progetto si distribuisce tramite Docker. L'immagine è pubblicata sul registry Gitea self-hosted (`santantonio.sytes.net`):
+6 -2
View File
@@ -242,8 +242,12 @@ Lo stato viene salvato in `.segnapunti/state.json` anche in modalità dev.
| `npm run test:e2e` | Playwright — Chromium, Firefox, Mobile Chrome | | `npm run test:e2e` | Playwright — Chromium, Firefox, Mobile Chrome |
Le suite Vitest coprono la logica di gioco (`gameState`, inclusa `formInizio`), Le suite Vitest coprono la logica di gioco (`gameState`, inclusa `formInizio`),
il protocollo e il routing del server, la persistenza, il client WebSocket il protocollo e il routing del server (incluso l'avvio effettivo di
(`wsMixin`), i componenti Vue e la generazione del referto. `startServer`), la persistenza, il rilevamento di rete/piattaforma
(`server-utils`, inclusi i rami WSL/PowerShell), il client WebSocket
(`wsMixin`, connessione/riconnessione/cleanup), i componenti Vue e la
generazione del referto (inclusi i suoi side-effect di stampa). La copertura
statement complessiva è oltre il 99%.
> I test E2E richiedono il server in esecuzione (`npm run serve`) e i browser Playwright installati: > I test E2E richiedono il server in esecuzione (`npm run serve`) e i browser Playwright installati:
> ```bash > ```bash
+16
View File
@@ -171,6 +171,13 @@ export function applyAction(state, action) {
case "startTimeout": { case "startTimeout": {
const team = action.team const team = action.team
if (team !== 'home' && team !== 'guest') break if (team !== 'home' && team !== 'guest') break
if (!s.timeout) {
const setCorrente = s.sp.striscia.at(-1)
;(setCorrente.timeouts ??= []).push({
team,
punteggio: punteggio(s.sp.striscia),
})
}
s.timeout = true s.timeout = true
s.timeoutTeam = team s.timeoutTeam = team
s.timeoutVideo = action.video ?? null s.timeoutVideo = action.video ?? null
@@ -209,6 +216,7 @@ export function applyAction(state, action) {
const cambi = action.cambi || [] const cambi = action.cambi || []
const form = s.sp.form[team].map((val) => String(val).trim()) const form = s.sp.form[team].map((val) => String(val).trim())
const formAggiornata = [...form] const formAggiornata = [...form]
const cambiApplicati = []
let valid = true let valid = true
for (const cambio of cambi) { for (const cambio of cambi) {
@@ -223,11 +231,19 @@ export function applyAction(state, action) {
const idx = formAggiornata.findIndex((val) => String(val).trim() === cout) const idx = formAggiornata.findIndex((val) => String(val).trim() === cout)
if (idx !== -1) { if (idx !== -1) {
formAggiornata.splice(idx, 1, cin) formAggiornata.splice(idx, 1, cin)
cambiApplicati.push({ in: cin, out: cout })
} }
} }
if (valid) { if (valid) {
s.sp.form[team] = formAggiornata s.sp.form[team] = formAggiornata
if (cambiApplicati.length > 0) {
const setCorrente = s.sp.striscia.at(-1)
const puntAttuale = punteggio(s.sp.striscia)
;(setCorrente.cambi ??= []).push(
...cambiApplicati.map(c => ({ team, ...c, punteggio: { ...puntAttuale } }))
)
}
} }
break break
} }
+119 -37
View File
@@ -44,6 +44,31 @@ export function buildRefertoHtml(state, now = new Date()) {
</div>` </div>`
} }
const nomeTeam = (team) => team === 'home' ? nomi.home : nomi.guest
const eventiHtml = (s) => {
const eventi = [
...(s.timeouts ?? []).map(t => ({ tipo: 'timeout', ...t })),
...(s.cambi ?? []).map(c => ({ tipo: 'cambio', ...c })),
].sort((a, b) => (a.punteggio.home + a.punteggio.guest) - (b.punteggio.home + b.punteggio.guest))
if (eventi.length === 0) return ''
const righe = eventi.map(e => {
const sul = `sul ${e.punteggio.home}-${e.punteggio.guest}`
const testo = e.tipo === 'timeout'
? `Time out <strong>${nomeTeam(e.team)}</strong> ${sul}`
: `Cambio <strong>${nomeTeam(e.team)}</strong>: entra ${e.in} per ${e.out} ${sul}`
return `<li class="evento evento-${e.tipo}">${testo}</li>`
}).join('')
return `
<div class="eventi-set">
<div class="eventi-label">Cambi e time out</div>
<ul class="eventi-lista">${righe}</ul>
</div>`
}
const setsHtml = setReali.map((s, i) => { const setsHtml = setReali.map((s, i) => {
let h = 0, g = 0 let h = 0, g = 0
const punti = [] const punti = []
@@ -62,17 +87,31 @@ export function buildRefertoHtml(state, now = new Date()) {
return ` return `
<div class="set-section"> <div class="set-section">
<div class="set-header"> <div class="set-barra"><span>SET</span><span class="set-numero">${i + 1}</span></div>
Set ${i + 1} &nbsp;—&nbsp; ${nomi.home} <strong>${h}</strong> · <strong>${g}</strong> ${nomi.guest}${etichettaVinc} <div class="set-corpo">
<div class="set-header">
${nomi.home} <strong>${h}</strong> · <strong>${g}</strong> ${nomi.guest}${etichettaVinc}
</div>
<div class="form-inizio">
<div class="form-inizio-label">Formazione di partenza</div>
${formazioneHtml(s.formInizio)}
</div>${eventiHtml(s)}
<div class="punti-grid">${puntiHtml || '<em style="color:#999;font-size:11px">Nessun punto registrato</em>'}</div>
</div> </div>
<div class="form-inizio">
<div class="form-inizio-label">Formazione di partenza</div>
${formazioneHtml(s.formInizio)}
</div>
<div class="punti-grid">${puntiHtml || '<em style="color:#999;font-size:11px">Nessun punto registrato</em>'}</div>
</div>` </div>`
}).join('') }).join('')
const risultatoSetHtml = setReali.map((s, i) => {
let h = 0, g = 0
for (const c of s.ris) c === 'h' ? h++ : g++
const vinc = vincitoreSet(s)
return `<tr>
<td class="ris-set-n">${i + 1}</td>
<td class="ris-set-p ${vinc === 'h' ? 'ris-vinc' : ''}">${h}</td>
<td class="ris-set-p ${vinc === 'g' ? 'ris-vinc' : ''}">${g}</td>
</tr>`
}).join('')
const html = `<!DOCTYPE html> const html = `<!DOCTYPE html>
<html lang="it"> <html lang="it">
<head> <head>
@@ -80,50 +119,93 @@ export function buildRefertoHtml(state, now = new Date()) {
<title>Referto — ${nomi.home} vs ${nomi.guest}</title> <title>Referto — ${nomi.home} vs ${nomi.guest}</title>
<style> <style>
* { box-sizing: border-box; margin: 0; padding: 0; } * { box-sizing: border-box; margin: 0; padding: 0; }
body { font-family: Arial, Helvetica, sans-serif; color: #111; background: #fff; padding: 28px; font-size: 14px; } :root { --teal: #23b8dc; --teal-dark: #0f8faf; }
@media print { body { padding: 12px; } } body { font-family: Arial, Helvetica, sans-serif; color: #111; background: #fff; padding: 20px; font-size: 14px; }
@media print { body { padding: 8px; } }
.header { text-align: center; border-bottom: 2px solid #111; padding-bottom: 12px; margin-bottom: 20px; } .foglio { border: 3px solid #111; padding: 14px; }
.titolo { font-size: 18px; font-weight: bold; letter-spacing: 3px; text-transform: uppercase; }
.meta { font-size: 12px; color: #666; margin-top: 5px; }
.squadre-row { display: flex; justify-content: space-around; align-items: center; margin: 18px 0 6px; } .header { display: flex; align-items: center; justify-content: space-between; gap: 16px; border: 2px solid #111; padding: 10px 16px; margin-bottom: 14px; }
.nome-sq { font-size: 22px; font-weight: bold; text-align: center; max-width: 40%; } .titolo { font-size: 15px; font-weight: bold; letter-spacing: 2px; text-transform: uppercase; }
.vs { font-size: 16px; color: #999; } .meta { font-size: 11px; color: #333; margin-top: 3px; text-transform: uppercase; letter-spacing: 0.5px; }
.risultato { text-align: center; font-size: 36px; font-weight: bold; letter-spacing: 6px; margin-bottom: 4px; }
.modalita-label { text-align: center; font-size: 12px; color: #888; margin-bottom: 22px; }
.set-section { margin-bottom: 12px; border: 1px solid #ddd; border-radius: 4px; overflow: hidden; } .squadre-row { display: flex; align-items: center; gap: 14px; }
.set-header { background: #f5f5f5; padding: 7px 12px; font-size: 13px; border-bottom: 1px solid #ddd; } .badge-ab { display: inline-flex; align-items: center; justify-content: center; width: 22px; height: 22px; border-radius: 50%; border: 2px solid #111; font-weight: bold; font-size: 12px; flex-shrink: 0; }
.punti-grid { display: flex; flex-wrap: wrap; gap: 3px; padding: 8px 10px; } .nome-sq { font-size: 18px; font-weight: bold; }
.punto { display: inline-block; padding: 2px 5px; border-radius: 3px; font-size: 11px; font-family: 'Courier New', monospace; white-space: nowrap; } .vs { font-size: 12px; color: #666; font-weight: bold; }
.risultato-riepilogo { display: flex; align-items: center; gap: 10px; }
.risultato { font-size: 30px; font-weight: bold; letter-spacing: 3px; }
.modalita-label { font-size: 10px; color: #555; text-transform: uppercase; letter-spacing: 0.5px; }
.set-section { display: flex; margin-bottom: 10px; border: 2px solid #111; }
.set-barra { flex: 0 0 26px; background: var(--teal); color: #fff; display: flex; flex-direction: column; align-items: center; justify-content: center; gap: 4px; border-right: 2px solid #111; }
.set-barra span:first-child { writing-mode: vertical-rl; transform: rotate(180deg); font-weight: bold; font-size: 11px; letter-spacing: 2px; }
.set-numero { font-weight: bold; font-size: 15px; }
.set-corpo { flex: 1; min-width: 0; }
.set-header { background: #eaf7fb; padding: 6px 12px; font-size: 13px; border-bottom: 1px solid #111; }
.punti-grid { display: flex; flex-wrap: wrap; gap: 0; padding: 0; border-top: 1px solid #111; }
.punto { display: inline-flex; align-items: center; justify-content: center; min-width: 34px; padding: 3px 4px; font-size: 11px; font-family: 'Courier New', monospace; white-space: nowrap; border-right: 1px solid #ccc; border-bottom: 1px solid #ccc; }
.punto-h { background: #d0e8ff; color: #003a6e; } .punto-h { background: #d0e8ff; color: #003a6e; }
.punto-g { background: #ffddc0; color: #6e2700; } .punto-g { background: #ffddc0; color: #6e2700; }
.form-inizio { padding: 8px 10px; border-bottom: 1px solid #eee; background: #fafafa; } .eventi-set { padding: 6px 12px; border-bottom: 1px solid #111; }
.form-inizio-label { font-size: 11px; font-weight: bold; letter-spacing: 0.5px; text-transform: uppercase; color: #888; margin-bottom: 7px; } .eventi-label { font-size: 10px; font-weight: bold; letter-spacing: 0.5px; text-transform: uppercase; color: #0f8faf; margin-bottom: 4px; }
.eventi-lista { list-style: none; }
.evento { font-size: 12px; padding: 1px 0; }
.form-inizio { padding: 6px 12px; border-bottom: 1px solid #111; background: #fafafa; }
.form-inizio-label { font-size: 10px; font-weight: bold; letter-spacing: 0.5px; text-transform: uppercase; color: #0f8faf; margin-bottom: 6px; }
.form-row { display: flex; gap: 40px; } .form-row { display: flex; gap: 40px; }
.form-team { flex: 1; } .form-team { flex: 1; }
.form-team-name { font-weight: bold; font-size: 13px; margin-bottom: 6px; } .form-team-name { font-weight: bold; font-size: 12px; margin-bottom: 5px; }
.giocatori { display: flex; gap: 5px; flex-wrap: wrap; } .giocatori { display: flex; gap: 5px; flex-wrap: wrap; }
.giocatore { background: #f0f0f0; border-radius: 50%; width: 30px; height: 30px; display: flex; align-items: center; justify-content: center; font-weight: bold; font-size: 12px; border: 1px solid #ccc; } .giocatore { background: #fff; border-radius: 50%; width: 26px; height: 26px; display: flex; align-items: center; justify-content: center; font-weight: bold; font-size: 11px; border: 2px solid #111; }
.finale { display: flex; gap: 14px; align-items: flex-start; margin-top: 4px; }
.finale-box { border: 2px solid #111; flex: 0 0 220px; }
.finale-box-header { background: var(--teal); color: #fff; text-align: center; font-weight: bold; font-size: 12px; letter-spacing: 1px; text-transform: uppercase; padding: 5px; }
.finale-tabella { width: 100%; border-collapse: collapse; font-size: 12px; }
.finale-tabella th, .finale-tabella td { border: 1px solid #111; padding: 4px 8px; text-align: center; }
.finale-tabella th { background: #eaf7fb; font-size: 10px; text-transform: uppercase; }
.ris-vinc { font-weight: bold; background: #d0e8ff; }
.finale-vince { padding: 8px; text-align: center; font-size: 14px; font-weight: bold; border-top: 2px solid #111; }
</style> </style>
</head> </head>
<body> <body>
<div class="header"> <div class="foglio">
<div class="titolo">Referto di Gara</div> <div class="header">
<div class="meta">${dataOra} &nbsp;·&nbsp; Modalità: ${modalitaPartita}</div> <div>
</div> <div class="titolo">Referto di Gara</div>
<div class="meta">${dataOra} &nbsp;·&nbsp; Modalità: ${modalitaPartita}</div>
</div>
<div class="squadre-row">
<span class="badge-ab">A</span>
<span class="nome-sq">${nomi.home}</span>
<span class="vs">vs</span>
<span class="nome-sq">${nomi.guest}</span>
<span class="badge-ab">B</span>
</div>
</div>
<div class="squadre-row"> ${setsHtml}
<div class="nome-sq">${nomi.home}</div>
<div class="vs">vs</div>
<div class="nome-sq">${nomi.guest}</div>
</div>
<div class="risultato">${setVinti.home} ${setVinti.guest}</div>
<div class="modalita-label">set vinti</div>
${setsHtml} <div class="finale">
<div class="finale-box">
<div class="finale-box-header">Risultato finale</div>
<table class="finale-tabella">
<thead><tr><th>Set</th><th>A</th><th>B</th></tr></thead>
<tbody>${risultatoSetHtml}</tbody>
</table>
<div class="finale-vince">Vince ${setVinti.home >= setVinti.guest ? nomi.home : nomi.guest} &nbsp;${setVinti.home} ${setVinti.guest}</div>
</div>
<div class="risultato-riepilogo">
<div class="risultato">${setVinti.home} ${setVinti.guest}</div>
<div class="modalita-label">set vinti</div>
</div>
</div>
</div>
</body> </body>
</html>` </html>`
+28 -7
View File
@@ -94,7 +94,9 @@ Esempi reali nel progetto:
- time out (`startTimeout`/`stopTimeout`: squadra, video, timestamp) - time out (`startTimeout`/`stopTimeout`: squadra, video, timestamp)
- generazione del referto (`buildRefertoHtml`) - generazione del referto (`buildRefertoHtml`)
- persistenza stato su disco (`persist.js`, con `fs` mockato) - persistenza stato su disco (`persist.js`, con `fs` mockato)
- utility di rete (`server-utils.js`) - utility di rete (`server-utils.js`), incluso il rilevamento piattaforma
(WSL/PowerShell vs. `os.networkInterfaces()`, con `fs`/`child_process`/`os`
mockati per non dipendere dalla macchina che esegue i test)
- elenco video spot (`spot-utils.js`, su cartella temporanea reale) - elenco video spot (`spot-utils.js`, su cartella temporanea reale)
Quando falliscono: Quando falliscono:
@@ -119,6 +121,8 @@ Esempi reali nel progetto:
- autorizzazioni (solo controller puo inviare certe azioni) - autorizzazioni (solo controller puo inviare certe azioni)
- timer server-side del time out (auto-`stopTimeout` dopo 30s, `startedAt` impostato dal server) - timer server-side del time out (auto-`stopTimeout` dopo 30s, `startedAt` impostato dal server)
- routing del server Express (`createApp`: rotte display/controller/asset, `/spot/list` e file statici `/spot/`) - routing del server Express (`createApp`: rotte display/controller/asset, `/spot/list` e file statici `/spot/`)
- rami di errore del WebSocket handler (`ws.send`/`ws.terminate` che lanciano, `applyAction` che fallisce a metà azione: lo stato precedente va ripristinato senza propagare l'eccezione)
- avvio effettivo di `startServer` (`server.js`): ascolto HTTP, upgrade su `/ws`, rifiuto su altri percorsi, persistenza dopo un'azione — con `persist.js` mockato per non toccare mai `.segnapunti/state.json` reale
Quando falliscono: Quando falliscono:
- spesso c'e problema nel protocollo messaggi o nei controlli di ruolo - spesso c'e problema nel protocollo messaggi o nei controlli di ruolo
@@ -138,8 +142,9 @@ Esempi reali nel progetto:
- dialog reset/config/cambi - dialog reset/config/cambi
- modal time out del controller (scelta squadra/video, banner, countdown) - modal time out del controller (scelta squadra/video, banner, countdown)
- overlay time out del display (video, placeholder, countdown, dissolvenza) - overlay time out del display (video, placeholder, countdown, dissolvenza)
- bottone REFERTO nel dialog PARTITA FINITA - bottone REFERTO nel dialog PARTITA FINITA e generazione del referto stampabile (`generaReferto`: `window.open`, scrittura HTML, stampa)
- client WebSocket (`wsMixin`: connessione, riconnessione, `sendWs`, errori) - client WebSocket (`wsMixin`: connessione, riconnessione con backoff, cleanup allo smontaggio, `sendWs`, errori)
- layout esteso/landscape del controller (pannelli squadra, popup automatico SET VINTO con soglia decisiva)
Quando falliscono: Quando falliscono:
- spesso hai rotto template, computed, metodi o binding - spesso hai rotto template, computed, metodi o binding
@@ -190,8 +195,8 @@ Note importanti:
Caso OK (verde): Caso OK (verde):
```text ```text
Test Files 10 passed (10) Test Files 15 passed (15)
Tests 212 passed (212) Tests 362 passed (362)
``` ```
Significa: Significa:
@@ -208,7 +213,7 @@ Caso KO (rosso):
Caso OK: Caso OK:
```text ```text
75 passed 81 passed
``` ```
Caso KO: Caso KO:
@@ -279,7 +284,23 @@ npm run test:e2e -- --update-snapshots
npm run test:e2e npm run test:e2e
``` ```
## 10) Come aggiungere un nuovo test (consigli pratici) ## 10) Copertura del codice
Non c'è uno script `npm` dedicato, ma si può controllare la copertura in
qualsiasi momento con:
```bash
npx vitest run tests/unit tests/integration tests/component --coverage
```
Il report a schermo mostra percentuale e righe scoperte per file. La suite
attuale copre oltre il 99% di statement/linee; le poche righe rimaste
scoperte sono documentate nei commit dei rispettivi test come codice morto,
difensivo, oppure legato a HMR (`import.meta.hot`) o all'avvio diretto
(`node server.js`) — casi non testabili in modo significativo senza side
effect reali sull'ambiente.
## 11) Come aggiungere un nuovo test (consigli pratici)
- metti i test nel posto giusto: - metti i test nel posto giusto:
- `tests/unit` per logica pura - `tests/unit` per logica pura
+608
View File
@@ -133,6 +133,14 @@ describe('ControllerPage.vue', () => {
const btn = wrapper.findAll('.btn-ctrl').find(b => b.text().includes('Cambio Palla')) const btn = wrapper.findAll('.btn-ctrl').find(b => b.text().includes('Cambio Palla'))
expect(btn.attributes('disabled')).toBeDefined() expect(btn.attributes('disabled')).toBeDefined()
}) })
it('il click quando abilitato invia cambiaPalla', async () => {
const wrapper = mountController()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
const btn = wrapper.findAll('.btn-ctrl').find(b => b.text().includes('Cambio Palla'))
await btn.trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'cambiaPalla' })
})
}) })
// ============================================= // =============================================
@@ -349,6 +357,25 @@ describe('ControllerPage.vue', () => {
expect(spy).toHaveBeenCalledWith({ type: 'stopTimeout' }) expect(spy).toHaveBeenCalledWith({ type: 'stopTimeout' })
}) })
it('avviaTimeout non invia nulla se nessuna squadra è selezionata', () => {
const wrapper = mountController()
wrapper.vm.timeoutTeamSel = null
const spy = vi.spyOn(wrapper.vm, 'sendAction')
wrapper.vm.avviaTimeout()
expect(spy).not.toHaveBeenCalled()
})
it('il ticker del time out aggiorna nowTick periodicamente', async () => {
const wrapper = mountController()
wrapper.vm.state.timeoutTeam = 'home'
wrapper.vm.state.timeoutStartedAt = Date.now()
wrapper.vm.state.timeout = true
await wrapper.vm.$nextTick()
const before = wrapper.vm.nowTick
vi.advanceTimersByTime(500)
expect(wrapper.vm.nowTick).toBeGreaterThan(before)
})
it('durante il time out mostra banner con squadra e countdown', async () => { it('durante il time out mostra banner con squadra e countdown', async () => {
const wrapper = mountController() const wrapper = mountController()
wrapper.vm.state.timeoutTeam = 'home' wrapper.vm.state.timeoutTeam = 'home'
@@ -365,6 +392,587 @@ describe('ControllerPage.vue', () => {
}) })
}) })
// =============================================
// MODALITÀ ESTESA (landscape)
// =============================================
describe('Modalità estesa', () => {
it('con isLandscape=true mostra il layout esteso con i pannelli squadra', async () => {
const wrapper = mountController()
wrapper.vm.isLandscape = true
await wrapper.vm.$nextTick()
expect(wrapper.find('.e-dash').exists()).toBe(true)
expect(wrapper.findAll('.e-panel')).toHaveLength(2)
// in modalità estesa non compare il layout mobile
expect(wrapper.find('.score-preview').exists()).toBe(false)
})
it('il resize della finestra aggiorna isLandscape', async () => {
const wrapper = mountController()
expect(wrapper.vm.isLandscape).toBe(false)
Object.defineProperty(window, 'innerWidth', { value: 800, writable: true, configurable: true })
Object.defineProperty(window, 'innerHeight', { value: 400, writable: true, configurable: true })
window.dispatchEvent(new Event('resize'))
await wrapper.vm.$nextTick()
expect(wrapper.vm.isLandscape).toBe(true)
// ripristina le dimensioni portrait usate dagli altri test
Object.defineProperty(window, 'innerWidth', { value: 400, writable: true, configurable: true })
Object.defineProperty(window, 'innerHeight', { value: 800, writable: true, configurable: true })
})
it('il click sul punteggio in modalità estesa invia incPunt', async () => {
const wrapper = mountController()
wrapper.vm.isLandscape = true
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.findAll('.e-panel__score')[0].trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'incPunt', team: 'home' })
})
it('smontare il componente rimuove i listener di resize e ferma il ticker del time out', async () => {
const wrapper = mountController()
wrapper.vm.state.timeout = true
await wrapper.vm.$nextTick()
expect(vi.getTimerCount()).toBeGreaterThan(0)
const removeSpy = vi.spyOn(window, 'removeEventListener')
wrapper.unmount()
expect(removeSpy).toHaveBeenCalledWith('resize', expect.any(Function))
expect(removeSpy).toHaveBeenCalledWith('orientationchange', expect.any(Function))
})
})
// =============================================
// SET VINTO AUTOMATICO (watch squadraVincente)
// =============================================
describe('Set vinto automatico', () => {
it('apre la modal SET VINTO quando il punteggio raggiunge la soglia', async () => {
const wrapper = mountController()
setScore(wrapper, 25, 10)
await wrapper.vm.$nextTick()
expect(wrapper.vm.showSetVinto).toBe(true)
expect(wrapper.vm.setVintoTeam).toBe('home')
expect(wrapper.find('.dialog-title').text()).toBe('SET VINTO')
})
it('non riapre la modal se è già aperta', async () => {
const wrapper = mountController()
setScore(wrapper, 25, 10)
await wrapper.vm.$nextTick()
wrapper.vm.setVintoTeam = 'guest'
setScore(wrapper, 25, 24)
await wrapper.vm.$nextTick()
// il watch non deve sovrascrivere setVintoTeam perché showSetVinto è già true
expect(wrapper.vm.setVintoTeam).toBe('guest')
})
it('INDIETRO annulla l\'ultimo punto e chiude la modal', async () => {
const wrapper = mountController()
setScore(wrapper, 25, 10)
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.find('.btn-cancel').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'decPunt' })
expect(wrapper.vm.showSetVinto).toBe(false)
})
it('VAI AL SET SUCCESSIVO invia nuovoSet e apre la configurazione', async () => {
const wrapper = mountController()
setScore(wrapper, 25, 10)
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
const btn = wrapper.findAll('.btn-confirm').find(b => b.text().includes('SET SUCCESSIVO'))
await btn.trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'nuovoSet', team: 'home' })
expect(wrapper.vm.showSetVinto).toBe(false)
expect(wrapper.vm.showConfig).toBe(true)
})
})
// =============================================
// CONFIGURAZIONE
// =============================================
describe('Configurazione', () => {
it('apre la modal di configurazione precompilata con lo stato corrente', async () => {
const wrapper = mountController()
await wrapper.find('.btn-ctrl:not(.btn-timeout)').exists()
const btn = wrapper.findAll('.btn-ctrl').find(b => b.text() === 'Config')
await btn.trigger('click')
expect(wrapper.vm.showConfig).toBe(true)
expect(wrapper.vm.configData.nomeHome).toBe('Antoniana')
expect(wrapper.vm.configData.nomeGuest).toBe('Guest')
expect(wrapper.vm.configData.modalita).toBe('3/5')
})
it('OK invia le action di configurazione e chiude la modal', async () => {
const wrapper = mountController()
wrapper.vm.showConfig = true
wrapper.vm.configData = {
nomeHome: 'Casa', nomeGuest: 'Ospiti', modalita: '2/3',
formHome: ['1', '2', '3', '4', '5', '6'],
formGuest: ['1', '2', '3', '4', '5', '6'],
}
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.find('.dialog-config .btn-confirm').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'setNomi', home: 'Casa', guest: 'Ospiti' })
expect(spy).toHaveBeenCalledWith({ type: 'setModalita', modalita: '2/3' })
expect(spy).toHaveBeenCalledWith({ type: 'setFormazione', team: 'home', form: ['1', '2', '3', '4', '5', '6'] })
expect(spy).toHaveBeenCalledWith({ type: 'setFormazione', team: 'guest', form: ['1', '2', '3', '4', '5', '6'] })
expect(wrapper.vm.showConfig).toBe(false)
})
it('Annulla chiude la modal di configurazione senza inviare nulla', async () => {
const wrapper = mountController()
wrapper.vm.showConfig = true
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.find('.dialog-config .btn-cancel').trigger('click')
expect(spy).not.toHaveBeenCalled()
expect(wrapper.vm.showConfig).toBe(false)
})
it('scegliendo la modalità 2/3 il bottone diventa attivo', async () => {
const wrapper = mountController()
wrapper.vm.showConfig = true
await wrapper.vm.$nextTick()
const btn23 = wrapper.findAll('.btn-mode').find(b => b.text() === '2/3')
await btn23.trigger('click')
expect(wrapper.vm.configData.modalita).toBe('2/3')
expect(btn23.classes()).toContain('active')
})
})
// =============================================
// GESTIONE CAMBI GIOCATORE
// =============================================
describe('Gestione cambi', () => {
it('il click su Cambi apre la selezione squadra, poi la modal cambio per la squadra scelta', async () => {
const wrapper = mountController()
const btnCambi = wrapper.findAll('.btn-ctrl').find(b => b.text() === 'Cambi')
await btnCambi.trigger('click')
expect(wrapper.vm.showCambiTeam).toBe(true)
await wrapper.find('.dialog-title').exists()
const btnHome = wrapper.find('.overlay .btn-set.home-bg')
await btnHome.trigger('click')
expect(wrapper.vm.showCambiTeam).toBe(false)
expect(wrapper.vm.showCambi).toBe(true)
expect(wrapper.vm.cambiTeam).toBe('home')
})
it('Annulla chiude la modal cambi e azzera i campi', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
wrapper.vm.cambiData = [{ in: '10', out: '1' }, { in: '', out: '' }]
await wrapper.vm.$nextTick()
await wrapper.find('.dialog .btn-cancel').trigger('click')
expect(wrapper.vm.showCambi).toBe(false)
expect(wrapper.vm.cambiData).toEqual([{ in: '', out: '' }, { in: '', out: '' }])
})
it('CONFERMA con un cambio valido invia confermaCambi e chiude la modal', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
wrapper.vm.cambiData = [{ in: '10', out: '1' }, { in: '', out: '' }]
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.find('.dialog .btn-confirm').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'confermaCambi', team: 'home', cambi: [{ in: '10', out: '1' }] })
expect(wrapper.vm.showCambi).toBe(false)
})
it('segnala errore se il numero del giocatore non è formato da sole cifre', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
wrapper.vm.cambiData = [{ in: 'ab', out: '1' }, { in: '', out: '' }]
wrapper.vm.confermaCambi()
expect(wrapper.vm.cambiError).toBe('I numeri dei giocatori devono essere cifre')
})
it('segnala errore se il giocatore entrante coincide con l\'uscente', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
wrapper.vm.cambiData = [{ in: '1', out: '1' }, { in: '', out: '' }]
wrapper.vm.confermaCambi()
expect(wrapper.vm.cambiError).toBe('Il giocatore 1 non può sostituire sé stesso')
})
it('segnala errore se il giocatore entrante è già in formazione', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
// "2" è già presente nella formazione di default ["1".."6"]
wrapper.vm.cambiData = [{ in: '2', out: '1' }, { in: '', out: '' }]
wrapper.vm.confermaCambi()
expect(wrapper.vm.cambiError).toBe('Il giocatore 2 è già in formazione')
})
it('segnala errore se il giocatore uscente non è in formazione', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
wrapper.vm.cambiData = [{ in: '10', out: '9' }, { in: '', out: '' }]
wrapper.vm.confermaCambi()
expect(wrapper.vm.cambiError).toBe('Il giocatore 9 non è in formazione')
})
it('confermaCambi non fa nulla se cambiValid è false', () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
const spy = vi.spyOn(wrapper.vm, 'sendAction')
wrapper.vm.cambiData = [{ in: '', out: '' }, { in: '', out: '' }]
wrapper.vm.confermaCambi()
expect(spy).not.toHaveBeenCalled()
})
})
// =============================================
// GESTIONE ERRORI WEBSOCKET
// =============================================
describe('Gestione errori WebSocket', () => {
it('onWsMessage con type error logga il messaggio', () => {
const wrapper = mountController()
const spy = vi.spyOn(console, 'error').mockImplementation(() => {})
wrapper.vm.onWsMessage({ type: 'error', message: 'boom' })
expect(spy).toHaveBeenCalledWith('[Controller] Error:', 'boom')
spy.mockRestore()
})
it('sendAction senza type non invia nulla', () => {
const wrapper = mountController()
const spy = vi.spyOn(wrapper.vm, 'sendWs')
wrapper.vm.sendAction({})
expect(spy).not.toHaveBeenCalled()
})
it('sendAction mostra un feedback di errore se non connesso', () => {
const wrapper = mountController()
const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
wrapper.vm.wsConnected = false
wrapper.vm.sendAction({ type: 'incPunt', team: 'home' })
expect(errSpy).toHaveBeenCalledWith('[Controller] Error:', 'Non connesso al server')
errSpy.mockRestore()
})
})
// =============================================
// DURATA VIDEO SPOT
// =============================================
describe('Durata video spot', () => {
it('formatDurata mostra i secondi arrotondati o un placeholder', () => {
const wrapper = mountController()
expect(wrapper.vm.formatDurata(12.4)).toBe('12s')
expect(wrapper.vm.formatDurata(undefined)).toBe('…')
expect(wrapper.vm.formatDurata(NaN)).toBe('…')
})
it('probeDurata popola spotDurations quando i metadata del video sono pronti', () => {
const wrapper = mountController()
const videoEls = []
const originalCreateElement = document.createElement.bind(document)
vi.spyOn(document, 'createElement').mockImplementation((tag) => {
const el = originalCreateElement(tag)
if (tag === 'video') videoEls.push(el)
return el
})
wrapper.vm.probeDurata('a.mp4')
const videoEl = videoEls[0]
Object.defineProperty(videoEl, 'duration', { value: 42, configurable: true })
videoEl.onloadedmetadata()
expect(wrapper.vm.spotDurations['a.mp4']).toBe(42)
document.createElement.mockRestore()
})
})
// =============================================
// PULSANTI MOBILE (click reali sui bottoni)
// =============================================
describe('Pulsanti layout mobile', () => {
it('ANNULLA PUNTO invia decPunt', async () => {
const wrapper = mountController()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.find('.btn-undo').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'decPunt' })
})
it('i due bottoni SET inviano incSet per la rispettiva squadra', async () => {
const wrapper = mountController()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
const setBtns = wrapper.findAll('.action-row .btn-set')
await setBtns[0].trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'incSet', team: 'home' })
await setBtns[1].trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'incSet', team: 'guest' })
})
it('Formazioni/Striscia/Inverti/Voce inviano le rispettive action', async () => {
const wrapper = mountController()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
const byLabel = (label) => wrapper.findAll('.btn-ctrl').find(b => b.text().includes(label))
await byLabel('Formazioni').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'toggleFormazione' })
await byLabel('Striscia').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'toggleStriscia' })
await byLabel('Inverti').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'toggleOrder' })
const wsSendSpy = vi.spyOn(wrapper.vm, 'sendWs')
wrapper.vm.wsConnected = true
wrapper.vm.ws = { readyState: 1, send: vi.fn() }
await byLabel('Voce').trigger('click')
expect(wsSendSpy).toHaveBeenCalled()
})
it('con order=false lo score-preview mostra prima guest e poi home', async () => {
const wrapper = mountController()
wrapper.vm.state.order = false
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
const scores = wrapper.findAll('.team-score')
expect(scores[0].classes()).toContain('guest-bg')
expect(scores[1].classes()).toContain('home-bg')
await scores[0].trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'incPunt', team: 'guest' })
})
})
// =============================================
// MODALITÀ ESTESA: interazioni aggiuntive
// =============================================
describe('Modalità estesa: interazioni', () => {
it('il click sul punteggio e sul bottone SET della seconda squadra funzionano', async () => {
const wrapper = mountController()
wrapper.vm.isLandscape = true
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.findAll('.e-panel__score')[1].trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'incPunt', team: 'guest' })
await wrapper.findAll('.e-panel__setbtn')[0].trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'incSet', team: 'home' })
await wrapper.findAll('.e-panel__setbtn')[1].trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'incSet', team: 'guest' })
})
it('la barra e-actions invia le action corrispondenti ai bottoni', async () => {
const wrapper = mountController()
wrapper.vm.isLandscape = true
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
const byLabel = (label) => wrapper.findAll('.e-act').find(b => b.text().includes(label))
await byLabel('Annulla').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'decPunt' })
await byLabel('Cambio Palla').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'cambiaPalla' })
await byLabel('Inverti').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'toggleOrder' })
await byLabel('Striscia').trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'toggleStriscia' })
expect(wrapper.vm.showCambiTeam).toBe(false)
const btnCambi = wrapper.findAll('.e-act').find(b => b.text() === 'Cambi')
await btnCambi.trigger('click')
expect(wrapper.vm.showCambiTeam).toBe(true)
wrapper.vm.showCambiTeam = false
await byLabel('Config').trigger('click')
expect(wrapper.vm.showConfig).toBe(true)
wrapper.vm.showConfig = false
await wrapper.find('.e-act--danger').trigger('click')
expect(wrapper.vm.confirmReset).toBe(true)
})
it('il bottone Voce e il bottone Time Out della barra estesa funzionano', async () => {
const wrapper = mountController()
wrapper.vm.isLandscape = true
await wrapper.vm.$nextTick()
wrapper.vm.wsConnected = true
wrapper.vm.ws = { readyState: 1, send: vi.fn() }
const wsSendSpy = vi.spyOn(wrapper.vm, 'sendWs')
const btnVoce = wrapper.findAll('.e-act').find(b => b.text() === 'Voce')
await btnVoce.trigger('click')
expect(wsSendSpy).toHaveBeenCalled()
const btnTimeout = wrapper.find('.e-act.btn-timeout')
await btnTimeout.trigger('click')
expect(wrapper.vm.showTimeoutModal).toBe(true)
wrapper.vm.showTimeoutModal = false
wrapper.vm.state.timeout = true
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await btnTimeout.trigger('click')
expect(spy).toHaveBeenCalledWith({ type: 'stopTimeout' })
})
})
// =============================================
// CHIUSURA DIALOG CLICCANDO FUORI (overlay self)
// =============================================
describe('Chiusura dialog cliccando fuori', () => {
it('conferma reset si chiude cliccando fuori dal dialog', async () => {
const wrapper = mountController()
wrapper.vm.confirmReset = true
await wrapper.vm.$nextTick()
await wrapper.find('.overlay').trigger('click')
expect(wrapper.vm.confirmReset).toBe(false)
})
it('configurazione si chiude cliccando fuori dal dialog', async () => {
const wrapper = mountController()
wrapper.vm.showConfig = true
await wrapper.vm.$nextTick()
await wrapper.find('.overlay').trigger('click')
expect(wrapper.vm.showConfig).toBe(false)
})
it('scelta squadra cambi si chiude cliccando fuori dal dialog', async () => {
const wrapper = mountController()
wrapper.vm.showCambiTeam = true
await wrapper.vm.$nextTick()
await wrapper.find('.overlay').trigger('click')
expect(wrapper.vm.showCambiTeam).toBe(false)
})
it('la modal cambi si chiude cliccando fuori dal dialog', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
await wrapper.vm.$nextTick()
await wrapper.find('.overlay').trigger('click')
expect(wrapper.vm.showCambi).toBe(false)
})
it('la modal time out si chiude cliccando fuori dal dialog', async () => {
const wrapper = mountController()
await wrapper.find('.btn-timeout').trigger('click')
await flushPromises()
await wrapper.find('.overlay').trigger('click')
expect(wrapper.vm.showTimeoutModal).toBe(false)
})
})
// =============================================
// DIALOG PARTITA FINITA / CONFIGURAZIONE: dettagli
// =============================================
describe('Dettagli dialog', () => {
it('CHIUDI chiude il dialog partita finita senza inviare action', async () => {
const wrapper = mountController()
wrapper.vm.state.modalitaPartita = '2/3'
wrapper.vm.state.sp.striscia = [
{ serv: 'h', ris: '', vinc: 'h' },
{ serv: 'h', ris: '', vinc: null },
]
wrapper.vm.setVintoTeam = 'home'
wrapper.vm.showSetVinto = true
await wrapper.vm.$nextTick()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
const chiudi = wrapper.findAll('.btn-confirm').find(b => b.text() === 'CHIUDI')
await chiudi.trigger('click')
expect(spy).not.toHaveBeenCalled()
expect(wrapper.vm.showSetVinto).toBe(false)
})
it('isPartitaFinita è false in modalità amichevole anche a set vinto', () => {
const wrapper = mountController()
wrapper.vm.state.modalitaPartita = 'amichevole'
wrapper.vm.setVintoTeam = 'home'
expect(wrapper.vm.isPartitaFinita).toBe(false)
})
it('la vittoria della squadra guest apre comunque la modal SET VINTO', async () => {
const wrapper = mountController()
setScore(wrapper, 10, 25, 'g')
await wrapper.vm.$nextTick()
expect(wrapper.vm.setVintoTeam).toBe('guest')
})
it('nel set decisivo la soglia di vittoria scende a 15 punti', async () => {
const wrapper = mountController()
// 4 set già assegnati (2 a testa): il quinto è il set decisivo (3/5)
wrapper.vm.state.sp.striscia = [
{ serv: 'h', ris: '', vinc: 'h' },
{ serv: 'h', ris: '', vinc: 'g' },
{ serv: 'h', ris: '', vinc: 'h' },
{ serv: 'h', ris: '', vinc: 'g' },
{ serv: 'h', ris: '', vinc: null },
]
setScore(wrapper, 15, 10)
await wrapper.vm.$nextTick()
expect(wrapper.vm.setVintoTeam).toBe('home')
})
it('i bottoni modalità 3/5 e amichevole aggiornano configData', async () => {
const wrapper = mountController()
wrapper.vm.showConfig = true
await wrapper.vm.$nextTick()
const btn35 = wrapper.findAll('.btn-mode').find(b => b.text() === '3/5')
await btn35.trigger('click')
expect(wrapper.vm.configData.modalita).toBe('3/5')
const btnAmichevole = wrapper.findAll('.btn-mode').find(b => b.text() === 'Amichevole')
await btnAmichevole.trigger('click')
expect(wrapper.vm.configData.modalita).toBe('amichevole')
expect(btnAmichevole.classes()).toContain('active')
})
it('i campi di testo di configurazione aggiornano configData tramite v-model', async () => {
const wrapper = mountController()
wrapper.vm.showConfig = true
await wrapper.vm.$nextTick()
const inputs = wrapper.findAll('.dialog-config .input-field')
await inputs[0].setValue('Casa Nuova')
await inputs[1].setValue('Ospiti Nuovi')
expect(wrapper.vm.configData.nomeHome).toBe('Casa Nuova')
expect(wrapper.vm.configData.nomeGuest).toBe('Ospiti Nuovi')
})
it('i campi numerici della formazione aggiornano configData tramite v-model', async () => {
const wrapper = mountController()
wrapper.vm.showConfig = true
await wrapper.vm.$nextTick()
const numInputs = wrapper.findAll('.dialog-config .input-num')
// 6 campi per home + 6 per guest: li valorizziamo tutti per coprire ogni binding v-model
for (let i = 0; i < numInputs.length; i++) {
await numInputs[i].setValue(String(i))
}
// il DOM ordina le celle come 3,2,1 / 4,5,0: verifichiamo l'insieme dei valori digitati
expect(new Set(wrapper.vm.configData.formHome)).toEqual(new Set(['0', '1', '2', '3', '4', '5']))
expect(new Set(wrapper.vm.configData.formGuest)).toEqual(new Set(['6', '7', '8', '9', '10', '11']))
})
it('la selezione squadra guest per i cambi apre la modal con cambiTeam=guest', async () => {
const wrapper = mountController()
wrapper.vm.showCambiTeam = true
await wrapper.vm.$nextTick()
await wrapper.find('.overlay .btn-set.guest-bg').trigger('click')
expect(wrapper.vm.cambiTeam).toBe('guest')
expect(wrapper.vm.showCambi).toBe(true)
})
it('i campi IN/OUT della modal cambi aggiornano cambiData tramite v-model', async () => {
const wrapper = mountController()
wrapper.vm.openCambi('home')
await wrapper.vm.$nextTick()
const inField = wrapper.find('.cambi-in-field')
const outField = wrapper.find('.cambi-out-field')
await inField.setValue('10')
await outField.setValue('1')
expect(wrapper.vm.cambiData[0]).toEqual({ in: '10', out: '1' })
})
it('nella modal time out si può selezionare un video dall\'elenco tramite radio', async () => {
fetchMock.mockResolvedValueOnce({ json: async () => ['a.mp4'] })
const wrapper = mountController()
await wrapper.find('.btn-timeout').trigger('click')
await flushPromises()
const radios = wrapper.findAll('input[type=radio]')
await radios[0].setValue(true)
expect(wrapper.vm.timeoutVideoSel).toBe('a.mp4')
await radios[1].setValue(true)
expect(wrapper.vm.timeoutVideoSel).toBe(null)
})
it('Annulla nella modal time out la chiude senza avviarlo', async () => {
const wrapper = mountController()
await wrapper.find('.btn-timeout').trigger('click')
await flushPromises()
const spy = vi.spyOn(wrapper.vm, 'sendAction')
await wrapper.find('.dialog-timeout .btn-cancel').trigger('click')
expect(spy).not.toHaveBeenCalled()
expect(wrapper.vm.showTimeoutModal).toBe(false)
})
})
// ============================================= // =============================================
// BARRA CONNESSIONE // BARRA CONNESSIONE
// ============================================= // =============================================
+259
View File
@@ -31,6 +31,15 @@ vi.stubGlobal('speechSynthesis', {
getVoices: () => [] getVoices: () => []
}) })
// happy-dom non implementa la Web Speech API: stub minimo dell'utterance.
vi.stubGlobal('SpeechSynthesisUtterance', class {
constructor(text) {
this.text = text
this.voice = null
this.lang = ''
}
})
// happy-dom non implementa la riproduzione media: stub di play/pause/load // happy-dom non implementa la riproduzione media: stub di play/pause/load
// così l'overlay time out può montare il <video> senza errori. // così l'overlay time out può montare il <video> senza errori.
Object.defineProperty(HTMLMediaElement.prototype, 'play', { value: vi.fn().mockResolvedValue(undefined), writable: true }) Object.defineProperty(HTMLMediaElement.prototype, 'play', { value: vi.fn().mockResolvedValue(undefined), writable: true })
@@ -257,6 +266,256 @@ describe('DisplayPage.vue', () => {
}) })
}) })
// =============================================
// ORDINE GUEST/HOME CON FORMAZIONE
// =============================================
describe('Ordine guest/home con formazione visibile', () => {
it('order=false e visuForm=true mostra punteggio inline e formazioni nell\'ordine guest poi home', async () => {
const wrapper = mountDisplay()
wrapper.vm.state.order = false
wrapper.vm.state.visuForm = true
wrapper.vm.state.sp.striscia.at(-1).ris = 'h'.repeat(3) + 'g'.repeat(2)
await wrapper.vm.$nextTick()
const scores = wrapper.findAll('.score-inline')
expect(scores[0].text()).toBe('2') // guest per primo
expect(scores[1].text()).toBe('3') // home per secondo
const formDivs = wrapper.findAll('.formdiv')
expect(formDivs).toHaveLength(12)
})
})
// =============================================
// MONTAGGIO: FULLSCREEN E TIME OUT GIÀ ATTIVO
// =============================================
describe('Comportamento al mount', () => {
it('su dispositivo mobile richiede la fullscreen', () => {
const uaSpy = vi.spyOn(navigator, 'userAgent', 'get').mockReturnValue('Mozilla/5.0 (iPhone; CPU iPhone OS 15_0 like Mac OS X)')
const fsSpy = vi.fn()
document.documentElement.requestFullscreen = fsSpy
const wrapper = mountDisplay()
expect(fsSpy).toHaveBeenCalled()
wrapper.unmount()
uaSpy.mockRestore()
})
it('non richiede la fullscreen su desktop', () => {
const fsSpy = vi.fn()
document.documentElement.requestFullscreen = fsSpy
const wrapper = mountDisplay()
expect(fsSpy).not.toHaveBeenCalled()
wrapper.unmount()
})
it('ignora l\'errore se requestFullscreen lancia un\'eccezione', () => {
const uaSpy = vi.spyOn(navigator, 'userAgent', 'get').mockReturnValue('Mozilla/5.0 (Linux; Android 10)')
document.documentElement.requestFullscreen = () => { throw new Error('non consentito') }
expect(() => mountDisplay()).not.toThrow()
uaSpy.mockRestore()
})
it('se il time out è già attivo al mount avvia subito la modalità time out', async () => {
const wrapper = mount(DisplayPage, {
global: { stubs: { 'w-app': true } },
data() {
const state = {
order: true, visuForm: false, visuStriscia: true,
timeout: true, timeoutTeam: 'home', timeoutVideo: null,
timeoutStartedAt: Date.now(), modalitaPartita: '3/5',
sp: {
striscia: [{ serv: 'h', ris: '', vinc: null }],
nomi: { home: 'Antoniana', guest: 'Guest' },
form: { home: ['1', '2', '3', '4', '5', '6'], guest: ['1', '2', '3', '4', '5', '6'] },
},
}
return { state }
}
})
await wrapper.vm.$nextTick()
expect(wrapper.vm.mostraPlaceholder).toBe(true)
expect(wrapper.find('.timeout-overlay').exists()).toBe(true)
})
})
// =============================================
// TICKER COUNTDOWN E CASI LIMITE
// =============================================
describe('Ticker del countdown', () => {
it('il ticker aggiorna nowTick periodicamente durante il time out', async () => {
const wrapper = mountDisplay()
wrapper.vm.state.timeoutTeam = 'home'
wrapper.vm.state.timeoutStartedAt = Date.now()
wrapper.vm.state.timeout = true
await wrapper.vm.$nextTick()
expect(wrapper.vm.timeoutRemaining).toBe(30)
vi.advanceTimersByTime(1000)
await wrapper.vm.$nextTick()
expect(wrapper.vm.timeoutRemaining).toBeLessThan(30)
})
it('timeoutRemaining resta 30 se manca timeoutStartedAt', async () => {
const wrapper = mountDisplay()
wrapper.vm.state.timeoutTeam = 'home'
wrapper.vm.state.timeoutStartedAt = null
wrapper.vm.state.timeout = true
await wrapper.vm.$nextTick()
expect(wrapper.vm.timeoutRemaining).toBe(30)
})
it('smontare il componente durante il time out ferma il ticker senza errori', async () => {
const wrapper = mountDisplay()
wrapper.vm.state.timeoutTeam = 'home'
wrapper.vm.state.timeoutStartedAt = Date.now()
wrapper.vm.state.timeout = true
await wrapper.vm.$nextTick()
expect(() => wrapper.unmount()).not.toThrow()
})
it('stricciaStrip non fallisce se la striscia del set corrente è assente', async () => {
const wrapper = mountDisplay()
wrapper.vm.state.sp.striscia = []
expect(wrapper.vm.stricciaStrip).toEqual({ home: [], guest: [] })
// ripristina uno stato valido prima che Vue ricalcoli gli altri computed (punt/servHome)
wrapper.vm.state.sp.striscia = [{ serv: 'h', ris: '', vinc: null }]
await wrapper.vm.$nextTick()
})
})
// =============================================
// FINE TIME OUT: STOP VIDEO E FALLBACK MUTO
// =============================================
describe('Stop del time out e riproduzione video', () => {
async function attivaTimeout(wrapper, { team = 'home', video = null, startedAt = Date.now() } = {}) {
wrapper.vm.state.timeoutTeam = team
wrapper.vm.state.timeoutVideo = video
wrapper.vm.state.timeoutStartedAt = startedAt
wrapper.vm.state.timeout = true
await wrapper.vm.$nextTick()
await wrapper.vm.$nextTick()
}
it('allo stop del time out con video in corso mette in pausa e resetta il video', async () => {
const wrapper = mountDisplay()
await attivaTimeout(wrapper, { video: 'a.mp4' })
const video = wrapper.find('.timeout-video').element
const pauseSpy = vi.spyOn(video, 'pause')
wrapper.vm.state.timeout = false
await wrapper.vm.$nextTick()
expect(pauseSpy).toHaveBeenCalled()
})
it('a metà riproduzione si riallinea al punto corretto (riconnessione)', async () => {
const wrapper = mountDisplay()
await attivaTimeout(wrapper, { video: 'a.mp4', startedAt: Date.now() - 10000 })
const video = wrapper.find('.timeout-video').element
Object.defineProperty(video, 'duration', { value: 60, configurable: true })
video.onloadedmetadata()
expect(video.currentTime).toBeCloseTo(10, 0)
})
it('senza timeoutStartedAt non tenta alcun riallineamento (trascorsi=0)', async () => {
const wrapper = mountDisplay()
await attivaTimeout(wrapper, { video: 'a.mp4', startedAt: null })
const video = wrapper.find('.timeout-video').element
Object.defineProperty(video, 'duration', { value: 60, configurable: true })
video.currentTime = 0
video.onloadedmetadata()
expect(video.currentTime).toBe(0)
})
it('non si riallinea se il tempo trascorso supera la durata del video', async () => {
const wrapper = mountDisplay()
await attivaTimeout(wrapper, { video: 'a.mp4', startedAt: Date.now() - 10000 })
const video = wrapper.find('.timeout-video').element
Object.defineProperty(video, 'duration', { value: 5, configurable: true })
video.currentTime = 0
video.onloadedmetadata()
expect(video.currentTime).toBe(0)
})
it('caricaSpotCorrente non fa nulla se manca il ref del video o il video del time out', () => {
const wrapper = mountDisplay()
// nessun overlay attivo: $refs.spotVideo non esiste
expect(() => wrapper.vm.caricaSpotCorrente()).not.toThrow()
expect(wrapper.vm.spotVideoDuration).toBe(null)
})
it('playSpot non fa nulla se manca il ref del video', () => {
const wrapper = mountDisplay()
expect(() => wrapper.vm.playSpot()).not.toThrow()
})
it('se l\'autoplay con audio viene bloccato ripiega sul muto', async () => {
const wrapper = mountDisplay()
HTMLMediaElement.prototype.play = vi.fn()
.mockRejectedValueOnce(new Error('autoplay bloccato'))
.mockResolvedValueOnce(undefined)
await attivaTimeout(wrapper, { video: 'a.mp4' })
const video = wrapper.find('.timeout-video').element
await Promise.resolve()
await Promise.resolve()
expect(video.muted).toBe(true)
})
})
// =============================================
// MESSAGGI WEBSOCKET E SINTESI VOCALE
// =============================================
describe('onWsMessage e sintesi vocale', () => {
it('un messaggio "speak" avvia la sintesi vocale con il testo ricevuto', () => {
const wrapper = mountDisplay()
wrapper.vm.onWsMessage({ type: 'speak', text: '15 a 10' })
expect(window.speechSynthesis.cancel).toHaveBeenCalled()
expect(window.speechSynthesis.speak).toHaveBeenCalled()
const utterance = window.speechSynthesis.speak.mock.calls.at(-1)[0]
expect(utterance.text).toBe('15 a 10')
expect(utterance.lang).toBe('it-IT')
})
it('un messaggio "error" viene loggato in console', () => {
const wrapper = mountDisplay()
const spy = vi.spyOn(console, 'error').mockImplementation(() => {})
wrapper.vm.onWsMessage({ type: 'error', message: 'boom' })
expect(spy).toHaveBeenCalledWith('[Display] Server error:', 'boom')
spy.mockRestore()
})
it('ignora testo vuoto o non stringa nella sintesi vocale', () => {
const wrapper = mountDisplay()
window.speechSynthesis.speak.mockClear()
wrapper.vm.speakOnDisplay(' ')
wrapper.vm.speakOnDisplay(42)
expect(window.speechSynthesis.speak).not.toHaveBeenCalled()
})
it('sceglie la voce "Google italiano" quando disponibile', () => {
const wrapper = mountDisplay()
const vIt = { name: 'voce it generica', lang: 'it-IT' }
const vGoogle = { name: 'Google italiano', lang: 'it-IT' }
window.speechSynthesis.getVoices = () => [vIt, vGoogle]
wrapper.vm.speakOnDisplay('prova')
const utterance = window.speechSynthesis.speak.mock.calls.at(-1)[0]
expect(utterance.voice).toBe(vGoogle)
})
it('ripiega su una voce con lingua "it" se manca "Google italiano"', () => {
const wrapper = mountDisplay()
const vIt = { name: 'voce italiana', lang: 'it-IT' }
const vEn = { name: 'english voice', lang: 'en-US' }
window.speechSynthesis.getVoices = () => [vEn, vIt]
wrapper.vm.speakOnDisplay('prova')
const utterance = window.speechSynthesis.speak.mock.calls.at(-1)[0]
expect(utterance.voice).toBe(vIt)
})
it('usa voice=null se nessuna voce italiana è disponibile', () => {
const wrapper = mountDisplay()
window.speechSynthesis.getVoices = () => [{ name: 'english voice', lang: 'en-US' }]
wrapper.vm.speakOnDisplay('prova')
const utterance = window.speechSynthesis.speak.mock.calls.at(-1)[0]
expect(utterance.voice).toBe(null)
})
})
// ============================================= // =============================================
// ICONA SERVIZIO // ICONA SERVIZIO
// ============================================= // =============================================
@@ -0,0 +1,31 @@
// @vitest-environment happy-dom
import { describe, it, expect, vi, afterEach } from 'vitest'
import { generaReferto } from '../../src/referto.js'
import { createInitialState } from '../../src/gameState.js'
// generaReferto è l'unico punto con effetti collaterali di referto.js (window.open
// + stampa): qui verifichiamo che orchestri correttamente le API del browser,
// mockando window.open per non aprire davvero una finestra durante i test.
describe('generaReferto (referto.js)', () => {
afterEach(() => {
vi.restoreAllMocks()
})
it('apre una nuova scheda, scrive il referto e avvia la stampa', () => {
const popup = {
document: { write: vi.fn(), close: vi.fn() },
print: vi.fn(),
}
const openSpy = vi.spyOn(window, 'open').mockReturnValue(popup)
const state = createInitialState()
state.sp.nomi = { home: 'Antoniana', guest: 'Rivali' }
generaReferto(state)
expect(openSpy).toHaveBeenCalledWith('', '_blank')
expect(popup.document.write).toHaveBeenCalledTimes(1)
expect(popup.document.write.mock.calls[0][0]).toContain('Antoniana')
expect(popup.document.close).toHaveBeenCalledTimes(1)
expect(popup.print).toHaveBeenCalledTimes(1)
})
})
+97
View File
@@ -224,5 +224,102 @@ describe('createWsMixin (wsMixin.js)', () => {
wrapper.unmount() wrapper.unmount()
expect(ws.close).toHaveBeenCalled() expect(ws.close).toHaveBeenCalled()
}) })
it('beforeUnmount con connessione OPEN chiude con codice 1000', () => {
const wrapper = mountWith()
const ws = ultimaWs()
ws.readyState = MockWebSocket.OPEN
wrapper.unmount()
expect(ws.close).toHaveBeenCalledWith(1000, 'Component unmounting')
})
it('beforeUnmount non lancia se ws.close lancia', () => {
const wrapper = mountWith()
const ws = ultimaWs()
ws.readyState = MockWebSocket.OPEN
ws.close.mockImplementation(() => { throw new Error('già chiusa') })
const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
expect(() => wrapper.unmount()).not.toThrow()
expect(consoleSpy).toHaveBeenCalled()
})
it('beforeUnmount annulla un reconnectTimeout pendente', () => {
vi.useFakeTimers()
try {
const wrapper = mountWith()
wrapper.vm.isConnecting = false
wrapper.vm.scheduleReconnect()
expect(wrapper.vm.reconnectTimeout).not.toBe(null)
wrapper.unmount()
// se il timer non fosse stato annullato, allo scoccare del delay
// verrebbe creata una nuova connessione anche a componente smontato
const primaDelTimeout = MockWebSocket.instances.length
vi.advanceTimersByTime(30000)
expect(MockWebSocket.instances.length).toBe(primaDelTimeout)
} finally {
vi.useRealTimers()
}
})
})
describe('rami aggiuntivi', () => {
it('connectWebSocket non fa nulla se è già in corso un tentativo di connessione', () => {
const wrapper = mountWith()
const primaDelTimeout = MockWebSocket.instances.length
wrapper.vm.isConnecting = true
wrapper.vm.connectWebSocket()
expect(MockWebSocket.instances.length).toBe(primaDelTimeout)
})
it('riconnettendosi con la WebSocket precedente OPEN, la chiude con codice 1000', () => {
const wrapper = mountWith()
const prima = ultimaWs()
prima.readyState = MockWebSocket.OPEN
wrapper.vm.isConnecting = false
wrapper.vm.connectWebSocket()
expect(prima.close).toHaveBeenCalledWith(1000, 'Reconnecting')
})
it('se la chiusura della WebSocket precedente lancia, logga e continua a riconnettersi', () => {
const wrapper = mountWith()
const prima = ultimaWs()
prima.readyState = MockWebSocket.OPEN
prima.close.mockImplementation(() => { throw new Error('già chiusa') })
const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
wrapper.vm.isConnecting = false
expect(() => wrapper.vm.connectWebSocket()).not.toThrow()
expect(consoleSpy).toHaveBeenCalled()
expect(ultimaWs()).not.toBe(prima)
})
it('onopen non lancia se il registro fallisce (send che lancia)', () => {
const wrapper = mountWith()
const ws = ultimaWs()
ws.readyState = MockWebSocket.OPEN
ws.send.mockImplementation(() => { throw new Error('boom') })
const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
expect(() => ws.onopen()).not.toThrow()
expect(wrapper.vm.wsConnected).toBe(true)
expect(consoleSpy).toHaveBeenCalled()
})
it('onmessage non lancia su JSON non valido', () => {
mountWith()
const ws = ultimaWs()
const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
expect(() => ws.onmessage({ data: 'non è JSON {{{' })).not.toThrow()
expect(consoleSpy).toHaveBeenCalled()
})
it('scheduleReconnect non arma un secondo timer se uno è già pendente', () => {
const wrapper = mountWith()
const spy = vi.spyOn(globalThis, 'setTimeout')
wrapper.vm.reconnectTimeout = 123 // timer già pendente (valore fittizio)
wrapper.vm.scheduleReconnect()
expect(spy).not.toHaveBeenCalled()
})
}) })
}) })
+1 -2
View File
@@ -40,8 +40,7 @@ test.describe('Referto di fine partita', () => {
// Verifica il contenuto del referto // Verifica il contenuto del referto
const body = popup.locator('body'); const body = popup.locator('body');
await expect(body).toContainText('Referto di Gara'); await expect(body).toContainText('Referto di Gara');
await expect(body).toContainText('Set 1'); await expect(popup.locator('.set-numero')).toHaveText(['1', '2']);
await expect(body).toContainText('Set 2');
// Home ha vinto 2 set a 0 // Home ha vinto 2 set a 0
await expect(body).toContainText('2 0'); await expect(body).toContainText('2 0');
}); });
+75
View File
@@ -0,0 +1,75 @@
import { describe, it, expect, vi, beforeAll, afterAll } from 'vitest'
import WebSocket from 'ws'
import { createInitialState } from '../../src/gameState.js'
// startServer() usa src/persist.js per caricare/salvare lo stato reale su disco
// (.segnapunti/state.json): lo mockiamo per non toccare lo stato di una partita
// vera durante i test, ed esercitare in isolamento l'avvio di HTTP + WebSocket.
const saveState = vi.fn()
vi.mock('../../src/persist.js', () => ({
loadState: () => createInitialState(),
saveState,
}))
const { startServer } = await import('../../server.js')
describe('startServer (server.js)', () => {
let server
let port
beforeAll(async () => {
const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
server = startServer(0)
await new Promise((resolve) => server.on('listening', resolve))
port = server.address().port
consoleSpy.mockRestore()
})
afterAll(async () => {
await new Promise((resolve) => server.close(resolve))
})
it('stampa le info del server all\'avvio', async () => {
const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {})
const s = startServer(0)
await new Promise((resolve) => s.on('listening', resolve))
const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n')
expect(allLogs).toContain('Segnapunti Server')
consoleSpy.mockRestore()
await new Promise((resolve) => s.close(resolve))
})
it('accetta una connessione WebSocket su /ws', async () => {
const ws = new WebSocket(`ws://127.0.0.1:${port}/ws`)
await new Promise((resolve, reject) => {
ws.on('open', resolve)
ws.on('error', reject)
})
ws.close()
})
it('rifiuta l\'upgrade su un percorso diverso da /ws', async () => {
const ws = new WebSocket(`ws://127.0.0.1:${port}/altro-percorso`)
await new Promise((resolve) => {
ws.on('error', () => resolve())
ws.on('open', () => { ws.close(); resolve() })
})
})
it('un\'azione dal controller invoca saveState (onStateChange)', async () => {
saveState.mockClear()
const ws = new WebSocket(`ws://127.0.0.1:${port}/ws`)
await new Promise((resolve, reject) => {
ws.on('open', resolve)
ws.on('error', reject)
})
ws.send(JSON.stringify({ type: 'register', role: 'controller' }))
await new Promise((resolve) => ws.once('message', resolve))
ws.send(JSON.stringify({ type: 'action', action: { type: 'incPunt', team: 'home' } }))
await new Promise((resolve) => ws.once('message', resolve))
expect(saveState).toHaveBeenCalled()
ws.close()
})
})
@@ -0,0 +1,134 @@
import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
import { EventEmitter } from 'events'
// Mock parziale di una WebSocket e del Server, analogo a websocket.test.js ma
// con send/terminate che possono essere forzati a lanciare per esercitare i
// rami di gestione errori altrimenti irraggiungibili.
class MockWebSocket extends EventEmitter {
constructor() {
super()
this.readyState = 1 // OPEN
}
send = vi.fn()
terminate = vi.fn()
}
class MockWebSocketServer extends EventEmitter {
clients = new Set()
}
describe('websocket-handler.js — rami di errore', () => {
afterEach(() => {
vi.restoreAllMocks()
vi.resetModules()
vi.doUnmock('../../src/gameState.js')
})
describe('errori di invio (ws.send che lancia)', () => {
let wss, setupWebSocketHandler, consoleSpy
beforeEach(async () => {
vi.resetModules()
;({ setupWebSocketHandler } = await import('../../src/websocket-handler.js'))
wss = new MockWebSocketServer()
consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
})
it('handleRegister: se ws.send lancia durante l\'invio dello stato iniziale, non deve propagare l\'errore', () => {
setupWebSocketHandler(wss)
const ws = new MockWebSocket()
ws.send.mockImplementation(() => { throw new Error('socket chiuso a metà') })
wss.emit('connection', ws)
wss.clients.add(ws)
expect(() => {
ws.emit('message', JSON.stringify({ type: 'register', role: 'display' }))
}).not.toThrow()
expect(consoleSpy).toHaveBeenCalledWith('Error sending initial state:', expect.any(Error))
})
it('sendError: se ws.send lancia rispondendo a un JSON non valido, non deve propagare l\'errore', () => {
setupWebSocketHandler(wss)
const ws = new MockWebSocket()
ws.send.mockImplementation(() => { throw new Error('socket chiuso a metà') })
wss.emit('connection', ws)
wss.clients.add(ws)
// JSON non valido → handleMessage va nel catch → sendError prova a rispondere → ws.send lancia
expect(() => {
ws.emit('message', 'non è JSON {{{')
}).not.toThrow()
expect(consoleSpy).toHaveBeenCalledWith('Failed to send error message:', expect.any(Error))
})
it('sendError: se ws.send lancia rispondendo a un\'azione invalida, non deve propagare l\'errore', () => {
setupWebSocketHandler(wss)
const controller = new MockWebSocket()
wss.emit('connection', controller)
wss.clients.add(controller)
controller.emit('message', JSON.stringify({ type: 'register', role: 'controller' }))
controller.send.mockImplementation(() => { throw new Error('socket chiuso a metà') })
expect(() => {
controller.emit('message', JSON.stringify({ type: 'action' }))
}).not.toThrow()
expect(consoleSpy).toHaveBeenCalledWith('Failed to send error message:', expect.any(Error))
})
})
describe('handleError: ws.terminate che lancia', () => {
it('non deve propagare l\'errore se la chiusura forzata fallisce', async () => {
vi.resetModules()
const { setupWebSocketHandler } = await import('../../src/websocket-handler.js')
const wss = new MockWebSocketServer()
setupWebSocketHandler(wss)
const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
const ws = new MockWebSocket()
ws.terminate.mockImplementation(() => { throw new Error('già chiusa') })
wss.emit('connection', ws)
const err = new Error('Invalid UTF8')
err.code = 'WS_ERR_INVALID_UTF8'
expect(() => ws.emit('error', err)).not.toThrow()
expect(consoleSpy).toHaveBeenCalledWith('Error closing connection:', expect.any(Error))
})
})
describe('handleAction: applyAction che lancia', () => {
it('non deve propagare l\'errore, ripristina lo stato precedente e notifica il client', async () => {
vi.resetModules()
vi.doMock('../../src/gameState.js', async (importOriginal) => {
const actual = await importOriginal()
return {
...actual,
applyAction: vi.fn((state, action) => {
if (action.type === 'boom') throw new Error('regola di gioco rotta')
return actual.applyAction(state, action)
}),
}
})
const { setupWebSocketHandler } = await import('../../src/websocket-handler.js')
const wss = new MockWebSocketServer()
const handler = setupWebSocketHandler(wss)
const consoleSpy = vi.spyOn(console, 'error').mockImplementation(() => {})
const controller = new MockWebSocket()
wss.emit('connection', controller)
wss.clients.add(controller)
controller.emit('message', JSON.stringify({ type: 'register', role: 'controller' }))
const statoPrecedente = handler.getState()
controller.send.mockClear()
expect(() => {
controller.emit('message', JSON.stringify({ type: 'action', action: { type: 'boom' } }))
}).not.toThrow()
expect(consoleSpy).toHaveBeenCalledWith('Error applying action:', expect.any(Error))
expect(handler.getState()).toEqual(statoPrecedente)
const msg = JSON.parse(controller.send.mock.calls.at(-1)[0])
expect(msg.type).toBe('error')
expect(msg.message).toContain('Failed to apply action')
})
})
})
+123
View File
@@ -365,6 +365,13 @@ describe('Game Logic (gameState.js)', () => {
expect(setDi(s).guest).toBe(0) expect(setDi(s).guest).toBe(0)
}) })
it('dovrebbe incrementare il set guest e servire guest nel nuovo set', () => {
const s = applyAction(state, { type: 'nuovoSet', team: 'guest' })
expect(setDi(s).home).toBe(0)
expect(setDi(s).guest).toBe(1)
expect(s.sp.striscia.at(-1).serv).toBe('g')
})
it('in 2/3 alla palla match registra il set vincente senza aprirne uno nuovo', () => { it('in 2/3 alla palla match registra il set vincente senza aprirne uno nuovo', () => {
state.modalitaPartita = '2/3' state.modalitaPartita = '2/3'
setSetVinti(state, 1, 0) setSetVinti(state, 1, 0)
@@ -508,6 +515,12 @@ describe('Game Logic (gameState.js)', () => {
expect(s.timeoutVideo).toBe(null) expect(s.timeoutVideo).toBe(null)
}) })
it('startTimeout senza startedAt dovrebbe lasciare timeoutStartedAt null', () => {
const s = applyAction(state, { type: 'startTimeout', team: 'home', video: 'spot.mp4' })
expect(s.timeout).toBe(true)
expect(s.timeoutStartedAt).toBe(null)
})
it('startTimeout senza squadra valida dovrebbe essere ignorato', () => { it('startTimeout senza squadra valida dovrebbe essere ignorato', () => {
for (const team of [undefined, null, 'arbitro', '']) { for (const team of [undefined, null, 'arbitro', '']) {
const s = applyAction(state, { type: 'startTimeout', team, video: 'spot.mp4', startedAt: 1000 }) const s = applyAction(state, { type: 'startTimeout', team, video: 'spot.mp4', startedAt: 1000 })
@@ -532,6 +545,46 @@ describe('Game Logic (gameState.js)', () => {
expect(s.timeout).toBe(false) expect(s.timeout).toBe(false)
expect(s.timeoutTeam).toBe(null) expect(s.timeoutTeam).toBe(null)
}) })
it('startTimeout dovrebbe registrare l\'evento nel set corrente con il punteggio', () => {
let s = state
for (let i = 0; i < 3; i++) s = applyAction(s, { type: 'incPunt', team: 'home' })
s = applyAction(s, { type: 'incPunt', team: 'guest' })
s = applyAction(s, { type: 'startTimeout', team: 'guest', startedAt: 1000 })
expect(s.sp.striscia.at(-1).timeouts).toEqual([
{ team: 'guest', punteggio: { home: 3, guest: 1 } },
])
})
it('startTimeout su time out già attivo non dovrebbe registrare un secondo evento', () => {
let s = applyAction(state, { type: 'startTimeout', team: 'home', startedAt: 1000 })
s = applyAction(s, { type: 'startTimeout', team: 'home', startedAt: 2000 })
expect(s.sp.striscia.at(-1).timeouts).toHaveLength(1)
})
it('time out successivi dovrebbero accumularsi nel set corrente', () => {
let s = applyAction(state, { type: 'startTimeout', team: 'home', startedAt: 1000 })
s = applyAction(s, { type: 'stopTimeout' })
s = applyAction(s, { type: 'incPunt', team: 'guest' })
s = applyAction(s, { type: 'startTimeout', team: 'guest', startedAt: 2000 })
expect(s.sp.striscia.at(-1).timeouts).toEqual([
{ team: 'home', punteggio: { home: 0, guest: 0 } },
{ team: 'guest', punteggio: { home: 0, guest: 1 } },
])
})
it('un nuovo set non dovrebbe ereditare i time out del precedente', () => {
let s = applyAction(state, { type: 'startTimeout', team: 'home', startedAt: 1000 })
s = applyAction(s, { type: 'stopTimeout' })
s = applyAction(s, { type: 'nuovoSet', team: 'home' })
expect(s.sp.striscia.at(-1).timeouts).toBeUndefined()
expect(s.sp.striscia.at(-2).timeouts).toHaveLength(1)
})
it('startTimeout ignorato non dovrebbe registrare eventi', () => {
const s = applyAction(state, { type: 'startTimeout', team: 'arbitro', startedAt: 1000 })
expect(s.sp.striscia.at(-1).timeouts).toBeUndefined()
})
}) })
// ============================================= // =============================================
@@ -604,6 +657,12 @@ describe('Game Logic (gameState.js)', () => {
// CAMBI GIOCATORI (confermaCambi) // CAMBI GIOCATORI (confermaCambi)
// ============================================= // =============================================
describe('confermaCambi', () => { describe('confermaCambi', () => {
it('senza array cambi non dovrebbe modificare la formazione', () => {
state.sp.form.home = ["1", "2", "3", "4", "5", "6"]
const s = applyAction(state, { type: 'confermaCambi', team: 'home' })
expect(s.sp.form.home).toEqual(["1", "2", "3", "4", "5", "6"])
})
it('dovrebbe effettuare una sostituzione valida', () => { it('dovrebbe effettuare una sostituzione valida', () => {
state.sp.form.home = ["1", "2", "3", "4", "5", "6"] state.sp.form.home = ["1", "2", "3", "4", "5", "6"]
const s = applyAction(state, { const s = applyAction(state, {
@@ -705,6 +764,64 @@ describe('Game Logic (gameState.js)', () => {
expect(s.sp.form.home).not.toContain("1") expect(s.sp.form.home).not.toContain("1")
expect(s.sp.form.home).not.toContain("10") expect(s.sp.form.home).not.toContain("10")
}) })
it('dovrebbe registrare il cambio nel set corrente con il punteggio', () => {
let s = state
for (let i = 0; i < 2; i++) s = applyAction(s, { type: 'incPunt', team: 'home' })
s = applyAction(s, { type: 'incPunt', team: 'guest' })
s = applyAction(s, {
type: 'confermaCambi',
team: 'home',
cambi: [{ in: "10", out: "3" }]
})
expect(s.sp.striscia.at(-1).cambi).toEqual([
{ team: 'home', in: "10", out: "3", punteggio: { home: 2, guest: 1 } },
])
})
it('dovrebbe registrare ogni cambio di una doppia sostituzione', () => {
const s = applyAction(state, {
type: 'confermaCambi',
team: 'guest',
cambi: [
{ in: "10", out: "1" },
{ in: "11", out: "2" }
]
})
expect(s.sp.striscia.at(-1).cambi).toEqual([
{ team: 'guest', in: "10", out: "1", punteggio: { home: 0, guest: 0 } },
{ team: 'guest', in: "11", out: "2", punteggio: { home: 0, guest: 0 } },
])
})
it('non dovrebbe registrare nulla se la validazione fallisce', () => {
const s = applyAction(state, {
type: 'confermaCambi',
team: 'home',
cambi: [{ in: "abc", out: "1" }]
})
expect(s.sp.striscia.at(-1).cambi).toBeUndefined()
})
it('non dovrebbe registrare nulla se tutti i cambi sono vuoti', () => {
const s = applyAction(state, {
type: 'confermaCambi',
team: 'home',
cambi: [{ in: "", out: "" }]
})
expect(s.sp.striscia.at(-1).cambi).toBeUndefined()
})
it('un nuovo set non dovrebbe ereditare i cambi del precedente', () => {
let s = applyAction(state, {
type: 'confermaCambi',
team: 'home',
cambi: [{ in: "10", out: "3" }]
})
s = applyAction(s, { type: 'nuovoSet', team: 'home' })
expect(s.sp.striscia.at(-1).cambi).toBeUndefined()
expect(s.sp.striscia.at(-2).cambi).toHaveLength(1)
})
}) })
// ============================================= // =============================================
@@ -843,6 +960,12 @@ describe('Game Logic (gameState.js)', () => {
expect(s.sp.striscia[0].ris).toBe('') expect(s.sp.striscia[0].ris).toBe('')
}) })
it('con striscia vuota dovrebbe usare "h" come servizio iniziale di default', () => {
state.sp.striscia = []
const s = applyAction(state, { type: 'resetta' })
expect(s.sp.striscia).toEqual([{ serv: 'h', ris: '', vinc: null }])
})
it('dovrebbe impostare visuForm a false', () => { it('dovrebbe impostare visuForm a false', () => {
state.visuForm = true state.visuForm = true
const s = applyAction(state, { type: 'resetta' }) const s = applyAction(state, { type: 'resetta' })
+45 -4
View File
@@ -21,10 +21,10 @@ describe('buildRefertoHtml (referto.js)', () => {
{ serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(18), vinc: 'h' }, { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(18), vinc: 'h' },
] ]
const html = buildRefertoHtml(statoConSet(striscia), NOW) const html = buildRefertoHtml(statoConSet(striscia), NOW)
// due set reali → "Set 1" e "Set 2", mai "Set 3" // due set reali → numeri "1" e "2" nella barra SET, mai "3"
expect(html).toContain('Set 1') expect(html).toContain('<span class="set-numero">1</span>')
expect(html).toContain('Set 2') expect(html).toContain('<span class="set-numero">2</span>')
expect(html).not.toContain('Set 3') expect(html).not.toContain('<span class="set-numero">3</span>')
}) })
it('calcola il punteggio finale di ogni set dalla ris', () => { it('calcola il punteggio finale di ogni set dalla ris', () => {
@@ -60,6 +60,15 @@ describe('buildRefertoHtml (referto.js)', () => {
expect(html).toContain('1 0') expect(html).toContain('1 0')
}) })
it('ricava il vincitore guest dal conteggio punti se vinc è nullo', () => {
const striscia = [
{ serv: 'h', ris: 'g'.repeat(25) + 'h'.repeat(20), vinc: null },
]
const html = buildRefertoHtml(statoConSet(striscia), NOW)
// conteggio punti guest > home, pur con vinc null → 0 1
expect(html).toContain('0 1')
})
it('include la progressione punto-punto con classi per squadra', () => { it('include la progressione punto-punto con classi per squadra', () => {
const striscia = [ const striscia = [
{ serv: 'h', ris: 'hhg', vinc: null }, { serv: 'h', ris: 'hhg', vinc: null },
@@ -97,6 +106,38 @@ describe('buildRefertoHtml (referto.js)', () => {
expect(html).toContain('Nessun punto registrato') expect(html).toContain('Nessun punto registrato')
}) })
it('elenca time out e cambi del set con squadra e punteggio', () => {
const striscia = [{
serv: 'h', ris: 'hhhg', vinc: null,
timeouts: [{ team: 'guest', punteggio: { home: 3, guest: 1 } }],
cambi: [{ team: 'home', in: '10', out: '3', punteggio: { home: 2, guest: 0 } }],
}]
const html = buildRefertoHtml(statoConSet(striscia), NOW)
expect(html).toContain('Cambi e time out')
expect(html).toContain('Time out <strong>Rivali</strong> sul 3-1')
expect(html).toContain('Cambio <strong>Antoniana</strong>: entra 10 per 3 sul 2-0')
})
it('ordina gli eventi per punteggio crescente', () => {
const striscia = [{
serv: 'h', ris: 'hhhgg', vinc: null,
timeouts: [{ team: 'home', punteggio: { home: 3, guest: 2 } }],
cambi: [{ team: 'guest', in: '9', out: '4', punteggio: { home: 1, guest: 0 } }],
}]
const html = buildRefertoHtml(statoConSet(striscia), NOW)
const idxCambio = html.indexOf('entra 9 per 4')
const idxTimeout = html.indexOf('Time out')
expect(idxCambio).toBeGreaterThan(-1)
expect(idxTimeout).toBeGreaterThan(-1)
expect(idxCambio).toBeLessThan(idxTimeout)
})
it('omette la sezione eventi se il set non ne ha', () => {
const striscia = [{ serv: 'h', ris: 'h', vinc: null }]
const html = buildRefertoHtml(statoConSet(striscia), NOW)
expect(html).not.toContain('Cambi e time out')
})
it('header contiene nomi squadre, modalità e data iniettata', () => { it('header contiene nomi squadre, modalità e data iniettata', () => {
const striscia = [{ serv: 'h', ris: '', vinc: null }] const striscia = [{ serv: 'h', ris: '', vinc: null }]
const state = statoConSet(striscia) const state = statoConSet(striscia)
+81
View File
@@ -0,0 +1,81 @@
import { describe, it, expect, vi, afterEach } from 'vitest'
// getNetworkIPs() senza argomenti dipende dalla piattaforma (fs, child_process,
// os.networkInterfaces): qui mockiamo quei moduli per esercitare in modo
// deterministico i rami WSL/non-WSL, altrimenti scoperti dagli altri test
// (che iniettano sempre `nets` per bypassare questo codice).
describe('server-utils.js — rilevamento piattaforma', () => {
afterEach(() => {
vi.restoreAllMocks()
vi.resetModules()
vi.doUnmock('fs')
vi.doUnmock('child_process')
vi.doUnmock('os')
})
it('senza WSL usa os.networkInterfaces()', async () => {
vi.doMock('fs', () => ({
existsSync: vi.fn(() => false),
readFileSync: vi.fn(),
}))
vi.doMock('child_process', () => ({
execSync: vi.fn(),
}))
vi.doMock('os', () => ({
networkInterfaces: vi.fn(() => ({
eth0: [{ family: 'IPv4', internal: false, address: '10.0.0.9' }],
})),
}))
const { getNetworkIPs } = await import('../../src/server-utils.js')
expect(getNetworkIPs()).toEqual(['10.0.0.9'])
})
it('su WSL interroga PowerShell e filtra gli IP LAN', async () => {
vi.doMock('fs', () => ({
existsSync: vi.fn(() => true),
readFileSync: vi.fn(() => 'Microsoft WSL2'),
}))
vi.doMock('child_process', () => ({
execSync: vi.fn(() => Buffer.from('192.168.1.42\n127.0.0.1\n172.17.0.1\n')),
}))
vi.doMock('os', () => ({
networkInterfaces: vi.fn(),
}))
const { getNetworkIPs } = await import('../../src/server-utils.js')
expect(getNetworkIPs()).toEqual(['192.168.1.42'])
})
it('su WSL se PowerShell fallisce restituisce array vuoto', async () => {
vi.doMock('fs', () => ({
existsSync: vi.fn(() => true),
readFileSync: vi.fn(() => 'Microsoft WSL2'),
}))
vi.doMock('child_process', () => ({
execSync: vi.fn(() => { throw new Error('powershell non disponibile') }),
}))
vi.doMock('os', () => ({
networkInterfaces: vi.fn(),
}))
const { getNetworkIPs } = await import('../../src/server-utils.js')
expect(getNetworkIPs()).toEqual([])
})
it('se la lettura di /proc/sys/kernel/osrelease lancia, tratta la piattaforma come non-WSL', async () => {
vi.doMock('fs', () => ({
existsSync: vi.fn(() => true),
readFileSync: vi.fn(() => { throw new Error('permesso negato') }),
}))
vi.doMock('child_process', () => ({
execSync: vi.fn(),
}))
vi.doMock('os', () => ({
networkInterfaces: vi.fn(() => ({})),
}))
const { getNetworkIPs } = await import('../../src/server-utils.js')
expect(getNetworkIPs()).toEqual([])
})
})