fix(ci): tag OpenAPI 'Coffres Secrets' declare + compteur de themes 15 -> 16 #34

Merged
bruno merged 3 commits from fix/ci-tests-openapi-themes into main 2026-10-28 08:27:32 -04:00
Owner

Corrige les 2 echecs du job CI 'Run tests' (2 failed, 1866 passed, 1 skipped).

  1. tests/test_openapi.py::test_used_tags_are_declared — undeclared tags: {'Coffres Secrets'}. Cause : le routeur backend/routers/coffres.py utilise le tag 'Coffres Secrets', mais ce nom n'etait pas dans TAGS_METADATA (backend/openapi_docs.py, passe en openapi_tags dans main.py). Correctif : 1 ligne ajoutee a TAGS_METADATA, apres l'entree Admin, dans le meme style (metadata only, values are never exposed). Le tag du routeur n'est pas change, rien n'est renomme ; _CANONICAL_TAGS etant derive de TAGS_METADATA, test_tags_metadata_declared reste coherent par construction.

  2. tests/test_themes.py::test_theme_count — Expected 15 themes, found 16. Le 16e est 'spark', ajoute deliberement (theme par defaut, release 2.80.2) : c'est le test qui etait obsolete, le theme n'est pas retire. Correctif : docstring module + docstring du test + assertion 15 -> 16 ; meme compteur perime corrige dans un commentaire de frontend/js/themes.js (16 built-in x 4 modes). Aucun autre fichier ne code ce compteur (verifie : unit.test.mjs n'asserte aucun nombre de themes).

Verifie : py_compile OK sur les 2 fichiers Python, node --check OK sur themes.js, ruff OK sur openapi_docs.py, assertions exactes des 2 tests repliquees en local (tag declare, 16 noms dont spark), fichiers re-telecharges depuis la branche identiques, main intact. Non execute en local : les 2 tests pytest cibles (pas de checkout complet dans le sandbox, boot de backend.main:app impossible ici) — la CI devra confirmer test_openapi + test_themes verts, puis le job test complet. Note : test_themes.py porte 1 erreur I001 pre-existante sur main (hors perimetre, job lint ne verifie que backend/), volontairement non touchee.

Corrige les 2 echecs du job CI 'Run tests' (2 failed, 1866 passed, 1 skipped). 1) tests/test_openapi.py::test_used_tags_are_declared — undeclared tags: {'Coffres Secrets'}. Cause : le routeur backend/routers/coffres.py utilise le tag 'Coffres Secrets', mais ce nom n'etait pas dans TAGS_METADATA (backend/openapi_docs.py, passe en openapi_tags dans main.py). Correctif : 1 ligne ajoutee a TAGS_METADATA, apres l'entree Admin, dans le meme style (metadata only, values are never exposed). Le tag du routeur n'est pas change, rien n'est renomme ; _CANONICAL_TAGS etant derive de TAGS_METADATA, test_tags_metadata_declared reste coherent par construction. 2) tests/test_themes.py::test_theme_count — Expected 15 themes, found 16. Le 16e est 'spark', ajoute deliberement (theme par defaut, release 2.80.2) : c'est le test qui etait obsolete, le theme n'est pas retire. Correctif : docstring module + docstring du test + assertion 15 -> 16 ; meme compteur perime corrige dans un commentaire de frontend/js/themes.js (16 built-in x 4 modes). Aucun autre fichier ne code ce compteur (verifie : unit.test.mjs n'asserte aucun nombre de themes). Verifie : py_compile OK sur les 2 fichiers Python, node --check OK sur themes.js, ruff OK sur openapi_docs.py, assertions exactes des 2 tests repliquees en local (tag declare, 16 noms dont spark), fichiers re-telecharges depuis la branche identiques, main intact. Non execute en local : les 2 tests pytest cibles (pas de checkout complet dans le sandbox, boot de backend.main:app impossible ici) — la CI devra confirmer test_openapi + test_themes verts, puis le job test complet. Note : test_themes.py porte 1 erreur I001 pre-existante sur main (hors perimetre, job lint ne verifie que backend/), volontairement non touchee.
bruno added 3 commits 2026-10-28 08:26:45 -04:00
docs: commentaire 15 -> 16 themes integres (meme compteur obsolete que test_themes.py)
CI / security (pull_request) Failing after 0s
CI / lint (pull_request) Successful in 3m11s
CI / test (pull_request) Failing after 5m8s
CI / build (pull_request) Skipped
CI / e2e (pull_request) Skipped
2a1780b43d
bruno merged commit 6713510ce9 into main 2026-10-28 08:27:32 -04:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Projets/ObsiGate#34