Snow-kal commented on code in PR #3198: URL: https://github.com/apache/dubbo-go/pull/3198#discussion_r2779150141
########## remoting/nacos/builder_test.go: ########## @@ -16,195 +16,3 @@ */ package nacos Review Comment: 好的，已更改，我主要是看它都被注释了还以为都不用了 简单概括一下 builder_test.go 的修改内容 已移除测试中对 config 包的依赖，并补充了基于 URL 解析的覆盖 删除了 config.RemoteConfig 相关结构与 TestNewNacosClient，避免引用不存在的构造路径。 新增 NewNacosClientByURL/NewNacosConfigClientByUrl 的缺少 client name 的错误用例 扩展 GetNacosConfig 用例：包含 context path、endpoint-only、nil url、空 location、非法 address 等，修正 endpoint-only 返回 nil serverConfigs 的预期。 修正测试中 Nacos constant 包路径与 Config client 的 mock 函数。 -- 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]
