From fd09a6ed5066d05992fc3d03c83766997a69a8c4 Mon Sep 17 00:00:00 2001 From: Mauren Berti Date: Sun, 28 Jan 2024 16:13:12 -0500 Subject: [PATCH] chore: changes from code review Remove error checks from the utility function and bubble them up instead. --- .../internal/configmapsecret/configmapsecret_testutils.go | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/kustomize/commands/internal/configmapsecret/configmapsecret_testutils.go b/kustomize/commands/internal/configmapsecret/configmapsecret_testutils.go index b4f763e0a..376b79c1f 100644 --- a/kustomize/commands/internal/configmapsecret/configmapsecret_testutils.go +++ b/kustomize/commands/internal/configmapsecret/configmapsecret_testutils.go @@ -81,16 +81,12 @@ func SetupEditSetConfigMapSecretTest( return nil, err } - require.NoError(t, err) - _, err = testutils_test.ReadTestKustomization(fSys) require.NoError(t, err) mf, err := kustfile.NewKustomizationFile(fSys) require.NoError(t, err) - kustomization, err := mf.Read() - require.NoError(t, err) - - return kustomization, nil + //nolint: wrapcheck + return mf.Read() }