test: fix e2e test 6

ratify-project · Aug 25, 2024 · fee6b7d · fee6b7d
1 parent 11de2c0
commit fee6b7d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/pkg/verifier/notation/notation.go b/pkg/verifier/notation/notation.go
@@ -243,7 +243,9 @@ func normalizeVerificationCertsStores(conf *NotationPluginVerifierConfig) error
 			return re.ErrorCodeConfigInvalid.NewError(re.Verifier, conf.Name, re.EmptyLink, err, nil, re.HideStackTrace)
 		}
 		var legacyCertStore map[string]interface{}
-		json.Unmarshal(legacyCertStoreBytes, &legacyCertStore)
+		if err := json.Unmarshal(legacyCertStoreBytes, &legacyCertStore); err != nil {
+			return re.ErrorCodeConfigInvalid.NewError(re.Verifier, conf.Name, re.EmptyLink, err, fmt.Sprintf("failed to unmarshal to legacyCertStore from: %+v.", legacyCertStoreBytes), re.HideStackTrace)
+		}
 		// support legacy verfier config format for backward compatibility
 		conf.VerificationCertStores = verificationCertStores{
 			trustStoreTypeCA: legacyCertStore,

diff --git a/pkg/verifier/notation/notation_test.go b/pkg/verifier/notation/notation_test.go
@@ -257,7 +257,7 @@ func TestParseVerifierConfig(t *testing.T) {
 				Name:              test,
 				VerificationCerts: []string{testPath, defaultCertDir},
 				VerificationCertStores: verificationCertStores{
-					trustStoreTypeCA: verificationCertStores{
+					trustStoreTypeCA: map[string]interface{}{
 						"certstore1": []interface{}{"akv1", "akv2"},
 						"certstore2": []interface{}{"akv3", "akv4"},
 					},