Ajoute le système de modèles/thèmes de document ("Utiliser un modèle")
Nouveau bouton dans le topbar de l'éditeur, à côté d'Aperçu, qui ouvre une modale listant les thèmes du catalogue (document_engine/themes/). Cliquer un thème charge un VRAI aperçu (rendu serveur réel dans un iframe, jamais une resucée CSS côté client) avec le choix de garder le contenu actuel ou de le remplacer par le contenu de démonstration du modèle. Architecture pensée pour une centaine de thèmes futurs : chaque thème est une feuille de style externe (static/document/themes/<id>.css) qui habille les classes fixes du moteur, jamais du code qui en changerait la structure. Premier thème implémenté pour valider le mécanisme : "Sécurité Incendie" (6 pages de contenu réel, quiz inclus). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Sonnet 5
parent
a34bcf4159
commit
864ae697fd
@@ -331,6 +331,52 @@ def test_bouton_default_attributes_include_empty_attachment_fields(tmp_support_s
|
||||
}
|
||||
|
||||
|
||||
def test_replace_document_content_rebuilds_pages_and_elements_from_seed(tmp_support_slug_cleanup: Any) -> None:
|
||||
slug, page_id = _new_support(tmp_support_slug_cleanup)
|
||||
doc_engine.add_document_element(slug, "titre", page_id=page_id)
|
||||
|
||||
seed_pages: list[list[dict[str, Any]]] = [
|
||||
[
|
||||
{"kind": "titre", "attributes": {"content": "Page 1 du modèle"}},
|
||||
{
|
||||
"kind": "row",
|
||||
"attributes": {"gap": 20},
|
||||
"children": [
|
||||
{"kind": "paragraphe", "attributes": {"content": "Colonne gauche"}},
|
||||
{"kind": "paragraphe", "attributes": {"content": "Colonne droite"}},
|
||||
],
|
||||
},
|
||||
],
|
||||
[{"kind": "liste_puces", "attributes": {"items": ["Un", "Deux"]}}],
|
||||
]
|
||||
doc_engine.replace_document_content(slug, seed_pages)
|
||||
|
||||
pages = doc_engine.list_document_pages(slug)
|
||||
assert len(pages) == 2
|
||||
|
||||
page1_elements = doc_engine.list_document_elements(slug, pages[0]["id"])
|
||||
titre = next(el for el in page1_elements if el["kind"] == "titre")
|
||||
assert titre["attributes"]["content"] == "Page 1 du modèle"
|
||||
row = next(el for el in page1_elements if el["kind"] == "row")
|
||||
children = [el for el in page1_elements if el["parent_id"] == row["id"]]
|
||||
assert len(children) == 2
|
||||
assert {c["attributes"]["content"] for c in children} == {"Colonne gauche", "Colonne droite"}
|
||||
|
||||
page2_elements = doc_engine.list_document_elements(slug, pages[1]["id"])
|
||||
assert page2_elements[0]["kind"] == "liste_puces"
|
||||
assert page2_elements[0]["attributes"]["items"] == ["Un", "Deux"]
|
||||
|
||||
|
||||
def test_replace_document_content_merges_attributes_onto_defaults(tmp_support_slug_cleanup: Any) -> None:
|
||||
slug, _page_id = _new_support(tmp_support_slug_cleanup)
|
||||
doc_engine.replace_document_content(slug, [[{"kind": "titre", "attributes": {"content": "Seulement le contenu"}}]])
|
||||
page_id = doc_engine.list_document_pages(slug)[0]["id"]
|
||||
element = doc_engine.list_document_elements(slug, page_id)[0]
|
||||
assert element["attributes"]["content"] == "Seulement le contenu"
|
||||
assert element["attributes"]["style"] == "titre1"
|
||||
assert element["attributes"]["align"] == "left"
|
||||
|
||||
|
||||
def test_bouton_with_attachment_filename_renders_data_attribute(tmp_support_slug_cleanup: Any) -> None:
|
||||
slug, page_id = _new_support(tmp_support_slug_cleanup)
|
||||
element_id = doc_engine.add_document_element(slug, "bouton", page_id=page_id)
|
||||
|
||||
@@ -400,3 +400,66 @@ def test_download_attachment_404s_when_nothing_was_uploaded(client: FlaskClient,
|
||||
element_id = _add_bouton(client, support)
|
||||
resp = client.get(f"/document/{support}/elements/{element_id}/download-attachment")
|
||||
assert resp.status_code == 404
|
||||
|
||||
|
||||
def test_document_edit_has_no_theme_link_by_default(client: FlaskClient, support: str) -> None:
|
||||
resp = client.get(f"/document/{support}/edit")
|
||||
assert "document/themes/securite-incendie.css" not in resp.get_data(as_text=True)
|
||||
|
||||
|
||||
def test_document_theme_apply_rejects_an_unknown_theme(client: FlaskClient, support: str) -> None:
|
||||
resp = client.post(
|
||||
f"/document/{support}/theme/apply",
|
||||
json={"theme_id": "n-importe-quoi", "mode": "keep_content"},
|
||||
)
|
||||
assert resp.status_code == 404
|
||||
|
||||
|
||||
def test_document_theme_apply_rejects_an_invalid_mode(client: FlaskClient, support: str) -> None:
|
||||
resp = client.post(
|
||||
f"/document/{support}/theme/apply",
|
||||
json={"theme_id": "securite-incendie", "mode": "n-importe-quoi"},
|
||||
)
|
||||
assert resp.status_code == 400
|
||||
|
||||
|
||||
def test_document_theme_apply_keep_content_only_sets_the_theme(client: FlaskClient, support: str) -> None:
|
||||
page_id = _page_id(support)
|
||||
client.post(f"/document/{support}/elements/add", data={"kind": "titre", "page_id": page_id})
|
||||
|
||||
resp = client.post(
|
||||
f"/document/{support}/theme/apply",
|
||||
json={"theme_id": "securite-incendie", "mode": "keep_content"},
|
||||
)
|
||||
assert resp.status_code == 200
|
||||
assert db.get_document_theme(support) == "securite-incendie"
|
||||
assert len(document_engine.list_document_elements(support, page_id)) == 1
|
||||
|
||||
edit_resp = client.get(f"/document/{support}/edit")
|
||||
assert "document/themes/securite-incendie.css" in edit_resp.get_data(as_text=True)
|
||||
|
||||
|
||||
def test_document_theme_apply_template_content_replaces_everything(client: FlaskClient, support: str) -> None:
|
||||
page_id = _page_id(support)
|
||||
client.post(f"/document/{support}/elements/add", data={"kind": "titre", "page_id": page_id})
|
||||
|
||||
resp = client.post(
|
||||
f"/document/{support}/theme/apply",
|
||||
json={"theme_id": "securite-incendie", "mode": "template_content"},
|
||||
)
|
||||
assert resp.status_code == 200
|
||||
pages = document_engine.list_document_pages(support)
|
||||
theme = document_engine.get_document_theme_entry("securite-incendie")
|
||||
assert theme is not None
|
||||
assert len(pages) == len(theme["seed_pages"])
|
||||
|
||||
|
||||
def test_document_theme_preview_renders_the_first_seed_page(client: FlaskClient, support: str) -> None:
|
||||
resp = client.get(f"/document/{support}/theme/securite-incendie/preview")
|
||||
assert resp.status_code == 200
|
||||
assert "docPage" in resp.get_data(as_text=True)
|
||||
|
||||
|
||||
def test_document_theme_preview_404s_for_an_unknown_theme(client: FlaskClient, support: str) -> None:
|
||||
resp = client.get(f"/document/{support}/theme/n-importe-quoi/preview")
|
||||
assert resp.status_code == 404
|
||||
|
||||
@@ -0,0 +1,74 @@
|
||||
"""Catalogue de thèmes visuels (document_engine/themes/) — conversion du
|
||||
contenu-seed en éléments "à plat" pour l'aperçu (seed_blocks_to_elements),
|
||||
et respect par CHAQUE thème du catalogue des règles structurelles du
|
||||
moteur (ex. un mini-jeu seul sur sa page, voir routes/document/
|
||||
document_element_add.py) — vérifié ici statiquement sur les données du
|
||||
catalogue, sans passer par une route HTTP."""
|
||||
|
||||
from typing import Any
|
||||
|
||||
import document_engine as doc_engine
|
||||
|
||||
|
||||
def test_get_document_theme_entry_returns_none_for_an_unknown_id() -> None:
|
||||
assert doc_engine.get_document_theme_entry("n-importe-quoi") is None
|
||||
|
||||
|
||||
def test_get_document_theme_entry_returns_the_matching_entry() -> None:
|
||||
entry = doc_engine.get_document_theme_entry("securite-incendie")
|
||||
assert entry is not None
|
||||
assert entry["id"] == "securite-incendie"
|
||||
assert entry["name"] == "Sécurité Incendie"
|
||||
assert len(entry["seed_pages"]) > 0
|
||||
|
||||
|
||||
def test_seed_blocks_to_elements_assigns_unique_synthetic_ids() -> None:
|
||||
blocks: list[dict[str, Any]] = [
|
||||
{"kind": "titre", "attributes": {"content": "Titre"}},
|
||||
{
|
||||
"kind": "row",
|
||||
"attributes": {"gap": 10},
|
||||
"children": [
|
||||
{"kind": "paragraphe", "attributes": {"content": "A"}},
|
||||
{"kind": "paragraphe", "attributes": {"content": "B"}},
|
||||
],
|
||||
},
|
||||
]
|
||||
elements = doc_engine.seed_blocks_to_elements(blocks)
|
||||
ids = [el["id"] for el in elements]
|
||||
assert len(ids) == len(set(ids))
|
||||
assert len(elements) == 4
|
||||
|
||||
row = next(el for el in elements if el["kind"] == "row")
|
||||
children = [el for el in elements if el["parent_id"] == row["id"]]
|
||||
assert len(children) == 2
|
||||
|
||||
|
||||
def test_seed_blocks_to_elements_merges_onto_default_attributes() -> None:
|
||||
elements = doc_engine.seed_blocks_to_elements([{"kind": "titre", "attributes": {"content": "Contenu seul"}}])
|
||||
assert elements[0]["attributes"]["content"] == "Contenu seul"
|
||||
assert elements[0]["attributes"]["style"] == "titre1"
|
||||
|
||||
|
||||
def test_seed_blocks_to_elements_renders_without_error() -> None:
|
||||
theme = doc_engine.get_document_theme_entry("securite-incendie")
|
||||
assert theme is not None
|
||||
elements = doc_engine.seed_blocks_to_elements(theme["seed_pages"][0])
|
||||
html = doc_engine.render_document(elements)
|
||||
assert "docText" in html
|
||||
|
||||
|
||||
def test_every_theme_seed_page_respects_the_minigame_exclusivity_rule() -> None:
|
||||
"""Un mini-jeu doit toujours être SEUL sur sa page (règle appliquée
|
||||
côté route pour un ajout manuel, voir routes/document/
|
||||
document_element_add.py) — le contenu-seed d'un thème la respecte
|
||||
dès sa conception puisque document_engine.replace_document_content
|
||||
ne la revérifie pas elle-même (contenu fourni par le thème, pas par
|
||||
l'utilisateur)."""
|
||||
for theme in doc_engine.DOCUMENT_THEMES:
|
||||
for page_blocks in theme["seed_pages"]:
|
||||
minigame_blocks = [b for b in page_blocks if b["kind"] in doc_engine.MINIGAME_KINDS]
|
||||
if minigame_blocks:
|
||||
assert len(page_blocks) == 1, (
|
||||
f"thème {theme['id']!r} : une page avec mini-jeu doit contenir uniquement ce mini-jeu"
|
||||
)
|
||||
@@ -59,3 +59,26 @@ def test_delete_support_removes_it_from_the_listing(tmp_support_slug_cleanup: An
|
||||
assert [s["slug"] for s in db.list_supports("48")] == [slug]
|
||||
db.delete_support(slug)
|
||||
assert db.list_supports("48") == []
|
||||
|
||||
|
||||
def test_new_support_has_no_theme_by_default(tmp_support_slug_cleanup: Any) -> None:
|
||||
slug = db.create_support("Sans thème", owner_folder="49")
|
||||
tmp_support_slug_cleanup(slug)
|
||||
assert db.get_document_theme(slug) is None
|
||||
assert db.support_meta(slug)["theme"] is None
|
||||
|
||||
|
||||
def test_set_document_theme_persists_and_is_readable_back(tmp_support_slug_cleanup: Any) -> None:
|
||||
slug = db.create_support("Avec thème", owner_folder="50")
|
||||
tmp_support_slug_cleanup(slug)
|
||||
db.set_document_theme(slug, "securite-incendie")
|
||||
assert db.get_document_theme(slug) == "securite-incendie"
|
||||
assert db.support_meta(slug)["theme"] == "securite-incendie"
|
||||
|
||||
|
||||
def test_set_document_theme_can_be_changed(tmp_support_slug_cleanup: Any) -> None:
|
||||
slug = db.create_support("Change de thème", owner_folder="51")
|
||||
tmp_support_slug_cleanup(slug)
|
||||
db.set_document_theme(slug, "securite-incendie")
|
||||
db.set_document_theme(slug, "autre-theme")
|
||||
assert db.get_document_theme(slug) == "autre-theme"
|
||||
|
||||
Reference in New Issue
Block a user