Improved the error message displayed when there are not allowed chara…

…cters
XLSForm · Oct 28, 2023 · bd6e285 · bd6e285
1 parent 89ce8b4
commit bd6e285
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pyxform/validators/pyxform/android_package_name.py b/pyxform/validators/pyxform/android_package_name.py
@@ -27,6 +27,6 @@ def validate_android_package_name(name: str) -> Optional[str]:
     pattern = re.compile(r"[^a-zA-Z0-9._]")
     for segment in segments:
         if pattern.search(segment):
-            return f"{prefix}the package name contains not allowed characters."
+            return f"{prefix}the package name contains characters that are not allowed in segment: {segment}. Package names can only include letters (a-z, A-Z), numbers (0-9), dots (.), and underscores (_)."
 
     return None
diff --git a/tests/validators/pyxform/test_android_package_name.py b/tests/validators/pyxform/test_android_package_name.py
@@ -34,7 +34,7 @@ def test_invalid_start_with_digit(self):
 
     def test_invalid_character(self):
         result = validate_android_package_name("com.example.app$")
-        expected_error = "Parameter 'app' has an invalid Android package name - the package name contains not allowed characters."
+        expected_error = "Parameter 'app' has an invalid Android package name - the package name contains characters that are not allowed in segment: app$. Package names can only include letters (a-z, A-Z), numbers (0-9), dots (.), and underscores (_)."
         self.assertEqual(result, expected_error)
 
     def test_package_name_segment_with_zero_length(self):