(PC-29597)[API] fix: use column name for error key

pass-culture · Oct 21, 2024 · 0486699 · 0486699
1 parent 4cc5344
commit 0486699
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/api/src/pcapi/validation/models/generic.py b/api/src/pcapi/validation/models/generic.py
@@ -28,7 +28,7 @@ def validate_generic(model: Model) -> ApiErrors:
 
         column = columns[key]
         value = getattr(model, key)
-
+        column_name = column.expression.name
         if (  # pylint: disable=too-many-boolean-expressions
             not getattr(column.expression, "nullable", False)
             and not column.expression.foreign_keys
@@ -37,7 +37,7 @@ def validate_generic(model: Model) -> ApiErrors:
             and getattr(column.expression, "server_default", ...) is None
             and value is None
         ):
-            api_errors.add_error(key, "Cette information est obligatoire")
+            api_errors.add_error(column_name, "Cette information est obligatoire")
 
         if value is None:
             continue
@@ -47,20 +47,22 @@ def validate_generic(model: Model) -> ApiErrors:
             and not isinstance(column.expression.type, sqlalchemy.Enum)
             and not isinstance(value, str)
         ):
-            api_errors.add_error(key, "doit être une chaîne de caractères")
+            api_errors.add_error(column_name, "doit être une chaîne de caractères")
 
         if (
             isinstance(column.expression.type, (String, CHAR))
             and isinstance(value, str)
             and column.expression.type.length
             and len(value) > column.expression.type.length
         ):
-            api_errors.add_error(key, f"Vous devez saisir moins de {str(column.expression.type.length)} caractères")
+            api_errors.add_error(
+                column_name, f"Vous devez saisir moins de {str(column.expression.type.length)} caractères"
+            )
 
         if isinstance(column.expression.type, Integer) and not isinstance(value, int):
-            api_errors.add_error(key, "doit être un entier")
+            api_errors.add_error(column_name, "doit être un entier")
 
         if isinstance(column.expression.type, Float) and not isinstance(value, float):
-            api_errors.add_error(key, "doit être un nombre")
+            api_errors.add_error(column_name, "doit être un nombre")
 
     return api_errors
diff --git a/api/tests/validation/models/generic_test.py b/api/tests/validation/models/generic_test.py
@@ -17,7 +17,7 @@ def test_should_return_error_when_information_requires_a_string_type():
 def test_should_return_error_when_information_requires_an_integer_type():
     offer = offers_factories.OfferFactory.build(durationMinutes="not a number")
     api_error = validate_generic(offer)
-    assert api_error.errors == {"_durationMinutes": ["doit être un entier"]}
+    assert api_error.errors == {"durationMinutes": ["doit être un entier"]}
 
 
 def test_should_return_error_when_information_exceeds_column_size():