infinispan / infinispan/protostream

Improve unit-test coverage for ProtobufUtil

Open
#720 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
56
Forks
39
Avg merge
5h 36m
Merged PRs (30d)
5

Description

`ProtobufUtil` has several public entry points whose edge-case behaviour is not pinned by the current suite. Add focused, additive unit tests (no production changes) for the cases a maintainer confirmed are worth covering:

- `testWriteToNullObject` — writing a null object
- `testFromEnumClass` — reading an enum type
- `testEstimateSize` — size estimation
- `testToByteBuffer` / `testFromByteBuffer` — ByteBuffer round-trip

Tracked for PR #710.

Contributor guide

Open the contributing guide

Research direction

Locate the existing unit tests for ProtobufUtil and review how its public entry points are exercised. Add focused tests for testWriteToNullObject, testFromEnumClass, testEstimateSize, and the testToByteBuffer/testFromByteBuffer round-trip, then run the relevant unit-test suite to confirm the coverage additions pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
76/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.