[DTP-Worker 20260812_080104] fix correctness · commissions vendeurs — la formule traçable base×taux cassait sur montants réels
Bug latent réel (crm/commissions/commlib/finance.py:75) : le libellé de base de
la formule traçable « commission = base × taux » utilisait f"{base:g}", qui casse
deux fois sur des montants immobiliers réels en RD (une unité USD 300k ≈ 18M DOP) :
(1) notation exponentielle dès 1e6 (18000000 → 1.8e+07, illisible/non auditable) ;
(2) arrondi silencieux à 6 chiffres significatifs (123456.78 → 123457) = une base
FABRIQUÉE ≠ de la réelle, l'invention interdite par #6 dans le module même qui
proclame l'anti-invention.
Fix : helper _amount_label (f"{x:f}" jamais exponentiel + strip zéros) → décimal
fidèle, identique à l'ancien pour tous les montants simples (200000 reste 200000) ;
seuls les cas buggés changent. 2 tests à dents (millions non-exponentiel + décimales
préservées) — teeth prouvé : les DEUX échouent sans le fix.
Byte-repro : 0 impact d'artefact du module (compute_line est runtime, jamais appelé
par le générateur). Cascade compteur-de-suite seule : matrice 631/614 → 633/616,
regression×3 + quality_report régénérés, fiches qa/erpnext/crm + README réalignés
(commissions 25→27, Total CRM 81→83). run_ci 33/0/0. 0 code moteur V18 (bloqué D-06).
0 gate ajouté (#5). 0 commande VPS (#8).
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Claude Code DTP Worker
@@ -58,7 +58,7 @@ duplication · workflow #5) :
|
||||
```bash
|
||||
python3 commissions_gen.py build # écrit out/ (refuse si invalide)
|
||||
python3 commissions_gen.py validate # schéma + 10 invariants, sans écrire
|
||||
python3 -m unittest discover -s tests -v # 25 tests (stdlib pur)
|
||||
python3 -m unittest discover -s tests -v # 27 tests (stdlib pur)
|
||||
```
|
||||
|
||||
## Les 10 invariants (le CLI refuse d'écrire si l'un casse)
|
||||
@@ -89,5 +89,5 @@ sert **uniquement aux tests** : ses chiffres sont des exemples fictifs portant u
|
||||
|
||||
**Auto-score 4Big : 96/100.** Réserve −4 : confirmation des taux réels + câblage
|
||||
du calcul en production côté VPS (agent ERPNext Backend · #8) ; ce module valide
|
||||
statiquement en-repo (25 tests verts + schéma + 10 invariants de cross-cohérence
|
||||
statiquement en-repo (27 tests verts + schéma + 10 invariants de cross-cohérence
|
||||
+ gate CI).
|
||||
|
||||
@@ -51,6 +51,22 @@ def rate(value: Any) -> Optional[float]:
|
||||
return None
|
||||
|
||||
|
||||
def _amount_label(x: float) -> str:
|
||||
"""Montant affiché dans la formule : décimal FIDÈLE.
|
||||
|
||||
`:g` (l'ancien encodeur) cassait deux fois sur des montants réels : (1) il bascule
|
||||
en notation exponentielle dès 1e6 (« 18000000 » → « 1.8e+07 » — illisible dans une
|
||||
formule censée être auditable) et (2) il arrondit à 6 chiffres significatifs
|
||||
(« 123456.78 » → « 123457 ») ce qui FABRIQUE une base différente de la réelle —
|
||||
exactement l'invention interdite par CLAUDE.md #6. `:f` (jamais exponentiel) puis
|
||||
strip des zéros/point superflus donne un décimal fidèle et lisible, identique à
|
||||
l'ancien pour les montants simples (« 200000 » reste « 200000 »)."""
|
||||
s = f"{x:f}" # décimal complet, jamais de notation exponentielle
|
||||
if "." in s:
|
||||
s = s.rstrip("0").rstrip(".")
|
||||
return s
|
||||
|
||||
|
||||
def _rate_label(value: Any) -> str:
|
||||
"""Libellé du taux tel qu'affiché dans la formule (verbatim si texte)."""
|
||||
if not is_filled(value):
|
||||
@@ -72,7 +88,7 @@ def compute_line(dossier: dict, event: dict) -> dict:
|
||||
devise = dossier.get("devise")
|
||||
|
||||
montant = base * taux if (base is not None and taux is not None) else None
|
||||
base_lbl = f"{base:g}" if base is not None else f"{{{base_field}}}"
|
||||
base_lbl = _amount_label(base) if base is not None else f"{{{base_field}}}"
|
||||
formule = f"{base_lbl} × {_rate_label(event.get('taux_pct'))}"
|
||||
|
||||
manquants: list[str] = []
|
||||
|
||||
@@ -214,6 +214,24 @@ class TraceableCalc(unittest.TestCase):
|
||||
self.assertIn("2.5", line["formule"])
|
||||
self.assertEqual(line["devise"], "USD")
|
||||
|
||||
def test_formule_base_millions_pas_exponentiel(self):
|
||||
# Régression : `:g` cassait les bases réelles ≥ 1e6 en notation exponentielle
|
||||
# (« 18000000 » → « 1.8e+07 ») — illisible et non auditable dans la formule.
|
||||
ev = {"update_value": "contrat", "role_id": "ventes-conseiller",
|
||||
"base_field": "montant_contrat", "taux_pct": self.taux}
|
||||
line = finance.compute_line({"montant_contrat": 18000000.0, "devise": "DOP"}, ev)
|
||||
self.assertIn("18000000", line["formule"])
|
||||
self.assertNotIn("e+", line["formule"])
|
||||
self.assertNotIn("E+", line["formule"])
|
||||
|
||||
def test_formule_base_preserve_les_decimales(self):
|
||||
# Régression : `:g` arrondissait à 6 chiffres significatifs (« 123456.78 »
|
||||
# → « 123457 »), fabriquant une base ≠ de la réelle (anti-invention #6).
|
||||
ev = {"update_value": "contrat", "role_id": "ventes-conseiller",
|
||||
"base_field": "montant_contrat", "taux_pct": self.taux}
|
||||
line = finance.compute_line({"montant_contrat": 123456.78, "devise": "USD"}, ev)
|
||||
self.assertIn("123456.78", line["formule"])
|
||||
|
||||
def test_none_si_taux_absent(self):
|
||||
ev = {"update_value": "contrat", "role_id": "ventes-conseiller",
|
||||
"base_field": "montant_contrat", "taux_pct": None}
|
||||
|
||||
@@ -366,7 +366,7 @@
|
||||
"criterion": "TESTS",
|
||||
"weight": 25,
|
||||
"passed": true,
|
||||
"evidence": "25 méthodes test_* dans 1 fichier(s) (seuil 8)"
|
||||
"evidence": "27 méthodes test_* dans 1 fichier(s) (seuil 8)"
|
||||
},
|
||||
{
|
||||
"criterion": "CLI",
|
||||
|
||||
@@ -5,7 +5,7 @@
|
||||
"version": "1.0",
|
||||
"verdict": "PASS",
|
||||
"suites": 24,
|
||||
"test_methods": 631,
|
||||
"test_methods": 633,
|
||||
"coverage_ok": true,
|
||||
"artifacts": [
|
||||
"regression_plan.json"
|
||||
|
||||
@@ -35,7 +35,7 @@
|
||||
"in_gate": true,
|
||||
"has_tests_dir": true,
|
||||
"test_files": 1,
|
||||
"test_methods": 25
|
||||
"test_methods": 27
|
||||
},
|
||||
{
|
||||
"id": "crm.dossier_vente",
|
||||
@@ -295,7 +295,7 @@
|
||||
"totals": {
|
||||
"suites": 24,
|
||||
"test_files": 24,
|
||||
"test_methods": 631,
|
||||
"test_methods": 633,
|
||||
"min_methods": 10,
|
||||
"under_threshold": 0
|
||||
},
|
||||
|
||||
@@ -6,9 +6,9 @@
|
||||
{
|
||||
"id": "crm.commissions",
|
||||
"path": "crm/commissions",
|
||||
"expected_methods": 25,
|
||||
"ran": 25,
|
||||
"passed": 24,
|
||||
"expected_methods": 27,
|
||||
"ran": 27,
|
||||
"passed": 26,
|
||||
"failures": 0,
|
||||
"errors": 0,
|
||||
"skipped": 1,
|
||||
@@ -296,8 +296,8 @@
|
||||
"suites": 24,
|
||||
"green": 24,
|
||||
"red": 0,
|
||||
"ran": 631,
|
||||
"passed": 614,
|
||||
"ran": 633,
|
||||
"passed": 616,
|
||||
"failures": 0,
|
||||
"errors": 0,
|
||||
"skipped": 17
|
||||
|
||||
Reference in New Issue
Block a user