yashisrani opened a new pull request, #3627:
URL: https://github.com/apache/dubbo-go/pull/3627
### Description
Adds comprehensive unit tests for `hessian2Codec` and `msgpackCodec` in the
Triple protocol codec layer, addressing the TODO at `codec.go:377`.
**hessian2Codec**: 17 test cases covering round-trips for primitives
(string, int32, int64, bool, []byte), collections (map[any]any, []string), nil
handling, and error cases (non-pointer target, nil target, nil pointer target).
**msgpackCodec**: Expanded from 1 basic test to 6 cases covering proto
message round-trips, name verification, nil handling, invalid data errors, and
empty payload edge cases.
Fixes #3626
## Special notes for your reviewer:
- Tests follow existing patterns from `codec_test.go` (table-driven subtests
with `t.Parallel()`)
- The `hessian2Codec` uses `map[any]any` instead of `map[string]string`
because hessian2 decodes map keys as `interface{}` — this reflects actual codec
behavior
- All tests pass with `go test` and `go vet`
### Checklist
- [x] I confirm the target branch is `develop`
- [x] Code has passed local testing
- [x] I have added tests that prove my fix is effective or that my feature
works
<img width="792" height="521" alt="Screenshot 2026-08-09 at 4 11 13 PM"
src="https://github.com/user-attachments/assets/688c6978-626c-4ea9-8607-637ce378f4c5";
/>
--
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]
