Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix:nacos client reuse #1633

Merged
merged 16 commits into from
Feb 1, 2022
Merged

Fix:nacos client reuse #1633

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
f19a829
up:处理nacos客户端共用问题
zhaoyunxing92 Dec 2, 2021
dbc19ce
fmt:import
zhaoyunxing92 Dec 2, 2021
d816d3c
Merge remote-tracking branch 'origin/3.0' into fix-nacos
zhaoyunxing92 Dec 12, 2021
39409b2
up:更新客户端获取
zhaoyunxing92 Dec 30, 2021
7021da8
修复测试用例问题
zhaoyunxing92 Dec 30, 2021
26fab4a
up:测试问题
zhaoyunxing92 Dec 30, 2021
dc6154a
fmt
zhaoyunxing92 Dec 30, 2021
8c2a9e3
修复测试用例
zhaoyunxing92 Dec 30, 2021
1eaa349
fmt
zhaoyunxing92 Dec 30, 2021
816ae72
修改impl测试问题
zhaoyunxing92 Jan 3, 2022
481de45
Merge remote-tracking branch 'upstream/3.0' into fix-nacos
zhaoyunxing92 Jan 4, 2022
371012a
Merge branch '3.0' into fix-nacos
zhaoyunxing92 Jan 14, 2022
81757f2
Merge remote-tracking branch 'origin/3.0' into fix-nacos
zhaoyunxing92 Jan 18, 2022
0951444
fix:测试用例
zhaoyunxing92 Jan 18, 2022
afe5605
Merge remote-tracking branch 'upstream/3.0' into fix-nacos
zhaoyunxing92 Jan 27, 2022
e7ddc1e
fix:修改应用级服务发现
zhaoyunxing92 Jan 27, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix:测试用例
  • Loading branch information
@zhaoyunxing92
zhaoyunxing92 committed Jan 18, 2022
commit 0951444ba4b596d2ba7a237996ee1601d417de5e
4 changes: 3 additions & 1 deletion registry/nacos/service_discovery_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@ import (
const testName = "test"

func TestNewNacosServiceDiscovery(t *testing.T) {
url, _ := common.NewURL("dubbo://127.0.0.1:8848")
url, _ := common.NewURL("dubbo://127.0.0.1:8848",
common.WithParamsValue(constant.ClientNameKey, "nacos-client"))
sd, err := newNacosServiceDiscovery(url)
assert.Nil(t, err)
err = sd.Destroy()
Expand Down Expand Up @@ -122,6 +123,7 @@ func newMockNacosServiceDiscovery(url *common.URL) (registry.ServiceDiscovery, e
common.WithParamsValue(constant.NacosGroupKey, url.GetParam(constant.RegistryGroupKey, defaultGroup)),
common.WithParamsValue(constant.NacosUsername, url.Username),
common.WithParamsValue(constant.NacosPassword, url.Password),
common.WithParamsValue(constant.ClientNameKey, "nacos-client"),
common.WithParamsValue(constant.NacosNamespaceID, url.GetParam(constant.RegistryNamespaceKey, "")))
discoveryURL.Location = url.Location
discoveryURL.Username = url.Username
Expand Down