fix: range

cluster/router/condition/app_router_test.go

@@ -83,8 +83,7 @@ conditions:
 	appRouteURL := getAppRouteURL(routerKey)
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	appRouter, err := NewAppRouter(appRouteURL, notify)
@@ -140,8 +139,7 @@ conditions:
 	appRouteURL := getAppRouteURL(routerKey)
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	appRouter, err := NewAppRouter(appRouteURL, notify)
@@ -188,8 +186,7 @@ conditions:
 	appRouteURL := getAppRouteURL(routerKey)
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	appRouter, err := NewAppRouter(appRouteURL, notify)

cluster/router/condition/factory_test.go

@@ -134,8 +134,7 @@ func TestRoute_matchWhen(t *testing.T) {
 	rule := base64.URLEncoding.EncodeToString([]byte("=> host = 1.2.3.4"))
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	router, _ := newConditionRouterFactory().NewPriorityRouter(getRouteUrl(rule), notify)
@@ -171,8 +170,7 @@ func TestRoute_matchWhen(t *testing.T) {
 func TestRoute_matchFilter(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()
@@ -212,8 +210,7 @@ func TestRoute_matchFilter(t *testing.T) {
 func TestRoute_methodRoute(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	inv := invocation.NewRPCInvocationWithOptions(invocation.WithMethodName("getFoo"), invocation.WithParameterTypes([]reflect.Type{}), invocation.WithArguments([]interface{}{}))
@@ -241,8 +238,7 @@ func TestRoute_methodRoute(t *testing.T) {
 func TestRoute_ReturnFalse(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	url, _ := common.NewURL("")
@@ -259,8 +255,7 @@ func TestRoute_ReturnFalse(t *testing.T) {
 func TestRoute_ReturnEmpty(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()
@@ -277,8 +272,7 @@ func TestRoute_ReturnEmpty(t *testing.T) {
 func TestRoute_ReturnAll(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()
@@ -303,8 +297,7 @@ func TestRoute_HostFilter(t *testing.T) {
 	url3, _ := common.NewURL(fmt.Sprintf(factoryDubboFormat, localIP))
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	invoker1 := NewMockInvoker(url1, 1)
@@ -324,8 +317,7 @@ func TestRoute_HostFilter(t *testing.T) {
 func TestRoute_Empty_HostFilter(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()
@@ -349,8 +341,7 @@ func TestRoute_Empty_HostFilter(t *testing.T) {
 func TestRoute_False_HostFilter(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()
@@ -374,8 +365,7 @@ func TestRoute_False_HostFilter(t *testing.T) {
 func TestRoute_Placeholder(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()
@@ -399,8 +389,7 @@ func TestRoute_Placeholder(t *testing.T) {
 func TestRoute_NoForce(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()
@@ -422,8 +411,7 @@ func TestRoute_NoForce(t *testing.T) {
 func TestRoute_Force(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	localIP := common.GetLocalIp()

cluster/router/condition/router_test.go

@@ -60,8 +60,7 @@ func TestParseRule(t *testing.T) {
 func TestNewConditionRouter(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	url, _ := common.NewURL(`condition://0.0.0.0:?application=mock-app&category=routers&force=true&priority=1&router=condition&rule=YSAmIGMgPT4gYiAmIGQ%3D`)

cluster/router/conncheck/conn_check_route_test.go

@@ -50,8 +50,7 @@ func TestConnCheckRouterRoute(t *testing.T) {
 	defer protocol.CleanAllStatus()
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	consumerURL, _ := common.NewURL(connCheck1001URL)

cluster/router/healthcheck/health_check_route_test.go

@@ -51,8 +51,7 @@ func TestHealthCheckRouterRoute(t *testing.T) {
 	defer protocol.CleanAllStatus()
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	consumerURL, _ := common.NewURL(healthCheck1001URL)
@@ -120,8 +119,7 @@ func TestNewHealthCheckRouter(t *testing.T) {
 	defer protocol.CleanAllStatus()
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	url, _ := common.NewURL(fmt.Sprintf(healthCheckDubboUrlFormat, healthCheckDubbo1010IP))

cluster/router/tag/factory_test.go

@@ -41,8 +41,7 @@ func TestTagRouterFactoryNewRouter(t *testing.T) {
 	factory := NewTagRouterFactory()
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	tagRouter, e := factory.NewPriorityRouter(u1, notify)

cluster/router/tag/tag_router_test.go

@@ -121,8 +121,7 @@ func TestTagRouterPriority(t *testing.T) {
 	u1, err := common.NewURL(tagRouterTestUserConsumerTag)
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	assert.Nil(t, err)
@@ -135,8 +134,7 @@ func TestTagRouterPriority(t *testing.T) {
 func TestTagRouterRouteForce(t *testing.T) {
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	u1, e1 := common.NewURL(tagRouterTestUserConsumerTag)
@@ -175,8 +173,7 @@ func TestTagRouterRouteNoForce(t *testing.T) {
 	assert.Nil(t, e1)
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	tagRouter, e := NewTagRouter(u1, notify)
@@ -244,8 +241,7 @@ func TestRouteBeijingInvoker(t *testing.T) {
 	url, _ := common.NewURL(tagRouterTestBeijingUrl)
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	tagRouter, _ := NewTagRouter(url, notify)
@@ -326,8 +322,7 @@ tags:
 
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	tagRouter, err := NewTagRouter(url, notify)
@@ -397,8 +392,7 @@ func TestProcess(t *testing.T) {
 	assert.Nil(t, err)
 	notify := make(chan struct{})
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	tagRouter, e := NewTagRouter(u1, notify)
@@ -420,8 +414,7 @@ tags:
     addresses: [192.168.1.3, 192.168.1.4]
 `
 	go func() {
-		for {
-			<-notify
+		for range notify {
 		}
 	}()
 	tagRouter.Process(&config_center.ConfigChangeEvent{Value: testYML, ConfigType: remoting.EventTypeAdd})