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
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>
175 lines
8.6 KiB
Python
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)
|