AcadeNice/corentin_wakdo
10
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

test: le scénario navigateur du responsable suit l'ADR-0020 (commandes visibles, création de compte refusée) #179

Merged
Corentin merged 1 commit from test/rbac-demo-manager-orders into dev 2026-09-27 19:05:00 +02:00
Conversation 0 Commits 1 Files changed 1 +10 -8
Corentin commented 2026-09-27 18:20:31 +02:00
Owner
Copy link

Trouvé en préparant la PR de la console de la page Santé : le test navigateur tests/e2e/rbac-demo.spec.js affirmait encore la grille d'avant l'ADR-0020 (« le responsable ne voit aucun lien vers les commandes »). Depuis #176, le responsable détient order.read et order.cancel : le test échouait. Il n'est pas joué par la CI (tests de bout en bout manuels, voir docs/TESTING.md section 5), d'où le retard.

Ce qui change

  • Le responsable voit désormais Commandes et Cuisine (les deux liens que order.read ouvre dans admin/layout.php).
  • Les refus démontrés sont /admin/roles (pas de role.manage) et /admin/users/new (pas de user.create, geste M5 de docs/demo/matrice-rbac.md).
  • Aucun code applicatif touché.

Preuves

  • rbac-demo.spec.js rejoué sur une pile jetable : 5 scénarios sur 5.
  • gitleaks v8.30.1 sur l'historique de la branche : aucune fuite.
Trouvé en préparant la PR de la console de la page Santé : le test navigateur `tests/e2e/rbac-demo.spec.js` affirmait encore la grille d'avant l'ADR-0020 (« le responsable ne voit aucun lien vers les commandes »). Depuis #176, le responsable détient `order.read` et `order.cancel` : le test échouait. Il n'est pas joué par la CI (tests de bout en bout manuels, voir `docs/TESTING.md` section 5), d'où le retard. ## Ce qui change - Le responsable voit désormais Commandes et Cuisine (les deux liens que `order.read` ouvre dans `admin/layout.php`). - Les refus démontrés sont `/admin/roles` (pas de `role.manage`) et `/admin/users/new` (pas de `user.create`, geste M5 de `docs/demo/matrice-rbac.md`). - Aucun code applicatif touché. ## Preuves - `rbac-demo.spec.js` rejoué sur une pile jetable : 5 scénarios sur 5. - gitleaks v8.30.1 sur l'historique de la branche : aucune fuite.
test: manager RBAC browser scenario follows ADR-0020 (sees orders, refused user creation)
All checks were successful
CI / secret-scan (pull_request) Successful in 25s
Details
CI / php-lint (pull_request) Successful in 28s
Details
CI / static-tests (pull_request) Successful in 2m49s
Details
CI / js-tests (pull_request) Successful in 40s
Details
CI / shell-tests (pull_request) Successful in 11s
Details
b6da033f85 
The e2e spec still asserted the pre-ADR-0020 grid (manager sees no order link), so it failed since #176. The manager now sees Commandes and Cuisine; the demonstrated refusals are /admin/roles and /admin/users/new (no user.create, matrice-rbac.md M5). Run on a throwaway stack: 5/5.
Corentin deleted branch test/rbac-demo-manager-orders 2026-09-27 19:05:00 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
auto-merge
Fusion automatique sur CI verte

No labels
auto-merge
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AcadeNice/corentin_wakdo!179
No description provided.