style: fix code lint

pkg/common/file_test.go

@@ -17,8 +17,9 @@
 package common
 
 import (
-	"github.com/stretchr/testify/assert"
 	"testing"
+
+	"github.com/stretchr/testify/assert"
 )
 
 func TestGetFileSize(t *testing.T) {

pkg/runtime/pluggable.go

@@ -90,18 +90,18 @@ func (m *MosnRuntime) RegisterPluggableComponent() error {
 	}
 
 	// 1. discover pluggable component
-	services, err := m.Discover()
+	services, err := m.discover()
 	if err != nil {
 		return err
 	}
 
 	// 2. callback to register factory into MosnRuntime
-	m.Callback(services)
+	m.callback(services)
 	return nil
 }
 
-// Discover use grpc reflect to get services information.
-func (m *MosnRuntime) Discover() ([]pluggableComponentService, error) {
+// discover use grpc reflect to get services information.
+func (m *MosnRuntime) discover() ([]pluggableComponentService, error) {
 	ctx := context.TODO()
 	services, err := serviceDiscovery(func(socket string) (client reflectServiceClient, closer func(), err error) {
 		conn, err := grpcdial.SocketDial(
@@ -189,8 +189,8 @@ func serviceDiscovery(reflectClientFactory func(socket string) (client reflectSe
 	return services, nil
 }
 
-// Callback use callback function to register pluggable component factories into MosnRuntime
-func (m *MosnRuntime) Callback(services []pluggableComponentService) {
+// callback use callback function to register pluggable component factories into MosnRuntime
+func (m *MosnRuntime) callback(services []pluggableComponentService) {
 	for _, service := range services {
 		callback, ok := onServiceDiscovered[service.protoRef]
 		if !ok {

pkg/runtime/pluggable_test.go

@@ -15,14 +15,16 @@ package runtime
 
 import (
 	"errors"
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
-	grpcdial "mosn.io/layotto/pkg/grpc"
 	"net"
 	"os"
 	"runtime"
 	"sync"
 	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+
+	grpcdial "mosn.io/layotto/pkg/grpc"
 )
 
 type fakeReflectService struct {
@@ -83,7 +85,7 @@ func TestMosnRuntime_Callback(t *testing.T) {
 			called++
 			assert.Equal(t, name, fakeComponentName)
 		})
-		m.Callback([]pluggableComponentService{{protoRef: fakeServiceName, componentName: fakeComponentName}})
+		m.callback([]pluggableComponentService{{protoRef: fakeServiceName, componentName: fakeComponentName}})
 		assert.Equal(t, 1, called)
 	})
 }
@@ -193,11 +195,11 @@ func Test_serviceDiscovery(t *testing.T) {
 
 func TestConnectionCloser(t *testing.T) {
 	t.Run("connection closer should call grpc close and client reset", func(t *testing.T) {
-		const close, reset = "close", "reset"
+		const closed, reset = "closed", "reset"
 		callOrder := []string{}
 		fakeCloser := &fakeGrpcCloser{
 			onCloseCalled: func() {
-				callOrder = append(callOrder, close)
+				callOrder = append(callOrder, closed)
 			},
 		}
 		fakeService := &fakeReflectService{
@@ -208,6 +210,6 @@ func TestConnectionCloser(t *testing.T) {
 		closer := reflectServiceConnectionCloser(fakeCloser, fakeService)
 		closer()
 		assert.Len(t, callOrder, 2)
-		assert.Equal(t, callOrder, []string{reset, close})
+		assert.Equal(t, callOrder, []string{reset, closed})
 	})
 }