diff --git a/server.js b/server.js index 82df178..6198b49 100644 --- a/server.js +++ b/server.js @@ -1,7 +1,7 @@ import { createServer } from 'http' import express from 'express' import { WebSocketServer } from 'ws' -import { fileURLToPath } from 'url' +import { fileURLToPath, pathToFileURL } from 'url' import { dirname, join } from 'path' import { setupWebSocketHandler } from './src/websocket-handler.js' import { printServerInfo } from './src/server-utils.js' @@ -10,36 +10,52 @@ import { loadState, saveState } from './src/persist.js' const __filename = fileURLToPath(import.meta.url) const __dirname = dirname(__filename) -const PORT = process.env.PORT || 3000 -const distDir = join(__dirname, 'dist') +const DIST_DIR = join(__dirname, 'dist') -const app = express() +// Crea l'app Express (asset statici + route display/controller) senza avviare il +// listen né il WebSocket: così il routing è testabile in isolamento. +export function createApp(distDir = DIST_DIR) { + const app = express() -app.use(express.static(distDir, { index: false })) + app.use(express.static(distDir, { index: false })) -app.get(['/', '/display', '/display/*splat'], (_req, res) => { - res.sendFile(join(distDir, 'index.html')) -}) + app.get(['/', '/display', '/display/*splat'], (_req, res) => { + res.sendFile(join(distDir, 'index.html')) + }) -app.get(['/controller', '/controller/*splat'], (_req, res) => { - res.sendFile(join(distDir, 'controller.html')) -}) + app.get(['/controller', '/controller/*splat'], (_req, res) => { + res.sendFile(join(distDir, 'controller.html')) + }) -const server = createServer(app) -const wss = new WebSocketServer({ noServer: true }) -setupWebSocketHandler(wss, { initialState: loadState(), onStateChange: saveState }) + return app +} -server.on('upgrade', (request, socket, head) => { - const pathname = new URL(request.url, `http://${request.headers.host}`).pathname - if (pathname === '/ws') { - wss.handleUpgrade(request, socket, head, (ws) => { - wss.emit('connection', ws, request) - }) - } else { - socket.destroy() - } -}) +// Avvia HTTP + WebSocket. Lo stato viene caricato da disco e ripersistito ad ogni azione. +export function startServer(port = process.env.PORT || 3000) { + const app = createApp() + const server = createServer(app) + const wss = new WebSocketServer({ noServer: true }) + setupWebSocketHandler(wss, { initialState: loadState(), onStateChange: saveState }) -server.listen(PORT, '0.0.0.0', () => { - printServerInfo(PORT) -}) + server.on('upgrade', (request, socket, head) => { + const pathname = new URL(request.url, `http://${request.headers.host}`).pathname + if (pathname === '/ws') { + wss.handleUpgrade(request, socket, head, (ws) => { + wss.emit('connection', ws, request) + }) + } else { + socket.destroy() + } + }) + + server.listen(port, '0.0.0.0', () => { + printServerInfo(port) + }) + + return server +} + +// Avvia solo se eseguito direttamente (`node server.js`), non quando importato nei test. +if (process.argv[1] && import.meta.url === pathToFileURL(process.argv[1]).href) { + startServer() +} diff --git a/src/referto.js b/src/referto.js index 6c6a431..c08e5ec 100644 --- a/src/referto.js +++ b/src/referto.js @@ -7,9 +7,11 @@ function vincitoreSet(s) { return null } -export function generaReferto(state) { +// Costruisce l'HTML del referto (funzione pura, testabile). +// `now` è iniettabile per rendere deterministica la data nei test. +export function buildRefertoHtml(state, now = new Date()) { const { sp, modalitaPartita } = state - const { nomi, striscia, form } = sp + const { nomi, striscia } = sp const setReali = striscia.filter(s => !s._phantom) @@ -20,7 +22,7 @@ export function generaReferto(state) { else if (v === 'g') setVinti.guest++ } - const dataOra = new Date().toLocaleString('it-IT', { + const dataOra = now.toLocaleString('it-IT', { day: '2-digit', month: '2-digit', year: 'numeric', hour: '2-digit', minute: '2-digit', }) @@ -125,6 +127,13 @@ export function generaReferto(state) { ` + return html +} + +// Apre il referto in una nuova scheda e avvia la stampa (effetto collaterale, +// solo browser). La generazione dell'HTML è delegata a buildRefertoHtml. +export function generaReferto(state) { + const html = buildRefertoHtml(state) const w = window.open('', '_blank') w.document.write(html) w.document.close() diff --git a/src/server-utils.js b/src/server-utils.js index 63fb3ec..7c5d8f0 100644 --- a/src/server-utils.js +++ b/src/server-utils.js @@ -9,7 +9,34 @@ function isWSL() { } catch { return false } } -export function getNetworkIPs() { +// Un IPv4 è "pubblicabile" in LAN se non è loopback, link-local o bridge Docker. +function isLanIPv4(address) { + return !!address + && !address.startsWith('127.') + && !address.startsWith('169.254.') + && !address.startsWith('172.') +} + +// Estrae gli IP LAN da un oggetto in stile os.networkInterfaces(). +// Esportata per poter essere testata in isolamento, senza dipendere dalla piattaforma. +export function collectIPs(nets) { + const out = [] + for (const name of Object.keys(nets || {})) { + for (const net of nets[name]) { + if (net.family === 'IPv4' && !net.internal && isLanIPv4(net.address)) { + out.push(net.address) + } + } + } + return out +} + +// Restituisce gli IP di rete della macchina. +// Passando `nets` (oggetto in stile os.networkInterfaces) si forza il percorso +// deterministico, scavalcando il rilevamento WSL/PowerShell: utile nei test. +export function getNetworkIPs(nets) { + if (nets) return collectIPs(nets) + if (isWSL()) { try { const out = execSync( @@ -18,23 +45,15 @@ export function getNetworkIPs() { ) return out.toString().trim().split('\n') .map(s => s.trim()) - .filter(ip => ip && !ip.startsWith('127.') && !ip.startsWith('169.254.') && !ip.startsWith('172.')) + .filter(isLanIPv4) } catch { return [] } } - const nets = networkInterfaces() - const networkIPs = [] - for (const name of Object.keys(nets)) { - for (const net of nets[name]) { - if (net.family === 'IPv4' && !net.internal) networkIPs.push(net.address) - } - } - return networkIPs + return collectIPs(networkInterfaces()) } -export function printServerInfo(port = 3000) { - const networkIPs = getNetworkIPs() - +// `networkIPs` è iniettabile per rendere la stampa testabile in modo deterministico. +export function printServerInfo(port = 3000, networkIPs = getNetworkIPs()) { console.log(`\nSegnapunti Server`) console.log(` Display: http://127.0.0.1:${port}/display`) console.log(` Controller: http://127.0.0.1:${port}/controller`) diff --git a/tests/component/ControllerPage.test.js b/tests/component/ControllerPage.test.js index d3877c2..3cad251 100644 --- a/tests/component/ControllerPage.test.js +++ b/tests/component/ControllerPage.test.js @@ -2,6 +2,10 @@ import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' import { mount } from '@vue/test-utils' import ControllerPage from '../../src/components/ControllerPage.vue' +import { generaReferto } from '../../src/referto.js' + +// Il referto apre una finestra/print: lo mockiamo per testarne solo l'invocazione. +vi.mock('../../src/referto.js', () => ({ generaReferto: vi.fn() })) // Mock globale WebSocket per jsdom class MockWebSocket { @@ -25,6 +29,21 @@ class MockWebSocket { vi.stubGlobal('WebSocket', MockWebSocket) +// Forza l'orientamento portrait → il controller usa il layout "mobile" +// (con .team-pts, .btn-ctrl, ecc.) su cui questi test fanno asserzioni. +Object.defineProperty(window, 'innerWidth', { value: 400, writable: true, configurable: true }) +Object.defineProperty(window, 'innerHeight', { value: 800, writable: true, configurable: true }) + +// Imposta il punteggio del set in corso costruendo una ris coerente. +// `serv` ('h'|'g') controlla l'ultimo punto, quindi chi risulta al servizio. +function setScore(wrapper, home, guest, serv = 'h') { + const altro = serv === 'h' ? 'g' : 'h' + const nAltro = serv === 'h' ? guest : home + const nServ = serv === 'h' ? home : guest + // mette per ultimo il carattere del battitore desiderato + wrapper.vm.state.sp.striscia.at(-1).ris = altro.repeat(nAltro) + serv.repeat(nServ) +} + // Helper per creare il componente con stato personalizzato function mountController(stateOverrides = {}) { const wrapper = mount(ControllerPage, { @@ -105,7 +124,7 @@ describe('ControllerPage.vue', () => { it('dovrebbe essere disabilitato se il punteggio non è 0-0', async () => { const wrapper = mountController() - wrapper.vm.state.sp.punt.home = 5 + setScore(wrapper, 5, 0) await wrapper.vm.$nextTick() const btn = wrapper.findAll('.btn-ctrl').find(b => b.text().includes('Cambio Palla')) expect(btn.attributes('disabled')).toBeDefined() @@ -194,8 +213,7 @@ describe('ControllerPage.vue', () => { it('dovrebbe generare "N pari" a punteggio uguale', () => { const wrapper = mountController() - wrapper.vm.state.sp.punt.home = 5 - wrapper.vm.state.sp.punt.guest = 5 + setScore(wrapper, 5, 5) wrapper.vm.wsConnected = true wrapper.vm.ws = { readyState: 1, send: vi.fn() } wrapper.vm.speak() @@ -205,9 +223,7 @@ describe('ControllerPage.vue', () => { it('dovrebbe annunciare prima il punteggio di chi batte (home serve)', () => { const wrapper = mountController() - wrapper.vm.state.sp.punt.home = 15 - wrapper.vm.state.sp.punt.guest = 10 - wrapper.vm.state.sp.servHome = true + setScore(wrapper, 15, 10, 'h') wrapper.vm.wsConnected = true wrapper.vm.ws = { readyState: 1, send: vi.fn() } wrapper.vm.speak() @@ -217,9 +233,7 @@ describe('ControllerPage.vue', () => { it('dovrebbe annunciare prima il punteggio di chi batte (guest serve)', () => { const wrapper = mountController() - wrapper.vm.state.sp.punt.home = 10 - wrapper.vm.state.sp.punt.guest = 15 - wrapper.vm.state.sp.servHome = false + setScore(wrapper, 10, 15, 'g') wrapper.vm.wsConnected = true wrapper.vm.ws = { readyState: 1, send: vi.fn() } wrapper.vm.speak() @@ -228,6 +242,51 @@ describe('ControllerPage.vue', () => { }) }) + // ============================================= + // REFERTO (modal PARTITA FINITA) + // ============================================= + describe('Referto', () => { + // Porta il componente allo stato "partita finita" per home in 2/3 + function setPartitaFinita(wrapper) { + 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 + } + + it('mostra il bottone REFERTO quando la partita è finita', async () => { + const wrapper = mountController() + setPartitaFinita(wrapper) + await wrapper.vm.$nextTick() + expect(wrapper.vm.isPartitaFinita).toBe(true) + const btn = wrapper.findAll('.btn-secondary').find(b => b.text().includes('REFERTO')) + expect(btn).toBeDefined() + }) + + it('il click su REFERTO invoca generaReferto con lo stato', async () => { + const wrapper = mountController() + setPartitaFinita(wrapper) + await wrapper.vm.$nextTick() + const btn = wrapper.findAll('.btn-secondary').find(b => b.text().includes('REFERTO')) + await btn.trigger('click') + expect(generaReferto).toHaveBeenCalledWith(wrapper.vm.state) + }) + + it('NON mostra il bottone REFERTO a set vinto (partita non finita)', async () => { + const wrapper = mountController() + wrapper.vm.state.modalitaPartita = '3/5' + wrapper.vm.setVintoTeam = 'home' + wrapper.vm.showSetVinto = true + await wrapper.vm.$nextTick() + expect(wrapper.vm.isPartitaFinita).toBe(false) + const btn = wrapper.findAll('.btn-secondary').find(b => b.text().includes('REFERTO')) + expect(btn).toBeUndefined() + }) + }) + // ============================================= // BARRA CONNESSIONE // ============================================= diff --git a/tests/component/DisplayPage.test.js b/tests/component/DisplayPage.test.js index a7468f2..4b8f52d 100644 --- a/tests/component/DisplayPage.test.js +++ b/tests/component/DisplayPage.test.js @@ -75,8 +75,8 @@ describe('DisplayPage.vue', () => { it('dovrebbe aggiornare il punteggio quando lo stato cambia', async () => { const wrapper = mountDisplay() - wrapper.vm.state.sp.punt.home = 15 - wrapper.vm.state.sp.punt.guest = 12 + // il punteggio si ricava dalla striscia: 15 punti home + 12 guest + wrapper.vm.state.sp.striscia.at(-1).ris = 'h'.repeat(15) + 'g'.repeat(12) await wrapper.vm.$nextTick() const punti = wrapper.findAll('.punt') expect(punti[0].text()).toBe('15') diff --git a/tests/component/wsMixin.test.js b/tests/component/wsMixin.test.js new file mode 100644 index 0000000..71b88c7 --- /dev/null +++ b/tests/component/wsMixin.test.js @@ -0,0 +1,174 @@ +// @vitest-environment happy-dom +import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' +import { mount } from '@vue/test-utils' +import { createWsMixin } from '../../src/wsMixin.js' +import { createInitialState } from '../../src/gameState.js' + +// WebSocket mock controllabile: i gestori (onopen/onmessage/onclose) vengono +// assegnati dal mixin e li invochiamo manualmente nei test. +class MockWebSocket { + static CONNECTING = 0 + static OPEN = 1 + static CLOSING = 2 + static CLOSED = 3 + constructor(url) { + this.url = url + this.readyState = MockWebSocket.CONNECTING + this.send = vi.fn() + this.close = vi.fn() + MockWebSocket.instances.push(this) + } +} +MockWebSocket.instances = [] + +vi.stubGlobal('WebSocket', MockWebSocket) + +// Monta un componente che usa il mixin. `extra` permette di aggiungere hook. +function mountWith(role = 'controller', extra = {}) { + const Comp = { + mixins: [createWsMixin(role)], + template: '
', + ...extra, + } + return mount(Comp) +} + +function ultimaWs() { + return MockWebSocket.instances.at(-1) +} + +describe('createWsMixin (wsMixin.js)', () => { + beforeEach(() => { + MockWebSocket.instances = [] + }) + + afterEach(() => { + vi.restoreAllMocks() + }) + + describe('computed derivati', () => { + it('punt/servHome/set delegano alle funzioni pure sulla striscia', async () => { + const wrapper = mountWith() + wrapper.vm.state.sp.striscia = [ + { serv: 'h', ris: 'h', vinc: 'h' }, + { serv: 'h', ris: 'h'.repeat(10) + 'g'.repeat(8), vinc: null }, + ] + await wrapper.vm.$nextTick() + expect(wrapper.vm.punt).toEqual({ home: 10, guest: 8 }) + expect(wrapper.vm.set).toEqual({ home: 1, guest: 0 }) + // ultimo punto 'g' → serve guest + expect(wrapper.vm.servHome).toBe(false) + }) + }) + + describe('connessione', () => { + it('apre una WebSocket verso /ws al mount', () => { + mountWith() + const ws = ultimaWs() + expect(ws).toBeDefined() + expect(ws.url).toMatch(/^ws:\/\/.+\/ws$/) + }) + + it('invia il messaggio register all\'apertura', () => { + const wrapper = mountWith('controller') + const ws = ultimaWs() + ws.readyState = MockWebSocket.OPEN + ws.onopen() + expect(ws.send).toHaveBeenCalledTimes(1) + const msg = JSON.parse(ws.send.mock.calls[0][0]) + expect(msg).toEqual({ type: 'register', role: 'controller' }) + expect(wrapper.vm.wsConnected).toBe(true) + }) + + it('un messaggio "state" aggiorna lo stato locale', () => { + const wrapper = mountWith() + const ws = ultimaWs() + const nuovo = createInitialState() + nuovo.sp.nomi.home = 'Nuova Squadra' + ws.onmessage({ data: JSON.stringify({ type: 'state', state: nuovo }) }) + expect(wrapper.vm.state.sp.nomi.home).toBe('Nuova Squadra') + }) + + it('un messaggio non-state invoca l\'hook onWsMessage', () => { + const onWsMessage = vi.fn() + const wrapper = mountWith('display', { methods: { onWsMessage } }) + const ws = ultimaWs() + ws.onmessage({ data: JSON.stringify({ type: 'speak', text: 'ciao' }) }) + expect(onWsMessage).toHaveBeenCalledWith({ type: 'speak', text: 'ciao' }) + }) + }) + + describe('riconnessione', () => { + it('scheduleReconnect usa backoff esponenziale con cap a 30s', () => { + const wrapper = mountWith() + const delays = [] + const spy = vi.spyOn(globalThis, 'setTimeout').mockImplementation(() => 123) + // intercetta i delay leggendoli dalle chiamate + spy.mockImplementation((_fn, d) => { delays.push(d); return 123 }) + + wrapper.vm.reconnectAttempts = 0 + wrapper.vm.reconnectTimeout = null + wrapper.vm.scheduleReconnect() // 1000 + wrapper.vm.reconnectTimeout = null + wrapper.vm.scheduleReconnect() // 2000 + wrapper.vm.reconnectTimeout = null + wrapper.vm.scheduleReconnect() // 4000 + expect(delays).toEqual([1000, 2000, 4000]) + + // attempts alto → cap a 30000 + delays.length = 0 + wrapper.vm.reconnectAttempts = 20 + wrapper.vm.reconnectTimeout = null + wrapper.vm.scheduleReconnect() + expect(delays[0]).toBe(30000) + + spy.mockRestore() + }) + + it('non riconnette su chiusura pulita (1000/1001)', () => { + const wrapper = mountWith() + const ws = ultimaWs() + const spy = vi.spyOn(wrapper.vm, 'scheduleReconnect') + ws.onclose({ code: 1000 }) + ws.onclose({ code: 1001 }) + expect(spy).not.toHaveBeenCalled() + expect(wrapper.vm.wsConnected).toBe(false) + }) + + it('riconnette su chiusura anomala (es. 1006)', () => { + const wrapper = mountWith() + const ws = ultimaWs() + const spy = vi.spyOn(wrapper.vm, 'scheduleReconnect') + ws.onclose({ code: 1006 }) + expect(spy).toHaveBeenCalled() + }) + }) + + describe('sendWs', () => { + it('ritorna false se non connesso', () => { + const wrapper = mountWith() + wrapper.vm.wsConnected = false + expect(wrapper.vm.sendWs({ type: 'action' })).toBe(false) + }) + + it('serializza e invia se connesso e aperto', () => { + const wrapper = mountWith() + const ws = ultimaWs() + ws.readyState = MockWebSocket.OPEN + wrapper.vm.wsConnected = true + const ok = wrapper.vm.sendWs({ type: 'action', action: { type: 'incPunt' } }) + expect(ok).toBe(true) + const inviato = JSON.parse(ws.send.mock.calls.at(-1)[0]) + expect(inviato.type).toBe('action') + }) + }) + + describe('cleanup', () => { + it('beforeUnmount chiude la WebSocket', () => { + const wrapper = mountWith() + const ws = ultimaWs() + wrapper.unmount() + expect(ws.close).toHaveBeenCalled() + }) + }) +}) diff --git a/tests/e2e/accessibility.spec.cjs b/tests/e2e/accessibility.spec.cjs index 886408a..5599b12 100644 --- a/tests/e2e/accessibility.spec.cjs +++ b/tests/e2e/accessibility.spec.cjs @@ -17,7 +17,8 @@ test.describe('Accessibility (a11y)', () => { }); test('Controller: non dovrebbe avere violazioni critiche a11y', async ({ page }) => { - await page.goto('http://localhost:3001'); + await page.setViewportSize({ width: 390, height: 844 }); + await page.goto('http://localhost:3000/controller'); await page.waitForTimeout(500); const results = await new AxeBuilder({ page }) @@ -42,7 +43,8 @@ test.describe('Accessibility (a11y)', () => { }); test('Controller: i touch target dovrebbero avere dimensione minima', async ({ page }) => { - await page.goto('http://localhost:3001'); + await page.setViewportSize({ width: 390, height: 844 }); + await page.goto('http://localhost:3000/controller'); await page.waitForSelector('.conn-bar.connected'); // Controlla che i bottoni principali abbiano dimensione minima 44x44px @@ -57,7 +59,8 @@ test.describe('Accessibility (a11y)', () => { }); test('Controller: i bottoni punteggio dovrebbero avere dimensione adeguata', async ({ page }) => { - await page.goto('http://localhost:3001'); + await page.setViewportSize({ width: 390, height: 844 }); + await page.goto('http://localhost:3000/controller'); await page.waitForSelector('.conn-bar.connected'); const scoreButtons = page.locator('.team-score'); diff --git a/tests/e2e/basic-flow.spec.cjs b/tests/e2e/basic-flow.spec.cjs index 386668c..fdb3e03 100644 --- a/tests/e2e/basic-flow.spec.cjs +++ b/tests/e2e/basic-flow.spec.cjs @@ -7,7 +7,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await expect(displayPage).toHaveTitle(/Segnapunti/); await expect(controllerPage).toHaveTitle(/Controller/); @@ -15,7 +16,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => { test('il punteggio iniziale dovrebbe essere 0-0', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); // Attende la connessione WebSocket await controllerPage.waitForSelector('.conn-bar.connected'); @@ -31,7 +33,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); // Attende la connessione WebSocket del controller await controllerPage.waitForSelector('.conn-bar.connected'); @@ -42,6 +45,11 @@ test.describe('Basic Flow: Controller ↔ Display', () => { if (await btnConfirm.isVisible()) { await btnConfirm.click(); } + // doReset apre automaticamente il dialog di configurazione: chiudilo + const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel'); + if (await cfgCancel.isVisible()) { + await cfgCancel.click(); + } await controllerPage.waitForTimeout(200); // Click +1 Home @@ -60,7 +68,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); @@ -70,6 +79,11 @@ test.describe('Basic Flow: Controller ↔ Display', () => { if (await btnConfirm.isVisible()) { await btnConfirm.click(); } + // doReset apre automaticamente il dialog di configurazione: chiudilo + const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel'); + if (await cfgCancel.isVisible()) { + await cfgCancel.click(); + } await controllerPage.waitForTimeout(200); // Click +1 Guest @@ -88,7 +102,8 @@ test.describe('Basic Flow: Controller ↔ Display', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); @@ -98,6 +113,11 @@ test.describe('Basic Flow: Controller ↔ Display', () => { if (await btnConfirm.isVisible()) { await btnConfirm.click(); } + // doReset apre automaticamente il dialog di configurazione: chiudilo + const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel'); + if (await cfgCancel.isVisible()) { + await cfgCancel.click(); + } await controllerPage.waitForTimeout(200); // Home +1, Guest +1, Home +1 diff --git a/tests/e2e/full-match.spec.cjs b/tests/e2e/full-match.spec.cjs index e6c12ba..838747d 100644 --- a/tests/e2e/full-match.spec.cjs +++ b/tests/e2e/full-match.spec.cjs @@ -7,6 +7,11 @@ async function resetGame(controllerPage) { if (await btnConfirm.isVisible()) { await btnConfirm.click(); } + // doReset apre automaticamente il dialog di configurazione: chiudilo + const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel'); + if (await cfgCancel.isVisible()) { + await cfgCancel.click(); + } await controllerPage.waitForTimeout(300); } @@ -36,7 +41,8 @@ test.describe('Full Match Simulation', () => { test('Partita 2/3: Home vince 2 set a 0', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); await resetGame(controllerPage); @@ -64,7 +70,8 @@ test.describe('Full Match Simulation', () => { test('Set decisivo 2/3: vittoria a 15 punti', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); await resetGame(controllerPage); @@ -101,7 +108,8 @@ test.describe('Full Match Simulation', () => { test('Set normale: punti oltre 25 fino ai vantaggi', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); await resetGame(controllerPage); diff --git a/tests/e2e/game-operations.spec.cjs b/tests/e2e/game-operations.spec.cjs index 0f9de2f..1e5984c 100644 --- a/tests/e2e/game-operations.spec.cjs +++ b/tests/e2e/game-operations.spec.cjs @@ -7,6 +7,11 @@ async function resetGame(controllerPage) { if (await btnConfirm.isVisible()) { await btnConfirm.click(); } + // doReset apre automaticamente il dialog di configurazione: chiudilo + const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel'); + if (await cfgCancel.isVisible()) { + await cfgCancel.click(); + } await controllerPage.waitForTimeout(300); } @@ -14,7 +19,8 @@ test.describe('Game Operations', () => { test('Undo: dovrebbe annullare l\'ultimo punto', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); await resetGame(controllerPage); @@ -32,7 +38,8 @@ test.describe('Game Operations', () => { test('Reset: dovrebbe azzerare tutto dopo conferma', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); // Imposta qualche punto @@ -54,7 +61,8 @@ test.describe('Game Operations', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); // Apri config @@ -84,7 +92,8 @@ test.describe('Game Operations', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); // Inizialmente mostra punteggio, non formazione @@ -103,7 +112,8 @@ test.describe('Game Operations', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); // Inizialmente la striscia è visibile @@ -125,7 +135,8 @@ test.describe('Game Operations', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); await resetGame(controllerPage); @@ -157,7 +168,8 @@ test.describe('Game Operations', () => { test('Cambi: dovrebbe mostrare errore per giocatore non in formazione', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); await resetGame(controllerPage); diff --git a/tests/e2e/game-simulation.spec.cjs b/tests/e2e/game-simulation.spec.cjs index f705a0f..4e135ce 100644 --- a/tests/e2e/game-simulation.spec.cjs +++ b/tests/e2e/game-simulation.spec.cjs @@ -7,7 +7,8 @@ test.describe('Game Simulation', () => { const controllerPage = await context.newPage(); await displayPage.goto('http://localhost:3000'); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); // Selettori (basati su ID ipotetici o classi, adattali al tuo HTML reale) // Assumo che nel DOM ci siano elementi con ID o classi riconoscibili @@ -23,6 +24,12 @@ test.describe('Game Simulation', () => { await btnConfirmReset.click(); } } + // doReset apre automaticamente il dialog di configurazione: chiudilo + const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel'); + if (await cfgCancel.isVisible()) { + await cfgCancel.click(); + } + await controllerPage.waitForTimeout(200); // 2. Loop per vincere il primo set (25 punti) // In ControllerPage.vue, il click su .team-score.home-bg incrementa i punti home @@ -55,15 +62,15 @@ test.describe('Game Simulation', () => { // 2. Cliccare "SET HOME". // 3. Verificare che Set Home = 1. - // Verifica che siamo a 25 + // A 25-0 compare automaticamente il dialog "SET VINTO" await expect(controllerPage.locator('.team-score.home-bg .team-pts')).toHaveText('25'); + await controllerPage.waitForSelector('.dialog-winner'); - // Clicca bottone SET - const btnSetHome = controllerPage.locator('.btn-set.home-bg'); - await btnSetHome.click(); + // Procedi al set successivo: registra il set vinto da Home + await controllerPage.getByText('VAI AL SET SUCCESSIVO').click(); + await controllerPage.waitForTimeout(300); - // Verifica che il set sia incrementato - // Nota: display potrebbe chiamarsi diversamente, controlliamo Controller per coerenza + // Verifica che il set Home sia incrementato a 1 await expect(controllerPage.locator('.team-score.home-bg .team-set')).toContainText('SET 1'); }); }); diff --git a/tests/e2e/visual-regression.spec.cjs b/tests/e2e/visual-regression.spec.cjs index 2c1443d..be2f68a 100644 --- a/tests/e2e/visual-regression.spec.cjs +++ b/tests/e2e/visual-regression.spec.cjs @@ -7,6 +7,11 @@ async function resetGame(controllerPage) { if (await btnConfirm.isVisible()) { await btnConfirm.click(); } + // doReset apre automaticamente il dialog di configurazione: chiudilo + const cfgCancel = controllerPage.locator('.dialog-config .btn-cancel'); + if (await cfgCancel.isVisible()) { + await cfgCancel.click(); + } await controllerPage.waitForTimeout(300); } @@ -16,7 +21,8 @@ test.describe('Visual Regression', () => { const controllerPage = await context.newPage(); const displayPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await displayPage.goto('http://localhost:3000'); await controllerPage.waitForSelector('.conn-bar.connected'); @@ -35,7 +41,8 @@ test.describe('Visual Regression', () => { const controllerPage = await context.newPage(); const displayPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await displayPage.goto('http://localhost:3000'); await controllerPage.waitForSelector('.conn-bar.connected'); @@ -59,7 +66,8 @@ test.describe('Visual Regression', () => { test('Controller: screenshot stato iniziale', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); await resetGame(controllerPage); @@ -71,7 +79,8 @@ test.describe('Visual Regression', () => { test('Controller: screenshot con modal config aperta', async ({ context }) => { const controllerPage = await context.newPage(); - await controllerPage.goto('http://localhost:3001'); + await controllerPage.setViewportSize({ width: 390, height: 844 }); + await controllerPage.goto('http://localhost:3000/controller'); await controllerPage.waitForSelector('.conn-bar.connected'); // Apri config diff --git a/tests/integration/server.test.js b/tests/integration/server.test.js new file mode 100644 index 0000000..8436aac --- /dev/null +++ b/tests/integration/server.test.js @@ -0,0 +1,66 @@ +import { describe, it, expect, beforeAll, afterAll } from 'vitest' +import { mkdtempSync, writeFileSync, rmSync } from 'fs' +import { tmpdir } from 'os' +import { join } from 'path' +import { createApp } from '../../server.js' + +// Usiamo una dist temporanea con file marcatori, così il test non dipende da +// una build reale ed è deterministico. +let server +let baseUrl +let distDir + +beforeAll(async () => { + distDir = mkdtempSync(join(tmpdir(), 'segnapunti-dist-')) + writeFileSync(join(distDir, 'index.html'), 'DISPLAY_MARKER') + writeFileSync(join(distDir, 'controller.html'), 'CONTROLLER_MARKER') + writeFileSync(join(distDir, 'app.js'), 'ASSET_MARKER') + + const app = createApp(distDir) + await new Promise((resolve) => { + server = app.listen(0, () => { + baseUrl = `http://127.0.0.1:${server.address().port}` + resolve() + }) + }) +}) + +afterAll(async () => { + await new Promise((resolve) => server.close(resolve)) + rmSync(distDir, { recursive: true, force: true }) +}) + +async function get(path) { + const res = await fetch(baseUrl + path) + return { status: res.status, body: await res.text() } +} + +describe('Routing server (server.js)', () => { + it('GET / serve la pagina display', async () => { + const { status, body } = await get('/') + expect(status).toBe(200) + expect(body).toBe('DISPLAY_MARKER') + }) + + it('GET /display serve la pagina display', async () => { + expect((await get('/display')).body).toBe('DISPLAY_MARKER') + }) + + it('GET /display/qualsiasi serve comunque la pagina display (SPA)', async () => { + expect((await get('/display/foo/bar')).body).toBe('DISPLAY_MARKER') + }) + + it('GET /controller serve la pagina controller', async () => { + expect((await get('/controller')).body).toBe('CONTROLLER_MARKER') + }) + + it('GET /controller/qualsiasi serve comunque la pagina controller', async () => { + expect((await get('/controller/foo')).body).toBe('CONTROLLER_MARKER') + }) + + it('serve gli asset statici dalla dist', async () => { + const { status, body } = await get('/app.js') + expect(status).toBe(200) + expect(body).toBe('ASSET_MARKER') + }) +}) diff --git a/tests/integration/websocket.test.js b/tests/integration/websocket.test.js index 2534bd5..21337a6 100644 --- a/tests/integration/websocket.test.js +++ b/tests/integration/websocket.test.js @@ -1,7 +1,11 @@ import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest' import { setupWebSocketHandler } from '../../src/websocket-handler.js' +import { punteggio } from '../../src/gameState.js' import { EventEmitter } from 'events' +// Il punteggio non è memorizzato nello stato: si ricava dalla striscia. +const puntHome = (state) => punteggio(state.sp.striscia).home + // Mock parziale di una WebSocket e del Server class MockWebSocket extends EventEmitter { constructor() { @@ -103,7 +107,7 @@ describe('WebSocket Integration (websocket-handler.js)', () => { expect(controller.send).toHaveBeenCalled() const sentMsg = lastSent(controller) expect(sentMsg.type).toBe('state') - expect(sentMsg.state.sp.punt.home).toBe(1) + expect(puntHome(sentMsg.state)).toBe(1) }) it('dovrebbe impedire al display di inviare azioni', () => { @@ -182,8 +186,8 @@ describe('WebSocket Integration (websocket-handler.js)', () => { const msg1 = lastSent(display1) const msg2 = lastSent(display2) expect(msg1.type).toBe('state') - expect(msg1.state.sp.punt.home).toBe(1) - expect(msg2.state.sp.punt.home).toBe(1) + expect(puntHome(msg1.state)).toBe(1) + expect(puntHome(msg2.state)).toBe(1) }) it('non dovrebbe inviare a client con readyState != OPEN', () => { @@ -302,7 +306,7 @@ describe('WebSocket Integration (websocket-handler.js)', () => { const msg = lastSent(controller) expect(msg.type).toBe('state') - expect(msg.state.sp.punt.home).toBe(1) + expect(puntHome(msg.state)).toBe(1) }) }) @@ -374,15 +378,15 @@ describe('WebSocket Integration (websocket-handler.js)', () => { describe('API pubblica', () => { it('getState dovrebbe restituire lo stato corrente', () => { const state = handler.getState() - expect(state.sp.punt.home).toBe(0) - expect(state.sp.punt.guest).toBe(0) + expect(puntHome(state)).toBe(0) + expect(punteggio(state.sp.striscia).guest).toBe(0) }) it('setState dovrebbe sovrascrivere lo stato', () => { const newState = handler.getState() - newState.sp.punt.home = 99 + newState.sp.striscia.at(-1).ris = 'hh' handler.setState(newState) - expect(handler.getState().sp.punt.home).toBe(99) + expect(puntHome(handler.getState())).toBe(2) }) it('broadcastState dovrebbe inviare a tutti i client', () => { diff --git a/tests/stress/websocket-load.test.js b/tests/stress/websocket-load.test.js index 9d89c3b..837dfcf 100644 --- a/tests/stress/websocket-load.test.js +++ b/tests/stress/websocket-load.test.js @@ -1,7 +1,11 @@ import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest' import { setupWebSocketHandler } from '../../src/websocket-handler.js' +import { punteggio } from '../../src/gameState.js' import { EventEmitter } from 'events' +// Il punteggio si ricava dalla striscia, non è memorizzato nello stato. +const punt = (state) => punteggio(state.sp.striscia) + class MockWebSocket extends EventEmitter { constructor() { super() @@ -57,7 +61,7 @@ describe('Stress Test WebSocket', () => { expect(display.send).toHaveBeenCalled() const msg = JSON.parse(display.send.mock.calls[display.send.mock.calls.length - 1][0]) expect(msg.type).toBe('state') - expect(msg.state.sp.punt.home).toBe(1) + expect(punt(msg.state).home).toBe(1) } }) @@ -81,11 +85,11 @@ describe('Stress Test WebSocket', () => { // Lo stato finale dipende da checkVittoria che blocca a 25+2 // Home arriva a 25-0 → vittoria → blocca. Quindi punti home = 25 const state = handler.getState() - expect(state.sp.punt.home).toBe(25) + expect(punt(state).home).toBe(25) // Guest: non può segnare dopo vittoria? No, checkVittoria blocca solo il team che ha vinto? // Controlliamo: checkVittoria controlla ENTRAMBI i team. // A 25-0 → vittoria=true → incPunt per guest è anche bloccato - expect(state.sp.punt.guest).toBe(0) + expect(punt(state).guest).toBe(0) }) it('dovrebbe garantire che tutti i display ricevano ogni update sotto carico', () => { @@ -112,7 +116,7 @@ describe('Stress Test WebSocket', () => { // Verifica stato finale su tutti i display for (const display of displays) { const lastMsg = JSON.parse(display.send.mock.calls[4][0]) - expect(lastMsg.state.sp.punt.home).toBe(5) + expect(punt(lastMsg.state).home).toBe(5) } }) }) diff --git a/tests/unit/gameState.test.js b/tests/unit/gameState.test.js index 0b671d0..862ba8f 100644 --- a/tests/unit/gameState.test.js +++ b/tests/unit/gameState.test.js @@ -1,5 +1,39 @@ import { describe, it, expect, beforeEach } from 'vitest' -import { createInitialState, applyAction, checkVittoria, checkVittoriaPartita } from '../../src/gameState.js' +import { + createInitialState, applyAction, checkVittoria, checkVittoriaPartita, + punteggio, setVinti, servizio, +} from '../../src/gameState.js' + +// ============================================= +// HELPER +// Lo stato non memorizza più punteggio/set/servizio direttamente: +// si ricavano dalla striscia tramite le funzioni pure esportate. +// ============================================= + +// Punteggio del set in corso +const puntDi = (s) => punteggio(s.sp.striscia) +// Set vinti nel match +const setDi = (s) => setVinti(s.sp.striscia) +// true se serve Home +const servDi = (s) => servizio(s.sp.striscia) + +// Imposta chi serve a inizio set (set in corso a 0-0) +function setServizioIniziale(state, team) { + state.sp.striscia.at(-1).serv = team === 'home' ? 'h' : 'g' +} + +// Imposta il punteggio del set in corso costruendo una ris coerente +function setPunteggio(state, home, guest) { + state.sp.striscia.at(-1).ris = 'h'.repeat(home) + 'g'.repeat(guest) +} + +// Aggiunge set già conclusi (vinti) PRIMA del set in corso +function setSetVinti(state, home, guest) { + const conclusi = [] + for (let i = 0; i < home; i++) conclusi.push({ serv: 'h', ris: '', vinc: 'h' }) + for (let i = 0; i < guest; i++) conclusi.push({ serv: 'g', ris: '', vinc: 'g' }) + state.sp.striscia = [...conclusi, state.sp.striscia.at(-1)] +} describe('Game Logic (gameState.js)', () => { let state @@ -13,17 +47,17 @@ describe('Game Logic (gameState.js)', () => { // ============================================= describe('Stato iniziale', () => { it('dovrebbe iniziare con 0-0', () => { - expect(state.sp.punt.home).toBe(0) - expect(state.sp.punt.guest).toBe(0) + expect(puntDi(state).home).toBe(0) + expect(puntDi(state).guest).toBe(0) }) it('dovrebbe avere i set a 0', () => { - expect(state.sp.set.home).toBe(0) - expect(state.sp.set.guest).toBe(0) + expect(setDi(state).home).toBe(0) + expect(setDi(state).guest).toBe(0) }) it('dovrebbe avere servizio Home', () => { - expect(state.sp.servHome).toBe(true) + expect(servDi(state)).toBe(true) }) it('dovrebbe avere formazione di default [1-6]', () => { @@ -69,43 +103,43 @@ describe('Game Logic (gameState.js)', () => { describe('incPunt', () => { it('dovrebbe incrementare i punti Home', () => { const newState = applyAction(state, { type: 'incPunt', team: 'home' }) - expect(newState.sp.punt.home).toBe(1) - expect(newState.sp.punt.guest).toBe(0) + expect(puntDi(newState).home).toBe(1) + expect(puntDi(newState).guest).toBe(0) }) it('dovrebbe incrementare i punti Guest', () => { const newState = applyAction(state, { type: 'incPunt', team: 'guest' }) - expect(newState.sp.punt.guest).toBe(1) - expect(newState.sp.punt.home).toBe(0) + expect(puntDi(newState).guest).toBe(1) + expect(puntDi(newState).home).toBe(0) }) it('dovrebbe gestire il cambio palla (Guest segna, batteva Home)', () => { - state.sp.servHome = true + setServizioIniziale(state, 'home') const s1 = applyAction(state, { type: 'incPunt', team: 'guest' }) - expect(s1.sp.servHome).toBe(false) + expect(servDi(s1)).toBe(false) }) it('dovrebbe gestire il cambio palla (Home segna, batteva Guest)', () => { - state.sp.servHome = false + setServizioIniziale(state, 'guest') const s1 = applyAction(state, { type: 'incPunt', team: 'home' }) - expect(s1.sp.servHome).toBe(true) + expect(servDi(s1)).toBe(true) }) it('non dovrebbe cambiare palla se segna chi batte', () => { - state.sp.servHome = true + setServizioIniziale(state, 'home') const s1 = applyAction(state, { type: 'incPunt', team: 'home' }) - expect(s1.sp.servHome).toBe(true) + expect(servDi(s1)).toBe(true) }) it('dovrebbe ruotare la formazione al cambio palla', () => { - state.sp.servHome = true + setServizioIniziale(state, 'home') state.sp.form.guest = ["1", "2", "3", "4", "5", "6"] const newState = applyAction(state, { type: 'incPunt', team: 'guest' }) expect(newState.sp.form.guest).toEqual(["2", "3", "4", "5", "6", "1"]) }) it('non dovrebbe ruotare la formazione se non c\'è cambio palla', () => { - state.sp.servHome = true + setServizioIniziale(state, 'home') state.sp.form.home = ["1", "2", "3", "4", "5", "6"] const newState = applyAction(state, { type: 'incPunt', team: 'home' }) expect(newState.sp.form.home).toEqual(["1", "2", "3", "4", "5", "6"]) @@ -129,10 +163,9 @@ describe('Game Logic (gameState.js)', () => { }) it('non dovrebbe incrementare i punti dopo vittoria', () => { - state.sp.punt.home = 25 - state.sp.punt.guest = 23 + setPunteggio(state, 25, 23) const s = applyAction(state, { type: 'incPunt', team: 'home' }) - expect(s.sp.punt.home).toBe(25) + expect(puntDi(s).home).toBe(25) }) }) @@ -143,33 +176,33 @@ describe('Game Logic (gameState.js)', () => { it('dovrebbe annullare l\'ultimo punto Home', () => { const s1 = applyAction(state, { type: 'incPunt', team: 'home' }) const s2 = applyAction(s1, { type: 'decPunt' }) - expect(s2.sp.punt.home).toBe(0) - expect(s2.sp.punt.guest).toBe(0) + expect(puntDi(s2).home).toBe(0) + expect(puntDi(s2).guest).toBe(0) }) it('dovrebbe annullare l\'ultimo punto Guest', () => { const s1 = applyAction(state, { type: 'incPunt', team: 'guest' }) const s2 = applyAction(s1, { type: 'decPunt' }) - expect(s2.sp.punt.home).toBe(0) - expect(s2.sp.punt.guest).toBe(0) + expect(puntDi(s2).home).toBe(0) + expect(puntDi(s2).guest).toBe(0) }) it('non dovrebbe fare nulla sullo stato iniziale', () => { const s = applyAction(state, { type: 'decPunt' }) - expect(s.sp.punt.home).toBe(0) - expect(s.sp.punt.guest).toBe(0) + expect(puntDi(s).home).toBe(0) + expect(puntDi(s).guest).toBe(0) }) it('dovrebbe ripristinare il servizio dopo undo con cambio palla', () => { - state.sp.servHome = true + setServizioIniziale(state, 'home') const s1 = applyAction(state, { type: 'incPunt', team: 'guest' }) - expect(s1.sp.servHome).toBe(false) + expect(servDi(s1)).toBe(false) const s2 = applyAction(s1, { type: 'decPunt' }) - expect(s2.sp.servHome).toBe(true) + expect(servDi(s2)).toBe(true) }) it('dovrebbe invertire la rotazione dopo undo con cambio palla', () => { - state.sp.servHome = true + setServizioIniziale(state, 'home') state.sp.form.guest = ["1", "2", "3", "4", "5", "6"] const s1 = applyAction(state, { type: 'incPunt', team: 'guest' }) expect(s1.sp.form.guest).toEqual(["2", "3", "4", "5", "6", "1"]) @@ -188,14 +221,66 @@ describe('Game Logic (gameState.js)', () => { s = applyAction(s, { type: 'incPunt', team: 'home' }) s = applyAction(s, { type: 'incPunt', team: 'guest' }) s = applyAction(s, { type: 'incPunt', team: 'home' }) - expect(s.sp.punt.home).toBe(2) - expect(s.sp.punt.guest).toBe(1) + expect(puntDi(s).home).toBe(2) + expect(puntDi(s).guest).toBe(1) s = applyAction(s, { type: 'decPunt' }) - expect(s.sp.punt.home).toBe(1) + expect(puntDi(s).home).toBe(1) s = applyAction(s, { type: 'decPunt' }) - expect(s.sp.punt.guest).toBe(0) + expect(puntDi(s).guest).toBe(0) s = applyAction(s, { type: 'decPunt' }) - expect(s.sp.punt.home).toBe(0) + expect(puntDi(s).home).toBe(0) + }) + }) + + // ============================================= + // FORMAZIONE DI PARTENZA (formInizio) + // ============================================= + describe('formInizio', () => { + it('dovrebbe salvare la formazione corrente al primo punto del set', () => { + const s = applyAction(state, { type: 'incPunt', team: 'home' }) + expect(s.sp.striscia.at(-1).formInizio).toEqual({ + home: ["1", "2", "3", "4", "5", "6"], + guest: ["1", "2", "3", "4", "5", "6"], + }) + }) + + it('formInizio è uno snapshot: la rotazione successiva non lo modifica', () => { + setServizioIniziale(state, 'home') + // 1° punto Home: nessun cambio palla, salva formInizio + let s = applyAction(state, { type: 'incPunt', team: 'home' }) + // 2° punto Guest: cambio palla → ruota la formazione guest + s = applyAction(s, { type: 'incPunt', team: 'guest' }) + expect(s.sp.form.guest).toEqual(["2", "3", "4", "5", "6", "1"]) + // lo snapshot resta quello iniziale + expect(s.sp.striscia.at(-1).formInizio.guest).toEqual(["1", "2", "3", "4", "5", "6"]) + }) + + it('decPunt che riporta il set a 0-0 cancella formInizio', () => { + const s1 = applyAction(state, { type: 'incPunt', team: 'home' }) + expect(s1.sp.striscia.at(-1).formInizio).toBeDefined() + const s2 = applyAction(s1, { type: 'decPunt' }) + expect(s2.sp.striscia.at(-1).formInizio).toBeUndefined() + }) + + it('decPunt con punti ancora presenti NON cancella formInizio', () => { + let s = applyAction(state, { type: 'incPunt', team: 'home' }) + s = applyAction(s, { type: 'incPunt', team: 'home' }) + s = applyAction(s, { type: 'decPunt' }) + expect(s.sp.striscia.at(-1).ris).toBe('h') + expect(s.sp.striscia.at(-1).formInizio).toBeDefined() + }) + + it('ogni set mantiene la propria formInizio', () => { + const custom = ['7', '8', '9', '10', '11', '12'] + let s = applyAction(state, { type: 'setFormazione', team: 'home', form: custom }) + // primo punto del set 1: salva formInizio custom + s = applyAction(s, { type: 'incPunt', team: 'home' }) + // chiude il set 1 e ne apre uno nuovo (formazioni resettate a default) + s = applyAction(s, { type: 'nuovoSet', team: 'home' }) + // primo punto del set 2: salva formInizio default + s = applyAction(s, { type: 'incPunt', team: 'home' }) + expect(s.sp.striscia[0].formInizio.home).toEqual(custom) + expect(s.sp.striscia.at(-1).formInizio.home).toEqual(['1', '2', '3', '4', '5', '6']) }) }) @@ -205,24 +290,27 @@ describe('Game Logic (gameState.js)', () => { describe('incSet', () => { it('dovrebbe incrementare il set Home', () => { const s = applyAction(state, { type: 'incSet', team: 'home' }) - expect(s.sp.set.home).toBe(1) + expect(setDi(s).home).toBe(1) }) it('dovrebbe incrementare il set Guest', () => { const s = applyAction(state, { type: 'incSet', team: 'guest' }) - expect(s.sp.set.guest).toBe(1) + expect(setDi(s).guest).toBe(1) }) it('dovrebbe fare wrap da 2 a 0', () => { - state.sp.set.home = 2 - const s = applyAction(state, { type: 'incSet', team: 'home' }) - expect(s.sp.set.home).toBe(0) + let s = applyAction(state, { type: 'incSet', team: 'home' }) + s = applyAction(s, { type: 'incSet', team: 'home' }) + expect(setDi(s).home).toBe(2) + s = applyAction(s, { type: 'incSet', team: 'home' }) + expect(setDi(s).home).toBe(0) }) it('dovrebbe incrementare da 1 a 2', () => { - state.sp.set.home = 1 - const s = applyAction(state, { type: 'incSet', team: 'home' }) - expect(s.sp.set.home).toBe(2) + let s = applyAction(state, { type: 'incSet', team: 'home' }) + expect(setDi(s).home).toBe(1) + s = applyAction(s, { type: 'incSet', team: 'home' }) + expect(setDi(s).home).toBe(2) }) }) @@ -231,18 +319,16 @@ describe('Game Logic (gameState.js)', () => { // ============================================= describe('nuovoSet', () => { it('dovrebbe incrementare il set della squadra vincente', () => { - state.sp.punt.home = 25 const s = applyAction(state, { type: 'nuovoSet', team: 'home' }) - expect(s.sp.set.home).toBe(1) - expect(s.sp.set.guest).toBe(0) + expect(setDi(s).home).toBe(1) + expect(setDi(s).guest).toBe(0) }) - it('dovrebbe azzerare i punti', () => { - state.sp.punt.home = 25 - state.sp.punt.guest = 10 + it('dovrebbe azzerare i punti nel nuovo set', () => { + setPunteggio(state, 25, 10) const s = applyAction(state, { type: 'nuovoSet', team: 'home' }) - expect(s.sp.punt.home).toBe(0) - expect(s.sp.punt.guest).toBe(0) + expect(puntDi(s).home).toBe(0) + expect(puntDi(s).guest).toBe(0) }) it('dovrebbe aggiungere un nuovo set vuoto alla striscia', () => { @@ -268,37 +354,35 @@ describe('Game Logic (gameState.js)', () => { it('dovrebbe ignorare team non valido', () => { const s = applyAction(state, { type: 'nuovoSet', team: 'invalid' }) - expect(s.sp.set.home).toBe(0) - expect(s.sp.set.guest).toBe(0) + expect(setDi(s).home).toBe(0) + expect(setDi(s).guest).toBe(0) }) - it('in 2/3 dovrebbe registrare il set vincente senza resettare il punteggio', () => { + it('in 2/3 alla palla match registra il set vincente senza aprirne uno nuovo', () => { state.modalitaPartita = '2/3' - state.sp.set.home = 1 - state.sp.punt.home = 25 - state.sp.punt.guest = 18 + setSetVinti(state, 1, 0) + setPunteggio(state, 25, 18) const s = applyAction(state, { type: 'nuovoSet', team: 'home' }) - expect(s.sp.set.home).toBe(2) - expect(s.sp.punt.home).toBe(25) - expect(s.sp.punt.guest).toBe(18) + expect(setDi(s).home).toBe(2) + expect(puntDi(s).home).toBe(25) + expect(puntDi(s).guest).toBe(18) }) - it('in 3/5 dovrebbe registrare il set vincente senza resettare il punteggio', () => { + it('in 3/5 alla palla match registra il set vincente senza aprirne uno nuovo', () => { state.modalitaPartita = '3/5' - state.sp.set.home = 2 - state.sp.punt.home = 25 - state.sp.punt.guest = 20 + setSetVinti(state, 2, 0) + setPunteggio(state, 25, 20) const s = applyAction(state, { type: 'nuovoSet', team: 'home' }) - expect(s.sp.set.home).toBe(3) - expect(s.sp.punt.home).toBe(25) - expect(s.sp.punt.guest).toBe(20) + expect(setDi(s).home).toBe(3) + expect(puntDi(s).home).toBe(25) + expect(puntDi(s).guest).toBe(20) }) it('dovrebbe ignorare nuovoSet se la partita è già finita', () => { state.modalitaPartita = '2/3' - state.sp.set.home = 2 + setSetVinti(state, 2, 0) const s = applyAction(state, { type: 'nuovoSet', team: 'home' }) - expect(s.sp.set.home).toBe(2) + expect(setDi(s).home).toBe(2) }) }) @@ -308,29 +392,33 @@ describe('Game Logic (gameState.js)', () => { describe('checkVittoriaPartita', () => { it('in 2/3 restituisce false se nessuno ha 2 set', () => { state.modalitaPartita = '2/3' - state.sp.set.home = 1 - state.sp.set.guest = 0 + setSetVinti(state, 1, 0) expect(checkVittoriaPartita(state)).toBe(false) }) it('in 2/3 restituisce true se home ha 2 set', () => { state.modalitaPartita = '2/3' - state.sp.set.home = 2 + setSetVinti(state, 2, 0) expect(checkVittoriaPartita(state)).toBe(true) }) it('in 3/5 restituisce false se nessuno ha 3 set', () => { state.modalitaPartita = '3/5' - state.sp.set.home = 2 - state.sp.set.guest = 2 + setSetVinti(state, 2, 2) expect(checkVittoriaPartita(state)).toBe(false) }) it('in 3/5 restituisce true se guest ha 3 set', () => { state.modalitaPartita = '3/5' - state.sp.set.guest = 3 + setSetVinti(state, 0, 3) expect(checkVittoriaPartita(state)).toBe(true) }) + + it('in amichevole restituisce sempre false', () => { + state.modalitaPartita = 'amichevole' + setSetVinti(state, 5, 0) + expect(checkVittoriaPartita(state)).toBe(false) + }) }) // ============================================= @@ -338,27 +426,29 @@ describe('Game Logic (gameState.js)', () => { // ============================================= describe('cambiaPalla', () => { it('dovrebbe invertire il servizio a 0-0', () => { - expect(state.sp.servHome).toBe(true) + expect(servDi(state)).toBe(true) const s = applyAction(state, { type: 'cambiaPalla' }) - expect(s.sp.servHome).toBe(false) + expect(servDi(s)).toBe(false) }) it('dovrebbe tornare a Home con doppio toggle', () => { let s = applyAction(state, { type: 'cambiaPalla' }) s = applyAction(s, { type: 'cambiaPalla' }) - expect(s.sp.servHome).toBe(true) + expect(servDi(s)).toBe(true) }) it('non dovrebbe cambiare palla se il punteggio non è 0-0', () => { - state.sp.punt.home = 1 + setPunteggio(state, 1, 0) + const prima = servDi(state) const s = applyAction(state, { type: 'cambiaPalla' }) - expect(s.sp.servHome).toBe(true) + expect(servDi(s)).toBe(prima) }) it('non dovrebbe cambiare palla se Guest ha punti', () => { - state.sp.punt.guest = 3 + setPunteggio(state, 0, 3) + const prima = servDi(state) const s = applyAction(state, { type: 'cambiaPalla' }) - expect(s.sp.servHome).toBe(true) + expect(servDi(s)).toBe(prima) }) }) @@ -569,44 +659,37 @@ describe('Game Logic (gameState.js)', () => { // ============================================= describe('checkVittoria', () => { it('non dovrebbe dare vittoria a 24-24', () => { - state.sp.punt.home = 24 - state.sp.punt.guest = 24 + setPunteggio(state, 24, 24) expect(checkVittoria(state)).toBe(false) }) it('dovrebbe dare vittoria a 25-23', () => { - state.sp.punt.home = 25 - state.sp.punt.guest = 23 + setPunteggio(state, 25, 23) expect(checkVittoria(state)).toBe(true) }) it('non dovrebbe dare vittoria a 25-24 (serve 2 punti di scarto)', () => { - state.sp.punt.home = 25 - state.sp.punt.guest = 24 + setPunteggio(state, 25, 24) expect(checkVittoria(state)).toBe(false) }) it('dovrebbe dare vittoria a 26-24', () => { - state.sp.punt.home = 26 - state.sp.punt.guest = 24 + setPunteggio(state, 26, 24) expect(checkVittoria(state)).toBe(true) }) it('dovrebbe dare vittoria Guest a 25-20', () => { - state.sp.punt.home = 20 - state.sp.punt.guest = 25 + setPunteggio(state, 20, 25) expect(checkVittoria(state)).toBe(true) }) it('dovrebbe dare vittoria ai vantaggi (30-28)', () => { - state.sp.punt.home = 30 - state.sp.punt.guest = 28 + setPunteggio(state, 30, 28) expect(checkVittoria(state)).toBe(true) }) it('non dovrebbe dare vittoria ai vantaggi senza scarto (28-27)', () => { - state.sp.punt.home = 28 - state.sp.punt.guest = 27 + setPunteggio(state, 28, 27) expect(checkVittoria(state)).toBe(false) }) }) @@ -617,82 +700,64 @@ describe('Game Logic (gameState.js)', () => { describe('Set decisivo', () => { it('modalità 3/5: set decisivo dopo 4 set totali → vittoria a 15', () => { state.modalitaPartita = "3/5" - state.sp.set.home = 2 - state.sp.set.guest = 2 - state.sp.punt.home = 15 - state.sp.punt.guest = 10 + setSetVinti(state, 2, 2) + setPunteggio(state, 15, 10) expect(checkVittoria(state)).toBe(true) }) it('modalità 3/5: non vittoria a 14-10 nel set decisivo', () => { state.modalitaPartita = "3/5" - state.sp.set.home = 2 - state.sp.set.guest = 2 - state.sp.punt.home = 14 - state.sp.punt.guest = 10 + setSetVinti(state, 2, 2) + setPunteggio(state, 14, 10) expect(checkVittoria(state)).toBe(false) }) it('modalità 3/5: vittoria a 15-13 nel set decisivo', () => { state.modalitaPartita = "3/5" - state.sp.set.home = 2 - state.sp.set.guest = 2 - state.sp.punt.home = 15 - state.sp.punt.guest = 13 + setSetVinti(state, 2, 2) + setPunteggio(state, 15, 13) expect(checkVittoria(state)).toBe(true) }) it('modalità 3/5: non vittoria a 15-14 nel set decisivo (serve scarto)', () => { state.modalitaPartita = "3/5" - state.sp.set.home = 2 - state.sp.set.guest = 2 - state.sp.punt.home = 15 - state.sp.punt.guest = 14 + setSetVinti(state, 2, 2) + setPunteggio(state, 15, 14) expect(checkVittoria(state)).toBe(false) }) it('modalità 3/5: vittoria a 16-14 nel set decisivo', () => { state.modalitaPartita = "3/5" - state.sp.set.home = 2 - state.sp.set.guest = 2 - state.sp.punt.home = 16 - state.sp.punt.guest = 14 + setSetVinti(state, 2, 2) + setPunteggio(state, 16, 14) expect(checkVittoria(state)).toBe(true) }) it('modalità 2/3: set decisivo dopo 2 set totali → vittoria a 15', () => { state.modalitaPartita = "2/3" - state.sp.set.home = 1 - state.sp.set.guest = 1 - state.sp.punt.home = 15 - state.sp.punt.guest = 10 + setSetVinti(state, 1, 1) + setPunteggio(state, 15, 10) expect(checkVittoria(state)).toBe(true) }) - it('modalità 2/3: non vittoria a 24-20 nel set decisivo (soglia 15)', () => { + it('modalità 2/3: non vittoria a 14-10 nel set decisivo (soglia 15)', () => { state.modalitaPartita = "2/3" - state.sp.set.home = 1 - state.sp.set.guest = 1 - state.sp.punt.home = 14 - state.sp.punt.guest = 10 + setSetVinti(state, 1, 1) + setPunteggio(state, 14, 10) expect(checkVittoria(state)).toBe(false) }) it('modalità 2/3: set non decisivo (1-0) → soglia 25', () => { state.modalitaPartita = "2/3" - state.sp.set.home = 1 - state.sp.set.guest = 0 - state.sp.punt.home = 15 - state.sp.punt.guest = 10 + setSetVinti(state, 1, 0) + setPunteggio(state, 15, 10) expect(checkVittoria(state)).toBe(false) }) it('modalità 3/5: set non decisivo (2-1) → soglia 25', () => { state.modalitaPartita = "3/5" - state.sp.set.home = 2 - state.sp.set.guest = 1 - state.sp.punt.home = 15 - state.sp.punt.guest = 10 + setSetVinti(state, 2, 1) + setPunteggio(state, 15, 10) expect(checkVittoria(state)).toBe(false) }) }) @@ -702,15 +767,13 @@ describe('Game Logic (gameState.js)', () => { // ============================================= describe('Reset', () => { it('dovrebbe resettare punti e set a zero', () => { - state.sp.punt.home = 10 - state.sp.punt.guest = 8 - state.sp.set.home = 1 - state.sp.set.guest = 1 + setSetVinti(state, 1, 1) + setPunteggio(state, 10, 8) const s = applyAction(state, { type: 'resetta' }) - expect(s.sp.punt.home).toBe(0) - expect(s.sp.punt.guest).toBe(0) - expect(s.sp.set.home).toBe(0) - expect(s.sp.set.guest).toBe(0) + expect(puntDi(s).home).toBe(0) + expect(puntDi(s).guest).toBe(0) + expect(setDi(s).home).toBe(0) + expect(setDi(s).guest).toBe(0) }) it('dovrebbe resettare formazioni a default', () => { @@ -721,7 +784,7 @@ describe('Game Logic (gameState.js)', () => { }) it('dovrebbe resettare la striscia a un set vuoto', () => { - state.sp.striscia = [{ serv: 'h', ris: 'hgh' }, { serv: 'h', ris: 'g' }] + state.sp.striscia = [{ serv: 'h', ris: 'hgh', vinc: 'h' }, { serv: 'h', ris: 'g', vinc: null }] const s = applyAction(state, { type: 'resetta' }) expect(s.sp.striscia).toHaveLength(1) expect(s.sp.striscia[0].ris).toBe('') @@ -748,8 +811,8 @@ describe('Game Logic (gameState.js)', () => { describe('Azione sconosciuta', () => { it('dovrebbe restituire lo stato invariato con azione non riconosciuta', () => { const s = applyAction(state, { type: 'azioneInesistente' }) - expect(s.sp.punt.home).toBe(0) - expect(s.sp.punt.guest).toBe(0) + expect(puntDi(s).home).toBe(0) + expect(puntDi(s).guest).toBe(0) }) }) }) diff --git a/tests/unit/persist.test.js b/tests/unit/persist.test.js new file mode 100644 index 0000000..28ff180 --- /dev/null +++ b/tests/unit/persist.test.js @@ -0,0 +1,71 @@ +import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest' + +// Mock dell'I/O su disco: i test non toccano il filesystem reale né dipendono +// dal path relativo a src/. +vi.mock('fs', () => ({ + readFileSync: vi.fn(), + writeFileSync: vi.fn(), + mkdirSync: vi.fn(), + existsSync: vi.fn(), +})) + +import * as fs from 'fs' +import { loadState, saveState } from '../../src/persist.js' +import { createInitialState } from '../../src/gameState.js' + +describe('Persistenza stato (persist.js)', () => { + beforeEach(() => { + vi.clearAllMocks() + }) + + afterEach(() => { + vi.restoreAllMocks() + }) + + describe('saveState', () => { + it('crea la directory e scrive lo stato serializzato', () => { + const state = createInitialState() + saveState(state) + + expect(fs.mkdirSync).toHaveBeenCalledWith(expect.any(String), { recursive: true }) + expect(fs.writeFileSync).toHaveBeenCalled() + + const [, contenuto, encoding] = fs.writeFileSync.mock.calls[0] + expect(JSON.parse(contenuto)).toEqual(state) + expect(encoding).toBe('utf8') + }) + + it('non lancia eccezioni se la scrittura fallisce', () => { + fs.mkdirSync.mockImplementation(() => { throw new Error('EACCES') }) + const errSpy = vi.spyOn(console, 'error').mockImplementation(() => {}) + + expect(() => saveState(createInitialState())).not.toThrow() + expect(errSpy).toHaveBeenCalled() + }) + }) + + describe('loadState', () => { + it('legge e fa il parse di un file valido', () => { + const salvato = createInitialState() + salvato.sp.nomi.home = 'Squadra X' + fs.existsSync.mockReturnValue(true) + fs.readFileSync.mockReturnValue(JSON.stringify(salvato)) + + expect(loadState()).toEqual(salvato) + }) + + it('ritorna lo stato iniziale se il file non esiste', () => { + fs.existsSync.mockReturnValue(false) + expect(loadState()).toEqual(createInitialState()) + }) + + it('ritorna lo stato iniziale se il JSON è corrotto', () => { + fs.existsSync.mockReturnValue(true) + fs.readFileSync.mockReturnValue('{ questo non è json') + const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {}) + + expect(loadState()).toEqual(createInitialState()) + expect(warnSpy).toHaveBeenCalled() + }) + }) +}) diff --git a/tests/unit/referto.test.js b/tests/unit/referto.test.js new file mode 100644 index 0000000..0ea4fb9 --- /dev/null +++ b/tests/unit/referto.test.js @@ -0,0 +1,110 @@ +import { describe, it, expect } from 'vitest' +import { buildRefertoHtml } from '../../src/referto.js' +import { createInitialState } from '../../src/gameState.js' + +// Data fissa per asserzioni deterministiche +const NOW = new Date('2026-03-14T20:30:00') + +// Costruisce uno stato con una striscia di set arbitraria +function statoConSet(striscia, extra = {}) { + const state = createInitialState() + state.sp.striscia = striscia + state.sp.nomi = { home: 'Antoniana', guest: 'Rivali' } + return { ...state, ...extra } +} + +describe('buildRefertoHtml (referto.js)', () => { + it('esclude i set _phantom dal referto', () => { + const striscia = [ + { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(20), vinc: 'h' }, + { serv: 'g', ris: '', vinc: 'g', _phantom: true }, + { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(18), vinc: 'h' }, + ] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + // due set reali → "Set 1" e "Set 2", mai "Set 3" + expect(html).toContain('Set 1') + expect(html).toContain('Set 2') + expect(html).not.toContain('Set 3') + }) + + it('calcola il punteggio finale di ogni set dalla ris', () => { + const striscia = [ + { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(20), vinc: 'h' }, + { serv: 'h', ris: '', vinc: null }, + ] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + // header del set: "Antoniana 25 · 20 Rivali" + expect(html).toContain('25') + expect(html).toContain('20') + }) + + it('conta i set vinti usando vinc', () => { + const striscia = [ + { serv: 'h', ris: '', vinc: 'h' }, + { serv: 'g', ris: '', vinc: 'g' }, + { serv: 'h', ris: '', vinc: 'h' }, + { serv: 'h', ris: '', vinc: null }, + ] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + // risultato 2 – 1 + expect(html).toContain('2 – 1') + }) + + it('ricava il vincitore dal conteggio punti se vinc è nullo', () => { + const striscia = [ + { serv: 'h', ris: 'h'.repeat(25) + 'g'.repeat(23), vinc: null }, + { serv: 'h', ris: '', vinc: null }, + ] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + // il primo set, pur con vinc null, conta come vinto da home → 1 – 0 + expect(html).toContain('1 – 0') + }) + + it('include la progressione punto-punto con classi per squadra', () => { + const striscia = [ + { serv: 'h', ris: 'hhg', vinc: null }, + ] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + expect(html).toContain('punto-h') + expect(html).toContain('punto-g') + expect(html).toContain('1-0') + expect(html).toContain('2-0') + expect(html).toContain('2-1') + }) + + it('rende la formazione di partenza quando presente', () => { + const striscia = [ + { + serv: 'h', ris: 'h', vinc: null, + formInizio: { home: ['4', '8', '15'], guest: ['16', '23', '42'] }, + }, + ] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + expect(html).toContain('Formazione di partenza') + expect(html).toContain('>4<') + expect(html).toContain('>42<') + }) + + it('mostra "non disponibile" se manca formInizio', () => { + const striscia = [{ serv: 'h', ris: 'h', vinc: null }] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + expect(html).toContain('non disponibile') + }) + + it('mostra "Nessun punto registrato" per un set senza punti', () => { + const striscia = [{ serv: 'h', ris: '', vinc: null }] + const html = buildRefertoHtml(statoConSet(striscia), NOW) + expect(html).toContain('Nessun punto registrato') + }) + + it('header contiene nomi squadre, modalità e data iniettata', () => { + const striscia = [{ serv: 'h', ris: '', vinc: null }] + const state = statoConSet(striscia) + state.modalitaPartita = '2/3' + const html = buildRefertoHtml(state, NOW) + expect(html).toContain('Antoniana') + expect(html).toContain('Rivali') + expect(html).toContain('Modalità: 2/3') + expect(html).toContain('14/03/2026') + }) +}) diff --git a/tests/unit/server-utils.test.js b/tests/unit/server-utils.test.js index 179dd89..9d2dea7 100644 --- a/tests/unit/server-utils.test.js +++ b/tests/unit/server-utils.test.js @@ -1,14 +1,9 @@ import { describe, it, expect, vi, afterEach } from 'vitest' -import * as os from 'os' +import { getNetworkIPs, collectIPs, printServerInfo } from '../../src/server-utils.js' -vi.mock('os', async (importOriginal) => { - return { - ...await importOriginal(), - networkInterfaces: vi.fn(() => ({})) - } -}) - -import { getNetworkIPs, printServerInfo } from '../../src/server-utils.js' +// Nota: gli IP vengono iniettati nei test (oggetto stile os.networkInterfaces o +// array di IP), così i risultati sono deterministici su qualsiasi piattaforma — +// incluso WSL, dove getNetworkIPs() userebbe altrimenti PowerShell. describe('Server Utils', () => { @@ -17,85 +12,73 @@ describe('Server Utils', () => { }) // ============================================= - // getNetworkIPs + // getNetworkIPs / collectIPs // ============================================= describe('getNetworkIPs', () => { it('dovrebbe restituire indirizzi IPv4 non-loopback', () => { - os.networkInterfaces.mockReturnValue({ - eth0: [ - { family: 'IPv4', internal: false, address: '192.168.1.100' } - ] - }) - expect(getNetworkIPs()).toEqual(['192.168.1.100']) + expect(getNetworkIPs({ + eth0: [{ family: 'IPv4', internal: false, address: '192.168.1.100' }] + })).toEqual(['192.168.1.100']) }) it('dovrebbe escludere indirizzi loopback (internal)', () => { - os.networkInterfaces.mockReturnValue({ - lo: [ - { family: 'IPv4', internal: true, address: '127.0.0.1' } - ], - eth0: [ - { family: 'IPv4', internal: false, address: '192.168.1.100' } - ] + const ips = getNetworkIPs({ + lo: [{ family: 'IPv4', internal: true, address: '127.0.0.1' }], + eth0: [{ family: 'IPv4', internal: false, address: '192.168.1.100' }] }) - const ips = getNetworkIPs() expect(ips).not.toContain('127.0.0.1') expect(ips).toContain('192.168.1.100') }) it('dovrebbe escludere indirizzi IPv6', () => { - os.networkInterfaces.mockReturnValue({ + const ips = getNetworkIPs({ eth0: [ { family: 'IPv6', internal: false, address: 'fe80::1' }, { family: 'IPv4', internal: false, address: '192.168.1.100' } ] }) - const ips = getNetworkIPs() expect(ips).toEqual(['192.168.1.100']) }) it('dovrebbe escludere bridge Docker 172.17.x.x', () => { - os.networkInterfaces.mockReturnValue({ - docker0: [ - { family: 'IPv4', internal: false, address: '172.17.0.1' } - ], - eth0: [ - { family: 'IPv4', internal: false, address: '10.0.0.5' } - ] + const ips = getNetworkIPs({ + docker0: [{ family: 'IPv4', internal: false, address: '172.17.0.1' }], + eth0: [{ family: 'IPv4', internal: false, address: '10.0.0.5' }] }) - const ips = getNetworkIPs() expect(ips).not.toContain('172.17.0.1') expect(ips).toContain('10.0.0.5') }) it('dovrebbe escludere bridge Docker 172.18.x.x', () => { - os.networkInterfaces.mockReturnValue({ - br0: [ - { family: 'IPv4', internal: false, address: '172.18.0.1' } - ] - }) - expect(getNetworkIPs()).toEqual([]) + expect(getNetworkIPs({ + br0: [{ family: 'IPv4', internal: false, address: '172.18.0.1' }] + })).toEqual([]) + }) + + it('dovrebbe escludere indirizzi link-local 169.254.x.x', () => { + expect(getNetworkIPs({ + eth0: [{ family: 'IPv4', internal: false, address: '169.254.1.1' }] + })).toEqual([]) }) it('dovrebbe restituire array vuoto se nessuna interfaccia disponibile', () => { - os.networkInterfaces.mockReturnValue({}) - expect(getNetworkIPs()).toEqual([]) + expect(getNetworkIPs({})).toEqual([]) }) it('dovrebbe restituire più indirizzi da interfacce diverse', () => { - os.networkInterfaces.mockReturnValue({ - eth0: [ - { family: 'IPv4', internal: false, address: '192.168.1.100' } - ], - wlan0: [ - { family: 'IPv4', internal: false, address: '192.168.1.101' } - ] + const ips = getNetworkIPs({ + eth0: [{ family: 'IPv4', internal: false, address: '192.168.1.100' }], + wlan0: [{ family: 'IPv4', internal: false, address: '192.168.1.101' }] }) - const ips = getNetworkIPs() expect(ips).toHaveLength(2) expect(ips).toContain('192.168.1.100') expect(ips).toContain('192.168.1.101') }) + + it('collectIPs gestisce input vuoto/undefined senza errori', () => { + expect(collectIPs()).toEqual([]) + expect(collectIPs({})).toEqual([]) + }) }) // ============================================= @@ -103,9 +86,8 @@ describe('Server Utils', () => { // ============================================= describe('printServerInfo', () => { it('dovrebbe stampare la porta di default (3000)', () => { - os.networkInterfaces.mockReturnValue({}) const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {}) - printServerInfo() + printServerInfo(3000, []) const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n') expect(allLogs).toContain('3000') expect(allLogs).toContain('/display') @@ -114,22 +96,16 @@ describe('Server Utils', () => { }) it('dovrebbe stampare la porta personalizzata', () => { - os.networkInterfaces.mockReturnValue({}) const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {}) - printServerInfo(8080) + printServerInfo(8080, []) const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n') expect(allLogs).toContain('8080') consoleSpy.mockRestore() }) it('dovrebbe mostrare gli URL remoti se ci sono IP di rete', () => { - os.networkInterfaces.mockReturnValue({ - eth0: [ - { family: 'IPv4', internal: false, address: '192.168.1.50' } - ] - }) const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {}) - printServerInfo(3000) + printServerInfo(3000, ['192.168.1.50']) const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n') expect(allLogs).toContain('192.168.1.50') expect(allLogs).toContain('remoti') @@ -137,9 +113,8 @@ describe('Server Utils', () => { }) it('non dovrebbe mostrare sezione remoti se nessun IP di rete', () => { - os.networkInterfaces.mockReturnValue({}) const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {}) - printServerInfo(3000) + printServerInfo(3000, []) const allLogs = consoleSpy.mock.calls.map(c => c[0]).join('\n') expect(allLogs).not.toContain('remoti') consoleSpy.mockRestore()