This is an automated email from the ASF dual-hosted git repository.

alexstocks pushed a commit to branch 1.5
in repository https://gitbox.apache.org/repos/asf/dubbo-go.git


The following commit(s) were added to refs/heads/1.5 by this push:
     new 7984ce3  fix:map iteration order
     new ad3a6c4  Merge pull request #1109 from zhaoyunxing92/fix-map
7984ce3 is described below

commit 7984ce383ed93ffe2f30261b603327c71321bdfd
Author: zhaoyunxing <[email protected]>
AuthorDate: Sun Mar 28 10:24:23 2021 +0800

    fix:map iteration order
---
 config/config_api_test.go | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/config/config_api_test.go b/config/config_api_test.go
index fd8d561..c0bba78 100644
--- a/config/config_api_test.go
+++ b/config/config_api_test.go
@@ -160,18 +160,14 @@ func TestNewProviderConfig(t *testing.T) {
                assert.Equal(t, v, serviceConfig)
        }
        assert.NotNil(t, testProviderConfig.Registries)
-       i := 0
-       for k, v := range testProviderConfig.Registries {
-               if i == 0 {
-                       assert.Equal(t, k, "demoConsul")
-                       assert.Equal(t, v, defaultConsulRegistry)
-                       i++
-               } else {
-                       assert.Equal(t, k, "demoNacos")
-                       assert.Equal(t, v, defaultNacosRegistry)
-               }
+
+       if registry, ok := testProviderConfig.Registries["demoConsul"]; ok {
+               assert.Equal(t, registry, defaultConsulRegistry)
        }
 
+       if registry, ok := testProviderConfig.Registries["demoNacos"]; ok {
+               assert.Equal(t, registry, defaultNacosRegistry)
+       }
        assert.NotNil(t, testProviderConfig.Protocols)
        assert.Equal(t, testProviderConfig.Protocols["dubbo"].Name, "dubbo")
        assert.Equal(t, testProviderConfig.Protocols["dubbo"].Port, "20000")

Reply via email to