erplibre/test/test_anonymize.py

1821 lines
68 KiB
Python
Raw Permalink Normal View History

[ADD] analyse: anonymiser une copie, sans IA et sans rien casser Des mots pris dans une liste, des nombres tirés entre 0 et 1000, écrits en SQL. Aucun modèle, aucun réseau — un test le vérifie sur les imports. Le difficile n'est pas de remplacer, c'est de savoir ce qu'on n'a PAS le droit de toucher. Mesuré sur une base 18 réelle : 505 champs `selection` sont stockés en varchar, 2693 many2one sont des entiers, 194 textes sont des jsonb par langue, 301 contraintes d'unicité attendent une collision. « Tous les champs string » n'existe pas ; on croise ir_model_fields, pg_attribute et pg_constraint, et aucune des trois ne suffit seule. Trois pièges ont été trouvés en LANÇANT l'outil, pas en le relisant : PostgreSQL refuse d'indexer un ARRAY[...] sans parenthèses, res_partner.credit_limit est un jsonb qu'Odoo appelle float, et crm_lead.probability porte un CHECK qui interdit 1000. Chaque fois l'écriture a échoué et la base est restée intacte : une seule transaction, tout ou rien. Preuve sur copie jetable : empreinte du schéma identique, 848 tables, 6495 contraintes, arch_db et xmlid intacts, lang et many2one inchangés — seules les colonnes visées ont changé. --- EN --- Words from a list, numbers drawn between 0 and 1000, written in SQL. No model, no network — a test checks that on the imports. The hard part is not replacing, it is knowing what must NOT be touched. Measured on a real 18 database: 505 `selection` fields are stored as varchar, 2693 many2one are integers, 194 texts are per-language jsonb, 301 unique constraints await a collision. "All string fields" does not exist; we cross ir_model_fields, pg_attribute and pg_constraint, and none of the three is enough alone. Three traps were found by RUNNING it, not by rereading it: PostgreSQL refuses to subscript a bare ARRAY[...], res_partner.credit_limit is a jsonb Odoo calls float, and crm_lead.probability has a CHECK forbidding 1000. Each time the write failed and the database stayed intact: one transaction, all or nothing. Proof on a throwaway copy: identical schema fingerprint, 848 tables, 6495 constraints, arch_db and xmlids intact, lang and many2one unchanged — only the targeted columns changed. Assisted-by: Claude Opus 5
2026-08-25 00:43:50 -04:00
#!/usr/bin/env python3
# © 2021-2026 TechnoLibre (http://www.technolibre.ca)
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
"""Anonymiser : ce qui compte, c'est ce qu'on REFUSE de toucher.
Remplacer des mots est facile. Ce qui casse une base, c'est de croire que
« tous les champs string » veut dire quelque chose. Mesuré sur une base
réelle en 18 : 505 champs `selection` sont stockés en varchar
(`res.partner.lang`, `sale.order.invoice_status`), 2693 many2one sont des
entiers, 194 textes sont des `jsonb` par langue, et 301 contraintes
d'unicité attendent une collision.
Trois de ces pièges ont été trouvés en LANÇANT l'outil sur une copie
jetable, pas en le relisant : PostgreSQL refuse d'indexer un
`ARRAY[...]` sans parenthèses, `res_partner.credit_limit` est un jsonb
alors qu'Odoo l'appelle `float`, et `crm_lead.probability` porte un CHECK
qui interdit 1000. Chacun aurait fait échouer l'écriture — et l'écriture
étant transactionnelle, chaque fois la base est restée intacte. Ce
fichier fige ces trois-là pour qu'ils ne reviennent pas.
Le plancher est l'objet du premier bloc : aucun mode, aucune liste
blanche, aucune insistance ne doit permettre d'écrire dans `ir.*`.
"""
import ast
import io
import math
import sys
[ADD] analyse: anonymiser une copie, sans IA et sans rien casser Des mots pris dans une liste, des nombres tirés entre 0 et 1000, écrits en SQL. Aucun modèle, aucun réseau — un test le vérifie sur les imports. Le difficile n'est pas de remplacer, c'est de savoir ce qu'on n'a PAS le droit de toucher. Mesuré sur une base 18 réelle : 505 champs `selection` sont stockés en varchar, 2693 many2one sont des entiers, 194 textes sont des jsonb par langue, 301 contraintes d'unicité attendent une collision. « Tous les champs string » n'existe pas ; on croise ir_model_fields, pg_attribute et pg_constraint, et aucune des trois ne suffit seule. Trois pièges ont été trouvés en LANÇANT l'outil, pas en le relisant : PostgreSQL refuse d'indexer un ARRAY[...] sans parenthèses, res_partner.credit_limit est un jsonb qu'Odoo appelle float, et crm_lead.probability porte un CHECK qui interdit 1000. Chaque fois l'écriture a échoué et la base est restée intacte : une seule transaction, tout ou rien. Preuve sur copie jetable : empreinte du schéma identique, 848 tables, 6495 contraintes, arch_db et xmlid intacts, lang et many2one inchangés — seules les colonnes visées ont changé. --- EN --- Words from a list, numbers drawn between 0 and 1000, written in SQL. No model, no network — a test checks that on the imports. The hard part is not replacing, it is knowing what must NOT be touched. Measured on a real 18 database: 505 `selection` fields are stored as varchar, 2693 many2one are integers, 194 texts are per-language jsonb, 301 unique constraints await a collision. "All string fields" does not exist; we cross ir_model_fields, pg_attribute and pg_constraint, and none of the three is enough alone. Three traps were found by RUNNING it, not by rereading it: PostgreSQL refuses to subscript a bare ARRAY[...], res_partner.credit_limit is a jsonb Odoo calls float, and crm_lead.probability has a CHECK forbidding 1000. Each time the write failed and the database stayed intact: one transaction, all or nothing. Proof on a throwaway copy: identical schema fingerprint, 848 tables, 6495 constraints, arch_db and xmlids intact, lang and many2one unchanged — only the targeted columns changed. Assisted-by: Claude Opus 5
2026-08-25 00:43:50 -04:00
import unittest
from pathlib import Path
from script.analyse import anonymize as anon
MOTEUR = (
Path(__file__).resolve().parent.parent
/ "script"
/ "analyse"
/ "anonymize.py"
)
def champ(
nom,
ttype="char",
pg="character varying",
unique=False,
checked=False,
modele="res.partner",
):
return {
"model": modele,
"name": nom,
"ttype": ttype,
"pg_type": pg,
"unique": unique,
"checked": checked,
}
class TestTheFloorNobodyCanLift(unittest.TestCase):
"""`ir.*` reste intouchable, quel que soit le chemin."""
def test_ir_models_are_refused_even_when_whitelisted(self):
tous = {"ir.ui.view", "ir.model.fields", "res.partner"}
for mode in anon.MODES:
choisis = anon.choisir_modeles(
tous, mode, whitelist=list(tous), blacklist=[]
)
self.assertNotIn("ir.ui.view", choisis, mode)
self.assertNotIn("ir.model.fields", choisis, mode)
def test_languages_and_currencies_are_refused_too(self):
"""Ce ne sont pas des données personnelles, et les casser casse
les adresses et les montants."""
tous = {"res.lang", "res.currency", "res.country", "res.partner"}
choisis = anon.choisir_modeles(tous, "whitelist", whitelist=list(tous))
self.assertEqual(choisis, ["res.partner"])
def test_a_blacklist_of_nothing_still_respects_the_floor(self):
tous = {"ir.ui.view", "res.partner", "res.lang"}
self.assertEqual(
anon.choisir_modeles(tous, "blacklist", blacklist=[]),
["res.partner"],
)
class TestWhatIsNeverReplaced(unittest.TestCase):
"""Le cœur : distinguer un vrai texte d'un varchar qui n'en est pas un."""
def test_a_selection_stored_as_varchar_is_left_alone(self):
"""505 dans la base mesurée. Y écrire un mot casse l'ORM."""
self.assertFalse(anon.champ_retenu(champ("lang", "selection")))
self.assertFalse(
anon.champ_retenu(champ("invoice_status", "selection"))
)
def test_a_many2one_is_left_alone(self):
"""2693 entiers qui sont des relations."""
self.assertFalse(
anon.champ_retenu(champ("parent_id", "many2one", "integer"))
)
def test_a_many2one_is_refused_by_its_type_not_only_its_name(self):
"""Sur la base mesurée, tous les many2one finissent par `_id` — mais
un module maison peut en nommer un `owner`, et la convention ne
peut pas être la seule barrière. C'est le TYPE qui décide."""
self.assertFalse(
anon.champ_retenu(champ("owner", "many2one", "integer"))
)
self.assertFalse(
anon.champ_retenu(champ("responsable", "many2one", "integer"))
)
def test_anything_named_like_a_relation_is_left_alone(self):
for nom in ("company_id", "tag_ids", "partner_id"):
self.assertFalse(anon.champ_retenu(champ(nom, "integer")), nom)
def test_a_number_living_in_a_jsonb_is_left_alone(self):
"""Mesuré : res_partner.credit_limit est un float DANS un jsonb."""
self.assertFalse(
anon.champ_retenu(champ("credit_limit", "float", "jsonb"))
)
def test_a_column_under_a_check_constraint_is_left_alone(self):
"""Mesuré : crm_lead.probability doit rester entre 0 et 100."""
self.assertFalse(
anon.champ_retenu(
champ("probability", "float", "numeric", checked=True)
)
)
def test_technical_columns_are_left_alone(self):
for nom in (
"id",
"create_uid",
"write_date",
"state",
"sequence",
"arch_db",
"active",
):
self.assertFalse(anon.champ_retenu(champ(nom, "char")), nom)
def test_logins_stay_unless_asked(self):
"""Sinon personne ne peut plus ouvrir la copie qu'on anonymise."""
self.assertFalse(anon.champ_retenu(champ("login")))
self.assertTrue(
anon.champ_retenu(champ("login"), inclure_connexion=True)
)
def test_a_real_text_is_taken(self):
for ttype in ("char", "text", "html"):
self.assertTrue(anon.champ_retenu(champ("name", ttype)), ttype)
def test_a_real_number_is_taken(self):
for ttype in ("integer", "float", "monetary"):
self.assertTrue(
anon.champ_retenu(champ("amount", ttype, "numeric")), ttype
)
class TestTheSqlItWrites(unittest.TestCase):
def test_a_null_stays_null(self):
"""Un NULL devenu mot créerait de la donnée là où il n'y en avait
pas : la copie mentirait dans l'autre sens."""
sql = anon.expression_texte(champ("name"), ["a"])
self.assertIn("IS NULL THEN NULL", sql)
self.assertIn(
"IS NULL THEN NULL",
anon.expression_nombre(champ("x", "integer", "integer")),
)
def test_the_array_is_parenthesised(self):
"""PostgreSQL refuse d'indexer un ARRAY[...] nu — mesuré."""
sql = anon.expression_texte(champ("name"), ["a", "b"])
self.assertIn("(ARRAY[", sql)
self.assertNotIn("] ARRAY[", sql)
self.assertRegex(sql, r"\(ARRAY\[[^\]]*\]\)\[")
def test_a_unique_column_gets_the_id_appended(self):
"""Deux lignes au même mot feraient échouer TOUT l'UPDATE."""
sql = anon.expression_texte(champ("ref", unique=True), ["a"])
self.assertIn('"id"::text', sql)
[ADD] analyse: anonymiser une copie, sans IA et sans rien casser Des mots pris dans une liste, des nombres tirés entre 0 et 1000, écrits en SQL. Aucun modèle, aucun réseau — un test le vérifie sur les imports. Le difficile n'est pas de remplacer, c'est de savoir ce qu'on n'a PAS le droit de toucher. Mesuré sur une base 18 réelle : 505 champs `selection` sont stockés en varchar, 2693 many2one sont des entiers, 194 textes sont des jsonb par langue, 301 contraintes d'unicité attendent une collision. « Tous les champs string » n'existe pas ; on croise ir_model_fields, pg_attribute et pg_constraint, et aucune des trois ne suffit seule. Trois pièges ont été trouvés en LANÇANT l'outil, pas en le relisant : PostgreSQL refuse d'indexer un ARRAY[...] sans parenthèses, res_partner.credit_limit est un jsonb qu'Odoo appelle float, et crm_lead.probability porte un CHECK qui interdit 1000. Chaque fois l'écriture a échoué et la base est restée intacte : une seule transaction, tout ou rien. Preuve sur copie jetable : empreinte du schéma identique, 848 tables, 6495 contraintes, arch_db et xmlid intacts, lang et many2one inchangés — seules les colonnes visées ont changé. --- EN --- Words from a list, numbers drawn between 0 and 1000, written in SQL. No model, no network — a test checks that on the imports. The hard part is not replacing, it is knowing what must NOT be touched. Measured on a real 18 database: 505 `selection` fields are stored as varchar, 2693 many2one are integers, 194 texts are per-language jsonb, 301 unique constraints await a collision. "All string fields" does not exist; we cross ir_model_fields, pg_attribute and pg_constraint, and none of the three is enough alone. Three traps were found by RUNNING it, not by rereading it: PostgreSQL refuses to subscript a bare ARRAY[...], res_partner.credit_limit is a jsonb Odoo calls float, and crm_lead.probability has a CHECK forbidding 1000. Each time the write failed and the database stayed intact: one transaction, all or nothing. Proof on a throwaway copy: identical schema fingerprint, 848 tables, 6495 constraints, arch_db and xmlids intact, lang and many2one unchanged — only the targeted columns changed. Assisted-by: Claude Opus 5
2026-08-25 00:43:50 -04:00
self.assertNotIn(
'"id"::text', anon.expression_texte(champ("ref"), ["a"])
[ADD] analyse: anonymiser une copie, sans IA et sans rien casser Des mots pris dans une liste, des nombres tirés entre 0 et 1000, écrits en SQL. Aucun modèle, aucun réseau — un test le vérifie sur les imports. Le difficile n'est pas de remplacer, c'est de savoir ce qu'on n'a PAS le droit de toucher. Mesuré sur une base 18 réelle : 505 champs `selection` sont stockés en varchar, 2693 many2one sont des entiers, 194 textes sont des jsonb par langue, 301 contraintes d'unicité attendent une collision. « Tous les champs string » n'existe pas ; on croise ir_model_fields, pg_attribute et pg_constraint, et aucune des trois ne suffit seule. Trois pièges ont été trouvés en LANÇANT l'outil, pas en le relisant : PostgreSQL refuse d'indexer un ARRAY[...] sans parenthèses, res_partner.credit_limit est un jsonb qu'Odoo appelle float, et crm_lead.probability porte un CHECK qui interdit 1000. Chaque fois l'écriture a échoué et la base est restée intacte : une seule transaction, tout ou rien. Preuve sur copie jetable : empreinte du schéma identique, 848 tables, 6495 contraintes, arch_db et xmlid intacts, lang et many2one inchangés — seules les colonnes visées ont changé. --- EN --- Words from a list, numbers drawn between 0 and 1000, written in SQL. No model, no network — a test checks that on the imports. The hard part is not replacing, it is knowing what must NOT be touched. Measured on a real 18 database: 505 `selection` fields are stored as varchar, 2693 many2one are integers, 194 texts are per-language jsonb, 301 unique constraints await a collision. "All string fields" does not exist; we cross ir_model_fields, pg_attribute and pg_constraint, and none of the three is enough alone. Three traps were found by RUNNING it, not by rereading it: PostgreSQL refuses to subscript a bare ARRAY[...], res_partner.credit_limit is a jsonb Odoo calls float, and crm_lead.probability has a CHECK forbidding 1000. Each time the write failed and the database stayed intact: one transaction, all or nothing. Proof on a throwaway copy: identical schema fingerprint, 848 tables, 6495 constraints, arch_db and xmlids intact, lang and many2one unchanged — only the targeted columns changed. Assisted-by: Claude Opus 5
2026-08-25 00:43:50 -04:00
)
def test_a_translated_column_is_rebuilt_key_by_key(self):
"""Écrire une chaîne dans un jsonb détruirait la colonne."""
sql = anon.expression_texte(champ("comment", "html", "jsonb"), ["a"])
self.assertIn("jsonb_object_agg", sql)
self.assertIn("jsonb_each_text", sql)
def test_a_plain_text_column_is_not_treated_as_json(self):
sql = anon.expression_texte(champ("comment", "text", "text"), ["a"])
self.assertNotIn("jsonb", sql)
def test_numbers_land_between_zero_and_a_thousand(self):
entier = anon.expression_nombre(champ("n", "integer", "integer"))
self.assertIn("1001", entier)
decimal = anon.expression_nombre(champ("x", "float", "numeric"))
self.assertIn("1000", decimal)
def test_a_word_with_a_quote_cannot_break_out(self):
"""Une liste de mots vient d'un fichier : elle n'est pas de confiance."""
sql = anon.expression_texte(champ("name"), ["l'ete"])
self.assertIn("'l''ete'", sql)
def test_one_update_per_table_not_per_column(self):
sql = anon.sql_pour_table(
"res_partner", [champ("name"), champ("ref")], None
)
self.assertEqual(sql.count("UPDATE"), 1)
self.assertTrue(sql.endswith(";"))
def test_no_column_means_no_statement(self):
self.assertIsNone(anon.sql_pour_table("res_partner", [], None))
class TestTheModes(unittest.TestCase):
TOUS = {"res.partner", "crm.lead", "sale.order", "ir.ui.view"}
def test_whitelist_takes_only_what_it_names(self):
self.assertEqual(
anon.choisir_modeles(self.TOUS, "whitelist", ["crm.lead"]),
["crm.lead"],
)
def test_blacklist_takes_everything_else(self):
choisis = anon.choisir_modeles(
self.TOUS, "blacklist", blacklist=["crm.lead"]
)
self.assertNotIn("crm.lead", choisis)
self.assertIn("res.partner", choisis)
def test_hybrid_starts_from_the_defaults_and_adjusts(self):
choisis = anon.choisir_modeles(
self.TOUS,
"hybrid",
whitelist=["sale.order"],
blacklist=["res.partner"],
)
self.assertIn("sale.order", choisis)
self.assertIn("crm.lead", choisis) # dans les défauts
self.assertNotIn("res.partner", choisis) # retiré
def test_an_unknown_mode_raises_rather_than_guessing(self):
with self.assertRaises(ValueError):
anon.choisir_modeles(self.TOUS, "peut-etre")
class TestTheWordList(unittest.TestCase):
def test_a_flat_list_is_used_everywhere(self):
self.assertEqual(anon.mots_pour("name", ["a", "b"]), ("a", "b"))
def test_a_dictionary_can_answer_per_field(self):
mots = {"email": ["a@b.c"], "*": ["mot"]}
self.assertEqual(anon.mots_pour("email", mots), ("a@b.c",))
self.assertEqual(anon.mots_pour("name", mots), ("mot",))
def test_an_empty_list_falls_back_to_the_built_in(self):
self.assertEqual(anon.mots_pour("name", []), anon.MOTS_PAR_DEFAUT)
class TestThereIsNoModelInTheLoop(unittest.TestCase):
"""« sans passer par un GPT » : vérifié sur le code, pas sur parole."""
def test_the_engine_imports_nothing_that_could_call_out(self):
arbre = ast.parse(MOTEUR.read_text(encoding="utf-8"))
interdits = {
"requests",
"urllib",
"urllib3",
"http",
"httpx",
"socket",
"openai",
"anthropic",
"xmlrpc",
"json",
}
for noeud in ast.walk(arbre):
noms = []
if isinstance(noeud, ast.Import):
noms = [a.name.split(".")[0] for a in noeud.names]
elif isinstance(noeud, ast.ImportFrom) and noeud.module:
noms = [noeud.module.split(".")[0]]
for nom in noms:
self.assertNotIn(nom, interdits, nom)
def test_the_write_is_one_transaction(self):
"""Une collision au dixième modèle laisserait une base à moitié
anonymisée, que rien ne rattrape sinon une restauration."""
arbre = ast.parse(MOTEUR.read_text(encoding="utf-8"))
fonction = [
n
for n in ast.walk(arbre)
if isinstance(n, ast.FunctionDef) and n.name == "ecrire"
]
self.assertEqual(len(fonction), 1)
# Le corps SANS la docstring : le texte la mentionne, l'argument
# doit être réellement passé.
corps = [
n
for n in fonction[0].body
if not (
isinstance(n, ast.Expr)
and isinstance(n.value, ast.Constant)
and isinstance(n.value.value, str)
)
]
litteraux = {
n.value
for bloc in corps
for n in ast.walk(bloc)
if isinstance(n, ast.Constant) and isinstance(n.value, str)
}
self.assertIn("-1", litteraux)
self.assertIn("ON_ERROR_STOP=1", litteraux)
class TestTheRefusalToWrite(unittest.TestCase):
"""Le refus doit précéder la connexion.
Sinon ces deux tests passent au vert parce que la base n'existe pas,
et ne prouvent rien du garde. On lit donc le message rendu, et l'on
vérifie qu'aucun psql n'a été appelé.
"""
def _refus(self, argv):
import contextlib
import io as flux
appels = []
vrai = anon.lib_analyse.require_odoo_database
def espion(*a, **k):
appels.append(a)
return vrai(*a, **k)
anon.lib_analyse.require_odoo_database = espion
sortie = flux.StringIO()
try:
with contextlib.redirect_stderr(sortie):
code = anon.main(argv)
finally:
anon.lib_analyse.require_odoo_database = vrai
return code, sortie.getvalue(), appels
def test_apply_without_a_matching_confirm_is_refused(self):
code, message, appels = self._refus(
["-d", "une_base", "--apply", "--confirm", "une_autre"]
)
self.assertEqual(code, 2)
self.assertIn(
anon.t("Refusing to write: --confirm must repeat"), message
)
self.assertEqual(appels, [], "la base a été contactée pour refuser")
def test_apply_with_no_confirm_at_all_is_refused(self):
code, message, appels = self._refus(["-d", "une_base", "--apply"])
self.assertEqual(code, 2)
self.assertIn(
anon.t("Refusing to write: --confirm must repeat"), message
)
self.assertEqual(appels, [])
class TestTheSqlNeverTravelsThroughArgv(unittest.TestCase):
"""La panne signalée : « OSError: [Errno 7] Argument list too long ».
Linux plafonne UN SEUL argument à MAX_ARG_STRLEN — 32 pages, soit
131 072 octets. Mesuré sur une base réelle : le mode hybride produit
58 Ko de SQL et passait, la liste noire en produit 342 Ko sur 410
modèles et cassait. Le mode qui couvre le plus était celui qui
échouait, donc celui qu'aucun de mes essais n'exerçait.
Le rendu de `render` reste borné, lui ; c'est bien l'exécution qu'il
faut regarder, et pas seulement le plan.
"""
def _executer(self, etapes):
"""Lancer `ecrire` avec un faux psql, et rendre ce qu'il a reçu."""
import script.analyse.anonymize as module
vu = {}
vrai_run = module.__dict__.get("subprocess")
class FauxFait:
returncode = 0
stdout = ""
stderr = ""
import subprocess as vrai_subprocess
def espion(cmd, **kwargs):
vu["cmd"] = list(cmd)
chemin = cmd[cmd.index("-f") + 1] if "-f" in cmd else None
if chemin:
with open(chemin, encoding="utf-8") as handle:
vu["fichier"] = handle.read()
vu["chemin"] = chemin
return FauxFait()
vrai_env = anon.lib_analyse.pg_env
anon.lib_analyse.pg_env = lambda *a, **k: {"PATH": "/usr/bin"}
vrai_subprocess_run = vrai_subprocess.run
vrai_subprocess.run = espion
try:
erreur = anon.ecrire("une_base", etapes)
finally:
vrai_subprocess.run = vrai_subprocess_run
anon.lib_analyse.pg_env = vrai_env
del vrai_run
vu["erreur"] = erreur
return vu
def _gros_plan(self, combien=400):
"""Un plan de la taille de celui qui cassait."""
etapes = []
for index in range(combien):
champ = {
"model": "m.%d" % index,
"name": "name",
"ttype": "char",
"pg_type": "character varying",
"unique": False,
"checked": False,
}
etapes.append(
{
"model": champ["model"],
"fields": [champ],
"sql": anon.sql_pour_table("m_%d" % index, [champ], None),
}
)
return etapes
def test_no_single_argument_comes_close_to_the_kernel_limit(self):
vu = self._executer(self._gros_plan())
plus_gros = max(len(a) for a in vu["cmd"])
self.assertLess(
plus_gros,
4096,
"un argument porte le SQL : c'est ce qui rendait E2BIG",
)
def test_the_sql_goes_through_a_file_not_through_c(self):
vu = self._executer(self._gros_plan(3))
self.assertIn("-f", vu["cmd"])
self.assertNotIn("-c", vu["cmd"])
self.assertIn('UPDATE "m_0"', vu["fichier"])
def test_the_single_transaction_survives_the_change(self):
"""`--single-transaction` n'est documenté qu'avec -c ou -f : passer
par l'entrée standard l'aurait perdu en silence."""
vu = self._executer(self._gros_plan(2))
self.assertIn("-1", vu["cmd"])
self.assertIn("ON_ERROR_STOP=1", vu["cmd"])
def test_the_temporary_file_does_not_survive(self):
import os as vrai_os
vu = self._executer(self._gros_plan(2))
self.assertFalse(vrai_os.path.exists(vu["chemin"]))
class TestABoundedColumnIsNeverOverflowed(unittest.TestCase):
"""`value too long for type character varying(3)`.
Mesuré sur une base réelle : 13 colonnes texte portent une longueur
déclarée, dont des codes à 1, 2 et 3 caractères — `res.country.code`,
`account.journal.code`. Y écrire « jonquille » fait échouer l'UPDATE,
et comme l'écriture est transactionnelle, TOUTE l'anonymisation.
Le mode hybride ne touchait aucune de ces colonnes ; la liste noire,
si. Le mode qui couvre le plus est celui qui cassait.
"""
def _champ(self, **kw):
base = {
"model": "m",
"name": "code",
"ttype": "char",
"pg_type": "character varying",
"unique": False,
"checked": False,
"max_len": None,
}
base.update(kw)
return base
def test_a_bounded_column_is_truncated(self):
sql = anon.expression_texte(self._champ(max_len=3), ["jonquille"])
self.assertIn("left(", sql)
self.assertIn(", 3)", sql)
def test_an_unbounded_column_is_left_alone(self):
sql = anon.expression_texte(self._champ(), ["jonquille"])
self.assertNotIn("left(", sql)
def test_a_bounded_unique_column_keeps_the_id_in_front(self):
"""Tronquer par la droite doit laisser l'identifiant intact :
c'est lui qui porte l'unicité."""
sql = anon.expression_texte(
self._champ(max_len=8, unique=True), ["jonquille"]
)
self.assertIn("left(\"id\"::text || '-'", sql)
def test_an_unbounded_unique_column_keeps_the_old_shape(self):
sql = anon.expression_texte(self._champ(unique=True), ["jonquille"])
self.assertTrue(sql.rstrip().endswith("|| '-' || \"id\"::text END"))
def test_the_length_is_read_from_the_database_not_guessed(self):
"""`atttypmod` est la seule source : une longueur devinée serait
fausse dès qu'un module en change une."""
self.assertIn("atttypmod", anon.REQUETE_CHAMPS)
class TestReservedWordsCannotBreakTheStatement(unittest.TestCase):
"""`syntax error at or near "user"`.
Odoo laisse nommer un champ `user`, `order` ou `group`. Un identifiant
nu fait alors échouer l'analyse syntaxique — et l'écriture étant
transactionnelle, c'est toute l'anonymisation qui tombe. Trouvé en
liste noire sur une base réelle, jamais en mode hybride : les quinze
modèles par défaut n'en portent aucun.
"""
def _champ(self, nom):
return {
"model": "m",
"name": nom,
"ttype": "char",
"pg_type": "character varying",
"unique": False,
"checked": False,
"max_len": None,
}
def test_a_column_named_like_a_keyword_is_quoted(self):
for nom in ("user", "order", "group", "check", "references", "limit"):
sql = anon.sql_pour_table("t", [self._champ(nom)], ["a"])
self.assertIn(f'"{nom}" =', sql, nom)
self.assertNotIn(f" {nom} =", sql, nom)
def test_a_table_named_like_a_keyword_is_quoted(self):
sql = anon.sql_pour_table("order", [self._champ("name")], ["a"])
self.assertTrue(sql.startswith('UPDATE "order" SET'))
def test_the_row_identifier_is_quoted_too(self):
"""Une seule règle vaut mieux que deux : tout identifiant est cité."""
sql = anon.expression_texte(self._champ("name"), ["a", "b"])
self.assertIn('"id"', sql)
def test_a_quote_inside_an_identifier_cannot_escape(self):
self.assertEqual(anon.ident('a"b'), '"a""b"')
class TestACheckDoesNotSilenceTheMainField(unittest.TestCase):
"""La règle « écarter toute colonne sous CHECK » était trop large.
Mesuré : `res_partner.name` porte
CHECK ((type='contact' AND name IS NOT NULL) OR type<>'contact')
— une garantie de non-nullité, qu'un mot satisfait. L'écarter rendait
une anonymisation qui n'anonymisait pas les noms, en annonçant 255
colonnes écrites. Le pire des deux mondes : silencieux et faux.
Sur un NOMBRE la distinction s'inverse : `credit * debit = 0` et
`amount >= 0` bornent la valeur, et un tirage à 1000 les viole.
"""
def test_the_query_tells_numbers_from_text(self):
"""La règle vit dans le SQL : c'est là qu'elle se vérifie."""
requete = anon.REQUETE_CHAMPS
self.assertIn("f.ttype IN ('integer','float','monetary')", requete)
self.assertIn("pg_get_constraintdef", requete)
def test_only_shape_constraints_disqualify_text(self):
for motif in ("char_length", "~~", "jsonb_typeof"):
self.assertIn(motif, anon.REQUETE_CHAMPS, motif)
def _champ_calibre(self, ttype="integer"):
return {
"model": "res.partner",
"name": "montant",
"ttype": ttype,
"pg_type": "numeric",
"unique": False,
"checked": False,
"max_len": None,
"borne_min": 1,
"borne_max": 99999,
}
def test_the_width_rule_reads_each_ROW(self):
"""La décade se lit sur la valeur de la ligne, pas sur la colonne.
Une colonne mêle des largeurs, et c'est la largeur de la VALEUR
que l'option promet de garder.
"""
sql = anon.expression_calibre(self._champ_calibre())
self.assertIn('trunc(abs("montant"::numeric))', sql)
self.assertIn("power(10::numeric", sql)
# `floor(log(...))` travaillait en flottant : log(999999999999999)
# y vaut 15 tout rond, et la décade gagnait un rang.
self.assertNotIn("floor(log(", sql)
def test_the_three_cases_the_rule_does_not_cover(self):
"""NULL, zéro et sous l'unité retombent sur l'étendue mesurée.
0,15 n'a aucun chiffre avant la virgule : appliquer la règle y
tirerait un taux entre 1 et 9, ce que l'étendue existe pour
empêcher.
"""
sql = anon.expression_calibre(self._champ_calibre("float"))
self.assertIn('WHEN "montant" IS NULL THEN NULL', sql)
self.assertIn('WHEN "montant" = 0 THEN "montant"', sql)
self.assertIn('WHEN abs("montant"::numeric) < 1', sql)
def test_the_sign_survives(self):
for ttype in ("integer", "float", "monetary"):
with self.subTest(ttype=ttype):
sql = anon.expression_calibre(self._champ_calibre(ttype))
self.assertIn('sign("montant")', sql)
def test_an_integer_field_gets_a_WHOLE_number(self):
"""Un `float` rendu dans un champ entier ne se réimporte plus.
Ce qui garantit l'entier est `floor`, non la coulée : celle-ci
suit le type qui accueille, et un `integer` d'Odoo peut vivre
dans une colonne `numeric` où `::integer` lèverait.
"""
sql = anon.expression_calibre(self._champ_calibre("integer"))
self.assertIn("floor(", sql)
self.assertNotIn("::numeric, 2)", sql.split("ELSE")[-1])
sql = anon.expression_calibre(self._champ_calibre("float"))
self.assertIn("::numeric, 2)", sql)
def test_the_cast_follows_the_column_when_it_is_a_bounded_int(self):
champ = dict(self._champ_calibre("integer"), pg_type="integer")
self.assertIn("::integer", anon.expression_calibre(champ))
def test_the_option_is_OFF_by_default(self):
"""L'étendue mesurée est ce qui protège une heure ou un taux :
elle reste la règle."""
champs = [self._champ_calibre()]
mots = {"MOTS": ["aa", "bb"]}
sans = anon.plan(champs, mode="blacklist", mots=mots)
avec = anon.plan(champs, mode="blacklist", mots=mots, calibre=True)
self.assertNotIn("power(10", sans[0]["sql"])
self.assertIn("power(10", avec[0]["sql"])
def test_the_dry_run_and_the_write_build_the_SAME_sql(self):
"""`appliquer_sondes` refait le SQL : sans l'option, la marche à
blanc montrerait autre chose que ce que `--apply` écrit."""
champs = [self._champ_calibre()]
mots = {"MOTS": ["aa", "bb"]}
etapes = anon.plan(champs, mode="blacklist", mots=mots, calibre=True)
refait = anon.appliquer_sondes(
etapes, {}, {"res.partner": {"montant": (1, 99)}}, mots, True
)
self.assertIn("power(10", refait[0]["sql"])
def _champ_numerique(self, unique):
return {
"model": "res.partner",
"name": "ref",
"ttype": "integer",
"pg_type": "int4",
"unique": unique,
"checked": False,
"max_len": None,
}
def test_a_unique_number_is_left_alone(self):
"""Aucun tirage ne garantit son unicité.
`expression_texte` colle l'id sur une colonne unique ; un nombre
n'a pas cette issue — y coller l'id changerait sa grandeur. Deux
lignes au même nombre font échouer l'UPDATE, et transaction
unique oblige, TOUTE l'anonymisation avec.
"""
self.assertFalse(anon.champ_retenu(self._champ_numerique(True)))
self.assertTrue(anon.champ_retenu(self._champ_numerique(False)))
def test_a_unique_TEXT_column_is_still_anonymised(self):
"""L'abstention ne vaut que pour les nombres : le texte a l'id
collé, et c'est lui qui porte l'unicité."""
champ = dict(
self._champ_numerique(True),
name="vat",
ttype="char",
pg_type="character varying",
)
self.assertTrue(anon.champ_retenu(champ))
def test_the_report_NAMES_what_it_left_alone(self):
"""Le taire laisserait une colonne identifiante partir sans que
rien ne le dise."""
champs = [
{
"model": "res.partner",
"name": "name",
"ttype": "char",
"pg_type": "character varying",
"unique": False,
"checked": False,
"max_len": None,
},
self._champ_numerique(True),
]
etapes = anon.plan(
champs, mode="blacklist", mots={"MOTS": ["aa", "bb"]}
)
self.assertEqual(etapes[0]["abstenus"], ["ref"])
self.assertNotIn("ref", [c["name"] for c in etapes[0]["fields"]])
self.assertIn("ref", anon.render(etapes))
def test_a_field_the_query_cleared_is_anonymised(self):
"""`checked=False` doit suffire : aucune seconde barrière cachée."""
champ = {
"model": "res.partner",
"name": "name",
"ttype": "char",
"pg_type": "character varying",
"unique": False,
"checked": False,
"max_len": None,
}
self.assertTrue(anon.champ_retenu(champ))
def test_a_field_the_query_flagged_is_left_alone(self):
champ = {
"model": "account.move.line",
"name": "credit",
"ttype": "monetary",
"pg_type": "numeric",
"unique": False,
"checked": True,
"max_len": None,
}
self.assertFalse(anon.champ_retenu(champ))
class TestStructuredCharFieldsSurvive(unittest.TestCase):
"""`ValueError: invalid literal for int() with base 10: 'bruyere'`.
Odoo déclare `parent_path` en `char`, mais y range un CHEMIN
D'IDENTIFIANTS — « 1/7/12/ » — qu'il reparse :
int(id) for id in company.parent_path.split('/')
base/models/res_company.py:117, models.py:203 et :221
Y écrire un mot fait lever le serveur au premier chargement de page.
Mesuré : sept modèles `_parent_store` dans une base ordinaire —
res.company, product.category, stock.location, hr.department,
website.menu, account.analytic.plan, helpdesk.ticket.category.
Même chose pour `days_next_month`, qu'Odoo passe à `int()`
(account/models/account_payment_term.py:321).
"""
def _champ(self, nom):
return {
"model": "res.company",
"name": nom,
"ttype": "char",
"pg_type": "character varying",
"unique": False,
"checked": False,
"max_len": None,
}
def test_the_known_parsers_are_named(self):
self.assertIn("parent_path", anon.CHAMPS_STRUCTURES)
self.assertIn("days_next_month", anon.CHAMPS_STRUCTURES)
def test_they_are_never_touched_whatever_the_model(self):
for nom in anon.CHAMPS_STRUCTURES:
self.assertFalse(anon.champ_retenu(self._champ(nom)), nom)
def test_a_phone_number_is_not_mistaken_for_a_structure(self):
"""Le motif EXIGE les barres obliques. Sans elles, un numéro tout
en chiffres passerait pour une structure et échapperait à
l'anonymisation — un défaut de confidentialité, pas de robustesse.
"""
import re
motif = re.compile(anon.MOTIF_CHEMIN)
for valeur in ("5141234567", "0", "42", "1234-5678"):
self.assertIsNone(motif.match(valeur), valeur)
for valeur in ("1/", "1/7/12/", "3/4/"):
self.assertIsNotNone(motif.match(valeur), valeur)
def test_the_probe_asks_once_per_table_not_once_per_column(self):
"""Sur 410 modèles, la différence est de 410 allers-retours au
lieu de 863."""
appels = []
def espion(database, sql, config_path=None):
appels.append(sql)
return "0:0\x1f0:0"
vrai = anon.lib_analyse.run_psql
anon.lib_analyse.run_psql = espion
etapes = [
{
"model": "res.company",
"fields": [self._champ("name"), self._champ("street")],
"sql": "x",
}
]
try:
[FIX] anonymisation : tirer les nombres dans l'étendue mesurée `resource.calendar.attendance.hour_from` est un `float` qui porte une HEURE DE LA JOURNÉE : un tirage uniforme sur 0 à 1000 le sort de 0..23 et l'affichage de la fiche lève « ValueError: hour must be in 0..23 ». La borne vit dans `time(int(integral), ...)` de resource/models/utils.py, et aucune contrainte PostgreSQL ne la déclare : le schéma ne dit pas le SENS de la donnée. « 0 à 1000 » est donc faux pour tout nombre borné par un usage : heures, pourcentages, taux. On respecte désormais la seule borne que les DONNÉES déclarent, leur propre étendue — min et max mesurés par table, et 0 à 1000 quand elle est inconnue. Vérifié : les tirages tiennent dans l'étendue mesurée, et la fiche s'affiche sans lever. --- EN --- `resource.calendar.attendance.hour_from` is a `float` holding an HOUR OF DAY: a uniform draw over 0 to 1000 takes it out of 0..23 and displaying the record raises "ValueError: hour must be in 0..23". The bound lives in `time(int(integral), ...)` in resource/models/utils.py, and no PostgreSQL constraint declares it: the schema does not say what the data MEANS. So "0 to 1000" is wrong for any number bounded by usage: hours, percentages, rates. We now respect the only bound the DATA declares, its own range — min and max measured per table, and 0 to 1000 when it is unknown. Checked: draws stay inside the measured range, and the record displays without raising. Assisted-by: Claude Opus 5 (cherry picked from commit 8e0a19823714b1d18f05613d498f8136b682c047)
2026-08-25 06:40:11 -04:00
anon.sonder_colonnes("base", etapes)
finally:
anon.lib_analyse.run_psql = vrai
self.assertEqual(len(appels), 1)
self.assertIn('"name"', appels[0])
self.assertIn('"street"', appels[0])
def test_one_counter_example_is_enough_to_keep_a_column(self):
"""Mieux vaut anonymiser une colonne douteuse que taire une
donnée personnelle."""
def espion(database, sql, config_path=None):
# 10 valeurs remplies, 9 seulement sont des chemins.
return "10:9"
vrai = anon.lib_analyse.run_psql
anon.lib_analyse.run_psql = espion
etapes = [
{"model": "m", "fields": [self._champ("chemin")], "sql": "x"}
]
try:
[FIX] anonymisation : tirer les nombres dans l'étendue mesurée `resource.calendar.attendance.hour_from` est un `float` qui porte une HEURE DE LA JOURNÉE : un tirage uniforme sur 0 à 1000 le sort de 0..23 et l'affichage de la fiche lève « ValueError: hour must be in 0..23 ». La borne vit dans `time(int(integral), ...)` de resource/models/utils.py, et aucune contrainte PostgreSQL ne la déclare : le schéma ne dit pas le SENS de la donnée. « 0 à 1000 » est donc faux pour tout nombre borné par un usage : heures, pourcentages, taux. On respecte désormais la seule borne que les DONNÉES déclarent, leur propre étendue — min et max mesurés par table, et 0 à 1000 quand elle est inconnue. Vérifié : les tirages tiennent dans l'étendue mesurée, et la fiche s'affiche sans lever. --- EN --- `resource.calendar.attendance.hour_from` is a `float` holding an HOUR OF DAY: a uniform draw over 0 to 1000 takes it out of 0..23 and displaying the record raises "ValueError: hour must be in 0..23". The bound lives in `time(int(integral), ...)` in resource/models/utils.py, and no PostgreSQL constraint declares it: the schema does not say what the data MEANS. So "0 to 1000" is wrong for any number bounded by usage: hours, percentages, rates. We now respect the only bound the DATA declares, its own range — min and max measured per table, and 0 to 1000 when it is unknown. Checked: draws stay inside the measured range, and the record displays without raising. Assisted-by: Claude Opus 5 (cherry picked from commit 8e0a19823714b1d18f05613d498f8136b682c047)
2026-08-25 06:40:11 -04:00
ecartees, _ = anon.sonder_colonnes("base", etapes)
finally:
anon.lib_analyse.run_psql = vrai
self.assertEqual(ecartees, {})
def test_an_all_paths_column_is_dropped_from_the_plan(self):
def espion(database, sql, config_path=None):
[FIX] anonymisation : tirer les nombres dans l'étendue mesurée `resource.calendar.attendance.hour_from` est un `float` qui porte une HEURE DE LA JOURNÉE : un tirage uniforme sur 0 à 1000 le sort de 0..23 et l'affichage de la fiche lève « ValueError: hour must be in 0..23 ». La borne vit dans `time(int(integral), ...)` de resource/models/utils.py, et aucune contrainte PostgreSQL ne la déclare : le schéma ne dit pas le SENS de la donnée. « 0 à 1000 » est donc faux pour tout nombre borné par un usage : heures, pourcentages, taux. On respecte désormais la seule borne que les DONNÉES déclarent, leur propre étendue — min et max mesurés par table, et 0 à 1000 quand elle est inconnue. Vérifié : les tirages tiennent dans l'étendue mesurée, et la fiche s'affiche sans lever. --- EN --- `resource.calendar.attendance.hour_from` is a `float` holding an HOUR OF DAY: a uniform draw over 0 to 1000 takes it out of 0..23 and displaying the record raises "ValueError: hour must be in 0..23". The bound lives in `time(int(integral), ...)` in resource/models/utils.py, and no PostgreSQL constraint declares it: the schema does not say what the data MEANS. So "0 to 1000" is wrong for any number bounded by usage: hours, percentages, rates. We now respect the only bound the DATA declares, its own range — min and max measured per table, and 0 to 1000 when it is unknown. Checked: draws stay inside the measured range, and the record displays without raising. Assisted-by: Claude Opus 5 (cherry picked from commit 8e0a19823714b1d18f05613d498f8136b682c047)
2026-08-25 06:40:11 -04:00
# Une mesure PAR COLONNE : la sonde compte ce qu'elle
# reçoit et renonce si le compte ne tombe pas juste.
return "10:10\x1f10:10"
vrai = anon.lib_analyse.run_psql
anon.lib_analyse.run_psql = espion
etapes = [
{
"model": "m",
"fields": [self._champ("chemin"), self._champ("nom")],
"sql": "x",
}
]
try:
[FIX] anonymisation : tirer les nombres dans l'étendue mesurée `resource.calendar.attendance.hour_from` est un `float` qui porte une HEURE DE LA JOURNÉE : un tirage uniforme sur 0 à 1000 le sort de 0..23 et l'affichage de la fiche lève « ValueError: hour must be in 0..23 ». La borne vit dans `time(int(integral), ...)` de resource/models/utils.py, et aucune contrainte PostgreSQL ne la déclare : le schéma ne dit pas le SENS de la donnée. « 0 à 1000 » est donc faux pour tout nombre borné par un usage : heures, pourcentages, taux. On respecte désormais la seule borne que les DONNÉES déclarent, leur propre étendue — min et max mesurés par table, et 0 à 1000 quand elle est inconnue. Vérifié : les tirages tiennent dans l'étendue mesurée, et la fiche s'affiche sans lever. --- EN --- `resource.calendar.attendance.hour_from` is a `float` holding an HOUR OF DAY: a uniform draw over 0 to 1000 takes it out of 0..23 and displaying the record raises "ValueError: hour must be in 0..23". The bound lives in `time(int(integral), ...)` in resource/models/utils.py, and no PostgreSQL constraint declares it: the schema does not say what the data MEANS. So "0 to 1000" is wrong for any number bounded by usage: hours, percentages, rates. We now respect the only bound the DATA declares, its own range — min and max measured per table, and 0 to 1000 when it is unknown. Checked: draws stay inside the measured range, and the record displays without raising. Assisted-by: Claude Opus 5 (cherry picked from commit 8e0a19823714b1d18f05613d498f8136b682c047)
2026-08-25 06:40:11 -04:00
ecartees, _ = anon.sonder_colonnes("base", etapes)
finally:
anon.lib_analyse.run_psql = vrai
# Les deux colonnes rendent le même compte ici : c'est le principe
# qu'on vérifie, pas la ligne exacte.
self.assertIn("m", ecartees)
[FIX] anonymisation : tirer les nombres dans l'étendue mesurée `resource.calendar.attendance.hour_from` est un `float` qui porte une HEURE DE LA JOURNÉE : un tirage uniforme sur 0 à 1000 le sort de 0..23 et l'affichage de la fiche lève « ValueError: hour must be in 0..23 ». La borne vit dans `time(int(integral), ...)` de resource/models/utils.py, et aucune contrainte PostgreSQL ne la déclare : le schéma ne dit pas le SENS de la donnée. « 0 à 1000 » est donc faux pour tout nombre borné par un usage : heures, pourcentages, taux. On respecte désormais la seule borne que les DONNÉES déclarent, leur propre étendue — min et max mesurés par table, et 0 à 1000 quand elle est inconnue. Vérifié : les tirages tiennent dans l'étendue mesurée, et la fiche s'affiche sans lever. --- EN --- `resource.calendar.attendance.hour_from` is a `float` holding an HOUR OF DAY: a uniform draw over 0 to 1000 takes it out of 0..23 and displaying the record raises "ValueError: hour must be in 0..23". The bound lives in `time(int(integral), ...)` in resource/models/utils.py, and no PostgreSQL constraint declares it: the schema does not say what the data MEANS. So "0 to 1000" is wrong for any number bounded by usage: hours, percentages, rates. We now respect the only bound the DATA declares, its own range — min and max measured per table, and 0 to 1000 when it is unknown. Checked: draws stay inside the measured range, and the record displays without raising. Assisted-by: Claude Opus 5 (cherry picked from commit 8e0a19823714b1d18f05613d498f8136b682c047)
2026-08-25 06:40:11 -04:00
propre = anon.appliquer_sondes(etapes, {"m": ["chemin"]}, {}, None)
self.assertEqual([c["name"] for c in propre[0]["fields"]], ["nom"])
self.assertNotIn('"chemin"', propre[0]["sql"])
def test_a_model_entirely_dropped_leaves_no_empty_statement(self):
etapes = [
{"model": "m", "fields": [self._champ("chemin")], "sql": "x"}
]
self.assertEqual(
[FIX] anonymisation : tirer les nombres dans l'étendue mesurée `resource.calendar.attendance.hour_from` est un `float` qui porte une HEURE DE LA JOURNÉE : un tirage uniforme sur 0 à 1000 le sort de 0..23 et l'affichage de la fiche lève « ValueError: hour must be in 0..23 ». La borne vit dans `time(int(integral), ...)` de resource/models/utils.py, et aucune contrainte PostgreSQL ne la déclare : le schéma ne dit pas le SENS de la donnée. « 0 à 1000 » est donc faux pour tout nombre borné par un usage : heures, pourcentages, taux. On respecte désormais la seule borne que les DONNÉES déclarent, leur propre étendue — min et max mesurés par table, et 0 à 1000 quand elle est inconnue. Vérifié : les tirages tiennent dans l'étendue mesurée, et la fiche s'affiche sans lever. --- EN --- `resource.calendar.attendance.hour_from` is a `float` holding an HOUR OF DAY: a uniform draw over 0 to 1000 takes it out of 0..23 and displaying the record raises "ValueError: hour must be in 0..23". The bound lives in `time(int(integral), ...)` in resource/models/utils.py, and no PostgreSQL constraint declares it: the schema does not say what the data MEANS. So "0 to 1000" is wrong for any number bounded by usage: hours, percentages, rates. We now respect the only bound the DATA declares, its own range — min and max measured per table, and 0 to 1000 when it is unknown. Checked: draws stay inside the measured range, and the record displays without raising. Assisted-by: Claude Opus 5 (cherry picked from commit 8e0a19823714b1d18f05613d498f8136b682c047)
2026-08-25 06:40:11 -04:00
anon.appliquer_sondes(etapes, {"m": ["chemin"]}, {}, None), []
)
[FIX] anonymisation : tirer les nombres dans l'étendue mesurée `resource.calendar.attendance.hour_from` est un `float` qui porte une HEURE DE LA JOURNÉE : un tirage uniforme sur 0 à 1000 le sort de 0..23 et l'affichage de la fiche lève « ValueError: hour must be in 0..23 ». La borne vit dans `time(int(integral), ...)` de resource/models/utils.py, et aucune contrainte PostgreSQL ne la déclare : le schéma ne dit pas le SENS de la donnée. « 0 à 1000 » est donc faux pour tout nombre borné par un usage : heures, pourcentages, taux. On respecte désormais la seule borne que les DONNÉES déclarent, leur propre étendue — min et max mesurés par table, et 0 à 1000 quand elle est inconnue. Vérifié : les tirages tiennent dans l'étendue mesurée, et la fiche s'affiche sans lever. --- EN --- `resource.calendar.attendance.hour_from` is a `float` holding an HOUR OF DAY: a uniform draw over 0 to 1000 takes it out of 0..23 and displaying the record raises "ValueError: hour must be in 0..23". The bound lives in `time(int(integral), ...)` in resource/models/utils.py, and no PostgreSQL constraint declares it: the schema does not say what the data MEANS. So "0 to 1000" is wrong for any number bounded by usage: hours, percentages, rates. We now respect the only bound the DATA declares, its own range — min and max measured per table, and 0 to 1000 when it is unknown. Checked: draws stay inside the measured range, and the record displays without raising. Assisted-by: Claude Opus 5 (cherry picked from commit 8e0a19823714b1d18f05613d498f8136b682c047)
2026-08-25 06:40:11 -04:00
class TestANumberKeepsItsMeaningfulRange(unittest.TestCase):
"""`ValueError: hour must be in 0..23`.
`resource.calendar.attendance.hour_from` est un `float` qui vaut une
heure de la journée — 8,00 à 13,00 dans la base d'origine. Un tirage
à 957 fait lever Odoo au premier affichage d'un employé :
time(int(integral), ...) resource/models/utils.py:45
Aucune contrainte PostgreSQL ne dit cela : la borne vit dans le code.
La seule que les DONNÉES déclarent est leur propre étendue, et c'est
la seule qu'on puisse respecter sans nommer les champs un par un.
"""
def _champ(self, ttype="float", **kw):
base = {
"model": "resource.calendar.attendance",
"name": "hour_from",
"ttype": ttype,
"pg_type": "numeric",
"unique": False,
"checked": False,
"max_len": None,
}
base.update(kw)
return base
def test_a_measured_range_bounds_the_draw(self):
sql = anon.expression_nombre(
self._champ(borne_min="8.0", borne_max="13.0")
)
self.assertIn("8.0 + random() * (13.0 - 8.0)", sql)
self.assertNotIn("1000", sql)
def test_an_integer_can_reach_its_upper_bound(self):
sql = anon.expression_nombre(
self._champ(ttype="integer", borne_min="0", borne_max="5")
)
self.assertIn("(5 - 0 + 1)", sql)
def test_without_a_range_it_falls_back_on_a_thousand(self):
self.assertIn("1000", anon.expression_nombre(self._champ()))
self.assertIn(
"1001", anon.expression_nombre(self._champ(ttype="integer"))
)
def test_a_half_known_range_is_not_used(self):
"""Une borne sans l'autre ne borne rien."""
for kw in ({"borne_min": "8.0"}, {"borne_max": "13.0"}):
self.assertIn("1000", anon.expression_nombre(self._champ(**kw)))
def test_a_value_that_is_not_a_number_never_reaches_the_sql(self):
"""Ces bornes viennent de la base et retournent dans du SQL."""
self.assertTrue(anon.nombre_valide("8.0"))
self.assertTrue(anon.nombre_valide("-3"))
for mauvais in ("8.0); DROP TABLE x; --", "", None, "huit"):
self.assertFalse(anon.nombre_valide(mauvais), mauvais)
def test_the_probe_reads_the_bounds(self):
recu = {}
def espion(database, sql, config_path=None):
recu["sql"] = sql
return "8.0:13.0"
vrai = anon.lib_analyse.run_psql
anon.lib_analyse.run_psql = espion
etapes = [{"model": "m", "fields": [self._champ()], "sql": "x"}]
try:
_, bornes = anon.sonder_colonnes("base", etapes)
finally:
anon.lib_analyse.run_psql = vrai
self.assertIn("min(", recu["sql"])
self.assertIn("max(", recu["sql"])
self.assertEqual(bornes["m"]["hour_from"], ("8.0", "13.0"))
def test_an_empty_column_yields_no_bound(self):
def espion(database, sql, config_path=None):
return ":"
vrai = anon.lib_analyse.run_psql
anon.lib_analyse.run_psql = espion
etapes = [{"model": "m", "fields": [self._champ()], "sql": "x"}]
try:
_, bornes = anon.sonder_colonnes("base", etapes)
finally:
anon.lib_analyse.run_psql = vrai
self.assertEqual(bornes, {})
def test_the_bounds_reach_the_generated_sql(self):
etapes = [{"model": "m", "fields": [self._champ()], "sql": "x"}]
propre = anon.appliquer_sondes(
etapes, {}, {"m": {"hour_from": ("8.0", "13.0")}}, None
)
self.assertIn("8.0 + random()", propre[0]["sql"])
class TestTheProbeDistrustsWhatItReads(unittest.TestCase):
"""Ce que la sonde reçoit repart dans du SQL : elle le vérifie.
Deux mutations ont survécu au premier tour, et les deux disaient la
même chose : j'éprouvais les fonctions de contrôle isolément sans
vérifier que la sonde s'en sert. Une garde qu'on n'exerce pas ne
garde rien.
"""
def _champ(self, nom="hour_from", ttype="float"):
return {
"model": "m",
"name": nom,
"ttype": ttype,
"pg_type": "numeric",
"unique": False,
"checked": False,
"max_len": None,
}
def _sonder(self, reponse, champs):
def espion(database, sql, config_path=None):
return reponse
vrai = anon.lib_analyse.run_psql
anon.lib_analyse.run_psql = espion
etapes = [{"model": "m", "fields": champs, "sql": "x"}]
try:
return anon.sonder_colonnes("base", etapes)
finally:
anon.lib_analyse.run_psql = vrai
def test_a_bound_that_is_not_a_number_is_refused(self):
"""La base peut rendre autre chose qu'un nombre ; ces valeurs
retournent telles quelles dans un littéral SQL."""
for reponse in ("abc:def", "8.0); DROP TABLE x; --:13", ":13"):
_, bornes = self._sonder(reponse, [self._champ()])
self.assertEqual(bornes, {}, reponse)
def test_a_valid_bound_still_passes(self):
_, bornes = self._sonder("8.0:13.0", [self._champ()])
self.assertEqual(bornes["m"]["hour_from"], ("8.0", "13.0"))
def test_a_short_answer_is_refused_whole(self):
"""Moins de mesures que de colonnes : `zip` tronquerait en silence
et attribuerait la mesure d'une colonne à une autre."""
champs = [self._champ("a"), self._champ("b"), self._champ("c")]
ecartees, bornes = self._sonder("8.0:13.0", champs)
self.assertEqual(bornes, {})
self.assertEqual(ecartees, {})
def test_a_long_answer_is_refused_too(self):
champs = [self._champ("a")]
_, bornes = self._sonder("8.0:13.0\x1f1.0:2.0", champs)
self.assertEqual(bornes, {})
def test_the_exact_count_is_accepted(self):
champs = [self._champ("a"), self._champ("b")]
_, bornes = self._sonder("8.0:13.0\x1f1.0:2.0", champs)
self.assertEqual(len(bornes["m"]), 2)
class TestTheCalibreCannotLeaveTheColumnType(unittest.TestCase):
"""Garder la largeur ne doit pas viser plus haut que la colonne ne tient.
PostgreSQL borne ses entiers par TAILLE. Une valeur à 10 chiffres dans
une colonne `integer` tire dans une bande qui monte à 9 999 999 999,
là où le type s'arrête à 2 147 483 647 : le dépassement lève, et
l'écriture tenant en une transaction unique, il emporte TOUTE
l'anonymisation. `smallint` est plus étroit encore — 32 767 — et une
valeur à 5 chiffres y suffit.
"""
def _champ(self, ttype="integer", pg="integer"):
return {
"model": "res.partner",
"name": "montant",
"ttype": ttype,
"pg_type": pg,
"unique": False,
"checked": False,
"max_len": None,
"borne_min": 1,
"borne_max": 99999,
}
def _tirer(self, valeur, pg="integer", entier=True, hasard=0.5):
"""Un MODÈLE de l'arithmétique du SQL, et non le SQL lui-même.
Il tourne sans base et couvre toute la plage de `random()` d'un
seul balayage, ce qu'une base rend coûteux. Mais il ne prouve que
lui-même : c'est `TestTheEmittedSqlUnderPostgres` qui éprouve la
chaîne réellement émise, et les deux doivent rester d'accord.
"""
decade = 10 ** (len(str(abs(int(valeur)))) - 1)
haut = decade * 10 - 1
coulee = anon.type_de_coulee({"ttype": "integer", "pg_type": pg})
if coulee in anon.PLAFOND_ENTIER:
haut = min(haut, anon.PLAFOND_ENTIER[coulee])
signe = -1 if valeur < 0 else 1
if entier:
return signe * math.floor(decade + hasard * (haut - decade + 1))
return round(signe * (decade + hasard * (haut - decade)), 2)
def test_each_integer_width_has_its_own_ceiling(self):
for pg, plafond in (
("smallint", 32767),
("integer", 2147483647),
("bigint", 9223372036854775807),
):
with self.subTest(pg=pg):
sql = anon.expression_calibre(self._champ(pg=pg))
self.assertIn("least(", sql)
self.assertIn(str(plafond), sql)
def test_the_alias_spelling_is_understood(self):
"""Un dump écrit `int4` là où `regtype` rend `integer`."""
for alias, canon in (
("int2", "smallint"),
("int4", "integer"),
("int8", "bigint"),
):
with self.subTest(alias=alias):
self.assertEqual(canon, anon.type_entier(alias))
def test_a_type_with_no_ceiling_keeps_the_open_band(self):
"""`numeric` et `double precision` n'ont aucun plafond à tenir."""
for pg in ("numeric", "double precision"):
with self.subTest(pg=pg):
self.assertIsNone(anon.type_entier(pg))
sql = anon.expression_calibre(self._champ("float", pg))
self.assertNotIn("least(", sql)
def test_the_cast_follows_the_column_and_is_not_always_integer(self):
"""Couler un tirage bigint en `::integer` lèverait à son tour."""
self.assertIn(
"::bigint", anon.expression_calibre(self._champ(pg="bigint"))
)
self.assertIn(
"::smallint", anon.expression_calibre(self._champ(pg="smallint"))
)
def test_a_ten_digit_integer_never_leaves_int4(self):
for cran in range(1000):
tire = self._tirer(2000000000, "integer", hasard=cran / 1000.0)
self.assertLessEqual(tire, 2147483647, cran)
self.assertGreaterEqual(tire, 10**9, cran)
def test_a_five_digit_smallint_never_leaves_int2(self):
for cran in range(1000):
tire = self._tirer(20000, "smallint", hasard=cran / 1000.0)
self.assertLessEqual(tire, 32767, cran)
self.assertGreaterEqual(tire, 10000, cran)
def test_the_sign_survives_the_ceiling(self):
for cran in range(1000):
tire = self._tirer(-2000000000, "integer", hasard=cran / 1000.0)
self.assertLess(tire, 0, cran)
self.assertGreaterEqual(tire, -2147483647, cran)
def test_the_top_of_the_band_is_reachable(self):
"""Sans le `+ 1`, 8839 ne pouvait jamais sortir 9999."""
atteints = {
self._tirer(8839, "numeric", hasard=c / 10000.0)
for c in range(10000)
}
self.assertEqual(1000, min(atteints))
self.assertEqual(9999, max(atteints))
def test_rounding_never_adds_a_digit(self):
"""`round(…, 2)` au haut de la bande rendrait 10000,0 : un chiffre
de plus, ce que l'option existe pour empêcher."""
for cran in range(10000):
tire = self._tirer(
8839.5, "numeric", entier=False, hasard=cran / 10000.0
)
self.assertLess(tire, 10000, cran)
self.assertGreaterEqual(tire, 1000, cran)
def test_the_measure_casts_before_taking_the_absolute_value(self):
"""`abs()` du plus petit entier signé lève : sa valeur absolue ne
tient pas dans son propre type."""
sql = anon.expression_calibre(self._champ())
self.assertIn('abs("montant"::numeric)', sql)
self.assertEqual(
sql.count('abs("montant"'),
sql.count('abs("montant"::numeric)'),
)
class TestWhyAColumnIsLeftAlone(unittest.TestCase):
"""La raison d'une abstention se LIT, elle ne se redevine pas.
Le rapport ne nomme qu'un motif — l'unicité numérique, la seule dont
le silence laisserait partir une colonne identifiante. Le redeviner
depuis le type et l'unicité nommait `id` sur CHAQUE modèle, `id`
étant un entier unique que le PLANCHER refuse ; et une colonne
`xxx_id`, sous contrainte CHECK ou en jsonb numérique se confondait
de la même façon.
"""
def _f(self, nom, ttype="integer", pg="integer", **kw):
return {
"model": kw.pop("modele", "res.partner"),
"name": nom,
"ttype": ttype,
"pg_type": pg,
"unique": kw.pop("unique", False),
"checked": kw.pop("checked", False),
"max_len": None,
}
def test_the_primary_key_is_refused_by_the_FLOOR(self):
"""`id` est un entier unique : c'est le plancher qui l'écarte."""
self.assertEqual(
anon.REFUS_PLANCHER,
anon.raison_du_refus(self._f("id", unique=True)),
)
def test_each_neighbour_gives_its_OWN_reason(self):
"""Quatre champs satisfont « nombre unique » sans être celui-là."""
for nom, kw, attendu in (
("id", {"unique": True}, anon.REFUS_PLANCHER),
("partner_id", {"unique": True}, anon.REFUS_RELATION),
(
"compteur",
{"unique": True, "checked": True},
anon.REFUS_CONTRAINTE,
),
(
"credit_limit",
{"unique": True, "pg": "jsonb"},
anon.REFUS_JSONB_NOMBRE,
),
):
with self.subTest(nom=nom):
champ = self._f(nom, **kw)
self.assertEqual(attendu, anon.raison_du_refus(champ))
self.assertNotEqual(
anon.REFUS_NOMBRE_UNIQUE, anon.raison_du_refus(champ)
)
def test_the_reason_the_report_names_is_still_reached(self):
self.assertEqual(
anon.REFUS_NOMBRE_UNIQUE,
anon.raison_du_refus(self._f("numero", unique=True)),
)
def test_the_two_questions_never_disagree(self):
"""`champ_retenu` n'est que la même question posée en oui/non."""
for champ in (
self._f("id", unique=True),
self._f("numero", unique=True),
self._f("name", "char", "character varying"),
self._f("login", "char", "character varying"),
self._f("parent_path", "char", "character varying"),
):
with self.subTest(nom=champ["name"]):
self.assertEqual(
anon.raison_du_refus(champ) is None,
anon.champ_retenu(champ),
)
def test_the_login_reason_follows_the_option(self):
champ = self._f("login", "char", "character varying")
self.assertEqual(anon.REFUS_CONNEXION, anon.raison_du_refus(champ))
self.assertIsNone(anon.raison_du_refus(champ, True))
def test_the_plan_does_not_name_the_primary_key(self):
"""Une ligne ⚠ par modèle noyait les vrais avertissements."""
champs = [
self._f("id", unique=True),
self._f("name", "char", "character varying"),
]
etapes = anon.plan(
champs, mode="blacklist", mots={"MOTS": ["aa", "bb"]}
)
self.assertEqual([], etapes[0]["abstenus"])
self.assertNotIn("⚠", anon.render(etapes))
class TestTheWarningSurvivesAnEmptyPlan(unittest.TestCase):
"""Le seul cas pour lequel l'avertissement existe le perdait.
Un modèle dont la seule colonne anonymisable EST la numérique unique
ne produit aucun UPDATE. L'étape était écartée, l'avertissement avec,
et le rapport affirmait « rien à anonymiser » sur la colonne même
qu'il existe pour nommer.
"""
MOTS = {"MOTS": ["aa", "bb"]}
def _f(self, nom, ttype="integer", pg="integer", **kw):
return {
"model": kw.pop("modele", "x.compteur"),
"name": nom,
"ttype": ttype,
"pg_type": pg,
"unique": kw.pop("unique", False),
"checked": False,
"max_len": None,
}
def test_a_model_with_only_that_column_keeps_its_warning(self):
etapes = anon.plan(
[self._f("numero", unique=True)],
mode="blacklist",
mots=self.MOTS,
)
self.assertEqual(1, len(etapes))
self.assertEqual(["numero"], etapes[0]["abstenus"])
def test_such_a_step_carries_NO_sql_and_no_field(self):
"""C'est ce que lisent le code de sortie et l'écriture."""
etapes = anon.plan(
[self._f("numero", unique=True)],
mode="blacklist",
mots=self.MOTS,
)
self.assertIsNone(etapes[0]["sql"])
self.assertEqual([], etapes[0]["fields"])
def test_the_report_NAMES_it_and_still_says_nothing_to_do(self):
etapes = anon.plan(
[self._f("numero", unique=True)],
mode="blacklist",
mots=self.MOTS,
)
rapport = anon.render(etapes)
self.assertIn("numero", rapport)
self.assertIn(anon.t("left in clear, numeric and unique:"), rapport)
self.assertIn(
anon.t("Nothing to anonymise with these lists."), rapport
)
def test_such_a_report_does_not_invite_to_write(self):
"""Il n'y a rien à écrire : proposer --apply serait un piège."""
etapes = anon.plan(
[self._f("numero", unique=True)],
mode="blacklist",
mots=self.MOTS,
)
self.assertNotIn(
anon.t("Use --apply --confirm <database> to write."),
anon.render(etapes),
)
def test_a_real_step_still_invites_to_write(self):
etapes = anon.plan(
[
self._f("numero", unique=True),
self._f("libelle", "char", "character varying"),
],
mode="blacklist",
mots=self.MOTS,
)
rapport = anon.render(etapes)
self.assertIn(
anon.t("Use --apply --confirm <database> to write."), rapport
)
self.assertIn(anon.t("left in clear, numeric and unique:"), rapport)
def test_the_probe_taking_the_last_column_keeps_the_warning(self):
"""Le second endroit où l'avertissement disparaissait."""
etapes = anon.plan(
[
self._f("ref_interne", unique=True),
self._f("chemin", "char", "character varying"),
],
mode="blacklist",
mots=self.MOTS,
)
apres = anon.appliquer_sondes(
etapes, {"x.compteur": ["chemin"]}, {}, self.MOTS
)
self.assertEqual(1, len(apres))
self.assertIsNone(apres[0]["sql"])
self.assertEqual(["ref_interne"], apres[0]["abstenus"])
self.assertIn("ref_interne", anon.render(apres))
def test_a_step_with_neither_sql_nor_warning_is_dropped(self):
"""Sans avertissement à porter, une étape vide n'a rien à dire."""
etapes = anon.plan(
[self._f("chemin", "char", "character varying")],
mode="blacklist",
mots=self.MOTS,
)
apres = anon.appliquer_sondes(
etapes, {"x.compteur": ["chemin"]}, {}, self.MOTS
)
self.assertEqual([], apres)
def _postgres_joignable():
"""PostgreSQL répond-il ? Le lanceur unitaire n'en exige aucun."""
try:
from script.analyse import lib_analyse
lib_analyse.run_psql("postgres", "SELECT 1", timeout=5)
return True
except Exception: # noqa: BLE001 - absent, refusé, injoignable : pareil
return False
PG_JOIGNABLE = _postgres_joignable()
@unittest.skipUnless(PG_JOIGNABLE, "aucun PostgreSQL joignable")
class TestTheEmittedSqlUnderPostgres(unittest.TestCase):
"""Le SQL RÉELLEMENT ÉMIS, évalué par PostgreSQL.
Un test bâti sur une transcription Python prouve la transcription.
Les deux décisions d'arithmétique de `expression_calibre` — le
plafond du type d'ARRIVÉE et le compte exact des chiffres — ne se
vérifient que dans la chaîne émise, sur le moteur qui l'exécutera.
Aucune table, aucune écriture : l'expression est évaluée sur une
liste de VALUES, et `pg_env` impose `default_transaction_read_only`.
Sans base joignable, le test SE DIT ignoré plutôt que de passer au
vert en silence.
"""
LIGNES = 3000
def _champ(self, ttype, pg, bmin, bmax):
return {
"model": "m",
"name": "montant",
"ttype": ttype,
"pg_type": pg,
"unique": False,
"checked": False,
"max_len": None,
"borne_min": bmin,
"borne_max": bmax,
}
def _largeurs_fautives(self, ttype, pg, bmin, bmax, valeurs):
"""Combien de tirages n'ont pas la largeur de leur source.
`random()` reste RÉEL : y substituer une constante laisse
PostgreSQL replier l'expression au plan, et lever alors sur une
branche que le CASE n'atteint jamais à l'exécution.
"""
from script.analyse import lib_analyse
expr = anon.expression_calibre(self._champ(ttype, pg, bmin, bmax))
source = ", ".join("((%s)::%s)" % (v, pg) for v in valeurs)
entiere = "ltrim(split_part(%s::text, '.', 1), '-')"
sql = (
"SELECT count(*) FILTER (WHERE length(%s) <> length(%s))"
' FROM (VALUES %s) t("montant"), generate_series(1, %d)'
% (
entiere % 't."montant"',
entiere % ("(%s)" % expr),
source,
self.LIGNES,
)
)
return int(lib_analyse.run_psql("postgres", sql, timeout=30).strip())
def test_an_odoo_integer_on_a_numeric_column_does_not_raise(self):
"""La bande s'INVERSAIT : son bas dépassait le plafond d'`integer`
dont on bornait le haut, et chaque ligne levait — emportant, en
transaction unique, toute l'anonymisation."""
self.assertEqual(
0,
self._largeurs_fautives(
"integer",
"numeric",
1,
10**12,
[10000000000, 500000000000, 2147483648],
),
)
def test_the_same_on_double_precision(self):
self.assertEqual(
0,
self._largeurs_fautives(
"integer", "double precision", 1, 10**12, [10000000000]
),
)
def test_a_bounded_integer_column_never_overflows(self):
for pg, bmax, valeurs in (
("smallint", 32767, [1, 999, 10000, 32767, -32768]),
(
"integer",
2000000000,
[1, 1000000000, 2147483647, -2147483648],
),
(
"bigint",
10**18,
[10**18, 999999999999999999, 9223372036854775807],
),
):
with self.subTest(pg=pg):
self.assertEqual(
0,
self._largeurs_fautives("integer", pg, 1, bmax, valeurs),
)
def test_the_digit_count_is_exact_past_fifteen(self):
"""`floor(log(999999999999999))` vaut 15 tout rond en flottant :
la décade gagnait un rang, et la copie un chiffre."""
self.assertEqual(
0,
self._largeurs_fautives(
"integer",
"numeric",
1,
10**20,
[
999999999999999,
9999999999999999,
999999999999999999,
10**19 - 1,
],
),
)
def test_the_decimal_branch_keeps_its_width_too(self):
for pg in ("numeric", "double precision"):
with self.subTest(pg=pg):
self.assertEqual(
0,
self._largeurs_fautives(
"float",
pg,
1,
10**9,
[8839.5, 999999999999999.0, -1234.56],
),
)
def test_the_smallest_signed_integer_does_not_break_abs(self):
"""`abs()` du minimum d'un type lève : sa valeur absolue ne tient
pas dans ce type. La mesure passe donc par `numeric`."""
for pg, mini in (
("smallint", -32768),
("integer", -2147483648),
("bigint", -9223372036854775808),
):
with self.subTest(pg=pg):
self.assertEqual(
0,
self._largeurs_fautives(
"integer", pg, 1, abs(mini) - 1, [mini]
),
)
class TestWhereADrawIsPoured(unittest.TestCase):
"""Le type qui ACCUEILLE le tirage n'est pas celui d'Odoo.
Une base montée de version garde la colonne `numeric` qu'un champ
`Float` avait créée, `ir_model_fields` disant désormais `integer` :
Odoo ne réécrit pas le type d'une colonne quand le champ change. Y
couler en `integer` lève dès 2 147 483 648.
"""
def test_a_bounded_integer_column_keeps_its_own_type(self):
for pg, attendu in (
("smallint", "smallint"),
("integer", "integer"),
("bigint", "bigint"),
("int2", "smallint"),
("int4", "integer"),
("int8", "bigint"),
):
with self.subTest(pg=pg):
self.assertEqual(
attendu,
anon.type_de_coulee({"ttype": "integer", "pg_type": pg}),
)
def test_an_integer_field_on_an_unbounded_column_pours_numeric(self):
for pg in ("numeric", "double precision", "real"):
with self.subTest(pg=pg):
self.assertEqual(
"numeric",
anon.type_de_coulee({"ttype": "integer", "pg_type": pg}),
)
def test_a_decimal_field_follows_its_column(self):
"""Un `float` d'Odoo sur une colonne entière garde le plafond de
celle-ci ; ailleurs, aucun plafond ne s'applique."""
self.assertEqual(
"integer",
anon.type_de_coulee({"ttype": "float", "pg_type": "integer"}),
)
self.assertIsNone(
anon.type_de_coulee({"ttype": "float", "pg_type": "numeric"})
)
def test_no_ceiling_is_applied_to_an_unbounded_cast(self):
"""Borner à `integer` le haut d'une bande dont le bas le dépasse
INVERSE la bande, et chaque ligne lève."""
champ = {
"model": "m",
"name": "montant",
"ttype": "integer",
"pg_type": "numeric",
"unique": False,
"checked": False,
"max_len": None,
"borne_min": 1,
"borne_max": 10**12,
}
sql = anon.expression_calibre(champ)
self.assertNotIn("least(", sql)
self.assertNotIn("2147483647", sql)
self.assertIn("::numeric", sql)
def test_the_measured_extent_pours_the_same_way(self):
"""`expression_nombre` portait le même `::integer` en dur."""
champ = {
"model": "m",
"name": "montant",
"ttype": "integer",
"pg_type": "numeric",
"unique": False,
"checked": False,
"max_len": None,
"borne_min": 1,
"borne_max": 10**12,
}
sql = anon.expression_nombre(champ)
self.assertIn("::numeric", sql)
self.assertNotIn("::integer", sql)
class TestTheExitCodeContract(unittest.TestCase):
"""Les codes de sortie sont lus par le menu qui écrit.
Ils doivent se distinguer d'une TRACE PYTHON, qui sort en 1 : le flux
lisait « tout ce qui n'est ni 0 ni 2 » comme du travail annoncé, et
demandait la confirmation destructrice après un plantage.
`--apply` sur un plan vide rendait 0 lui aussi, psql acceptant un
script vide : l'appelant y lisait « écriture faite » et tirait une
sauvegarde de la base intacte en l'annonçant anonymisée.
"""
def setUp(self):
self.champs = []
for nom, remplacant in (
("require_odoo_database", lambda *a, **k: None),
):
self.addCleanup(
setattr, anon.lib_analyse, nom, getattr(anon.lib_analyse, nom)
)
setattr(anon.lib_analyse, nom, remplacant)
for nom, remplacant in (
("inspect", lambda *a, **k: self.champs),
("sonder_colonnes", lambda *a, **k: ({}, {})),
("ecrire", lambda *a, **k: None),
):
self.addCleanup(setattr, anon, nom, getattr(anon, nom))
setattr(anon, nom, remplacant)
sortie = io.StringIO()
vrai = sys.stdout
sys.stdout = sortie
self.sortie = sortie
self.addCleanup(setattr, sys, "stdout", vrai)
def _f(self, nom, ttype="char", pg="character varying", unique=False):
return {
"model": "res.partner",
"name": nom,
"ttype": ttype,
"pg_type": pg,
"unique": unique,
"checked": False,
"max_len": None,
}
def _code(self, champs, applique=False):
self.champs = champs
extra = ["--apply", "--confirm", "b"] if applique else []
return anon.main(["--database", "b"] + extra)
def test_the_codes_do_not_collide_with_a_python_traceback(self):
"""Une exception non rattrapée sort en 1 : aucun code du contrat
ne doit valoir 1."""
codes = (
anon.SORTIE_RIEN,
anon.SORTIE_REFUS,
anon.SORTIE_A_FAIRE,
anon.SORTIE_SANS_EFFET,
)
self.assertNotIn(1, codes)
self.assertEqual(len(set(codes)), len(codes))
def test_an_empty_plan_announces_nothing_to_do(self):
self.assertEqual(anon.SORTIE_RIEN, self._code([]))
def test_real_work_announces_itself_with_its_OWN_code(self):
self.assertEqual(anon.SORTIE_A_FAIRE, self._code([self._f("name")]))
def test_a_warning_only_plan_announces_NO_work(self):
"""Le seul cas pour lequel l'avertissement existe : le compter
pour du travail ferait confirmer une écriture sans objet."""
champs = [self._f("numero", "integer", "integer", unique=True)]
self.assertEqual(anon.SORTIE_RIEN, self._code(champs))
self.assertIn(
anon.t("left in clear, numeric and unique:"),
self.sortie.getvalue(),
)
def test_apply_on_an_empty_plan_is_not_a_write(self):
self.assertEqual(anon.SORTIE_SANS_EFFET, self._code([], True))
def test_apply_on_a_warning_only_plan_is_not_a_write_either(self):
champs = [self._f("numero", "integer", "integer", unique=True)]
self.assertEqual(anon.SORTIE_SANS_EFFET, self._code(champs, True))
def test_apply_that_writes_returns_the_success_code(self):
self.assertEqual(anon.SORTIE_RIEN, self._code([self._f("name")], True))
def test_a_write_error_returns_the_refusal_code(self):
anon.ecrire = lambda *a, **k: "collision d'unicité"
self.assertEqual(
anon.SORTIE_REFUS, self._code([self._f("name")], True)
)
def test_a_confirm_that_does_not_repeat_the_name_refuses(self):
self.champs = [self._f("name")]
self.assertEqual(
anon.SORTIE_REFUS,
anon.main(["--database", "b", "--apply", "--confirm", "autre"]),
)
def test_no_empty_script_ever_reaches_psql(self):
"""`ecrire` recevait un script vide, que psql accepte."""
appels = []
anon.ecrire = lambda *a, **k: appels.append(a) or None
self._code([], True)
self._code(
[self._f("numero", "integer", "integer", unique=True)], True
)
self.assertEqual([], appels)
self._code([self._f("name")], True)
self.assertEqual(1, len(appels))
class TestWhichAbstentionGetsNamed(unittest.TestCase):
"""« Laquelle toucher » et « laquelle nommer » sont deux questions.
L'ordre des contrôles répond à la première : il rend UN motif, le
premier rencontré. Le réutiliser comme prédicat du rapport tait une
colonne numérique unique dès qu'un autre motif la précède — une
contrainte CHECK, un jsonb, un nom en `_id` — alors que c'est
exactement ce que la ligne ⚠ affirme.
Le cas qui compte : un module déclarant `unique(numero)` ET
`check(numero > 0)` sur un numéro de document ou d'employé. La
requête lève `checked` pour TOUTE contrainte sur un nombre, donc le
motif rendu est la contrainte, jamais l'unicité.
"""
MOTS = {"MOTS": ["aa", "bb"]}
def _f(self, nom, ttype="integer", pg="integer", **kw):
return {
"model": "x.y",
"name": nom,
"ttype": ttype,
"pg_type": pg,
"unique": kw.pop("unique", False),
"checked": kw.pop("checked", False),
"max_len": None,
}
def _nommes(self, champ):
"""Ce que le rapport nomme, la colonne étant accompagnée d'un
texte pour qu'une étape existe."""
etapes = anon.plan(
[champ, self._f("libelle", "char", "character varying")],
mode="blacklist",
mots=self.MOTS,
)
return etapes[0]["abstenus"] if etapes else []
def test_the_primary_key_stays_SILENT(self):
"""`id` est un entier unique sur CHAQUE modèle : le nommer
partout noyait les vrais avertissements."""
self.assertEqual([], self._nommes(self._f("id", unique=True)))
def test_every_unique_number_left_in_clear_IS_named(self):
for nom, kw in (
("numero", {"unique": True}),
("numero", {"unique": True, "checked": True}),
("numero", {"unique": True, "pg": "jsonb"}),
("compteur_id", {"unique": True}),
):
with self.subTest(nom=nom, **kw):
self.assertEqual([nom], self._nommes(self._f(nom, **kw)))
def test_the_reason_returned_is_NOT_the_uniqueness_one(self):
"""La preuve que le prédicat ne peut pas s'y adosser."""
for kw in (
{"unique": True, "checked": True},
{"unique": True, "pg": "jsonb"},
):
with self.subTest(**kw):
champ = self._f("numero", **kw)
self.assertNotEqual(
anon.REFUS_NOMBRE_UNIQUE, anon.raison_du_refus(champ)
)
self.assertTrue(
anon.abstention_a_nommer(
champ, anon.raison_du_refus(champ)
)
)
def test_a_unique_TEXT_column_is_not_named(self):
"""Le texte a une issue : `expression_texte` y colle l'id."""
champ = self._f("ref", "char", "character varying", unique=True)
self.assertEqual([], self._nommes(champ))
def test_a_number_that_is_not_unique_is_not_named(self):
self.assertEqual([], self._nommes(self._f("montant")))
def test_a_column_that_is_TAKEN_is_not_named(self):
self.assertFalse(anon.abstention_a_nommer(self._f("montant"), None))
def test_the_floor_is_the_only_reason_that_silences(self):
"""Toute autre raison laisse la question au type et à l'unicité."""
champ = self._f("numero", unique=True)
for raison in (
anon.REFUS_RELATION,
anon.REFUS_CONTRAINTE,
anon.REFUS_JSONB_NOMBRE,
anon.REFUS_NOMBRE_UNIQUE,
anon.REFUS_STRUCTURE,
anon.REFUS_CONNEXION,
):
with self.subTest(raison=raison):
self.assertTrue(anon.abstention_a_nommer(champ, raison))
self.assertFalse(anon.abstention_a_nommer(champ, anon.REFUS_PLANCHER))
if __name__ == "__main__":
unittest.main()