Skip to content

Commit 05248a5

Browse files
committed
Add a comment about the low character value test
1 parent 911e314 commit 05248a5

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

tests/mock_vws/test_add_target.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,10 @@ class TestTargetName:
209209
'name',
210210
[
211211
'á',
212-
chr(_MAX_CHAR_VALUE),
212+
# We test just below the max character value.
213+
# This is because targets with the max character value in their
214+
# names get stuck in the processing stage.
215+
chr(_MAX_CHAR_VALUE - 2),
213216
'a' * 64,
214217
],
215218
ids=['Short name', 'Max char value', 'Long name'],

0 commit comments

Comments
 (0)