uros-b opened a new pull request, #17528: URL: https://github.com/apache/iceberg/pull/17528
`UnicodeUtil` had no dedicated test class. This adds one covering its public surface: - `isCharHighSurrogate` at both ends of the high-surrogate range (0xD800-0xDBFF), plus a low surrogate and an ASCII char - `truncateString` when no truncation is needed, when truncating by code points, and that a surrogate pair (emoji) is not split - the `IllegalArgumentException` guard for a non-positive truncate length - `truncateStringMin` / `truncateStringMax`, including the overflow case where every retained code point is the maximum and no greater bound exists (returns null) Test-only; documents existing behavior. --- **AI Disclosure** - Model: Claude Opus 4.8 - Platform/Tool: Claude Code - Human Oversight: fully reviewed - Prompt Summary: Add unit tests documenting the existing behavior of a pure utility class. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
