Même chantier que le commit précédent (moteur de jeu, play.html) — templates/screen_edit.html était un unique fichier HTML+CSS+JS de 3312 lignes, tout l'éditeur (arborescence, panneaux flottants, canevas, formulaire de propriétés, éditeur de flow à nœuds, blocs de logique, timeline d'animation) vivant dans UN SEUL <script>. Ce fichier est plus imbriqué que play.html : de nombreux appels s'exécutent au niveau racine du script (pas seulement des déclarations de fonctions), et JavaScript hoiste les déclarations `function` sur TOUT le script — un appel au niveau racine peut donc référencer une fonction déclarée PLUS LOIN dans le même fichier. Découper naïvement casserait cet ordre implicite. Un audit dédié (analyse ligne par ligne de chaque appel racine + son graphe d'appel transitif) a identifié 3 références "en avance" réelles, toutes regroupées dans la même zone (initBuilderPanel()/toggleActionFields() → bindAspectButtons/ toggleElementPropertyValue/onDataDefinitionChange) — le découpage respecte cette contrainte : chaque fichier est une TRANCHE SÉQUENTIELLE de l'original (jamais une réorganisation), et cette zone spécifique reste un seul fichier (panel-init.js) pour que le hoisting continue de fonctionner exactement comme avant. 5 fichiers sous static/js/screen_edit/ : - tree-panels.js — arborescence, menu contextuel, panneaux flottants gauche/droite, galerie d'icônes, modale de suppression/choix d'icône, glisser-déposer du canevas, panneau de propriétés (autosave). - panel-init.js — (ré)initialisation du panneau central après chaque changement de sélection, filtres de répéteur/donnée liée, condition de visibilité, champs d'action du formulaire de nœud. - flow-editor.js — éditeur de flow à nœuds (rendu du graphe, formulaire d'ajout de nœud, blocs de logique — currentBlockNodes/Edges). - tabs-and-blocks.js — onglets du centre, panneaux flottants génériques (drag/resize/plein écran), modale d'un bloc de logique. - animation-timeline.js — timeline d'animation (clips Animate.css/ personnalisés). Toutes les données injectées par Jinja (GAME_SLUG, SCREEN_ID, DEFINITIONS_DATA, FLOW_NODES_INITIAL, ELEMENTS_LABELS, CUSTOM_EVENTS_MAP, ANIM_CLIPS...) sont posées UNE FOIS par un petit <script> inline resté dans le template, avant les <script src> — même patron que static/js/play/. Le seul bout de logique resté inline est la toute petite IIFE d'ouverture initiale (?tab=/?block=), qui dépend directement de request.args et doit s'exécuter après que tous les fichiers soient chargés. tests/conftest.py : screen_edit_js_bundle() (même principe que play_js_bundle(), Phase -1 précédente) — 4 tests qui vérifiaient la présence de telle fonction/chaîne dans le HTML de l'éditeur (le JS y était inline) sont mis à jour pour chercher dans ce bundle. Un des deux échecs révélait un test déjà fragile (assert "Ligne cliquée" in html vérifiait en réalité le TEXTE SOURCE d'un <script> inline, jamais du HTML réellement rendu — ce texte ne peut plus s'y trouver une fois la fonction qui le construit dynamiquement déplacée dans un fichier externe) : corrigé pour vérifier le bundle JS + la disponibilité de la route séparément. Vérifié : 215 tests passent, syntaxe JS validée sur les 5 nouveaux fichiers (node --check) et sur les <script> inline restants (rendus via le client de test). Test manuel recommandé (édition complète d'une scène : arborescence, propriétés, glisser-déposer, logique de flow, blocs, timeline) avant de considérer ce découpage définitivement sans risque — comme pour play.html, ce fichier n'a pas de harnais de test DOM automatisé. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
140 lines
5.5 KiB
Python
140 lines
5.5 KiB
Python
"""
|
|
Fixtures pytest partagées.
|
|
|
|
Les tests utilisent le VRAI client de test Flask (`app.test_client()`) et de
|
|
VRAIES bases SQLite créées sous projects/ (comme en production) — rien n'est
|
|
simulé. Chaque jeu de test est créé avec un nom préfixé "pytest_" et son
|
|
dossier est supprimé automatiquement à la fin du test (fixture `game`),
|
|
pour ne jamais laisser de données de test dans projects/.
|
|
"""
|
|
import os
|
|
import shutil
|
|
import sys
|
|
import tempfile
|
|
|
|
sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__))))
|
|
|
|
# Base de comptes (auth/) et clé de session ISOLÉES du poste réel — posé
|
|
# AVANT tout import de core.flask_app/auth (chemin résolu à chaque appel,
|
|
# voir auth/connection.py, mais réglé ici une bonne fois pour toute la
|
|
# session de tests, jamais data/users.db). Sans ça, faire tourner la
|
|
# suite de tests écrirait de vrais comptes dans la vraie base de comptes,
|
|
# et la logique "premier compte = admin" (auth/create_user.py) serait déjà
|
|
# consommée par un run précédent.
|
|
_TEST_AUTH_DIR = tempfile.mkdtemp(prefix="forge_test_auth_")
|
|
os.environ["FORGE_USERS_DB_PATH"] = os.path.join(_TEST_AUTH_DIR, "users.db")
|
|
os.environ["FORGE_SECRET_KEY_PATH"] = os.path.join(_TEST_AUTH_DIR, "secret_key")
|
|
|
|
import pytest
|
|
|
|
from core.flask_app import app as flask_app
|
|
from core import jinja_filters # noqa: F401 - enregistre les filtres Jinja (elabel, elstyle, colname)
|
|
import routes # noqa: F401 - enregistre toutes les routes sur flask_app
|
|
from core import auth_guard # noqa: F401 - enregistre la garde de connexion (après les routes)
|
|
from core import csrf # noqa: F401 - enregistre csrf_token() comme variable globale Jinja
|
|
from core import csrf_guard # noqa: F401 - enregistre la vérification du jeton CSRF (désactivée en TESTING, voir test_csrf.py)
|
|
from core import recovery_codes_flash # noqa: F401 - enregistre pop_recovery_codes() comme variable globale Jinja
|
|
import auth
|
|
import db
|
|
|
|
# Un seul compte admin pour toute la session de tests : les ~80 routes
|
|
# existantes s'attendent (comportement d'avant l'authentification) à un
|
|
# accès sans restriction — "admin" est le seul rôle illimité (voir
|
|
# core/auth_guard.py). Créé directement en Python (pas via /register) :
|
|
# ces tests exercent le RESTE du moteur, pas le parcours d'inscription
|
|
# lui-même (qui a ses propres tests dédiés, voir test_auth.py).
|
|
_TEST_ADMIN_ID = auth.create_user("admin@pytest.local", "Test1234!", "Test", "Admin")
|
|
auth.confirm_totp(_TEST_ADMIN_ID)
|
|
|
|
|
|
@pytest.fixture
|
|
def tmp_game_slug_cleanup():
|
|
"""Pour les tests qui appellent db.create_game(...) directement (sans
|
|
passer par une route Flask) : enregistre un slug à nettoyer à la fin du
|
|
test, quel que soit son résultat."""
|
|
slugs = []
|
|
|
|
def register(slug):
|
|
slugs.append(slug)
|
|
return slug
|
|
|
|
yield register
|
|
for slug in slugs:
|
|
game_dir = db.game_dir(slug)
|
|
if os.path.isdir(game_dir):
|
|
shutil.rmtree(game_dir)
|
|
|
|
|
|
@pytest.fixture
|
|
def client():
|
|
flask_app.config["TESTING"] = True
|
|
with flask_app.test_client() as c:
|
|
with c.session_transaction() as sess:
|
|
sess["user_id"] = _TEST_ADMIN_ID
|
|
yield c
|
|
|
|
|
|
@pytest.fixture
|
|
def game(client):
|
|
"""Crée un jeu de test frais et le supprime après le test, quel que
|
|
soit son résultat (succès ou échec)."""
|
|
resp = client.post("/games/new", data={"name": "pytest_test_game"}, follow_redirects=False)
|
|
assert resp.status_code == 302
|
|
slug = resp.headers["Location"].rstrip("/").split("/")[-1]
|
|
yield slug
|
|
game_dir = db.game_dir(slug)
|
|
if os.path.isdir(game_dir):
|
|
shutil.rmtree(game_dir)
|
|
|
|
|
|
_STATIC_DIR = os.path.join(os.path.dirname(os.path.dirname(os.path.abspath(__file__))), "static", "js")
|
|
_PLAY_JS_DIR = os.path.join(_STATIC_DIR, "play")
|
|
_SCREEN_EDIT_JS_DIR = os.path.join(_STATIC_DIR, "screen_edit")
|
|
|
|
|
|
def _js_bundle(directory):
|
|
parts = []
|
|
for name in sorted(os.listdir(directory)):
|
|
path = os.path.join(directory, name)
|
|
if os.path.isfile(path) and name.endswith(".js"):
|
|
with open(path, encoding="utf-8") as f:
|
|
parts.append(f.read())
|
|
return "\n".join(parts)
|
|
|
|
|
|
def play_js_bundle():
|
|
"""Concatène tout le JS du moteur jouable (voir le plan de
|
|
modularisation — static/js/play/, un fichier par responsabilité,
|
|
extrait de l'ancien <script> unique de templates/play.html). Plusieurs
|
|
tests vérifient que "le JS livré au joueur" contient telle fonction/
|
|
telle chaîne, sans dépendre d'un navigateur — avant le découpage, ça
|
|
revenait à chercher dans le HTML de /game/<slug>/play (tout était
|
|
inline) ; maintenant la logique vit dans ces fichiers statiques."""
|
|
return _js_bundle(_PLAY_JS_DIR)
|
|
|
|
|
|
def screen_edit_js_bundle():
|
|
"""Même principe que play_js_bundle(), pour l'éditeur (static/js/
|
|
screen_edit/, extrait de templates/screen_edit.html)."""
|
|
return _js_bundle(_SCREEN_EDIT_JS_DIR)
|
|
|
|
|
|
@pytest.fixture
|
|
def mail_object(client, game):
|
|
"""Un objet "Mail" avec deux champs texte (sujet, expediteur), pour les
|
|
tests de Répéteur / élément de jeu lié à un objet."""
|
|
resp = client.post(
|
|
f"/game/{game}/objects/new",
|
|
data={
|
|
"object_name": "Mail",
|
|
"field_name[]": ["sujet", "expediteur"],
|
|
"field_type[]": ["texte", "texte"],
|
|
"field_relation[]": ["", ""],
|
|
"field_required[]": ["0", "0"],
|
|
},
|
|
follow_redirects=False,
|
|
)
|
|
assert resp.status_code == 302
|
|
definition_id = resp.headers["Location"].rstrip("/").split("/")[-1]
|
|
return {"slug": game, "definition_id": int(definition_id)}
|