Aller au contenu

PD-231 — Acceptabilité

📚 Navigation User Story | Document | | | ---------- | -- | | 📋 [Spécification](PD-231-specification.md) | | | 🛠️ [Plan d'implémentation](PD-231-plan.md) | | | ✅ **Critères d'acceptation** | *(ce document)* | | 📝 [Retour d'expérience](PD-231-rex.md) | | [← Retour à site-vitrine](../PD-225-epic.md) · [↑ Index User Story](index.md)

Objectif

Vérifier que l’implémentation est conforme à la spécification, respecte l’ensemble des invariants ProbatioVault et ne présente aucune incohérence ou oubli critique.

Périmètre de vérification

La revue d’acceptabilité vérifie explicitement :

  • la conformité stricte à la spécification fonctionnelle
  • le respect de tous les invariants applicables
  • la couverture des scénarios de test définis
  • l’absence d’incohérences, oublis ou régressions

Écarts identifiés

Classification des écarts

Niveau Définition
BLOQUANT Violation d’un invariant, faille de sécurité, non-conformité majeure à la spec
MAJEUR Fonction incomplète, comportement non conforme mais sans rupture de sécurité
MINEUR Détail, dette acceptable, amélioration non critique

Détail des écarts

ID Description Référence Gravité
E-01 Pas de prévisualisation MR/branches : seul le job pages sur main existe, aucune génération d’artefact Pages par branche ni URL de preview exposée dans la MR. L’invariant 2 (preview + URL cliquable + validation manuelle avant prod) n’est pas respecté. Spec §Invariants 2 ; Plan §Pipeline preview ; .gitlab-ci.yml BLOQUANT
E-02 Mise en production non conditionnée à validation manuelle : le job pages déploie directement sur main sans étape d’approbation après revue de l’artefact de preview, contrairement à l’invariant 2. Spec §Invariants 2 ; Plan §Pipeline preview (when: manual pour prod) ; .gitlab-ci.yml MAJEUR
E-03 Étapes build/test manquantes : pipeline réduit à un seul job pages (stage deploy). Aucune séparation build/test ni npm run lint, astro check, test:a11y prévus dans le plan ; pas de cache keyed par branche. Plan §Flux techniques (stages build/test/deploy) ; .gitlab-ci.yml ; package.json (scripts manquants) MAJEUR
E-04 Scripts npm incomplets : package.json ne contient ni lint, ni check, ni test:a11y nécessaires aux jobs test/a11y décrits dans le plan. Plan §Scripts package.json ; package.json MINEUR

Conclusion d’acceptabilité

⛔ REFUSÉ — Présence d’un écart BLOQUANT (absence totale de prévisualisation MR/URL d’artefact) et de plusieurs écarts majeurs sur le pipeline CI/CD GitLab Pages.

Suivi des écarts (append-only)

[2025-12-19] — Suivi E-01

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Job pages:preview ajouté avec règles MR non-main, artefacts public/ issus de dist/, URL d’environnement review/$CI_COMMIT_REF_SLUG exposée via $CI_PAGES_URL, expiration 7 jours et on_stop prévu.
  • Référence vérification :
  • .gitlab-ci.yml:101-135

[2025-12-19] — Suivi E-02

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Job pages sur main avec when: manual, déploiement sur environnement production (https://probatiovault.com), empêchant la mise en prod sans action explicite.
  • Référence vérification :
  • .gitlab-ci.yml:71-97

[2025-12-19] — Suivi E-03

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Pipeline à 4 stages : build (artefact dist/), validate (validate-i18n), test (lint, check), deploy (pages, pages:preview), cache clé ${CI_COMMIT_REF_SLUG}.
  • Référence vérification :
  • .gitlab-ci.yml:1-69

[2025-12-19] — Suivi E-04

  • Statut précédent : OUVERT
  • Statut actuel : RÉSOLU
  • Justification factuelle :
  • Scripts et dépendances ajoutés : check, lint, lint:fix, test:a11y dans package.json avec configs associées (eslint.config.js, .pa11yci.json) et devDependencies requises.
  • Référence vérification :
  • package.json:1-29 ; eslint.config.js ; .pa11yci.json

Verdict d’acceptabilité (courant)

  • ✅ ACCEPTÉ
  • Date : 2025-12-19
  • Motif synthétique : Écarts E-01 à E-04 résolus (preview MR, déploiement prod manuel, pipeline complet, scripts/tests ajoutés).

Historique des verdicts

Date Verdict Version Commentaire
2025-12-18 ⛔ REFUSÉ n/a Verdict initial avec E-01 BLOQUANT (pas de preview MR)
2025-12-19 ✅ ACCEPTÉ n/a Écarts E-01 à E-04 résolus (preview MR, validation manuelle prod, pipeline complet, scripts npm/tests)