chore: fmt

Author: spydaNVIDIA

common/pkg/credential/credential_test.go

@@ -36,14 +36,14 @@ func TestCredentialEqual(t *testing.T) {
 		b      *Credential
 		expect bool
 	}{
-		"both nil":            {a: nil, b: nil, expect: true},
-		"first nil":           {a: nil, b: &c1, expect: false},
-		"second nil":          {a: &c1, b: nil, expect: false},
-		"identical":           {a: &c2, b: &c2, expect: true},
-		"equal values":        {a: &c5, b: &c6, expect: true},
-		"different user":      {a: &c2, b: &c3, expect: false},
-		"different password":  {a: &c2, b: &c4, expect: false},
-		"both differ":         {a: &c2, b: &c5, expect: false},
+		"both nil":           {a: nil, b: nil, expect: true},
+		"first nil":          {a: nil, b: &c1, expect: false},
+		"second nil":         {a: &c1, b: nil, expect: false},
+		"identical":          {a: &c2, b: &c2, expect: true},
+		"equal values":       {a: &c5, b: &c6, expect: true},
+		"different user":     {a: &c2, b: &c3, expect: false},
+		"different password": {a: &c2, b: &c4, expect: false},
+		"both differ":        {a: &c2, b: &c5, expect: false},
 	}
 
 	for name, tc := range tests {

nvswitch-manager/cmd/serve.go

@@ -30,8 +30,8 @@ import (
 
 	"github.com/spf13/cobra"
 
-	svc "github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/internal/service"
 	"github.com/NVIDIA/ncx-infra-controller-rest/common/pkg/credential"
+	svc "github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/internal/service"
 	"github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/pkg/credentials"
 	"github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/pkg/db"
 	"github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/pkg/nvswitchmanager"

nvswitch-manager/internal/service/server_impl.go

@@ -26,8 +26,8 @@ import (
 	"github.com/google/uuid"
 	log "github.com/sirupsen/logrus"
 
-	pb "github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/internal/proto/v1"
 	"github.com/NVIDIA/ncx-infra-controller-rest/common/pkg/credential"
+	pb "github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/internal/proto/v1"
 	"github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/pkg/common/vendor"
 	"github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/pkg/converter/protobuf"
 	"github.com/NVIDIA/ncx-infra-controller-rest/nvswitch-manager/pkg/firmwaremanager"

nvswitch-manager/internal/service/server_impl_test.go

@@ -127,9 +127,9 @@ func TestValidateFirmwareTarget_MissingBmcCredentials(t *testing.T) {
 	tests := map[string]struct {
 		cred *pb.Credentials
 	}{
-		"nil credentials":   {cred: nil},
-		"empty username":    {cred: &pb.Credentials{Username: "", Password: "pass"}},
-		"empty password":    {cred: &pb.Credentials{Username: "admin", Password: ""}},
+		"nil credentials": {cred: nil},
+		"empty username":  {cred: &pb.Credentials{Username: "", Password: "pass"}},
+		"empty password":  {cred: &pb.Credentials{Username: "admin", Password: ""}},
 	}
 
 	for name, tc := range tests {
@@ -155,9 +155,9 @@ func TestValidateFirmwareTarget_MissingNvosCredentials(t *testing.T) {
 	tests := map[string]struct {
 		cred *pb.Credentials
 	}{
-		"nil credentials":   {cred: nil},
-		"empty username":    {cred: &pb.Credentials{Username: "", Password: "pass"}},
-		"empty password":    {cred: &pb.Credentials{Username: "nvos", Password: ""}},
+		"nil credentials": {cred: nil},
+		"empty username":  {cred: &pb.Credentials{Username: "", Password: "pass"}},
+		"empty password":  {cred: &pb.Credentials{Username: "nvos", Password: ""}},
 	}
 
 	for name, tc := range tests {

nvswitch-manager/internal/service/service.go

@@ -61,12 +61,12 @@ func New(ctx context.Context, c Config) (*Service, error) {
 		if err != nil {
 			return nil, fmt.Errorf("database connection required for persistent mode: %w", err)
 		}
-		
+
 		if err := migrations.Migrate(ctx, pg); err != nil {
 			return nil, fmt.Errorf("failed to run database migrations: %w", err)
 		}
 		db = pg.DB()
-		log.Info("Connected to database")		
+		log.Info("Connected to database")
 	}
 
 	nsmConfig, err := c.ToNsmConf()

nvswitch-manager/pkg/credentials/inmemory_test.go

@@ -109,15 +109,15 @@ func TestInMemoryBMCPutGet(t *testing.T) {
 			ctx := context.Background()
 			mgr := NewInMemoryCredentialManager()
 
-		// Optional initial put
-		if tc.initialPut {
-			mac := parseMAC(t, tc.putMAC)
-			assert.NoError(t, mgr.PutBMC(ctx, mac, tc.putCred))
-			// For the idempotent scenario, put the same credential again
-			if name == "put same credential is no-op" {
-				assert.NoError(t, mgr.PutBMC(ctx, mac, newCredential("user1", "p1")))
+			// Optional initial put
+			if tc.initialPut {
+				mac := parseMAC(t, tc.putMAC)
+				assert.NoError(t, mgr.PutBMC(ctx, mac, tc.putCred))
+				// For the idempotent scenario, put the same credential again
+				if name == "put same credential is no-op" {
+					assert.NoError(t, mgr.PutBMC(ctx, mac, newCredential("user1", "p1")))
+				}
 			}
-		}
 
 			// Get flow
 			got, err := mgr.GetBMC(ctx, parseMAC(t, tc.getMAC))
@@ -132,9 +132,9 @@ func TestInMemoryBMCPutGet(t *testing.T) {
 			assert.Equal(t, tc.wantUser, got.User)
 			assert.Equal(t, tc.wantPass, got.Password.Value)
 
-		if tc.samePtr && tc.initialPut {
-			assert.Same(t, tc.putCred, got)
-		}
+			if tc.samePtr && tc.initialPut {
+				assert.Same(t, tc.putCred, got)
+			}
 		})
 	}
 }