Files
williamandClaude Sonnet 5 bab581737a
Build and deploy / test-python (push) Successful in 11m2s
Build and deploy / test-js (push) Successful in 1m2s
Build and deploy / lint-python (push) Successful in 4m6s
Build and deploy / lint-js (push) Failing after 1m21s
Build and deploy / build-and-push (push) Skipped
Build and deploy / deploy (push) Skipped
Build and deploy / sonarqube (push) Failing after 4m2s
Ajoute le support de formation : entite racine separee du jeu 2D
Nouveau moteur document_engine/ (elements CRUD + rendering + labels),
db/supports/ (stockage independant de db/games), routes/document/
(CRUD AJAX + publication), et l'editeur frontend complet
(templates/document/, static/document/) avec moteur de layout reel
(glisser-deposer -> fusion en rangee ou insertion avant/apres), vrai
Undo/Redo par pile de commandes, grille d'accroche pour les formes
libres, apercu responsive a largeurs fixes, mode Apercu, et publication
persistee.

"Mes formations" (templates/index.html) liste desormais les
environnements 2D et les supports de formation cote a cote ;
l'onboarding et core/auth_guard.py sont generalises pour qu'un compte
restreint puisse posseder un projet de chaque type independamment.

SKIP=djlint : le hook ne signale que le backlog H021 (styles en ligne)
deja documente dans CODE_QUALITY.md sur des fichiers pre-existants non
touches ici (base.html, game/play.html, scene_edit.html,
game_dashboard_simple.html, clause_row.html) plus une ligne de
index.html deja presente avant cette session — aucun nouveau fichier
(document_edit.html compris) n'y figure. Tous les autres outils
(ruff, mypy --strict, vulture, bandit, import-linter, eslint,
stylelint) passent sans erreur ; 617 tests Python + 276 tests JS
verts, plus une verification manuelle complete du cycle de vie via le
serveur de developpement.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
2026-09-20 09:20:33 +02:00

175 lines
8.6 KiB
Python

