From 5381f37e2a1d4661b5f31fbcc5f8a7b855ee36a0 Mon Sep 17 00:00:00 2001
From: Mauren Berti <stormqueen1990@gmail.com>
Date: Wed, 25 Oct 2023 20:43:10 -0400
Subject: [PATCH] fix: lint issues

---
 kustomize/commands/edit/set/all.go            |  2 +-
 kustomize/commands/edit/set/configmap.go      | 10 +++++++---
 kustomize/commands/edit/set/configmap_test.go |  7 ++++---
 3 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/kustomize/commands/edit/set/all.go b/kustomize/commands/edit/set/all.go
index 0c716c91b2..b621482f28 100644
--- a/kustomize/commands/edit/set/all.go
+++ b/kustomize/commands/edit/set/all.go
@@ -32,7 +32,7 @@ func NewCmdSet(
 	}
 
 	c.AddCommand(
-		newCmdSetConfigMap(fSys, ldr, rf),
+		NewCmdSetConfigMap(fSys, ldr, rf),
 		newCmdSetNamePrefix(fSys),
 		newCmdSetNameSuffix(fSys),
 		newCmdSetNamespace(fSys, v),
diff --git a/kustomize/commands/edit/set/configmap.go b/kustomize/commands/edit/set/configmap.go
index f91047d9f4..70c9f111fe 100644
--- a/kustomize/commands/edit/set/configmap.go
+++ b/kustomize/commands/edit/set/configmap.go
@@ -16,7 +16,7 @@ import (
 	"sigs.k8s.io/kustomize/kyaml/filesys"
 )
 
-func newCmdSetConfigMap(
+func NewCmdSetConfigMap(
 	fSys filesys.FileSystem,
 	ldr ifc.KvLoader,
 	rf *resource.Factory,
@@ -115,7 +115,7 @@ func setConfigMap(
 	if len(flags.LiteralSources) > 0 {
 		err := util.UpdateLiteralSources(&args.GeneratorArgs, flags)
 		if err != nil {
-			return err
+			return fmt.Errorf("failed to update literal sources: %w", err)
 		}
 	}
 
@@ -129,7 +129,11 @@ func setConfigMap(
 		args.Options, k.GeneratorOptions)
 
 	_, err = rf.MakeConfigMap(ldr, args)
-	return err
+	if err != nil {
+		return fmt.Errorf("failed to validate ConfigMap structure: %w", err)
+	}
+
+	return nil
 }
 
 // findConfigMapArgs finds the generator arguments corresponding to the specified
diff --git a/kustomize/commands/edit/set/configmap_test.go b/kustomize/commands/edit/set/configmap_test.go
index 0359ac2b9d..7a4179c983 100644
--- a/kustomize/commands/edit/set/configmap_test.go
+++ b/kustomize/commands/edit/set/configmap_test.go
@@ -1,4 +1,4 @@
-package set
+package set_test
 
 import (
 	"testing"
@@ -7,6 +7,7 @@ import (
 	"sigs.k8s.io/kustomize/api/kv"
 	"sigs.k8s.io/kustomize/api/pkg/loader"
 	"sigs.k8s.io/kustomize/api/provider"
+	. "sigs.k8s.io/kustomize/kustomize/v5/commands/edit/set"
 	"sigs.k8s.io/kustomize/kustomize/v5/commands/internal/kustfile"
 	testutils_test "sigs.k8s.io/kustomize/kustomize/v5/commands/internal/testutils"
 	"sigs.k8s.io/kustomize/kyaml/filesys"
@@ -69,7 +70,7 @@ configMapGenerator:
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			cmd := newCmdSetConfigMap(
+			cmd := NewCmdSetConfigMap(
 				fSys,
 				kv.NewLoader(
 					loader.NewFileLoaderAtCwd(fSys),
@@ -131,7 +132,7 @@ configMapGenerator:
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			cmd := newCmdSetConfigMap(
+			cmd := NewCmdSetConfigMap(
 				fSys,
 				kv.NewLoader(
 					loader.NewFileLoaderAtCwd(fSys),