Dhairya-007 commented on PR #29696: URL: https://github.com/apache/camel/pull/29696#issuecomment-6825043473
Sorry — I broke the build with my own review follow-up. The AssertJ switch replaced the `assertEquals` **import** but left the two calls, so `UtilsTest` did not compile, and the two raw-byte cases I described in the comment above never actually made it into the file. `UtilsTest` is rewritten: `assertThat` throughout, both new cases present (the `byte[]` overload agreeing with the `String` one for valid UTF-8, and differing for a body that is not), and the non-ASCII literals as `\u00eb`/`\u00e3` so the source stays ASCII. The main-source changes — the `byte[]` overload, hashing `readAllBytes()` directly, and `MessageDigest.isEqual` — were all in the previous push and are untouched. -- 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]
