fix lint errors

cluster/cluster_impl/failback_cluster_invoker.go

@@ -121,8 +121,11 @@ func (invoker *failbackClusterInvoker) checkRetry(retryTask *retryTimerTask, err
 	if retryTask.retries > invoker.maxRetries {
 		logger.Errorf("Failed retry times exceed threshold (%v), We have to abandon, invocation-> %v.\n",
 			retryTask.retries, retryTask.invocation)
-	} else {
-		invoker.taskList.Put(retryTask)
+		return
+	}
+
+	if err := invoker.taskList.Put(retryTask); err != nil {
+		logger.Errorf("invoker.taskList.Put(retryTask:%#v) = error:%v", retryTask, err)
 	}
 }
 

cluster/router/condition/file_test.go

@@ -89,18 +89,21 @@ func TestParseServiceRouterKey(t *testing.T) {
 	assert.Equal(t, "mock-group", grp)
 	assert.Equal(t, "mock-service", srv)
 	assert.Equal(t, "1.0.0", ver)
+	assert.Nil(t, err)
 
 	testString = "mock-group/mock-service"
 	grp, srv, ver, err = parseServiceRouterKey(testString)
 	assert.Equal(t, "mock-group", grp)
 	assert.Equal(t, "mock-service", srv)
 	assert.Equal(t, "", ver)
+	assert.Nil(t, err)
 
 	testString = "mock-service:1.0.0"
 	grp, srv, ver, err = parseServiceRouterKey(testString)
 	assert.Equal(t, "", grp)
 	assert.Equal(t, "mock-service", srv)
 	assert.Equal(t, "1.0.0", ver)
+	assert.Nil(t, err)
 
 	testString = "mock-service"
 	grp, srv, ver, err = parseServiceRouterKey(testString)

cluster/router/tag/tag_router_test.go

@@ -73,7 +73,7 @@ const (
 )
 
 var (
-	zkFormat        = "zookeeper://%s:%d"
+	zkFormat = "zookeeper://%s:%d"
 )
 
 // MockInvoker is only mock the Invoker to support test tagRouter
@@ -309,7 +309,8 @@ tags:
 
 func (suite *DynamicTagRouter) TearDownTest() {
 	suite.zkClient.Close()
-	suite.testCluster.Stop()
+	err := suite.testCluster.Stop()
+	suite.Nil(err)
 }
 
 func (suite *DynamicTagRouter) TestDynamicTagRouterSetByIPv4() {