Addressed the comment

cfpb · Mar 15, 2024 · f42a273 · f42a273
1 parent 231a10c
commit f42a273
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/entities/models/dto.py b/src/entities/models/dto.py
@@ -90,7 +90,8 @@ def validate_fi(self) -> "FinancialInstitutionDto":
             match = re.match(r"^([a-zA-Z0-9]{20})", self.lei)
             if not match:
                 raise ValueError(
-                    f"Invalid lei {self.lei}. FinancialInstitution lei can only contain letters and numbers."
+                    f"Invalid lei {self.lei}. FinancialInstitution lei must be 20 characaters long and contain only "
+                    "letters and numbers."
                 )
         return self
 

diff --git a/tests/api/routers/test_institutions_api.py b/tests/api/routers/test_institutions_api.py
@@ -97,7 +97,8 @@ def test_invalid_lei(self, mocker: MockerFixture, app_fixture: FastAPI, authed_u
         )
         assert (
             res.json()["detail"][0]["msg"]
-            == "Value error, Invalid lei test_Lei. FinancialInstitution lei can only contain letters and numbers."
+            == "Value error, Invalid lei test_Lei. FinancialInstitution lei must be 20 characaters long and contain "
+            "only letters and numbers."
         )
         assert res.status_code == 422