From 85ac83cc754b67193272ec34468a4b1dffe4a370 Mon Sep 17 00:00:00 2001 From: Mathieu Benoit Date: Wed, 9 Sep 2026 04:34:46 -0400 Subject: [PATCH] [FIX] conventions : retirer un nom d'organisation des exemples MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Un nom d'organisation servait d'exemple dans une docstring et dans quatre tests — nom de base, chemin de compte, préfixe de copie. La règle du dépôt l'interdit partout hors de `private/`, et ces fichiers suivent le dépôt en amont : sur un fork rendu public, l'exemple le devient aussi. Un test le fige pour toujours, et trois des cinq sites servaient justement à démontrer qu'un chemin de compte ou un nom est refusé. Une valeur inventée démontre aussi bien, et celle-ci ne paraît nulle part ailleurs. La docstring perd son exemple : il n'illustrait que « long », et la phrase le dit maintenant. Le détecteur du dépôt ne voit pas cette classe — il n'annonçait rien ici. --- EN --- An organisation's name served as an example in one docstring and in four tests — a database name, an account path, a copy prefix. The repository's rule forbids it anywhere outside `private/`, and these files follow the repository upstream: on a fork made public, the example becomes public too. A test freezes it forever, and three of the five sites existed precisely to demonstrate that an account path or a name is refused. An invented value demonstrates just as well, and this one appears nowhere else. The docstring loses its example: it only illustrated "long", which the sentence now says. The repository's detector does not see this class — it reported nothing here. Assisted-by: Claude Opus 5 --- script/todo/todo.py | 4 ++-- test/test_db_duplicate.py | 2 +- test/test_git_commit_msg.py | 2 +- test/test_monitoring.py | 2 +- test/test_prompt_defaults.py | 2 +- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/script/todo/todo.py b/script/todo/todo.py index 8e45978..efdefbf 100755 --- a/script/todo/todo.py +++ b/script/todo/todo.py @@ -4425,8 +4425,8 @@ class TODO( de modèles et de colonnes, et lesquels sont traduits ou uniques. La confirmation redemande le NOM de la base. Une frappe sur « o » - se donne par réflexe ; recopier « sireine_neutralize_upgrade_18 » - oblige à regarder ce qu'on détruit. + se tape par réflexe ; recopier un nom long oblige à regarder ce + qu'on détruit. """ from script.analyse import monitoring diff --git a/test/test_db_duplicate.py b/test/test_db_duplicate.py index ee32a61..6608c5a 100644 --- a/test/test_db_duplicate.py +++ b/test/test_db_duplicate.py @@ -26,7 +26,7 @@ class TestWhatANameMayBe(unittest.TestCase): """Le nom entre dans du SQL par `database_identifier` : il se filtre.""" def test_ordinary_names_pass(self): - for nom in ("sireine", "el_essai", "a", "base-2024", "_interne"): + for nom in ("garance", "el_essai", "a", "base-2024", "_interne"): self.assertTrue(dup.nom_valide(nom), nom) def test_a_name_that_could_carry_sql_is_refused(self): diff --git a/test/test_git_commit_msg.py b/test/test_git_commit_msg.py index 3bc9036..af5743e 100644 --- a/test/test_git_commit_msg.py +++ b/test/test_git_commit_msg.py @@ -214,7 +214,7 @@ class TestLeCorps(unittest.TestCase): def test_un_chemin_de_compte(self): problemes = check( - _message("Le venv vit dans /home/sireine/git/erplibre/.") + _message("Le venv vit dans /home/garance/git/erplibre/.") ) self.assertEqual(1, len(problemes)) self.assertIn("chemin de compte", problemes[0]) diff --git a/test/test_monitoring.py b/test/test_monitoring.py index 9b858aa..9e0181e 100644 --- a/test/test_monitoring.py +++ b/test/test_monitoring.py @@ -667,7 +667,7 @@ class TestTheVerdictsSection(unittest.TestCase): ) def test_a_plain_name_is_its_own_lineage(self): - self.assertEqual("copy_sireine3", residue.famille("copy_sireine3")) + self.assertEqual("copy_garance3", residue.famille("copy_garance3")) def test_another_migration_verdicts_are_not_shown(self): # Deux migrations partagent le fichier. Attribuer l'échec de diff --git a/test/test_prompt_defaults.py b/test/test_prompt_defaults.py index 14e96a8..88cfc97 100644 --- a/test/test_prompt_defaults.py +++ b/test/test_prompt_defaults.py @@ -257,7 +257,7 @@ class TestTheDatabaseNameTheFileSuggests(unittest.TestCase): return database_name_from_file(chemin, **kw) def test_the_zip_extension_goes_away(self): - self.assertEqual("sireine3", self.nom("image_db/sireine3.zip")) + self.assertEqual("garance3", self.nom("image_db/garance3.zip")) def test_an_uppercase_extension_goes_away_too(self): self.assertEqual("client", self.nom("image_db/CLIENT.ZIP"))