RavenDB-26793 Skip all C0 control characters in custom-entity-name test

poissoncorp · poissoncorp · commit af94dd87b115 · 2026-07-08T23:57:35.000+02:00
RavenDB now rejects control characters in document ids (including the generated
HiLo id). The test injected control chars 1..13 into the collection name via a
guard that only skipped 14..31, so it failed against current 6.2/7.2 servers.
Skip the whole C0 control range (ord &lt;= 31).
diff --git a/ravendb/tests/jvm_migrated_tests/client_tests/test_custom_entity_name.py b/ravendb/tests/jvm_migrated_tests/client_tests/test_custom_entity_name.py
@@ -100,7 +100,9 @@ def _customize_store(self, store: DocumentStore) -> None:
         )
 
     def __test_when_collection_and_id_contain_special_chars(self, c: str) -> None:
-        if 14 <= ord(c) <= 31:
+        # RavenDB now rejects control characters in document ids (including the generated HiLo id),
+        # so skip the whole C0 control range rather than only 14..31.
+        if ord(c) <= 31:
             return
 
         self.c = c

Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,9 @@ def _customize_store(self, store: DocumentStore) -> None:`
`100`	`100`	`)`
`101`	`101`
`102`	`102`	`def __test_when_collection_and_id_contain_special_chars(self, c: str) -> None:`
`103`		`- if 14 <= ord(c) <= 31:`
	`103`	`+ # RavenDB now rejects control characters in document ids (including the generated HiLo id),`
	`104`	`+ # so skip the whole C0 control range rather than only 14..31.`
	`105`	`+ if ord(c) <= 31:`
`104`	`106`	`return`
`105`	`107`
`106`	`108`	`self.c = c`