Set-OPS-Public/scripts/tests/test_adressage_derive.py
Daniel Allaire 98ab74d047 underlay : les tenants declares sont confrontes aux dossiers reels
Suite du filtre de portee : `underlay.tenants` nomme des DOSSIERS FRERES, et une faute de
frappe y etait invisible — le tenant disparaissait des trois devis du site, qui restaient
« conformes » sur ce qu'il en restait.

Sur un site a UN SEUL tenant — le cas de la prochaine implantation — la faute rend un
devis VIDE : une frontiere sans regle, un commutateur sans VLAN. Rien dans le mot
« conforme » ne dirait qu'on vient de dessiner le vide.

L'ecart est lisible sans toucher au materiel : d'un cote une liste de noms, de l'autre
les dossiers presents. Il se dit donc a `make underlay` (D-75). Quatre situations, quatre
messages distincts : dossier absent ; dossier sans plan/nomenclature.yml ; nomenclature
non federee (index absent, categories vide, federe: false) ; plus aucun nom qui
corresponde.

CE QU'UN GABARIT NE DOIT PAS SUBIR. Un modele decrit du materiel, pas un site deploye :
sans garde, tout modele portant un exemple de `tenants` echouerait chez quiconque n'a pas
ce dossier, et P17 deviendrait rouge sur la machine du voisin. La distinction existait
deja : modeles.py passe des reperes de tenants EXPLICITES (gabarit), le site les laisse
deriver. La verification ne s'applique qu'au second cas.

Trois tests dans test_adressage_derive.py — nom introuvable, clef absente, gabarit
epargne — avec un nom absurde pour qu'aucun test ne depende des dossiers de la machine.

make test 15 + 9 ; prouver 37 OK, 0 echec, 0 saute.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-18 16:55:49 -04:00

158 lines
6.5 KiB
Python

