yashisrani opened a new issue, #3626:
URL: https://github.com/apache/dubbo-go/issues/3626
### Description
`hessian2Codec` currently has no test coverage and contains a TODO in
`protocol/triple/triple_protocol/codec.go:377`:
```go
// todo(DMwangnima): add unit tests
type hessian2Codec struct{}
```
msgpackCodec also has only a basic round-trip test. We should add
comprehensive unit tests similar to the existing protobuf codec tests.
Tasks
Hessian2
* Test Name() returns "hessian2"
* Test Marshal/Unmarshal round-trips for primitives and collections
* Test nil values
* Test invalid inputs and error cases:
* Non-pointer target
* Nil target
* Invalid/corrupted data
* Nil pointer target
MsgPack
* Test Name() returns "msgpack"
* Add protobuf message round-trip tests
* Test nil and empty payload behavior
* Test invalid data and nil targets
Why?
Better codec coverage will help detect serialization bugs, prevent
regressions during dependency upgrades, and ensure Hessian2 Java
interoperability remains reliable.
--
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]
