fix(spanner): address review comments and format imports cleanly

Author: sakthivelmanii

spanner/grpc_client.go

@@ -34,8 +34,6 @@ import (
 	"google.golang.org/grpc/status"
 )
 
-
-
 type contextKey string
 
 const metricsTracerKey contextKey = "metricsTracer"

spanner/integration_test.go

@@ -57,8 +57,8 @@ import (
 	"google.golang.org/api/option/internaloption"
 	"google.golang.org/grpc"
 	"google.golang.org/grpc/codes"
-	"google.golang.org/grpc/credentials/insecure"
 	"google.golang.org/grpc/credentials/alts"
+	"google.golang.org/grpc/credentials/insecure"
 	"google.golang.org/grpc/peer"
 	"google.golang.org/grpc/status"
 	"google.golang.org/protobuf/proto"

spanner/metrics.go

@@ -37,8 +37,8 @@ import (
 	"google.golang.org/api/option"
 	"google.golang.org/grpc"
 	"google.golang.org/grpc/codes"
-	"google.golang.org/grpc/experimental/stats"
 	"google.golang.org/grpc/credentials/alts"
+	"google.golang.org/grpc/experimental/stats"
 	"google.golang.org/grpc/peer"
 	"google.golang.org/grpc/stats/opentelemetry"
 	"google.golang.org/grpc/status"
@@ -534,7 +534,7 @@ func (o *opTracer) incrementAttemptCount() {
 // setDirectPathUsed sets whether DirectPath was used for the attempt.
 func (a *attemptTracer) setDirectPathUsed(ctx context.Context) {
 	peerInfo, ok := peer.FromContext(ctx)
-	if ok && peerInfo.AuthInfo != nil {
+	if ok {
 		if _, isALTS := peerInfo.AuthInfo.(alts.AuthInfo); isALTS {
 			a.directPathUsed = true
 		}

spanner/metrics_test.go

@@ -31,8 +31,8 @@ import (
 	"google.golang.org/genproto/googleapis/api/metric"
 	"google.golang.org/grpc"
 	"google.golang.org/grpc/codes"
-	"google.golang.org/grpc/credentials/insecure"
 	"google.golang.org/grpc/credentials/alts"
+	"google.golang.org/grpc/credentials/insecure"
 	"google.golang.org/grpc/peer"
 	"google.golang.org/grpc/status"
 )
@@ -320,9 +320,11 @@ func parseHex(hexStr string) (int64, error) {
 type mockALTSAuthInfo struct {
 	alts.AuthInfo
 }
+
 func (m mockALTSAuthInfo) AuthType() string { return "alts" }
 
 type mockOtherAuthInfo struct{}
+
 func (m mockOtherAuthInfo) AuthType() string { return "other" }
 
 func TestSetDirectPathUsed(t *testing.T) {