#!/usr/bin/env python3
"""Adressage derive : la bande basse de l'underlay (D-77) et les bornes de l'index.
Ce que ce test garde, et pourquoi il existe. La regle « underlay dans la bande basse »
n'est sure que parce que les zones d'un tenant commencent a `OCTET_ZONE + 1`. C'est une
propriete de `inventory_rules`, pas une place qu'on aurait constatee libre un jour : si
`OCTET_ZONE` bougeait sans que la borne suive, la validation continuerait a dire « oui »
en laissant l'underlay recouvrir les zones — et personne ne le verrait avant que deux
machines ne se disputent une adresse.
Le test derive donc sa frontiere de la MEME constante que le code, et verifie les trois
cas qui comptent, y compris ceux ou un prefixe FRANCHIT la frontiere (un /19 couvre les
octets 0 a 31 : il deborde, meme s'il commence dans la bande basse).
"""
import sys
from pathlib import Path
RACINE = Path(__file__).resolve().parent.parent.parent
sys.path.insert(0, str(RACINE / "scripts"))
import underlay as u # noqa: E402
from inventory_rules import OCTET_ZONE # noqa: E402
SUPERNETS = ["10.11.0.0/16", "10.17.0.0/16"] # deux sites : index 11 et 17
MIEN = 11
def _erreurs(sous: str, index: int | None = MIEN) -> list[str]:
site = {"reseaux": [{"nom": "g", "vlan": 10, "sous_reseau": sous}]}
if index is not None:
site["index"] = index
return [e for e in u.valider(site, tenants_vlans=set(), tenants_supernets=SUPERNETS)
if "deborde" in e or "chevauche le supernet" in e]
def test_la_borne_derive_de_la_regle_des_zones():
"""La frontiere n'est pas 16 en dur : elle suit `OCTET_ZONE`."""
assert u.PREMIER_OCTET_ZONE == OCTET_ZONE + 1
def test_bande_basse_de_son_propre_supernet_acceptee():
for sous in ("10.11.0.0/24", "10.11.4.0/24", "10.11.5.0/24", "10.11.15.0/24"):
assert _erreurs(sous) == [], f"{sous} devrait etre conforme (bande basse)"
def test_bande_des_zones_refusee():
for sous in (f"10.11.{OCTET_ZONE + 1}.0/24", "10.11.18.0/24", "10.11.16.0/20"):
assert _erreurs(sous), f"{sous} devrait etre REFUSE (bande des zones)"
def test_un_prefixe_qui_franchit_la_frontiere_est_refuse():
"""Le piege : il COMMENCE dans la bande basse et deborde. Un test qui ne
regarderait que le premier octet le laisserait passer."""
for sous in ("10.11.0.0/19", "10.11.0.0/16"):
assert _erreurs(sous), f"{sous} couvre au-dela de {OCTET_ZONE} : a refuser"
def test_supernet_d_un_autre_site_refuse():
"""C'est ce qui empeche deux sites d'etre relies — l'erreur la plus couteuse."""
assert _erreurs("10.17.0.0/24"), "le supernet d'un autre site doit etre refuse"
def test_hors_de_tout_supernet_accepte():
"""Le stockage (192.168.x, identique partout) et l'heritage 10.0.x avant migration."""
for sous in ("192.168.20.0/24", "192.168.31.0/24", "10.0.0.0/24"):
assert _erreurs(sous) == [], f"{sous} est hors des supernets : conforme"
def test_sans_index_on_retombe_sur_la_regle_stricte():
"""Un underlay qui n'a pas encore migre ne declare pas d'index : aucun chevauchement
n'est alors tolere, ce qui reste le comportement sur."""
assert _erreurs("10.11.0.0/24", index=None), \
"sans index, tout chevauchement doit etre refuse"
# --- Bornes de l'index (2026-08-12) -----------------------------------------------
# Sans elles, `supernet_de(300)` rendait « 10.300.0.0/16 » : une chaine qui RESSEMBLE
# a un reseau. Elle traverse tout le moteur sans bruit et n'echoue qu'au premier
# `ip_network()` qui la lit — tres loin de l'index fautif.
def test_les_bornes_couvrent_exactement_l_octet():
from inventory_rules import INDEX_MIN, INDEX_MAX
assert (INDEX_MIN, INDEX_MAX) == (0, 255)
def test_index_aux_bornes_accepte():
from inventory_rules import supernet_de
assert supernet_de(0) == "10.0.0.0/16"
assert supernet_de(255) == "10.255.0.0/16"
def test_index_hors_bornes_refuse():
from inventory_rules import supernet_de, base3_de, vlan_de
for fn, args in ((supernet_de, (256,)), (supernet_de, (-1,)),
(base3_de, (300, 1)), (vlan_de, (256, 1))):
try:
fn(*args)
except ValueError:
continue
raise AssertionError(f"{fn.__name__}{args} aurait du etre refuse")
def test_index_non_entier_refuse():
from inventory_rules import supernet_de
for mauvais in ("abc", None, 1.5e400):
try:
supernet_de(mauvais)
except ValueError:
continue
raise AssertionError(f"index {mauvais!r} aurait du etre refuse")
def test_le_vlan_reste_sous_le_plafond_802_1q():
"""La meme borne protege un SECOND plafond, moins visible."""
from inventory_rules import vlan_de, INDEX_MAX
assert vlan_de(INDEX_MAX, 9) < 4094
def test_un_tenant_declare_introuvable_est_refuse_statiquement():
"""`tenants` nomme des dossiers freres : une faute de frappe doit se voir ICI.
Sinon elle ne se voit nulle part — le tenant disparait des trois devis du site, qui
restent « conformes » sur ce qu'il en reste. Le nom est volontairement absurde : le
test ne doit dependre d'aucun dossier reel de la machine qui l'execute.
"""
erreurs = u._erreurs_tenants_declares({"tenants": ["OPS-NExistePas-9f2c"]})
assert any("aucun dossier frere" in e for e in erreurs), erreurs
assert any("aucun nom declare ne correspond" in e for e in erreurs), erreurs
def test_sans_la_clef_tenants_rien_n_est_verifie():
"""Clef absente = toute la federation : un site unique n'a rien a declarer."""
assert u._erreurs_tenants_declares({"reseaux": []}) == []
assert u._erreurs_tenants_declares(None) == []
def test_un_gabarit_n_est_pas_confronte_aux_dossiers_freres():
"""Un modele decrit du MATERIEL : il ne peut pas nommer de tenants reels.
La distinction tient a l'appelant — `modeles.py` passe des reperes de tenants
explicites, ce qui dit « gabarit ». Sans cette garde, tout modele portant un exemple
de `tenants` echouerait chez quiconque n'a pas ce dossier.
"""
gabarit = {"tenants": ["OPS-NExistePas-9f2c"],
"reseaux": [{"nom": "g", "vlan": 10, "sous_reseau": "10.11.0.0/24"}]}
erreurs = u.valider(gabarit, tenants_vlans=set(), tenants_supernets=SUPERNETS)
assert not any("tenants:" in e for e in erreurs), erreurs
if __name__ == "__main__":
n = 0
for nom, fn in sorted(globals().items()):
if nom.startswith("test_") and callable(fn):
fn()
n += 1
print(f"OK — {n} test(s) adressage derive : bande basse (OCTET_ZONE={OCTET_ZONE}) + bornes de l'index.")