Add unit tests for ProtobufUtil

[vasiliy-mikhailov]

Additive unit tests for ProtobufUtil — edge cases and current behavior pinned with explicit assertions. No existing test or production code changed (append-only).

Verified green under Java 17 (mvn -pl core test -Dtest=ProtobufUtilTest → 59 tests, 0 failures). On this class the additions raise line coverage from 42 to 65/69 lines and the PIT mutation kill-rate from 65 to 109/121 mutants.

---

How this was produced

This PR was generated with an AI-assisted pipeline built around mutation testing (PIT). The pipeline mutates the target class (flipping conditions and changing boundary/edge cases) and runs the existing tests against each mutant. Where a mutant survives (the existing tests do not catch that edge case), it writes a focused test for that case and reruns PIT to confirm the new test actually kills that specific mutant. So every added test is verified to catch a concrete edge case the suite missed before, rather than being speculative or redundant. The change is additive only (no production code modified), and the module builds green under its CI JDK.

[tristantarrant]

Thanks for the contribution !

Most of the calls are invoking thin wrapper overloads, so I don't think they are worth the tests.
Please keep the tests for the following:

• testWriteToNullObject
• testFromEnumClass
• testEstimateSize
• testToByteBuffer / testFromByteBuffer

[tristantarrant]

Also, can you create an issue and format the commit message according to the commit rules ?

[vasiliy-mikhailov]

Thanks @tristantarrant! Addressed both points:

• Trimmed the PR down to just the five tests you flagged (testWriteToNullObject, testFromEnumClass, testEstimateSize, testToByteBuffer / testFromByteBuffer) and dropped the thin-wrapper-overload tests.
• Created issue Improve unit-test coverage for ProtobufUtil #720 and reformatted the commit to start with [#720] per the commit rules.

Ready for another look whenever you have a moment.

[tristantarrant]

Sorry, you need to adjust to the new Jupiter assertion method argument order (the message goes in last position now)

[vasiliy-mikhailov]

Thanks @tristantarrant! Addressed the Jupiter feedback:

• Rebased onto current main (which migrated the suite to JUnit 5 Jupiter) and reordered the assertion in testEstimateSize so the message is in last position, per the new Jupiter argument order.
• The PR is now exactly the five tests you asked to keep: testWriteToNullObject, testFromEnumClass, testEstimateSize, testToByteBuffer, testFromByteBuffer. The diff is just those 50 lines.
• Linked to issue Improve unit-test coverage for ProtobufUtil #720, and the commit uses the [#720] format.

Verified locally under JDK 17: ProtobufUtilTest compiles and all of its tests pass. CI is re-running for 17/21/25-ea.

🤖 Addressed by Claude Code

[tristantarrant]

Looks good now, thanks

[tristantarrant]

Merged, thanks!