MaoMengww opened a new pull request, #3666:
URL: https://github.com/apache/dubbo-go/pull/3666

   ### Description
   Fixes # (issue)
   This PR improves the comments and test coverage of 
metadata/info/metadata_info.go.
   
   Comments the `MetadataInfo` and `ServiceInfo` types, including their fields 
- app, revision, tag, services, name, group, version, protocol, port, path, 
params, serviceKey,matchKey and so on. Also completes the doc comments of their 
methods (`AddService`, `RemoveService`, `GetServices`, `GetMethods`, 
`GetParams`, `DeepCopy`, `CalRevision`, etc.)
   
   Adds unit tests covering ServiceInfo construction (`NewServiceInfo`, 
`NewServiceInfoWithURL`), method and param retrieval, and the empty map / 
nil-value edge cases ,
   
   本地测试
   ```
   ~/project/dubbo-go$ go test ./metadata/info/...
   ok      dubbo.apache.org/dubbo-go/v3/metadata/info      0.006s
   ```
   
   Refs [OSPP] 补齐 Dubbo-Go 注释与单测覆盖 #3644 (task 7)
   
   ### Checklist
   - [x] I confirm the target branch is `develop`
   - [x] I have run `make fmt` to format my code
   - [x] I have run `make test` to run local tests
   - [ ] I have added tests that prove my fix is effective or that my feature 
works
   


-- 
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]

Reply via email to