Merge pull request 'Barre : supprimer le texte parasite « Chantier 5 » rendu au-dessus du header' (#37) from flowdeck-texte-parasite into main
This commit was merged in pull request #37.
This commit is contained in:
@@ -5,10 +5,10 @@
|
|||||||
CSS/JS : /static/css/draco-barre.css, /static/js/vendor/draco-barre.js.
|
CSS/JS : /static/css/draco-barre.css, /static/js/vendor/draco-barre.js.
|
||||||
v7.82.0 (dev/prod) : les href des onglets ne sont plus figés —
|
v7.82.0 (dev/prod) : les href des onglets ne sont plus figés —
|
||||||
l'hôte les injecte depuis sa config (draco_urls(), noms officiels
|
l'hôte les injecte depuis sa config (draco_urls(), noms officiels
|
||||||
par défaut, noms en -dev déclarés dans le .env de l'environnement). #}
|
par défaut, noms en -dev déclarés dans le .env de l'environnement).
|
||||||
Chantier 5 (fusion #69/#71, sans bump) : 5e onglet Console Spark
|
5e onglet Console Spark (data-app="agent", draco_urls().agent),
|
||||||
(data-app="agent", draco_urls().agent), après Kanbante ; l'entrée
|
après Kanbante ; l'entrée « Console Spark » du menu de la marque
|
||||||
« Console Spark » du menu de la marque est conservée.
|
est conservée (menu construit par draco-barre.js depuis les onglets). #}
|
||||||
<link rel="stylesheet" href="/static/css/draco-alertes.css?v={{ asset_version }}">
|
<link rel="stylesheet" href="/static/css/draco-alertes.css?v={{ asset_version }}">
|
||||||
<header class="draco-bar" data-current="flowdeck">
|
<header class="draco-bar" data-current="flowdeck">
|
||||||
<div class="draco-bar-zone draco-bar-gauche">
|
<div class="draco-bar-zone draco-bar-gauche">
|
||||||
|
|||||||
@@ -0,0 +1,48 @@
|
|||||||
|
"""Barre d'apps Flowdeck : aucun texte de chantier rendu hors commentaire.
|
||||||
|
|
||||||
|
Defaut 5 : les lignes « Chantier 5 (fusion #69/#71, sans bump) : ... »
|
||||||
|
etaient placees APRES la fermeture `#}` du commentaire Jinja d'en-tete
|
||||||
|
de `app/templates/_draco_bar.html` (lignes 9-11) : elles etaient donc
|
||||||
|
rendues telles quelles, juste au-dessus du <header> de la barre, sur
|
||||||
|
toutes les pages (la barre est incluse par base.html).
|
||||||
|
|
||||||
|
Test sans dependance (lecture du gabarit seul, comme
|
||||||
|
test_draco_alertes.py) : il tourne sans serveur ni base, donc aussi
|
||||||
|
en sandbox. La note documentaire peut vivre DANS le commentaire Jinja
|
||||||
|
(jamais rendu) ; elle ne doit jamais en sortir.
|
||||||
|
"""
|
||||||
|
|
||||||
|
import re
|
||||||
|
from pathlib import Path
|
||||||
|
|
||||||
|
ROOT = Path(__file__).resolve().parent.parent
|
||||||
|
|
||||||
|
|
||||||
|
def _hors_commentaires(texte):
|
||||||
|
"""Texte du gabarit sans les commentaires Jinja {# ... #}."""
|
||||||
|
return re.sub(r"\{#.*?#\}", "", texte, flags=re.S)
|
||||||
|
|
||||||
|
|
||||||
|
def test_aucun_texte_chantier_rendu():
|
||||||
|
gabarit = (ROOT / "app/templates/_draco_bar.html").read_text()
|
||||||
|
rendu = _hors_commentaires(gabarit)
|
||||||
|
assert "Chantier 5 (fusion" not in rendu
|
||||||
|
assert "sans bump" not in rendu
|
||||||
|
# Aucune ligne de texte libre entre le commentaire d'en-tete et <link> :
|
||||||
|
# le rendu doit commencer (apres commentaires) par <link> ou <header>.
|
||||||
|
assert rendu.lstrip().startswith("<link")
|
||||||
|
|
||||||
|
|
||||||
|
def test_barre_intacte_apres_suppression():
|
||||||
|
"""Le 5e onglet Console Spark et la marque survivent a la suppression."""
|
||||||
|
gabarit = (ROOT / "app/templates/_draco_bar.html").read_text()
|
||||||
|
rendu = _hors_commentaires(gabarit)
|
||||||
|
nav = rendu.split('<nav class="draco-apptabs"')[1].split("</nav>")[0]
|
||||||
|
assert nav.index('data-app="kanbante"') < nav.index('data-app="agent"')
|
||||||
|
assert "Console Spark" in nav
|
||||||
|
assert 'data-app="agent"' in nav
|
||||||
|
assert 'class="draco-brand"' in rendu
|
||||||
|
# L'entree « Console Spark » du menu de la marque est construite par
|
||||||
|
# draco-barre.js depuis les onglets : l'onglet agent suffit a l'alimenter.
|
||||||
|
js_barre = (ROOT / "static/js/vendor/draco-barre.js").read_text()
|
||||||
|
assert "Console Spark" in js_barre
|
||||||
Reference in New Issue
Block a user