[FIX] conventions : retirer un nom d'organisation des exemples
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
This commit is contained in:
parent
7c83dd1fc8
commit
85ac83cc75
5 changed files with 6 additions and 6 deletions
|
|
@ -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
|
||||
|
||||
|
|
|
|||
|
|
@ -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):
|
||||
|
|
|
|||
|
|
@ -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])
|
||||
|
|
|
|||
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -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"))
|
||||
|
|
|
|||
Loading…
Reference in a new issue