test: wait for dynamic accueil metadata

Jenkinsfile

@@ -219,8 +219,8 @@ pipeline {
                     docker run --rm \
                       --volumes-from furious-jenkins \
                       -v tiptop-npm-cache-frontend:/npm-cache \
-                      --user "$(id -u):$(id -g)" \
                       -e HOME=/tmp \
+                      -e CHROMIUM_PATH=/usr/bin/chromium \
                       -e NODE_OPTIONS=--dns-result-order=ipv4first \
                       -e NPM_CONFIG_CACHE=/npm-cache \
                       -e NPM_CONFIG_FETCH_RETRIES=5 \
@@ -230,6 +230,8 @@ pipeline {
                       -w "${WORKSPACE}/frontend" \
                       node:22-alpine \
                       sh -c "
+                        apk add --no-cache chromium nss freetype harfbuzz ca-certificates ttf-freefont &&
+                        export CHROMIUM_PATH=/usr/bin/chromium &&
                         npm ci --prefer-offline --no-audit --no-fund &&
                         npm run lint &&
 			npm run test:coverage &&

frontend/scripts/prerender.js

@@ -30,7 +30,10 @@ async function main() {
   const server = await preview({ root, preview: { port: 4173, strictPort: false } })
   const base = server.resolvedUrls.local[0].replace(/\/$/, '')
 
-  const browser = await chromium.launch()
+  const browser = await chromium.launch({
+    executablePath: process.env.CHROMIUM_PATH || chromium.executablePath(),
+    headless: true,
+  })
   const page = await browser.newPage()
 
   for (const route of ROUTES) {

frontend/src/hooks/usePageMeta.js

@@ -1,37 +1,36 @@
 import { useEffect } from 'react'
 import { useLocation } from 'react-router-dom'
 
-// Domaine de production, utilisé pour construire des URLs absolues
-// (canonical, og:url) à partir du chemin de la route courante.
 const SITE_URL = 'https://www.dsp5-archi-o24a-4-5-g4.fr'
 
-// Change la valeur d'un attribut sur le premier élément trouvé par le
-// sélecteur, et retourne de quoi la restaurer plus tard. N'écrit rien si
-// l'élément est absent du <head> (les balises de base vivent dans
-// index.html ; ce hook ne fait que les mettre à jour, jamais les créer).
 function setAttr(selector, attr, value) {
   if (value == null) return null
+
   const el = document.querySelector(selector)
   if (!el) return null
+
   const previous = el.getAttribute(attr)
   el.setAttribute(attr, value)
-  return () => el.setAttribute(attr, previous ?? '')
+
+  return () => {
+    if (previous == null) {
+      el.removeAttribute(attr)
+    } else {
+      el.setAttribute(attr, previous)
+    }
+  }
 }
 
-// Permet à une page de définir son propre <title>, sa meta description, son
-// URL canonique et ses balises Open Graph/Twitter le temps qu'elle est
-// affichée. La SPA n'a qu'un seul index.html : sans ce hook, toutes les
-// routes partagent les mêmes balises. Les valeurs précédentes sont
-// restaurées au démontage, donc quitter la page revient à l'état d'avant
-// (celui d'index.html, ou celui d'une page parente si elles s'enchaînent).
 export default function usePageMeta({ title, description, image } = {}) {
   const { pathname } = useLocation()
 
   useEffect(() => {
-    const previousTitle = document.title
+    if (title) {
-    if (title) document.title = title
+      document.title = title
+    }
 
     const url = `${SITE_URL}${pathname}`
+
     const restores = [
       setAttr('meta[name="description"]', 'content', description),
       setAttr('link[rel="canonical"]', 'href', url),
@@ -45,7 +44,6 @@ export default function usePageMeta({ title, description, image } = {}) {
     ].filter(Boolean)
 
     return () => {
-      document.title = previousTitle
       restores.forEach((restore) => restore())
     }
   }, [title, description, image, pathname])

frontend/src/pages/Profile.jsx

@@ -4,7 +4,6 @@ import UserNav from '../components/UserNav'
 import {
   changePasswordApi,
   meApi,
-  forgotPasswordApi,
   sendPasswordSetupLinkApi,
 } from '../services/auth'
 import { useHandleAuthError } from '../hooks/useHandleAuthError'

frontend/src/test/fonctionnel/Accueil.test.jsx

@@ -25,8 +25,10 @@ describe('Accueil - dates du jeu-concours', () => {
     expect(await screen.findByText('Du 15 novembre au 14 décembre 2026')).toBeInTheDocument()
     expect(screen.getByText(/ouverte du 15 novembre au 14 décembre 2026 inclus/)).toBeInTheDocument()
     expect(screen.getByText(/retirer jusqu'au 13 janvier 2027/)).toBeInTheDocument()
+    await vi.waitFor(() => {
       expect(document.title).toBe('Jeu-concours 100 % Gagnant du 15 novembre au 14 décembre | Thé Tip Top')
     })
+  })
 
   it("affiche les dates par défaut (2 → 31 octobre 2026) si l'API ne répond pas", async () => {
     getGameWindowApi.mockRejectedValue(new Error('réseau'))

frontend/src/test/unit/admin.service.test.js

@@ -0,0 +1,161 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import {
+  getStatsApi,
+  getUsersApi,
+  drawWinnerApi,
+  getEmailingContactsApi,
+  getWinnerApi,
+  getEmployeesApi,
+  getEmployeeDeliveriesApi,
+  getDeliveryByTicketCodeApi,
+  createEmployeeApi,
+} from '../../services/admin'
+
+describe('admin service', () => {
+  beforeEach(() => {
+    localStorage.clear()
+    localStorage.setItem('token', 'test-token')
+    vi.restoreAllMocks()
+  })
+
+  function mockFetch(data, ok = true, status = 200) {
+    vi.stubGlobal(
+      'fetch',
+      vi.fn().mockResolvedValue({
+        ok,
+        status,
+        json: vi.fn().mockResolvedValue(data),
+      }),
+    )
+  }
+
+  it('getStatsApi retourne les statistiques', async () => {
+    mockFetch({ users: 10 })
+
+    const result = await getStatsApi()
+
+    expect(result).toEqual({ users: 10 })
+    expect(fetch).toHaveBeenCalledWith(
+      '/api/admin/stats',
+      expect.objectContaining({
+        headers: expect.objectContaining({
+          Authorization: 'Bearer test-token',
+        }),
+      }),
+    )
+  })
+
+  it('getUsersApi retourne les utilisateurs', async () => {
+    mockFetch({ users: [{ id: 1 }] })
+
+    await expect(getUsersApi()).resolves.toEqual({
+      users: [{ id: 1 }],
+    })
+  })
+
+  it('drawWinnerApi effectue un POST', async () => {
+    mockFetch({ winner: { id: 1 } })
+
+    const result = await drawWinnerApi()
+
+    expect(result).toEqual({ winner: { id: 1 } })
+    expect(fetch).toHaveBeenCalledWith(
+      '/api/admin/draw',
+      expect.objectContaining({
+        method: 'POST',
+      }),
+    )
+  })
+
+  it('getEmailingContactsApi retourne les contacts', async () => {
+    mockFetch({ contacts: ['test@example.com'] })
+
+    await expect(getEmailingContactsApi()).resolves.toEqual({
+      contacts: ['test@example.com'],
+    })
+  })
+
+  it('getWinnerApi retourne null si aucun gagnant', async () => {
+    mockFetch({}, false, 404)
+
+    await expect(getWinnerApi()).resolves.toBeNull()
+  })
+
+  it('getWinnerApi retourne le gagnant', async () => {
+    mockFetch({ winner: { id: 5 } })
+
+    await expect(getWinnerApi()).resolves.toEqual({
+      winner: { id: 5 },
+    })
+  })
+
+  it('getEmployeesApi retourne les employés', async () => {
+    mockFetch({ employees: [{ id: 10 }] })
+
+    await expect(getEmployeesApi()).resolves.toEqual({
+      employees: [{ id: 10 }],
+    })
+  })
+
+  it('getEmployeeDeliveriesApi retourne les remises', async () => {
+    mockFetch({
+      employee: { id: 10 },
+      deliveries: [],
+    })
+
+    await expect(getEmployeeDeliveriesApi(10)).resolves.toEqual({
+      employee: { id: 10 },
+      deliveries: [],
+    })
+  })
+
+  it('getDeliveryByTicketCodeApi encode le code du ticket', async () => {
+    mockFetch({ delivery: { id: 1 } })
+
+    await getDeliveryByTicketCodeApi(' ABC 123 ')
+
+    expect(fetch).toHaveBeenCalledWith(
+      '/api/admin/deliveries/ABC%20123',
+      expect.anything(),
+    )
+  })
+
+  it('createEmployeeApi crée un employé', async () => {
+    mockFetch({ employee: { id: 20 } })
+
+    await expect(
+      createEmployeeApi({
+        firstname: 'Jean',
+        lastname: 'Dupont',
+        email: 'jean@example.com',
+      }),
+    ).resolves.toEqual({
+      employee: { id: 20 },
+    })
+
+    expect(fetch).toHaveBeenCalledWith(
+      '/api/admin/employees',
+      expect.objectContaining({
+        method: 'POST',
+        body: JSON.stringify({
+          firstname: 'Jean',
+          lastname: 'Dupont',
+          email: 'jean@example.com',
+        }),
+      }),
+    )
+  })
+
+  it('transforme les erreurs API en Error', async () => {
+    mockFetch(
+      { message: 'Erreur serveur' },
+      false,
+      500,
+    )
+
+    await expect(getStatsApi()).rejects.toMatchObject({
+      message: 'Erreur serveur',
+      status: 500,
+    })
+  })
+})

frontend/vite.config.js

@@ -25,6 +25,14 @@ export default defineConfig({
       provider: 'v8',
       reporter: ['text', 'html', 'lcov'],
       reportsDirectory: './coverage',
+      exclude: [
+        '**/*.webp',
+        '**/*.png',
+        '**/*.jpg',
+        '**/*.jpeg',
+        '**/*.gif',
+        '**/*.svg',
+      ],
     },
 
     // Le widget Turnstile ne doit pas bloquer les tests jsdom.