Progress towards max char value

Author: adamtheturtle

CONTRIBUTING.rst

@@ -178,6 +178,8 @@ A response to an invalid query may have an ``application/json`` content type but
 
 After deleting a target, for up to approximately 30 seconds, matching it with a query returns a 500 response.
 
+A target with the name ``\uffff`` gets stuck in processing.
+
 Performing a release
 --------------------
 

tests/mock_vws/test_add_target.py

@@ -203,13 +203,16 @@ class TestTargetName:
     Tests for the target name field.
     """
 
+    _MAX_CHAR_VALUE = 65535
+
     @pytest.mark.parametrize(
         'name',
         [
             'á',
+            chr(_MAX_CHAR_VALUE),
             'a' * 64,
         ],
-        ids=['Short name', 'Long name'],
+        ids=['Short name', 'Max char value', 'Long name'],
     )
     def test_name_valid(
         self,
@@ -249,9 +252,10 @@ def test_name_invalid(
         vuforia_database_keys: VuforiaDatabaseKeys,
     ) -> None:
         """
-        A target's name must be a UTF-8 encoded string of length 0 < N < 65.
+        A target's name must be a string of length 0 < N < 65.
 
-        We test bad encoding in another test as it has a different error.
+        We test characters out of range in another test as that gives a
+        different error.
         """
         image_data = png_rgb.read()
         image_data_encoded = base64.b64encode(image_data).decode('ascii')
@@ -273,13 +277,12 @@ def test_name_invalid(
             result_code=ResultCodes.FAIL,
         )
 
-    # TODO also on update
-    def test_not_utf8_encoded(
+    def test_character_out_of_range(
         self,
         png_rgb: io.BytesIO,
         vuforia_database_keys: VuforiaDatabaseKeys,
     ) -> None:
-        name = '\U0001f604'
+        name = chr(self._MAX_CHAR_VALUE + 1)
         image_data = png_rgb.read()
         image_data_encoded = base64.b64encode(image_data).decode('ascii')