review: fix broken config test

Signed-off-by: Andreas Auernhammer <[email protected]>
minio · Oct 23, 2023 · 9123d01 · 9123d01
1 parent 785b657
commit 9123d01
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/internal/keystore/vault/config_test.go b/internal/keystore/vault/config_test.go
@@ -5,13 +5,14 @@
 package vault
 
 import (
+	"reflect"
 	"testing"
 	"time"
 )
 
 func TestCloneConfig(t *testing.T) {
 	for i, a := range cloneConfigTests {
-		if b := a.Clone(); *a != *b {
+		if b := a.Clone(); !reflect.DeepEqual(a, b) {
 			t.Fatalf("Test %d: cloned config does not match original", i)
 		}
 	}
@@ -35,6 +36,10 @@ var cloneConfigTests = []*Config{
 			Role:   "kes",
 			JWT:    "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoxNTE2MjM5MDIyfQ.SflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c",
 		},
+		Transit: &Transit{
+			Engine:  "transit",
+			KeyName: "my-key",
+		},
 		StatusPingAfter: 15 * time.Second,
 		PrivateKey:      "/tmp/kes/vault.key",
 		Certificate:     "/tmp/kes/vault.crt",