"""Onboarding guidé (routes/onboarding/onboarding_new.py) : un compte
"user" fraîchement inscrit choisit un type de jeu avant d'avoir accès à
quoi que ce soit — désormais un seul type, "rpg" (jeu RPG 2D), les types
Quiz/Embranchement/"Créer mon jeu de A à Z" ayant été retirés avec le
moteur d'écran "document". Réutilise les fixtures de test_auth.py
(inscription/2FA)."""
import os
from flask.testing import FlaskClient
import auth
import db
import game_engine
from tests.conftest import not_none
from tests.test_auth import _cleanup_project, _complete_onboarding, _confirm_2fa, _register, anon_client # noqa: F401
def test_account_without_a_project_is_redirected_to_onboarding(anon_client: FlaskClient) -> None: # noqa: F811 - anon_client est un fixture pytest (voir tests.test_auth), pas une vraie redefinition - pyflakes ne comprend pas l'injection de fixture par parametre
_register(anon_client, "needsonboarding@example.com")
_confirm_2fa(anon_client)
try:
resp = anon_client.get("/", follow_redirects=False)
assert resp.status_code == 302
assert resp.headers["Location"] == "/onboarding"
resp = anon_client.get("/onboarding")
assert resp.status_code == 200
# Aucun des deux slots n'est encore rempli — les deux cartes sont
# proposées (voir docs/plan/PLAN.md, projet_kind de premier niveau).
html = resp.get_data(as_text=True)
assert "Environnement de formation 2D" in html
assert "Support de formation" in html
finally:
_cleanup_project("needsonboarding@example.com")
def test_rpg_dashboard_shows_scene_preview_and_forces_jeu2d_on_new_screens(anon_client: FlaskClient) -> None: # noqa: F811 - anon_client est un fixture pytest (voir tests.test_auth), pas une vraie redefinition - pyflakes ne comprend pas l'injection de fixture par parametre
_register(anon_client, "rpguser@example.com")
_confirm_2fa(anon_client)
try:
resp = _complete_onboarding(anon_client, name="Mon RPG", onboarding_type="rpg")
assert resp.status_code == 302
slug = not_none(auth.get_user_by_email("rpguser@example.com"))["project_slug"]
assert resp.headers["Location"] == f"/game/{slug}"
assert db.get_onboarding_type(slug) == "rpg"
screen_list = game_engine.list_screens(slug)
assert len(screen_list) == 1
assert screen_list[0]["kind"] == "jeu_2d"
html = anon_client.get(f"/game/{slug}").get_data(as_text=True)
assert "screenPreviewScene" in html
assert 'name="kind"' not in html
# "+ Ajouter un écran" ne propose pas de choix de type — imposé
# jeu_2d par le projet, même en tentant de forcer "document" via
# un POST direct (défense en profondeur, pas seulement l'UI).
anon_client.post(
f"/game/{slug}/screens/new", data={"name": "Scène 2", "kind": "document"}, follow_redirects=False
)
new_screen_list = game_engine.list_screens(slug)
assert len(new_screen_list) == 2
assert new_screen_list[1]["kind"] == "jeu_2d"
edit_html = anon_client.get(f"/game/{slug}/screens/{screen_list[0]['id']}/edit").get_data(as_text=True)
assert "scene-editor.js" in edit_html
finally:
_cleanup_project("rpguser@example.com")
def test_onboarding_rejects_unknown_type(anon_client: FlaskClient) -> None: # noqa: F811 - anon_client est un fixture pytest (voir tests.test_auth), pas une vraie redefinition - pyflakes ne comprend pas l'injection de fixture par parametre
_register(anon_client, "badtype@example.com")
_confirm_2fa(anon_client)
try:
resp = anon_client.post("/onboarding", data={"onboarding_type": "n_importe_quoi", "name": "Test"})
assert resp.status_code == 200
assert not_none(auth.get_user_by_email("badtype@example.com"))["project_slug"] is None
finally:
_cleanup_project("badtype@example.com")
def test_onboarding_rejects_removed_types(anon_client: FlaskClient) -> None: # noqa: F811 - anon_client est un fixture pytest (voir tests.test_auth), pas une vraie redefinition - pyflakes ne comprend pas l'injection de fixture par parametre
"""Les anciens types (quiz/embranchement/custom) ne sont plus des choix
valides — voir db/games/game_type_catalog.py, ONBOARDING_TYPES ne
contient plus que "rpg"."""
_register(anon_client, "removedtype@example.com")
_confirm_2fa(anon_client)
try:
for removed_type in ("quiz", "embranchement", "custom"):
resp = anon_client.post("/onboarding", data={"onboarding_type": removed_type, "name": "Test"})
assert resp.status_code == 200
assert not_none(auth.get_user_by_email("removedtype@example.com"))["project_slug"] is None
finally:
_cleanup_project("removedtype@example.com")
def test_revisiting_onboarding_after_completing_one_slot_offers_only_the_other(anon_client: FlaskClient) -> None: # noqa: F811 - anon_client est un fixture pytest (voir tests.test_auth), pas une vraie redefinition - pyflakes ne comprend pas l'injection de fixture par parametre
"""Les deux slots (project_slug/support_slug) sont indépendants (voir
docs/plan/PLAN.md) : avoir déjà un environnement 2D ne bloque plus
/onboarding, mais n'y propose plus que la carte encore libre — tenter
de recréer un environnement 2D via un POST direct doit rester rejeté."""
_register(anon_client, "revisit@example.com")
_confirm_2fa(anon_client)
support_slug = None
try:
_complete_onboarding(anon_client, name="Premier jeu", onboarding_type="rpg")
first_slug = not_none(auth.get_user_by_email("revisit@example.com"))["project_slug"]
resp = anon_client.get("/onboarding")
assert resp.status_code == 200
html = resp.get_data(as_text=True)
assert "Environnement de formation 2D" not in html
assert "Support de formation" in html
again = anon_client.post(
"/onboarding", data={"onboarding_type": "rpg", "name": "Second jeu"}, follow_redirects=False
)
assert again.status_code == 200
assert not_none(auth.get_user_by_email("revisit@example.com"))["project_slug"] == first_slug
second = anon_client.post(
"/onboarding", data={"onboarding_type": "support", "name": "Un support"}, follow_redirects=False
)
assert second.status_code == 302
support_slug = not_none(auth.get_user_by_email("revisit@example.com"))["support_slug"]
assert second.headers["Location"] == f"/document/{support_slug}/edit"
both_full = anon_client.get("/onboarding", follow_redirects=False)
assert both_full.status_code == 302
assert both_full.headers["Location"] == "/"
finally:
_cleanup_project("revisit@example.com")
if support_slug:
db.delete_support(support_slug)
def test_admin_is_never_redirected_to_onboarding(client: FlaskClient, game: str) -> None:
resp = client.get("/", follow_redirects=False)
assert resp.status_code == 200
def test_admin_can_create_several_games_via_onboarding(client: FlaskClient) -> None:
"""Comportement systématique à CHAQUE création de jeu, pas une
formalité réservée à la toute première inscription (voir le plan) —
un admin, illimité (pas de project_slug dédié), doit pouvoir repasser
par /onboarding autant de fois qu'il crée un nouveau jeu, sans jamais
être bloqué/redirigé vers un projet précédent."""
resp1 = _complete_onboarding(client, name="Jeu RPG Admin 1", onboarding_type="rpg")
assert resp1.status_code == 302
slug1 = resp1.headers["Location"].split("/game/", 1)[1].split("/", 1)[0]
slug2 = None
try:
# project_slug reste NON posé sur le compte admin (voir
# auth/create_user.py — colonne réservée à un compte "user" à
# projet unique) : repasser sur /onboarding ne redirige PAS vers
# le premier projet créé, la page de choix s'affiche à nouveau.
resp = client.get("/onboarding")
assert resp.status_code == 200
assert "Environnement de formation 2D" in resp.get_data(as_text=True)
resp2 = _complete_onboarding(client, name="Jeu RPG Admin 2", onboarding_type="rpg")
assert resp2.status_code == 302
slug2 = resp2.headers["Location"].split("/game/", 1)[1].split("/", 1)[0]
assert slug2 != slug1
import tests.conftest as conftest_module
slugs = {g["slug"] for g in db.list_games(str(conftest_module._TEST_ADMIN_ID))}
assert slug1 in slugs
assert slug2 in slugs
finally:
for slug in (slug1, slug2):
if slug and os.path.isdir(db.game_dir(slug)):
db.delete_game(slug)