atharvalade opened a new pull request, #3037: URL: https://github.com/apache/iggy/pull/3037
## Which issue does this PR close? Closes #2978 ## Rationale Go SDK Codecov coverage was only 36.37%. This PR adds comprehensive unit tests across all major Go SDK packages and adjusts codecov.yml to exclude non-library code (samples, benchmarks) from the coverage denominator. ## What changed? Most Go SDK packages (contracts, internal/command, binary_serialization, internal/util) had minimal or no unit tests for their serialization, deserialization, and constructor logic. The coverage profile was also diluted by untestable sample/benchmark code counting against the denominator. Added 20+ new test files with ~130 test functions covering all command MarshalBinary types, all contract marshal/unmarshal roundtrips, all binary response deserializers, and leader redirection decision paths. Excluded `foreign/go/samples/**` and `foreign/go/benchmarks/**` from Codecov ignore list. Also fixed two buffer allocation bugs in `UpdateUser` and `UpdatePermissions` MarshalBinary discovered by the new tests. ## Local Execution - Passed - Pre-commit hooks ran ## AI Usage 1. Opus 4.6 2. Minimal AI used 3. Verified by running full `go test -race` suite locally with coverage profiling, plus gofmt, trailing whitespace/newline, and license header checks 4. Yes, all code can be explained -- 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]
